Comment 3 for bug 2053074

Reviewed: https://review.opendev.org/c/starlingx/config/+/913717
Committed: https://opendev.org/starlingx/config/commit/b1b160f48bab21af56b617a911218877cde13550
Submitter: "Zuul (22348)"
Branch: master

commit b1b160f48bab21af56b617a911218877cde13550
Author: Igor Soares <email address hidden>
Date: Tue Mar 19 17:29:11 2024 -0300

    Fix charts upload when there are existing ones

    This fixes a bug that prevents StarlingX application charts from
    being uploaded to the helm repository when one or more of them have been
    uploaded before.

    The charts upload logic was changed to check if all charts provided by
    the given application are valid prior to uploading. If a chart is
    invalid then no charts for that application will be uploaded, since the
    upload process cannot proceed in that scenario.

    Test Plan:
    PASS: build-pkgs -a && build-image
    PASS: AIO-SX fresh install
    PASS: Build a platform-integ-apps version containing one existing chart
          and two nonexistent charts in the local Helm repository.
          Update platform-integ-apps to the built version.
          Confirm that the existing chart was not re-uploaded and that the
          nonexistent ones were correctly uploaded to the Helm repository.
    PASS: Apply/remove/delete platform-integ-apps

    Closes-Bug: 2053074
    Depends-on: https://review.opendev.org/c/starlingx/integ/+/912305

    Change-Id: I155d457f58be1986cc6f25178929aedfbe1d0693
    Signed-off-by: Igor Soares <email address hidden>