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

fix "podman system prune networks" flake #15994

Merged
merged 1 commit into from
Sep 29, 2022

Conversation

Luap99
Copy link
Member

@Luap99 Luap99 commented Sep 29, 2022

Since by default the network config dir is shared in the e2e tests any
other parallel running test could remove a network and cause this test to
fail.

Fixes #15990

Does this PR introduce a user-facing change?

None

Since by default the network config dir is shared in the e2e tests any
other parallel running test could remove a network and cause this test to
fail.

Fixes containers#15990

Signed-off-by: Paul Holzinger <[email protected]>
@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 Sep 29, 2022
@Luap99
Copy link
Member Author

Luap99 commented Sep 29, 2022

@edsantiago PTAL

Copy link
Member

@edsantiago edsantiago left a comment

Choose a reason for hiding this comment

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

Nice catch. Thank you for the quick work.

@rhatdan
Copy link
Member

rhatdan commented Sep 29, 2022

/approve
/lgtm
/hold

@openshift-ci openshift-ci bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged. labels Sep 29, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 29, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: edsantiago, Luap99, rhatdan

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:
  • OWNERS [Luap99,edsantiago,rhatdan]

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

@mheon
Copy link
Member

mheon commented Sep 29, 2022

Restarted one flake, changes LGTM

@edsantiago
Copy link
Member

I got to thinking: it takes two to tango, no? If this test is failing because some other test is stepping on it, shouldn't we see failures in that other test too? And shouldn't that other test be fixed? Unfortunately I can't find any clear indications in my flake logs, and I tried reading the test code, and that didn't go so well. Anyhow, just wondering if there's another place in the tests where we need to do the tmpdir thing.

@Luap99
Copy link
Member Author

Luap99 commented Sep 29, 2022

I got to thinking: it takes two to tango, no? If this test is failing because some other test is stepping on it, shouldn't we see failures in that other test too? And shouldn't that other test be fixed? Unfortunately I can't find any clear indications in my flake logs, and I tried reading the test code, and that didn't go so well. Anyhow, just wondering if there's another place in the tests where we need to do the tmpdir thing.

The problem is that in that test file many test do podman system prune which also removes networks, the other test doesn't care if it removes more than it should. Generally speaking it may be more future proof if all system prune/reset tests would use their own network config dir to prevent causing failures in other network tests.

@edsantiago
Copy link
Member

/hold cancel

thanks again

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 29, 2022
@openshift-merge-robot openshift-merge-robot merged commit 472e824 into containers:main Sep 29, 2022
@Luap99 Luap99 deleted the flake-15990 branch September 29, 2022 16:51
@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 20, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 20, 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. release-note-none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI Flake: podman system prune networks: expected string length 1
5 participants