Skip to content

Makeflow vine inputs shared - #4551

Merged
btovar merged 2 commits into
cooperative-computing-lab:masterfrom
btovar:makeflow-vine-inputs-shared
Oct 27, 2026
Merged

btovar merged 2 commits into
cooperative-computing-lab:masterfrom
btovar:makeflow-vine-inputs-shared

Conversation

@btovar

@btovar btovar commented Oct 27, 2026

Copy link
Copy Markdown
Member

First part of #4544. Transfer from shared filesystem if remote name is different.

Merge Checklist

The following items must be completed before PRs can be merged.
Check these off to verify you have completed all steps.

  • make test Run local tests prior to pushing.
  • make format Format source code to comply with lint policies. Note that some lint errors can only be resolved manually (e.g., Python)
  • make lint Run lint on source code prior to pushing.
  • Manual Update: Update the manual to reflect user-visible changes.
  • Type Labels: Select a github label for the type: bugfix, enhancement, etc.
  • Product Labels: Select a github label for the product: TaskVine, Makeflow, etc.
  • PR RTM: Mark your PR as ready to merge.

ZacKienzle2 and others added 2 commits October 27, 2026 11:40
The shared_fs hook drops every input and output whose outer name lies on a shared
filesystem. A file renamed for the task is not reachable under its inner name there.
The resource monitor executable is such a file: it is staged as cctools-monitor and
the wrapper runs ./cctools-monitor, so under -T vine with `--shared-fs` every monitored
task fails and its summary is missing.

Files whose inner name differs from the outer name are now kept in the transfer lists.
@btovar btovar added this to the 7.17.5 milestone Oct 27, 2026
@btovar
btovar merged commit 292ff53 into cooperative-computing-lab:master Oct 27, 2026
13 checks passed
@btovar
btovar deleted the makeflow-vine-inputs-shared branch October 27, 2026 16:37
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