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
Bug 1732124: Azure: don't allow installing with the same cluster name as an existing install #3120
Bug 1732124: Azure: don't allow installing with the same cluster name as an existing install #3120
Conversation
/hold needs #2764 |
/retitle Bug 1732124: Azure: don't allow installing with the same cluster name as an existing install |
@jhixson74: This pull request references Bugzilla bug 1732124, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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. |
abf524b
to
a3573e6
Compare
/lgtm but it'll probably need it again once the other PR merges and you rebase with it. |
a3573e6
to
b4ddf9b
Compare
b4ddf9b
to
b4d82f8
Compare
b4d82f8
to
bf268bc
Compare
I have eliminated the need for the other PR. This works without it now. |
/test e2e-azure |
bf268bc
to
1e824e5
Compare
/test e2e-azure |
@jhixson74: This pull request references Bugzilla bug 1732124, which is valid. 3 validation(s) were run on this bug
In response to this:
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. |
/test e2e-azure |
1 similar comment
/test e2e-azure |
0cb98cb
to
5803fe0
Compare
5803fe0
to
06e56f4
Compare
/test e2e-azure |
…install Check for an existing "api.clusterName" CNAME, A, and AAA record. If it exists, then the install will fail. See https://bugzilla.redhat.com/show_bug.cgi?id=1732124
06e56f4
to
ddf3973
Compare
/test e2e-azure |
/approve /hold cancel |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya 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 |
@jhixson74: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@jhixson74: All pull requests linked via external trackers have merged: openshift/installer#3120. Bugzilla bug 1732124 has been moved to the MODIFIED state. In response to this:
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. |
This code checks to see if an existing DNS record exists for api.clusterName and fails if it exists.
See https://bugzilla.redhat.com/show_bug.cgi?id=1732124