-
Notifications
You must be signed in to change notification settings - Fork 98
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
fix(k8s-eks): use latest haproxy ingress controller version #6539
Conversation
With the scylla-operator-1.10 we started getting HA problems [1] with the 'haproxy' service. So, fix it by using the latest available ingress controller version for haproxy. [1] scylladb/scylla-operator#1341
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
Question: how we assure we use the same as on production?
We have used the same. But, the latest one is more stable. And I would say that they should switch to the latest too. And, finally, the |
One more run showed that the problem is still in place: https://jenkins.scylladb.com/job/scylla-operator/job/operator-1.10/job/upgrade/job/upgrade-platform-k8s-eks/4/consoleFull |
Third run with this PR passed: https://jenkins.scylladb.com/job/scylla-operator/job/operator-1.10/job/upgrade/job/upgrade-platform-k8s-eks/5/ |
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
With the scylla-operator-1.10 we started getting HA problems [1] with the
haproxy
service.So, fix it by using the latest available ingress controller version for haproxy.
[1] scylladb/scylla-operator#1341
PR pre-checks (self review)
backport
labelsNew configuration option are added and documented (insdcm/sct_config.py
)I have added tests to cover my changes (Infrastructure only - underunit-test/
folder)I have updated the Readme/doc folder accordingly (if needed)