From 161f447d8b7ce644bb85a935588b3e6221c9e6e3 Mon Sep 17 00:00:00 2001 From: Tristan Wilson Date: Tue, 23 Jul 2024 10:52:51 +0200 Subject: [PATCH] Change the test data stored by das fs healthcheck For some installations, if they had already stored 65000 expiry index entries for the hold health check data, simply setting the expiry time to the start of epoch isn't enough to fix the health check for them since it already has 65000 entries and start of epoch would be another entry, exceeding the max inode links. Simply changing the test data preimage fixes this becauese it will be stored at a different hash. --- das/local_file_storage_service.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/das/local_file_storage_service.go b/das/local_file_storage_service.go index 5623195657..66a08d0d95 100644 --- a/das/local_file_storage_service.go +++ b/das/local_file_storage_service.go @@ -228,7 +228,7 @@ func (s *LocalFileStorageService) String() string { } func (s *LocalFileStorageService) HealthCheck(ctx context.Context) error { - testData := []byte("Test-Data") + testData := []byte("Test Data") // Store some data with an expiry time at the start of the epoch. // If expiry is disabled it will only create an index entry for the // same timestamp each time the health check happens. @@ -236,7 +236,7 @@ func (s *LocalFileStorageService) HealthCheck(ctx context.Context) error { // runs. There is a slight chance of a race between pruning and the // Put and Get calls, but systems using the HealthCheck will just retry // and succeed the next time. - err := s.Put(ctx, testData /* start of epoch */, 0) + err := s.Put(ctx, testData, 0 /* start of epoch */) if err != nil { return err }