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(BA-530): Update SDK and CLI to follow-up per-user uid/gid #3361

Open
wants to merge 3 commits into
base: topic/01-02-feat_add_per-user_uid_gid_apis
Choose a base branch
from

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Jan 3, 2025

resolves #3479 (BA-530)

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • API server-client counterparts (e.g., manager API -> client SDK)

Copy link
Member Author

fregataa commented Jan 3, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fregataa fregataa added this to the 25Q1 milestone Jan 3, 2025
@fregataa fregataa marked this pull request as ready for review January 3, 2025 13:09
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from e056ec6 to 7614158 Compare January 6, 2025 03:43
@fregataa fregataa force-pushed the topic/01-03-feat_update_sdk_and_cli_to_follow-up_per-user_uid_gid branch from 10fc52d to 9975779 Compare January 6, 2025 03:44
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 4f4e9a9 to e70f1cc Compare January 17, 2025 07:16
@fregataa fregataa force-pushed the topic/01-03-feat_update_sdk_and_cli_to_follow-up_per-user_uid_gid branch from 9975779 to cf418f0 Compare January 17, 2025 07:16
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from ca469e0 to 9660c3e Compare January 21, 2025 02:44
@fregataa fregataa force-pushed the topic/01-03-feat_update_sdk_and_cli_to_follow-up_per-user_uid_gid branch from cf418f0 to bb05310 Compare January 21, 2025 02:45
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 9660c3e to 9b5ca7f Compare January 22, 2025 12:21
@fregataa fregataa force-pushed the topic/01-03-feat_update_sdk_and_cli_to_follow-up_per-user_uid_gid branch 2 times, most recently from f1cece1 to 20f5b34 Compare January 24, 2025 06:26
@fregataa fregataa changed the title feat: Update SDK and CLI to follow-up per-user uid/gid feat(BA-530): Update SDK and CLI to follow-up per-user uid/gid Jan 24, 2025
@fregataa fregataa requested a review from HyeockJinKim January 24, 2025 16:21
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 33f328b to 46ba1b0 Compare January 24, 2025 16:38
@fregataa fregataa force-pushed the topic/01-03-feat_update_sdk_and_cli_to_follow-up_per-user_uid_gid branch from 20f5b34 to 6d21151 Compare January 24, 2025 16:38
help="Unset the user's container primary GID.",
)
@click.option(
"--cgids",
Copy link
Collaborator

Choose a reason for hiding this comment

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

I feel that abbreviations like cuid and cguid might overlap with other topics.
How about avoiding using abbreviations for now?

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:L 100~500 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants