Skip to content

Helm: fix hardcoded datadog ad url#3877

Closed
dgdelahera wants to merge 2 commits intohashicorp:mainfrom
dgdelahera:fix_dd_ad_annotations
Closed

Helm: fix hardcoded datadog ad url#3877
dgdelahera wants to merge 2 commits intohashicorp:mainfrom
dgdelahera:fix_dd_ad_annotations

Conversation

@dgdelahera
Copy link

Changes proposed in this PR

How I've tested this PR

Result can be checked in the test. Now the ad URL will always point to the correct server service name.

How I expect reviewers to test this PR

Checklist

@hashicorp-cla-app
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@dgdelahera dgdelahera changed the title fix hardcoded datadog ad url Helm: fix hardcoded datadog ad url Apr 4, 2024
@dgdelahera
Copy link
Author

I'm unable to sign the CLA:

image

@david-yu
Copy link
Contributor

david-yu commented Apr 4, 2024

Hi @dgdelahera We have a PR in the works that should resolve this! #3685 Thanks for filing. We'll resolve your issue when that PR is closed.

@david-yu david-yu closed this Apr 4, 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.

2 participants