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

CalcJob: Add support for nested targets in remote_symlink_list #5974

Merged

Conversation

sphuber
Copy link
Contributor

@sphuber sphuber commented Apr 18, 2023

Fixes #4726

It is now possible to specify a target in the remote_symlink_list that contains nested directories that do not necessarily exist. The upload_calculation will automatically create them before creating the symlink.

@mbercx
Copy link
Member

mbercx commented Apr 18, 2023

Ha, nice! I was actually just running into this issue when testing aiidateam/aiida-quantumespresso#918. Let me give this a spin in the field.

@sphuber
Copy link
Contributor Author

sphuber commented Sep 6, 2023

Any updates on this @mbercx ?

@mbercx
Copy link
Member

mbercx commented Sep 7, 2023

Haven't managed to test it yet. Can I rebase the branch? My storage in the project I can test this in has already been migrated to main_0002.

It is now possible to specify a target in the `remote_symlink_list` that
contains nested directories that do not necessarily exist. The
`upload_calculation` will automatically create them before creating the
symlink.
@sphuber sphuber force-pushed the feature/4726/nested-remote-symlinks branch from b05d89b to bc391fb Compare September 7, 2023 09:07
@sphuber
Copy link
Contributor Author

sphuber commented Sep 7, 2023

Haven't managed to test it yet. Can I rebase the branch? My storage in the project I can test this in has already been migrated to main_0002.

done

@mbercx mbercx self-requested a review September 7, 2023 12:47
Copy link
Member

@mbercx mbercx left a comment

Choose a reason for hiding this comment

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

Tested in the field, works like a charm. Thanks @sphuber!

@sphuber sphuber merged commit 0ec650c into aiidateam:main Sep 7, 2023
@sphuber sphuber deleted the feature/4726/nested-remote-symlinks branch September 7, 2023 13:00
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.

Allow nested remote symlinks
2 participants