-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
move ovn config helper functions to util lib #51771
move ovn config helper functions to util lib #51771
Conversation
/test e2e-aws-ovn-clusternetwork-cidr-expansion |
@jluhrsen: The specified target(s) for
Use
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,
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
901ba2a
to
b7c5114
Compare
@jluhrsen,
If the problem persists, please contact Test Platform. |
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
@jluhrsen,
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
@jluhrsen,
If the problem persists, please contact Test Platform. |
b7c5114
to
84ededd
Compare
@jluhrsen,
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
84ededd
to
0b97c2d
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
@jluhrsen,
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
0b97c2d
to
c5dc250
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
c5dc250
to
e23b42e
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
e23b42e
to
a380e75
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
2b55276
to
681420d
Compare
@danwinship , thank you for reviewing this PR :) |
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
@jluhrsen,
If the problem persists, please contact Test Platform. |
681420d
to
b69111e
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
b69111e
to
b27fbee
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
planning to use this for work on SDN-4776 [0] which covers more day-2 configs Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
b27fbee
to
d284350
Compare
/pj-rehearse |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
/pj-rehearse ack |
@jluhrsen: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
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.
As long as the content is small enough to fit into the Secret backing SHARED_DIR
, I guess this works. And reverts or other pivots should be fairly straightforward if we do run into trouble.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jluhrsen, wking 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: 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-sigs/prow repository. I understand the commands that are listed here. |
946a917
into
openshift:master
planning to use this for work on SDN-4776 [0] which covers more day-2 configs Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
planning to use this for work on SDN-4776 [0] which covers more day-2 configs