Skip to content

feat: add option to set scaling_groups.use_host_network option from CLI#941

Merged
kyujin-cho merged 4 commits intomainfrom
feature/add-use-host-network-option
Jan 5, 2023
Merged

feat: add option to set scaling_groups.use_host_network option from CLI#941
kyujin-cho merged 4 commits intomainfrom
feature/add-use-host-network-option

Conversation

@agatha197
Copy link
Copy Markdown
Contributor

@agatha197 agatha197 commented Dec 21, 2022

follows #838

This PR enables CRUD the use_host_network option of scaling group in the client-py.
related PR: https://github.com/lablup/backend.ai-control-panel/pull/459

@agatha197 agatha197 added type:feature Add new features comp:manager Related to Manager component comp:client Related to Client component effort:easy Need to understand only a specific region of codes (good first issue, easy). labels Dec 21, 2022
@agatha197 agatha197 added this to the 22.09 milestone Dec 21, 2022
@agatha197 agatha197 self-assigned this Dec 21, 2022
@fregataa fregataa self-requested a review December 21, 2022 08:10
Copy link
Copy Markdown
Member

@fregataa fregataa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
I tested it and works well

Copy link
Copy Markdown
Member

@adrysn adrysn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Copy Markdown
Member

@kyujin-cho kyujin-cho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@kyujin-cho kyujin-cho merged commit eb20433 into main Jan 5, 2023
@kyujin-cho kyujin-cho deleted the feature/add-use-host-network-option branch January 5, 2023 08:31
@kyujin-cho kyujin-cho changed the title feat: corresponds to client-py to CRUD scaling group's use_host_network option feat: add option to set scaling_groups.use_host_network option from CLI (#941) Jan 5, 2023
@kyujin-cho kyujin-cho changed the title feat: add option to set scaling_groups.use_host_network option from CLI (#941) feat: add option to set scaling_groups.use_host_network option from CLI Jan 5, 2023
kyujin-cho pushed a commit that referenced this pull request Jan 10, 2023
… CLI (#941)

Co-authored-by: Joongi Kim <joongi@lablup.com>
Backported-from: main
Backported-to: 22.09
@Yaminyam Yaminyam added the size:M 30~100 LoC label Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:client Related to Client component comp:manager Related to Manager component effort:easy Need to understand only a specific region of codes (good first issue, easy). size:M 30~100 LoC type:feature Add new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants