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

feat: update CLI to support controlling extra_mounts parameter #2169

Merged
merged 1 commit into from
May 28, 2024

Conversation

kyujin-cho
Copy link
Member

@kyujin-cho kyujin-cho commented May 22, 2024

This PR updates CLI to be aware of newly added extra_mounts parameter when creating or updating model service.

Copy link

graphite-app bot commented May 22, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@github-actions github-actions bot added comp:client Related to Client component comp:cli Related to CLI component size:M 30~100 LoC labels May 22, 2024
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 40a3300 to d82ba12 Compare May 22, 2024 09:20
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from d82ba12 to ace548f Compare May 22, 2024 09:26
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from ace548f to 4955e6b Compare May 22, 2024 09:27
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 4955e6b to 80f6bfd Compare May 22, 2024 09:28
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 80f6bfd to d62f374 Compare May 22, 2024 09:34
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from d62f374 to 3a077a1 Compare May 24, 2024 06:47
@kyujin-cho kyujin-cho added the skip:changelog Make the action workflow to skip towncrier check label May 24, 2024 — with Graphite App
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 3a077a1 to dc4935e Compare May 24, 2024 06:54
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from dc4935e to 4c077bc Compare May 28, 2024 02:27
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 073d34a to 2eb7e3c Compare May 28, 2024 02:33
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 2eb7e3c to d62a667 Compare May 28, 2024 02:55
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from d62a667 to 2f1f74b Compare May 28, 2024 03:09
@kyujin-cho kyujin-cho changed the title feat: update CLI to support passing extra_mounts flag feat: update CLI to support controlling extra_mounts parameter May 28, 2024
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 2f1f74b to 8a1ade8 Compare May 28, 2024 03:15
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 8a1ade8 to 91482ff Compare May 28, 2024 03:20
@kyujin-cho kyujin-cho force-pushed the topic/update-cli branch 3 times, most recently from f199977 to e0fbc89 Compare May 28, 2024 03:45
@kyujin-cho kyujin-cho added this to the 24.03 milestone May 28, 2024
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 8a57e67 to 76fa1ea Compare May 28, 2024 03:59
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 622ab57 to 8cfe0f1 Compare May 28, 2024 05:06
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 8cfe0f1 to 132bd3d Compare May 28, 2024 05:21
Copy link

graphite-app bot commented May 28, 2024

Merge activity

This PR updates CLI to be aware of newly added `extra_mounts` parameter when creating or updating model service.
@kyujin-cho kyujin-cho force-pushed the topic/add-support-for-extra-mounts branch from 132bd3d to 1b81caa Compare May 28, 2024 05:40
@kyujin-cho kyujin-cho changed the base branch from topic/add-support-for-extra-mounts to main May 28, 2024 05:48
@graphite-app graphite-app bot merged commit be3a305 into main May 28, 2024
28 checks passed
@graphite-app graphite-app bot deleted the topic/update-cli branch May 28, 2024 05:49
lablup-octodog pushed a commit that referenced this pull request May 28, 2024
This PR updates CLI to be aware of newly added  parameter when creating or updating model service.

Backported-from: main (24.09)
Backported-to: 24.03
Backport-of: 2169
kyujin-cho added a commit that referenced this pull request May 28, 2024
Co-authored-by: Kyujin Cho <kyujin.cho@lablup.com>
Backported-from: main (24.09)
Backported-to: 24.03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:cli Related to CLI component comp:client Related to Client component size:M 30~100 LoC skip:changelog Make the action workflow to skip towncrier check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant