-
Notifications
You must be signed in to change notification settings - Fork 506
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
Add --auto-reload-all flag #487
Add --auto-reload-all flag #487
Conversation
dce599e
to
ffedbaf
Compare
fd7e66c
to
5db6edc
Compare
@itaispiegel Image is available for testing. |
5db6edc
to
97b0e6a
Compare
@itaispiegel Image is available for testing. |
@itaispiegel Thank you for the contribution, the PR looks good. Can you please also write some test cases here: https://github.com/stakater/Reloader/blob/master/internal/pkg/handler/upgrade_test.go. You can write some scenarios from your last commit where you can test if the reload-all is set to true then you reload more than one deployment upon a config map change. Having at least one positive and one negative test case should be enough :) |
@faizanahmad055 Great! I'll work on it this week and will keep in touch with you :) |
@itaispiegel Yikes! You better fix it before anyone else finds out! Build has Failed! |
5f45814
to
67ac0f6
Compare
@itaispiegel Yikes! You better fix it before anyone else finds out! Build has Failed! |
@itaispiegel Yikes! You better fix it before anyone else finds out! Build has Failed! |
@itaispiegel Yikes! You better fix it before anyone else finds out! Build has Failed! |
67ac0f6
to
f29cc82
Compare
f29cc82
to
b9e24b3
Compare
@itaispiegel Yikes! You better fix it before anyone else finds out! Build has Failed! |
@itaispiegel Yikes! You better fix it before anyone else finds out! Build has Failed! |
@itaispiegel Image is available for testing. |
@karl-johan-grahn Image is available for testing. |
This is my PR for issue #478.
I added the flag
--auto-reload-all
, which commands the controller to reload all resources when their configmaps/secrets change, unless they have theauto
annotation set tofalse
(or an undefined value).I'd be glad to get feedback on this, so please tell me if the documentation isn't clear enough, or if you think this should be implemented differently.