Support for cross-plugin conditions

Bug #1587068 reported by Michael Polenchuk
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Confirmed
High
Fuel Toolbox
Mitaka
Won't Fix
High
Fuel Toolbox

Bug Description

Environment:
 latest 9.0 iso with ODL plugin

Steps to reproduce:
 * create new env with ODL plugin enabled
 * try to change some settings Other->OpenDaylight plugin
 * save <- failed (src: http://paste.ubuntu.com/16833847/)

Actual result:
 Gently failed with just one line in api.log
 Response code '500 Internal Server Error' for PUT /api/clusters/1/attributes from 127.0.0.1:53353
 on "condition":"settings:bgpvpn == null or settings:bgpvpn.metadata.enabled == false"
 (https://github.com/openstack/fuel-plugin-opendaylight/blob/master/environment_config.yaml#L43)

 It happens due to bgpvpn plugin absence.

Expected results:
 Settings saved w/o errors or UI showed an error message w/ restriction.

Additional information:
 https://github.com/openstack/fuel-plugin-opendaylight

Changed in fuel:
milestone: none → 9.0
assignee: nobody → Fuel Sustaining (fuel-sustaining-team)
tags: added: area-python
Changed in fuel:
importance: Undecided → High
Dmitry Klenov (dklenov)
Changed in fuel:
status: New → Confirmed
Ilya Kutukov (ikutukov)
Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Nikolas Hermanns (nikolas-hermanns)
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

This bug seems to be not related to Fuel project itself. Marking as incomplete.

Changed in fuel:
status: Confirmed → Incomplete
Revision history for this message
Arthur Svechnikov (asvechnikov) wrote :

This bug is related to Fuel. This bug occurred due to an error in the validation of cluster attributes restrictions. RestrictionBase uses `strict` by default, however, there is no other option how to check whether some plugin is installed or not, only "settings:some_plugin == null". In this case, BaseRestriction will raise the error that settings doesn't contain some_plugin entity.

Changed in fuel:
status: Incomplete → Confirmed
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

Fuel doesn't support cross-plugin dependencies at the moment. Marking this request as a feature request for the next release.

Changed in fuel:
assignee: Nikolas Hermanns (nikolas-hermanns) → Fuel Toolbox (fuel-toolbox)
milestone: 9.0 → 10.0
tags: added: feature
summary: - [nailgun] Failed with internal server error on restriction
+ Support for cross-plugin conditions
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.