StarlingX

Containers: cinder and glance clients do not display the correct version

Bug #1814573 reported by Yang Liu on 2019-02-04

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	StarlingX	Fix Released	Medium	Al Bailey

Bug Description

Brief Description
-----------------
cinder and glance clients both return version as 0.0.0

Severity
--------
Minor

Steps to Reproduce
------------------
[wrsroot@controller-0 ~(keystone_admin)]$ glance --version
0.0.0
[wrsroot@controller-0 ~(keystone_admin)]$ cinder --version
0.0.0

Expected Behavior
------------------
Actual version should be returned.
such as in stx master:
controller-0:~$ cinder --version
3.1.0

Reproducibility
---------------
Reproducible

System Configuration
--------------------
Any

Branch/Pull Time/Commit
-----------------------
f/stein as of 2019-01-30

Timestamp/Logs
--------------
Easy to reproduce

Tags:

Ghada Khalil (gkhalil) on 2019-02-04

tags:

added: stx.containers

Al Bailey (albailey1974) on 2019-02-05

Changed in starlingx:
assignee:	nobody → Al Bailey (albailey1974)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2019-02-05: Fix proposed to stx-upstream (f/stein)

Fix proposed to branch: f/stein
Review: https://review.openstack.org/635015

Revision history for this message

Al Bailey (albailey1974) wrote on 2019-02-05: Re: Containers: cinder and glance clients do not display the corrret version

This bug was introduced in the stein branch only.

Revision history for this message

Al Bailey (albailey1974) wrote on 2019-02-05:

Fix is up for review
https://review.openstack.org/#/c/635015/

Al Bailey (albailey1974) on 2019-02-05

Changed in starlingx:
status:	New → In Progress

Revision history for this message

Ghada Khalil (gkhalil) wrote on 2019-02-06:

Marking as release gating -- issue related to containers feature.

Changed in starlingx:
importance:	Undecided → Medium
tags:	added: stx.2019.05

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2019-02-21: Fix merged to stx-upstream (f/stein)

Reviewed: https://review.openstack.org/635015
Committed: https://git.openstack.org/cgit/openstack/stx-upstream/commit/?id=b6e3badac62686a14a0d837d2677feccfa1dfd70
Submitter: Zuul
Branch: f/stein

commit b6e3badac62686a14a0d837d2677feccfa1dfd70
Author: Al Bailey <email address hidden>
Date: Mon Feb 4 11:59:06 2019 -0600

Fix the version string in cinder and glance clients

cinderclient was showing 0.0.0 for cinder --version
Same problem for glance.

The pbr version needed to be set when building from
outside of a git tree.

All other clients had this set properly.

This bug was introduced when the new stein clients were
added.

The cinderclient also needed some BuildRequires updated.
These BuildRequires were for building wheels.

    Closes-Bug: 1814573
    Change-Id: I4afe783e25ab2172ae999787e6b0e3ec91f78419
    Signed-off-by: Al Bailey <email address hidden>

tags:

added: in-f-stein

Ghada Khalil (gkhalil) on 2019-02-25

Changed in starlingx:
status:	In Progress → Fix Released

Ghada Khalil (gkhalil) on 2019-02-27

summary:

- Containers: cinder and glance clients do not display the corrret version
+ Containers: cinder and glance clients do not display the correct version

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2019-03-05: Fix proposed to stx-upstream (f/centos76)

Fix proposed to branch: f/centos76
Review: https://review.openstack.org/640918

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2019-03-05: Fix merged to stx-upstream (f/centos76)

Download full text (10.7 KiB)

Reviewed: https://review.openstack.org/640918
Committed: https://git.openstack.org/cgit/openstack/stx-upstream/commit/?id=1443710d8f88509b5e31ebdb2ce302767cd229df
Submitter: Zuul
Branch: f/centos76

commit 53cfbc9f06b58dee663a8f857dc269bde610c349
Author: Don Penney <email address hidden>
Date: Fri Mar 1 23:50:56 2019 -0500

Remove openstack images from pike build

    Change-Id: Idd211511e42fa4c1290fdfa382ac579a1a8de91a
    Story: 2004751
    Task: 29788
    Signed-off-by: Don Penney <email address hidden>

commit bcce8c810f56a32710277341b0d6fe23226e7532
Author: Al Bailey <email address hidden>
Date: Thu Feb 21 09:52:23 2019 -0600

Setting the worker_rlimit_nofile minimum to 2048 for nginx

    In the docker image for mariadb-ingress if there are many cores
    the calculated value for worker_rlimit_nofile ends up being 1024
    which is too small. This change sets the min to 2048.

    Closes-Bug: 1816479
    Change-Id: I4f198b703eda61d9a9531640ec01a2770f9ec172
    Signed-off-by: Al Bailey <email address hidden>

