-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Enable azure_private_dns to work with non "AzurePublicCloud" clouds #1578
Enable azure_private_dns to work with non "AzurePublicCloud" clouds #1578
Conversation
Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA. It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Welcome @daddonpa! |
bf8576a
to
d7f54c9
Compare
I signed it |
@daddonpa is there a way we can add a test or prove this behaviour? |
Added test to verify that I am not sure if my way of testing (setting environment variable and casting an interface to the type I know it should be) is fine for go in general and this repository in particular but I could not find a better way to do it. The same test can be added for |
@Raffo is there something else I should add? |
/assign @njuettner |
/kind feature |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@daddonpa can you rebase your PR to pull in the latest changes from master? I see that the new GItHub action CI check did not run for this PR. Thanks! |
/hold Needs a rebase before it can be merged. |
…like "AzureUSGovernmentCloud", "AzureChinaCloud" or "AzureGermanCloud"
…nagerEndpoint from the configured cloud
…re/azure_privatedns_test.go
6106ef5
to
83d7364
Compare
@seanmalloy rebased commits onto master |
/hold cancel @tariq1890 please review one more time now that the PR has been rebased. Thanks! |
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add you PR to the CHANGELOG.md, we can merge it afterwards
@njuettner Added Pull Request to CHANGELOG.md |
/assign @Raffo @njuettner |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: daddonpa, Raffo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The PrivateZoneClient by default communicates to the AzurePublicCloud resource manager endpoint.
To make the
azure_private_dns
work with other clouds likeit should use the endpoint specific to the selected cloud.
This change makes it behave like the
NewAzureProvider
fromazure.go