StarlingX

Bug #1908720
Comment #3

Comment 3 for bug 1908720

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2021-06-01: Fix merged to openstack-armada-app (f/centos8)

Reviewed: https://review.opendev.org/c/starlingx/openstack-armada-app/+/792235
Committed: https://opendev.org/starlingx/openstack-armada-app/commit/1bf694661282a019bf79f253fc148baede65db64
Submitter: "Zuul (22348)"
Branch: f/centos8

commit 963e63cd55d5be4f5ddfc148ae00b6a46e071295
Author: Thiago Brito <email address hidden>
Date: Fri May 14 15:36:07 2021 -0300

Fix cpu_shared/dedicated_set config location

    Change I61514389b616db754b0d2f35deb0101f90dbdd02 removed the deprecated
    property vcpu_pin_set in favor of the newer cpu_shared_set and
    cpu_dedicated_set, but those new configs are placed under the [compute]
    section of nova.conf instead of [DEFAULT]. This is causing VMs to be
    scheduled on platform reserved cores. This commit will fix it.

Closes-Bug: #1928683

Signed-off-by: Thiago Brito <email address hidden>
Change-Id: I541760619f4c79c66a2bf22715afdc873b8343ce

commit 58f4d9ffcaf47fe969267149135201aec01624a8
Author: Gustavo Santos <email address hidden>
Date: Mon Mar 8 14:56:55 2021 -0300

Add k8s proxy-body-size to horizon overrides

    The current network.dashboard.ingress.annotations in horizon's
    values.yaml helm charts do not include the kubernetes property
    'proxy-body-size'. This makes the resulting nginx.conf file in ingress
    add the default rule 'max_body_size 1m' to the horizon servers,
    which limits all http requests' size inside horizon to 1MiB, making it
    impossible to upload images larger than that to glance using the
    horizon GUI, for example.

    This change adds said property to the horizon overrides, making
    horizon's servers in nginx.conf include a 'max_body_size' of 2500MiB,
    which makes uploading images up to that size possible again.

    Story: 2008692
    Task: 41996
    Change-Id: I91888ce238d5304c08eb1e97918989b8f93ee34f

commit b5c1f62088778287e4b50aeac1f17d166a7a177a
Author: Dan Voiculeasa <email address hidden>
Date: Wed Feb 3 16:00:47 2021 +0200

Introduce metadata for app behavior control

Keep existing behavior when evaluating app reapplies.

    Story: 2007960
    Task: 41755
    Signed-off-by: Dan Voiculeasa <email address hidden>
    Change-Id: Ie02743cdf056dda3feb66911c74f9dabe69d98dd

commit eab750b7ff03808002acf35deebdf762e687b332
Author: Martin, Chen <email address hidden>
Date: Sat May 30 09:11:05 2020 +0800

Add override setting in openstack helm plugin for rook-ceph

    Deploy with rook-ceph, without "system storage-backend-add ceph"
    there is no object storage-ceph in database. As current openstack
    helm plugin fixed on object storage-ceph, in rook-ceph case
    use a fixed override setting

Story: 2005527
Task: 39914

Depends-On: https://review.opendev.org/#/c/713084/

Change-Id: Ied852d60e8b15d55865747e0b6f4b54f2392d6df
Signed-off-by: Martin, Chen <email address hidden>

commit 852d8d61dbfc4f9f29afe8da10924731a58028ea
Author: Dan Voiculeasa <email address hidden>
Date: Mon Nov 16 12:41:55 2020 +0200

Introduce lifecycle operator to openstack app

A big chunk of logic is moved from sysinv conductor to application
itself.

    Following hooks were necessary:
    pre-apply, post-apply, pre-manifest-apply, pre-apply-rbd,
    pre-apply-resource, post-remove-rbd, post-remove-resource, post-remove

    Change-Id: I41858c831a4af564dbdf38934d51d34489bf8a9a
    Story: 2007960
    Task: 41293
    Signed-off-by: Dan Voiculeasa <email address hidden>

commit b64b0204465dc51c57b45bf9925b0fbb88749a45
Author: hbrito <email address hidden>
Date: Fri Dec 18 13:52:51 2020 +0000

Increase proxy-connect-timeout to avoid nginx timeout errors

This patch increases the proxy-connect-timeout from 5 to 30 seconds,
avoiding the Bad Gateway 502 error when CLI commands are executed.

    Closes-bug: 1908720
    Change-Id: I557456e9d0550a906b6d849d682de7ea3f0f42ad
    Signed-off-by: hbrito <email address hidden>

commit ca527c227653a13e617122051c18591aa1212f98
Author: Don Penney <email address hidden>
Date: Wed Jan 6 14:35:03 2021 -0500

Remove empty package from python-k8sapp-openstack

    Packages defined in a spec with no files do not result in an RPM
    produced by the build. On a rebuild, the build tools scan the spec and
    sees the package defined but does not find a corresponding RPM, and so
    flags the package for a rebuild as a result.

