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

log version control: diff from outside the workflow source dir listed #4220

Closed
oliver-sanders opened this issue May 18, 2021 · 2 comments · Fixed by #4423
Closed

log version control: diff from outside the workflow source dir listed #4220

oliver-sanders opened this issue May 18, 2021 · 2 comments · Fixed by #4423
Assignees
Labels
Milestone

Comments

@oliver-sanders
Copy link
Member

oliver-sanders commented May 18, 2021

The log/version/ files log changes for files that aren't in the workflow source dir.

For example if we have this directory structure:

myproject/
    src/
    ...
    test-workflow/
        bin/
        flow.cylc
        ...
    build/workflow/
        flow.cylc
        ...

And we install like so:

$ cylc install -c 'myproject/test-workflow'

We could see uncommitted diff from myproject/build-workflow listed in the log files. This is the case for git vc, haven't tested svn.

The rose-stem pattern often sees test workflows embedded in much larger projects, listing the diff of the whole project would be over the top for Cylc purposes. It could be useful for provenance, however, Cylc doesn't capture full information about its run environment so should this be captured via other means?

Question:

  • Should we restrict this to just show changes from within the workflow source directory? - Yes

Pull requests welcome!

@oliver-sanders oliver-sanders added small question Flag this as a question for the next Cylc project meeting. labels May 18, 2021
@oliver-sanders oliver-sanders added this to the cylc-8.0.0 milestone May 18, 2021
@hjoliver
Copy link
Member

Should we restrict this to just show changes from within the workflow source directory?

IMO yes. That's all that affects the running of the workflow in question (unless it accesses external files, which could just as well be anywhere, not just under an umbrella project source),

@oliver-sanders
Copy link
Member Author

I think so too, no one's screamed, removing the question label.

@oliver-sanders oliver-sanders removed the question Flag this as a question for the next Cylc project meeting. label Sep 1, 2021
@wxtim wxtim self-assigned this Sep 8, 2021
@hjoliver hjoliver modified the milestones: cylc-8.0rc1, cylc-8.0b3 Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants