Skip to content
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

pkg/tar: ignore EOPNOTSUPP from return value of Setxattr for Stable/Beta/Alpha #2

Merged
merged 1 commit into from
Dec 4, 2019

Conversation

dongsupark
Copy link
Member

@dongsupark dongsupark commented Oct 31, 2019

In some cases, docker service does not start, because torcx is not able to unpack tarball like the following error:

failed to unpack: unpacking \"/usr/share/torcx/store/docker:com.coreos.cl.torcx.tgz\": error extracting tar: operation not supported" image=docker reference=com.coreos.cl

This means Setxattr() returns EOPNOTSUPP, especially when SELinux is disabled on the host.
To work around the issue, let's ignore EOPNOTSUPP.

@dongsupark dongsupark added the bug Something isn't working label Oct 31, 2019
In some cases, docker service does not start, because torcx is
not able to unpack tarball like the following error:

```
failed to unpack: unpacking \"/usr/share/torcx/store/docker:com.coreos.cl.torcx.tgz\":
error extracting tar: operation not supported" image=docker reference=com.coreos.cl
```

This means `Setxattr()` returns EOPNOTSUPP, especially when SELinux
is disabled on the host. To work around the issue, let's ignore
EOPNOTSUPP.
Copy link
Member

@iaguis iaguis left a comment

Choose a reason for hiding this comment

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

LGTM.

I've built this, tested on a machine with SELINUX=disabled and it worked for me.

@dongsupark dongsupark force-pushed the dongsu/fix-setxattr-eopnotsupp branch from 758eed6 to fe37695 Compare October 31, 2019 17:29
@dongsupark dongsupark changed the title pkg/tar: ignore EOPNOTSUPP from return value of Setxattr pkg/tar: ignore EOPNOTSUPP from return value of Setxattr for Stable/Beta/Alpha Oct 31, 2019
@dongsupark dongsupark merged commit 6333bc8 into flatcar-master Dec 4, 2019
@dongsupark dongsupark deleted the dongsu/fix-setxattr-eopnotsupp branch December 4, 2019 11:36
dongsupark pushed a commit to flatcar-archive/coreos-overlay that referenced this pull request Dec 4, 2019
To fix the recent Setxattr issue in torcx, bump torcx to the
latest commit.

See also flatcar/torcx#2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants