search

Garden-AI / garden

https://garden-ai.readthedocs.io
MIT License
16 stars 4 forks source link

hotfix: use new endpoint uuid in `constants.py` #430

Closed OwenPriceSkelly closed 5 months ago

OwenPriceSkelly commented 5 months ago

Part of #428. Waiting to call #418 and #425 closed until I've got the helm chart cleaned up and documented etc in my other branch.

Overview

🪦 Here lies DLHub Endpoint. May he CrashLoopBackOff in eternal peace. 🪦

Discussion

Thursday is upon us, and sending anything to the dlhub endpoint just gets you: 

globus_sdk.exc.api.GlobusAPIError: 
('POST', 'https://compute.api.globus.org/v2/submit', 'Bearer', 403, 'ENDPOINT_ACCESS_FORBIDDEN', 'Unauthorized access to endpoint 86a47061-f3d9-44f0-90dc-56ddc642c000')

This PR just a) changes the default endpoint to the new one and b) pours one out 🫗 for the old.

What's not included

Since only members of the associated globus group can actually use the new endpoint, the rest of #428 is going to be totally docs/error message focused to point users toward the link to join the group.

In the meantime, I've sent Ryan Jacobs a group invite link directly and can send a followup once this change lands on pypi and he should update.

Testing

I've been frequently testing against both dlhub and the new endpoint for the past few days as I worked on getting it up and running, and as of today the new one is the less broken.

The new one is functional for both the one migrated dlhub model I've been using to test and the tutorial seedling. Still a couple of quirks, like the fact that a mysterious encountered unknown data fields while reading a result message: {'details'} warning now prints along with the results.

But we do get results!

To kick the tires, make sure you've joined this group and try running any entrypoint.

Documentation

not yet!

📚 Documentation preview 📚: https://garden-ai--430.org.readthedocs.build/en/430/

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.64%. Comparing base (cf5a849) to head (6012018).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #430 +/- ## ======================================= Coverage 76.64% 76.64% ======================================= Files 46 46 Lines 2834 2834 ======================================= Hits 2172 2172 Misses 662 662 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.