Skip to content

feat(BA-528): Update GQL user mutation to allow set uid/gid #3352

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

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Jan 2, 2025

resolves #3478 (BA-528)

mutation ModifyUser {
    modify_user (
        email: "[email protected]",
        props: {
            container_uid: 1101,
            container_main_gid: 1102,
            container_gids: [1103,1104]
        }
    ){
        ok
        msg
        user {
            username
            container_uid
            container_main_gid
            container_gids
        }
    }
}

The fields are described as follows:

  • container_uid: The user ID (UID) assigned to processes running inside the container. Integer type, nullable.
  • container_main_gid: The primary group ID (GID) assigned to processes running inside the container. Integer type, nullable.
  • container_gids: Supplementary group IDs assigned to processes running inside the container. Array of integers, nullable.

When connecting to a container created with container_uid = 1100, container_main_gid = 1101, and container_gids = [1102, 1103], the id values should be displayed as follows:
image

Checklist: (if applicable)

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

📚 Documentation preview 📚: https://sorna--3352.org.readthedocs.build/en/3352/


📚 Documentation preview 📚: https://sorna-ko--3352.org.readthedocs.build/ko/3352/

Copy link
Member Author

fregataa commented Jan 2, 2025

@fregataa fregataa changed the title feat: Update GQL user mutation to allow set uid/gid feat(BA-99): Update GQL user mutation to allow set uid/gid Jan 2, 2025
@fregataa fregataa added this to the 25Q1 milestone Jan 2, 2025
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 3010ad3 to 61b7dc9 Compare January 2, 2025 10:24
@github-actions github-actions bot added size:S 10~30 LoC and removed size:XS ~10 LoC labels Jan 2, 2025
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from c463435 to 25a6684 Compare January 2, 2025 10:26
@github-actions github-actions bot added comp:agent Related to Agent component comp:common Related to Common component require:db-migration Automatically set when alembic migrations are added or updated size:L 100~500 LoC and removed size:S 10~30 LoC labels Jan 2, 2025
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 1f0c0b5 to 3aa27eb Compare January 2, 2025 10:27
@github-actions github-actions bot added size:S 10~30 LoC size:M 30~100 LoC and removed size:L 100~500 LoC size:S 10~30 LoC labels Jan 2, 2025
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 332cd9f to 2e76a36 Compare January 2, 2025 10:43
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from 25a6684 to 8687663 Compare January 3, 2025 07:05
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 87dd4b6 to f6442af Compare January 3, 2025 07:06
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from 8687663 to e4b9ddd Compare January 3, 2025 12:04
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from f6442af to 553894d Compare January 3, 2025 12:04
@github-actions github-actions bot added size:L 100~500 LoC and removed size:M 30~100 LoC labels Jan 3, 2025
@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
@github-actions github-actions bot added the area:docs Documentations label Jan 17, 2025
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from d21854a to ace1096 Compare January 21, 2025 02:44
@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/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from ace1096 to 3238c46 Compare January 21, 2025 08:26
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from e47a61f to 97bf1d8 Compare January 22, 2025 12:21
@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 changed the title feat: Update GQL user mutation to allow set uid/gid feat(BA-528): Update GQL user mutation to allow set uid/gid Jan 23, 2025
@fregataa fregataa requested a review from agatha197 January 23, 2025 07:09
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from 97bf1d8 to 08c0f58 Compare January 24, 2025 16:37
@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/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from 08c0f58 to 2dbf4aa Compare February 3, 2025 14:48
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 46ba1b0 to 23bca21 Compare February 3, 2025 14:49
@fregataa fregataa force-pushed the topic/12-20-feat_enable_per-user_uid_gid_set_for_containers branch from 2dbf4aa to 6a15d5b Compare February 5, 2025 06:18
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 23bca21 to 675bbaf Compare February 5, 2025 09:05
Copy link
Collaborator

@HyeockJinKim HyeockJinKim left a comment

Choose a reason for hiding this comment

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

lgtm

@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 675bbaf to 4c18c69 Compare February 5, 2025 09:20
Base automatically changed from topic/12-20-feat_enable_per-user_uid_gid_set_for_containers to main February 5, 2025 09:43
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch 2 times, most recently from 7047355 to 6c4b571 Compare February 5, 2025 10:08
@fregataa fregataa force-pushed the topic/01-02-feat_add_per-user_uid_gid_apis branch from 6c4b571 to 8a23986 Compare February 5, 2025 10:14
@fregataa fregataa added this pull request to the merge queue Feb 5, 2025
Merged via the queue into main with commit 68374b6 Feb 5, 2025
25 checks passed
@fregataa fregataa deleted the topic/01-02-feat_add_per-user_uid_gid_apis branch February 5, 2025 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:docs Documentations comp:agent Related to Agent component comp:common Related to Common component comp:manager Related to Manager component require:db-migration Automatically set when alembic migrations are added or updated size:L 100~500 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add UID/GID fields to user creation and update APIs
2 participants