Skip to content

Commit

Permalink
shell: fix signal plugin to work with Fluxion scheduler
Browse files Browse the repository at this point in the history
Problem: The shell signal plugin doesn't work with Fluxion because
it expects a float value for expiration in R and Fluxion encodes
an integer.

Update the format specifier in the signal plugin's use of json_unpack
on R to use `s:F` instead of `s:f` so that the plugin works with
Fluxion.

Fixes flux-framework#6521
  • Loading branch information
grondo committed Dec 18, 2024
1 parent 368b85a commit b4cd80c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/shell/signal.c
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ static int set_timeleft_watcher (struct shell_signal *sig)
double remaining;

if (flux_shell_info_unpack (sig->shell,
"{s:{s:{s:f}}}",
"{s:{s:{s:F}}}",
"R",
"execution",
"expiration", &expiration) < 0) {
Expand Down

0 comments on commit b4cd80c

Please sign in to comment.