Intermittent UT failure due to non-specific SG check

Bug #1487942 reported by Sumit Naiksatam
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Group Based Policy
Fix Released
High
Sumit Naiksatam

Bug Description

The commit 09379c3ff72588841e1cde278965007c531cbe4d added a few egress SG rules by default. However, the following line:
https://github.com/stackforge/group-based-policy/blob/09379c3ff72588841e1cde278965007c531cbe4d/gbpservice/neutron/tests/unit/services/grouppolicy/test_resource_mapping.py#L1680

only checks for the first rule. This check passes sometime and fails at other times.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to group-based-policy (master)

Fix proposed to branch: master
Review: https://review.openstack.org/216080

Changed in group-based-policy:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to group-based-policy (master)

Reviewed: https://review.openstack.org/216080
Committed: https://git.openstack.org/cgit/stackforge/group-based-policy/commit/?id=bdeae71869ab50dcaef3d2b01e7339dd9268436d
Submitter: Jenkins
Branch: master

commit bdeae71869ab50dcaef3d2b01e7339dd9268436d
Author: Sumit Naiksatam <email address hidden>
Date: Sun Aug 23 19:29:10 2015 -0700

    Fix policy rule update UT to check specific SG rule

    This fixes a UT which was updated in:
    09379c3ff72588841e1cde278965007c531cbe4d

    Note: This patch does not need to be backported since
    the fix was already included in the previously backported
    commit.

    Change-Id: I0d24047711505016cac5e095cc09c15c270f0df8
    Closes-bug: 1487942

Changed in group-based-policy:
status: In Progress → Fix Committed
Changed in group-based-policy:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.