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

Disk should return the size of the disk #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

goneri
Copy link

@goneri goneri commented Dec 30, 2015

Disk key should give the size of the first disk in GB, not the number of hard
drive. During the nova boot, nova will compare this value with the
minimal mandatory size associated with the Glance image.

Without this patch, nova will always fail and returns the "no valid host was found"
message as soon as the size is greater than 1GB.

See: https://www.rdoproject.org/install/instack-faq/#What_is_the_NODES_JSON_file_format.3F

@goneri goneri changed the title Disk should returns the size of the disk Disk should return the size of the disk Dec 30, 2015
Disk key should give the size of the first disk in GB, not the number of hard
drive. During the nova boot, nova will compare this value with the
minimal mandatory size associated with the Glance image.

Without this patch, nova will always fail and returns the "no valid host was found"
message as soon as the size is greater than 1.
This parameter allow the user to manually specify what NIC should be
used to boot the system. There is three reasons to prefer this solution:
- depending on how the NIC are negotiated, we may have a 10Gb NIC seen
  has a 1G NIC. In this case, idracula will pick the wrong MAC.
- this speed up the discovery
- we don't need anymore to turn the nodes on first.
For the moment, ironic ramdisk doesn't support PERC RAID. It makes more
sense to use the size of the first physical disk.
With this patch, it's also possible to drop all the virtual disk
configuration and do JBOD instead.
goneri pushed a commit to goneri/idracula that referenced this pull request Jan 19, 2016
Default to IPMI instead of iDrac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant