Skip to content

Commit

Permalink
remove - after prefix in storage snapshot name
Browse files Browse the repository at this point in the history
  • Loading branch information
mschuchard committed Jan 7, 2025
1 parent 7e8013b commit 4a2e7dc
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
### 1.2.1 (Next)
- Add timestamp suffix to default snapshot file name.
- Migrate AWS storage support to SDK v2.
- Remove `-` after `prefix` in storage snapshot name.

### 1.2.0
- Add GCP storage support.
Expand Down
2 changes: 1 addition & 1 deletion storage/aws_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ func TestSnapshotS3Upload(test *testing.T) {
}
defer fooFile.Close()

if err := snapshotS3Upload(util.Container, fooFile, ""); err == nil || !strings.Contains(err.Error(), "get credentials: failed to refresh cached credentials") {
if err := snapshotS3Upload(util.Container, fooFile, "empty"); err == nil || !strings.Contains(err.Error(), "get credentials: failed to refresh cached credentials") {
test.Errorf("expected error (contains): get credentials: failed to refresh cached credentials, actual: %v", err)
}
}
2 changes: 1 addition & 1 deletion storage/storage.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import (
// unified function for interfacing with all snapshot storage transfers
func StorageTransfer(config *util.CloudConfig, snapshotPath string, cleanup bool) error {
// use supplied prefix and snapshot base filename for full name
snapshotName := config.Prefix + "-" + filepath.Base(snapshotPath)
snapshotName := config.Prefix + filepath.Base(snapshotPath)

// open snapshot file
snapshotFile, err := os.Open(snapshotPath)
Expand Down

0 comments on commit 4a2e7dc

Please sign in to comment.