Tool generates commands for a wrong Cinder version

Bug #1673425 reported by Olena Logvinova on 2017-03-16
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-doc-tools
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.

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) on 2017-03-17
description: updated
Changed in openstack-doc-tools:
status: New → Confirmed
Brian Moss (bmoss) on 2017-03-21
Changed in openstack-doc-tools:
importance: Undecided → Critical
importance: Critical → Medium
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)

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

Changed in openstack-doc-tools:
status: Confirmed → In Progress

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

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  Edit
Everyone can see this information.

Other bug subscribers