Skip to content

Fix flake on machine cp e2e test#25619

Merged
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
l0rd:machine-cp-flake
Mar 19, 2025
Merged

Fix flake on machine cp e2e test#25619
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
l0rd:machine-cp-flake

Conversation

@l0rd
Copy link
Member

@l0rd l0rd commented Mar 18, 2025

Explicitly close file to avoid machine e2e test to fail on CI from time to time.

Fixes #25614

Does this PR introduce a user-facing change?

None

Explicitly close file to avoid machine e2e test
to fail on CI from time to time.

See for example this failed execution of the machine-wsl task:
https://api.cirrus-ci.com/v1/artifact/task/6697640798191616/html/machine-wsl-podman-windows-rootless-host-sqlite.log.html

Fix containers#25614

Signed-off-by: Mario Loriedo <mario.loriedo@gmail.com>
@openshift-ci openshift-ci bot added release-note-none approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 18, 2025
Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jakecorrenti jakecorrenti left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for fixing this!

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jakecorrenti, l0rd, Luap99

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

The pull request process is described here

Details 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

@Luap99
Copy link
Member

Luap99 commented Mar 19, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 19, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 2b0aef5 into containers:main Mar 19, 2025
49 checks passed
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Jun 18, 2025
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Jun 18, 2025
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. machine release-note-none

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: podman machine DeferCleanup callback error because file is being used by another process.

3 participants