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
Changing StatsAddress flag from Int to String to prevent Init container crash loop. #742
Conversation
f1034ed
to
dd12689
Compare
Thanks for this fix. Assigning @alexbrand to review. @josebiro in the future would you please raise a bug so we can track defects across releases. Thanks |
This may need a possible backport for 0.6.2 if gimbal users need it, /cc @stevesloka @rosskukulinski |
Thanks @josebiro. I think this looks good, but do you mind adding a test case to internal/envoy/config_test.go where we set the stats address and port? |
Test added. Checks should pass, everything is signed off, but for some reason github went from passing to waiting. |
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.
Thanks @josebiro. This LGTM. Do you mind rebasing from master
as well?
3b04268
to
eca1d06
Compare
Rebased; feel free to merge at your will. |
…ults to match reality. Signed-off-by: Joseph Bironas <josebiro@gmail.com> Signed-off-by: Joseph Bironas <josebiro@gmail.com>
Signed-off-by: Joseph Bironas <josebiro@gmail.com>
Signed-off-by: Joseph Bironas <josebiro@gmail.com> Signed-off-by: Joseph Bironas <josebiro@gmail.com>
Signed-off-by: Joseph Bironas <josebiro@gmail.com> Signed-off-by: Joseph Bironas <josebiro@gmail.com>
Thank you very much. |
And fixing deployment defaults for prometheus scraping to match reality.
This is a small bugfix, so didn't file an issue. Let me know if that's not appropriate.
All tests pass. Built and tested via docker and confirmed that: