<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 29, 2018 at 5:26 PM, Piotr Kliczewski <span dir="ltr"><<a href="mailto:pkliczew@redhat.com" target="_blank">pkliczew@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div>Richard,<br><br></div><div>Great progress. I really like it!!!<br></div></div></div></div></div></div></div></blockquote><div><br></div><div>That's crazy, Rich! :)<br></div><div>And actually pretty straight forward … to read.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div></div><div><br></div><div>Here is what I noticed:<br><br></div>I see that in the yaml file we provide short-id as:<br></div><br>  os:<br>    osinfo: 'rhel7.2'<br><br></div>whereas kubevirt expects it in metadata:<br><br>metadata:<br>  labels:<br>    <a href="http://kubevirt.io/os" target="_blank">kubevirt.io/os</a>: win10<br><br></div></div></div></div></blockquote><div><br></div><div>Yes, such a label is needed - The value should be the libosinfo OS short-id.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div></div>For windows vms we need to create perset like this one [1]. We can add separator </div></div></div></blockquote><div><br></div><div>We need to see how this evolves.<br></div><div>I'd aim at saing that KubeVirt will provide - as part of it's releases - presets for a given set of short-ids.<br><br></div><div>Then a client - like v2v - can assume - or knows - for which OSses presets exist and can just reference them.<br><br></div><div>Really cool.<br></div><div><br></div><div>- fabian<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><pre>---</pre>between the objects.<br><br></div>Thanks,<br></div>Piotr<br><div><div><br>[1] <a href="https://github.com/kubevirt/kubevirt/blob/master/manifests/release/demo-content.yaml.in" target="_blank">https://github.com/kubevirt/<wbr>kubevirt/blob/master/<wbr>manifests/release/demo-<wbr>content.yaml.in</a><br><div><br><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 27, 2018 at 6:31 PM, Richard W.M. Jones <span dir="ltr"><<a href="mailto:rjones@redhat.com" target="_blank">rjones@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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>
</blockquote></div><br></div>
</blockquote></div><br></div></div>