-
Couldn't load subscription status.
- Fork 13
Adding a new function to override the client args from terraform provider of synthetics #37
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
Conversation
|
CLA Assistant Lite bot CLA Assistant Lite bot All contributors have signed the COC ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
Testing results along with changes in splunk/terraform-provider-synthetics#78. 1. Failure logs with worng custom URL - 2025-08-18T19:58:54.916+0530 [ERROR] vertex "synthetics_create_browser_check_v2.browser_v2_foo_check" error: Post "https://api.lab.signalfx.com/v2/synthetics/tests/browser": dial tcp: lookup api.lab.signalfx.com: no such host
2. Logs with no custom URL field [ I abrupty failed as I dont want to apply]
3. Logs with custom URL and working [ 403 because of VPN issue]
|
|
I have read the Code of Conduct and I hereby accept the Terms |
Before the change?
In one of our example, instead of using api..signalfx.com we are making use of external-api..signalfx.com api URL. So in order to achieve it, following support is added in which if apiUrl is passed in providers.tf, it will take precedence otherwise default one will be used.
In order to make that work in synthetics teraform provider, it is required to update the synthetics client code as well so that the non exportable variables can be set from Upstream.
After the change?
This is totally a new function in client code, so it will not affect the existing functionality in any way.
Pull request checklist
make testacc), and pasted in this PR (for bug fixes / features)Acceptance Test Output
Does this introduce a breaking change?