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

shell: add shell.post-init plugin calllback topic between shell.init and first task.init #5719

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

grondo
Copy link
Contributor

@grondo grondo commented Feb 5, 2024

This simple PR just adds a new shell plugin callback topic: shell.post-init. This will be convenient for plugins that want to run after all other plugin shell.init callbacks have run, but before the first task.init callback (which is supposed to be task-specific).

It will be used in the solution for #5718.

Copy link
Member

@garlick garlick left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link

codecov bot commented Feb 5, 2024

Codecov Report

Merging #5719 (d94d01b) into master (2cebb8f) will decrease coverage by 0.01%.
Report is 5 commits behind head on master.
The diff coverage is 75.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5719      +/-   ##
==========================================
- Coverage   83.46%   83.45%   -0.01%     
==========================================
  Files         486      486              
  Lines       82933    82937       +4     
==========================================
- Hits        69221    69218       -3     
- Misses      13712    13719       +7     
Files Coverage Δ
src/shell/shell.c 78.73% <75.00%> (-0.14%) ⬇️

... and 8 files with indirect coverage changes

@grondo
Copy link
Contributor Author

grondo commented Feb 5, 2024

Thanks! I set MWP.

Problem: The job shell does not issue any plugin callbacks between
shell.init and task.init for the first task, but this may be useful
if a plugin wants to take an action after all shell.init callbacks
have been run, but before the first task.init callback.

Add a shell.post-init callback. This runs after the shell barrier is
complete but before the shell begins launching tasks.
Problem: The new shell.post-init shell plugin callback topic is
not documented.

Add it to flux-shell(1).
@mergify mergify bot merged commit 96ff0f4 into flux-framework:master Feb 5, 2024
33 checks passed
@grondo grondo deleted the shell.post-init branch February 6, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants