There is misleading cli output in env levels

Bug #1623907 reported by Alexandr Kostrikov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Alexander Kislitsky

Bug Description

There is a problem that hierarchy levels are show in wrong order.
Is shows that servers are first and then nodes.

Actual:
[root@nailgun tuning-box]# fuel2 config env create -l nodes,servers -i 1
+------------------+------------------------+
| Field | Value |
+------------------+------------------------+
| hierarchy_levels | [u'servers', u'nodes'] |
| id | 6 |
| components | [1] |
+------------------+------------------------+
[root@nailgun tuning-box]# fuel2 config override --env 6 --resource 1 --value '{"a": 1, "b": "144override"}' --level nodes=1 --type json --key key
ResourceOverride for key key was set
[root@nailgun tuning-box]# fuel2 config override --env 6 --resource 1 --value '{"a": 1, "b": "144override"}' --level servers=1 --type json --key key
{u'message': u"Unexpected level name 'servers'. Expected 'nodes'."}[root@nailgun tuning-box]#

Expected:
Hierarchy levels represent actual hierarchy.

Dmitry Klenov (dklenov)
Changed in fuel:
status: New → Confirmed
Changed in fuel:
milestone: 9.1 → 9.2
Revision history for this message
Alexander Kislitsky (akislitsky) wrote :

We have wrong hierarchy_levels order in environment creation output only. If you execute 'fuel2 config env show -e ENV_ID' hierarchy levels will be in right order.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tuning-box (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tuning-box (master)

Reviewed: https://review.openstack.org/370947
Committed: https://git.openstack.org/cgit/openstack/tuning-box/commit/?id=5f456cefc87b0466be6f7e039f0f855ded735594
Submitter: Jenkins
Branch: master

commit 5f456cefc87b0466be6f7e039f0f855ded735594
Author: Alexander Kislitsky <email address hidden>
Date: Thu Sep 15 18:32:18 2016 +0300

    Output of env creation fixed

    Hierarchy levels order fixed in POST environment response.

    Change-Id: Iff0b73a8515b9d15c5c531d8691225b97b6460e4
    Closes-Bug: #1623907

Changed in fuel:
status: In Progress → Fix Committed
tags: added: on-verification
Revision history for this message
Alexey. Kalashnikov (akalashnikov) wrote :

verified on 9.2
[root@nailgun /]# fuel2 config env create -l nodes,servers
+------------------+------------------------+
| Field | Value |
+------------------+------------------------+
| hierarchy_levels | [u'nodes', u'servers'] |
| id | 1 |
| components | [] |
+------------------+------------------------+

shotgun2 report:
http://paste.openstack.org/show/586692/

Changed in fuel:
status: Fix Committed → Fix Released
tags: removed: on-verification
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.