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

fix vmtype to make more sense, add ostype #381

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix vmtype to make more sense, add ostype #381

wants to merge 1 commit into from

Conversation

sjorge
Copy link
Contributor

@sjorge sjorge commented Dec 20, 2014

some cleanups to the vmadm man page

@rmustacc
Copy link
Contributor

@joshwilsdon can you please review? Let me know when you're happy and I'll go from there.

@joshwilsdon
Copy link
Contributor

Why do you think that this change is necessary? I'm not sure that I agree that this change makes more sense.

From my perspective the "type" field here is intended to expose which type of virtualization / emulation we're using. All of the current "OS" type brands use native OS virtualization where all syscalls are untranslated. The current "KVM" type uses the kvm hardware virtualization and is obviously emulating everything at the hardware level. The "LX" type was set here because the virtualization is not directly native since we're emulating Linux syscalls for applications in this zone.

@sjorge
Copy link
Contributor Author

sjorge commented Apr 9, 2015

The intent is to make it more clear that all stuff that applies to OS type also applies to LX but not the other way around.

Current types are KVM, OS, and LX but OS and LX are basically the same, except that lx takes a few extra's like kernel version.

@joshwilsdon joshwilsdon removed their assignment Dec 5, 2019
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.

None yet

4 participants