-
Notifications
You must be signed in to change notification settings - Fork 152
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
Add Type S3Compliant to Kopia Repository Storage Args #2159
Add Type S3Compliant to Kopia Repository Storage Args #2159
Conversation
Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
Thanks for submitting this pull request 🎉. The team will review it soon and get back to you. If you haven't already, please take a moment to review our project contributing guideline and Code of Conduct document. |
@r4rajat could you please add a test plan? Against which objectstore have you tested these changes? |
|
pkg/secrets/secrets.go
Outdated
@@ -60,6 +60,8 @@ func getLocationSecret(secret *corev1.Secret) (reposerver.Secret, error) { | |||
switch reposerver.LocType(string(locationType)) { | |||
case reposerver.LocTypeS3: | |||
return reposerver.NewAWSLocation(secret), nil | |||
case reposerver.LocTypes3Compliant: | |||
return reposerver.NewAWSLocation(secret), nil |
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.
endpoint
is compulsory for S3Compliant but not S3
, hence we may need to add validation for endpont key for LocTypeS3Compliant
under pkg/secrets
.
You can refer to #1940 PR to see how to add validation
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.
Added !
Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
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.
LGTM
* Add S3Compliant to Storage Args Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Test for s3Compliant storage args Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Validation for s3Compliant Location Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Format Imports Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Bucket Validation Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Tests for S3 Compliant Secrets Validation Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> --------- Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
* Add S3Compliant to Storage Args Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Test for s3Compliant storage args Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Validation for s3Compliant Location Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Format Imports Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Bucket Validation Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> * Add Tests for S3 Compliant Secrets Validation Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com> --------- Signed-off-by: Rajat Gupta <rajat.gupta@veeam.com>
Change Overview
Add Type S3Compliant to Kopia Repository Storage Args
s3compliant
as Type for Storage Arguments while testing Repository Server Workflows inkando
command linePull request type
Please check the type of change your PR introduces:
Issues
Test Plan
Manual Testing Steps
1) Create Images for Kanister and Repo Server controller
2) Install Kanister
3) Apply Repo Server CRD
4) Create Test Application [Time Logger]
5) Create OpenSSL Certificate
6) Create S3 Location and Location Secret Config Files
make install-minio
target]7) Apply Secrets
8) Create Repository
9) Create Repository Server CR
Wait till the status of Repository Server CR gets to
ServerReady
, You could check it by running following command10) Create Blueprint
11) Build kanctl with latest changes
12) Take Backup of the Application
Check Status of the actionset
13) Restore the Application
Check status