<div dir="ltr"><div><div>Hi,<br><br>some additional info on the problem: if I create some Hiera override for the nodes list and use node key which is hostname bond, then after node rename (rename during LCM or reset/rename/redeploy - doesn't matter) my override will create a "ghost" node in the list and will not change settings I wanted to change. So node keys in that hash should remain immutable.<br><br>Let's fix LP#1538220 and keep 'node-{uid}' simply because that's how it was working before (and does not require new patches like [0]) and we're too late in the release cycle to change keys to '{uid}'.<br><br></div>Regards,<br></div>Alex<br><br>[0] <a href="https://review.openstack.org/#/c/284046/">https://review.openstack.org/#/c/284046/</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 15, 2016 at 11:22 AM, Kyrylo Galanov <span dir="ltr"><<a href="mailto:kgalanov@mirantis.com" target="_blank">kgalanov@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-size:12.8px">Hi,</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Currently nailgun and puppet process network_metadata hash slightly different.</div><div style="font-size:12.8px">Nailgun uses short hostname as a hash key for each node, while library code assumes that key is always 'node-{uid}'. Sometimes it can result in a deployment failure.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">During code review[0] it turned out that there are two points of view, which approach is correct [1].</div><div style="font-size:12.8px">Both are one-line fixes, however it have been lasting too long with no result.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">I would like to start a discussion with a hope to close the issue shortly.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Best regards.</div><div style="font-size:12.8px">Kyrylo</div><div style="font-size:12.8px"><br></div><span style="font-size:12.8px">[0] </span><a href="https://review/" style="font-size:12.8px" target="_blank">https://review</a><span style="font-size:12.8px">.</span><span style="font-size:12.8px">openstack</span><span style="font-size:12.8px">.org/#</span><span style="font-size:12.8px">/c/284046/</span><br style="font-size:12.8px"><div style="font-size:12.8px">[1] <a href="https://bugs.launchpad.net/fuel/+bug/1538220" target="_blank">https://bugs.launchpad.net/fuel/+bug/1538220</a></div></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>