<div dir="ltr">Do you mean the callback event for AFTER_CREATE is missing the rules when it's for default security groups?</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 18, 2017 at 4:44 AM, Gary Kotton <span dir="ltr"><<a href="mailto:gkotton@vmware.com" target="_blank">gkotton@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
That patch looks good. We still have an issue in that the create security groups does not return the list of the default rules.<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Gary<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 6/17/17, 2:33 AM, "Isaku Yamahata" <<a href="mailto:isaku.yamahata@gmail.com">isaku.yamahata@gmail.com</a>> wrote:<br>
<br>
    It also broke networking-odl.<br>
    The patch[1] is needed to unbreak.<br>
    [1] <a href="https://review.openstack.org/#/c/448420/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/448420/</a><br>
<br>
    necessary db info is taken from context.session.new.<br>
    But with OVO, those expunge themselves with create method.<br>
    Those info needs to be passed as callback argument.<br>
<br>
    Thanks,<br>
<br>
    On Fri, Jun 16, 2017 at 01:25:28PM -0700,<br>
    Ihar Hrachyshka <<a href="mailto:ihrachys@redhat.com">ihrachys@redhat.com</a>> wrote:<br>
<br>
    > To close the loop here,<br>
    ><br>
    > - this also broke heat py3 job (<a href="https://launchpad.net/bugs/1698355" rel="noreferrer" target="_blank">https://launchpad.net/bugs/<wbr>1698355</a>)<br>
    > - we polished <a href="https://review.openstack.org/474575" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>474575</a> to fix both<br>
    > vmware-nsx and heat issues<br>
    > - I also posted a patch for oslo.serialization for the bug that<br>
    > triggered MemoryError in heat gate:<br>
    > <a href="https://review.openstack.org/475052" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>475052</a><br>
    > - the vmware-nsx adoption patch is at:<br>
    > <a href="https://review.openstack.org/#/c/474608/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/474608/</a> and @boden is working on it,<br>
    > should be ready to go in due course.<br>
    ><br>
    > Thanks and sorry for inconveniences,<br>
    > Ihar<br>
    ><br>
    > On Thu, Jun 15, 2017 at 6:17 AM, Gary Kotton <<a href="mailto:gkotton@vmware.com">gkotton@vmware.com</a>> wrote:<br>
    > > Hi,<br>
    > ><br>
    > > The commit <a href="https://review.openstack.org/284738" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>284738</a> has broken decomposed plugins<br>
    > > (those that extend security groups and rules). The reason for this is that<br>
    > > there is a extend callback that we use which expects to get a database<br>
    > > object and the aforementioned patch passes a new neutron object.<br>
    > ><br>
    > > I have posted [i] to temporarily address the issue. An alternative is to<br>
    > > revert the patch until the decomposed plugins can figure out how to<br>
    > > correctly address this.<br>
    > ><br>
    > > Thanks<br>
    > ><br>
    > > Gary<br>
    > ><br>
    > > [i] <a href="https://review.openstack.org/474575" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>474575</a><br>
    > ><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>
    > ><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>
<br>
    --<br>
    Isaku Yamahata <<a href="mailto:isaku.yamahata@gmail.com">isaku.yamahata@gmail.com</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>
<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></div><br></div>