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

PW Pool: Reduce task-to-task corruption risk #199

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

cevich
Copy link
Member

@cevich cevich commented Jun 3, 2024

Previously instances would shutdown and auto-terminate if the controlling VM's SetupInstances.sh examined the remote worker log and found >= PW_MAX_TASKS logged. However after examining the production Cron.log, it was found that nowhere near this number of tasks is actually running during PW_MAX_HOURS. Cut the value in half to lower the risk of one/more tasks corrupting processes or the filesystem for other tasks.

Note: Eyeball average tasks before timed auto-shutdown was about 7

Previously instances would shutdown and auto-terminate if the
controlling VM's `SetupInstances.sh` examined the remote worker
log and found >= `PW_MAX_TASKS` logged.  However after examining the
production `Cron.log`, it was found that nowhere near this number of
tasks is actually running during `PW_MAX_HOURS`.  Cut the value in
half to lower the risk of one/more tasks corrupting processes or the
filesystem for other tasks.

Note: Eyeball average tasks before timed auto-shutdown was about 7

Signed-off-by: Chris Evich <[email protected]>
@cevich
Copy link
Member Author

cevich commented Jun 3, 2024

The cirrus-ci/build-push_test error Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist has nothing to do with the subject or content of this PR. Force-merging this because it's such a minor risk parameter change.

@cevich cevich merged commit ae7f68a into containers:main Jun 3, 2024
6 of 9 checks passed
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.

1 participant