diff --git a/pkg/cloudprovider/backup_service.go b/pkg/cloudprovider/backup_service.go index 4414209c63..ff6863584e 100644 --- a/pkg/cloudprovider/backup_service.go +++ b/pkg/cloudprovider/backup_service.go @@ -64,8 +64,8 @@ type BackupGetter interface { const ( metadataFileFormatString = "%s/ark-backup.json" - backupFileFormatString = "%s/%s.tar.gz" - logFileFormatString = "%s/%s.log.gz" + backupFileFormatString = "%s/%s-data.tar.gz" + logFileFormatString = "%s/%s-logs.gz" ) func getMetadataKey(backup string) string { @@ -186,7 +186,6 @@ func (br *backupService) DeleteBackupDir(bucket, backupName string) error { var errs []error for _, key := range objects { glog.V(4).Infof("Trying to delete bucket=%s, key=%s", bucket, key) - fmt.Printf("Trying to delete bucket=%s, key=%s\n", bucket, key) if err := br.objectStorage.DeleteObject(bucket, key); err != nil { errs = append(errs, err) } diff --git a/pkg/cloudprovider/backup_service_test.go b/pkg/cloudprovider/backup_service_test.go index 86da45cc47..45aea8d480 100644 --- a/pkg/cloudprovider/backup_service_test.go +++ b/pkg/cloudprovider/backup_service_test.go @@ -88,10 +88,10 @@ func TestUploadBackup(t *testing.T) { objStore.On("PutObject", bucket, backupName+"/ark-backup.json", test.metadata).Return(test.metadataError) } if test.backup != nil { - objStore.On("PutObject", bucket, backupName+"/"+backupName+".tar.gz", test.backup).Return(test.backupError) + objStore.On("PutObject", bucket, backupName+"/"+backupName+"-data.tar.gz", test.backup).Return(test.backupError) } if test.log != nil { - objStore.On("PutObject", bucket, backupName+"/"+backupName+".log.gz", test.log).Return(test.logError) + objStore.On("PutObject", bucket, backupName+"/"+backupName+"-logs.gz", test.log).Return(test.logError) } if test.expectMetadataDelete { objStore.On("DeleteObject", bucket, backupName+"/ark-backup.json").Return(nil) @@ -116,7 +116,7 @@ func TestDownloadBackup(t *testing.T) { o := &testutil.ObjectStorageAdapter{} bucket := "b" backup := "bak" - o.On("GetObject", bucket, backup+"/"+backup+".tar.gz").Return(ioutil.NopCloser(strings.NewReader("foo")), nil) + o.On("GetObject", bucket, backup+"/"+backup+"-data.tar.gz").Return(ioutil.NopCloser(strings.NewReader("foo")), nil) s := NewBackupService(o) rc, err := s.DownloadBackup(bucket, backup) require.NoError(t, err)