-
Notifications
You must be signed in to change notification settings - Fork 324
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] The image list packing part of the build script should be architecture-aware #5313
Comments
Pre Ready-For-Testing Checklist
➜ brandboat@suse /mnt/iso_image/bundle/harvester/images-lists-archive ls -la
total 26
drwxr-xr-x 1 root root 2048 Oct 9 17:56 .
drwxr-xr-x 1 root root 2048 Oct 9 17:56 ..
drwxr-xr-x 1 root root 2048 Oct 9 17:56 current
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.1.0
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.1.1
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.1.2
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.1.3
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.2.0
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.2.1
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.2.2
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.3.0
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.3.1
drwxr-xr-x 1 root root 2048 Oct 9 17:56 v1.3.2
|
Automation e2e test issue: harvester/tests#1185 |
milestone/release required. |
thanks @lanfon72 , milestone added, this one should be included in v1.4.0 and backport to v1.3 (it was merged, and I thought is in v1.3.1 release... harvester/harvester-installer#691) |
Test PASS, both AMD64 and ARM64 arch has image_list_all.txt for Environment
Test Steps |
Describe the bug
We will soon publish ARM64 artifacts for the v1.3.0 release. The build pipeline includes an image list packing stage, which packs the image lists for the current and previous versions (ref). It isn't arch-aware yet, so the results (the packed image lists) will be the same for both AMD64 and ARM64 built artifacts.
The packed image lists inside the built ISO will be referenced in the final stage of Harvester upgrades to purge unneeded container images. If the image lists contain wrong image names, the container images cannot be removed.
In addition, future releases will publish the image lists with
ARCH
as the name suffix. We need to adapt this to prevent build failure.To Reproduce
Steps to reproduce the behavior:
Expected behavior
Fetch the correct image list according to the target architecture and pack them into the ISO image.
Support bundle
N/A
Environment
Additional context
N/A
The text was updated successfully, but these errors were encountered: