8000 chore: inherit encrytion config from parent backup by gnolong · Pull Request #9419 · apecloud/kubeblocks · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: inherit encrytion config from parent backup #9419

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 5, 2025
Merged

chore: inherit encrytion config from parent backup #9419

merged 2 commits into from
Jun 5, 2025

Conversation

gnolong
Copy link
Contributor
@gnolong gnolong commented Jun 5, 2025 •
edited
Loading
  • inherit encrytion config from parent backup
  • validate parent backup's encrytion config

@gnolong gnolong requested review from ldming, wangyelei, zjx20 and a team as code owners June 5, 2025 07:42
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines. label Jun 5, 2025
@codecov Codecov
Copy link
codecov bot commented Jun 5, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 64.28571% with 10 lines in your changes missing coverage. Please review.

Project coverage is 60.11%. Comparing base (8904667) to head (06fa16a).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
controllers/dataprotection/utils.go 54.54% 7 Missing and 3 partials ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #9419      +/-   ##
==========================================
+ Coverage   60.09%   60.11%   +0.02%     
==========================================
  Files         391      391              
  Lines       47752    47771      +19     
==========================================
+ Hits        28697    28718      +21     
- Misses      16248    16261      +13     
+ Partials     2807     2792      -15
Flag Coverage Δ
unittests 60.11% <64.28%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@apecloud-bot apecloud-bot added the approved PR Approved Test label Jun 5, 2025
@gnolong gnolong merged commit 36b3722 into main Jun 5, 2025
41 checks passed
@gnolong gnolong deleted the bugfix/backup-encrytion-config-alignment branch June 5, 2025 09:08
@github-actions github-actions bot added this to the Release 1.0.0 milestone Jun 5, 2025
@gnolong
Copy link
Contributor Author
gnolong commented Jun 5, 2025

/cherry-pick release-0.9

@gnolong
Copy link
Contributor Author
gnolong commented Jun 5, 2025

/cherry-pick release-1.0

apecloud-bot pushed a commit that referenced this pull request Jun 5, 2025
@gnolong
chore: inherit encrytion config from parent backup (#9419)
074e66f 
(cherry picked from commit 36b3722)
@apecloud-bot
Copy link
Collaborator

🤖 says: cherry pick action finished successfully 🎉!
See: https://github.com/apecloud/kubeblocks/actions/runs/15463112431

@apecloud-bot
Copy link
Collaborator

🤖 says: cherry pick action finished successfully 🎉!
See: https://github.com/apecloud/kubeblocks/actions/runs/15463119657

apecloud-bot pushed a commit that referenced this pull request Jun 5, 2025
@gnolong
chore: inherit encrytion config from parent backup (#9419)
eb0f8b4 
(cherry picked from commit 36b3722)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zjx20 zjx20 zjx20 approved these changes

@wangyelei wangyelei wangyelei approved these changes

@ldming ldming Awaiting requested review from ldming ldming is a code owner

Assignees
No one assigned
Labels
approved PR Approved Test size/L Denotes a PR that changes 100-499 lines.
Projects
None yet
Milestone
Release 1.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@gnolong @apecloud-bot @zjx20 @wangyelei
0