few tiny bugs , no biggie just panicing

Author: krasi-georgiev

db.go

@@ -505,8 +505,15 @@ func (db *DB) reload(deleteable ...string) (err error) {
 	for _, dir := range dirs {
 		meta, err := readMetaFile(dir)
 		if err != nil {
-			deleteable = append(deleteable, dir)
-			level.Error(db.logger).Log("msg", "dir set for deletion due to error in the meta file", "dir", dir, "err", err.Error())
+			if os.IsNotExist(err) {
+				deleteable = append(deleteable, dir)
+				level.Error(db.logger).Log("msg", "dir set for deletion due to error in the meta file", "dir", dir, "err", err.Error())
+				continue
+			}
+			return errors.Wrapf(err, "read meta information %s", dir)
+		}
+
+		if stringsContain(deleteable, dir) {
 			continue
 		}
 

repair.go

@@ -31,7 +31,7 @@ func repairBadIndexVersion(logger log.Logger, dir string) error {
 		}
 		d = path.Join(dir, d)
 		// Skip dirs with missing meta. These will be deleted when reloading the db.
-		if _, err := os.Stat(filepath.Join(dir, metaFilename)); os.IsNotExist(err) {
+		if _, err := os.Stat(filepath.Join(d, metaFilename)); os.IsNotExist(err) {
 			continue
 		}
 

repair_test.go

@@ -76,7 +76,7 @@ func TestRepairBadIndexVersion(t *testing.T) {
 	}
 
 	// On DB opening all blocks in the base dir should be repaired.
-	db, _ := Open("testdata/repair_index_version", nil, nil, nil)
+	db, err := Open("testdata/repair_index_version", nil, nil, nil)
 	if err != nil {
 		t.Fatal(err)
 	}