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

Change LCRC endpoints #319

Merged
merged 1 commit into from
Nov 30, 2023
Merged

Change LCRC endpoints #319

merged 1 commit into from
Nov 30, 2023

Conversation

forsyth2
Copy link
Collaborator

Change LCRC endpoints. Resolves #318.

@forsyth2 forsyth2 added the semver: bug Bug fix (will increment patch version) label Nov 30, 2023
@forsyth2 forsyth2 self-assigned this Nov 30, 2023
@forsyth2 forsyth2 merged commit bed55cc into main Nov 30, 2023
3 checks passed
@forsyth2 forsyth2 deleted the issue-318-endpoints branch November 30, 2023 19:01
@forsyth2
Copy link
Collaborator Author

forsyth2 commented Dec 8, 2023

In testing for the next patch release (to be included in Unified 1.9.2), I found that the Globus test hung on Chrysalis. I was able to get it working, using the following steps.

$ mkdir pre_unified_test_20231207
$ cd pre_unified_test_20231207
$ mkdir zstash_demo
$ echo 'file0 stuff' > zstash_demo/file0.txt
# The following endpoint is the NERSC HPSS endpoint.
$ zstash create --hpss=globus://9cd89cfd-6d04-11e5-ba46-22000b92c6ec/~/zstash_test_20231207 zstash_demo
### I had to copy the auth code. I suppose the unit tests never got around the issue of the
### auth code request not showing up in the output.
### Note that the auth code link required authentications to BOTH LCRC and NERSC on Globus.
$ mkdir zstash_extraction
$ cd zstash_extraction
$ zstash extract --hpss=globus://9cd89cfd-6d04-11e5-ba46-22000b92c6ec/~/zstash_test_20231207
# This was successful, so clearly the new LCRC endpoints are working.

$ cd <zstash directory>
$ python -m unittest tests/test_globus.py
### Passes. The auth code must have been the cause of the hang.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: bug Bug fix (will increment patch version)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Replace all expired Globus endpoints
1 participant