head-only rule fails to make policy table
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
congress |
Fix Released
|
Medium
|
Unassigned |
Bug Description
(openstack) congress policy create new
+------
| Field | Value |
+------
| abbreviation | new |
| description | |
| id | 71b75b39-
| kind | nonrecursive |
| name | new |
| owner_id | user |
+------
(openstack) congress policy rule create new 'p(0)'
+------
| Field | Value |
+------
| comment | |
| id | 2ee165e3-
| name | None |
| rule | p(0) |
+------
(openstack) congress policy row list new p
Not Found::table 'p' doesn't exist (HTTP 404) (Request-ID: req-7065d85c-
(openstack) congress policy rule create new 'p(x) :- q(x)'
+------
| Field | Value |
+------
| comment | |
| id | d8e1fca3-
| name | None |
| rule | p(x) :- |
| | q(x) |
+------
(openstack) congress policy row list new p
+------+
| Col0 |
+------+
| 0 |
+------+
(openstack) congress policy rule delete new d8e1fca3-
(openstack) congress policy rule list new
// ID: 2ee165e3-
// Name: None
p(0)
(openstack) congress policy row list new p
Not Found::table 'p' doesn't exist (HTTP 404) (Request-ID: req-cdc4c39a-
Reviewed: https:/ /review. openstack. org/427992 /git.openstack. org/cgit/ openstack/ congress/ commit/ ?id=cfecb039b9b 20aaa6bdd775cf9 821f058eb99c37
Committed: https:/
Submitter: Jenkins
Branch: master
commit cfecb039b9b20aa a6bdd775cf9821f 058eb99c37
Author: Eric K <email address hidden>
Date: Wed Feb 1 18:37:11 2017 -0800
Add tables contributed by atomic rules
datalog/* fail to consider tables contributed by atomic rules,
leading to the problem that user cannot list rows from the
policy table.
This change adds the tables contributed by atomic rules. datalog/ *.
This patch is a quick fix intended to have minimal risk
of disrupting other things (for example by introducing a new
optional argument for new behavior). More permanent fix is
needed that respects the abstractions intended by
congress/
Closes-Bug: 1658263
Change-Id: Ie0fd108ebcae01 b1f136ff7e859e2 5b5253a788d