Upstream helm-charts and helm-charts-elastic have modified content

Bug #1857049 reported by Saul Wold on 2019-12-19
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Medium
Saul Wold

Bug Description

Something seems to have changed with downloading github.com archive tarballs. The tarballs currently on the mirror contain the toplevel directory of helm-charts/ or helm-charts-elastic/, but if they are freshly downloaded the toplevel directory now contains or charts-92b6289ae93816717a8453cfe62bad51cbdb8ad0/ helm-charts-2bd7616ceddbdf2eee88965e2028ee37d304c79c/

This causes a build failure in monitor-armada-app then the RPM build happens and it tries to do the "%setup -n helm-charts" and gets the following error that rpmbuild can't change cd to the helm-charts directory.

I think the fix would be to use "%setup -n charts-%{sha}", it will also require coordination with updating the mirror with the newer tarball contents.

Ghada Khalil (gkhalil) on 2020-01-03
tags: added: stx.build
Ghada Khalil (gkhalil) wrote :

@Saul, Is this an issue specific to developer builds? I believe the nightly CENGN builds are successful.

Changed in starlingx:
status: New → Incomplete
Saul Wold (sgw-starlingx) wrote :

We investigated this and found that the tarball on CENGN is an older one and that this will affect developers if they don't download from the CENGN mirror as a source for those tarballs.

Frank Miller (sensfan22) on 2020-01-09
Changed in starlingx:
status: Incomplete → Triaged
importance: Undecided → Medium
assignee: nobody → Saul Wold (sgw-starlingx)
tags: added: stx.4.0

Fix proposed to branch: master
Review: https://review.opendev.org/702795

Changed in starlingx:
status: Triaged → In Progress
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers