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: Enable ironic on the bootstrap VM #2060

Closed
hardys opened this issue Jul 22, 2019 · 4 comments
Closed

baremetal: Enable ironic on the bootstrap VM #2060

hardys opened this issue Jul 22, 2019 · 4 comments
Assignees
Labels
platform/baremetal IPI bare metal hosts platform

Comments

@hardys
Copy link
Contributor

hardys commented Jul 22, 2019

openshift-install currently assumes Ironic has been stood up outside of the installer. Terraform will drive that instance of Ironic to provision the masters which works but isn't the desired self-contained integration.

Instead, we'd like to be able to minimize external pre-requisites and be able to spin up Ironic within the bootstrap VM for provisioning the masters.

In addition, this instance of Ironic is supposed to be temporary. Once the cluster comes up, the baremetal-operator should be managing Ironic within the cluster so moving the master deployment instance of Ironic to the bootstrap VM solves this problem, because the bootstrap VM will be destroyed when boostrap of the masters is completed.

Copied from openshift-metal3/kni-installer#68

@hardys
Copy link
Contributor Author

hardys commented Jul 22, 2019

/assign

@hardys
Copy link
Contributor Author

hardys commented Jul 22, 2019

/label platform/baremetal

@hardys
Copy link
Contributor Author

hardys commented Jul 22, 2019

cc @russellb please can you add the platform/baremetal label - thanks!

@russellb russellb added the platform/baremetal IPI bare metal hosts platform label Jul 22, 2019
stbenjam pushed a commit to hardys/installer that referenced this issue Jul 29, 2019
For the baremetal platform we require two different bootimages,
the QEMU one for the libvirt based bootstrap VM, and the OpenStack
one that contains the necessary Ironic config drive support to pass
data to ignition.

So we rework the Image abstraction by adding a new BootstrapImage
type/asset, which returns the same as Image in all cases except for
the baremetal platform.

This also aligns with the tfvars renamed in openshift#2044 and allows us to
pass the rhcos.QEMU image via BootstrapImage to terraform but leaves
the OpenStack image URL available for future use to deploy masters
via follow-up PRs that implement issue openshift#2060 and also correctly set
the worker machineset providerSpec for the baremetal-operator.

Related: openshift#2037
hardys pushed a commit to hardys/installer that referenced this issue Aug 13, 2019
This adds ironic to the bootstrap VM for the baremetal IPI platform.

Related: openshift#2060
@hardys
Copy link
Contributor Author

hardys commented Aug 22, 2019

Closed via #2079

@hardys hardys closed this as completed Aug 22, 2019
jhixson74 pushed a commit to jhixson74/installer that referenced this issue Dec 6, 2019
This adds ironic to the bootstrap VM for the baremetal IPI platform.

Related: openshift#2060
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

No branches or pull requests

2 participants