[nailgun] Roles assignment validation doesn't work on server-side

Bug #1286203 reported by Ivan Kolodyazhny
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Triaged
Medium
Vladimir Sharshov
Mitaka
Won't Fix
Medium
Vladimir Sharshov
Newton
Triaged
Medium
Vladimir Sharshov

Bug Description

Roles assignment validation works only on Web UI. E.g. You aren't able to assign Compute+Controller roles to a one node, only one Controller could be added on simple mode (multinode mode).

UI adds nodes to a cluster using NodeCollectionHandler (/api/nodes/). It doesn't contain any kind of roles validation. Some of needed validation is in NodeAssignmentHandler, but I'm not sure that it is using now.

Steps to reproduce:
1. Create cluster in multinode mode.
2. Add 3 controllers using REST API or by hacking UI in firebug, etc
3. Start deployment.

Actual results:
Deployment started.

Expected results:
You shouldn't be able to add 3 controllers in simple mode or deployment should not start.

Tested on latest master (ISO & Fake UI)

Ivan Kolodyazhny (e0ne)
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
Mike Scherbakov (mihgen)
Changed in fuel:
milestone: none → 5.0
Revision history for this message
Evgeniy L (rustyrobot) wrote :

Additional research is required, my opinion is we shouldn't set cluster and roles via '/api/nodes/' handler.

Changed in fuel:
status: New → Confirmed
importance: Undecided → Low
importance: Low → Medium
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Alexander Kislitsky (akislitsky)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
assignee: Alexander Kislitsky (akislitsky) → Fuel Python Team (fuel-python)
Kamil Sambor (ksambor)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Kamil Sambor (ksambor)
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 5.0 → 5.1
Kamil Sambor (ksambor)
Changed in fuel:
assignee: Kamil Sambor (ksambor) → nobody
Łukasz Oleś (loles)
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
Revision history for this message
Alexander Kislitsky (akislitsky) wrote : Re: Roles assignment validation doesn't work on server-side

We have no restrictions on number of nodes in openstack.yaml. This issue should be blueprinted and designed as feature.

Dmitry Pyzhov (dpyzhov)
Changed in fuel:
status: In Progress → Confirmed
Dima Shulyak (dshulyak)
tags: added: low-hanging-fruit
Dmitry Ilyin (idv1985)
summary: - Roles assignment validation doesn't work on server-side
+ [nailgun] Roles assignment validation doesn't work on server-side
tags: added: feature
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 5.1 → 6.0
Nikolay Markov (nmarkov)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Nikolay Markov (nmarkov)
Nikolay Markov (nmarkov)
Changed in fuel:
milestone: 6.0 → 6.1
Dmitry Pyzhov (dpyzhov)
tags: added: feature-validation
Nikolay Markov (nmarkov)
Changed in fuel:
milestone: 6.1 → 7.0
tags: added: qa-agree-7.0
Revision history for this message
Nikolay Markov (nmarkov) wrote :

Proper language parser and validator for YAML-defined checks is needed on backend.

Changed in fuel:
assignee: Nikolay Markov (nmarkov) → Fuel Python Team (fuel-python)
Revision history for this message
Andriy Popovych (popovych-andrey) wrote :
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

This is feature. It is need blueprint. Can be safely moving to 8.0

tags: added: known-issue need-bp
Changed in fuel:
status: Confirmed → Won't Fix
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
no longer affects: fuel/8.0.x
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Changed in fuel:
milestone: 8.0 → 9.0
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.