21036: Report remaining balancerStats as prometheus metrics.
[arvados.git] / services / keep-balance / balance_run_test.go
index b7b3fb61233249beb6c6df48e46c8de4ea678e22..7f6deb1e54ae2668728a0e504b0fc0c0d0715fae 100644 (file)
@@ -15,6 +15,7 @@ import (
        "os"
        "strings"
        "sync"
+       "syscall"
        "time"
 
        "git.arvados.org/arvados.git/lib/config"
@@ -555,6 +556,10 @@ func (s *runSuite) TestDryRun(c *check.C) {
        c.Check(bal.stats.trashesDeferred, check.Not(check.Equals), 0)
        c.Check(bal.stats.underrep.replicas, check.Not(check.Equals), 0)
        c.Check(bal.stats.overrep.replicas, check.Not(check.Equals), 0)
+
+       metrics := arvadostest.GatherMetricsAsString(srv.Metrics.reg)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_trash_entries_deferred_count [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_pull_entries_deferred_count [1-9].*`)
 }
 
 func (s *runSuite) TestCommit(c *check.C) {
@@ -592,6 +597,19 @@ func (s *runSuite) TestCommit(c *check.C) {
        c.Check(metrics, check.Matches, `(?ms).*\narvados_keepbalance_changeset_compute_seconds_count 1\n.*`)
        c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_dedup_byte_ratio [1-9].*`)
        c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_dedup_block_ratio [1-9].*`)
+
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_replicated_block_count{replicas="0"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_replicated_block_count{replicas="1"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_replicated_block_count{replicas="9"} 0\n.*`)
+
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_usage_replicas{status="needed",storage_class="default"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_usage_blocks{status="needed",storage_class="default"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_usage_bytes{status="needed",storage_class="default"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_usage_bytes{status="unneeded",storage_class="default"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_usage_bytes{status="unachievable",storage_class="default"} [1-9].*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keep_usage_bytes{status="pulling",storage_class="default"} [1-9].*`)
+
+       c.Logf("%s", metrics)
 }
 
 func (s *runSuite) TestChunkPrefix(c *check.C) {
@@ -623,7 +641,7 @@ func (s *runSuite) TestChunkPrefix(c *check.C) {
        c.Check(string(lost), check.Equals, "")
 }
 
-func (s *runSuite) TestRunForever(c *check.C) {
+func (s *runSuite) TestRunForever_TriggeredByTimer(c *check.C) {
        s.config.ManagementToken = "xyzzy"
        opts := RunOptions{
                Logger: ctxlog.TestLogger(c),
@@ -639,7 +657,7 @@ func (s *runSuite) TestRunForever(c *check.C) {
 
        ctx, cancel := context.WithCancel(context.Background())
        defer cancel()
-       s.config.Collections.BalancePeriod = arvados.Duration(time.Millisecond)
+       s.config.Collections.BalancePeriod = arvados.Duration(10 * time.Millisecond)
        srv := s.newServer(&opts)
 
        done := make(chan bool)
@@ -650,10 +668,11 @@ func (s *runSuite) TestRunForever(c *check.C) {
 
        // Each run should send 4 pull lists + 4 trash lists. The
        // first run should also send 4 empty trash lists at
-       // startup. We should complete all four runs in much less than
-       // a second.
+       // startup. We should complete at least four runs in much less
+       // than 10s.
        for t0 := time.Now(); time.Since(t0) < 10*time.Second; {
-               if pullReqs.Count() >= 16 && trashReqs.Count() == pullReqs.Count()+4 {
+               pulls := pullReqs.Count()
+               if pulls >= 16 && trashReqs.Count() == pulls+4 {
                        break
                }
                time.Sleep(time.Millisecond)
@@ -661,8 +680,70 @@ func (s *runSuite) TestRunForever(c *check.C) {
        cancel()
        <-done
        c.Check(pullReqs.Count() >= 16, check.Equals, true)
-       c.Check(trashReqs.Count(), check.Equals, pullReqs.Count()+4)
+       c.Check(trashReqs.Count() >= 20, check.Equals, true)
+
+       // We should have completed 4 runs before calling cancel().
+       // But the next run might also have started before we called
+       // cancel(), in which case the extra run will be included in
+       // the changeset_compute_seconds_count metric.
+       completed := pullReqs.Count() / 4
+       metrics := arvadostest.GatherMetricsAsString(srv.Metrics.reg)
+       c.Check(metrics, check.Matches, fmt.Sprintf(`(?ms).*\narvados_keepbalance_changeset_compute_seconds_count (%d|%d)\n.*`, completed, completed+1))
+}
+
+func (s *runSuite) TestRunForever_TriggeredBySignal(c *check.C) {
+       s.config.ManagementToken = "xyzzy"
+       opts := RunOptions{
+               Logger: ctxlog.TestLogger(c),
+               Dumper: ctxlog.TestLogger(c),
+       }
+       s.stub.serveCurrentUserAdmin()
+       s.stub.serveFooBarFileCollections()
+       s.stub.serveKeepServices(stubServices)
+       s.stub.serveKeepstoreMounts()
+       s.stub.serveKeepstoreIndexFoo4Bar1()
+       trashReqs := s.stub.serveKeepstoreTrash()
+       pullReqs := s.stub.serveKeepstorePull()
+
+       ctx, cancel := context.WithCancel(context.Background())
+       defer cancel()
+       s.config.Collections.BalancePeriod = arvados.Duration(time.Minute)
+       srv := s.newServer(&opts)
+
+       done := make(chan bool)
+       go func() {
+               srv.runForever(ctx)
+               close(done)
+       }()
+
+       procself, err := os.FindProcess(os.Getpid())
+       c.Assert(err, check.IsNil)
+
+       // Each run should send 4 pull lists + 4 trash lists. The
+       // first run should also send 4 empty trash lists at
+       // startup. We should be able to complete four runs in much
+       // less than 10s.
+       completedRuns := 0
+       for t0 := time.Now(); time.Since(t0) < 10*time.Second; {
+               pulls := pullReqs.Count()
+               if pulls >= 16 && trashReqs.Count() == pulls+4 {
+                       break
+               }
+               // Once the 1st run has started automatically, we
+               // start sending a single SIGUSR1 at the end of each
+               // run, to ensure we get exactly 4 runs in total.
+               if pulls > 0 && pulls%4 == 0 && pulls <= 12 && pulls/4 > completedRuns {
+                       completedRuns = pulls / 4
+                       c.Logf("completed run %d, sending SIGUSR1 to trigger next run", completedRuns)
+                       procself.Signal(syscall.SIGUSR1)
+               }
+               time.Sleep(time.Millisecond)
+       }
+       cancel()
+       <-done
+       c.Check(pullReqs.Count(), check.Equals, 16)
+       c.Check(trashReqs.Count(), check.Equals, 20)
 
        metrics := arvadostest.GatherMetricsAsString(srv.Metrics.reg)
-       c.Check(metrics, check.Matches, `(?ms).*\narvados_keepbalance_changeset_compute_seconds_count `+fmt.Sprintf("%d", pullReqs.Count()/4)+`\n.*`)
+       c.Check(metrics, check.Matches, `(?ms).*\narvados_keepbalance_changeset_compute_seconds_count 4\n.*`)
 }