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(collaboration): support for configuring database administrators and participating in approvals #2168

Merged
merged 8 commits into from
Apr 12, 2024

Conversation

smallsheeeep
Copy link
Contributor

What type of PR is this?

type-feature
module-collaboration

What this PR does / why we need it:

In common usage, users wanna that the tickets related to different databases are approved by different user. Which means that the approvers is related to the database.
This PR implements these features:

  1. Support configuring database adminiatrators;
  2. Support make the database adminiatrators as the approvers of the tickets.
    by:
  3. In DatabaseController, add a new API modifyDatabasesOwner for configuring database adminiatrators;
  4. In V_4_3_0_1__add_resource_role.yaml, add OWNER of ODC_DATABASE;
  5. Add some new template variables for integration, including:
    SQL_CONTENT_JSON_ARRAY("sql.content.json.array")
    DATABASE_OWNERS_IDS("database.owner.ids")
    DATABASE_OWNERS_ACCOUNTS("database.owner.accounts")
    DATABASE_OWNERS_NAMES("database.owner.names")
    DATABASE_NAME("database.name")
    ENVIRONMENT_NAME("environment.name")
    TASK_DESCRIPTION("task.description")
    RISK_LEVEL("risk.level")
    PROJECT_OWNER_IDS("project.owner.ids")
    PROJECT_OWNER_ACCOUNTS("project.owner.accounts")
    PROJECT_OWNER_NAMES("project.owner.names")
    ODC_TASK_URL("odc.task.url")

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:


ISADBA and others added 2 commits March 8, 2024 17:35
* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level

2024-02-29

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level

2024-02-29

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level

2024-02-29

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level
odc.task.url
3. Adjust the database permission request form, and split the form when applying for multiple databases.

2024-03-06

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level
odc.task.url
3. Adjust the database permission request form, and split the form when applying for multiple databases.

2024-03-06

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level
odc.task.url
3. Adjust the database permission request form, and split the form when applying for multiple databases.

2024-03-07

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level
odc.task.url
3. Adjust the database permission request form, and split the form when applying for multiple databases.

2024-03-08

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level
odc.task.url
3. Adjust the database permission request form, and split the form when applying for multiple databases.

2024-03-08

* feat(database-owner):
1. Add the owner role to the database, supporting approval workflow to use the owner role of the database.
2. Add variables for integrating external approval.
such as:
database.owner.accounts
database.owner.ids
database.owner.names
database.name
environment.name
task.description
risk.level
odc.task.url
3. Adjust the database permission request form, and split the form when applying for multiple databases.

2024-03-08
# Conflicts:
#	server/odc-service/src/main/java/com/oceanbase/odc/service/connection/database/DatabaseService.java
@smallsheeeep smallsheeeep added type-feature The functionality to be implemented priority-medium labels Apr 8, 2024
@smallsheeeep smallsheeeep added this to the ODC 4.3.0 milestone Apr 8, 2024
@smallsheeeep smallsheeeep changed the title feat(collaboration): Support for configuring database administrators and participating in approvals feat(collaboration): support for configuring database administrators and participating in approvals Apr 8, 2024
Copy link
Contributor

@yizhouxw yizhouxw 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
Collaborator

@ungreat ungreat 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
Collaborator

@krihy krihy left a comment

Choose a reason for hiding this comment

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

lgtm

suggest merge to 4.2.x after 424BP published

Copy link
Collaborator

@MarkPotato777 MarkPotato777 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
Contributor

@yhilmare yhilmare left a comment

Choose a reason for hiding this comment

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

lgtm

@yhilmare yhilmare merged commit cb13278 into dev/4.3.x Apr 12, 2024
11 checks passed
@yhilmare yhilmare deleted the feat/community/add_database_role branch April 12, 2024 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority-medium type-feature The functionality to be implemented
Projects
Status: Developed
Development

Successfully merging this pull request may close these issues.

8 participants