chore(treewide): stabilize CI

TestServiceStorageIntegration was inherently flaky, as it didn't ensure
listed cluster order on validation. Sorting cluster fixes that.
TestRestoreGetTargetUnitsViewsIntegration started failing because
the default tombstone gc mode of some system tables has changed
from timeout to repair. It's annoying to keep track of it across
different scylla versions, and we don't really care for the specific
value, so this check was replaced with non-empty tombstone gc mode check.
TestServiceRepairIntegration/repair_ignore_hosts behaved flaky due
to a small timeout which is increased.

Author: Michal-Leszczynski

pkg/service/cluster/service_integration_test.go

@@ -315,6 +315,9 @@ func TestServiceStorageIntegration(t *testing.T) {
 		UUIDComparer(),
 		cmpopts.IgnoreFields(cluster.Cluster{}, "Host"),
 		cmpopts.IgnoreFields(cluster.Cluster{}, "KnownHosts"),
+		cmpopts.SortSlices(func(a, b *cluster.Cluster) bool {
+			return a.ID.String() < b.ID.String()
+		}),
 	}
 
 	t.Run("list empty", func(t *testing.T) {

pkg/service/repair/service_repair_integration_test.go

@@ -1088,7 +1088,7 @@ func TestServiceRepairIntegration(t *testing.T) {
 		Print("When: repair is done")
 		rd := scyllaclient.NewRingDescriber(ctx, h.Client)
 		if rd.IsTabletKeyspace("test_repair") {
-			h.assertDonePartialTabletRepair(shortWait, startNodeFunc)
+			h.assertDonePartialTabletRepair(longWait, startNodeFunc)
 		} else {
 			h.assertDone(longWait)
 		}

pkg/service/restore/service_restore_integration_test.go

@@ -483,11 +483,19 @@ func TestRestoreGetTargetUnitsViewsIntegration(t *testing.T) {
 				cmpopts.SortSlices(func(a, b Table) bool { return a.Table < b.Table }),
 				cmpopts.IgnoreFields(Unit{}, "Size"),
 				cmpopts.IgnoreFields(Table{}, "Size"),
+				cmpopts.IgnoreFields(Table{}, "TombstoneGC"),
 				cmpopts.IgnoreSliceElements(func(v Unit) bool { return slices.Contains(ignoreUnits, v.Keyspace) }),
 				cmpopts.IgnoreSliceElements(func(v Table) bool { return slices.Contains(ignoreUnits, v.Table) }),
 			); diff != "" {
 				t.Fatal(tc.units, diff)
 			}
+			for _, u := range units {
+				for _, tab := range u.Tables {
+					if tab.TombstoneGC == "" {
+						t.Fatalf("Expected tombstone gc mode of %s.%s to be set", u.Keyspace, tab.Table)
+					}
+				}
+			}
 
 			if goldenViews == nil {
 				goldenViews = make([]View, 0)