<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">czw., 29.03.2018, 17:41 użytkownik Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> napisał:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Mar 29, 2018 at 05:26:13PM +0200, Piotr Kliczewski wrote:<br>
> Richard,<br>
><br>
> Great progress. I really like it!!!<br>
><br>
> Here is what I noticed:<br>
><br>
> I see that in the yaml file we provide short-id as:<br>
><br>
>   os:<br>
>     osinfo: 'rhel7.2'<br>
><br>
> whereas kubevirt expects it in metadata:<br>
><br>
> metadata:<br>
>   labels:<br>
>     <a href="http://kubevirt.io/os" rel="noreferrer noreferrer" target="_blank">kubevirt.io/os</a>: win10<br>
><br>
> For windows vms we need to create perset like this one [1]. We can add<br>
> separator<br>
><br>
> ---<br>
><br>
> between the objects.<br>
<br>
I'm a bit confused about the second part.  Is the VirtualMachinePreset<br>
file a separate file from the .yaml file that we're creating already<br>
or is it a replacement for it?<br>
<br>
We can create as many files as needed, or create different files<br>
depending on the machine type, or whatever is needed.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Sorry for not being clear enough. It can be single file with separator as i mentioned above. For windows we need 2 objects in the file for rhel just one.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Anyway I was hoping you'd take this patch and run with it.  Certainly<br>
the osinfo part (which I just made up on the spot) is dead simple to<br>
change.  You can post patches on this mailing list (eg. using ‘git<br>
send-email’) and we can review them together.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Yeah, this is the missing part. I like your approach is much better. Thank you!</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Rich.<br>
<br>
> Thanks,<br>
> Piotr<br>
><br>
> [1]<br>
> <a href="https://github.com/kubevirt/kubevirt/blob/master/manifests/release/demo-content.yaml.in" rel="noreferrer noreferrer" target="_blank">https://github.com/kubevirt/kubevirt/blob/master/manifests/release/demo-content.yaml.in</a><br>
><br>
><br>
><br>
> On Tue, Mar 27, 2018 at 6:31 PM, Richard W.M. Jones <<a href="mailto:rjones@redhat.com" target="_blank" rel="noreferrer">rjones@redhat.com</a>><br>
> wrote:<br>
><br>
> > Fixes some of the more egregious problems with v1, and also<br>
> > applies properly to the head of git without needing any other<br>
> > patches.<br>
> ><br>
> > Rich.<br>
> ><br>
> ><br>
<br>
--<br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
virt-df lists disk usage of guests without needing to install any<br>
software inside the virtual machine.  Supports Linux and Windows.<br>
<a href="http://people.redhat.com/~rjones/virt-df/" rel="noreferrer noreferrer" target="_blank">http://people.redhat.com/~rjones/virt-df/</a><br>
</blockquote></div></div></div>