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(key-management-service): add kms termination #11800

Merged
merged 1 commit into from
May 31, 2024

Conversation

darsene
Copy link
Contributor

@darsene darsene commented May 23, 2024

ref: MANAGER-13980

Question Answer
Branch? key-management-service
Bug fix? no
New feature? yes
Breaking change? no
Tickets Fix #MANAGER-13980
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

Related

@qpavy qpavy self-requested a review May 24, 2024 09:49
@darsene darsene force-pushed the feat/key-management-service branch from 21a156f to 78353c5 Compare May 24, 2024 11:52
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label May 24, 2024
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label May 24, 2024
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label May 27, 2024
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label May 27, 2024
@darsene darsene force-pushed the feat/key-management-service branch from ff4077c to 17cf3b7 Compare May 30, 2024 15:15
@github-actions github-actions bot added the has conflicts Has conflicts to resolve before merging label May 30, 2024
ref: MANAGER-13980

Signed-off-by: David Arsène <david.arsene.ext@ovhcloud.com>
@github-actions github-actions bot removed manager-components has conflicts Has conflicts to resolve before merging labels May 30, 2024
Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@darsene darsene merged commit 20c4e95 into feat/key-management-service May 31, 2024
11 of 12 checks passed
@darsene darsene deleted the feat/MANAGER-13980 branch May 31, 2024 14:43
vovh pushed a commit that referenced this pull request Jun 12, 2024
ref: MANAGER-13980

Signed-off-by: David Arsène <david.arsene.ext@ovhcloud.com>
vovh pushed a commit that referenced this pull request Jun 19, 2024
ref: MANAGER-13980

Signed-off-by: David Arsène <david.arsene.ext@ovhcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants