Skip to content

Commit

Permalink
orchestrator/clickhouse: fix panic in networks CSV refresher
Browse files Browse the repository at this point in the history
Upon second refresh, we close again the "ready" channel, leading to a
panic.
  • Loading branch information
vincentbernat committed Apr 14, 2024
1 parent 02a9aef commit 1360464
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions console/data/docs/99-changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ identified with a specific icon:
## 1.10.1 - 2024-04-14

- 🩹 *inlet*: fix versioning of metadata cache
- 🩹 *orchestrator*: fix panic in networks CSV refresher

## 1.10.0 - 2024-04-08

Expand Down
1 change: 1 addition & 0 deletions orchestrator/clickhouse/networks.go
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,7 @@ func (c *Component) networksCSVRefresher() {

if once {
close(c.networksCSVReady)
once = false
}

ctx, cancel := context.WithTimeout(c.t.Context(nil), time.Minute)
Expand Down

0 comments on commit 1360464

Please sign in to comment.