Skip to content

Commit

Permalink
satellite/repair/checker: persist placement information to the queue
Browse files Browse the repository at this point in the history
Change-Id: I51c7fd5a2a38f9f6620c16eddaed3b4915ffd792
  • Loading branch information
elek authored and Storj Robot committed Sep 25, 2023
1 parent 18d5caa commit b4fdc49
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions satellite/repair/checker/observer.go
Expand Up @@ -405,6 +405,7 @@ func (fork *observerFork) process(ctx context.Context, segment *rangedloop.Segme
Position: segment.Position,
UpdatedAt: time.Now().UTC(),
SegmentHealth: segmentHealth,
Placement: segment.Placement,
}, func() {
// Counters are increased after the queue has determined
// that the segment wasn't already queued for repair.
Expand Down
4 changes: 3 additions & 1 deletion satellite/repair/checker/observer_test.go
Expand Up @@ -607,7 +607,7 @@ func TestObserver_PlacementCheck(t *testing.T) {
_, err := planet.Satellites[0].API.Buckets.Service.UpdateBucket(ctx, buckets.Bucket{
ProjectID: planet.Uplinks[0].Projects[0].ID,
Name: "testbucket",
Placement: storj.EU,
Placement: storj.PlacementConstraint(1),
})
require.NoError(t, err)

Expand Down Expand Up @@ -672,6 +672,8 @@ func TestObserver_PlacementCheck(t *testing.T) {
require.NoError(t, err)

require.Equal(t, segments[0].StreamID, injuredSegment.StreamID)
require.Equal(t, segments[0].Placement, injuredSegment.Placement)
require.Equal(t, storj.PlacementConstraint(1), injuredSegment.Placement)

count, err := repairQueue.Count(ctx)
require.Zero(t, err)
Expand Down

0 comments on commit b4fdc49

Please sign in to comment.