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

pb-3494: vendor latest kdmp repo from 1.2.4 branch #1274

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

siva-portworx
Copy link
Contributor

What type of PR is this?

Uncomment only one and also add the corresponding label in the PR:
bug

What this PR does / why we need it:

pb-3494: vendor latest kdmp repo from 1.2.4 branch

Does this PR change a user-facing CRD or CLI?:
no

Is a release note needed?:

Issue: csi migrated volumes from k8s 1.23 are not backed up with generic backup as volume mount was selected with incorrect value.
User Impact: generic backup of csi migrated volumes does not get restored properly.
Resolution: Added check to take proper volume mount path for csi migrated volumes.

Does this change need to be cherry-picked to a release branch?:
no

sgajawada-px and others added 12 commits January 31, 2023 18:19
* PB-3397: Add Logic to support namespace label selector in the application backup controller
	- These feature is supported with manual and schedule backups
	- Currently Namespaces come along the application backup spec consider to be backed up,
	  Now these namespaces are selected using the namespace selectors provided as part of the same spec at execution time.
	- Filtered namespaces that to be backed up are also updated to the data-store object while reconciling.
Signed-off-by: Kesavan Thiruvenkadasamy <[email protected]>
Signed-off-by: Kesavan Thiruvenkadasamy <[email protected]>
* Added test for Clone PVCs after DR on destination cluster

* Added test for Clone PVCs after DR on dest cluster

* modified test to chec pvc bound

* modified test to check pvc bound

* Addressed review comments

* Added changes to use storageclassName instead of storage class in annotations for clone pvc test

---------

Co-authored-by: root <[email protected]>
Co-authored-by: GEET JAIN <[email protected]>
Co-authored-by: geet <[email protected]>
Co-authored-by: root <[email protected]>
Co-authored-by: Rohit-PX <[email protected]>
@cnbu-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@siva-portworx siva-portworx force-pushed the pb-3494 branch 2 times, most recently from 7de03f9 to cbd024e Compare February 8, 2023 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants