Skip to content

Conversation

@cybertron
Copy link
Contributor

Add a static mdns record that points to the API VIP. This is to
support the new required DNS record from OpenShift.

@cybertron
Copy link
Contributor Author

This is going to require changes in mdns-publisher as well. It also doesn't address the bootstrap node, which doesn't currently run mdns-publisher.

@celebdor celebdor added the CI check this PR with CI label May 9, 2019
@celebdor celebdor self-requested a review May 9, 2019 14:53
@derekhiggins
Copy link
Collaborator

Build SUCCESS, see build http://10.8.144.11:8080/job/dev-tools/629/

@markmc
Copy link
Contributor

markmc commented May 15, 2019

Please add a reference in the commit to the background context on the openshift change - e.g. openshift/installer#1633 is a good reference

@cybertron
Copy link
Contributor Author

I updated my local commit message, so references should be there when I'm ready to drop the WIP from this. We need openshift/mdns-publisher#7 before this will be effective though.

This is part of the work to fix openshift-metal3#538 and remove the requirement to
configure api-int as an external DNS record.
@cybertron
Copy link
Contributor Author

This version no longer has a dependency on mdns-publisher.

@cybertron cybertron changed the title WIP: Configure api-int mdns record Add api-int DNS record to coredns May 22, 2019
@derekhiggins
Copy link
Collaborator

Build SUCCESS, see build http://10.8.144.11:8080/job/dev-tools/667/

@hardys
Copy link

hardys commented May 24, 2019

Hmm does this depend on other commits? Since @cybertron mentioned it no longer depends on mdns-publisher I tried it locally and the masters can't resolve the api-int address to collect the ignition config.

@derekhiggins
Copy link
Collaborator

Build SUCCESS, see build http://10.8.144.11:8080/job/dev-tools/675/

@hardys
Copy link

hardys commented May 24, 2019

Ignore my previous comment, I accidentally tested on a branch with #560 applied and CI confirms this as working so lgtm

@hardys hardys merged commit 100c20a into openshift-metal3:master May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI check this PR with CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants