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 option for the Datadog scaler to use the Datadog Cluster Agent as source of metrics #5355

Closed
arapulido opened this issue Jan 8, 2024 · 2 comments · Fixed by #5496
Closed
Assignees
Labels
feature All issues for new features that have been committed to

Comments

@arapulido
Copy link
Contributor

Proposal

Right now, the Datadog scaler polls the Datadog REST API to get the query values that will drive scaling events. This solutions works, but many users, particularly those who have big infrastructures, are getting rate limited, limiting the usefulness of the feature.

The Datadog Cluster Agent, a component that is present in clusters that are monitored with Datadog, already has functionality to get metric values from Datadog with the same purpose (autoscaling).

Ideally, the Datadog KEDA scaler would have an option to connect to the Cluster Agent to get metrics values, instead of calling the REST API, as it will decrease the risk of being rate limited.

The user would have the choice of choosing either the REST API (that would remain default for now, for backwards compatibility), or choose to connect to the Cluster Agent instead.

Use-Case

No response

Is this a feature you are interested in implementing yourself?

Yes

Anything else?

No response

@arapulido arapulido added feature-request All issues for new features that have not been committed to needs-discussion labels Jan 8, 2024
@tomkerkhove tomkerkhove removed needs-discussion feature-request All issues for new features that have not been committed to labels Jan 9, 2024
@tomkerkhove tomkerkhove added the feature All issues for new features that have been committed to label Jan 9, 2024
@tomkerkhove
Copy link
Member

I like this idea 👍

@JorTurFer
Copy link
Member

I love it! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature All issues for new features that have been committed to
Projects
Status: Ready To Ship
Development

Successfully merging a pull request may close this issue.

3 participants