Skip to content

Conversation

@imain
Copy link
Contributor

@imain imain commented Aug 7, 2019

metal3-config is a better name for the configmap and better
matches other configmap names.

metal3-config is a better name for the configmap and better
matches other configmap names.  This also matches the patch that
was applied to the MAO.
@sadasu
Copy link
Contributor

sadasu commented Aug 7, 2019

Yes, the name of the configmap matches what was just merged into MAO via openshift/machine-api-operator#302. Thanks for making the corresponding change here.

@sadasu
Copy link
Contributor

sadasu commented Aug 7, 2019

/lgtm

@dhellmann dhellmann added the CI check this PR with CI label Aug 7, 2019
sadasu
sadasu approved these changes Aug 7, 2019
@dhellmann
Copy link
Member

Waiting for CI http://10.8.144.11:8080/job/dev-tools/1008/

@metal3ci
Copy link

metal3ci commented Aug 7, 2019

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

@dhellmann dhellmann merged commit 820abd8 into openshift-metal3:master Aug 7, 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.

4 participants