This commit removes the empty package definition from the spec.

    Partial-Bug: 1910439
    Signed-off-by: Don Penney <email address hidden>
    Change-Id: Ie1f18b1592f8187900624d993434ba04b23cbcff

commit cb9854c701ab631628902ae1e9d9e76f0e2785b0
Author: Zhipeng Liu <email address hidden>
Date: Thu Dec 24 22:21:41 2020 +0800

Update cpu_shared_set and cpu_dedicated_set in nova config

    Starting from Ussuri, OpenStack is deprecating vcpu_pin_set
    in favor of cpu_dedicated_set and cpu_shared_set. These
    overriders must be supported to be generated via Starlingx
    system commands.

    Closes-Bug: 1904729
    Change-Id: I61514389b616db754b0d2f35deb0101f90dbdd02
    Signed-off-by: Zhipeng Liu <email address hidden>

commit 0c30ffc410d9ca720ce80cb5fb08ae81adf05d2a
Author: Don Penney <email address hidden>
Date: Thu Dec 17 13:23:11 2020 -0500

Add auto-version for remaining stx/openstack-armada-app packages

    Update remaining StarlingX packages with hardcoded TIS_PATCH_VER to
    use PKG_GITREVCOUNT where possible, with offsets as needed to ensure
    the version is incremented above the hardcoded version.

    Story: 2008455
    Task: 41455
    Signed-off-by: Don Penney <email address hidden>
    Change-Id: Icdc9d71d1268a4d3dd9e569c8642717bceadda5e

commit fc68439414816b2384aed1e88120713f645db8d8
Author: Mihnea Saracin <email address hidden>
Date: Fri Sep 11 18:46:48 2020 +0300

Fix apply of stx-openstack when host is locked

    Currently, all of the stx-openstack services have the
    replica count set to the number of the controllers.
    If one of the controllers is locked their replicas
    number will still be 2 which is incorrect.
    We solve this by changing the number of replicas
    to be equal to the number of the active controllers.
    The rabbitmq and mariadb services cannot use this approach because
    they are unable to work properly if their replica number
    is decreased from 2 to 1. So a kubernetes toleration
    is used here to allow the rabbitmq and mariadb pods to be
    deployed on the locked controller.

    Change-Id: I15cf2a3f62525751435ddbe66760935f3ab21d2b
    Closes-Bug: 1879018
    Signed-off-by: Mihnea Saracin <email address hidden>

Reviewed:  https://review.opendev.org/c/starlingx/openstack-armada-app/+/792235
Committed: https://opendev.org/starlingx/openstack-armada-app/commit/1bf694661282a019bf79f253fc148baede65db64
Submitter: "Zuul (22348)"
Branch:    f/centos8

commit 963e63cd55d5be4f5ddfc148ae00b6a46e071295
Author: Thiago Brito <thiago.brito@windriver.com>
Date:   Fri May 14 15:36:07 2021 -0300

Fix cpu_shared/dedicated_set config location
    
    Change I61514389b616db754b0d2f35deb0101f90dbdd02 removed the deprecated
    property vcpu_pin_set in favor of the newer cpu_shared_set and
    cpu_dedicated_set, but those new configs are placed under the [compute]
    section of nova.conf instead of [DEFAULT]. This is causing VMs to be
    scheduled on platform reserved cores. This commit will fix it.
    
    Closes-Bug: #1928683
    
    Signed-off-by: Thiago Brito <thiago.brito@windriver.com>
    Change-Id: I541760619f4c79c66a2bf22715afdc873b8343ce

commit 58f4d9ffcaf47fe969267149135201aec01624a8
Author: Gustavo Santos <gustavofaganello.santos@windriver.com>
Date:   Mon Mar 8 14:56:55 2021 -0300

Add k8s proxy-body-size to horizon overrides
    
    The current network.dashboard.ingress.annotations in horizon's
    values.yaml helm charts do not include the kubernetes property
    'proxy-body-size'. This makes the resulting nginx.conf file in ingress
    add the default rule 'max_body_size 1m' to the horizon servers,
    which limits all http requests' size inside horizon to 1MiB, making it
    impossible to upload images larger than that to glance using the
    horizon GUI, for example.
    
    This change adds said property to the horizon overrides, making
    horizon's servers in nginx.conf include a 'max_body_size' of 2500MiB,
    which makes uploading images up to that size possible again.
    
    Story: 2008692
    Task: 41996
    Change-Id: I91888ce238d5304c08eb1e97918989b8f93ee34f

commit b5c1f62088778287e4b50aeac1f17d166a7a177a
Author: Dan Voiculeasa <dan.voiculeasa@windriver.com>
Date:   Wed Feb 3 16:00:47 2021 +0200

Introduce metadata for app behavior control
    
    Keep existing behavior when evaluating app reapplies.
    
    Story: 2007960
    Task: 41755
    Signed-off-by: Dan Voiculeasa <dan.voiculeasa@windriver.com>
    Change-Id: Ie02743cdf056dda3feb66911c74f9dabe69d98dd

commit eab750b7ff03808002acf35deebdf762e687b332
Author: Martin, Chen <haochuan.z.chen@intel.com>
Date:   Sat May 30 09:11:05 2020 +0800

