Skip to content

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

Merged
zroubalik merged 4 commits into
kedacore:mainfrom
aliaqel-stripe:aliaqel/update-klogr
Oct 8, 2025
Merged

Update helm charts to support zap logging in metrics server#741
zroubalik merged 4 commits into
kedacore:mainfrom
aliaqel-stripe:aliaqel/update-klogr

Conversation

@aliaqel-stripe

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

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor Author

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

@JorTurFer

Copy link
Copy Markdown
Member

Could you run docker run -v "$PWD:/helm-docs" -u $(id -u) jnorwood/helm-docs:latest and commit the changes?

@zroubalik zroubalik merged commit 2662aca into kedacore:main Oct 8, 2025
68 of 69 checks passed
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

3 participants