commit db10c94d9e26e4150b2a57c5d2e5673fd97a3481
Author: Alex Kozyrev <email address hidden>
Date: Wed Feb 20 12:15:55 2019 -0500

Create Docker image for Barbican in StarlingX

    In order to provide the secure management of secrets service
    as a container in StarlingX we need to create Barbican Docker
    image and include it into StarlingX repository.

    Change-Id: I3b4483f74d233348ec49729deff11ba7776af01b
    Story: 2003108
    Task: 29579
    Signed-off-by: Alex Kozyrev <email address hidden>

commit b6e3badac62686a14a0d837d2677feccfa1dfd70
Author: Al Bailey <email address hidden>
Date: Mon Feb 4 11:59:06 2019 -0600

Fix the version string in cinder and glance clients

cinderclient was showing 0.0.0 for cinder --version
Same problem for glance.

The pbr version needed to be set when building from
outside of a git tree.

All other clients had this set properly.

This bug was introduced when the new stein clients were
added.

The cinderclient also needed some BuildRequires updated.
These BuildRequires were for building wheels.

    Closes-Bug: 1814573
    Change-Id: I4afe783e25ab2172ae999787e6b0e3ec91f78419
    Signed-off-by: Al Bailey <email address hidden>

commit cb921ff9348fec9c634ef191dba293e33088daae
Author: Gerry Kopec <email address hidden>
Date: Fri Feb 15 18:11:55 2019 -0500

Update nova helm chart to fix console addressing

    Upstream nova helm chart attempts to figure out the address for VM
    consoles by running an init container that checks for ip routes and
    addresses on a compute host. It then sets the appropriate nova config
    options in a config file which it passes to nova-compute. However this
    effectively overwrites the same config option that stx has already
    set in nova.conf via per host overrides causing us to communicate over
    the wrong network or not to connect at all.

    This fix introduces an option to enable/disable passing of this
    additional config file to nova-compute. Default upstream behaviour is
    unchange...

Reviewed:  https://review.openstack.org/640918
Committed: https://git.openstack.org/cgit/openstack/stx-upstream/commit/?id=1443710d8f88509b5e31ebdb2ce302767cd229df
Submitter: Zuul
Branch:    f/centos76

commit 53cfbc9f06b58dee663a8f857dc269bde610c349
Author: Don Penney <don.penney@windriver.com>
Date:   Fri Mar 1 23:50:56 2019 -0500

Remove openstack images from pike build
    
    Change-Id: Idd211511e42fa4c1290fdfa382ac579a1a8de91a
    Story: 2004751
    Task: 29788
    Signed-off-by: Don Penney <don.penney@windriver.com>

commit bcce8c810f56a32710277341b0d6fe23226e7532
Author: Al Bailey <Al.Bailey@windriver.com>
Date:   Thu Feb 21 09:52:23 2019 -0600

Setting the worker_rlimit_nofile minimum to 2048 for nginx
    
    In the docker image for mariadb-ingress if there are many cores
    the calculated value for worker_rlimit_nofile ends up being 1024
    which is too small.   This change sets the min to 2048.
    
    Closes-Bug: 1816479
    Change-Id: I4f198b703eda61d9a9531640ec01a2770f9ec172
    Signed-off-by: Al Bailey <Al.Bailey@windriver.com>

commit db10c94d9e26e4150b2a57c5d2e5673fd97a3481
Author: Alex Kozyrev <alex.kozyrev@windriver.com>
Date:   Wed Feb 20 12:15:55 2019 -0500

Create Docker image for Barbican in StarlingX
    
    In order to provide the secure management of secrets service
    as a container in StarlingX we need to create Barbican Docker
    image and include it into StarlingX repository.
    
    Change-Id: I3b4483f74d233348ec49729deff11ba7776af01b
    Story: 2003108
    Task: 29579
    Signed-off-by: Alex Kozyrev <alex.kozyrev@windriver.com>

commit b6e3badac62686a14a0d837d2677feccfa1dfd70
Author: Al Bailey <Al.Bailey@windriver.com>
Date:   Mon Feb 4 11:59:06 2019 -0600

Fix the version string in cinder and glance clients
    
    cinderclient was showing 0.0.0 for cinder --version
    Same problem for glance.
    
    The pbr version needed to be set when building from
    outside of a git tree.
    
    All other clients had this set properly.
    
    This bug was introduced when the new stein clients were
    added.
    
    The cinderclient also needed some BuildRequires updated.
    These BuildRequires were for building wheels.
    
    Closes-Bug: 1814573
    Change-Id: I4afe783e25ab2172ae999787e6b0e3ec91f78419
    Signed-off-by: Al Bailey <Al.Bailey@windriver.com>

commit cb921ff9348fec9c634ef191dba293e33088daae
Author: Gerry Kopec <gerry.kopec@windriver.com>
Date:   Fri Feb 15 18:11:55 2019 -0500

Update nova helm chart to fix console addressing
    
    Upstream nova helm chart attempts to figure out the address for VM
    consoles by running an init container that checks for ip routes and
    addresses on a compute host.  It then sets the appropriate nova config
    options in a config file which it passes to nova-compute.  However this
    effectively overwrites the same config option that stx has already
    set in nova.conf via per host overrides causing us to communicate over
    the wrong network or not to connect at all.
    
    This fix introduces an option to enable/disable passing of this
    additional config file to nova-compute.  Default upstream behaviour is
    unchanged, but we will disable it in our overrides.
    
    Partial-Bug: #1815490
    Change-Id: I86eb80578b23fd89b7f9643b943ee759f26a15be
    Signed-off-by: Gerry Kopec <gerry.kopec@windriver.com>

commit aceb7fecbaf103f4ed0b269e916f238881b5340f
Author: Don Penney <don.penney@windriver.com>
Date:   Fri Feb 8 12:43:50 2019 -0500

TEMPORARY: Patch openstackclient until upstream commit merged
    
    This update must be reverted once the following commit
    is merged upstream:
    https://review.openstack.org/#/c/625544/11
    
    Until then, this update includes it as a patch against the
    python-openstackclient source.
    
    Change-Id: I527f889e797259a5cf816e71f2ca2a7f5fd949a3
    Story: 2004751
    Task: 29417
    Signed-off-by: Don Penney <don.penney@windriver.com>

commit 314e07a8f8d1e9fa2fc41182e0338be67e254972
Author: Robert Church <robert.church@windriver.com>
Date:   Sat Feb 9 02:27:56 2019 -0500

Remove problematic upstream mariadb commit
    
    When removing the mariadb release from the cluster, this upstream commit
    produces mariadb-ingress pods that are stuck in the "Terminating" state
    with the associated containers becoming hung. This ultimately impacts
    certain docker operations leading to PLEG health issues in the cluster.
    
    The root cause of this is that the ingress pod uses dumb-init to start
    the nginx-ingress-controller process. When the mariadb-ingress pod
    terminates (via kill -TERM 1) all child processes are terminated but the
    docker-containerd-shim remains causing the hung container condition.
    
    Temporarily reverting this commit. A fix will be introduced upstream
    dealing with dumb-init and this commit will be pick up again on the next
    full chart rebase.
    
    Change-Id: I25ad9bc3213468a9060e741917d96d9ac5c01b40
    Story: 2004520
    Task: 29420
    Signed-off-by: Robert Church <robert.church@windriver.com>

commit 98fa1202436d26a399a804f74e07954cb93eb056
Author: Don Penney <don.penney@windriver.com>
Date:   Thu Feb 7 10:55:06 2019 -0500

Update stx-neutron.master to use forked neutron repo
    
    This update temporarily points the stx-neutron master build
    to a forked neutron repo.  This is necessary to pull in specific
    commits that have not yet merged upstream.  Once all required
    commits are merged, this can revert back to the upstream repo.
    
    Story: 2004751
    Task: 29396
    Change-Id: I7453ffe943ff735338a06246a08399f79d6918b7
    Signed-off-by: Don Penney <don.penney@windriver.com>

commit 793b3e0ae9a7540f317b5bc19cf577c6217ebc83
Author: Al Bailey <Al.Bailey@windriver.com>
Date:   Mon Feb 4 15:05:06 2019 -0600

Fix horizon imports related to stein novaclient
    
    The pike novaclient contains wrs extensions were being
    imported by horizon during startup.
    
    Those imports fail with a stein novaclient, and so the
    horizon code will no longer import those files.
    
    When Horizon is moved to Stein (story 2004765)
    this change will end up being removed.
    
    Story: 2004751
    Task: 29356
    Change-Id: Ia51e8b592c58eab0df09bf9b345cbae9dbf0a319
    Signed-off-by: Al Bailey <Al.Bailey@windriver.com>

commit e936d8acec383c824bcf18b055de83b3a46bfd7b
Author: Al Bailey <Al.Bailey@windriver.com>
Date:   Fri Feb 1 08:49:53 2019 -0600

Adding ceilometerclient to the ISO
    
    Platform horizon (pike) needs ceilometerclient
    to satisy python imports.
    
    When Horizon is moved to Stein (story 2004765)
    this can be reverted.
    
    Story: 2004751
    Task: 29288
    Change-Id: I636296966f1c293fe8d3bbd1b3b1a335fb6a863a
    Depends-On: Iee76e932383493aea7d9fa0d18425f01e2f6a65d
    Signed-off-by: Al Bailey <Al.Bailey@windriver.com>

commit 7856f0ab35e5ff439102bb8a8effa1f92664c9c9
Author: Robert Church <robert.church@windriver.com>
Date:   Tue Jan 15 03:48:03 2019 -0500

Stein: Update neutron's daemonset-ovs-agent.yaml
    
    neutron-sanity-check command triggers privsep code and produces the
    following:
    
    WARNING oslo.privsep.daemon [-] privsep log:
    oslo_log.log.LogConfigError: Error loading logging config
    /etc/neutron/logging.conf: [Errno 2] No such file or directory:
    '/etc/neutron/logging.conf'
    
    This will add log_config_append to neutron-etc and allow ovs-agent to
    successfully deploy
    
    Change-Id: I82dd56dc686e2efbd10921b248497955c37279f1
    Story: 2004751
    Task: 28894
    Signed-off-by: Robert Church <robert.church@windriver.com>

commit 564e23702ba79f4234ca5ff59d9bef1b9cc82ee8
Author: Robert Church <robert.church@windriver.com>
Date:   Mon Jan 14 10:57:45 2019 -0500

Stein: Update Cinder to include resource_filters.json
    
    During Stein the deprecated query_volume_filters config option was
    removed. As a result, /etc/cinder/resource_filters.json is required to
    allow volume display operations to function.
    
    This adds the file to the configmap and provides the default values for
    the filters.
    
    Change-Id: I5b21949160430c72eb63db20475efa9f098e809f
    Story: 2004751
    Task: 28894
    Signed-off-by: Robert Church <robert.church@windriver.com>

commit 536fd15055fcbb3d15b3a591eb04e39d37716f2a
Author: Robert Church <robert.church@windriver.com>
Date:   Mon Jan 14 10:33:50 2019 -0500

Stein: Remove ceilometer-upgrade option
    
    This removes the --skip-metering-database option from _db-sync.sh.tpl.
    This option was removed with the deprecated storage drivers in
    Queens.
    
    Change-Id: I6c88f882c9e54841c9eeb97c5010cf0691804a62
    Story: 2004751
    Task: 28894
    Signed-off-by: Robert Church <robert.church@windriver.com>

commit d7964a17fb9391356c867bcae0ad938bf63b79a4
Author: Al Bailey <Al.Bailey@windriver.com>
Date:   Wed Jan 9 14:48:12 2019 -0600

Update openstack clients to stein/master
    
    python-aodhclient spec file is based off 1.1.1 (rocky)
    python-barbicanclient spec file is based off 4.7.1 (rocky)
    python-cinderclient spec file is based off 4.0.1 (rocky)
    python-glanceclient spec file is based off 2.13.1 (rocky)
     - docs disabled
    python-gnocchiclient spec file is based off 7.0.4 (rocky)
    python-heatclient spec file is based off 1.16.1 (rocky)
     - docs disabled
    python-ironicclient spec file is based off 2.5.0 (rocky)
    python-keystoneclient spec file is based off 3.17.0 (rocky)
    python-magnumclient spec file is based off 2.10.0 (rocky)
    python-muranoclient spec file is based off 1.1.1 (rocky)
    python-neutronclient spec file is based off 6.9.1 (rocky)
    python-novaclient spec file is based off 11.0.0 (rocky)
     - docs disabled
    python-openstackclient spec file is based off 3.16.2 (rocky)
     - docs disabled
     - unit tests disabled
    python-openstacksdk spec file is based of 0.17.2 (rocky)
     - unit tests disabled
    python-pankoclient spec file is based off 0.5.0 (rocky)
    
    The primary changes to each spec files are
    1) version has tis extension
    2) sdk package added
    3) wheels package added
    
    Disable building:
     - openstack-aodh
     - python-osc-lib
    
    ceilometerclient no longer exists.
    openstack-heat no longer requires ceilometerclient
    
    Story: 2004751
    Task: 28864
    Change-Id: Ifa905bea2e95ded72a327f8ff43667c8c5429363
    Depends-On: Iea58cb9484c75cf757397d53d7d1576a2f436d81
    Signed-off-by: Al Bailey <Al.Bailey@windriver.com>

commit bd659dc88a40c62dbd464b11a2d33e9f3b782e0f
Author: Scott Little <scott.little@windriver.com>
Date:   Tue Jan 8 16:31:48 2019 -0500

Update .gitreview for f/stein
    
    Change-Id: I470b2e162f8ee0ceaddf901078215fd458335b03
    Signed-off-by: Scott Little <scott.little@windriver.com>

tags:

added: in-f-centos76

Ken Young (kenyis) on 2019-04-05

tags:

added: stx.2.0
removed: stx.2019.05

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.