precommit of securitygroup/rule is missing necessary info

Bug #1698284 reported by Isaku Yamahata on 2017-06-16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Isaku Yamahata
Isaku Yamahata

Bug Description
breaks mechanism driver. especially networking-odl

precommit callbacks of security group/security group rule creation are missing necessary info.
They should pass more info. For example id, egress/ingress_rule as deafult rule.
Before ovo patch, those information can retrieved from
However with ovo patch, those newly created orm object is dropped from session as
NeutronDbObject expung itself when it's created. See neutron.objects.base._detach_db_obj for db relationship.

Changed in networking-odl:
importance: Undecided → Critical
assignee: nobody → Isaku Yamahata (yamahata)
milestone: none → 11.0.0
Changed in neutron:
assignee: nobody → Isaku Yamahata (yamahata)

Fix proposed to branch: master

Changed in neutron:
status: New → In Progress
Changed in networking-odl:
status: New → In Progress

Change abandoned by Isaku Yamahata (<email address hidden>) on branch: master

Changed in neutron:
importance: Undecided → High

Submitter: Jenkins
Branch: master

commit 9e208e90942f9553fde4d1f32b1aa462349dcbda
Author: Isaku Yamahata <email address hidden>
Date: Thu Jun 15 14:14:01 2017 -0700

    unbreak gate

    This patch is squashed patches of the following patch.
    There are several issues to break networking-odl gate,
    which requires several patches. In order to unbreak gate,
    those patches need to be squashed into one.

    devstack: bug workaround 1698129
    caused jobs breakage.
    Define AGENT_BINARY as bug work around.
    Once it's fixed, revert this patch.

    Related-bug: #1698129

    unit: make dummy uuid follow uuid format

    test_mechanism_odl_v2 test_sg fails due to update of neutron security
    group ovo because dummy uuid doesn't follow uuid format.
    Make those dummy uuid's follow uuid format.
    This also reveals the hidden bug of test_l3_odl. this patch fixes it.

    Closes-bug: #1698221
    unbreak test_update/delete_floating_ip

    test_update/delete_floating_ip tests floating ip, not port.
    So floating ip needs to be mocked. not port.

    unbreak security group with ovo

    [1] breaks security group. unbreak it.

    Closes-bug: #1698284
    Depends-On: I6f49f25eb2ad16221357024f45a6bb6175d5cd55

    skip functional test_security_group_update

    Due to security group update issue,
    functional test of test_security_group_update doesn't pass.
    skip it until the patch [1] is merged.
    Once the patch is merged, revert this patch.


    Change-Id: Ifad6929f7700dd3588bd43a45c978b0e071bf385

Changed in networking-odl:
status: In Progress → Fix Released

Change abandoned by Isaku Yamahata (<email address hidden>) on branch: master

This issue was fixed in the openstack/networking-odl development milestone.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers