<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; ">Hi guys,</span></div><div apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><br></span></div><div apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; ">As already discussed in the Nova meeting pointed out by Sean</span> <a href="http://eavesdrop.openstack.org/meetings/nova/2012/nova.2012-11-29-21.01.html">http://eavesdrop.openstack.org/meetings/nova/2012/nova.2012-11-29-21.01.html</a></div><div apple-content-edited="true">we are already replacing the current Nova tests binary serialized stubs with text Json and as a next step to refactor the driver in order to add an abstraction layer on top of WMI that can be mocked in a consistent way with the rest of the Nova project. I'm not discussing here the reasons that brought us to implement the existing testing framework in the Folsom timeframe, as it has already discussed in the Nova meeting. I'd be glad to talk about it anytime for detailed explanations, so please reach out for me on IRC (alexpilotti). </div><div apple-content-edited="true"><br></div><div apple-content-edited="true"><blockquote type="cite"><span style="font-family: Calibri, sans-serif; font-size: 14px; ">It looks like a 'future statement' said that json will go in @ (</span><a href="https://github.com/openstack/nova/blob/master/nova/tests/hyperv/README.rst" style="font-family: Calibri, sans-serif; font-size: 14px; ">https://github.com/openstack/nova/blob/master/nova/tests/hyperv/README.rst</a><span style="font-family: Calibri, sans-serif; font-size: 14px; ">), but shouldn't that have happened to begin with?</span></blockquote><br></div><div apple-content-edited="true">The reason why we opted for serialized and gzipped pickled files is speed and size, but I definitely agree that the benefits of Json serializiation in this context, related to project management more than technical reasons, makes it a more suitable choice.</div><div apple-content-edited="true"><br></div><div apple-content-edited="true">Here's the blueprint, we'll implement it in time for G-2:</div><div apple-content-edited="true"><a href="https://blueprints.launchpad.net/nova/+spec/hyper-v-testing-serialization-improvements">https://blueprints.launchpad.net/nova/+spec/hyper-v-testing-serialization-improvements</a></div><div apple-content-edited="true"><br></div><div apple-content-edited="true">As for blocking any Hyper-V related commit until this feature is implemented, IMO this would not bring any benefit to the Nova project, since we are already committed at doing what is needed, while it would definitely hinder the Hyper-V and Windows related features in the Grizzly release. </div><div apple-content-edited="true"><br></div><div apple-content-edited="true"> </div><div apple-content-edited="true">Thanks,</div><div apple-content-edited="true"><br></div><div apple-content-edited="true"><div><div>Alessandro Pilotti</div><div apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px; ">Cloudbase Solutions | CEO<br>-------------------------------------<br>MVP <a href="http://ASP.Net">ASP.Net</a> / IIS</span></div><div apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px; ">Windows Azure Insider<br>Red Hat Certified Engineer <br>-------------------------------------</span></div></div><div><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px; "><br></span></div></div><div apple-content-edited="true">
<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br></div></span></div></span><br class="Apple-interchange-newline"></span><br class="Apple-interchange-newline">
</div>
<br><div><div>On Dec 5, 2012, at 07:14 , Sean Dague <<a href="mailto:sdague@linux.vnet.ibm.com">sdague@linux.vnet.ibm.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">On 12/04/2012 05:25 PM, Joshua Harlow wrote:<br><blockquote type="cite">Hi all,<br><br>I was just looking into some of the tests and came across some hyperv<br>stuff that I don't understand.<br><br>Possibly somehow can help me understand whats happening here.<br><br><a href="https://github.com/openstack/nova/tree/master/nova/tests/hyperv/stubs">https://github.com/openstack/nova/tree/master/nova/tests/hyperv/stubs</a><br><br>There seem to be a bunch of binary serialized objects here that contain<br>some secret hidden code used during testing (probably not harmful but<br>who am I to say when the contents aren't visible). From a little decode<br>script that I wrote to try to see whats going on here @<br><a href="http://paste.ubuntu.com/1411286/">http://paste.ubuntu.com/1411286/</a> (that’s just a decode of one of those<br>files) is it possible that in the future we can not accept such binary<br>blobs (even for testing). Not to set off a /panic/, but if binary blobs<br>are allowed in, then who is really sure that said blob isn't executing<br>some code on my CI machine (or elsewhere) that could be doing malicious<br>activities. Without clear-text source files, its pretty hard to say that<br>they aren't (yes its just for testing, but that’s besides the point).<br><br>Did anyone besides the committers peek into those files to determine<br>what is going on there (reviewers?)?<br><br>It looks like a 'future statement' said that json will go in @<br>(<a href="https://github.com/openstack/nova/blob/master/nova/tests/hyperv/README.rst">https://github.com/openstack/nova/blob/master/nova/tests/hyperv/README.rst</a>),<br>but shouldn't that have happened to begin with?<br><br>Crappy decoder script @ <a href="http://paste.ubuntu.com/1411300/">http://paste.ubuntu.com/1411300/</a><br><br>Thoughts welcome.<br></blockquote><br>I agree with you. The last nova meeting actually had the hyperv driver as a topic for 30 minutes because of this issue - <a href="http://eavesdrop.openstack.org/meetings/nova/2012/nova.2012-11-29-21.01.html">http://eavesdrop.openstack.org/meetings/nova/2012/nova.2012-11-29-21.01.html</a><br><br>I'd be fine saying that new hyper-v patches are blocked until this is fixed, but that didn't seem to be the general feeling on things.<br><br>It's also incredibly helpful if others raise the issues in the review queue as well - <a href="https://review.openstack.org/#/c/16843/">https://review.openstack.org/#/c/16843/</a>. You don't need -2 powers to raise concerns.<br><br><span class="Apple-tab-span" style="white-space:pre"> </span>-Sean<br><br>-- <br>Sean Dague<br>IBM Linux Technology Center<br>email: <a href="mailto:sdague@linux.vnet.ibm.com">sdague@linux.vnet.ibm.com</a><br>alt-email: <a href="mailto:sldague@us.ibm.com">sldague@us.ibm.com</a><br><br><br>_______________________________________________<br>OpenStack-dev mailing list<br><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></blockquote></div><br></body></html>