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

Docker: folder for scripts run in container startup stages #6170

Merged
merged 2 commits into from
Nov 9, 2023

Conversation

unkcpz
Copy link
Member

@unkcpz unkcpz commented Nov 4, 2023

In order to simplify the implementation of using aiida-core image as the base image to construct customized images, the run-before-daemon-start and run-after-daemon-start script folders are created. The executables in these two folders will run before/after the aiida daemon start in the container respectively.

The standard linux run-parts tool is used to scan the folder and files are run in the lexical sort order (according to the C/POSIX locale character collation rules) of their names.

With this change, the customized image construction can be simplified, see aiidateam/aiida-quantumespresso@58e074a for the different before and after this change.

@unkcpz unkcpz force-pushed the fea/x/run-parts-s6-overlay-services branch from 559881d to 36bf9fa Compare November 4, 2023 22:43
Copy link

codecov bot commented Nov 4, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6123f52) 79.77% compared to head (ce2ef02) 79.77%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6170   +/-   ##
=======================================
  Coverage   79.77%   79.77%           
=======================================
  Files         547      547           
  Lines       40131    40131           
=======================================
  Hits        32010    32010           
  Misses       8121     8121           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@sphuber sphuber left a comment

Choose a reason for hiding this comment

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

Thanks @unkcpz

@sphuber sphuber merged commit 5fe6f4f into main Nov 9, 2023
34 checks passed
@sphuber sphuber deleted the fea/x/run-parts-s6-overlay-services branch November 9, 2023 12:34
sphuber pushed a commit that referenced this pull request Nov 14, 2023
…n start (#6170)

In order to simplify the implementation of using the `aiida-core` image
as the base for customized images, the `run-before-daemon-start` and
`run-after-daemon-start` script folders are created. Any executables in
these two folders will be executed before and after the AiiDA daemon is
started in the container, respectively.

The standard linux `run-parts` tool is used to scan these folders for
files, which are run in the lexical sort order of their names, according
to the C/POSIX locale character collation rules

Cherry-pick: 5fe6f4f
sphuber pushed a commit that referenced this pull request Nov 15, 2023
…n start (#6170)

In order to simplify the implementation of using the `aiida-core` image
as the base for customized images, the `run-before-daemon-start` and
`run-after-daemon-start` script folders are created. Any executables in
these two folders will be executed before and after the AiiDA daemon is
started in the container, respectively.

The standard linux `run-parts` tool is used to scan these folders for
files, which are run in the lexical sort order of their names, according
to the C/POSIX locale character collation rules

Cherry-pick: 5fe6f4f
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