Comment 9 for bug 1877548

Revision history for this message
Frank Miller (sensfan22) wrote :

I have confirmed this is an issue with the incorrect helm chart being used by the build system. Austin's analysis is correct that this commit is not being included in the generated helm-charts:
https://review.opendev.org/#/c/724385

Looking at the stx-openstack.yaml in the helm-charts-stx-openstack-centos-stable-latest.tgz tarball we see:
data:
  chart_name: ingress
  dependencies:
  - helm-toolkit
  install:
    no_hooks: false
  namespace: kube-system
  release: kube-system-ingress

But Sabeel's commit changed this to release: openstack-ingress

The reason for the build output to not include Sabeel's change is unclear at this time. The theory is that the layered build system builds this package but because the version did not change it is not able to properly distinguish a previously built package with the same version from the newly built package. Scott will need to confirm this theory. As a workaround the package is being upversioned to see if this addresses the issue and results in the right helm chart package in the build output directory. Even if this works a more robust solution will be required in the build system.

On a side note, now that we have cutover to the build layering method, the monolithic method still runs nightly but does not generate docker images or helm charts. As such when using a monolithic build the helm charts picked up by sanity are from the build layer method - this is not correct and needs to be addressed separately.