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: Id24227cd100a3c29809f1dd01f61ea7174e9d779
    Signed-off-by: Robert Church <email address hidden>

commit 88570d9f0539c6f3827fb708c65e5f497e5ce776
Author: Daniel Safta <email address hidden>
Date:   Thu Mar 11 13:32:42 2021 +0000

    Fixed cephfs-provisioner chart update
    
    cephfs-provisioner fails to upgrade when overrides
    are changed due to missing annotations.
    
    This commit address an application issue with
    https://review.opendev.org/c/starlingx/platform-armada-app/+/758786
    
    Closes-bug: 1918674
    Signed-off-by: Daniel Safta <email address hidden>
    Change-Id: I270c551d09c8b0c4440406cad68a8b5799cd9fb9

commit f48ac19d64072c52a69f38f63f131dc6396667d8
Author: Daniel Safta <email address hidden>
Date:   Thu Mar 4 15:23:19 2021 +0000

    Fixed the restartPolicy and serviceAccount rules
    
    When cephfs-storage-init gets scheduled by k8s
    on a compute node it can not pull images because
    it was using the wrong serviceAccount.
    
    This serviceAccount, along with new restartPolicy
    and backOffLimit rules are fixed in this commit.
    
    This commit address an application issue with
    https://review.opendev.org/c/starlingx/platform-armada-app/+/758786
    
    Closes-bug: 1917788
    Signed-off-by: Daniel Safta <email address hidden>
    Change-Id: I270c551d09c8b1c2870406cad68a8b5799cd9fb9

commit d30187c950cf622c7d6e65cc2d0995e51b41641f
Author: Dan Voiculeasa <email address hidden>
Date:   Thu Feb 4 12:12:54 2021 +0200

    Introduce metadata for app behavior control
    
    App should achieve applied state.
    Keep existing behavior when evaluating app reapplies.
    
    Note: Sibling commits modify metadata for other apps. Cert-manager and
    rook apps will no longer be evaluated strictly after
    platform-integ-apps.
    
    Story: 2007960
    Task: 41754
    Signed-off-by: Dan Voiculeasa <email address hidden>
    Change-Id: Ia7bbca906e343ffffa019885a790befdf5ccb565

commit d155fb78ca24288a05f02e7fc068dbab6d7d0c62
Author: Daniel Safta <email address hidden>
Date:   Fri Jan 15 15:49:55 2021 +0000

    Ceph-pools-audit job misses start time
    
    The Cronjob controller will check every 10s if there is a job to
    start, but if there are more than 100 retries to start the job,
    it will fail with "Cannot determine if job needs to be started.
    Too many missed start time (> 100). Set or decrease
    .spec.startingDeadlineSeconds or check clock skew." error.
    
    If the Cronjob controller happens to be down for a long period
    and 100 retries occured since lastScheduledTime, after that it
    will NOT retry to run the job.
    
    Because concurrencyPolicy is set to Forbid and
    startingDeadlineSeconds was not set, the Cronjob controller
    will fail to start a new job when another job is already running
    and will increase an internal counter.
    
    If startingDeadlineSeconds is set, the Cronjob controller will
    count how many fails occured in the last startingDeadlineSeconds
    seconds.
    
    Closes-bug: 1913057
    Signed-off-by: Daniel Safta <email address hidden>
    Change-Id: I3ca5ad04290580a8d0bf20f787cf5eefef6ac422

commit 3236e99affb558b0b25ea50ecdbea2072d61b7f9
Author: Robert Church <email address hidden>
Date:   Thu Jan 21 20:37:38 2021 +0200

    Insert dependency between python-k8sapp and stx-app-helm
    
    A change in python-k8sapp-platform is not always detected when building
    stx-platform-helm.
    
    Make sure changes are detected.
    Keep stx-platform-helm and python-k8sapp-platform version in sync.
    
    Closes-Bug: 1912490
    Signed-off-by: Dan Voiculeasa <email address hidden>
    Change-Id: If2b4efb9213db5f621f55ffe9770253251e6fccc

