-
Notifications
You must be signed in to change notification settings - Fork 129
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
Adding logic to backpack to only run on a node once for a given uuid #315
Conversation
6d40b5b
to
e221891
Compare
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.
Single nit...
e221891
to
6d60500
Compare
/rerun all |
Results for crc_jjb
|
/rerun all |
Results for crc_jjb
|
/rerun all |
Results for crc_jjb
|
/rerun all |
Results for crc_jjb
|
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.
LGTM, still fearful, but LGTM!
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.
Nice work @dry923 , I added just a few comments.
… Additionally, converted metadata_* variables to an object metadata.*
6d60500
to
a82ab0f
Compare
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.
lgtm
Results for crc_jjb
|
Note: This needs to be merged after cloud-bulldozer/bohica#10 and cloud-bulldozer/metadata-collector#3
This will add the ability for backpack to only run on a node once per each uuid. So if 3 pods land on the same node for the same workload it will only collect the data once. The init container will still run and then complete with a message saying that the data was already collected.