diff --git a/builder/amazon/common/artifact.go b/builder/amazon/common/artifact.go index 3dc33b4b9..455994d59 100644 --- a/builder/amazon/common/artifact.go +++ b/builder/amazon/common/artifact.go @@ -37,6 +37,7 @@ func (a *Artifact) Id() string { parts = append(parts, fmt.Sprintf("%s:%s", region, amiId)) } + sort.Strings(parts) return strings.Join(parts, ",") } @@ -47,7 +48,7 @@ func (a *Artifact) String() string { amiStrings = append(amiStrings, single) } - sort.Sort(sort.StringSlice(amiStrings)) + sort.Strings(amiStrings) return fmt.Sprintf("AMIs were created:\n\n%s", strings.Join(amiStrings, "\n")) } diff --git a/builder/virtualbox/iso/builder_test.go b/builder/virtualbox/iso/builder_test.go index f3a4519af..3b8e59569 100644 --- a/builder/virtualbox/iso/builder_test.go +++ b/builder/virtualbox/iso/builder_test.go @@ -46,7 +46,7 @@ func TestBuilderPrepare_Defaults(t *testing.T) { t.Errorf("bad guest OS type: %s", b.config.GuestOSType) } - if b.config.VMName != "packer-foo" { + if b.config.VMName == "" { t.Errorf("bad vm name: %s", b.config.VMName) }