cephfs-provisioner can not update

Bug #1918674 reported by Daniel Safta
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Medium
Daniel Safta

Bug Description

Brief Description
-----------------
After changing the overrides cephfs-provisioner chart fails to update.

Severity
--------
Provide the severity of the defect.
Major

Steps to Reproduce
------------------
Do an override using system helm-override-update and reapply platform-integ-appps.

Expected Behavior
------------------
platform-integ-apps works.

Actual Behavior
----------------
platform-integ-apps fails to update

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

System Configuration
--------------------
AIO-SX

Branch/Pull Time/Commit
-----------------------
master

Last Pass
---------

Timestamp/Logs
--------------

Test Activity
-------------

Workaround
----------

Revision history for this message
Bob Church (rchurch) wrote :
Ghada Khalil (gkhalil)
tags: added: stx.5.0 stx.storage
Changed in starlingx:
importance: Undecided → Medium
status: New → Triaged
assignee: nobody → Daniel Safta (dsafta)
Revision history for this message
Bob Church (rchurch) wrote :
Changed in starlingx:
status: Triaged → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to platform-armada-app (f/centos8)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to platform-armada-app (f/centos8)
Download full text (11.0 KiB)

Reviewed: https://review.opendev.org/c/starlingx/platform-armada-app/+/792234
Committed: https://opendev.org/starlingx/platform-armada-app/commit/f3bb236173d8bb336513a868d36f601a23ab87dc
Submitter: "Zuul (22348)"
Branch: f/centos8

commit 66fa48f04d926848a37f5fbb7689cf4b114cb3ba
Author: Pedro Henrique Linhares <email address hidden>
Date: Tue Apr 6 21:45:28 2021 -0300

    Update helm charts config maps after sx-dx migration with new CEPH monitors

    This commit adds annotations that allows config maps to be recreated
    after ceph monitor IP changes due to DX migration so that existing
    StorageClasses can get a reference to the correct monitor. StorageClasses
    and provisioners are recreated during platform-integ-apps auto re-apply.

    Story: 2008587
    Task: 42242

    Signed-off-by: Pedro Linhares <email address hidden>
    Depends-On: https://review.opendev.org/c/starlingx/stx-puppet/+/783727
    Change-Id: I9cedc70326e92796f03520deed7f0857e119257f

commit 9d45149d29b8c05808bcd7e5a129f135b3386931
Author: Isac Souza <email address hidden>
Date: Mon Apr 5 11:07:21 2021 -0300

    Use new method for setting num of replicas

    Use the new _num_replicas_for_platform_app method from the
    helm base class to set the number of replicas in the chart.
    The new method will return the number of provisioned
    controllers with a minimum of 1.

    Tested by building an ISO and installing the armada apps.

    Partial-Bug: 1922278
    Signed-off-by: Isac Souza <email address hidden>
    Change-Id: Idb3c93274a1cb5c410d885d459784382525427a0

commit 1021d50142af6422c9a3f0853f4b7c525e724ab8
Author: Daniel Safta <email address hidden>
Date: Wed Mar 24 12:56:29 2021 +0000

    Removed extra serviceAccount from cephfs-provisioner

    cephfs-provisioner may need to create new resources
    in the kubernetes cluster. It was granted access to
    some of the resources including namespaces but when
    https://review.opendev.org/c/starlingx/platform-armada-app/+/778746
    got merged the serviceAccount was changed.

    I have updated the serviceAccount with access to
    creating new namespaces and secrets.

    The serviceAccount that was initially used to create
    namespaces and secrets is not needed anymore, so I
    have removed it.

    Closes-bug: 1921197
    Change-Id: I3c683776f3ecaf9c78d1a6b5b1108e9582497dde
    Signed-off-by: Daniel Safta <email address hidden>

commit 45fd0a6b2cd2dbaaf7ff21ef989824377c5b17dc
Author: Robert Church <email address hidden>
Date: Thu Mar 18 11:43:21 2021 -0400

    Build: Isolate platform plugins to an app specific directory

    When building the stx-platform-helm RPM for platform-integ-apps the helm
    plugins are installed in a location that could be populated with other
    app plugins if their spec files are not properly set up.

    Adjust the spec to provide an app specific location for the plugins to
    ensure that no other app plugins are included in the application tarball

    Closes-Bug: #1920066
    Change-Id: Id24227cd100a3c29809f1dd01f61ea717...

tags: added: in-f-centos8
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.