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

Add adapter metrics service flags to 2.14 docs #1302

Merged

Conversation

aliaqel-stripe
Copy link
Contributor

Add section in the cluster docs in version 2.13 to describe the metrics-service flags (including the one added by kedacore/keda#5450

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO)

Fixes #kedacore/keda#5449

@aliaqel-stripe aliaqel-stripe requested a review from a team as a code owner January 31, 2024 20:02
Copy link

netlify bot commented Jan 31, 2024

Deploy Preview for keda ready!

Name Link
🔨 Latest commit 97d4f9e
🔍 Latest deploy log https://app.netlify.com/sites/keda/deploys/65ca6981be444600080347b9
😎 Deploy Preview https://deploy-preview-1302--keda.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

Thank you for your contribution! 🙏 We will review your PR as soon as possible.

While you are waiting, make sure to:

  • Add your contribution to all applicable KEDA versions
  • GitHub checks are passing
  • Is the DCO check failing? Here is how you can fix DCO issues

Learn more about:

@aliaqel-stripe aliaqel-stripe changed the title Add adapter metrics service flags to 2.13 docs Add adapter metrics service flags to 2.14 docs Jan 31, 2024
| Adapter Flag | Default Value | Description |
| ----------------------------------- | ------------------------------------------------- | ---------------------------------------------- |
| metrics-service-address | keda-operator.keda.svc.cluster.local:9666 | The address of the GRPC Metrics Service Server |
| metrics-service-grpc-authority | "" | Host Authority override for the Metrics Service if the Host Authority is not the same as the address used for the GRPC Metrics Service Server. |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we add a note explaining where this can be useful? a brief description of your use case would be nice (from top level, but just for giving a real world case to improve the understanding of the parameter)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let me know what you think of the updated comment

Copy link
Member

@JorTurFer JorTurFer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@aliaqel-stripe aliaqel-stripe force-pushed the aliaqel/add-adapter-flags-to-docs branch from 5268971 to f34f712 Compare February 7, 2024 19:51
content/docs/2.14/operate/cluster.md Outdated Show resolved Hide resolved
content/docs/2.14/operate/cluster.md Outdated Show resolved Hide resolved
content/docs/2.14/operate/cluster.md Outdated Show resolved Hide resolved
Signed-off-by: Ali Aqel <[email protected]>
@aliaqel-stripe aliaqel-stripe force-pushed the aliaqel/add-adapter-flags-to-docs branch from 4a7f056 to 571f9c3 Compare February 12, 2024 18:53
@zroubalik zroubalik merged commit 8f7e33e into kedacore:main Feb 12, 2024
8 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.

3 participants