5964 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
copilot-swe-agent[bot]	acd4d5b183	Initial plan	2026-02-02 03:19:24 +00:00
Xun Jiang/Bruce Jiang	386599638f	Merge pull request #9510 from Lyndon-Li/ignore-cache-volume-config-without-backup-repo-config ... Log when cache volume configured but backup repo is not	2026-02-02 10:50:20 +08:00
Lyndon-Li	9796da389d	ignore cache volume config when backup repo config is not provided ... Signed-off-by: Lyndon-Li <lyonghui@vmware.com>	2026-01-30 18:35:36 +08:00
Scott Seago	dfb1d45831	Remove backup from running list when backup fails validation (#9498) ... Signed-off-by: Scott Seago <sseago@redhat.com>	2026-01-27 16:25:30 -05:00
Xun Jiang/Bruce Jiang	72beb35edc	Maintenance Job only uses the first element of the LoadAffinity array from the ConfigMap. (#9494) ... Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>	2026-01-23 14:27:50 -05:00
Wenkai Yin(尹文开)	7442d20f9d	Merge pull request #9481 from Lyndon-Li/issue-fix-9478 ... Issue 9478: Diagnose expose on peek error	2026-01-15 16:53:57 +08:00
lyndon-li	4dfb47dd21	Merge pull request #9487 from Lyndon-Li/issue-fix-for-cache-volume ... Fix issue for cache volume	2026-01-15 11:43:36 +08:00
Lyndon-Li	e72fea8ecd	fix issue for cache volume ... Signed-off-by: Lyndon-Li <lyonghui@vmware.com>	2026-01-14 17:45:01 +08:00
lyndon-li	f388a5ce51	update doc statement for Windows support (#9486) ... Signed-off-by: Lyndon-Li <lyonghui@vmware.com>	2026-01-14 00:25:58 -05:00
Lyndon-Li	e703e06eeb	diagnose expose on peek error ... Signed-off-by: Lyndon-Li <lyonghui@vmware.com>	2026-01-13 16:33:14 +08:00
Wenkai Yin(尹文开)	1feaafc03e	Merge pull request #9474 from blackpiglet/add_role_rolebinding_in_restore_sequence ... Add Role, RoleBinding, ClusterRole, and ClusterRoleBinding in restore sequence	2026-01-08 11:58:06 +08:00
Xun Jiang/Bruce Jiang	e446ce54f6	Merge pull request #9473 from vmware-tanzu/fix_e2e_version_check_issue ... Fix the version regexp to make sure releaseRe and tegRe can support string like v1.16	2026-01-07 22:41:40 +08:00
Xun Jiang	b7289b51c7	Add Role, RoleBinding, ClusterRole, and ClusterRoleBinding in restore sequence. ... Ensure the RBAC resources are restored before pods. The change help to avoid pod starting error when pod depends on the RBAC resources, e.g., prometheus operator check whether it has enough permission before launching controller, if prometheus operator pod starts before RBAC resources created, it will not launch controllers, and it will not retry. f7f07bcdfb/cmd/operator/main.go (L392-L400) Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>	2026-01-07 12:40:23 +08:00
lyndon-li	6eae73f0bf	Merge pull request #9466 from Lyndon-Li/collect-kopia-content-log ... Collect kopia content log	2026-01-06 15:51:47 +08:00
Xun Jiang	07a3cf759d	Fix the version regexp to make sure releaseRe and tegRe can support string like v1.16. ... Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>	2026-01-06 11:51:59 +08:00
lyndon-li	8f8367be65	Merge pull request #9456 from lubronzhan/topic/lubron/fix_logger ... Fix: Update the logging to print correct affinity field	2026-01-05 13:50:55 +08:00
Lubron Zhan	0d80995e62	Update the logging to print correct affinity field ... Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>	2026-01-01 11:37:21 -08:00
Lyndon-Li	1425ebb369	collect kopia content log ... Signed-off-by: Lyndon-Li <lyonghui@vmware.com>	2025-12-31 15:42:14 +08:00
Lubron Zhan	04364ef2ca	Update the logging to print correct affinity field ... Signed-off-by: Lubron Zhan <lubronzhan@gmail.com>	2025-12-30 11:00:49 -08:00
lyndon-li	3b5118b45e	Merge pull request #9463 from Lyndon-Li/kopia-bump-up-0.22.3 ... Bump kopia to 0.22.3	2025-12-30 18:18:33 +08:00
Lyndon-Li	c556603ce2	bump kopia to 0.22.3 ... Signed-off-by: Lyndon-Li <lyonghui@vmware.com>	2025-12-30 16:44:52 +08:00
Xun Jiang/Bruce Jiang	8708d4fda8	Merge pull request #9457 from blackpiglet/9380_fix ... Add function to validate the E2E Velero version.	2025-12-29 14:04:39 +08:00
Xun Jiang/Bruce Jiang	0db7816fa6	Merge branch 'main' into 9380_fix	2025-12-26 18:09:22 +08:00
Xun Jiang	b5ccc4373d	Add functions to validate and compare the E2E Velero version. ... Remove 'self' from MIGRATE_FROM_VELERO_VERSION. * Need to modify the BYOT case to specify MIGRATE_FROM_VELERO_VERSION. Remove the CSI plugin installation check for no older than v1.14 Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>	2025-12-26 12:53:09 +08:00
lyndon-li	8f2016000a	Merge pull request #9166 from kaovilai/claude/issue-107-20250807-1638 ... Add VolumePolicy support for Pending PVC Phase conditions	2025-12-24 11:05:34 +08:00
lyndon-li	327ea3ea13	Merge pull request #9206 from Joeavaikath/label-cleanup ... Remove labels associated with previous backups	2025-12-22 13:20:04 +08:00
claude[bot]	80211d77e5	feat: Add VolumePolicy support for PVC Phase conditions to allow skipping Pending PVCs ... This commit implements VolumePolicy support for PVC Phase conditions, resolving vmware-tanzu/velero#7233 where backups fail with ''PVC has no volume backing this claim'' for Pending PVCs. Changes made: - Extended VolumePolicy API to support PVC phase conditions - Added pvcPhaseCondition struct with matching logic - Modified getMatchAction() to evaluate policies for unbound PVCs before returning errors - Added case to GetMatchAction() to handle PVC-only scenarios (nil PV) - Added comprehensive unit tests for PVC phase parsing and matching Users can now skip Pending PVCs through volume policy configuration: apiVersion: v1 kind: ConfigMap metadata: name: volume-policy namespace: velero data: policy.yaml: | version: v1 volumePolicies: - conditions: pvcPhase: [Pending] action: type: skip chore: rename changelog file to match PR #9166 Renamed changelogs/unreleased/7233-claude to changelogs/unreleased/9166-claude to match the opened PR at https://github.com/vmware-tanzu/velero/pull/9166 docs: Add PVC phase condition support to VolumePolicy documentation - Added pvcPhase field to YAML template example - Documented pvcPhase as a supported condition in the list - Added comprehensive examples for using PVC phase conditions - Included examples for Pending, Bound, and Lost phases - Demonstrated combining PVC phase with other conditions Co-Authored-By: Tiger Kaovilai <kaovilai@users.noreply.github.com>	2025-12-22 10:07:05 +07:00
Michal Pryc	4b6708de2c	Fix plugin init container names exceeding DNS-1123 limit (#9445) ... Ensure plugin init container names satisfy DNS-1123 label constraints (max 63 chars). Long names are truncated with an 8-char hash suffix to maintain uniqueness. Fixes: #9444 Signed-off-by: Michal Pryc <mpryc@redhat.com>	2025-12-19 13:39:27 -05:00
Tiger Kaovilai	65eaceee0b	Refactor cleanupStaleVeleroLabels to use constants for label keys ... Signed-off-by: Tiger Kaovilai <tkaovila@redhat.com>	2025-12-19 20:17:47 +07:00
Joseph	2d8a87fec4	Add condition to catch backup.velero.io labels ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
Joseph	eae5bea469	Remove backup.velero.io/must-include-additional-items label ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
Joseph	87dbc16b0a	Revert registry local debug value ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
Joseph	db2193c53a	Make update ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
Joseph	643dd784ea	Refactor into func and add tests ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
Joseph	e7166fc9e9	Add changelog ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
Joseph	bfb431fcdf	Clean up stale labels before backing up PVC ... Signed-off-by: Joseph <jvaikath@redhat.com>	2025-12-19 20:17:46 +07:00
lyndon-li	2d93ab261e	Merge pull request #9141 from kaovilai/9097 ... feat: Enhance BackupStorageLocation with Secret-based CA certificate support	2025-12-19 13:13:47 +08:00
Xun Jiang/Bruce Jiang	fcb7fc9356	Merge pull request #9366 from blackpiglet/9359_fix ... Use hookIndex for recording multiple restore exec hooks.	2025-12-18 17:00:42 +08:00
Xun Jiang/Bruce Jiang	727a4fd0ed	Merge pull request #9452 from blackpiglet/9201_fix ... Add maintenance job and data mover pod's labels and annotations setting.	2025-12-18 16:39:24 +08:00
Xun Jiang/Bruce Jiang	aa3bd251dd	Merge branch 'main' into 9097	2025-12-18 14:18:04 +08:00
Shubham Pampattiwar	dad85b6fc3	Merge pull request #9441 from shubham-pampattiwar/fix-volume-policy-performance-9179 ... Add PVC-to-Pod cache to improve volume policy performance	2025-12-17 21:47:11 -08:00
Shubham Pampattiwar	78e9470028	Remove unnecessary nolint directives from test file ... The nolint:staticcheck directives are not needed in the test file because it calls NewVolumeHelperImpl within the same package, which doesn't trigger deprecation warnings. Only cross-package calls need the directive. Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-17 11:01:00 -08:00
Shubham Pampattiwar	4ba2effaac	Add nolint directives for intentional deprecated function usage ... The ShouldPerformSnapshotWithVolumeHelper function and tests intentionally use NewVolumeHelperImpl (deprecated) for backwards compatibility with third-party plugins. Add nolint:staticcheck to suppress the linter warnings with explanatory comments. Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-17 10:54:11 -08:00
Shubham Pampattiwar	f592a264a6	Address review feedback: remove deprecated functions ... Remove deprecated functions that were marked for removal per review: - Remove GetPodsUsingPVC (replaced by GetPodsUsingPVCWithCache) - Remove IsPVCDefaultToFSBackup (replaced by IsPVCDefaultToFSBackupWithCache) - Remove associated tests for deprecated functions - Add deprecation marker to NewVolumeHelperImpl - Add deprecation marker to ShouldPerformSnapshotWithBackup Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-17 10:44:15 -08:00
Xun Jiang	e39374f335	Add maintenance job and data mover pod's labels and annotations setting. ... Add wait in file_system_test's async test cases. Add related documents. Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>	2025-12-17 13:21:07 +08:00
Shubham Pampattiwar	10ef43e147	Fix gofmt formatting issues in test files ... Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-16 12:40:55 -08:00
Shubham Pampattiwar	b7052c2cb1	Implement lazy per-namespace PVC-to-Pod caching for plugin path ... This commit addresses reviewer feedback on PR #9441 regarding concurrent backup caching concerns. Key changes: 1. Added lazy per-namespace caching for the CSI PVC BIA plugin path: - Added IsNamespaceBuilt() method to check if namespace is cached - Added BuildCacheForNamespace() for lazy, per-namespace cache building - Plugin builds cache incrementally as namespaces are encountered 2. Added NewVolumeHelperImplWithCache constructor for plugins: - Accepts externally-managed PVC-to-Pod cache - Follows pattern from PR #9226 (Scott Seago's design) 3. Plugin instance lifecycle clarification: - Plugin instances are unique per backup (created via newPluginManager) - Cleaned up via CleanupClients at backup completion - No mutex or backup UID tracking needed 4. Test coverage: - Added tests for IsNamespaceBuilt and BuildCacheForNamespace - Added tests for NewVolumeHelperImplWithCache constructor - Added test verifying cache usage for fs-backup determination This maintains the O(N+M) complexity improvement from issue #9179 while addressing architectural concerns about concurrent access. Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-16 12:28:47 -08:00
Xun Jiang	57370296ab	Use hookIndex for recording multiple restore exec hooks. ... Signed-off-by: Xun Jiang <xun.jiang@broadcom.com>	2025-12-16 16:39:42 +08:00
Shubham Pampattiwar	f4c4653c08	Fix linter errors: use 'any' instead of 'interface{}' ... Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-15 14:18:05 -08:00
Shubham Pampattiwar	987edf5037	Add global VolumeHelper caching in CSI PVC BIA plugin ... Address review feedback to have a global VolumeHelper instance per plugin process instead of creating one on each ShouldPerformSnapshot call. Changes: - Add volumeHelper, cachedForBackup, and mu fields to pvcBackupItemAction struct for caching the VolumeHelper per backup - Add getOrCreateVolumeHelper() method for thread-safe lazy initialization - Update Execute() to use cached VolumeHelper via ShouldPerformSnapshotWithVolumeHelper() - Update filterPVCsByVolumePolicy() to accept VolumeHelper parameter - Add ShouldPerformSnapshotWithVolumeHelper() that accepts optional VolumeHelper for reuse across multiple calls - Add NewVolumeHelperForBackup() factory function for BIA plugins - Add comprehensive unit tests for both nil and non-nil VolumeHelper paths This completes the fix for issue #9179 by ensuring the PVC-to-Pod cache is built once per backup and reused across all PVC processing, avoiding O(N*M) complexity. Fixes #9179 Signed-off-by: Shubham Pampattiwar <spampatt@redhat.com>	2025-12-15 14:18:05 -08:00