"nova --help" needs authentication causes cli-reference generation trouble

Bug #1491677 reported by Atsushi SAKAI
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
Undecided
Unassigned
python-novaclient
Fix Released
Medium
Ramaraja

Bug Description

nova --help needs authentication

$ nova --help
ERROR (CommandError): You must provide a username or user id via --os-username, --os-user-id, env[OS_USERNAME] or env[OS_USER_ID]

Would you treat same authentication level?
following command does not need authentication
nova help
nova help xxxx

Current authentication level for --help causes cli-reference generation trouble.

Revision history for this message
Atsushi SAKAI (sakaia) wrote :

Affected python-novaclient version is
2.27.0

Ramaraja (ramaraja-r)
Changed in python-novaclient:
assignee: nobody → Ramaraja (ramaraja-r)
Revision history for this message
Ramaraja (ramaraja-r) wrote :

There is a change in novaclient 2.27.0 do_help logic. unfortunately this logic will work for only 'help' argument. I think we may need to add '--help' and '-h' in addition.

I suspect this line:

https://github.com/openstack/python-novaclient/blob/2.27.0/novaclient/shell.py#L559

melanie witt (melwitt)
Changed in python-novaclient:
importance: Undecided → Medium
status: New → Confirmed
Ramaraja (ramaraja-r)
Changed in python-novaclient:
status: Confirmed → In Progress
Revision history for this message
Atsushi SAKAI (sakaia) wrote :

[FYI] still have a problem on python-novaclient 2.28.1

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to python-novaclient (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/221488

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

Reviewed: https://review.openstack.org/220919
Committed: https://git.openstack.org/cgit/openstack/python-novaclient/commit/?id=ca031f991565a3764f8c36bdd28c942ab383b4d3
Submitter: Jenkins
Branch: master

commit ca031f991565a3764f8c36bdd28c942ab383b4d3
Author: Ramaraja <email address hidden>
Date: Mon Sep 7 14:33:16 2015 +0530

    Fix nova --help needs authentication

    Added --help,-h in do_help flag so that authentication will not be prompted.

    Change-Id: Iafb766304eeda58eff0c92b87a127018820832fd
    Closes-Bug: #1491677

Changed in python-novaclient:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to python-novaclient (master)

Reviewed: https://review.openstack.org/221488
Committed: https://git.openstack.org/cgit/openstack/python-novaclient/commit/?id=d0b44893ac4fc3cda215cba20b8b9330e9fa92c3
Submitter: Jenkins
Branch: master

commit d0b44893ac4fc3cda215cba20b8b9330e9fa92c3
Author: melanie witt <email address hidden>
Date: Tue Sep 8 20:08:37 2015 +0000

    Add unit tests for different help commands

    Related-Bug: #1491677

    Change-Id: I7e6b74e60dafa0e869a5b3355793cda1cf08c389

Changed in python-novaclient:
milestone: none → 2.29.0
status: Fix Committed → Fix Released
Revision history for this message
Atsushi SAKAI (sakaia) wrote :

Currently Nova documentation is up to date.

Changed in openstack-manuals:
status: New → Confirmed
status: Confirmed → Fix Committed
Changed in openstack-manuals:
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.