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

Update helm charts to support zap logging in metrics server #741

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aliaqel-stripe
Copy link

@aliaqel-stripe aliaqel-stripe commented Feb 28, 2025

Update metrics-server helm charts to support kedacore/keda#6578

Checklist

  • I have verified that my change is according to the deprecations & breaking changes policy
  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • README is updated with new configuration values (if applicable) learn more
  • A PR is opened to update KEDA core (repo) (if applicable, ie. when deployment manifests are modified)

Fixes #5732

Signed-off-by: Ali Aqel <[email protected]>
@aliaqel-stripe
Copy link
Author

I suspect the chart failures are due to the flags not existing on the api server yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Metrics Server: klogr.New is deprecated use textlogger.NewLogger instead
1 participant