chore(deps): update dependency containers/automation_images to v20230726#18612
chore(deps): update dependency containers/automation_images to v20230726#18612openshift-merge-robot merged 7 commits intomainfrom
Conversation
|
One test is failing, https://api.cirrus-ci.com/v1/artifact/task/4968184671895552/html/int-podman-fedora-38-root-host-boltdb.log.html#t--podman-checkpoint-container-with-export--migration--and-ipc-host--1 @giuseppe PTAL look like a possible crun bug in f38/rawhide, f37 and debian seems to work fine. |
|
Also selinux tests are failing on f38: https://api.cirrus-ci.com/v1/artifact/task/4528380020785152/html/sys-podman-fedora-38-root-host-sqlite.log.html |
|
Thanks Paul for checking & pinging. I managed to completely loose track of this PR 😞 |
|
Added to #18577 to see if this fixes both problems. |
10045a0 to
2dd5ff9
Compare
|
force-push: Attempted rebasing ontop of Dan's #18577 since it's CI went all green. |
|
@rhatdan FWIW: rebased with your PR, seems we get a handful of checkpoint and SELinux test failures. |
|
I don't see how #18577 is relevant for the test failures? |
Honestly I'm just throwing a bucket of golf-balls and hoping one lands in the hole. I read Dan's comment above and thought I'd give it a try. I should probably pull it out and rebase again, see if anything's changed from your assessment. |
2dd5ff9 to
28f4a2b
Compare
|
@giuseppe ping - Any feedback on Paul's observation of a possible crun bug in this |
|
are we using crun 1.8.5? There was an issue in crun 1.8.4 with CRIU on Fedora that could have caused this problem |
|
CI logs says |
|
Thanks @giuseppe and @Luap99. Darn, so that means |
|
...opened containers/automation_images#281 |
28f4a2b to
24f9b0b
Compare
|
Force-push: New CI VM images. |
24f9b0b to
0900704
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. ⚠ Warning: custom changes will be lost. |
0900704 to
cd9d243
Compare
|
force-push: Rebased. |
|
All system tests failing with: I did not look deeply, for now I'm assuming it's related to the fixes Dan's working on. |
74a68a5 to
4c93d14
Compare
|
Force-push: I've reached my limit on this PR, and lack of F38 + updated rawhide testing at-large is hampering ongoing development and pending/imminent releases. Added skips for all persistently failing tests, with "TODO/FIXME" comments and references where possible. @edsantiago IIRC, you have a preferred format for these kinds of skip messages. Please advise how you'd like them to appear. This push is to see if we can even get a clean-run at this point (w/o new images). |
edsantiago
left a comment
There was a problem hiding this comment.
Yes, just skip. Two requests inline, neither is a blocker.
|
You also need to skip this test in rawhide, same BZ: podman/test/system/280-update.bats Line 19 in 608f484 |
95124bc to
20e31bc
Compare
Ref: #19407 Signed-off-by: Chris Evich <cevich@redhat.com>
Ref: #19471 Signed-off-by: Chris Evich <cevich@redhat.com>
Requires currently unavailable 0.0~git20230625.32660ce-1 or later. Signed-off-by: Chris Evich <cevich@redhat.com>
Also update Fedora and Debian names. Include ID of broken images in `validate_task` comment about broken rawhide. This should get picked up by renovate next time images are updated - hopefully prompting someone to re-enable the check. Signed-off-by: Renovate Bot <bot@renovateapp.com> Signed-off-by: Chris Evich <cevich@redhat.com>
20e31bc to
d74c225
Compare
|
Force-push: 43rd time's the charm? |
|
@containers/podman-maintainers PTAL. This isn't perfect, I've had to add some test skips and workarounds. But for months worth of trying, this is the best I can get it. The skips/workarounds can be addressed by followup PRs. Assuming it's good, let's get this in so I can backport it to the v4.6 branch (unless there's a reason not to?). |
Behold, for 'tis my word. Debian unstable (sid) package is now updated to 0.0~git20230627.289301b-1. |
edsantiago
left a comment
There was a problem hiding this comment.
/lgtm
/hold
I would like to merge this ASAP, then deal with pasta-debian later. Whoever agrees, release the krakhold.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cevich, edsantiago, renovate[bot] The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/hold cancel |
This PR contains the following updates:
20230426t140447z-f38f37d12->20230517t144652z-f38f37d12Release Notes
containers/automation_images
v20230517t144652z-f38f37d12Compare Source
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Never, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.