WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@kdp-cloud
Copy link
Collaborator

No description provided.

@kdp-cloud kdp-cloud marked this pull request as ready for review December 10, 2025 19:45
@kdp-cloud kdp-cloud requested a review from stuzart December 10, 2025 19:45
@fbacall
Copy link
Contributor

fbacall commented Dec 10, 2025

Can we not just add a healthcheck for the worker container?

@kdp-cloud
Copy link
Collaborator Author

Can we not just add a healthcheck for the worker container?

I added a healthcheck to a workers container but it did not affect the outcome of the workflow. It seems the seek container needs some time to pick up the workers.

@fbacall
Copy link
Contributor

fbacall commented Dec 11, 2025

I added a healthcheck to a workers container but it did not affect the outcome of the workflow. It seems the seek container needs some time to pick up the workers.

Probably due to this:

unless Seek::Docker.using_docker?
status.refresh
end

seek/config/schedule.rb

Lines 66 to 68 in 5e8b151

every 1.minute do
runner 'ApplicationStatus.instance.refresh'
end

@kdp-cloud kdp-cloud self-assigned this Dec 11, 2025
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.

3 participants