Comment 18 for bug 1951245

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-armada-app (master)

Reviewed: https://review.opendev.org/c/starlingx/openstack-armada-app/+/818621
Committed: https://opendev.org/starlingx/openstack-armada-app/commit/2755a004578d0e22dc94bdd3bf683a11b234f0d4
Submitter: "Zuul (22348)"
Branch: master

commit 2755a004578d0e22dc94bdd3bf683a11b234f0d4
Author: Heitor Matsui <email address hidden>
Date: Fri Nov 19 14:47:22 2021 -0300

    Get admin password and use it on config template

    The pci-irq-affinity-agent uses the platform keyring file to
    fetch the admin password to communicate with OpenStack services,
    but now that the agent is raised during the application apply it
    can use the same approach the other helm charts use to get the
    credentials, using it's plugin to capture the information and
    generate the system overrides with the values.

    This commit:
    - Changes the plugin to get the credentials
    - Changes the pci-irq-affinity-agent helm chart to include the
    password on the chart values and use it on the agent config file
    template
    - Adds an init container with dependencies on libvirt and nova
    compute pods, so that the agent pod is only create when those
    are available
    - Removes the keyring mount on the container, which will not be
    needed anymore with the previous changes and is causing failure
    when raising the pod
    - Removes additional keyring tools

    Depends-On: https://review.opendev.org/c/starlingx/utilities/+/818620
    Closes-Bug: 1951245
    Signed-off-by: Heitor Matsui <email address hidden>
    Change-Id: I26f993146b8a17b7602a45f0cd5d983c1d93b0c1