Skip to content

Conversation

hsiangkao
Copy link
Contributor

@hsiangkao hsiangkao commented May 24, 2025

Currently, Kata EROFS support needs it, otherwise it will:

[    0.564610] erofs: (device sda): mounted with root inode @ nid 36.
[    0.564858] overlayfs: failed to set xattr on upper
[    0.564859] overlayfs: ...falling back to index=off,metacopy=off.
[    0.564860] overlayfs: ...falling back to xino=off.

since currently upperdir is on the /run filesystem, which is a tmpfs.

Currently, Kata EROFS support needs it, otherwise it will:
[    0.564610] erofs: (device sda): mounted with root inode @ nid 36.
[    0.564858] overlayfs: failed to set xattr on upper
[    0.564859] overlayfs: ...falling back to index=off,metacopy=off.
[    0.564860] overlayfs: ...falling back to xino=off.

Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>
@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label May 24, 2025
Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

tested this locally, and it solves the apt -y install $pkg failures I had.
Thanks @hsiangkao!

@fidencio
Copy link
Member

cc @ChengyuZhu6 @Champ-Goblem

Copy link
Member

@RuoqingHe RuoqingHe left a comment

Choose a reason for hiding this comment

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

I'm testing build on risc-v runners locally

Copy link
Member

@RuoqingHe RuoqingHe left a comment

Choose a reason for hiding this comment

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

Green light, good to go, thanks @hsiangkao ❤️

@RuoqingHe RuoqingHe merged commit b6cafba into kata-containers:main May 26, 2025
576 of 632 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants