Merge pull request #10611 from filecoin-project/fix/check-sector-exists

fix: cli: Check if the sectorID exists before removing
filecoin-project · Apr 4, 2023 · 4394187 · 4394187
2 parents 740a7ca + d211b5e
commit 4394187
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/cmd/lotus-miner/sectors.go b/cmd/lotus-miner/sectors.go
@@ -1392,6 +1392,12 @@ var sectorsRemoveCmd = &cli.Command{
 			return xerrors.Errorf("could not parse sector number: %w", err)
 		}
 
+		// Check if the sector exists
+		_, err = minerAPI.SectorsStatus(ctx, abi.SectorNumber(id), false)
+		if err != nil {
+			return xerrors.Errorf("sectorID %d has not been created yet: %w", id, err)
+		}
+
 		return minerAPI.SectorRemove(ctx, abi.SectorNumber(id))
 	},
 }