Skip to content
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

baremetal: installer should register hosts for masters and workers #2063

Closed
russellb opened this issue Jul 22, 2019 · 3 comments · Fixed by #2087
Closed

baremetal: installer should register hosts for masters and workers #2063

russellb opened this issue Jul 22, 2019 · 3 comments · Fixed by #2087
Assignees
Labels
platform/baremetal IPI bare metal hosts platform

Comments

@russellb
Copy link
Member

russellb commented Jul 22, 2019

Original issue from @dhellmann here: openshift-metal3/kni-installer#46


Platform:

baremetal

What happened?

After the cluster was running, there were no BareMetalHost objects associated with the Machines for master Nodes.

What you expected to happen?

At least the information for the BareMetalHost CRs for the masters should be registered. If those are set up with the machineRef linked to existing Machines, the hosts will be marked as "externally provisioned" and managed more safely (they won't be rebooted to inspect their hardware).

How to reproduce it (as minimally and precisely as possible)?

$ cd dev-scripts
$ make

Anything else we need to know?

To populate the hardware details for masters we will have to update the Status block of the host objects.

References

@zeenix
Copy link
Contributor

zeenix commented Jul 23, 2019

We modified the issue template to make it harder for issue creators to add names of non-relevant platforms in the description recently but seems we didn't do a good job of that. Could you please remove them from description, thanks.

EDIT: Ah, you copied from an older issue, that's why. :)

@russellb
Copy link
Member Author

We modified the issue template to make it harder for issue creators to add names of non-relevant platforms in the description recently but seems we didn't do a good job of that. Could you please remove them from description, thanks.

EDIT: Ah, you copied from an older issue, that's why. :)

I've removed the platform list, as well.

@dhellmann
Copy link
Contributor

/assign dhellmann

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/baremetal IPI bare metal hosts platform
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants