<div dir="ltr"><div><div><div><div><div><div>Richard,<br><br></div><div>Great progress. I really like it!!!<br></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">kubevirt.io/os</a>: win10<br><br></div>For windows vms we need to create perset like this one [1]. We can add separator <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">https://github.com/kubevirt/kubevirt/blob/master/manifests/release/demo-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>