commit 2aedbcda09915c0b21b4fc963901243e184e8035
Author: Dan Voiculeasa <email address hidden>
Date:   Mon Nov 9 13:21:15 2020 +0200

    Introduce lifecycle operator to platform app
    
    A big chunk of logic is moved from sysinv conductor to application
    itself.
    
    Following hooks were necessary:
      pre-apply-rbd, pre-apply-resource, post-remove-rbd,
      post-remove-resource, auto-apply-semantic-check,
      post-armada-request
    
    Change-Id: Ibe994411fee55c84fa86770fad5497040f13b78f
    Story: 2007960
    Task: 41292
    Signed-off-by: Dan Voiculeasa <email address hidden>

commit 3f7acd5c3214fb93afc75ee23c0eabe6089d0c32
Author: Daniel Safta <email address hidden>
Date:   Tue Jan 5 14:14:33 2021 +0000

    Fixed the default storageclass and replicas number
    
    A previous update that added a new storageclass
    using cephfs-provisioner affects the installation of
    some application that are trying to use the
    default storageclass.
    
    When an application will try to use the default
    storageclass it will find two default storageclasses
    and we want to avoid this problem by using a single
    default storageclass that can be changed by the user
    through helm overrides.
    
    Also this fixes the replicas number for the deployment,
    it was hardcoded with 1, but now it can be changed by the user.
    
    This commit address an application issue with
    https://review.opendev.org/c/starlingx/platform-armada-app/+/758786
    
    Signed-off-by: Daniel Safta <email address hidden>
    Story: 2008162
    Task: 40908
    Change-Id: Ib1e4db29397e674a17d45120f6c0b3babaf1880b

commit a7681c3f3246830cc4c6fb58d3a9da0b9777e4cf
Author: Don Penney <email address hidden>
Date:   Wed Jan 6 14:36:19 2021 -0500

    Remove empty package from python-k8sapp-platform
    
    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: I48275e45b5c5628564f9d40807ba7828510ff80a

commit e2f6893ed0855b4f6b061ab0c7501d285d52c2f6
Author: Daniel Safta <email address hidden>
Date:   Fri Dec 18 14:34:51 2020 +0000

    Fixed missing key in cephfs-provisioner chart
    
    A previous update broke the build
    of stx-platform-helm, due to the
    missing defaultStorageClass key.
    This update defines this value in
    the cephfs-provisioner chart to fix
    the issue.
    
    This commit address a build issue with
    https://review.opendev.org/c/starlingx/platform-armada-app/+/758786
    
    Signed-off-by: Daniel Safta <email address hidden>
    Story: 2008162
    Task: 40908
    Change-Id: I0e70a5b504f8240e8d661415b27f6a86875c17dd

commit 3466dd67da245ebc678aa75c3c95c6c90257326e
Author: Daniel Safta <email address hidden>
Date:   Mon Oct 19 10:01:42 2020 +0000

    CephFS provisioner for k8sapp_platform
    
    This commit implements CephFS Provisioner as a helm chart
    in platform-integ-apps.
    CephFS Provisioner implements the same design as RBD Provisioner,
    but with CephFS Provisioner we can have mounts shared by differents
    components, such as pods.
    
    It is activated automatically same as RBD Provisioner.
    
    Signed-off-by: Daniel Safta <email address hidden>
    Story: 2008162
    Task: 40908
    Change-Id: Ic4270e401b2c3e51c3aecfab23af1e874e733831

commit 086a81deea855d51c65d70cc2288a266d671b9b4
Author: Robert Church <email address hidden>
Date:   Tue Dec 1 21:43:12 2020 -0500

    Enable Zuul tox jobs for platform plugin validation
    
    NOTE: As part of enabling this, --use-deprecated legacy-resolver was
    required. This is a result of a new version of pip installed on Dec 3,
    2020. A followup commit is required to remove this a deal with the
    fallout.
    
    Change-Id: I29d9484f1f3ffbb7b949c5d4177f95e7a60a99f4
    Story: 2008162
    Task: 41347
    Related-Bug: #1907125
    Signed-off-by: Robert Church <email address hidden>