-
Notifications
You must be signed in to change notification settings - Fork 81
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
Collect all images into one single txt file #558
Conversation
The all image list from master-head building 20230821
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only a nit discussion, others LGTM.
@w13915984028 Does the PR intend to publish the BTW, we do publish the image list tarball. e.g., https://releases.rancher.com/harvester/v1.1.2/images-list.tar.gz |
@bk201 Added a new PR in Harvester to publish the harvester-images-list.txt, thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Nice enhancement.
wondering if we know where |
@guangbochen , yes, the
let me check why the image list is not updated. |
We bumped all related event-router images in #545 The issue seems related to my local branch Is re-building the ISO and will update the image list soon. |
Signed-off-by: Jian Wang <[email protected]>
@guangbochen The newly generated image list file, should be correct. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
@mergify backport v1.2 |
✅ Backports have been created
|
Problem:
Harvester puts all container image names into a tarball
image-lists.tar.gz
, and this file is not published.It is not very convenient to check if an image is packed into ISO file.
Solution:
Generate an additional single txt file, which includes all images in the ISO, and
also publish this file
.Related Issue:
harvester/harvester#4176
Test plan:
image-all.txt
is existing in the pathdist/artifacts