weeix
80da461458
clarify VolumeSnapshotClass error for mismatched driver/provisioner
...
Signed-off-by: weeix <weeix@users.noreply.github.com >
2025-09-02 18:31:13 -05:00
Xun Jiang
c62a486765
Add ConfigMap parameters validation for install CLI and server start.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-08-22 20:31:38 +08:00
lyndon-li
ae29030917
Merge branch 'main' into implement8869
2025-08-06 13:45:35 +08:00
Tiger Kaovilai
35d2cc0890
Add priority class support for Velero server and node-agent
...
- Add --server-priority-class-name and --node-agent-priority-class-name flags to velero install command
- Configure data mover pods (PVB/PVR/DataUpload/DataDownload) to use priority class from node-agent-configmap
- Configure maintenance jobs to use priority class from repo-maintenance-job-configmap (global config only)
- Add priority class validation with ValidatePriorityClass and GetDataMoverPriorityClassName utilities
- Update e2e tests to include PriorityClass testing utilities
- Move priority class design document to Implemented folder
- Add comprehensive unit tests for all priority class implementations
- Update documentation for priority class configuration
- Add changelog entry for #8883
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
remove unused test utils
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
feat: add unit test for getting priority class name in maintenance jobs
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
doc update
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
feat: add priority class validation for repository maintenance jobs
- Add ValidatePriorityClassWithClient function to validate priority class existence
- Integrate validation in maintenance.go when creating maintenance jobs
- Update tests to cover the new validation functionality
- Return boolean from ValidatePriorityClass to allow fallback behavior
This ensures maintenance jobs don't fail due to non-existent priority classes,
following the same pattern used for data mover pods.
Addresses feedback from:
https://github.com/vmware-tanzu/velero/pull/8883#discussion_r2238681442
Refs #8869
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
refactor: clean up priority class handling for data mover pods
- Fix comment in node_agent.go to clarify PriorityClassName is only for data mover pods
- Simplify server.go to use dataPathConfigs.PriorityClassName directly
- Remove redundant priority class logging from controllers as it's already logged during server startup
- Keep logging centralized in the node-agent server initialization
This reduces code duplication and clarifies the scope of priority class configuration.
🤖 Generated with [Claude Code](https://claude.ai/code )
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
refactor: remove GetDataMoverPriorityClassName from kube utilities
Remove GetDataMoverPriorityClassName function and its tests as priority
class is now read directly from dataPathConfigs instead of parsing from
ConfigMap. This simplifies the codebase by eliminating the need for
indirect ConfigMap parsing.
Refs #8869
🤖 Generated with [Claude Code](https://claude.ai/code )
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
refactor: remove priority class validation from install command
Remove priority class validation during install as it's redundant
since validation already occurs during server startup. Users cannot
see console logs during install, making the validation warnings
ineffective at this stage.
The validation remains in place during server and node-agent startup
where it's more appropriate and visible to users.
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
Co-Authored-By: Claude <noreply@anthropic.com >
2025-08-06 01:36:22 -04:00
Daniel Jiang
249d8f581a
Add include/exclude policy to resources policy
...
fixes #8610
This commit extends the resources policy, such that user can define
resource include exclude filters in the policy and reuse it in different backups.
Signed-off-by: Daniel Jiang <daniel.jiang@broadcom.com >
2025-08-05 15:16:59 +08:00
Xun Jiang/Bruce Jiang
9cb421c26f
Fix the dd and du's node affinity issue. ( #9130 )
...
Run the E2E test on kind / build (push) Failing after 12m11s
Run the E2E test on kind / setup-test-matrix (push) Successful in 4s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 27s
Close stale issues and PRs / stale (push) Successful in 12s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 1m22s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-aws, main) (push) Failing after 1m3s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-gcp, main) (push) Failing after 1m0s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-microsoft-azure, main) (push) Failing after 1m6s
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-08-04 16:21:35 -04:00
Xun Jiang
c84aab7f6f
Remove the WaitUntilVSCHandleIsReady from vs BIA.
...
Becasue the pvc BIA already run WaitUntilVSCHandleIsReady,
no need to do the same work in vs BIA.
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-07-31 15:35:05 +08:00
Xun Jiang
7b872473df
Use const to represent snapshot.storage.kubernetes.io/managed-by.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-07-30 08:06:22 +08:00
Xun Jiang
a61a073aea
Avoid checking the VS and VSC status in the backup finalizing phase.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-07-24 13:28:05 +08:00
Xun Jiang/Bruce Jiang
770ff142d7
Add imagePullSecrets inheritage for VGDP pod and maintenance job. ( #9096 )
...
Run the E2E test on kind / build (push) Failing after 8m21s
Run the E2E test on kind / setup-test-matrix (push) Successful in 4s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 42s
Close stale issues and PRs / stale (push) Successful in 21s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 1m49s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-aws, main) (push) Failing after 1m17s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-gcp, main) (push) Failing after 3m30s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-microsoft-azure, main) (push) Failing after 3m12s
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-07-23 13:55:16 -04:00
Lyndon-Li
34f8b73507
bump up kopia to v0.21.1
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-07-22 15:56:04 +08:00
Shubham Pampattiwar
a73a150d98
Accommodate VGS workflows in PVC CSI plugin
...
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
Add changelog file
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
make update
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
lint fix
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
add unit tests for getVSForPVC func
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
Use v1beta1 instead of v1 v1alpha1
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
go mod tidy
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
update updateVGSCreatedVS func to use retry on conflict
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
make update minor fix
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
fix ut assert
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
Address PR feedback
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
minor updates
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
remove unsused func and add todo for dep upgrades
Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com >
2025-07-21 11:36:40 -07:00
Daniel Jiang
479c21c58a
Merge pull request #8994 from mmorel-35/usetesting
...
chore: enable usetesting linter
2025-07-11 17:34:30 +08:00
Lyndon-Li
a752b54614
issue 8857: support third party tolerations
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-07-08 14:23:00 +08:00
Xun Jiang
b5426516da
Make the backup repository controller doesn't invalidate the BSL on restart.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-07-02 17:57:06 +08:00
Xun Jiang
f2133c7d22
The backup and restore data mover node selection.
...
Update Makefile to let the `make serve-docs` work again.
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-07-01 16:26:12 +08:00
Priyansh Choudhary
bd609db395
Remove username/password credential handling from newConfigCredential ( #9041 )
...
Run the E2E test on kind / build (push) Failing after 6m56s
Run the E2E test on kind / setup-test-matrix (push) Successful in 4s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 1m1s
Close stale issues and PRs / stale (push) Successful in 16s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 1m30s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-aws, main) (push) Failing after 1m17s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-gcp, main) (push) Failing after 1m18s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-microsoft-azure, main) (push) Failing after 1m15s
* remove username/password credential handling from newConfigCredential
Signed-off-by: Priyansh Choudhary <im1706@gmail.com >
* changelog update
Signed-off-by: Priyansh Choudhary <im1706@gmail.com >
* remove username/password credential tests from newConfigCredential.
Signed-off-by: Priyansh Choudhary <im1706@gmail.com >
---------
Signed-off-by: Priyansh Choudhary <im1706@gmail.com >
2025-06-27 10:02:41 -04:00
Matthieu MOREL
4089a92147
chore: enable usetesting linter
...
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-06-26 08:05:50 +02:00
Jan Jansen
81105031a7
copy security context from origin pod
...
Signed-off-by: Jan Jansen <jan.jansen@gdata.de >
2025-06-24 16:17:08 +08:00
Matthieu MOREL
07ea14962c
fix require-error rule from testifylint
...
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-06-23 15:39:54 +00:00
Matthieu MOREL
59825a0506
fix len rule from testifylint
...
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-06-22 22:41:08 +02:00
Lyndon-Li
2e7d11e3b6
bump kopia v0.20.1
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-06-18 13:35:12 +08:00
Matthieu MOREL
00ce103a50
Refactor affinity tests to use require and assert for better readability and error reporting
...
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-06-09 07:33:22 +02:00
Xun Jiang
1b7175394a
Skip VS and VSC not created by backup.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-06-07 10:51:05 +08:00
Lyndon-Li
5d5d4cd657
Merge branch 'main' into pod-volume-exposer
2025-06-06 14:40:26 +08:00
Xun Jiang/Bruce Jiang
2390bc8e71
Merge pull request #8976 from blackpiglet/7785_fix
...
Add BSL status check for backup/restore operations.
2025-06-06 11:22:10 +08:00
lyndon-li
bd23062c24
Merge branch 'main' into pod-volume-exposer
2025-06-03 14:05:11 +08:00
Lyndon-Li
5ccf22e0b0
Merge branch 'main' into vgdp-ms-pvb-data-path
2025-06-03 13:26:52 +08:00
Lyndon-Li
92c72b1a63
data path for vgdp ms pvb
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-06-03 13:25:48 +08:00
Xun Jiang
f604a5da48
Add BSL status check for backup/restore operations.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-05-30 10:21:32 +08:00
Lyndon-Li
b7d997130d
issue 8960: implement PodVolume exposer for PVB/PVR
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-05-29 17:09:03 +08:00
Wenkai Yin(尹文开)
d0374fadb6
Mark BackupRepository not ready when BSL changed
...
Mark BackupRepository not ready when BSL changed
Fixes #8860
Signed-off-by: Wenkai Yin(尹文开) <yinw@vmware.com >
2025-05-27 14:43:14 +08:00
lyndon-li
4d10047d67
issue-8878: relief node os deduction error checks ( #8891 )
...
Run the E2E test on kind / build (push) Failing after 6m28s
Run the E2E test on kind / setup-test-matrix (push) Successful in 3s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 36s
Close stale issues and PRs / stale (push) Successful in 9s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 1m5s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-aws, main) (push) Failing after 49s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-gcp, main) (push) Failing after 44s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-microsoft-azure, main) (push) Failing after 1m4s
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-05-06 12:38:05 -04:00
Matthieu MOREL
c6a420bd3a
chore: define common aliases for k8s packages ( #8672 )
...
Run the E2E test on kind / build (push) Failing after 6m48s
Run the E2E test on kind / setup-test-matrix (push) Successful in 3s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 35s
Close stale issues and PRs / stale (push) Successful in 8s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 1m11s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-aws, main) (push) Failing after 47s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-gcp, main) (push) Failing after 49s
Trivy Nightly Scan / Trivy nightly scan (velero-plugin-for-microsoft-azure, main) (push) Failing after 43s
* lchore: define common alias for k8s packages
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
* Update .golangci.yaml
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
* Update .golangci.yaml
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
* Update .golangci.yaml
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
---------
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-04-22 06:14:47 -04:00
hu-keyu
5160f96c2e
issue8294: inherit k8s default volumeSnapshotClass ( #8721 )
...
Run the E2E test on kind / build (push) Failing after 7m33s
Run the E2E test on kind / setup-test-matrix (push) Successful in 2s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 38s
* inherit k8s default volumeSnapshotClass
Signed-off-by: hu-keyu <hzldd999@gmail.com >
* add doc change for issue8294
Signed-off-by: hu-keyu <hzldd999@gmail.com >
---------
Signed-off-by: hu-keyu <hzldd999@gmail.com >
2025-04-07 12:21:13 +05:30
Lyndon-Li
f1dcb7ba11
add third party annotation support for maintenance job
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-03-25 13:43:38 +08:00
Roger Zimmermann
38a52980cc
Issue #8772 ensure pv removed ( #8777 )
...
Run the E2E test on kind / build (push) Failing after 6m16s
Run the E2E test on kind / setup-test-matrix (push) Successful in 3s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 37s
Close stale issues and PRs / stale (push) Successful in 8s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 1m10s
Trivy Nightly Scan / Trivy nightly scan (velero-restore-helper, main) (push) Failing after 53s
* ensure pv has been deleted
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
* ensure delete pv unit test
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
* comment, errors
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
* updated changelog
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
* pass value
Co-authored-by: Tiger Kaovilai <passawit.kaovilai@gmail.com >
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
* function renamed as suggested
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
---------
Signed-off-by: Roger Zimmermann <roger.zimmermann@inventx.ch >
Co-authored-by: Tiger Kaovilai <passawit.kaovilai@gmail.com >
2025-03-13 10:39:25 -04:00
Lyndon-Li
b170892e64
issue 8754: add third party annotation support
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-03-10 10:38:26 +08:00
Matthieu MOREL
6a6a237ba7
Bump golangci-lint from v1.57.2 to v1.64.5 ( #8641 )
...
Run the E2E test on kind / build (push) Failing after 5m43s
Run the E2E test on kind / setup-test-matrix (push) Successful in 2s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
build-image / Build (push) Failing after 8s
Main CI / Build (push) Failing after 33s
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-03-04 13:55:29 +05:30
Wenkai Yin(尹文开)
a45c9f27e8
Merge pull request #8715 from Lyndon-Li/issue-fix-8706
...
Run the E2E test on kind / build (push) Failing after 6m45s
Run the E2E test on kind / setup-test-matrix (push) Successful in 3s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 36s
Close stale issues and PRs / stale (push) Successful in 9s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 55s
Trivy Nightly Scan / Trivy nightly scan (velero-restore-helper, main) (push) Failing after 44s
Issue 8706: for immediate volumes, get node from volumeattachment
2025-02-25 14:25:45 +08:00
Xun Jiang
6b7dd12bf7
Modify VS and VSC restore actions.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-02-25 10:44:45 +08:00
Lyndon-Li
bf0d909524
issue 8706: for immediate volumes, get node from volumeattachment
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-02-21 13:27:44 +08:00
Lyndon-Li
e2a7986629
issue 8706: for immediate volumes, get node from volumeattachment
...
Signed-off-by: Lyndon-Li <lyonghui@vmware.com >
2025-02-20 19:19:28 +08:00
Xun Jiang
eb77151f48
Delete VSC after backup completes.
...
Signed-off-by: Xun Jiang <xun.jiang@broadcom.com >
2025-02-19 14:36:59 +08:00
Tiger Kaovilai
a3cee616dc
Upgrade go.mod k8s.io/ go.mod to v0.31.3 and set klog.SetLogger() for client-go ( #8450 )
...
Run the E2E test on kind / build (push) Failing after 5m44s
Run the E2E test on kind / setup-test-matrix (push) Successful in 3s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
build-image / Build (push) Failing after 10s
Main CI / Build (push) Failing after 31s
Close stale issues and PRs / stale (push) Successful in 7s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 59s
Trivy Nightly Scan / Trivy nightly scan (velero-restore-helper, main) (push) Failing after 45s
Also bumped to support upgraded k8s.io/ deps.
- controller-gen to v0.16.5
- sigs.k8s.io/controller-runtime v0.19.2
Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com >
2025-02-17 15:05:10 -05:00
Matthieu MOREL
ae5e94e822
chore: enable increment-decrement from revive
...
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-02-07 20:58:39 +01:00
Xun Jiang/Bruce Jiang
10a5b7b702
Merge pull request #8624 from mmorel-35/revive/use-any
...
chore: enable use-any from revive
2025-02-07 15:09:05 +08:00
Daniel Jiang
bedea9c74c
Merge pull request #8637 from reasonerjt/rm-leaked-vs
...
Clean up leaked CSI snapshot for incomplete backup
2025-01-23 12:56:12 +08:00
Matthieu MOREL
1e54f1cb15
chore: enable var-declaration from revive ( #8636 )
...
Run the E2E test on kind / build (push) Failing after 5m28s
Run the E2E test on kind / setup-test-matrix (push) Successful in 3s
Run the E2E test on kind / run-e2e-test (push) Has been skipped
Main CI / Build (push) Failing after 33s
Close stale issues and PRs / stale (push) Successful in 8s
Trivy Nightly Scan / Trivy nightly scan (velero, main) (push) Failing after 52s
Trivy Nightly Scan / Trivy nightly scan (velero-restore-helper, main) (push) Failing after 56s
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com >
2025-01-22 15:56:44 -05:00
Daniel Jiang
1c372893ec
Clean up leaked CSI snapshot for incomplete backup
...
This commit makes sure when a backup is deleted the controller will
delete the CSI snapshot even when the bakckup tarball is not uploaded.
fixes #8160
Signed-off-by: Daniel Jiang <daniel.jiang@broadcom.com >
2025-01-22 17:17:41 +08:00