| Tim Windelschmidt | 6d33a43 | 2025-02-04 14:34:25 +0100 | [diff] [blame] | 1 | // Copyright The Monogon Project Authors. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 2 | // SPDX-License-Identifier: Apache-2.0 |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 3 | |
| 4 | package supervisor |
| 5 | |
| 6 | import ( |
| 7 | "context" |
| 8 | "errors" |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 9 | "fmt" |
| 10 | "runtime/debug" |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 11 | "sort" |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 12 | "time" |
| Tim Windelschmidt | 22c1a2a | 2025-03-24 17:23:17 +0100 | [diff] [blame^] | 13 | |
| 14 | "source.monogon.dev/osbase/logtree" |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 15 | ) |
| 16 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 17 | // The processor maintains runnable goroutines - ie., when requested will start |
| Jan Schär | aa6b42a | 2024-12-18 18:03:26 +0100 | [diff] [blame] | 18 | // one, and then once it exits, it will record the result and act accordingly. |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 19 | // It is also responsible for detecting and acting upon supervision subtrees |
| 20 | // that need to be restarted after death (via a 'GC' process) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 21 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 22 | // processorRequest is a request for the processor. Only one of the fields can |
| 23 | // be set. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 24 | type processorRequest struct { |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 25 | schedule *processorRequestSchedule |
| 26 | died *processorRequestDied |
| 27 | waitSettled *processorRequestWaitSettled |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 28 | } |
| 29 | |
| 30 | // processorRequestSchedule requests that a given node's runnable be started. |
| 31 | type processorRequestSchedule struct { |
| 32 | dn string |
| 33 | } |
| 34 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 35 | // processorRequestDied is a signal from a runnable goroutine that the runnable |
| 36 | // has died. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 37 | type processorRequestDied struct { |
| 38 | dn string |
| 39 | err error |
| 40 | } |
| 41 | |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 42 | type processorRequestWaitSettled struct { |
| 43 | waiter chan struct{} |
| 44 | } |
| 45 | |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 46 | // processor is the main processing loop. |
| 47 | func (s *supervisor) processor(ctx context.Context) { |
| 48 | s.ilogger.Info("supervisor processor started") |
| 49 | |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 50 | // Waiters waiting for the GC to be settled. |
| 51 | var waiters []chan struct{} |
| 52 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 53 | // The GC will run every millisecond if needed. Any time the processor |
| 54 | // requests a change in the supervision tree (ie a death or a new runnable) |
| 55 | // it will mark the state as dirty and run the GC on the next millisecond |
| 56 | // cycle. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 57 | gc := time.NewTicker(1 * time.Millisecond) |
| 58 | defer gc.Stop() |
| 59 | clean := true |
| 60 | |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 61 | // How long has the GC been clean. This is used to notify 'settled' waiters. |
| 62 | cleanCycles := 0 |
| 63 | |
| 64 | markDirty := func() { |
| 65 | clean = false |
| 66 | cleanCycles = 0 |
| 67 | } |
| 68 | |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 69 | for { |
| 70 | select { |
| 71 | case <-ctx.Done(): |
| Serge Bazanski | c735967 | 2020-10-30 16:38:57 +0100 | [diff] [blame] | 72 | s.ilogger.Infof("supervisor processor exiting: %v", ctx.Err()) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 73 | s.processKill() |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 74 | s.ilogger.Info("supervisor exited, starting liquidator to clean up remaining runnables...") |
| 75 | go s.liquidator() |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 76 | return |
| 77 | case <-gc.C: |
| 78 | if !clean { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 79 | s.processGC() |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 80 | } |
| 81 | clean = true |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 82 | cleanCycles += 1 |
| 83 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 84 | // This threshold is somewhat arbitrary. It's a balance between |
| 85 | // test speed and test reliability. |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 86 | if cleanCycles > 50 { |
| 87 | for _, w := range waiters { |
| 88 | close(w) |
| 89 | } |
| 90 | waiters = nil |
| 91 | } |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 92 | case r := <-s.pReq: |
| 93 | switch { |
| 94 | case r.schedule != nil: |
| 95 | s.processSchedule(r.schedule) |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 96 | markDirty() |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 97 | case r.died != nil: |
| 98 | s.processDied(r.died) |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 99 | markDirty() |
| 100 | case r.waitSettled != nil: |
| 101 | waiters = append(waiters, r.waitSettled.waiter) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 102 | default: |
| 103 | panic(fmt.Errorf("unhandled request %+v", r)) |
| 104 | } |
| 105 | } |
| 106 | } |
| 107 | } |
| 108 | |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 109 | // The liquidator is a context-free goroutine which the supervisor starts after |
| 110 | // its context has been canceled. Its job is to take over listening on the |
| 111 | // processing channels that the supervisor processor would usually listen on, |
| 112 | // and implement the minimum amount of logic required to mark existing runnables |
| 113 | // as DEAD. |
| 114 | // |
| 115 | // It exits when all runnables have exited one way or another, and the |
| 116 | // supervision tree is well and truly dead. This will also be reflected by |
| 117 | // liveRunnables returning an empty list. |
| 118 | func (s *supervisor) liquidator() { |
| 119 | for { |
| Tim Windelschmidt | 931b3a3 | 2024-04-18 23:39:20 +0200 | [diff] [blame] | 120 | r := <-s.pReq |
| 121 | switch { |
| 122 | case r.schedule != nil: |
| 123 | s.ilogger.Infof("liquidator: refusing to schedule %s", r.schedule.dn) |
| 124 | s.mu.Lock() |
| 125 | n := s.nodeByDN(r.schedule.dn) |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 126 | n.state = NodeStateDead |
| Tim Windelschmidt | 931b3a3 | 2024-04-18 23:39:20 +0200 | [diff] [blame] | 127 | s.mu.Unlock() |
| 128 | case r.died != nil: |
| 129 | s.ilogger.Infof("liquidator: %s exited", r.died.dn) |
| 130 | s.mu.Lock() |
| 131 | n := s.nodeByDN(r.died.dn) |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 132 | n.state = NodeStateDead |
| Tim Windelschmidt | 931b3a3 | 2024-04-18 23:39:20 +0200 | [diff] [blame] | 133 | s.mu.Unlock() |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 134 | } |
| 135 | live := s.liveRunnables() |
| 136 | if len(live) == 0 { |
| 137 | s.ilogger.Infof("liquidator: complete, all runnables dead or done") |
| 138 | return |
| 139 | } |
| 140 | } |
| 141 | } |
| 142 | |
| Jan Schär | aa6b42a | 2024-12-18 18:03:26 +0100 | [diff] [blame] | 143 | // liveRunnables returns a list of runnable DNs that aren't DONE/DEAD/CANCELED. |
| 144 | // This is used by the liquidator to figure out when its job is done, and by the |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 145 | // TestHarness to know when to unblock the test cleanup function. |
| 146 | func (s *supervisor) liveRunnables() []string { |
| 147 | s.mu.RLock() |
| 148 | defer s.mu.RUnlock() |
| 149 | |
| Jan Schär | aa6b42a | 2024-12-18 18:03:26 +0100 | [diff] [blame] | 150 | // DFS through supervision tree, making note of live (non-DONE/DEAD/CANCELED |
| 151 | // runnables). |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 152 | var live []string |
| 153 | seen := make(map[string]bool) |
| 154 | q := []*node{s.root} |
| 155 | for { |
| 156 | if len(q) == 0 { |
| 157 | break |
| 158 | } |
| 159 | |
| 160 | // Pop from DFS queue. |
| 161 | el := q[0] |
| 162 | q = q[1:] |
| 163 | |
| 164 | // Skip already visited runnables (this shouldn't happen because the supervision |
| 165 | // tree is, well, a tree - but better stay safe than get stuck in a loop). |
| 166 | eldn := el.dn() |
| 167 | if seen[eldn] { |
| 168 | continue |
| 169 | } |
| 170 | seen[eldn] = true |
| 171 | |
| Jan Schär | aa6b42a | 2024-12-18 18:03:26 +0100 | [diff] [blame] | 172 | if el.state != NodeStateDead && el.state != NodeStateDone && el.state != NodeStateCanceled { |
| Serge Bazanski | ec19b60 | 2022-03-09 20:41:31 +0100 | [diff] [blame] | 173 | live = append(live, eldn) |
| 174 | } |
| 175 | |
| 176 | // Recurse. |
| 177 | for _, child := range el.children { |
| 178 | q = append(q, child) |
| 179 | } |
| 180 | } |
| 181 | |
| 182 | sort.Strings(live) |
| 183 | return live |
| 184 | } |
| 185 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 186 | // processKill cancels all nodes in the supervision tree. This is only called |
| 187 | // right before exiting the processor, so they do not get automatically |
| 188 | // restarted. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 189 | func (s *supervisor) processKill() { |
| 190 | s.mu.Lock() |
| 191 | defer s.mu.Unlock() |
| 192 | |
| 193 | // Gather all context cancel functions. |
| 194 | var cancels []func() |
| 195 | queue := []*node{s.root} |
| 196 | for { |
| 197 | if len(queue) == 0 { |
| 198 | break |
| 199 | } |
| 200 | |
| 201 | cur := queue[0] |
| 202 | queue = queue[1:] |
| 203 | |
| 204 | cancels = append(cancels, cur.ctxC) |
| 205 | for _, c := range cur.children { |
| 206 | queue = append(queue, c) |
| 207 | } |
| 208 | } |
| 209 | |
| 210 | // Call all context cancels. |
| 211 | for _, c := range cancels { |
| 212 | c() |
| 213 | } |
| 214 | } |
| 215 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 216 | // processSchedule starts a node's runnable in a goroutine and records its |
| 217 | // output once it's done. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 218 | func (s *supervisor) processSchedule(r *processorRequestSchedule) { |
| 219 | s.mu.Lock() |
| 220 | defer s.mu.Unlock() |
| 221 | |
| 222 | n := s.nodeByDN(r.dn) |
| Serge Bazanski | cf864da | 2024-07-31 11:23:34 +0000 | [diff] [blame] | 223 | if n.state != NodeStateNew { |
| 224 | panic("programming error: scheduled node not new") |
| 225 | } |
| 226 | s.metrics.NotifyNodeState(r.dn, n.state) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 227 | go func() { |
| Serge Bazanski | 19bb412 | 2020-05-04 17:57:50 +0200 | [diff] [blame] | 228 | if !s.propagatePanic { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 229 | defer func() { |
| 230 | if rec := recover(); rec != nil { |
| 231 | s.pReq <- &processorRequest{ |
| 232 | died: &processorRequestDied{ |
| 233 | dn: r.dn, |
| 234 | err: fmt.Errorf("panic: %v, stacktrace: %s", rec, string(debug.Stack())), |
| 235 | }, |
| 236 | } |
| 237 | } |
| 238 | }() |
| 239 | } |
| 240 | |
| 241 | res := n.runnable(n.ctx) |
| 242 | |
| 243 | s.pReq <- &processorRequest{ |
| 244 | died: &processorRequestDied{ |
| 245 | dn: r.dn, |
| 246 | err: res, |
| 247 | }, |
| 248 | } |
| 249 | }() |
| 250 | } |
| 251 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 252 | // processDied records the result from a runnable goroutine, and updates its |
| 253 | // node state accordingly. If the result is a death and not an expected exit, |
| 254 | // related nodes (ie. children and group siblings) are canceled accordingly. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 255 | func (s *supervisor) processDied(r *processorRequestDied) { |
| 256 | s.mu.Lock() |
| 257 | defer s.mu.Unlock() |
| 258 | |
| 259 | // Okay, so a Runnable has quit. What now? |
| 260 | n := s.nodeByDN(r.dn) |
| 261 | ctx := n.ctx |
| 262 | |
| Jan Schär | 08c1c72 | 2024-12-19 12:03:17 +0100 | [diff] [blame] | 263 | // Simple case: it has signaled Done and quit with no error. |
| 264 | if n.signaledDone && r.err == nil { |
| 265 | // Mark the node as DONE. |
| 266 | n.state = NodeStateDone |
| Serge Bazanski | cf864da | 2024-07-31 11:23:34 +0000 | [diff] [blame] | 267 | s.metrics.NotifyNodeState(r.dn, n.state) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 268 | return |
| 269 | } |
| 270 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 271 | // Simple case: the context was canceled and the returned error is the |
| 272 | // context error. |
| Tim Windelschmidt | 47d0344 | 2024-04-23 15:08:44 +0200 | [diff] [blame] | 273 | if r.err != nil && ctx.Err() != nil && errors.Is(r.err, ctx.Err()) { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 274 | // Mark the node as canceled successfully. |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 275 | n.state = NodeStateCanceled |
| Serge Bazanski | cf864da | 2024-07-31 11:23:34 +0000 | [diff] [blame] | 276 | s.metrics.NotifyNodeState(r.dn, n.state) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 277 | return |
| 278 | } |
| 279 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 280 | // Otherwise, the Runnable should not have died or quit. Handle |
| 281 | // accordingly. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 282 | err := r.err |
| 283 | // A lack of returned error is also an error. |
| 284 | if err == nil { |
| Serge Bazanski | 0164c71 | 2023-03-16 17:54:07 +0100 | [diff] [blame] | 285 | err = fmt.Errorf("returned nil when %s", n.state) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 286 | } |
| 287 | |
| Tim Windelschmidt | 22c1a2a | 2025-03-24 17:23:17 +0100 | [diff] [blame^] | 288 | s.logtree.MustLeveledFor(logtree.DN(n.dn())).Fatalf("runnable returned error: %v", err) |
| 289 | |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 290 | // Mark as dead. |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 291 | n.state = NodeStateDead |
| Serge Bazanski | cf864da | 2024-07-31 11:23:34 +0000 | [diff] [blame] | 292 | s.metrics.NotifyNodeState(r.dn, n.state) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 293 | |
| 294 | // Cancel that node's context, just in case something still depends on it. |
| 295 | n.ctxC() |
| 296 | |
| 297 | // Cancel all siblings. |
| 298 | if n.parent != nil { |
| Tim Windelschmidt | 6b6428d | 2024-04-11 01:35:41 +0200 | [diff] [blame] | 299 | for name := range n.parent.groupSiblings(n.name) { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 300 | if name == n.name { |
| 301 | continue |
| 302 | } |
| 303 | sibling := n.parent.children[name] |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 304 | // TODO(q3k): does this need to run in a goroutine, ie. can a |
| 305 | // context cancel block? |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 306 | sibling.ctxC() |
| 307 | } |
| 308 | } |
| 309 | } |
| 310 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 311 | // processGC runs the GC process. It's not really Garbage Collection, as in, it |
| 312 | // doesn't remove unnecessary tree nodes - but it does find nodes that need to |
| 313 | // be restarted, find the subset that can and then schedules them for running. |
| 314 | // As such, it's less of a Garbage Collector and more of a Necromancer. |
| 315 | // However, GC is a friendlier name. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 316 | func (s *supervisor) processGC() { |
| 317 | s.mu.Lock() |
| 318 | defer s.mu.Unlock() |
| 319 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 320 | // The 'GC' serves is the main business logic of the supervision tree. It |
| 321 | // traverses a locked tree and tries to find subtrees that must be |
| 322 | // restarted (because of a DEAD/CANCELED runnable). It then finds which of |
| 323 | // these subtrees that should be restarted can be restarted, ie. which ones |
| 324 | // are fully recursively DEAD/CANCELED. It also finds the smallest set of |
| 325 | // largest subtrees that can be restarted, ie. if there's multiple DEAD |
| 326 | // runnables that can be restarted at once, it will do so. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 327 | |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 328 | // Phase one: Find all leaves. |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 329 | // This is a simple DFS that finds all the leaves of the tree, ie all nodes |
| 330 | // that do not have children nodes. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 331 | leaves := make(map[string]bool) |
| 332 | queue := []*node{s.root} |
| 333 | for { |
| 334 | if len(queue) == 0 { |
| 335 | break |
| 336 | } |
| 337 | cur := queue[0] |
| 338 | queue = queue[1:] |
| 339 | |
| 340 | for _, c := range cur.children { |
| 341 | queue = append([]*node{c}, queue...) |
| 342 | } |
| 343 | |
| 344 | if len(cur.children) == 0 { |
| 345 | leaves[cur.dn()] = true |
| 346 | } |
| 347 | } |
| 348 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 349 | // Phase two: traverse tree from node to root and make note of all subtrees |
| 350 | // that can be restarted. |
| 351 | // A subtree is restartable/ready iff every node in that subtree is either |
| 352 | // CANCELED, DEAD or DONE. Such a 'ready' subtree can be restarted by the |
| 353 | // supervisor if needed. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 354 | |
| 355 | // DNs that we already visited. |
| 356 | visited := make(map[string]bool) |
| 357 | // DNs whose subtrees are ready to be restarted. |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 358 | // These are all subtrees recursively - ie., root.a.a and root.a will both |
| 359 | // be marked here. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 360 | ready := make(map[string]bool) |
| 361 | |
| 362 | // We build a queue of nodes to visit, starting from the leaves. |
| 363 | queue = []*node{} |
| Tim Windelschmidt | 6b6428d | 2024-04-11 01:35:41 +0200 | [diff] [blame] | 364 | for l := range leaves { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 365 | queue = append(queue, s.nodeByDN(l)) |
| 366 | } |
| 367 | |
| 368 | for { |
| 369 | if len(queue) == 0 { |
| 370 | break |
| 371 | } |
| 372 | |
| 373 | cur := queue[0] |
| 374 | curDn := cur.dn() |
| 375 | |
| 376 | queue = queue[1:] |
| 377 | |
| 378 | // Do we have a decision about our children? |
| 379 | allVisited := true |
| 380 | for _, c := range cur.children { |
| 381 | if !visited[c.dn()] { |
| 382 | allVisited = false |
| 383 | break |
| 384 | } |
| 385 | } |
| 386 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 387 | // If no decision about children is available, it means we ended up in |
| 388 | // this subtree through some shorter path of a shorter/lower-order |
| 389 | // leaf. There is a path to a leaf that's longer than the one that |
| 390 | // caused this node to be enqueued. Easy solution: just push back the |
| 391 | // current element and retry later. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 392 | if !allVisited { |
| 393 | // Push back to queue and wait for a decision later. |
| 394 | queue = append(queue, cur) |
| 395 | continue |
| 396 | } |
| 397 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 398 | // All children have been visited and we have an idea about whether |
| 399 | // they're ready/restartable. All of the node's children must be |
| 400 | // restartable in order for this node to be restartable. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 401 | childrenReady := true |
| Serge Bazanski | ba7bf7d | 2021-10-29 16:59:00 +0200 | [diff] [blame] | 402 | var childrenNotReady []string |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 403 | for _, c := range cur.children { |
| 404 | if !ready[c.dn()] { |
| Serge Bazanski | ba7bf7d | 2021-10-29 16:59:00 +0200 | [diff] [blame] | 405 | childrenNotReady = append(childrenNotReady, c.dn()) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 406 | childrenReady = false |
| 407 | break |
| 408 | } |
| 409 | } |
| 410 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 411 | // In addition to children, the node itself must be restartable (ie. |
| 412 | // DONE, DEAD or CANCELED). |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 413 | curReady := false |
| 414 | switch cur.state { |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 415 | case NodeStateDone: |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 416 | curReady = true |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 417 | case NodeStateCanceled: |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 418 | curReady = true |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 419 | case NodeStateDead: |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 420 | curReady = true |
| Tim Windelschmidt | 9b2c156 | 2024-04-11 01:39:25 +0200 | [diff] [blame] | 421 | default: |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 422 | } |
| 423 | |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 424 | if cur.state == NodeStateDead && !childrenReady { |
| Serge Bazanski | ba7bf7d | 2021-10-29 16:59:00 +0200 | [diff] [blame] | 425 | s.ilogger.Warningf("Not restarting %s: children not ready to be restarted: %v", curDn, childrenNotReady) |
| 426 | } |
| 427 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 428 | // Note down that we have an opinion on this node, and note that |
| 429 | // opinion down. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 430 | visited[curDn] = true |
| 431 | ready[curDn] = childrenReady && curReady |
| 432 | |
| 433 | // Now we can also enqueue the parent of this node for processing. |
| 434 | if cur.parent != nil && !visited[cur.parent.dn()] { |
| 435 | queue = append(queue, cur.parent) |
| 436 | } |
| 437 | } |
| 438 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 439 | // Phase 3: traverse tree from root to find largest subtrees that need to |
| 440 | // be restarted and are ready to be restarted. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 441 | |
| 442 | // All DNs that need to be restarted by the GC process. |
| 443 | want := make(map[string]bool) |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 444 | // All DNs that need to be restarted and can be restarted by the GC process |
| 445 | // - a subset of 'want' DNs. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 446 | can := make(map[string]bool) |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 447 | // The set difference between 'want' and 'can' are all nodes that should be |
| 448 | // restarted but can't yet (ie. because a child is still in the process of |
| 449 | // being canceled). |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 450 | |
| 451 | // DFS from root. |
| 452 | queue = []*node{s.root} |
| 453 | for { |
| 454 | if len(queue) == 0 { |
| 455 | break |
| 456 | } |
| 457 | |
| 458 | cur := queue[0] |
| 459 | queue = queue[1:] |
| 460 | |
| Jan Schär | fce7c76 | 2024-12-19 14:07:24 +0100 | [diff] [blame] | 461 | // If this node's context is canceled and it has exited, it should be |
| 462 | // restarted. |
| 463 | exited := cur.state == NodeStateDead || cur.state == NodeStateCanceled || cur.state == NodeStateDone |
| 464 | if cur.ctx.Err() != nil && exited { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 465 | want[cur.dn()] = true |
| 466 | } |
| 467 | |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 468 | // If it should be restarted and is ready to be restarted... |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 469 | if want[cur.dn()] && ready[cur.dn()] { |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 470 | // And its parent context is valid (ie hasn't been canceled), mark |
| 471 | // it as restartable. |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 472 | if cur.parent == nil || cur.parent.ctx.Err() == nil { |
| 473 | can[cur.dn()] = true |
| 474 | continue |
| 475 | } |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 476 | } |
| 477 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 478 | // Otherwise, traverse further down the tree to see if something else |
| 479 | // needs to be done. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 480 | for _, c := range cur.children { |
| 481 | queue = append(queue, c) |
| 482 | } |
| 483 | } |
| 484 | |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 485 | // Reinitialize and reschedule all subtrees |
| Tim Windelschmidt | 6b6428d | 2024-04-11 01:35:41 +0200 | [diff] [blame] | 486 | for dn := range can { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 487 | n := s.nodeByDN(dn) |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 488 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 489 | // Only back off when the node unexpectedly died - not when it got |
| 490 | // canceled. |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 491 | bo := time.Duration(0) |
| Serge Bazanski | eca8ee3 | 2024-07-30 14:32:19 +0000 | [diff] [blame] | 492 | if n.state == NodeStateDead { |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 493 | bo = n.bo.NextBackOff() |
| 494 | } |
| Serge Bazanski | ac6b644 | 2020-05-06 19:13:43 +0200 | [diff] [blame] | 495 | |
| Serge Bazanski | 216fe7b | 2021-05-21 18:36:16 +0200 | [diff] [blame] | 496 | // Prepare node for rescheduling - remove its children, reset its state |
| 497 | // to new. |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 498 | n.reset() |
| Serge Bazanski | c735967 | 2020-10-30 16:38:57 +0100 | [diff] [blame] | 499 | s.ilogger.Infof("rescheduling supervised node %s with backoff %s", dn, bo.String()) |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 500 | |
| 501 | // Reschedule node runnable to run after backoff. |
| 502 | go func(n *node, bo time.Duration) { |
| Jan Schär | 6560209 | 2024-12-19 10:37:34 +0100 | [diff] [blame] | 503 | select { |
| 504 | case <-time.After(bo): |
| 505 | s.pReq <- &processorRequest{ |
| 506 | schedule: &processorRequestSchedule{dn: n.dn()}, |
| 507 | } |
| 508 | case <-n.ctx.Done(): |
| 509 | s.pReq <- &processorRequest{ |
| 510 | died: &processorRequestDied{ |
| 511 | dn: n.dn(), |
| 512 | err: n.ctx.Err(), |
| 513 | }, |
| 514 | } |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 515 | } |
| 516 | }(n, bo) |
| 517 | } |
| Serge Bazanski | 9c09c4e | 2020-03-24 13:58:01 +0100 | [diff] [blame] | 518 | } |