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: support tenant #363

Merged
merged 6 commits into from
Nov 11, 2022
Merged

feat: support tenant #363

merged 6 commits into from
Nov 11, 2022

Conversation

lixd
Copy link
Contributor

@lixd lixd commented Nov 10, 2022

What type of PR is this?

/kind feature

What this PR does / why we need it:

this is a huge commit, include #303#306#326#341#354, and #341 are manual merged.

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:


lixd and others added 5 commits November 10, 2022 15:57
* feat: add project type and apis

Signed-off-by: lixd <xueduan.li@gmail.com>

* feat: add project controller

Signed-off-by: lixd <xueduan.li@gmail.com>

* fix: add project.status and add、remove node api

Signed-off-by: lixd <xueduan.li@gmail.com>

Signed-off-by: lixd <xueduan.li@gmail.com>
* feat: project user and role binding under project

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* add update rolebinding api and role operation api

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* third commmit

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* modify request path and return data type

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* add project check and the way to set label and annotation

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* modify response code

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* modify the way to handle API group

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* modify request body data

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* modify response code

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

* add delete role check

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>

Signed-off-by: xu.jingwen <xu.jingwen@99cloud.net>
* feat: update authorizer to support project scope

Signed-off-by: lixd <xueduan.li@gmail.com>

* fix: update roles name

Signed-off-by: lixd <xueduan.li@gmail.com>

Signed-off-by: lixd <xueduan.li@gmail.com>
move join node to project api to global scope
update permission,add some api

Signed-off-by: lixd <xueduan.li@gmail.com>
* fix: add global resource view api

Signed-off-by: lixd <xueduan.li@gmail.com>

* fix: update role module and api format

Signed-off-by: lixd <xueduan.li@gmail.com>

* fix: update role

Signed-off-by: lixd <xueduan.li@gmail.com>

* fix: role rename

Signed-off-by: lixd <xueduan.li@gmail.com>

Signed-off-by: lixd <xueduan.li@gmail.com>
@kubeclipper-bot kubeclipper-bot added release-note-none kind/feature Categorizes issue or PR as related to a new feature. dco-signoff: yes size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 10, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 10, 2022

Codecov Report

Merging #363 (9a7234a) into master (7ce88bf) will decrease coverage by 0.53%.
The diff coverage is 2.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #363      +/-   ##
==========================================
- Coverage   12.86%   12.33%   -0.54%     
==========================================
  Files         102      102              
  Lines       15738    16733     +995     
==========================================
+ Hits         2025     2064      +39     
- Misses      13471    14422     +951     
- Partials      242      247       +5     
Impacted Files Coverage Δ
pkg/apis/core/v1/registry.go 0.00% <0.00%> (ø)
pkg/models/iam/utils.go 0.00% <0.00%> (ø)
pkg/query/query.go 10.73% <0.00%> (-0.86%) ⬇️
pkg/apis/core/v1/handler.go 0.30% <0.56%> (-0.01%) ⬇️
pkg/models/iam/iam.go 7.76% <0.73%> (-2.29%) ⬇️
pkg/authorization/rbac/rbac.go 53.44% <2.22%> (-17.87%) ⬇️
pkg/server/request/request.go 80.00% <92.00%> (+5.00%) ⬆️
pkg/scheme/scheme.go 100.00% <100.00%> (ø)
pkg/component/nfs/nfs.go 0.57% <0.00%> (+<0.01%) ⬆️
... and 3 more

@lixd lixd force-pushed the master branch 2 times, most recently from 28586b8 to 0af0b63 Compare November 10, 2022 10:01
@lixd
Copy link
Contributor Author

lixd commented Nov 11, 2022

/cc @x893675

Signed-off-by: lixd <xueduan.li@gmail.com>
@x893675
Copy link
Collaborator

x893675 commented Nov 11, 2022

/lgtm
/approve

@kubeclipper-bot kubeclipper-bot added the lgtm Indicates that a PR is ready to be merged. label Nov 11, 2022
@kubeclipper-bot
Copy link
Collaborator

LGTM label has been added.

Git tree hash: b8ca7e4d5fd1f1441d951af869d3f7c1ad676882

@kubeclipper-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lixd, x893675

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubeclipper-bot kubeclipper-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 11, 2022
@kubeclipper-bot kubeclipper-bot merged commit f8dad9f into kubeclipper:master Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note-none size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants