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

virt-build-kata: review the parameters passed to virt-install #3

Open
fidencio opened this issue Feb 15, 2021 · 1 comment
Open

virt-build-kata: review the parameters passed to virt-install #3

fidencio opened this issue Feb 15, 2021 · 1 comment

Comments

@fidencio
Copy link

Here, I'd just like to check & suggest whether we need the parameters currently used.

We use, for now:

  • --ram, I'd suggest using --memory instead, just because it was introduced in order to deprecate --ram;
  • --vcpus`, that's ok;
  • --disk, that's ok;
  • --graphics vnc,listen=0.0.0.0, do we have to pass that? or can se just fallback to the console and happy about that?
  • --os-type=$osvariant, similarly to the --ram / --memory case, we better use --os-variant here;
  • --network network=default, why do we need this?

Last but not least, I wonder whether we could be opinionated and force people to use -c qemu:///system just to ensure we'll, later on, be able to ssh into the machine.

What do you think?

@alicefr
Copy link
Owner

alicefr commented Feb 16, 2021

Here, I'd just like to check & suggest whether we need the parameters currently used.

We use, for now:

  • --ram, I'd suggest using --memory instead, just because it was introduced in order to deprecate --ram;

Ok, I'll change

  • --vcpus`, that's ok;
  • --disk, that's ok;
  • --graphics vnc,listen=0.0.0.0, do we have to pass that? or can se just fallback to the console and happy about that?

I had to add it because for ubuntu I was not able to see the console. This seems to be a known issue and you need to activate the serial getty consoe. I actually have it as firstboot param, so I can remove this.

  • --os-type=$osvariant, similarly to the --ram / --memory case, we better use --os-variant here;

Ok thanks

  • --network network=default, why do we need this?

No, I don't think we need it. It was me with the wrong setup. Maybe I'll put an optional flag is people want to configure the network

Last but not least, I wonder whether we could be opinionated and force people to use -c qemu:///system just to ensure we'll, later on, be able to ssh into the machine.

I'm not very familiar with the session mode. If you think, it's better we can configure it.

What do you think?

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

No branches or pull requests

2 participants