Remove fs_index version/entry_key check

* Fixes `Internal inconsistency: version 'qwertyuiopsha' vs '2' errors [#132106783] Signed-off-by: Ming Xiao <[email protected]>
cloudfoundry · Oct 11, 2016 · 57771c2 · 57771c2
1 parent f48a284
commit 57771c2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/releasedir/index/fs_index.go b/releasedir/index/fs_index.go
@@ -189,10 +189,6 @@ func (i FSIndex) entries(name string) ([]indexEntry, error) {
 	var entries []indexEntry
 
 	for versionKey, entry := range schema.Builds {
-		if versionKey != entry.Version {
-			return nil, bosherr.Errorf("Internal inconsistency: version '%s' vs '%s'", versionKey, entry.Version)
-		}
-
 		entries = append(entries, indexEntry{
 			Key:     versionKey,
 			Version: entry.Version,

diff --git a/releasedir/index/fs_index_test.go b/releasedir/index/fs_index_test.go
@@ -103,15 +103,14 @@ format-version: "2"`)
 			Expect(err).To(Equal(errors.New("fake-err")))
 		})
 
-		It("returns error if found entry does not have matching version with its build key", func() {
+		It("does not require version to equal entry key", func() {
 			fs.WriteFileString("/dir/name/index.yml", `---
 builds:
   fp: {version: other-fp}
 format-version: "2"`)
 
 			_, _, err := index.Find("name", "fp")
-			Expect(err).To(HaveOccurred())
-			Expect(err.Error()).To(Equal("Internal inconsistency: version 'fp' vs 'other-fp'"))
+			Expect(err).ToNot(HaveOccurred())
 		})
 
 		It("returns error if name is empty", func() {