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

ci: update Dockerfile to fix broken tests #176

Merged
merged 3 commits into from
Sep 29, 2017

Conversation

cyphar
Copy link
Member

@cyphar cyphar commented Sep 21, 2017

Due to some weird py3k issue that has manifested recently, our CI
started breaking due to errors within xattr. Switch to a newer version
(since it looks like the python3-* version is not maintained anymore in
the py3k repo).

Signed-off-by: Aleksa Sarai asarai@suse.de

@cyphar
Copy link
Member Author

cyphar commented Sep 29, 2017

xattr breakage is upstream (tracked in vbatts/go-mtree#140).

This requires backporting a fix for a bug introduced in the new version.

Signed-off-by: Aleksa Sarai <asarai@suse.de>
Signed-off-by: Aleksa Sarai <asarai@suse.de>
Due to some weird py3k issue that has manifested recently, our CI
started breaking due to errors within xattr. Switch to a newer version
(since it looks like the python3-* version is not maintained anymore in
the py3k repo).

Signed-off-by: Aleksa Sarai <asarai@suse.de>
@cyphar
Copy link
Member Author

cyphar commented Sep 29, 2017

LGTM.

@cyphar cyphar merged commit c33d421 into opencontainers:master Sep 29, 2017
cyphar added a commit that referenced this pull request Sep 29, 2017
  ci: update Dockerfile to fix broken tests
  vendor: update to github.com/urfave/cli@v1.20.0
  vendor: update to github.com/vbatts/go-mtree@v0.4.0

LGTMs: @cyphar
Closes #176
@cyphar cyphar mentioned this pull request Sep 29, 2017
@cyphar cyphar deleted the ci-fix-dockerfile branch October 1, 2017 22:10
@cyphar
Copy link
Member Author

cyphar commented Oct 2, 2017

This also includes a backport of opencontainers/runtime-tools#491, which is necessary because the current validator violates the spec.

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.

1 participant