Add override setting in openstack helm plugin for rook-ceph
    
    Deploy with rook-ceph, without "system storage-backend-add ceph"
    there is no object storage-ceph in database. As current openstack
    helm plugin fixed on object storage-ceph, in rook-ceph case
    use a fixed override setting
    
    Story: 2005527
    Task: 39914
    
    Depends-On: https://review.opendev.org/#/c/713084/
    
    Change-Id: Ied852d60e8b15d55865747e0b6f4b54f2392d6df
    Signed-off-by: Martin, Chen <haochuan.z.chen@intel.com>

commit 852d8d61dbfc4f9f29afe8da10924731a58028ea
Author: Dan Voiculeasa <dan.voiculeasa@windriver.com>
Date:   Mon Nov 16 12:41:55 2020 +0200

Introduce lifecycle operator to openstack app
    
    A big chunk of logic is moved from sysinv conductor to application
    itself.
    
    Following hooks were necessary:
    pre-apply, post-apply, pre-manifest-apply, pre-apply-rbd,
    pre-apply-resource, post-remove-rbd, post-remove-resource, post-remove
    
    Change-Id: I41858c831a4af564dbdf38934d51d34489bf8a9a
    Story: 2007960
    Task: 41293
    Signed-off-by: Dan Voiculeasa <dan.voiculeasa@windriver.com>

commit b64b0204465dc51c57b45bf9925b0fbb88749a45
Author: hbrito <hugo.brito@windriver.com>
Date:   Fri Dec 18 13:52:51 2020 +0000

Increase proxy-connect-timeout to avoid nginx timeout errors
    
    This patch increases the proxy-connect-timeout from 5 to 30 seconds,
    avoiding the Bad Gateway 502 error when CLI commands are executed.
    
    Closes-bug: 1908720
    Change-Id: I557456e9d0550a906b6d849d682de7ea3f0f42ad
    Signed-off-by: hbrito <hugo.brito@windriver.com>

commit ca527c227653a13e617122051c18591aa1212f98
Author: Don Penney <don.penney@windriver.com>
Date:   Wed Jan 6 14:35:03 2021 -0500

Remove empty package from python-k8sapp-openstack
    
    Packages defined in a spec with no files do not result in an RPM
    produced by the build. On a rebuild, the build tools scan the spec and
    sees the package defined but does not find a corresponding RPM, and so
    flags the package for a rebuild as a result.
    
    This commit removes the empty package definition from the spec.
    
    Partial-Bug: 1910439
    Signed-off-by: Don Penney <don.penney@windriver.com>
    Change-Id: Ie1f18b1592f8187900624d993434ba04b23cbcff

commit cb9854c701ab631628902ae1e9d9e76f0e2785b0
Author: Zhipeng Liu <zhipengs.liu@intel.com>
Date:   Thu Dec 24 22:21:41 2020 +0800

Update cpu_shared_set and cpu_dedicated_set in nova config
    
    Starting from Ussuri, OpenStack is deprecating vcpu_pin_set
    in favor of cpu_dedicated_set and cpu_shared_set. These
    overriders must be supported to be generated via Starlingx
    system commands.
    
    Closes-Bug: 1904729
    Change-Id: I61514389b616db754b0d2f35deb0101f90dbdd02
    Signed-off-by: Zhipeng Liu <zhipengs.liu@intel.com>

commit 0c30ffc410d9ca720ce80cb5fb08ae81adf05d2a
Author: Don Penney <don.penney@windriver.com>
Date:   Thu Dec 17 13:23:11 2020 -0500

Add auto-version for remaining stx/openstack-armada-app packages
    
    Update remaining StarlingX packages with hardcoded TIS_PATCH_VER to
    use PKG_GITREVCOUNT where possible, with offsets as needed to ensure
    the version is incremented above the hardcoded version.
    
    Story: 2008455
    Task: 41455
    Signed-off-by: Don Penney <don.penney@windriver.com>
    Change-Id: Icdc9d71d1268a4d3dd9e569c8642717bceadda5e

commit fc68439414816b2384aed1e88120713f645db8d8
Author: Mihnea Saracin <Mihnea.Saracin@windriver.com>
Date:   Fri Sep 11 18:46:48 2020 +0300

Fix apply of stx-openstack when host is locked
    
    Currently, all of the stx-openstack services have the
    replica count set to the number of the controllers.
    If one of the controllers is locked their replicas
    number will still be 2 which is incorrect.
    We solve this by changing the number of replicas
    to be equal to the number of the active controllers.
    The rabbitmq and mariadb services cannot use this approach because
    they are unable to work properly if their replica number
    is decreased from 2 to 1. So a kubernetes toleration
    is used here to allow the rabbitmq and mariadb pods to be
    deployed on the locked controller.
    
    Change-Id: I15cf2a3f62525751435ddbe66760935f3ab21d2b
    Closes-Bug: 1879018
    Signed-off-by: Mihnea Saracin <Mihnea.Saracin@windriver.com>