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

[Subtask] Add adminManager #2759

Closed
Tracked by #2232
qqqttt123 opened this issue Apr 1, 2024 · 0 comments · Fixed by #2758
Closed
Tracked by #2232

[Subtask] Add adminManager #2759

qqqttt123 opened this issue Apr 1, 2024 · 0 comments · Fixed by #2758
Assignees
Labels
subtask Subtasks of umbrella issue

Comments

@qqqttt123
Copy link
Contributor

qqqttt123 commented Apr 1, 2024

Describe the subtask

Role in the Gravitino has 3 kinds: service admin, metalake admin, normal roles.
Service admin is responsible for creating metalake admin. It can't be changed by API. It is configured by the configuration.
Metalake admin is used for creating metalake. It can't be bind to any metalake.

Parent issue

#2232

@qqqttt123 qqqttt123 added the subtask Subtasks of umbrella issue label Apr 1, 2024
@qqqttt123 qqqttt123 mentioned this issue Apr 1, 2024
19 tasks
@qqqttt123 qqqttt123 self-assigned this Apr 1, 2024
@jerryshao jerryshao added this to the Gravitino 0.5.0 milestone Apr 1, 2024
jerryshao pushed a commit that referenced this issue Apr 7, 2024
### What changes were proposed in this pull request?
Role in the Gravitino has 3 kinds: service admins, metalake admins,
normal roles.
Service admins are responsible for creating metalake admins. They can't
be changed by API. It is a config option.
Metalake admin is used for creating metalake. It can't be bind to any
metalake. So it's hard to unify it with normal users.
This pull request add the management of service admins and metalake
admins.

### Why are the changes needed?

Fix: #2759 

### Does this PR introduce _any_ user-facing change?
Yes, I will add openapi and document.

### How was this patch tested?
New ut.

---------

Co-authored-by: Heng Qin <[email protected]>
unknowntpo pushed a commit to unknowntpo/gravitino that referenced this issue Apr 9, 2024
…apache#2758)

### What changes were proposed in this pull request?
Role in the Gravitino has 3 kinds: service admins, metalake admins,
normal roles.
Service admins are responsible for creating metalake admins. They can't
be changed by API. It is a config option.
Metalake admin is used for creating metalake. It can't be bind to any
metalake. So it's hard to unify it with normal users.
This pull request add the management of service admins and metalake
admins.

### Why are the changes needed?

Fix: apache#2759 

### Does this PR introduce _any_ user-facing change?
Yes, I will add openapi and document.

### How was this patch tested?
New ut.

---------

Co-authored-by: Heng Qin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
subtask Subtasks of umbrella issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants