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
Bug 1948585: Add periodic for OVN using local gw mode #18322
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jluhrsen The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retitle Bug 1948585: Add periodic for OVN using local gw mode |
@jluhrsen: This pull request references Bugzilla bug 1948585, 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. 3 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (anusaxen@redhat.com), skipping review request. 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. |
8511862
to
99775ea
Compare
8229154
to
323a75c
Compare
Thanks Jamo. FYI there was a bug in services with local gw mode that @astoycos fixed. I dont think it's downstream yet. |
/test pj-rehearse |
52ec093
to
4b7d57a
Compare
The metal-ipi rehearsals do not work. The config they generate produces something too big for the OS running CI to handle.
|
For metal-ipi, typically we add a flag to our development scripts https://github.com/openshift-metal3/dev-scripts, and then set the flag in CI. This would let a developer easily use local gateway mode as well. If we did it this way, then we'd avoid making our scripts even bigger in CI. |
ok, I can try to do it that way. I remember quickly glancing through the dev scripts which is where I found that ASSESTS_EXTRA_FOLDER variable which led me to doing it this way. @stbenjam, can you help me understand the best way I can test my changes to dev-scripts so I know what I end up with |
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
@trozet , let's just use this PR to get OVN + local gw mode working in AWS. I'm guessing it will take me a bit to figure out the right way to a dev-scripts PR to make this happen on baremetal. |
You can make the changes you need to dev-scripts, have some flag like OVN_LOCAL_GW. To test it, add a temporary second commit that swaps the default value to true and you can see what CI does. |
👍🏻 working on that here |
/retest |
1 similar comment
/retest |
@trozet this is ready. these presubmits are never going to fully pass. the non-local job is using shared |
/retest |
@jluhrsen: The following tests failed, say
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. |
/approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dcbw, jluhrsen, knobunc 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 |
@jluhrsen: All pull requests linked via external trackers have merged: Bugzilla bug 1948585 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. |
@jluhrsen: Updated the following 3 configmaps:
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. |
Signed-off-by: Jamo Luhrsen jluhrsen@gmail.com