Feature all assets from catalog upload #256
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This removes the need to toggle both the Published and Featured checkboxes for each of the (46) uploaded assets. Since uploading a curated catalog can only be done by admins, we assume that all the uploaded assets are to be published and featured. Resolves machine-learning-exchange#237 Signed-off-by: Christian Kadner <[email protected]>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ckadner The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/lgtm |
ckadner
added a commit
to ckadner/mlx
that referenced
this pull request
Dec 2, 2021
* Since we have /catalog/upload_from_url API now (machine-learning-exchange#259), we no longer need to keep a copy of the catalog_upload.json file under quickstart * Since all assets get published and featured automatically with catalog upload (machine-learning-exchange#256), there is no longer a need for the init_catalog.sh script either * Since the MLX API also creates the Pipelines table (3c10fda) we no longer need to mount the init_db.sql to the mysql service * Since the Docker Compose File v3+ spec no longer supports condition as an option of depends_on we no longer need the healthcheck on services, but instead have to use simple while-wait-loops in the commands of downstream services Related machine-learning-exchange#256 Related machine-learning-exchange#259 Related 3c10fda Related https://forums.docker.com/t/depends-on-version-3/29463 Signed-off-by: Christian Kadner <[email protected]>
Merged
mlx-bot
pushed a commit
that referenced
this pull request
Dec 3, 2021
* Since we have /catalog/upload_from_url API now (#259), we no longer need to keep a copy of the catalog_upload.json file under quickstart * Since all assets get published and featured automatically with catalog upload (#256), there is no longer a need for the init_catalog.sh script either * Since the MLX API also creates the Pipelines table (3c10fda) we no longer need to mount the init_db.sql to the mysql service * Since the Docker Compose File v3+ spec no longer supports condition as an option of depends_on we no longer need the healthcheck on services, but instead have to use simple while-wait-loops in the commands of downstream services Related #256 Related #259 Related 3c10fda Related https://forums.docker.com/t/depends-on-version-3/29463 Signed-off-by: Christian Kadner <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This removes the need to toggle both the
PublishedandFeaturedcheck-boxes for each of the (currently 46) uploaded assets.Since uploading a curated catalog can only be done by admins, we assume that all the uploaded assets are to be published and featured.
Resolves #237