<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">+1 to merging the CLI part, if all our comments there are filed as High priority bugs and then fixed ASAP<div class=""><br class=""></div><div class="">- romcheg<br class=""><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">24 лип. 2015 о 07:58 Mike Scherbakov <<a href="mailto:mscherbakov@mirantis.com" class="">mscherbakov@mirantis.com</a>> написав(ла):</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Colleagues,<div class="">it sounds like we need to complete what was requested by Julia here (and it would take about a day as I understand), plus Andrey's request (which seems to be very important for partner story and flexibility), plus additional pieces which turned into bugs [1].</div><div class=""><br class=""></div><div class="">I'd like to hear opinion from fuel-web cores on this. I don't think we can do all of what is requested.</div><div class=""><br class=""></div><div class="">[1] <span style="line-height:1.5;font-size:13.1999998092651px" class=""><a href="https://bugs.launchpad.net/fuel/+bugs?field.tag=feature-node-labels-cli" class="">https://bugs.launchpad.net/fuel/+bugs?field.tag=feature-node-labels-cli</a></span></div>







</div><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Thu, Jul 23, 2015 at 6:13 PM Andrey Danin <<a href="mailto:adanin@mirantis.com" class="">adanin@mirantis.com</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class=""><div class=""><div class=""><div class="">Hi, folks.<br class=""><br class=""></div>I understand it may be not a good time but I want to make a proposal regarding this feature.<br class=""></div>The feature may be extremely useful for plugin developers if these labels would be serialized into astute.yaml. They may be used by plugin tasks to do node-specific modifications. Let me provide some examples:<br class="">* For Xen integration we need to provide unique Xen Server credentials for each Compute node. But with current architecture we don't have any customizable per-node parameters.<br class="">* It may be possible to use special labels to override global values (i.e. libvirt_type, thus implementing BP <a href="https://blueprints.launchpad.net/fuel/+spec/auto-virt-type" target="_blank" class="">https://blueprints.launchpad.net/fuel/+spec/auto-virt-type</a>).<br class="">* Another case may be the fencing. A user may put IPMI credentials into labels.<br class=""></div><div class="">And there are more cases like that.<br class=""></div><div class=""><br class=""></div><div class="">Despite the original spec doesn't have this idea I propose to implement that. Moreover, I've already did it. Here are my two commits with a spec update [0] and an implementation[1]. They are pretty simple.<br class=""><br class="">[0] <a href="https://review.openstack.org/#/c/205105/" target="_blank" class="">https://review.openstack.org/#/c/205105/</a><br class="">[1] <a href="https://review.openstack.org/#/c/205113/" target="_blank" class="">https://review.openstack.org/#/c/205113/</a><br class=""><br class=""><br class=""></div><div class="">Please grant FFE to this feature with my additions till tomorrow evening.<br class=""></div></div><div class="gmail_extra"></div><div class="gmail_extra"><br class=""><div class="gmail_quote">On Thu, Jul 23, 2015 at 10:05 PM, Julia Aranovich <span dir="ltr" class=""><<a href="mailto:jkirnosova@mirantis.com" target="_blank" class="">jkirnosova@mirantis.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class=""><br class=""><div class="">Mike, thanks for the important points you've provided.</div><div class=""><br class=""></div><div class="">My main argument for this FFE is the following: we've already got a confirmation from SME for this patch. But also got some not critical comments at the last minute before we were going to merge it and have to handle it now. But it looks that these comments don't block the feature and we can fix it after merging a base patch.</div><div class=""><br class=""></div><div class="">We tested the patch and it matches an acceptance criteria for the feature with some not critical known issues that already converted to launchpad tickets.</div><div class=""><br class=""></div><div class="">I believe we can land it in master tomorrow with +1 from SME.</div><div class=""><br class=""></div><div class="">BTW, I see no intersection in reviewers with this patch <a href="https://review.openstack.org/#/c/204321/" target="_blank" class="">https://review.openstack.org/#/c/204321/</a>.</div><div class=""><br class=""></div><div class="">Thank you,<br class="">Julia</div><div class=""><div class=""><div class=""><br class=""></div><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Thu, Jul 23, 2015 at 9:40 PM Mike Scherbakov <<a href="mailto:mscherbakov@mirantis.com" target="_blank" class="">mscherbakov@mirantis.com</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="">-1 <div class="">My concerns are the following:</div><div class=""><ol class=""><li class="">This feature is of a High priority, not Essential [1]</li><li class="">We already had to give exception for flexible networking CLI part [2], as it is essential one. So basically that means we have a conflict of focus for SMEs in the area.</li><li class="">Just by working on this, we don't spend time on bugs. Which increases risk of delivering on time with expected level of quality</li></ol><div class="">+390, -35 LOC also scare me a little bit, it's not a tiny change.</div></div><div class=""><br class=""></div><div class="">One of the possible workarounds can be, if we deliver this patch after HCF, and have an updated package of client. If someone want it in experimental mode, then the one could update client package and have this functionality.</div><div class=""><br class=""></div><div class="">If you convince me though that it can be finished by end of the week with only code reviews from SMEs (and only after flexible networking part is done), only after it I can change my mind.</div><div class=""><br class=""></div><div class="">[1] <a href="https://blueprints.launchpad.net/fuel/+spec/node-custom-attributes" target="_blank" class="">https://blueprints.launchpad.net/fuel/+spec/node-custom-attributes</a></div><div class="">[2] <a href="https://review.openstack.org/#/c/204321/" target="_blank" class="">https://review.openstack.org/#/c/204321/</a></div></div><div dir="ltr" class=""><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Thu, Jul 23, 2015 at 10:53 AM Sebastian Kalinowski <<a href="mailto:skalinowski@mirantis.com" target="_blank" class="">skalinowski@mirantis.com</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="">+1 for this FFE as it's important to have this functionality covered in CLI<br class=""></div><div class="gmail_extra"><br class=""><div class="gmail_quote">2015-07-23 19:46 GMT+02:00 Igor Kalnitsky <span dir="ltr" class=""><<a href="mailto:ikalnitsky@mirantis.com" target="_blank" class="">ikalnitsky@mirantis.com</a>></span>:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Julia,<br class="">
<br class="">
I'm ok with FF exception for CLI part. I don't think it can somehow<br class="">
decrease product quality, so as a core I'll help to land it.<br class="">
<br class="">
Thanks,<br class="">
Igor<br class="">
<span class=""><br class="">
On Thu, Jul 23, 2015 at 7:50 PM, Julia Aranovich<br class="">
<<a href="mailto:jkirnosova@mirantis.com" target="_blank" class="">jkirnosova@mirantis.com</a>> wrote:<br class="">
> Team,<br class="">
><br class="">
> I would like to request an exception from the Feature Freeze for CLI changes<br class="">
> of working with custom node labels added to fuelclient (fuel2) [1]. UI and<br class="">
> Nailgun parts of the story are already merged [2].<br class="">
><br class="">
> There CLI request is being actively reviewed, the base flow is accepted.<br class="">
> There are minimal risks here since the changes added to fuel2 version.<br class="">
><br class="">
> Please, respond if you have any questions or concerns related to this<br class="">
> request.<br class="">
><br class="">
> Thanks in advance,<br class="">
> Julia<br class="">
><br class="">
> [1] <a href="https://review.openstack.org/#/c/204524/" rel="noreferrer" target="_blank" class="">https://review.openstack.org/#/c/204524/</a><br class="">
</span>> [2] <a href="https://review.openstack.org/#/c/201472/" rel="noreferrer" target="_blank" class="">https://review.openstack.org/#/c/201472/</a><br class="">
><br class="">
> __________________________________________________________________________<br class="">
> OpenStack Development Mailing List (not for usage questions)<br class="">
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank" class="">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="">
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="">
><br class="">
<br class="">
__________________________________________________________________________<br class="">
OpenStack Development Mailing List (not for usage questions)<br class="">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank" class="">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="">
</blockquote></div><br class=""></div>
__________________________________________________________________________<br class="">
OpenStack Development Mailing List (not for usage questions)<br class="">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank" class="">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="">
</blockquote></div></div><div dir="ltr" class="">-- <br class=""></div><div dir="ltr" class="">Mike Scherbakov<br class="">#mihgen</div>
__________________________________________________________________________<br class="">
OpenStack Development Mailing List (not for usage questions)<br class="">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank" class="">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="">
</blockquote></div></div></div></div>
<br class="">__________________________________________________________________________<br class="">
OpenStack Development Mailing List (not for usage questions)<br class="">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank" class="">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="">
<br class=""></blockquote></div><br class=""><br clear="all" class=""><br class=""></div><div class="gmail_extra">-- <br class=""><div class="">Andrey Danin<br class=""><a href="mailto:adanin@mirantis.com" target="_blank" class="">adanin@mirantis.com</a><br class="">skype: gcon.monolake<br class=""></div>
</div>
__________________________________________________________________________<br class="">
OpenStack Development Mailing List (not for usage questions)<br class="">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org/?subject:unsubscribe" rel="noreferrer" target="_blank" class="">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="">
</blockquote></div><div dir="ltr" class="">-- <br class=""></div><div dir="ltr" class="">Mike Scherbakov<br class="">#mihgen</div>
__________________________________________________________________________<br class="">OpenStack Development Mailing List (not for usage questions)<br class="">Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" class="">OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<br class=""><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class=""></div></blockquote></div><br class=""></div></div></body></html>