Remove dead differ-state-reporter stack and its workflow - #442
Merged
Merged
Conversation
The nested environment-reporter stack deployed a lambda, differ-state-reporter, intended to report differ's Terraform state file to a Kosli environment named terraform-state-differ-<env> every minute. It had been broken since ~2025-04: every invocation crashed at init with Runtime.InvalidEntrypoint, so it never reported anything, and its target Kosli environments were never created. It was superseded by the consolidated terraform-statefile-paths-reporter, which covers every service at once. The live AWS resources have now been destroyed by hand in both accounts (beta 244531986313 and prod 274425519734), 16 resources each, so removing the configuration no longer risks orphaning them. Delete the directory and the workflow together: the workflow triggers on pushes under the stack's path with tf_apply set, so removing the directory alone would fire a run against a working_directory that no longer exists. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
gsavage
force-pushed
the
remove-old-environment-reporter
branch
from
July 30, 2026 14:10
54b1ca0 to
ea5012e
Compare
JonJagger
approved these changes
Jul 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes the nested
deployment/terraform/environment-reporter/stack and thetf-environment-reporter.ymlworkflow that applied it.The stack deployed a lambda,
differ-state-reporter, meant to report differ's Terraformstate file to a Kosli environment named
terraform-state-differ-<env>on arate(1 minute)cron. It had been broken since ~2025-04: every invocation crashed at init with
Runtime.InvalidEntrypoint, so it never reported anything and its target Kosli environmentswere never created. It is superseded by a consolidated state-file reporter that covers every
service at once, and differ was the only service still carrying a per-service reporter of
this kind.
The live AWS resources were destroyed by hand first, in both accounts — beta
244531986313and prod274425519734, 16 resources each (the lambda and its permissions,two EventBridge rules and their targets, the IAM role, two policies and their attachments,
and the log group). Neither destroy touched anything shared: the state buckets and the
kosli_api_tokenSSM parameter are read-only inputs to the stack, and the stack's own statelives under a separate key from the differ app stack. So this PR removes configuration for
infrastructure that no longer exists, rather than orphaning anything.
The directory and the workflow go together deliberately. The workflow triggers on pushes
under the stack's path with
tf_apply: 'true', so deleting the directory on its own wouldfire a run against a
working_directorythat no longer exists.docs/dead-state-reporter.mdis marked completed rather than deleted, so the fullinvestigation record stays in the repo.
🤖 Generated with Claude Code