-
Notifications
You must be signed in to change notification settings - Fork 82
OADP-6235: Add CaCert support for DPT #1862
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
base: oadp-dev
Are you sure you want to change the base?
OADP-6235: Add CaCert support for DPT #1862
Conversation
Skipping CI for Draft Pull Request. |
@shubham-pampattiwar: This pull request references OADP-6235 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
/cherry-pick oadp-1.5 |
@shubham-pampattiwar: once the present PR merges, I will cherry-pick it on top of In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
✅ Dev Testing Results - CA Certificate Support for DPTSuccessfully tested the CA certificate functionality for DataProtectionTest with MinIO using custom TLS certificates. Test Environment
BackupStorageLocation ConfigurationapiVersion: velero.io/v1
kind: BackupStorageLocation
metadata:
name: minio-bsl
namespace: openshift-adp
spec:
config:
region: minio
s3ForcePathStyle: "true"
s3Url: https://minio-s3-minio.apps.mtvoadpvgs.rhos-psi.cnv-qe.rhood.us
credential:
key: cloud
name: cloud-credentials-minio
objectStorage:
bucket: dpt-minio-test
caCert: LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0t... # [REDACTED - Base64 encoded CA cert]
prefix: velero
provider: aws
status:
lastSyncedTime: "2025-08-05T20:32:47Z"
lastValidationTime: "2025-08-05T20:32:57Z"
phase: Available Test Results✅ Test 1: With TLS Verification Enabled (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just my opinion non blockers
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kaovilai, shubham-pampattiwar The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@shubham-pampattiwar: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Why the changes were made
This PR adds CA certificate support for Data Protection Tests (DPT) to enable secure connections to object storage backends that use custom or self-signed certificates. This enhancement allows DPT to work with enterprise environments and private cloud deployments where object storage services use custom certificate authorities, addressing scenarios where the default system certificate store doesn't contain the required CA certificates.
The changes implement a flexible TLS configuration system that respects user preferences while maintaining security by:
How to test the changes made
- When skipTLSVerify: true is set, it takes precedence over any CA certificate configuration
- When CA certificate is provided without skipTLSVerify, the custom certificate is used
🤖 Generated with https://claude.ai/code
Co-Authored-By: Claude [email protected]