Fuyaql is broken for 9.1 release

Bug #1626965 reported by Stanislaw Bogatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Bulat Gaifullin
Mitaka
Fix Released
High
Bulat Gaifullin
Newton
Fix Committed
High
Bulat Gaifullin

Bug Description

Current 9.1 ISO (got this on 9.0 upgraded to 9.1 with proposed-2016-09-17-040338 snapshot).

For current 9.1 release we cannot debug fuyaql anymore:

[root@nailgun etc]# manage.py yaql
2016-09-23 10:33:18.535 DEBUG [7f2000e5c740] (settings) Looking for settings.yaml package config using old style __file__
2016-09-23 10:33:18.535 DEBUG [7f2000e5c740] (settings) Trying to read config file /usr/lib/python2.7/site-packages/nailgun/settings.yaml
2016-09-23 10:33:18.821 DEBUG [7f2000e5c740] (settings) Trying to read config file /etc/nailgun/settings.yaml
fuel-yaql> :show clusters
id | name | status
---------------------------
1 | test | error
fuel-yaql> :use cluster 1
/usr/lib64/python2.7/site-packages/sqlalchemy/sql/compiler.py:575: SAWarning: Can't resolve label reference '-id'; converting to text() (this warning may be suppressed after 10 occurrences)
  util.ellipses_string(element.element))
fuel-yaql> :show tasks
id | status
------------------
6 | error
fuel-yaql> :use task1 6
fuel-yaql> :use task2 6
fuel-yaql> :show nodes
uid | status | roles
-----------------------------
Unexpected error: 'uid'
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/nailgun/fuyaql/fuyaql.py", line 324, in run_loop
    r = self.execute_command(command)
  File "/usr/lib/python2.7/site-packages/nailgun/fuyaql/fuyaql.py", line 305, in execute_command
    return f(*args)
  File "/usr/lib/python2.7/site-packages/nailgun/fuyaql/fuyaql.py", line 233, in show_nodes
    lambda x: node_ids.index(x['uid'])
  File "/usr/lib/python2.7/site-packages/nailgun/fuyaql/fuyaql.py", line 380, in print_list
    print('*' * (get_selected_index(column) + 1), end=' ')
  File "/usr/lib/python2.7/site-packages/nailgun/fuyaql/fuyaql.py", line 233, in <lambda>
    lambda x: node_ids.index(x['uid'])
KeyError: 'uid'
fuel-yaql>

It lead us to inability of troubleshooting problems related to YAQL extensions in cluster.

How to reproduce:
1. Install Fuel 9.1 master node
2. Create cluster
3. Add nodes to cluster
4. Deploy
5. Try to get nodes list from fuyaql cli (called by 'manage.py yaql' on master node)

Expected: it works
Actual: it fails with python traceback

Tags: area-python
tags: added: area-python
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/375384

Changed in fuel:
assignee: nobody → Bulat Gaifullin (bgaifullin)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/375385

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/375384
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=c11cdf6730b656b9c72834392782b03778597457
Submitter: Jenkins
Branch: master

commit c11cdf6730b656b9c72834392782b03778597457
Author: Bulat Gaifullin <email address hidden>
Date: Fri Sep 23 13:41:32 2016 +0300

    Fixed fuyaql after spliting common and node part of deployment info

    Change-Id: I98055b786eacda504df0c99f974ee0aba37a3b4d
    Closes-Bug: 1626965

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0rc1

This issue was fixed in the openstack/fuel-web 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/375385
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=bccde34259fd961f77d0713679e6c16ed138acfa
Submitter: Jenkins
Branch: stable/mitaka

commit bccde34259fd961f77d0713679e6c16ed138acfa
Author: Bulat Gaifullin <email address hidden>
Date: Fri Sep 23 13:41:32 2016 +0300

    Fixed fuyaql after spliting common and node part of deployment info

    Change-Id: I98055b786eacda504df0c99f974ee0aba37a3b4d
    Closes-Bug: 1626965

tags: added: on-verification
Revision history for this message
Dmitry Belyaninov (dbelyaninov) wrote :

Waiting for #1646475

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0

This issue was fixed in the openstack/fuel-web 10.0.0 release.

Revision history for this message
Dmitry Belyaninov (dbelyaninov) wrote :

Verified on snapshot #601

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.