<div dir="ltr">Hi All,<div><br></div><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 13, 2016 at 12:28 PM, Michal Adamczyk <span dir="ltr"><<a href="mailto:vanditboy@gmail.com" target="_blank">vanditboy@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">On Fri, May 13, 2016 at 9:42 AM, Michal Adamczyk <span dir="ltr"><<a href="mailto:vanditboy@gmail.com" target="_blank">vanditboy@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Thu, May 12, 2016 at 2:14 PM, Michal Adamczyk <span dir="ltr"><<a href="mailto:vanditboy@gmail.com" target="_blank">vanditboy@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><span>On Wed, May 11, 2016 at 7:47 PM, Hongbin Lu <span dir="ltr"><<a href="mailto:hongbin.lu@huawei.com" target="_blank">hongbin.lu@huawei.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><br>
<br>
> -----Original Message-----<br>
> From: Emilien Macchi [mailto:<a href="mailto:emilien@redhat.com" target="_blank">emilien@redhat.com</a>]<br>
> Sent: May-11-16 9:44 AM<br>
> To: OpenStack Development Mailing List (not for usage questions)<br>
> Subject: Re: [openstack-dev] [puppet] magnum module -<br>
> fixes/improvements for Mitaka release<br>
><br>
> On Wed, May 11, 2016 at 9:22 AM, Michal Adamczyk <<a href="mailto:vanditboy@gmail.com" target="_blank">vanditboy@gmail.com</a>><br>
> wrote:<br>
> > Hi,<br>
> ><br>
> > I am working on some fixes/improvements to puppet magnum module:<br>
> > <a href="https://review.openstack.org/#/c/313293/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/313293/</a><br>
><br>
> reviewed & commented. Almost good, excellent work here!<br>
><br></div></div></blockquote><div><br></div></span><div>Thanks, another patch-set has been committed today.</div><div><div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>
> > I have found some issues while creating a bays with magnum<br>
> > (<a href="https://bugs.launchpad.net/magnum/+bug/1575524" rel="noreferrer" target="_blank">https://bugs.launchpad.net/<wbr>magnum/+bug/1575524</a>) and I still need to<br>
> > address few things:<br>
> ><br>
> >  - getting ID of the domain and user used to create trust (see my<br>
> > patch set<br>
> > 10 commit info). I have added domain class like in heat<br>
> ><br>
> <a href="https://review.openstack.org/#/c/313293/10/manifests/keystone/domain.p" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/313293/10/manifests/<wbr>keystone/domain.p</a><br>
> > p but the requirements is to get domain and user id, not a name.<br>
> >    With names module fails to create trust...<br>
> ><br>
> >   Do you know if there is simple way to get user/domain ID via<br>
> > existing keystone module?<br>
><br>
> We already had this issue in the paste, with neutron.conf that needed<br>
> the tenant id from nova service, to manage notifications.<br>
> It was a bug and it was fixed very early.<br>
> Using ID in production deployments is:<br>
> * hard to deploy, you need some magic that deploy the resource and get<br>
> the id to write it somewhere<br>
> * not flexible: everytime the resource change, the ID change and you<br>
> need to update conf<br>
><br>
> So please fix Magnum to allow to use names (or maybe it's in Keystone,<br>
> I haven't looked).<br>
> Otherwise, you'll need to write a provider that will get the ID for you,<br>
> look this example:<br>
> <a href="https://github.com/openstack/puppet-" rel="noreferrer" target="_blank">https://github.com/openstack/<wbr>puppet-</a><br>
> tempest/blob/master/lib/<wbr>puppet/provider/tempest_<wbr>glance_id_setter/openst<br>
> ack.rb<br>
<br>
</div></div>No problem from me. Please file a bug for that.</blockquote><div> </div></div></div><div>Should I rise a bug for that or it's already done? </div></div></div></div></blockquote><div> </div></div></div><div>Bug created: <a href="https://bugs.launchpad.net/puppet-magnum/+bug/1581372" target="_blank">https://bugs.launchpad.net/<wbr>puppet-magnum/+bug/1581372</a></div><div><div><div> </div></div></div></div></div></div></blockquote></div></div></div></div></div></blockquote><div>Hi,</div><div><br></div><div>As the above bug has been fixed how we can merge it to Mitaka and RDO/Ubuntu? Or we don't do it for Mitaka and leave it as it is now...</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span><br>
><br>
> >  - as magnum requires lbaas and in Mitaka v2 is supported according<br>
> to<br>
> > the docs<br>
> > <a href="http://docs.openstack.org/mitaka/networking-guide/adv-config-" rel="noreferrer" target="_blank">http://docs.openstack.org/<wbr>mitaka/networking-guide/adv-<wbr>config-</a><br>
> lbaas.htm<br>
</span>> > l we should add to neutron module or integration class directly the<br>
<span>> > following<br>
> > changes:<br>
> ><br>
> >  class { '::neutron::agents::lbaas':<br>
> >     interface_driver => $driver,<br>
> >     debug            => true,<br>
> >     enable_v1        => false,<br>
> >     enable_v2        => true,<br>
> >   }<br>
> ><br>
> >   neutron_config { 'service_providers/service_<wbr>provider':<br>
> >     value =><br>
> ><br>
> 'LOADBALANCERV2:Haproxy:<wbr>neutron_lbaas.drivers.haproxy.<wbr>plugin_driver.Hap<br>
> roxyOnHostPluginDriver:<wbr>default';<br>
> >   }<br>
> ><br>
><br>
> Good to know, we recently did some work to stabilize puppet-neutron so<br>
> we can deploy LBaaS v2, mjblack worked on it, maybe we can have a<br>
> status about it here.<br>
<br>
</span>FYI, Magnum is using LBaaS v1. There is a blueprint [1] to migrate to v2, but the blueprint is not finished yet.<br>
<br>
[1] <a href="https://blueprints.launchpad.net/magnum/+spec/magnum-lbaasv2-support" rel="noreferrer" target="_blank">https://blueprints.launchpad.<wbr>net/magnum/+spec/magnum-<wbr>lbaasv2-support</a></blockquote><div><br></div></span><div>This indicates that we have to wait till this is finished for magnum and my error:</div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px;line-height:18px">ERROR: HEAT-E99001 Service neutron is not available for resource type OS::Neutron:</span><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px;line-height:18px">:HealthMonitor, reason: Service endpoint not in service catalog.\n']</span><br></div><div>might be related to the fact I have lbaasv2 enabled, correct? Magnum won't work with Mitaka release as we don't have lbaasv1 there :/</div><div><br></div><div>So we wait for mjblack status about lbaasv2 switch for puppet-neutron.</div><span><div><br></div></span></div></div></div></blockquote></div></div></div></div></div></blockquote></div></div></div></div></div></blockquote><div>Any update from mjblack or does this topic of lbaasv2 died? I have created a bug for it  <a href="https://bugs.launchpad.net/bugs/1583024">https://bugs.launchpad.net/bugs/1583024</a> but appears that it was closed without any explanation. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span><br>
><br>
> > - add a parameter to api.pp or creates a new class with this<br>
> parameter<br>
> > to manage certificate manager entry inside [certificates] section of<br>
> > magnum.conf. Any preferences here?<br>
><br>
> Is it some entries for enabling SSL? Or related to Barbican?<br>
> If related to barbican, I suggest we take the puppet-oslo approach, and<br>
> create a Define resource with the common parameters that we'll have in<br>
> our puppet modules for barbican section.<br>
> And consume this class/define or write this code in api.pp makes sense<br>
> to me, now.<br>
<br>
</span>It is related to Barbican.<br>
<div><div><br></div></div></blockquote><div><br></div></span><div>I am adding the class "certificates" with cert_manager_type parameter for the moment and we can think about barbican resource.</div><span><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>
><br>
> ><br>
> > - should we extend glance_image resource type to contains --os-distro<br>
> > properties so we can add the fedora-atomic or core-os image via<br>
> > glance_image resource type?<br>
><br>
> Definitely yes. Once we have all of this, we might want to create a 4th<br>
> scenario in our integration CI with magnum + containers + barbican<br>
> + neutron lbaas v2, so we test the full stack.<br></div></div></blockquote><div><br></div></span><div>Yes, good idea, this will be my next quest :)</div></div></div></div></blockquote></div></div></div></div></div></blockquote></div></div><div>Quest here is done - glance client shows it a separate property for "os_distro" but in fact the image in glance_image resource type is added via openstack client so all we need here is </div><div><br></div><div> glance_image { 'fedora23-atomic':</div><div>     ensure           => present,</div><div>     container_format => 'bare',</div><div>     disk_format      => 'qcow2',</div><div>     is_public        => 'yes',</div><div>     source           => "${repourl}/Fedora-Cloud-<wbr>Atomic-23-20160420.x86_64.<wbr>qcow2",</div><div>     properties       => "os_distro='fedora-atomic'"</div><div>  }</div><div><br></div><div>So at the end glance image-show <id>:</div><div> +------------------+---------<wbr>-----------------------------+</div><div>| Property         | Value                                |</div><div>+------------------+----------<wbr>----------------------------+</div><div>| checksum         | 7466b305e32ae83e438c592e347d59<wbr>fb     |</div><div>| container_format | bare                                 |</div><div>| created_at       | 2016-05-13T10:55:59Z                 |</div><div>| disk_format      | qcow2                                |</div><div>| id               | a759c9e3-ae35-4b39-b24d-<wbr>d887f12ec305 |</div><div>| min_disk         | 0                                    |</div><div>| min_ram          | 0                                    |</div><div>| name             | fedora23-atomic                      |</div><div>| os_distro        | fedora-atomic                        |</div><div>| owner            | 0360f4288e834ab0afd3bac0f88123<wbr>e4     |</div><div>| protected        | False                                |</div><div>| s_distro         | fedora-atomic                        |</div><div>| size             | 497786368                            |</div><div>| status           | active                               |</div><div>| tags             | []                                   |</div><div>| updated_at       | 2016-05-13T11:06:12Z                 |</div><div>| virtual_size     | None                                 |</div><div>| visibility       | public                               |</div><div>+------------------+----------<wbr>----------------------------+</div><div>No extra work to be done here.</div><div><div class="h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>
> Question: how do you test Magnum, do you have Tempest tests?<br>
> I see<br>
> <a href="https://github.com/openstack/magnum/tree/master/magnum/tests/functional" rel="noreferrer" target="_blank">https://github.com/openstack/<wbr>magnum/tree/master/magnum/<wbr>tests/functional</a><br>
> /tempest_tests<br>
> kind of empty.<br>
> Our CI is running Tempest, it would be very useful for us to have<br>
> Magnum tests.<br>
></div></div></blockquote><div> </div></span><div>Hmm, well, I don't have Tempest tests. I am testing it on some old laptops/vms, wherever I can. I am just lunching baker/lint tests now to check if no errors.<br></div><div>I will think about creating them when I finish all tasks in the patch I am doing atm. </div><span><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>> Thanks for your work here,<br></div></blockquote></span><div>No - thank you for the guidance and support here, I am very happy to work on such great project! </div><span><div><br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>
> --<br>
> Emilien Macchi<br>
><br>
> ______________________________<wbr>______________________________<wbr>___________<br>
> ___<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: OpenStack-dev-<br>
> <a href="http://request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">request@lists.openstack.org?<wbr>subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></span></div><br><br clear="all"><span><div><br></div>-- <br><div>Kind regards,<br><br>Michal Adamczyk</div>
</span></div></div>
</blockquote></div></div></div><div><div><br><br clear="all"><div><br></div>-- <br><div>Kind regards,<br><br>Michal Adamczyk</div>
</div></div></div></div>
</blockquote></div></div></div><div><div class="h5"><br><br clear="all"><div><br></div>-- <br><div>Kind regards,<br><br>Michal Adamczyk</div>
</div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Kind regards,<br><br>Michal Adamczyk</div>
</div></div></div>