Skip to content
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

[operator] explicitly set metrics bind address to 8080 #304

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmazzitelli
Copy link
Contributor

the default value used to be 8080 but in ansible operator 1.36.0, that default was changed to 8443. So we need to explicitly set it 8080 now.

This goes with operator PR kiali/kiali-operator#857

the default value used to be 8080 but in ansible operator 1.36.0, that default was changed to 8443. So we need to explicitly set it 8080 now.
@jmazzitelli
Copy link
Contributor Author

Confirmed this works by the metrics-test molecule test that now passes when this helm chart PR is tested along with the operator PR kiali/kiali-operator#857

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires operator PR Requires changes to the operator
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

1 participant