congress

'table list <policy>' will return builtin tablenames such as 'equal'

Bug #1489313 reported by Yingxin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Fix Released
Undecided
Yingxin
congress liberty-rc2

Bug Description

Rule.tablenames() should not return builtin tablenames:
1. There is no row in builtin tables, so they should not be exposed to users when listing tables.
2. It is meaningless to subscribe another theory's builtin tables because there is no row inside.

Yingxin (cyx1231st)
Changed in congress:
assignee: nobody → Yingxin (cyx1231st)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to congress (master)

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

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

Reviewed: https://review.openstack.org/217569
Committed: https://git.openstack.org/cgit/openstack/congress/commit/?id=c121cb5f70d91be971496deb228bef68bdfd24a2
Submitter: Jenkins
Branch: master

commit c121cb5f70d91be971496deb228bef68bdfd24a2
Author: Yingxin Cheng <email address hidden>
Date: Thu Aug 27 15:38:29 2015 +0800

    Rule.tablenames() should not return builtin tables

    1. There is no row in builtin tables, so they should not be exposed to users in
    the first place when listing policy tables.
    2. It is also meaningless to track builtin table changes nor to subscribe
    builtin tables because there are no rows inside.
    3. It is meaningless to calculate references to builtin tables in
    agnostic.DseRuntime.delete_policy()

    Change-Id: Idcfbe719953fb5a6bf951afdc14d028665e26a43
    Closes-Bug: #1489313

Changed in congress:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to congress (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/224409

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to congress (stable/liberty)

Reviewed: https://review.openstack.org/224409
Committed: https://git.openstack.org/cgit/openstack/congress/commit/?id=21582e863515cce73e0ee4cc564c46d5d379abb8
Submitter: Jenkins
Branch: stable/liberty

commit 21582e863515cce73e0ee4cc564c46d5d379abb8
Author: Yingxin Cheng <email address hidden>
Date: Thu Aug 27 15:38:29 2015 +0800

    Rule.tablenames() should not return builtin tables

    1. There is no row in builtin tables, so they should not be exposed to users in
    the first place when listing policy tables.
    2. It is also meaningless to track builtin table changes nor to subscribe
    builtin tables because there are no rows inside.
    3. It is meaningless to calculate references to builtin tables in
    agnostic.DseRuntime.delete_policy()

    Change-Id: Idcfbe719953fb5a6bf951afdc14d028665e26a43
    Closes-Bug: #1489313
    (cherry picked from commit c121cb5f70d91be971496deb228bef68bdfd24a2)

tags: added: in-stable-liberty
Tim Hinrichs (thinrichs)
Changed in congress:
milestone: none → liberty-rc2
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/congress 2.0.0

This issue was fixed in the openstack/congress 2.0.0 release.

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

Other bug subscribers

Remote bug watches

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