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
MGMT-16330: disable SDN for 4.15 and above #5833
MGMT-16330: disable SDN for 4.15 and above #5833
Conversation
@rccrdpccl: This pull request references MGMT-16330 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the epic to target the "4.16.0" version, but no target version was set. 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. |
Skipping CI for Draft Pull Request. |
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 sure if these changes are enough to forbid the setting on back-end side, is that what we want?
f60ebea
to
9721820
Compare
Yes that's what we want, I misread a bit of code and thought it was automatic, so you're right it is not enough |
9721820
to
1b8fd88
Compare
/refresh |
/hold Holding to control merge once back from PTO |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #5833 +/- ##
==========================================
+ Coverage 67.96% 69.27% +1.31%
==========================================
Files 235 235
Lines 34482 35855 +1373
==========================================
+ Hits 23435 24838 +1403
+ Misses 8981 8926 -55
- Partials 2066 2091 +25
|
1b8fd88
to
b5e55c4
Compare
Signed-off-by: Riccardo Piccoli <rpiccoli@redhat.com>
b5e55c4
to
aa3cb2b
Compare
/retest |
1 similar comment
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adriengentil, gamli75, rccrdpccl 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 |
/retest |
3 similar comments
/retest |
/retest |
/retest |
/override e2e-agent-compact-ipv4 |
@rccrdpccl: /override requires failed status contexts, check run or a prowjob name to operate on.
Only the following failed contexts/checkruns were expected:
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context. 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. |
/override ci/prow/e2e-agent-compact-ipv4 |
@rccrdpccl: Overrode contexts on behalf of rccrdpccl: ci/prow/e2e-agent-compact-ipv4 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. |
@rccrdpccl: all tests passed! Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-agent-installer-api-server-container-v4.16.0-202401082351.p0.g2f45852.assembly.stream for distgit ose-agent-installer-api-server. |
This PR aims at disabling SDN option when installing OCP 4.15 and above
List all the issues related to this PR
What environments does this code impact?
How was this code tested?
Checklist
docs
, README, etc)Reviewers Checklist