Update backup and log file name formats

Signed-off-by: Andy Goldstein <andy.goldstein@gmail.com>
vmware-tanzu · Sep 11, 2017 · 3013c2d · 3013c2d
1 parent 767324d
commit 3013c2d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git 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
@@ -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)