<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></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; "><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; "><blockquote type="cite">Coincidentally, I have a WIP oslo-config branch for this:<br><br> <a href="https://github.com/markmc/oslo-config/commits/deprecated-group">https://github.com/markmc/oslo-config/commits/deprecated-group</a><br><br>It's getting pretty tight for Grizzly, but I hope to push this up for<br>review today.</blockquote></div></span><br class="Apple-interchange-newline"></div><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">That's perfect! Please let me know if you can make it for Grizzly, I'd be glad to test it. I'll wait to send my patch for review in the meantime, as with yours any compatibility issue would be solved.</div><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br></div></span><blockquote type="cite">Usually, we avoid breaking existing configurations. But, in the case of<br>hyper-v, I'd be fine with you guys saying "we did't have many users in<br>Folsom, so backwards compat isn't a big concern".<br></blockquote></span><br class="Apple-interchange-newline">
</div><div apple-content-edited="true">Folsom was the first release for us (I'm not counting the pre Essex driver) with the initial Nova support. For Grizzly we're quite overwhelmed by customers response, especially after we added Quantum and another gazillion features to Nova, Cinder, our Windows Cloud-Init etc. So it's a good moment for doing some small cleanup by moving those few options to the proper place. :-) </div><div apple-content-edited="true"><br></div><div apple-content-edited="true">Thanks!</div><div apple-content-edited="true"><br></div>
<div>Alessandro</div><div><br></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; "><div><div><div><div><div><br></div></div></div></div></div></div></span></div></span></span>
</div>
<br><div><div>On Feb 15, 2013, at 13:07 , Mark McLoughlin <<a href="mailto:markmc@redhat.com">markmc@redhat.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">On Fri, 2013-02-15 at 12:22 +0200, Alessandro Pilotti wrote:<br><blockquote type="cite">Hi guys,<br><br>As discussed with russelb and during the OpenStack Hyper-V meetings, it'd be a great to move compute driver specific onfigurations from nova.conf's [DEFAULT] group to a driver specific group (e.g. [LIBVIRT], [HYPERV], etc).<br><br>We already added new options in Grizzly to the [HYPERV] group and for consistency, before closing Grizzly, we'd like to move the existing Hyper-V ones added in Folsom there as well.<br><br>This approach could be IMO applied to all the drivers (in the Havana timeframe at this point, I guess).<br>The obvious issue is that breaking backwards compatibility is not an option.<br><br>openstack.common.cfg.Opt() offers a "deprecated_name" argument, but AFAIK this applies only for options belonging to the same group:<br><a href="https://github.com/openstack/nova/blob/master/nova/openstack/common/cfg.py#L518">https://github.com/openstack/nova/blob/master/nova/openstack/common/cfg.py#L518</a><br><br>What about adding a "deprecated_group" as well or a "<group>.<option>" syntax for "deprecated_name"? This way we could handle the transition smoothly.<br></blockquote><br>Coincidentally, I have a WIP oslo-config branch for this:<br><br> <a href="https://github.com/markmc/oslo-config/commits/deprecated-group">https://github.com/markmc/oslo-config/commits/deprecated-group</a><br><br>It's getting pretty tight for Grizzly, but I hope to push this up for<br>review today.<br><br><blockquote type="cite">Getting back to the specific Hyper-V case, if deprecated options are not possible at this stage, we'd prefer to rename them anyway now. The Hyper-V support is far more complete in this release then it was in Folsom. Moving or deprecating the options in Havana will create way more confusion than doing it now.<br><br>Here are the Hyper-V config options that need to be moved to the [HYPERV] group:<br><a href="https://github.com/openstack/nova/blob/master/nova/virt/hyperv/vmops.py#L43">https://github.com/openstack/nova/blob/master/nova/virt/hyperv/vmops.py#L43</a><br>https://github.com/openstack/nova/blob/master/nova/virt/hyperv/volumeops.py#L35<br><br>Any additional ideas?<br></blockquote><br>Usually, we avoid breaking existing configurations. But, in the case of<br>hyper-v, I'd be fine with you guys saying "we did't have many users in<br>Folsom, so backwards compat isn't a big concern".<br><br>Cheers,<br>Mark.<br><br><br>_______________________________________________<br>OpenStack-dev mailing list<br><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev<br></blockquote></div><br></body></html>