Skip to content

Commit 00d6b27

Browse files
committed
plume/release: drop metadata URL check
When we update the release index, plume checks if the release we want to add already exists by checking for a release with the same version _and_ metadata URL. The latter check seems unnecessary and in fact harmful because there shouldn't ever be two releases in the list with the same version number at all. So drop that check. Noticed this when testing that code locally (with `--local-mode`) in which we don't pass S3 information, so it's normal that the metadata URL isn't the same.
1 parent 9ad8441 commit 00d6b27

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mantle/cmd/plume/release.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ func modifyReleaseMetadataIndex(api *aws.API, rel release.Release) {
368368
}
369369

370370
func compareStaticReleaseInfo(a, b release.IndexRelease) bool {
371-
if a.Version != b.Version || a.MetadataURL != b.MetadataURL {
371+
if a.Version != b.Version {
372372
return false
373373
}
374374
return true

0 commit comments

Comments
 (0)