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 to provide custom datadog sites #5978

Closed
nishalshettigar opened this issue Jul 24, 2024 · 6 comments
Closed

Add option to provide custom datadog sites #5978

nishalshettigar opened this issue Jul 24, 2024 · 6 comments
Labels
feature-request All issues for new features that have not been committed to needs-discussion stale All issues that are marked as stale due to inactivity

Comments

@nishalshettigar
Copy link

nishalshettigar commented Jul 24, 2024

Proposal

We are using an enterprise version of datadog in my company and we have a separate custom site for our datadog instance (<org-name>.datadoghq.com). We were trying to implement keda autoscaling using the datadog scaler, but were facing an issue while doing so. We passed our datadog site in the datadogSite parameter under TriggerAuthentication object.

When we did that we got the following error:

error establishing Datadog connection: error connecting to Datadog API endpoint: The variable site in the server URL has invalid value <org-name>.datadoghq.com. Must be [datadoghq.com us3.datadoghq.com us5.datadoghq.com datadoghq.eu ddog-gov.com]

Hence, we need a mechanism to pass these custom datadog site instances to keda as well.

Use-Case

It will allow teams having their own DD subdomain (<org-name>.datadoghq.com) to be able to integrate keda with their datadog instance

Is this a feature you are interested in implementing yourself?

Yes

Anything else?

I had a brief look at the datadog scaler code and could see that the datadogSite parameter is mapped to the site parameter in the datadog API, which only accepts the values [datadoghq.com us3.datadoghq.com us5.datadoghq.com datadoghq.eu ddog-gov.com]. I also noticed that there is another argument called subdomain in the datadog API and hence I would propose to add another subdomain parameter under the trigger authentication object and pass that along to the datadog API while creating the DatadogConnection object.

@nishalshettigar nishalshettigar added feature-request All issues for new features that have not been committed to needs-discussion labels Jul 24, 2024
@arapulido
Copy link
Contributor

Hello @nishalshettigar ! I don't think this is needed, even if you have a custom URL, your organization is hosted in one of our regions, so adding just the Datadog region you are (datadoghq.com, us3.datadoghq.com, us5.datadoghq.com, datadoghq.eu or ddog-gov.com) and a correct API and APP keys that should work.

Can you try that and let me know, please? Thanks!

@nishalshettigar
Copy link
Author

Hi @arapulido, I tried without the site parameter, and I'm getting the following error in the scaled object:

error establishing Datadog connection: error connecting to Datadog API endpoint: 403 Forbidden

Not able to verify which URL it is trying to connect to, but I'm guessing it's not connecting to the org URL.

@nishalshettigar
Copy link
Author

So I was checking our open telemetry integration with datadog and I noticed that it is working without the site parameter. So, I'm guessing that API token might not have read access (it is working with open telemetry) and that might be reason for that 403 error. I'll just try to verify that and update here.

@arapulido
Copy link
Contributor

@nishalshettigar yes, I confirmed that the subdomain is not required when connecting to the API, so it has to be something related to your permissions

Copy link

stale bot commented Sep 29, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale All issues that are marked as stale due to inactivity label Sep 29, 2024
Copy link

stale bot commented Oct 19, 2024

This issue has been automatically closed due to inactivity.

@stale stale bot closed this as completed Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request All issues for new features that have not been committed to needs-discussion stale All issues that are marked as stale due to inactivity
Projects
None yet
Development

No branches or pull requests

2 participants