<div dir="ltr">I think the real problem here is that in Nova there are bug fixes that are tiny and very important to a particular subset of the user population and yet have been around for well over a month without getting a single core review.<div>
<br></div><div>Take for example <a href="https://review.openstack.org/#/c/40298/" target="_blank">https://review.openstack.org/#/c/40298/</a> , which fixes an important snapshot bug for the vmwareapi driver. This was posted well over a month ago on August 5th. It is a solid patch, is 54 new/changed lines including unit test enhancements. The commit message clearly shows which tempest tests it fixes. It has been reviewed by many vmware reviewers with +1s for a long time, but the patch just keeps having to be rebased as it sits waiting for core reviewer attention. </div>
<div><br></div><div>To me, the high-level take away is that it is hard to get new contributors excited about working on Nova when their well-written and well-targeted bug fixes just sit there, getting no feedback and not moving closer to merging. The bug above was the developer's first patch to OpenStack and while he hasn't complained a bit, I think the experience is far from the community behavior that we need to encourages new, high-quality contributors from diverse sources. For Nova to succeed in its goals of being a platform agnostic cloud layer, I think this is something we need a community strategy to address and I'd love to see it as part of the discussion put forward by those people nominating themselves as PTL. </div>
<div><br></div><div>Dan</div><div><br></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 20, 2013 at 7:07 AM, Gary Kotton <span dir="ltr"><<a href="mailto:gkotton@vmware.com" target="_blank">gkotton@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
The following two patches are really important (they are really simple and<br>
have been around since beginning of August - they are rebased every couple<br>
of days):-<br>
- <a href="https://review.openstack.org/#/c/40298/" target="_blank">https://review.openstack.org/#/c/40298/</a> - Tempest snapshot fails<br>
- <a href="https://review.openstack.org/#/c/43994/" target="_blank">https://review.openstack.org/#/c/43994/</a> - Disk copy fails<br>
There are a number of patches that we need to base on top of these that<br>
are high/critical (depending on how one looks at it). They are namely:<br>
- <a href="https://review.openstack.org/#/c/46730/" target="_blank">https://review.openstack.org/#/c/46730/</a> - flavor root disk sizes are not<br>
honored<br>
- <a href="https://review.openstack.org/#/c/47503/" target="_blank">https://review.openstack.org/#/c/47503/</a> - disabling linked clone and<br>
cacheing of images<br>
- <a href="https://review.openstack.org/#/c/46231/" target="_blank">https://review.openstack.org/#/c/46231/</a> - VM resize<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Gary<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 9/19/13 11:39 PM, "Shawn Hartsock" <<a href="mailto:hartsocks@vmware.com">hartsocks@vmware.com</a>> wrote:<br>
<br>
>Greetings stackers!<br>
><br>
>A quick mid-week update on the patches we're tracking for Havana-rc1.<br>
>There was a bug in my vote counting code that I use to query votes. Some<br>
>of the older patches were getting their votes counted wrong. Tracking the<br>
>"age" of a submitted patchset (number of days since a patchset was<br>
>posted) and the revision number helps spot these problems. I try to<br>
>validate these reports by hand, but I do miss things on occasion. Let me<br>
>know if I need to add or edit something.<br>
><br>
>Ordered by priority:<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1223709" target="_blank">https://bugs.launchpad.net/bugs/1223709</a><br>
><a href="https://review.openstack.org/46027" target="_blank">https://review.openstack.org/46027</a> readiness:ready for core<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1216510" target="_blank">https://bugs.launchpad.net/bugs/1216510</a><br>
><a href="https://review.openstack.org/43616" target="_blank">https://review.openstack.org/43616</a> readiness:needs one more +2/approval<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1226211" target="_blank">https://bugs.launchpad.net/bugs/1226211</a><br>
><a href="https://review.openstack.org/46789" target="_blank">https://review.openstack.org/46789</a> readiness:ready for core<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1217541" target="_blank">https://bugs.launchpad.net/bugs/1217541</a><br>
><a href="https://review.openstack.org/43621" target="_blank">https://review.openstack.org/43621</a> readiness:needs review<br>
>* High/High <a href="https://bugs.launchpad.net/bugs/1187853" target="_blank">https://bugs.launchpad.net/bugs/1187853</a><br>
><a href="https://review.openstack.org/45349" target="_blank">https://review.openstack.org/45349</a> readiness:ready for core<br>
>* Medium/High <a href="https://bugs.launchpad.net/bugs/1190515" target="_blank">https://bugs.launchpad.net/bugs/1190515</a><br>
><a href="https://review.openstack.org/33100" target="_blank">https://review.openstack.org/33100</a> readiness:ready for core<br>
>* High <a href="https://bugs.launchpad.net/bugs/1184807" target="_blank">https://bugs.launchpad.net/bugs/1184807</a><br>
><a href="https://review.openstack.org/40298" target="_blank">https://review.openstack.org/40298</a> readiness:ready for core<br>
>* High <a href="https://bugs.launchpad.net/bugs/1214850" target="_blank">https://bugs.launchpad.net/bugs/1214850</a><br>
><a href="https://review.openstack.org/43270" target="_blank">https://review.openstack.org/43270</a> readiness:needs review<br>
>* High <a href="https://bugs.launchpad.net/bugs/1226052" target="_blank">https://bugs.launchpad.net/bugs/1226052</a><br>
><a href="https://review.openstack.org/46730" target="_blank">https://review.openstack.org/46730</a> readiness:needs review<br>
>* High <a href="https://bugs.launchpad.net/bugs/1226826" target="_blank">https://bugs.launchpad.net/bugs/1226826</a><br>
><a href="https://review.openstack.org/47030" target="_blank">https://review.openstack.org/47030</a> readiness:needs review<br>
>* High <a href="https://bugs.launchpad.net/bugs/1225002" target="_blank">https://bugs.launchpad.net/bugs/1225002</a><br>
><a href="https://review.openstack.org/41977" target="_blank">https://review.openstack.org/41977</a> readiness:ready for core<br>
>* High <a href="https://bugs.launchpad.net/bugs/1194018" target="_blank">https://bugs.launchpad.net/bugs/1194018</a><br>
><a href="https://review.openstack.org/43641" target="_blank">https://review.openstack.org/43641</a> readiness:ready for core<br>
>* High <a href="https://bugs.launchpad.net/bugs/1171226" target="_blank">https://bugs.launchpad.net/bugs/1171226</a><br>
><a href="https://review.openstack.org/43994" target="_blank">https://review.openstack.org/43994</a> readiness:ready for core<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1183654" target="_blank">https://bugs.launchpad.net/bugs/1183654</a><br>
><a href="https://review.openstack.org/45203" target="_blank">https://review.openstack.org/45203</a> readiness:needs revision<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1223074" target="_blank">https://bugs.launchpad.net/bugs/1223074</a><br>
><a href="https://review.openstack.org/45864" target="_blank">https://review.openstack.org/45864</a> readiness:needs review<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1199954" target="_blank">https://bugs.launchpad.net/bugs/1199954</a><br>
><a href="https://review.openstack.org/46231" target="_blank">https://review.openstack.org/46231</a> readiness:needs review<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1222349" target="_blank">https://bugs.launchpad.net/bugs/1222349</a><br>
><a href="https://review.openstack.org/45570" target="_blank">https://review.openstack.org/45570</a> readiness:needs one more +2/approval<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1216961" target="_blank">https://bugs.launchpad.net/bugs/1216961</a><br>
><a href="https://review.openstack.org/43721" target="_blank">https://review.openstack.org/43721</a> readiness:needs one more +2/approval<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1215352" target="_blank">https://bugs.launchpad.net/bugs/1215352</a><br>
><a href="https://review.openstack.org/43268" target="_blank">https://review.openstack.org/43268</a> readiness:needs one more +2/approval<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1197041" target="_blank">https://bugs.launchpad.net/bugs/1197041</a><br>
><a href="https://review.openstack.org/43621" target="_blank">https://review.openstack.org/43621</a> readiness:needs review<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1222948" target="_blank">https://bugs.launchpad.net/bugs/1222948</a><br>
><a href="https://review.openstack.org/46400" target="_blank">https://review.openstack.org/46400</a> readiness:needs revision<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1226238" target="_blank">https://bugs.launchpad.net/bugs/1226238</a><br>
><a href="https://review.openstack.org/46824" target="_blank">https://review.openstack.org/46824</a> readiness:needs review<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1224479" target="_blank">https://bugs.launchpad.net/bugs/1224479</a><br>
><a href="https://review.openstack.org/46277" target="_blank">https://review.openstack.org/46277</a> readiness:ready for core<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1207064" target="_blank">https://bugs.launchpad.net/bugs/1207064</a><br>
><a href="https://review.openstack.org/42024" target="_blank">https://review.openstack.org/42024</a> readiness:needs revision<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1180044" target="_blank">https://bugs.launchpad.net/bugs/1180044</a><br>
><a href="https://review.openstack.org/43270" target="_blank">https://review.openstack.org/43270</a> readiness:needs review<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1226425" target="_blank">https://bugs.launchpad.net/bugs/1226425</a><br>
><a href="https://review.openstack.org/46895" target="_blank">https://review.openstack.org/46895</a> readiness:needs revision<br>
>* Low <a href="https://bugs.launchpad.net/bugs/1215958" target="_blank">https://bugs.launchpad.net/bugs/1215958</a><br>
><a href="https://review.openstack.org/43665" target="_blank">https://review.openstack.org/43665</a> readiness:needs review<br>
>* Low <a href="https://bugs.launchpad.net/bugs/1226450" target="_blank">https://bugs.launchpad.net/bugs/1226450</a><br>
><a href="https://review.openstack.org/46896" target="_blank">https://review.openstack.org/46896</a> readiness:ready for core<br>
><br>
>--------------------------------------------------------------------------<br>
>------<br>
>Ordered by fitness for review:<br>
><br>
>== needs one more +2/approval ==<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1222349" target="_blank">https://bugs.launchpad.net/bugs/1222349</a> review:<br>
><a href="https://review.openstack.org/45570" target="_blank">https://review.openstack.org/45570</a><br>
> title: 'VMware: datastore_regex is not honoured'<br>
> votes: +2:1, +1:5, -1:0, -2:0 age: 11 days revision: 4<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1216961" target="_blank">https://bugs.launchpad.net/bugs/1216961</a> review:<br>
><a href="https://review.openstack.org/43721" target="_blank">https://review.openstack.org/43721</a><br>
> title: 'VMware: exceptions for RetrievePropertiesEx incorrectly handled'<br>
> votes: +2:1, +1:5, -1:0, -2:0 age: 1 days revision: 2<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1215352" target="_blank">https://bugs.launchpad.net/bugs/1215352</a> review:<br>
><a href="https://review.openstack.org/43268" target="_blank">https://review.openstack.org/43268</a><br>
> title: 'VMware: unable to access VNC console if password is not<br>
>configured'<br>
> votes: +2:1, +1:3, -1:0, -2:0 age: 1 days revision: 12<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1216510" target="_blank">https://bugs.launchpad.net/bugs/1216510</a> review:<br>
><a href="https://review.openstack.org/43616" target="_blank">https://review.openstack.org/43616</a><br>
> title: 'VMware: exception when accessing invalid nodename'<br>
> votes: +2:1, +1:5, -1:0, -2:0 age: 3 days revision: 8<br>
><br>
>== ready for core ==<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1223709" target="_blank">https://bugs.launchpad.net/bugs/1223709</a> review:<br>
><a href="https://review.openstack.org/46027" target="_blank">https://review.openstack.org/46027</a><br>
> title: 'VMware: boot from volume exception'<br>
> votes: +2:0, +1:5, -1:0, -2:0 age: 1 days revision: 3<br>
>* High <a href="https://bugs.launchpad.net/bugs/1184807" target="_blank">https://bugs.launchpad.net/bugs/1184807</a> review:<br>
><a href="https://review.openstack.org/40298" target="_blank">https://review.openstack.org/40298</a><br>
> title: 'Snapshot failure with VMwareVCDriver'<br>
> votes: +2:0, +1:5, -1:0, -2:0 age: 1 days revision: 18<br>
>* High <a href="https://bugs.launchpad.net/bugs/1225002" target="_blank">https://bugs.launchpad.net/bugs/1225002</a> review:<br>
><a href="https://review.openstack.org/41977" target="_blank">https://review.openstack.org/41977</a><br>
> title: 'VMware: no VM connectivity when opaque network does not match<br>
>bridge id'<br>
> votes: +2:0, +1:5, -1:0, -2:0 age: 4 days revision: 5<br>
>* Low <a href="https://bugs.launchpad.net/bugs/1226450" target="_blank">https://bugs.launchpad.net/bugs/1226450</a> review:<br>
><a href="https://review.openstack.org/46896" target="_blank">https://review.openstack.org/46896</a><br>
> title: 'no image cache tests for VMware Hyper'<br>
> votes: +2:0, +1:4, -1:0, -2:0 age: 2 days revision: 1<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1224479" target="_blank">https://bugs.launchpad.net/bugs/1224479</a> review:<br>
><a href="https://review.openstack.org/46277" target="_blank">https://review.openstack.org/46277</a><br>
> title: '[vmware] InvalidURL nonnumeric port in read_write_ util with<br>
>ipv6 host'<br>
> votes: +2:0, +1:4, -1:0, -2:0 age: 7 days revision: 5<br>
>* Medium/High <a href="https://bugs.launchpad.net/bugs/1190515" target="_blank">https://bugs.launchpad.net/bugs/1190515</a> review:<br>
><a href="https://review.openstack.org/33100" target="_blank">https://review.openstack.org/33100</a><br>
> title: 'Incorrect host stats reported by VMWare VCDriver'<br>
> votes: +2:0, +1:7, -1:0, -2:0 age: 1 days revision: 19<br>
>* High <a href="https://bugs.launchpad.net/bugs/1194018" target="_blank">https://bugs.launchpad.net/bugs/1194018</a> review:<br>
><a href="https://review.openstack.org/43641" target="_blank">https://review.openstack.org/43641</a><br>
> title: 'Can't use vmware distributed virtual switch with nova'<br>
> votes: +2:0, +1:5, -1:0, -2:0 age: 25 days revision: 14<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1226211" target="_blank">https://bugs.launchpad.net/bugs/1226211</a> review:<br>
><a href="https://review.openstack.org/46789" target="_blank">https://review.openstack.org/46789</a><br>
> title: 'VMware: some API's return only data on the first cluster and not<br>
>the correct cluster'<br>
> votes: +2:0, +1:7, -1:0, -2:0 age: 3 days revision: 1<br>
>* High <a href="https://bugs.launchpad.net/bugs/1171226" target="_blank">https://bugs.launchpad.net/bugs/1171226</a> review:<br>
><a href="https://review.openstack.org/43994" target="_blank">https://review.openstack.org/43994</a><br>
> title: 'VMwareVCDriver: Sparse disk copy error on spawn'<br>
> votes: +2:0, +1:5, -1:0, -2:0 age: 1 days revision: 18<br>
>* High/High <a href="https://bugs.launchpad.net/bugs/1187853" target="_blank">https://bugs.launchpad.net/bugs/1187853</a> review:<br>
><a href="https://review.openstack.org/45349" target="_blank">https://review.openstack.org/45349</a><br>
> title: 'VMWAREAPI: Problem with starting Windows instances on ESXi 5.1'<br>
> votes: +2:0, +1:6, -1:0, -2:0 age: 14 days revision: 9<br>
><br>
>== needs review ==<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1223074" target="_blank">https://bugs.launchpad.net/bugs/1223074</a> review:<br>
><a href="https://review.openstack.org/45864" target="_blank">https://review.openstack.org/45864</a><br>
> title: 'vsphere: Instance with volume attachment stuck with DELETING<br>
>status'<br>
> votes: +2:0, +1:4, -1:0, -2:0 age: 1 days revision: 4<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1199954" target="_blank">https://bugs.launchpad.net/bugs/1199954</a> review:<br>
><a href="https://review.openstack.org/46231" target="_blank">https://review.openstack.org/46231</a><br>
> title: 'VCDriver: Failed to resize instance'<br>
> votes: +2:0, +1:3, -1:0, -2:0 age: 2 days revision: 6<br>
>* Low <a href="https://bugs.launchpad.net/bugs/1215958" target="_blank">https://bugs.launchpad.net/bugs/1215958</a> review:<br>
><a href="https://review.openstack.org/43665" target="_blank">https://review.openstack.org/43665</a><br>
> title: 'vim_util get_dynamic_property propSet need not be set - makes<br>
>vmware driver hard to diagnose'<br>
> votes: +2:0, +1:3, -1:0, -2:0 age: 11 days revision: 5<br>
>* High <a href="https://bugs.launchpad.net/bugs/1214850" target="_blank">https://bugs.launchpad.net/bugs/1214850</a> review:<br>
><a href="https://review.openstack.org/43270" target="_blank">https://review.openstack.org/43270</a><br>
> title: 'vmware driver selection of vm_folder_ref does not account for<br>
>datastore'<br>
> votes: +2:0, +1:1, -1:0, -2:0 age: 28 days revision: 1<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1197041" target="_blank">https://bugs.launchpad.net/bugs/1197041</a> review:<br>
><a href="https://review.openstack.org/43621" target="_blank">https://review.openstack.org/43621</a><br>
> title: 'nova compute crashes if you do not have any hosts in your<br>
>cluster'<br>
> votes: +2:0, +1:2, -1:0, -2:0 age: 0 days revision: 5<br>
>* High <a href="https://bugs.launchpad.net/bugs/1226052" target="_blank">https://bugs.launchpad.net/bugs/1226052</a> review:<br>
><a href="https://review.openstack.org/46730" target="_blank">https://review.openstack.org/46730</a><br>
> title: 'VMware: VM creation ignores the size of the flavor root disk'<br>
> votes: +2:0, +1:2, -1:0, -2:0 age: 0 days revision: 7<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1226238" target="_blank">https://bugs.launchpad.net/bugs/1226238</a> review:<br>
><a href="https://review.openstack.org/46824" target="_blank">https://review.openstack.org/46824</a><br>
> title: 'VMware Driver reports SwitchNotFound when network already exists'<br>
> votes: +2:0, +1:3, -1:0, -2:0 age: 3 days revision: 1<br>
>* High <a href="https://bugs.launchpad.net/bugs/1226826" target="_blank">https://bugs.launchpad.net/bugs/1226826</a> review:<br>
><a href="https://review.openstack.org/47030" target="_blank">https://review.openstack.org/47030</a><br>
> title: 'VMwareVCDriver: detaching volume fails with TypeError'<br>
> votes: +2:0, +1:4, -1:0, -2:0 age: 1 days revision: 2<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1180044" target="_blank">https://bugs.launchpad.net/bugs/1180044</a> review:<br>
><a href="https://review.openstack.org/43270" target="_blank">https://review.openstack.org/43270</a><br>
> title: 'nova failures when vCenter has multiple datacenters'<br>
> votes: +2:0, +1:1, -1:0, -2:0 age: 28 days revision: 1<br>
>* High/Critical <a href="https://bugs.launchpad.net/bugs/1217541" target="_blank">https://bugs.launchpad.net/bugs/1217541</a> review:<br>
><a href="https://review.openstack.org/43621" target="_blank">https://review.openstack.org/43621</a><br>
> title: 'vmwareapi drivers - most summary fields need not be set'<br>
> votes: +2:0, +1:2, -1:0, -2:0 age: 0 days revision: 5<br>
><br>
>== needs revision ==<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1183654" target="_blank">https://bugs.launchpad.net/bugs/1183654</a> review:<br>
><a href="https://review.openstack.org/45203" target="_blank">https://review.openstack.org/45203</a><br>
> title: 'vmware code is using deprecated API that may not handle<br>
>inventories over 100 items properly'<br>
> votes: +2:0, +1:1, -1:1, -2:0 age: 14 days revision: 2<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1222948" target="_blank">https://bugs.launchpad.net/bugs/1222948</a> review:<br>
><a href="https://review.openstack.org/46400" target="_blank">https://review.openstack.org/46400</a><br>
> title: 'VMware: booting multiple instances fails if image is not in<br>
>cache'<br>
> votes: +2:0, +1:1, -1:2, -2:0 age: 7 days revision: 6<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1207064" target="_blank">https://bugs.launchpad.net/bugs/1207064</a> review:<br>
><a href="https://review.openstack.org/42024" target="_blank">https://review.openstack.org/42024</a><br>
> title: 'VMWare : Disabling linked clone does not cache images on the<br>
>datastore'<br>
> votes: +2:0, +1:1, -1:3, -2:0 age: 1 days revision: 11<br>
>* Medium <a href="https://bugs.launchpad.net/bugs/1226425" target="_blank">https://bugs.launchpad.net/bugs/1226425</a> review:<br>
><a href="https://review.openstack.org/46895" target="_blank">https://review.openstack.org/46895</a><br>
> title: 'Problem specifying VMware network name in FlatNetworking'<br>
> votes: +2:0, +1:2, -1:1, -2:0 age: 2 days revision: 3<br>
><br>
># Shawn Hartsock<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" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>Dan Wendlandt <div>Nicira, Inc: <a href="http://www.nicira.com" target="_blank">www.nicira.com</a><br><div>twitter: danwendlandt<br>
~~~~~~~~~~~~~~~~~~~~~~~~~~~<br></div></div>
</div></div>