Andy/multiple bundle location #3899
Conversation
…into andy/multiple-bundle-location
|
|
||
|
|
||
| @post('/bundles/<bundle_uuid:re:%s>/locations/', apply=AuthenticatedProtectedPlugin()) | ||
| def _add_bundle_location(bundle_uuid: str): |
There was a problem hiding this comment.
the final thing you should do is actually test these REST endpoints to make sure they work. You can run python codalab_service.py start -bd and then manually call the endpoints using postman -- see https://www.postman.com/ and https://buttercms.com/blog/postman-testing-the-api-development-environment
There was a problem hiding this comment.
Or if you don't want to use postman, you could instead create an automated test under rest1 in test_cli.py: https://github.com/codalab/codalab-worksheets/blob/master/tests/cli/test_cli.py#L2394. postman might be easier / quicker to do though
There was a problem hiding this comment.
@andyjin2000 how about let's just fix the documentation issues and then merge this PR for now? @jzwang43 can then test out the REST endpoints as he's using the CLI and then we can fix issues as needed.
|
Sounds good, I will update documentation as soon as I get a chance |
|
@jzwang43 now that these endpoints have been merged, you should be able to start working on the CLI now |
Reasons for making this change
Adding REST API endpoints to help support multiple bundle locations. Added 2 GET endpoints (one to fetch all BundleLocations associated with a given bundle, and another to fetch a specific BundleLocation), as well as 1 POST endpoint to add a new BundleLocation to a bundle
Design doc: https://docs.google.com/document/d/1TOsO4gnxUm1XMmlGUb7BJ4Z6y_olFbv9APbx0-IEy6M/edit#
Related issues
Fixes #3801, which is part of multi-bundle store megaissue #3799
Screenshots
Checklist