Updated to include deleted_at in whitelist and updated migration to c…

…reate logidze snapshots for existing workflow executions
phac-nml · Jan 9, 2025 · 72a8bbb · 72a8bbb
1 parent 42f6b53
commit 72a8bbb
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/db/migrate/20250109162252_add_logidze_to_workflow_executions.rb b/db/migrate/20250109162252_add_logidze_to_workflow_executions.rb
@@ -5,6 +5,10 @@ def change
     reversible do |dir|
       dir.up do
         create_trigger :logidze_on_workflow_executions, on: :workflow_executions
+
+        execute <<-SQL.squish
+          UPDATE "workflow_executions" as t SET log_data = logidze_snapshot(to_jsonb(t), 'created_at', '{"run_id","name","state","deleted_at"}', true);
+        SQL
       end
 
       dir.down do

diff --git a/db/schema.rb b/db/schema.rb
diff --git a/db/triggers/logidze_on_workflow_executions_v01.sql b/db/triggers/logidze_on_workflow_executions_v01.sql
@@ -3,4 +3,4 @@ BEFORE UPDATE OR INSERT ON "workflow_executions" FOR EACH ROW
 WHEN (coalesce(current_setting('logidze.disabled', true), '') <> 'on')
 -- Parameters: history_size_limit (integer), timestamp_column (text), filtered_columns (text[]),
 -- include_columns (boolean), debounce_time_ms (integer)
-EXECUTE PROCEDURE logidze_logger(null, 'updated_at', '{run_id,name,state}', true);
+EXECUTE PROCEDURE logidze_logger(null, 'updated_at', '{run_id,name,state,deleted_at}', true);