From e5483adb9dd3622d7df19098a4adf723d4743b41 Mon Sep 17 00:00:00 2001 From: Ivan Zaitsev Date: Tue, 24 Mar 2026 15:53:07 -0700 Subject: [PATCH] Add 'advisor' to autorevert_events_v2 action enum The autorevert AI advisor lambda writes action='advisor' to ClickHouse, but the Enum8 column only accepted 'none', 'restart', 'revert'. This caused all advisor rows to be silently stored as 'none', breaking the dedup query (prior_advisor_exists) which filters on action='advisor'. The result was unlimited re-dispatches of the advisor workflow for the same (commit, signal) pair every ~5 minutes. Authored with Claude. --- clickhouse_db_schema/misc.autorevert_events_v2/schema.sql | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clickhouse_db_schema/misc.autorevert_events_v2/schema.sql b/clickhouse_db_schema/misc.autorevert_events_v2/schema.sql index 9ff5150872..a7e69da5fa 100644 --- a/clickhouse_db_schema/misc.autorevert_events_v2/schema.sql +++ b/clickhouse_db_schema/misc.autorevert_events_v2/schema.sql @@ -2,7 +2,7 @@ CREATE TABLE misc.autorevert_events_v2 ( `ts` DateTime DEFAULT now(), `repo` LowCardinality(String) DEFAULT 'pytorch/pytorch', - `action` Enum8('none' = 0, 'restart' = 1, 'revert' = 2), + `action` Enum8('none' = 0, 'restart' = 1, 'revert' = 2, 'advisor' = 3), `commit_sha` FixedString(40), `workflows` Array(String), `source_signal_keys` Array(String),