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

Create New VM dialog inconsistent handling of Standalone + HVM #1251

Closed
tasket opened this Issue Sep 29, 2015 · 9 comments

Comments

5 participants
@tasket

tasket commented Sep 29, 2015

In Qubes Manager 'Create New VM' dialog window, selecting first Standalone then HVM will give you the option of choosing a template (and this appears to work). However, first selecting HVM will auto-check the Standalone option, then ghost out the template dropdown box; There is no reason given why the template option should differ from selecting dialog options in a different order.

As it is, having Standalone checked /sometimes/ results in the creation of a standalone HVM having a clone of an HVM template, and /sometimes/ creates an HVM with a blank root.

Expected behavior: The status of the template dropdown box should not depend on the order in which Standalone or HVM options were selected. The dialog should be normalized so that each combination of VM type + Standalone produces a unique and discernible state. An additional radio button, checkbox or dropdown box item may be what's needed to clarify the operation of this UI function.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Sep 29, 2015

Member

Original intention was to provide only two options:

  1. New, blank standalone HVM (so no template option allowed)
  2. Template-based HVM

But you're right, this approach is wrong.

Member

marmarek commented Sep 29, 2015

Original intention was to provide only two options:

  1. New, blank standalone HVM (so no template option allowed)
  2. Template-based HVM

But you're right, this approach is wrong.

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Sep 29, 2015

It seems that a third option is manifesting: A new Standalone HVM that gets a cloned root from a template. Is this useful?

tasket commented Sep 29, 2015

It seems that a third option is manifesting: A new Standalone HVM that gets a cloned root from a template. Is this useful?

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Sep 29, 2015

Member

I don't know, do you see any use case for it? It need to be consistent,
but I have no opinion whether "standalone HVM created from template"
should be supported or not.

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Member

marmarek commented Sep 29, 2015

I don't know, do you see any use case for it? It need to be consistent,
but I have no opinion whether "standalone HVM created from template"
should be supported or not.

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

@entr0py

This comment has been minimized.

Show comment
Hide comment
@entr0py

entr0py Dec 26, 2016

I have no opinion whether "standalone HVM created from template"
should be supported or not.

I may be missing something obvious but why would need for this case (template-based standalone HVM) differ any from need for its PVM counterpart?

do you see any use case for it?

Windows, specifically, has a torturous update and config process. If I need a persistent file system, it would be beneficial to be able to fork a standaloneVM from a Windows template, instead of having to start from blank. The other option would be to clone and use the templateVM but that would obviously not be canonical use of a templateVM. Using HVM-template clones is what I did to test Debian-9 with different desktop environments.

entr0py commented Dec 26, 2016

I have no opinion whether "standalone HVM created from template"
should be supported or not.

I may be missing something obvious but why would need for this case (template-based standalone HVM) differ any from need for its PVM counterpart?

do you see any use case for it?

Windows, specifically, has a torturous update and config process. If I need a persistent file system, it would be beneficial to be able to fork a standaloneVM from a Windows template, instead of having to start from blank. The other option would be to clone and use the templateVM but that would obviously not be canonical use of a templateVM. Using HVM-template clones is what I did to test Debian-9 with different desktop environments.

@unman

This comment has been minimized.

Show comment
Hide comment
@unman

unman Apr 16, 2017

Member

@andrewdavidwong I believe this issue is resolved in 3.2, and can be closed. The same options appear regardless of order of selection.

Member

unman commented Apr 16, 2017

@andrewdavidwong I believe this issue is resolved in 3.2, and can be closed. The same options appear regardless of order of selection.

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Apr 16, 2017

@unman If one or more HVM templates exist in the system, the inconsistency still occurs as described.

tasket commented Apr 16, 2017

@unman If one or more HVM templates exist in the system, the inconsistency still occurs as described.

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Apr 16, 2017

I'll try to fix this and submit a PR.

tasket commented Apr 16, 2017

I'll try to fix this and submit a PR.

@unman

This comment has been minimized.

Show comment
Hide comment
@unman

unman Apr 16, 2017

Member

@tasket You are, of course, quite right. I reverted my QubesManager, removed internal mark on template and indeed the problem remains.
I would favour an additional checkbox as you suggested earlier, if that doesnt make the UI too cumbersome

Member

unman commented Apr 16, 2017

@tasket You are, of course, quite right. I reverted my QubesManager, removed internal mark on template and indeed the problem remains.
I would favour an additional checkbox as you suggested earlier, if that doesnt make the UI too cumbersome

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Apr 16, 2017

So far I'm trying it with same checkboxes, but user can choose 'none' (default) in the template list when both HVM and Standalone are selected.

tasket commented Apr 16, 2017

So far I'm trying it with same checkboxes, but user can choose 'none' (default) in the template list when both HVM and Standalone are selected.

tasket pushed a commit to tasket/qubes-manager that referenced this issue Apr 16, 2017

tasket
Begin fix for create_new_vm
Per QubesOS/qubes-issues#1251

Makes HVM+Standalone choices consistent: Allows choosing creation of Standalone HVM from HVM template unless 'none' (the default) is selected in template list.

@marmarta marmarta referenced this issue in QubesOS/qubes-manager Sep 13, 2017

Merged

Create new VM GUI #41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment