-
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
Port mapping #165
Port mapping #165
Conversation
public void setPorts(List<PortMapping> ports) { | ||
if (ports != null) { | ||
this.ports.clear(); | ||
this.ports.addAll(ports); |
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.
the way jenkins (de)serializes is a bit confusing. In upgrades this.ports will be null because the field is not in the xml config.
What you want is to guard against that, so just remove final
(I don't think there's any reason to have it) and do this.ports=ports
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.
Sounds good, I'll update it in a few and push it up.
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.
@carlossg fixed.
@@ -169,8 +169,7 @@ public void setLivenessProbe(ContainerLivenessProbe livenessProbe) { | |||
@DataBoundSetter | |||
public void setPorts(List<PortMapping> ports) { | |||
if (ports != null) { | |||
this.ports.clear(); | |||
this.ports.addAll(ports); | |||
this.ports = ports; |
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.
no need to check for null now
public void setPorts(List<PortMapping> ports) {
this.ports = ports;
}
I am merging it because it's been around for long, but it would be nice to have a pipeline test |
@carlossg Do you have a feeling of when the next release will be? No pressure - just wondering whether I should wait or build from master for now. |
no pressure taken ;) |
Hey Guys,
Unfortunately I get this:
|
This is to finish up the port mapping ticket in #135.
I added the help documents, and i can fix the other issue if @carlossg and provide guidance on how it should be fixed.