<div dir="ltr">I've pinged you on IRC to this effect but this has broken the stable branches which now have unresolvable dependencies on the service whose name has changed.<div><br></div><div><div><br></div><div>Error: Could not find resource 'Keystone_endpoint[RegionOne/glance]' for relationship on 'Service[glance-api]' on node openstack-puppet-test.openstacklocal</div><div>Error: Could not find resource 'Keystone_endpoint[RegionOne/glance]' for relationship on 'Service[glance-api]' on node openstack-puppet-test.openstacklocal</div></div><div><br></div><div>This break is specifically in glance, here:</div><div><br></div><div>manifests/keystone/auth.pp</div><div><br></div><div><div>  if $configure_endpoint {</div><div>    Keystone_endpoint["${region}/${real_service_name}"]  ~> Service <| name == 'glance-api' |></div><div>    Keystone_endpoint["${region}/${real_service_name}"] -> Glance_image<||></div><div>  }</div></div><div><br></div><div>I have not checked the other modules. I will be around for reviews on this if you ping me via email.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 22, 2015 at 1:42 PM, Matt Fischer <span dir="ltr"><<a href="mailto:matt@mattfischer.com" target="_blank">matt@mattfischer.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">Thanks Emilien,<div><br></div><div>This is what I was mentioning to you on IRC last week as a must fix for Mitaka. I'd like to also backport this to Liberty once it lands.</div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Dec 21, 2015 at 10:48 AM, Emilien Macchi <span dir="ltr"><<a href="mailto:emilien@redhat.com" target="_blank">emilien@redhat.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">Hello,<br>
<br>
I just reported [1] which affects puppet-keystone but also *all* modules.<br>
Since [2], you now have a lot of warnings about the new way to declare<br>
keystone_endpoint resource.<br>
<br>
This is not really acceptable and provides a poor end-user experience to<br>
have (by default) a lot of warnings.<br>
<br>
The patch that will fix it in puppet-keystone is [3] (please review it).<br>
To fix all other modules, we need to update unit tests and sometimes<br>
keystone/auth.pp in the module. It will requires a Depends-On the<br>
puppet-keystone patch, which means puppet-keystone patch will fail<br>
integration tests (circular dependency). Ex with [4] (puppet-glance)<br>
<br>
So here is the plan:<br>
* let's review [3] but do not merge it.<br>
* let's review [4] and other that will follow (on same Gerrit topic).<br>
* Once all patches have been submitted, I'll send a patch to<br>
puppet-openstack-integration with Depends-On of all other patches and<br>
see Integration testing, so we don't break our CI.<br>
<br>
You can follow all this work on the "endpoint/warnings" Gerrit topic [5].<br>
<br>
Any other suggestion is welcome,<br>
Please review,<br>
<br>
[1] <a href="https://bugs.launchpad.net/puppet-keystone/+bug/1528308" rel="noreferrer" target="_blank">https://bugs.launchpad.net/puppet-keystone/+bug/1528308</a><br>
[2]<br>
<a href="http://git.openstack.org/cgit/openstack/puppet-keystone/commit/?id=0a4e06abb0f5b3f324464ff5219d2885816311ce" rel="noreferrer" target="_blank">http://git.openstack.org/cgit/openstack/puppet-keystone/commit/?id=0a4e06abb0f5b3f324464ff5219d2885816311ce</a><br>
[3] <a href="https://review.openstack.org/#/c/259996/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/259996/</a><br>
[4] <a href="https://review.openstack.org/#/c/260044/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/260044/</a><br>
[5] <a href="https://review.openstack.org/#/q/topic:endpoint/warnings" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/topic:endpoint/warnings</a><br>
<span><font color="#888888">--<br>
Emilien Macchi<br>
<br>
</font></span><br></div></div><span class="">__________________________________________________________________________<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></span></blockquote></div><br></div>
</blockquote></div><br></div>