EarthSystemCoG / COG

COG source code
BSD 3-Clause "New" or "Revised" License
8 stars 16 forks source link

Update Globus downloads to use Globus SDK #1422

Closed mauzey1 closed 4 years ago

mauzey1 commented 4 years ago

This change will replace the old Globus transfer API with Globlus SDK. It also removes the X.509 endpoint activation section in the activateEndpoint fucntion that didn't appear to be in use in CoG.

sashakames commented 4 years ago

Thanks, Lets ensure testing is successful on this before merge into devel so no need to back out changes.

nathanlcarlson commented 4 years ago

This has successfully completed a transfer @sashakames . I am going to approve.

sashakames commented 4 years ago

Great! @nathanlcarlson on what server were you able to test? I thought on esgf-fedtest was registered (or did you register another client)?

mauzey1 commented 4 years ago

@sashakames I tested the Globus transfer by running CoG on my RHEL 7 workstation.

sashakames commented 4 years ago

Thanks @mauzey1! Nathan explained the peculiarities of testing from your desktop. I'm inclined to attempt a test on esgf-fedtest prior to a merge and will try to do so Monday next week if not later today.

sashakames commented 4 years ago

On esgf-fedtest the selected datasets aren't picked up by the transfer view. Will need to look into whats going on there. Can't imagine that complicated a problem.

mauzey1 commented 4 years ago

@sashakames @nathanlcarlson Are there Globus-enabled datasets on esgf-fedtest? I don't see any on there.

sashakames commented 4 years ago

https://esgf-fedtest.llnl.gov/search/esgf-llnl-dev/ - search for data_node aims3 or esgf-data1

mauzey1 commented 4 years ago

I'm getting a 504 error with that link.

sashakames commented 4 years ago

Seems to be working when I opened a private firefox sesssion. Might be some cookie issue as I saw the same issue, and don't quite understand the error in the log at first glance.

sashakames commented 4 years ago

Apologies that was just the project view page. The search page is timing out now and not sure why.

nathanlcarlson commented 4 years ago

Going to use #1423