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
[release-4.5] Bug 1878624: Invalid egressCIDR value causes sdn pods to fail on startup #187
[release-4.5] Bug 1878624: Invalid egressCIDR value causes sdn pods to fail on startup #187
Conversation
HostSubnet is half system-maintained (subnet allocations) and half user-maintained (EgressIPs). If there is any invalid value that is set on the user-maintained fields, we should just ignore them and continue with the SDN startup procedure instead of failing. In this patch we remove the validation of the user-maintained fields from ValidateHostSubnet function. We can do a separate validation for the user-maintained fields. Signed-off-by: Surya Seetharaman suryaseetharaman.9@gmail.com
We don't have a proper validation procedure for egress ips/cidrs. This patch adds a ValidateHostSubnetEgress function which will check for the validity of the user-maintained values in the hostsubet object. Signed-off-by: Surya Seetharaman suryaseetharaman.9@gmail.com
@openshift-cherrypick-robot: Bugzilla bug 1848478 has been cloned as Bugzilla bug 1878624. Retitling PR to link against new bug. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@openshift-cherrypick-robot: This pull request references Bugzilla bug 1878624, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/approve |
/retest |
I also need the lgtm label to get picked by patch-manager, @rcarrillocruz could you also add that please ? |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: juanluisvaladas, openshift-cherrypick-robot, rcarrillocruz The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
(patch manager) affecting real world users |
/retest Please review the full test history for this PR and help us cut down flakes. |
8 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
22 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@markmc can we put the cherry-pick label back ? The e2egcp job was facing some CI cluster issues (openshift/release#12386), that's why so many retests. |
@tssurya you can ask Ben, hje can also add the cherry-pick-approved label |
ah really ? I always get confused with this part. I thought only patch-managers for that week could do this and pick up backports. |
Not sure how that works but he has a lot of privileges |
@openshift-cherrypick-robot: All pull requests linked via external trackers have merged: Bugzilla bug 1878624 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This is an automated cherry-pick of #169
/assign tssurya