-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Always define an explicit workspace volume, it defaults to an empty dir #622
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not clear if you reproduced the bug in a test here—there are a couple of test patches but they just look like accommodating the change.
src/main/java/org/csanchez/jenkins/plugins/kubernetes/PodTemplate.java
Outdated
Show resolved
Hide resolved
src/main/java/org/csanchez/jenkins/plugins/kubernetes/PodTemplate.java
Outdated
Show resolved
Hide resolved
src/main/resources/org/csanchez/jenkins/plugins/kubernetes/PodTemplate/config.jelly
Outdated
Show resolved
Hide resolved
Co-Authored-By: Jesse Glick <jglick@cloudbees.com>
@jglick Ideally it would need a html unit (since this is driven by the form UI) adding a pod template, saving then check that the resulting pod template has an empty dir volume. But I have no clue how to write it |
So the opening of the relevant form and saving it are both routine things to do with a |
Yes I agree about form handling, difficulty mostly lies in dealing with the weird add button |
src/main/resources/org/csanchez/jenkins/plugins/kubernetes/PodTemplate/config.jelly
Outdated
Show resolved
Hide resolved
src/main/resources/org/csanchez/jenkins/plugins/kubernetes/PodTemplate/config.jelly
Outdated
Show resolved
Hide resolved
|
Add a HtmlUnit test for default workspace volume
@jglick I think I have a working test, and I converted the |
Should fix #600 (comment)