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

Create local conda-store dir for celery beat data files #902

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

soapy1
Copy link
Contributor

@soapy1 soapy1 commented Oct 19, 2024

Description

Previously, when running docker compose up -d the worker container would have some errors output because it is not able to create it's celery beat file. For example,

$ docker compose up 
. . .
conda-store-worker-1  | [2024-10-19 00:44:41,925: INFO/Beat] beat: Starting...
conda-store-worker-1  | [2024-10-19 00:44:41,952: ERROR/Beat] Removing corrupted schedule file '/.local/share/conda-store/celerybeat-schedule': FileNotFoundError(2, 'No such file or directory')
. . .

conda-store-worker-1  | [2024-10-19 00:44:41,967: WARNING/Beat] FileNotFoundError: [Errno 2] No such file or directory: b'/.local/share/conda-store/celerybeat-schedule.dat'
. . .

This PR creates the missing directory and gives the user permission to it so celery can create it's beat file. Now the output is:

$ docker compose up
. . .
conda-store-worker-1  | [2024-10-19 00:48:24,340: INFO/Beat] beat: Starting...
. . .

Pull request checklist

  • Did you test this change locally?
  • Did you update the documentation (if required)?
  • Did you add/update relevant tests for this change (if required)?

Copy link

netlify bot commented Oct 19, 2024

Deploy Preview for conda-store canceled.

Name Link
🔨 Latest commit 17eeeb6
🔍 Latest deploy log https://app.netlify.com/sites/conda-store/deploys/6713034440cdac0008ef4346

Copy link

netlify bot commented Oct 19, 2024

Deploy Preview for conda-store canceled.

Name Link
🔨 Latest commit 0bc82b6
🔍 Latest deploy log https://app.netlify.com/sites/conda-store/deploys/67130350dc4c840008b33a29

Copy link
Collaborator

@trallard trallard left a comment

Choose a reason for hiding this comment

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

Thank you, this has been annoying, but nobody got around to fix it until now.

@trallard trallard merged commit dc1134f into conda-incubator:main Oct 21, 2024
27 checks passed
@soapy1 soapy1 deleted the celerey-beat-dir branch October 21, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done 💪🏾
Development

Successfully merging this pull request may close these issues.

2 participants