Closed activeshadow closed 1 year ago
@activeshadow lmk if you have any issues with the change I made in 2e6bae5. Based on the description I expected it to work with the vm name, but that wasn't originally the case
@jacdavi yeah it's been too long. 😂 So what's the difference between "vm name" and "hostname" now?
Based on how the "FindVMs" function works, perhaps we should use "<vm id, name, or uuid>" for the command args?
So what you had was matching by UUID or hostname where the hostname is the VM's actual hostname as reported by miniccc. Whereas FindVMs
will match by the vm name used when creating the VM (e.g., vm launch kvm <name>
).
@jacdavi ah yes. I remember now. 😂 So in reality, the VM ID, name, or UUID, or the hostname as reported by miniccc should work... should we worry about showing the VM ID should work too, or what the difference is between VM name and hostname?
I guess I mostly copied vm mount
which mentions it works by name or UUID, though ID would work too. So it's fine to leave that off.
I'm not sure how to further clarify the vm name vs hostname. There's also a potential issue there where multiple VMs can have the same hostname (which would be poor design, but not technically wrong).
LGTM. Let me know if you want me to squash and rebase.
@activeshadow lmk if you have any issues with the change I made in 2e6bae5. Based on the description I expected it to work with the vm name, but that wasn't originally the case