-
Notifications
You must be signed in to change notification settings - Fork 22
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
Added scp transport for hotstart #1088
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you want to take this out of draft and confirm for me it was working for you last time you used it, I'd be happy to merge it if you're ready for it.
Ok thank you. I think I made these changes as a conversation starter and never got to the testing phase. This is an important enabler, so this will be next on my readiness agenda. |
Thanks @jasonfleming - bear in mind if we just want to support it in this particular case, the I think the current implementation is fine. If we want to add |
752e328
to
fc31bc7
Compare
@jasonfleming just wondering what the state of this PR draft was |
Implemented but not tested. I just bumped the priority of the underlying issue to important non-critical |
We can discuss this after #1277 gets in. I've opted to not use a URI scheme for now, instead assuming more details: Since we lean heavily on
...
The command to get a file and pipe it to
|
The usual transport for hotstart files is https ... maybe if the ":" is missing then we assume the transport is https? |
Need to include a fix for #1002 for this to be useful. |
resolves #715 not tested yet
(edited by @wwlwpd below)
We discussed on 4/9/24, updating this PR:
scp://
tossh://
due to the upstream desire to deprecatedscp
http://
,https://
, orssh://
), look for the standardHOST:/path/to/file
; this can be done easily by splitting on the:
, withawk -F: '{print $1}'
andawk -F '{print $2}'
.These things to do will round out this support nicely. In the future, other protocols may become important (e.g.,
s3://
, etc); so we definitely want to be able to have a clear place to support them and a consistent way to support them across the ASGS toolset.