Skip to content

Conversation

@RuoqingHe
Copy link
Member

Once we are releasing with more architectures (like riscv64), assets naming with image format will conflict with other architecture when using the same format. Append with kernel arch string to avoid possible confliction.

Signed-off-by: Ruoqing He heruoqing@iscas.ac.cn

@RuoqingHe RuoqingHe requested a review from rbradford March 28, 2025 02:32
Copy link
Member

@rbradford rbradford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea!

@RuoqingHe
Copy link
Member Author

Good idea!

I have a little question, should we name Image.gz to Image-arm64.gz or Image.gz-arm64, WDYT 🤔

@RuoqingHe RuoqingHe force-pushed the 2025-03-28-rename-assets-with-arch branch from c312680 to cb26ca9 Compare April 4, 2025 15:03
upload_url: ${{ steps.create_release.outputs.upload_url }}
asset_path: arch/arm64/boot/Image.gz
asset_name: Image.gz
asset_name: Image-arm64.gz
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using Image-<arch>.gz format here

@RuoqingHe RuoqingHe force-pushed the 2025-03-28-rename-assets-with-arch branch from cb26ca9 to becbeed Compare April 7, 2025 10:51
Once we are releasing with more architectures (like riscv64), assets
naming with image format will conflict with other architecture when
using the same format. Append with kernel arch string to avoid possible
confliction.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
@RuoqingHe RuoqingHe force-pushed the 2025-03-28-rename-assets-with-arch branch from becbeed to 132835c Compare April 7, 2025 10:59
@RuoqingHe RuoqingHe merged commit 4ef36ab into cloud-hypervisor:ch-6.12.8 Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants