Skip to content

CMA should support bound service account tokens #96335

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mburke5678
Copy link
Contributor

@mburke5678 mburke5678 commented Jul 18, 2025

DO NOT MERGE UNTIL CUSTOM METRICS AUTOSCALER 2.17.2 RELEASES

https://issues.redhat.com/browse/OSDOCS-12581

Link to docs preview:
Understanding custom metrics autoscaler trigger authentications -- Added three example trigger authentications that use bound service account tokens. Moved example secret to after the associated trigger auth example.
Using trigger authentications -- Updated example trigger auth to use bound service token
Configuring the custom metrics autoscaler to use OpenShift Container Platform monitoring -- Updated example trigger auth to use bound service token; removed prereq to have a secret (not needed for new token)

QE review:

  • QE has approved this change.

@mburke5678 mburke5678 added this to the Continuous Release milestone Jul 18, 2025
@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 18, 2025
@mburke5678 mburke5678 changed the title CMA should support bound service account tokens/default Keda Controller CMA should support bound service account tokens Jul 18, 2025
@mburke5678
Copy link
Contributor Author

@maxcao13 Can you PTAL?

Copy link

openshift-ci bot commented Jul 21, 2025

@mburke5678: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

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

Successfully merging this pull request may close these issues.

2 participants