Tool generates commands for a wrong Cinder version

Bug #1673425 reported by Olena Logvinova
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstackdocstheme
Fix Released
Medium
KATO Tomoyuki

Bug Description

*Steps to reproduce:*

1. Use the following instructions to set up openstack-doc-tools:
   https://docs.openstack.org/contributor-guide/doc-tools/cli-reference.html
2. Generate an updated config file for **Cinder** using openstack-doc-tools. See output:
   https://review.openstack.org/#/c/445296.
3. Verify the version python-cinderclient that this file was generated for:
   https://review.openstack.org/#/c/445296/1/doc/cli-reference/source/cinder.rst (line 26) -
   version **2.0.1**.
4. Verify that the `manageable-list` command is deleted (line 194).
5. Verify the shell.py file of python-cinderclient for v2:
   https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/shell.py#L2500

**Actual result:**

The `manageable-list` command is present in the shell.py file of python-cinderclient for v2 although it was deleted by the openstack-doc-tools while generating new content.

**Expected result:**

The openstack-doc-tools generates updated content for the corresponding version of the python-clients correctly.

Revision history for this message
neilvixied (neilvixied) wrote :
Download full text (4.7 KiB)

Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech consultants #wetechconsultants {wetechconsultants} [wetechconsultants] "wetechconsultants" |wetechconsultants|||Wetechconsultants we tech ...

Read more...

Chason Chan (chen-xing)
description: updated
Changed in openstack-doc-tools:
status: New → Confirmed
Brian Moss (bmoss)
Changed in openstack-doc-tools:
importance: Undecided → Critical
importance: Critical → Medium
Revision history for this message
KATO Tomoyuki (kato-tomoyuki) wrote :

hmmm, we need to handle the latest Block Storage API microversion.

Changed in openstack-doc-tools:
assignee: nobody → KATO Tomoyuki (kato-tomoyuki)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-doc-tools (master)

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

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

Reviewed: https://review.openstack.org/448145
Committed: https://git.openstack.org/cgit/openstack/openstack-doc-tools/commit/?id=89100368b50e9377bf4e587c091fcc03609f8fe0
Submitter: Jenkins
Branch: master

commit 89100368b50e9377bf4e587c091fcc03609f8fe0
Author: KATO Tomoyuki <email address hidden>
Date: Wed Mar 22 00:05:07 2017 +0900

    [cli-ref] add API version for cinder explicitly

    Python-cinderclient does not use the latest Block Storage API.
    As work around, we need to set OS_VOLUME_API_VERSION environment
    variable explicitly so that we can generate all commands.

    Change-Id: I5de94f1fd5e8a4068cb7209062588eb6ade73fc8
    Closes-Bug: #1673425

Changed in openstack-doc-tools:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-doc-tools 1.6.0

This issue was fixed in the openstack/openstack-doc-tools 1.6.0 release.

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.