Status API for a rule doesn't work after Congress restarts

Bug #1507329 reported by Masahito Muroi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Fix Released
Undecided
Masahito Muroi

Bug Description

Loaded rule's info, id, name, original_str and so on, are set to different from original info persisted in DB after Congress restarts. It causes status API for a rule error.

When Policy Engine starts, it loads persisted rules in DB but doesn't put these into Rule instances Policy Engine maintains in itself. In this case, Status API returns 'could not find rule' error as a response of the API for rule even though an user calls the API with correct URL with correct parameters.

Changed in congress:
assignee: nobody → Masahito Muroi (muroi-masahito)
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/237026
Committed: https://git.openstack.org/cgit/openstack/congress/commit/?id=19dd02831ec995c4bcabc256bac1b0dde5088bb5
Submitter: Jenkins
Branch: master

commit 19dd02831ec995c4bcabc256bac1b0dde5088bb5
Author: Masahito Muroi <email address hidden>
Date: Mon Oct 19 23:48:15 2015 +0900

    Sets loaded rule info to Rule instances

    Policy Engine initializes Rule instances when Congress starts
    if rules are persited in db. The bug is that Policy Engine only
    loads persited rule string but doesn't set other infomation,
    id, name and comment. This patch lets Policy Engine set the
    lacked infomations to Rule instance.

    Closes-Bug: #1507329
    Change-Id: Id8354d9bff0d5b5938406473e2f344e726bd2f3c

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/237825

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

Reviewed: https://review.openstack.org/237825
Committed: https://git.openstack.org/cgit/openstack/congress/commit/?id=04eb262cea276fae61b624ba00155d5027119ff0
Submitter: Jenkins
Branch: stable/liberty

commit 04eb262cea276fae61b624ba00155d5027119ff0
Author: Masahito Muroi <email address hidden>
Date: Mon Oct 19 23:48:15 2015 +0900

    Sets loaded rule info to Rule instances

    Policy Engine initializes Rule instances when Congress starts
    if rules are persited in db. The bug is that Policy Engine only
    loads persited rule string but doesn't set other infomation,
    id, name and comment. This patch lets Policy Engine set the
    lacked infomations to Rule instance.

    Closes-Bug: #1507329
    Change-Id: Id8354d9bff0d5b5938406473e2f344e726bd2f3c
    (cherry picked from commit 19dd02831ec995c4bcabc256bac1b0dde5088bb5)

tags: added: in-stable-liberty
Tim Hinrichs (thinrichs)
Changed in congress:
status: Fix Committed → Fix Released
Tim Hinrichs (thinrichs)
Changed in congress:
milestone: none → mitaka-1
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.