-
justinw
Hi! Does anyone have any experience getting RHEL9 based distros (Rocky, specifically) to boot as a bhyve hvm on smartos? Would someone mind looking at this vmadm get and tell me if I'm doing something dumb? (vmadm get:
pastebin.com/W7rx6SNc // booting with: vmadm boot 669bfb47-bdc5-6cd3-efa2-c26ac1ff9c96 order=cd,once=d cdrom=/Rocky-9.1-x86_64-minimal.iso,ide)
-
justinw
Over vnc, I get a uefi prompt where it looks like there's only one storage device but I'm really not sure next steps for uefi debugging. :(
-
justinw
Mapping table BLK0: Alias(s) PciRoot(0x0)/Pci(0x4,0x0) ... i'd think with the disk and iso there should be two things?
-
justinw
Oh okay, looks like Pci(0x4,0x0) is the virtio disk 0 and its just not picking up the CD.
-
justinw
Do I have to specify in the vmspec json that the machine has a cdrom drive, maybe?
-
justinw
Same behavior w/ Debian 11 netinstall ... so I think I'm just doing something dumb. :(
-
bahamat
-
pmooney
how many folks are running smartos under modern qemu/kvm/ovmf?
-
pmooney
I just updated my (linux) hypervisor, and now a smartos guest is griping about the garbage smbios info it's getting from that qemu/ovmf combo
-
bahamat
We've moved our jenkins agents to running in bhyve instead of kvm.
-
bahamat
I've never run it under linux kvm.
-
pmooney
the fields for the cpu information have some garbage in them, which makes its way through sysinfo to vminfod, which goes into maintenance
-
bahamat
How garbage is it?
-
pmooney
"CPU Type": "\x01",
-
bahamat
Maybe I need to do something like this?
TritonDataCenter/smartos-live 27dee23
-
bahamat
OS-8428
-
jinni
-
bahamat
So back story on that, illumos#13560 changed how smbios expresses non-printable characters.
-
jinni
-
bahamat
And sysinfo just mostly tries to take a bare string and put quotes around it, which isn't always valid JSON encoded text.
-
pmooney
yeah
-
bahamat
So in OS-8248 I JSON.stringify()'d those strings.
-
jinni
Sorry, OS-8248 is not public.
-
bahamat
8428, even
-
bahamat
So maybe I need to do that in more places?
-
pmooney
probably anywhere data is being emitted from smbios?
-
pmooney
CPU_Version looks like the one straggler
-
bahamat
But yours is hanging up on CPU_Type?
-
bahamat
Oh, but that's just CPU_Version
-
pmooney
Pardon, CPU_Version is the variable name used in the script
-
pmooney
yeah
-
bahamat
Yeah, ok.
-
bahamat
vendor also
-
bahamat
Ok, I'll get to that on Monday.
-
pmooney
vendor isn't being emitted as part of the json, though
-
pmooney
I don't think
-
bahamat
Oh, ok, yeah that's just used to check if it's bhyve.
-
bahamat
I don't see vendor referenced anywhere else.
-
bahamat
If you're in a hurry, and you want to do a PR, I'll fast track it for you (not to dump work on you...)
-
bahamat
But otherwise, we just had a release, so I can get you a build early next week.
-
bahamat
Sorry about that. I had wondered about CPU_Vendor at the time. I should have just done it along with the rest of them.
-
bahamat
If you want to try hotpatching it, I think this will fix it:
gist.github.com/afb15c96135162e1f93d7e29613da268
-
pmooney
that function doesn't output the bits directly
-
pmooney
it just sets variables for later use
-
pmooney
AFAICT
-
bahamat
Yeah, but that's all we need.
-
pmooney
will it not be also emitted by output_parsable()"
-
bahamat
Once it's strigified, it'll be valid later.
-
pmooney
?
-
pmooney
oh, pardon, you're evaling that
-
pmooney
yeah
-
bahamat
Yeah.
-
bahamat
-
bahamat
If you can verify that works, I'll get that merged in on Monday, and I can cut a build for you if you'd like.
-
pmooney
seems to work
-
bahamat
OK, cool.
-
pmooney
the sysinfo caching means I don't even need the overlayed version
-
pmooney
since the "real" sysinfo just uses the cached version, haha
-
pmooney
*cached data
-
bahamat
Oh, that's right. Yeah, we did that last time I tested it.
-
pmooney
I personally don't need a special build
-
pmooney
there's been enough drama lately
-
bahamat
Also kind of makes testing it a pain in the ass, because when I was running it repeatedly I had to hack up sysinfo so that it *wouldn't* read or write the cache.
-
pmooney
yeah
-
bahamat
Ok, smartos-live#1068
-
jinni
-
bahamat
Can you put a comment there that you tested it and it fixed the issue for you?
-
justinw
bahamat: thanks i'll check that out!
-
pmooney
bahamat: added some commentary to the PR
-
bahamat
Perfect, thanks!
-
pmooney
thanks for looking at it on a saturday
-
bahamat
Well, luckily it was one I had remembered from just a few weeks ago.
-
bahamat
Having already worked it out then, the fix was simple.
-
bahamat
You're just lucky you caught me when I happened to be at my computer.