FUEL - CLI: same description for several help commands

Bug #1465547 reported by Artem Hrechanychenko
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Fuel Python (Deprecated)
8.0.x
Fix Released
Medium
Fuel Python (Deprecated)

Bug Description

release_versions:
  2014.2.2-6.1:
    VERSION:
      api: '1.0'
      astute_sha: 7766818f079881e2dbeedb34e1f67e517ed7d479
      build_id: 2015-06-08_06-13-27
      build_number: '521'
      feature_groups:
      - mirantis
      fuel-library_sha: f43c2ae1af3b493ee0e7810eab7bb7b50c986c7d
      fuel-ostf_sha: 7c938648a246e0311d05e2372ff43ef1eb2e2761
      fuelmain_sha: bcc909ffc5dd5156ba54cae348b6a07c1b607b24
      nailgun_sha: 4340d55c19029394cd5610b0e0f56d6cb8cb661b
      openstack_version: 2014.2.2-6.1
      production: docker
      python-fuelclient_sha: 4fc55db0265bbf39c369df398b9dc7d6469ba13b
      release: '6.1'

The same descripions for help commands:

  node list - Show list of all avaliable nodes.
  task list - Show list of all avaliable nodes.

  node show - Show info about node with given id.
  task show - Show info about node with given id.

Changed in fuel:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Fuel Python Team (fuel-python)
milestone: 6.1 → 7.0
tags: added: module-client
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Alexander Gubanov (ogubanov)
status: Confirmed → In Progress
Revision history for this message
Aniket Anikhindi (aniketanikhindi) wrote :

Assigning to myself for resolution.

Changed in fuel:
assignee: Alexander Gubanov (ogubanov) → Aniket Anikhindi (aniketanikhindi)
Revision history for this message
Alexander Gubanov (ogubanov) wrote :
Revision history for this message
Aniket Anikhindi (aniketanikhindi) wrote :

Sorry I didn't see that you were already on it.

My fix covers the said issues and fixes a couple other typos too. It's up for review at:
https://review.openstack.org/#/c/198673

I also see your patch failed review for some reason.

Once again, sorry for overstepping..

Aniket

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-fuelclient (master)

Change abandoned by Alexander Gubanov (<email address hidden>) on branch: master
Review: https://review.openstack.org/198660
Reason: duplicate of https://review.openstack.org/#/c/198673

Revision history for this message
Aniket Anikhindi (aniketanikhindi) wrote :

Any review comment or progress on
https://review.openstack.org/#/c/198673

Revision history for this message
Aniket Anikhindi (aniketanikhindi) wrote :

I have uploaded a working patch set (Patch-Set-3) which has passed the builds successfully. There has been no review action on it since yesterday.. Thanks

Changed in fuel:
importance: Medium → Low
status: In Progress → Won't Fix
assignee: Aniket Anikhindi (aniketanikhindi) → Fuel Python Team (fuel-python)
Revision history for this message
Nastya Urlapova (aurlapova) wrote :

Sergey, issue that is related to UX cannot be in Low priority.

Changed in fuel:
importance: Low → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-fuelclient (master)

Reviewed: https://review.openstack.org/198673
Committed: https://git.openstack.org/cgit/stackforge/python-fuelclient/commit/?id=7857474bc2d0036704110e78b42021d7957705fd
Submitter: Jenkins
Branch: master

commit 7857474bc2d0036704110e78b42021d7957705fd
Author: Aniket Anikhindi <email address hidden>
Date: Fri Jul 3 03:55:17 2015 +0530

    Fix description for "fuel2 --help" commands

    FUEL-CLI: Fixed issue wherein several help commands had same description.
    Also fixed typos in the messages in the said files.

    Change-Id: Ic2437106ad6d30c0562e3420b7eaf4936e5c093c
    DocImpact:
    Closes-Bug: #1465547

Changed in fuel:
status: Won't Fix → Fix Committed
Revision history for this message
Alexander Gubanov (ogubanov) wrote :

Verified on MOS 7.0 (build 257) - fixed!
Proof:

[root@nailgun ~]# fuel2 --help | grep -P '(node|task) (list|show) '
DEPRECATION WARNING: /etc/fuel/client/config.yaml exists and will be used as the source for settings. This behavior is deprecated. Please specify the path to your custom settings file in the FUELCLIENT_CUSTOM_SETTINGS environment variable.
  node list Show list of all available nodes.
  node show Show info about node with given id.
  task list Show list of all available tasks.
  task show Show info about task with given id.

Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.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.