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

Make Custom Metrics Basic Authentication Optional #5

Closed
wants to merge 5 commits into from

Conversation

asalan316
Copy link
Contributor

As part of the PR#3044, Application Autoscaler allows users to generate their required credential types while authenticating with custom metrics. The two available authentication schemes are basic and mTLS.

Generation of such credentials can be done by setting credential_type binding parameter.

credential-type = "binding-secret" => allow basic authentication
credential-type = "x509" => allow mTLS authentication only

This PR documents the above mentioned behaviour

@asalan316 asalan316 added the in-discussion This item is being discussed internally label Jul 8, 2024
@asalan316 asalan316 force-pushed the custom-metrics-basic-auth-optional branch 2 times, most recently from 9ffac17 to c35c405 Compare July 9, 2024 08:15
@asalan316 asalan316 force-pushed the custom-metrics-basic-auth-optional branch from c35c405 to 21f4283 Compare July 9, 2024 08:22
@asalan316 asalan316 requested review from sapsy and silvestre July 24, 2024 11:36
docs/defining-a-custom-metric-87e657e.md Outdated Show resolved Hide resolved
docs/defining-a-custom-metric-87e657e.md Outdated Show resolved Hide resolved
docs/defining-a-custom-metric-87e657e.md Outdated Show resolved Hide resolved
docs/defining-a-custom-metric-87e657e.md Outdated Show resolved Hide resolved
sapsy and others added 4 commits July 31, 2024 17:20
Co-authored-by: Silvestre Zabala <silvestre.zabala@sap.com>
Co-authored-by: Silvestre Zabala <silvestre.zabala@sap.com>
Co-authored-by: Silvestre Zabala <silvestre.zabala@sap.com>
@asalan316
Copy link
Contributor Author

asalan316 commented Aug 1, 2024

Suggestions applied. Thanks

This change should be part of

  • Release 7b ( if possible). Otherwise, Release 8b

@asalan316 asalan316 removed the in-discussion This item is being discussed internally label Aug 2, 2024
@asalan316
Copy link
Contributor Author

@sapsy The PR look good from dev team side. Could you please merge it after your approval. Thanks

@silvestre
Copy link
Contributor

changes have already been applied

@silvestre silvestre closed this Oct 28, 2024
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.

3 participants