missing validation on val-col for lists

Bug #1399493 reported by Aaron Rosen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Fix Released
Undecided
Aaron Rosen

Bug Description

2014-12-04 16:29:21.696 DEBUG congress.service [-] 'val-col' from (pid=22225) wrapper /opt/stack/congress/congress/service.py:34
2014-12-04 16:29:21.696 TRACE congress.service Traceback (most recent call last):
2014-12-04 16:29:21.696 TRACE congress.service File "/opt/stack/congress/congress/service.py", line 32, in wrapper
2014-12-04 16:29:21.696 TRACE congress.service return f(*args, **kw)
2014-12-04 16:29:21.696 TRACE congress.service File "/opt/stack/congress/congress/service.py", line 56, in congress_app_factory
2014-12-04 16:29:21.696 TRACE congress.service cage = harness.create(root_path, policy_path, data_path)
2014-12-04 16:29:21.696 TRACE congress.service File "/opt/stack/congress/congress/harness.py", line 164, in create
2014-12-04 16:29:21.696 TRACE congress.service engine.set_schema(name, service.get_schema())
2014-12-04 16:29:21.696 TRACE congress.service File "/opt/stack/congress/congress/datasources/datasource_driver.py", line 422, in get_schema
2014-12-04 16:29:21.696 TRACE congress.service _get_schema(trans, all_schemas)
2014-12-04 16:29:21.696 TRACE congress.service File "/opt/stack/congress/congress/datasources/datasource_driver.py", line 399, in _get_schema
2014-12-04 16:29:21.696 TRACE congress.service value_col = translator[self.VAL_COL]
2014-12-04 16:29:21.696 TRACE congress.service KeyError: 'val-col'
2014-12-04 16:29:21.696 TRACE congress.service
2014-12-04 16:29:21.697 CRITICAL congress.service [-] 'val-col'

Aaron Rosen (arosen)
Changed in congress:
assignee: nobody → Aaron Rosen (arosen)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to congress (master)

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

commit fb5173d234262db5d5ecdfef2bff433045b74d02
Author: Aaron Rosen <email address hidden>
Date: Thu Dec 4 16:31:50 2014 -0800

    Add validation for val-col LIST

    val-col is a required field for lists so we should validate that it is present.

    Closes-bug: 1399493

    Change-Id: I317f97b41deca2d5da9fc0690a8f81117afb62af

Changed in congress:
status: New → Fix Committed
Tim Hinrichs (thinrichs)
Changed in congress:
milestone: none → kilo-3
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.