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

vendor containers/common@main #10848

Merged
merged 1 commit into from
Jul 16, 2021

Conversation

vrothberg
Copy link
Member

The IgnorePlatform options has been removed from the
LookupImageOptions in libimage to properly support multi-arch images.

Signed-off-by: Valentin Rothberg [email protected]

@openshift-ci openshift-ci bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jul 2, 2021
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 4, 2021
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 6, 2021
@vrothberg vrothberg changed the title WIP - vendor containers/common@main vendor containers/common@main Jul 7, 2021
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 7, 2021
@vrothberg
Copy link
Member Author

Merged in c/common and c/buildah. Ready from my side.

@containers/podman-maintainers PTAL

@mheon
Copy link
Member

mheon commented Jul 8, 2021

Restarted one test, LGTM

@vrothberg
Copy link
Member Author

$GOSRC/$SCRIPT_BASE/setup_environment.sh
/tmp/scripts81855ad8681d0d86d1e91e00167939cb.sh: line 5: /var/tmp/go/src/github.com/containers/podman/./contrib/cirrus/setup_environment.sh: No such file or directory

2nd time in this PR. Really looks like a CI infra fart.

@vrothberg
Copy link
Member Author

Argh... the buds tests need massaging. I'll be out on Monday.

@edsantiago, do you know what's going on? In case not, please don't bother and I'll take a look on Tuesday morning.

@edsantiago
Copy link
Member

This is almost certainly fallout from containers/buildah#3292

TL;DR something changed in runc (I think?) and changed the debug log messages. The fix in the above PR seems to have fixed things for new-runc, but perhaps our podman CI systems are still using old-runc, and the fix breaks.

Solution: make a fix that handles both old-runc and new-runc. Or: bump up podman CI systems to use new-runc.

@cevich PTAL

@cevich
Copy link
Member

cevich commented Jul 13, 2021

Solution: make a fix that handles both old-runc and new-runc. Or: bump up podman CI systems to use new-runc.

and wait, there's more! I'm having trouble bringing in new images due to some deep/dark SELinux issue only Dan and Giuseppe are qualified to describe 😢

@cevich
Copy link
Member

cevich commented Jul 13, 2021

Update: Since the update here merged, maybe try rebasing and see if it helps?

@vrothberg
Copy link
Member Author

Update: Since the update here merged, maybe try rebasing and see if it helps?

Thank you! Just rebased.

@vrothberg
Copy link
Member Author

Still failing:

[+0766s] not ok 211 bud with --runtime and --runtime-flag
[+0766s] # (from function `expect_output' in file tests/helpers.bash, line 317,
[+0766s] #  in test file tests/bud.bats, line 2899)
[+0766s] #   `expect_output --substring "$flag_accepted_rx"' failed

@cevich
Copy link
Member

cevich commented Jul 14, 2021

I have those same new images for this repo, but it's failing horribly in CI 😢

The `IgnorePlatform` options has been removed from the
`LookupImageOptions` in libimage to properly support multi-arch images.

Skip one buildah-bud test which requires updated CI images.  This is
currently being done in github.com/containers/pull/10829 but
we need to unblock merging common and buildah into podman.

[NO TESTS NEEDED]

Signed-off-by: Valentin Rothberg <[email protected]>
@vrothberg
Copy link
Member Author

@containers/podman-maintainers @rhatdan please merge me. This will unblock merging c/common and buildah but note that the bud tests is now skipped until the CI images have been updated.

@rhatdan
Copy link
Member

rhatdan commented Jul 16, 2021

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 16, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 16, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rhatdan, vrothberg

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit 68585b0 into containers:main Jul 16, 2021
@vrothberg vrothberg deleted the update-libimage branch July 16, 2021 15:13
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 22, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants