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.

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

[controller] Add data wipe #122

Merged
merged 122 commits into from
Feb 25, 2025
Merged

[controller] Add data wipe #122

merged 122 commits into from
Feb 25, 2025
+508 −50

Conversation

AleksZimin
Copy link
Member

@AleksZimin AleksZimin commented Feb 3, 2025 •
edited by duckhawk
Loading

Description

Added the ability to delete data from volumes upon deletion in the EE edition.

What is the expected result?

Working feature of data wipe while deleting persistentVolumes

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

astef added 3 commits December 20, 2024 01:10
@astef
Move LLVS to EE
eaa249f 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
also move snapshots restore to EE
14e1e82 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix linter
fe9cc5c 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@z9r5 z9r5 requested a review from maxinsky February 3, 2025 08:56
krpsh123 and others added 26 commits February 3, 2025 14:56
@krpsh123
Resolve merge conflict
ac7d566 
Signed-off-by: Pavel Karpov <pavel.karpov@flant.com>
@krpsh123
[github] CI: add editions
867f1ad 
Signed-off-by: Pavel Karpov <pavel.karpov@flant.com>
@astef
include llvs always
0556941 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@NikolayDemchuk
added webhook
dd3ae07 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@NikolayDemchuk
fixes
d502bfb 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@NikolayDemchuk
change
1401ca3 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@NikolayDemchuk
fixes
6f9209f 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@NikolayDemchuk
fixes
536dac9 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@NikolayDemchuk
changed go version
3624ce5 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@NikolayDemchuk
fix
15735cf 
Signed-off-by: Nikolay Demchuk <nikolay.demchuk@flant.com>
@astef
try
583354f 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@krpsh123
[webhooks] fix werf
5b14900 
Signed-off-by: Pavel Karpov <pavel.karpov@flant.com>
@krpsh123
bump go version to 1.23.5
441d42e 
Signed-off-by: Pavel Karpov <pavel.karpov@flant.com>
@astef
update go
060d427 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix yamls
62fdc34 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix caBundle
b26795b 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
better message
a008529 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
optionally disable validating webhook when featureLLVSEnabled
8b772d2 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix overwriting openapi folder in bundle
4972310 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix bundle problem; change build trigger
6284f38 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix typo
13e4c0e 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix reversed logic of featureLLVSEnabled
a426efa 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
Merge pull request #123 from deckhouse/ndemchuk-webhook-add
71223ab 
[ci] Add webhook
@astef
upgrade Go in workflows
6847728 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@astef
fix linter
3bbf5ec 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@AleksZimin
some changes
21866a6 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
denmaloyreb
denmaloyreb reviewed Feb 20, 2025
View reviewed changes
@asergunov @denmaloyreb
Apply suggestions from code review
4c0f4af 
Co-authored-by: Denis Rebenok <60424108+denmaloyreb@users.noreply.github.com>
Signed-off-by: Anton Sergunov <setosha@gmail.com>
denmaloyreb
denmaloyreb reviewed Feb 20, 2025
View reviewed changes
asergunov and others added 5 commits February 20, 2025 18:36
@asergunov
Update images/agent/src/internal/utils/volume_cleanup_ee.go
167fb64 
Signed-off-by: Anton Sergunov <setosha@gmail.com>
@asergunov
early return for less indentation
15493b4 
Signed-off-by: Anton Sergunov <anton.sergunov@flant.com>
@asergunov @denmaloyreb
Update docs/USAGE.ru.md
3e06611 
Co-authored-by: Denis Rebenok <60424108+denmaloyreb@users.noreply.github.com>
Signed-off-by: Anton Sergunov <setosha@gmail.com>
@asergunov
don't requeue cleaning up errors, don't update phase when not heeded
fd04ca2 
Signed-off-by: Anton Sergunov <anton.sergunov@flant.com>
docs: add Protection against data leakage between volumes section in …
bb4bba7 
…USAGE.md

Signed-off-by: Denis.Rebenok <denis.rebenok@flant.ru>
denmaloyreb
denmaloyreb previously approved these changes Feb 21, 2025
View reviewed changes
@AleksZimin
some fixes
3ef50b1 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
AleksZimin
AleksZimin commented Feb 21, 2025
View reviewed changes
@AleksZimin
Update docs/USAGE.md
5e9f33d 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin AleksZimin marked this pull request as ready for review February 21, 2025 11:29
asergunov
asergunov previously approved these changes Feb 21, 2025
View reviewed changes
@AleksZimin AleksZimin requested a review from astef February 21, 2025 12:31
astef
astef requested changes Feb 21, 2025
View reviewed changes
@asergunov
Remove unused constants
edca6e2 
Signed-off-by: Anton Sergunov <anton.sergunov@flant.com>
@asergunov
Fix by review
0c7b1ee 
Signed-off-by: Anton Sergunov <anton.sergunov@flant.com>
@asergunov
Remove check duplication
d1924e6 
Signed-off-by: Anton Sergunov <anton.sergunov@flant.com>
@asergunov asergunov requested a review from astef February 24, 2025 10:06
@astef
improve naming andadd comment
9437bcd 
Signed-off-by: Aleksandr Stefurishin <aleksandr.stefurishin@flant.com>
@AleksZimin AleksZimin merged commit 1778068 into main Feb 25, 2025
11 checks passed
@AleksZimin AleksZimin deleted the add-data-wipe branch February 25, 2025 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astef astef astef approved these changes

@asergunov asergunov asergunov approved these changes

@denmaloyreb denmaloyreb denmaloyreb left review comments

Assignees

@asergunov asergunov

@AleksZimin AleksZimin

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@AleksZimin @astef @asergunov @denmaloyreb @krpsh123 @NikolayDemchuk @duckhawk