-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fail fast on buildScalers
when not able to resolve a secret
#2394
fail fast on buildScalers
when not able to resolve a secret
#2394
Conversation
/run-e2e |
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!
Thanks for the fix :)
Please, when you have time update the changelog
5a8a213
to
1925b06
Compare
Signed-off-by: Xiayang Wu <xwu@rippling.com>
Co-authored-by: Jorge Turrado Ferrero <Jorge_turrado@hotmail.es> Signed-off-by: Xiayang Wu <xwu@rippling.com>
Signed-off-by: Xiayang Wu <xwu@rippling.com>
1925b06
to
05a891a
Compare
@JorTurFer sorry, I totally forgot about this PR because of holiday. changelog is updated alongside the code suggestion |
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!
Thanks for your contribution ❤️
/run-e2e mongo* |
buildScalers
when not able to resolve a secret
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, thanks a lot @fivesheep !
Signed-off-by: Xiayang Wu xwu@rippling.com
Instead of creating an unexpected HPA, fail fast when not able to resolve a secret a deployment is relying on.
the missing secret can be a mistake or just timing issue. more details can be found from #2393
Checklist
Fixes #2393