Skip to content

Commit 3a49cc7

Browse files
authored
[nexus] re-enable sp_ereport_ingester post-R16 (#8838)
For R16, the `sp_ereport_ingester` background task was disabled in the production Nexus config file (see #8709). This is because the corresponding Hubris code for evacuating ereports from the SP had not yet merged, resulting in Nexus yelling constantly about trying to collect ereports from SPs that weren't listening on the ereport port. Now, however, oxidecomputer/hubris#2126 has merged, and R16 has been cut, so we can turn this back on. This commit does that.
1 parent b36f032 commit 3a49cc7

File tree

2 files changed

+0
-8
lines changed

2 files changed

+0
-8
lines changed

smf/nexus/multi-sled/config-partial.toml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,10 +94,6 @@ read_only_region_replacement_start.period_secs = 30
9494
alert_dispatcher.period_secs = 60
9595
webhook_deliverator.period_secs = 60
9696
sp_ereport_ingester.period_secs = 30
97-
# Disabled in R16, as the Hubris task that handles ereport ingestion requests
98-
# has not merged yet, and trying to ingest them will just result in Nexus
99-
# logging a bunch of errors.
100-
sp_ereport_ingester.disable = true
10197

10298
[default_region_allocation_strategy]
10399
# by default, allocate across 3 distinct sleds

smf/nexus/single-sled/config-partial.toml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,10 +94,6 @@ read_only_region_replacement_start.period_secs = 30
9494
alert_dispatcher.period_secs = 60
9595
webhook_deliverator.period_secs = 60
9696
sp_ereport_ingester.period_secs = 30
97-
# Disabled in R16, as the Hubris task that handles ereport ingestion requests
98-
# has not merged yet, and trying to ingest them will just result in Nexus
99-
# logging a bunch of errors.
100-
sp_ereport_ingester.disable = true
10197

10298
[default_region_allocation_strategy]
10399
# by default, allocate without requirement for distinct sleds.

0 commit comments

Comments
 (0)