Revert "openstack: Del DNS from service VM update docs"#1094
Revert "openstack: Del DNS from service VM update docs"#1094openshift-merge-robot merged 1 commit intoopenshift:masterfrom tomassedovic:reverte-openstack-dns-removal
Conversation
This reverts commit ae54392. The proposed solution (including the exact same CoreDNS configuration) does not work for several of our developers deployments causing a regression (i.e. we're not able to get the etcd cluster started). It appears that the DNS requests are getting the CNAME values for the etcd records, but they never resolve to an IP address. We've not figured out why that is yet, so reverting this change lets us unblock the OpenStack deployments.
openshift-ci-robot
added
size/L
openshift-ci-robot
added
the
do-not-merge/hold
|
I think we should go with this revert to allow deployments. As next step, we should think how to flip the /etc/resolv.conf and in parallel try to figure out the cause for the different behavior. |
|
I'm ok with this. I'd like to get green light from @wking or someone from the openshift team, tho. |
|
/test e2e-aws |
|
Fine with me. Should not affect anything but OpenStack, and can reasonably
be considered a bug fix. (but need a real exception before you can merge)
…On Thu, Jan 17, 2019 at 1:07 PM Flavio Percoco ***@***.***> wrote:
/test e2e-aws
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1094 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAS4CuVY3pe3tmoB58cUh7mNYNNt7GLBks5vELvIgaJpZM4aGICO>
.
--
Russell Bryant
|
|
Once I see the written approval, I'll go ahead and merge this. |
|
/test e2e-aws |
|
/hold cancel |
openshift-ci-robot
removed
the
do-not-merge/hold
|
moving forward with this as it's been approved already: 👍 /lgtm |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
moving forward with this as it's been approved already: 👍 /lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flaper87, tomassedovic The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/test e2e-aws |
1 similar comment
|
/test e2e-aws |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
9 participants
This reverts commit ae54392.
The proposed solution (including the exact same CoreDNS configuration)
does not work for several of our developers deployments causing a
regression (i.e. we're not able to get the etcd cluster started).
It appears that the DNS requests are getting the CNAME values for the
etcd records, but they never resolve to an IP address.
We've not figured out why that is yet, so reverting this change lets us
unblock the OpenStack deployments.