Skip to content

test: Prevent bash TCO when keeping a mount point busy#17823

Merged
mvollmer merged 1 commit into
cockpit-project:mainfrom
mvollmer:test-no-bash-tco
Oct 19, 2022
Merged

test: Prevent bash TCO when keeping a mount point busy#17823
mvollmer merged 1 commit into
cockpit-project:mainfrom
mvollmer:test-no-bash-tco

Conversation

@mvollmer
Copy link
Copy Markdown
Member

Different versions of bash do tail call optimization in different scenarios, and the "Currently in use" pop-up would either show the bash process or not (depending on whether TCO was done). So let's prevent bash from doing TCO of the "sleep" call.

@mvollmer
Copy link
Copy Markdown
Member Author

The alternative would be to expect TCO always and update the pixel
test references. That's probably also feasible and less tricky.

@mvollmer
Copy link
Copy Markdown
Member Author

This is needed for the fedora-36 refresh in cockpit-project/bots#3955

Comment thread test/verify/check-storage-used Outdated
Different versions of bash do tail call optimization in different
scenarios, and the "Currently in use" pop-up would either show the
bash process or not (depending on whether TCO was done). So let's
prevent bash from doing TCO of the "sleep" call.
@mvollmer mvollmer requested a review from jelly October 18, 2022 09:19
@mvollmer mvollmer temporarily deployed to cockpit-dist October 18, 2022 09:22 Inactive
@mvollmer
Copy link
Copy Markdown
Member Author

TestNetworkingCheckpoints.testNoRollback is fixed in #17829

@mvollmer mvollmer merged commit 4db4991 into cockpit-project:main Oct 19, 2022
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.

2 participants