NVMe madness

[Fog_Newb]

@Sebastian-Roth

I don’t want to be that guy but…

Any updates?

[Sebastian Roth]

@Fog_Newb Not yet, sorry.

[Sebastian Roth]

@Fog_Newb Finally found some time to work on this. At first I was hoping to come up with kind of a logic that would detect disk size and select the correct disk according to this. As I said we have this working for images set to multiple disk type.

But half way into it I figured that my solution would only work for deployments where we have the size information from capturing beforehand. But this wouldn’t solve the issue of NVMe drives switching position on capturing in the first place.

The only way we can solve this is by telling FOS to look for a disk of certain size when running a task. So far we only allowed the host setting Host Primary Disk to name a Linux device file to use - e.g. /dev/nvme0n1. But now I added the functionality for you to specify disk size (byte count) using this same field.

So you just need to do is download the modified init.xz and put into /var/www/html/fog/service/ipxe/ directory (rename the original file beforehand) and set Host Primary Disk to an integer value matching exactly the byte count of the disk (value you get from blockdev --getsize64 /dev/...).

Please give it a try and let me knwo what you think,

[Fog_Newb]

@Sebastian-Roth

Thanks. I will test as soon as I can. Probably middle of the night or early tomorrow.

[Fog_Newb]

@Sebastian-Roth

I copied over the updated init.xz to

/var/www/html/fog/service/ipxe/

Then set the host primary disk to 1000204886016 and attempted to capture

It worked great

Thank you very much

[Sebastian Roth]

@Fog_Newb said in NVMe madness:

Then set the host primary disk to 1000204886016 and attempted to capture

So this was capturing the one tera byte disk, right?

[Fog_Newb]

@Sebastian-Roth

Yes.

[Fog_Newb]

@Sebastian-Roth

So yes, this is a perfect solution since Primary host disk can now be set by size. I have one image for the OS disk, and one for the “D” drive. I just switch the Primary Host disk setting depending on which image I want to capture or deploy.

[Sebastian Roth]

This is committed to the fos repo and will be available in the next release.

@Tom-Elliott Should we re-phrase the name of that field or just leave it like that and add this information to the documentation?

[Tom Elliott]

@Sebastian-Roth I think leave as is but update documentation.

[Sebastian Roth]

@JJ-Fullmer @Jurgen-Goedbloed Would one of you take care of adding this new feature to the documentation?

[Tom Elliott]

@Sebastian-Roth hmmm what about a new field specifically designed for the size? If hard drive specified default to that, else if size specified match drive based on that?

[Sebastian Roth]

@Tom-Elliott But why have another file if they can’t be used in combination anyway? It’s either or!

[Tom Elliott]

@Sebastian-Roth that’s what I mean. If you have specified the hdd use that. If hdd not found but you have size defined, try to find matching size. If both or either defined but not exist error out.

[Quazz]

@Tom-Elliott I kind of like that idea of a fallback mechanism, but I fear that it would kind of nullify what it’s trying to solve in a way.

Few people would know about the NVME naming scheme being essentially random on each boot and thus may presumably fill in the name of the drive. This would then get used and the size (if filled in) would be ignored.

I think if it were to be implemented as such, then the GUI should inform to use size over name for NVME drives.

PS: What happens when you have 2 identical drives?

[Sebastian Roth]

@Quazz Right, using a second field will just allow for people to confuse things. While we try to give people more responsibility instead of trying to solve all things within the code I don’t think adding another field is valuable in this term. It’s a simple case of either or in one field and there is no situation where specifying both size and dev name at the same time makes sense.

PS: What happens when you have 2 identical drives?

I have thought about this while implementing the multi disk NVMe code and this new stuff as well but I don’t think there is any possible way we can help people with two identical sized disks. If we store disk identifiers (globally unique) it cannot be deployed to a different target machine. As far as I see size is the only parameter helping us to solve this dilemma.

[Fog_Newb]

@Sebastian-Roth

Quick question. Does 1.5.9.36 dev build have these NVME related intits?

[Sebastian Roth]

@Fog_Newb No, not yet.

[Fog_Newb]

@Sebastian-Roth

Thanks. I updated and instead of checking I got lazy and posted. Where can I find info on what’s changed in each dev build? Github didn’t seem to have anything listed.

[Sebastian Roth]

@Fog_Newb Every dev version is just one single new commit to the dev-branch. So you know what changed looking at the commit history on github.