-
Notifications
You must be signed in to change notification settings - Fork 25
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
Allow setting properties and root_device #10
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - a couple of comments below
Due to limitations in terraform, we can't pass a nested map here so instead surface the root_device as a top-level interface, then set the map value internally. More info at hashicorp/terraform#2114 Fixes: openshift-metal3#8
This is required for enabling extra disks, e.g for rook/ceph This also enables an extra disk so we can prove this works and support testing with ceph/rook. Note this requires openshift-metal3/terraform-provider-ironic#10
This is required for enabling extra disks, e.g for rook/ceph This also enables an extra disk so we can prove this works and support testing with ceph/rook. Note this requires openshift-metal3/terraform-provider-ironic#10
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
We'll need to add root_device
to the installer instead of utils.sh
as well
Good point, I added it to utils.sh assuming that patch may land before the switch to kni-installer. Are you planning a rebase to the kni-installer patch, or should I post a follow-up? I guess we'll need to expose this via install-config for the baremetal platform as well. |
If this goes first I'll update the kni-installer PR's to include root_device |
This is required for enabling extra disks, e.g for rook/ceph This also enables an extra disk so we can prove this works and support testing with ceph/rook. Note this requires openshift-metal3/terraform-provider-ironic#10
This is required for enabling extra disks, e.g for rook/ceph This also enables an extra disk so we can prove this works and support testing with ceph/rook. Note this requires openshift-metal3/terraform-provider-ironic#10
This is required for enabling extra disks, e.g for rook/ceph This also enables an extra disk so we can prove this works and support testing with ceph/rook. Note this requires openshift-metal3/terraform-provider-ironic#10
This is required for enabling extra disks, e.g for rook/ceph This also enables an extra disk so we can prove this works and support testing with ceph/rook. Note this requires openshift-metal3/terraform-provider-ironic#10
Fixes #8