feat(etl): replace durable-job lease worker on repaired stack - #138
Closed
seonghobae wants to merge 8 commits into
Closed
feat(etl): replace durable-job lease worker on repaired stack#138seonghobae wants to merge 8 commits into
seonghobae wants to merge 8 commits into
Conversation
Contributor
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
This was referenced Aug 7, 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.
Closed as superseded by #143
This Draft pull request is closed without deleting or rewriting its branch because its strict stacked ancestry diverged from the exact live predecessor and a non-destructive replacement has now been established.
Exact superseded state
778956039f6e41d9d8f264097c6ebf14cd3306f3;ci/hourly-opencode-nvidia-nim;b24bed946d3da2e62d89ded828d877dac2f3c65c;75265fae9fcf5faac6f8b9cab63247ac86f23dd4;b24bed946d3da2e62d89ded828d877dac2f3c65c, this head 8 commits ahead of the merge base and 33 behind the live predecessor;refs/pull/138/mergesynthetic commit was generated against the old predecessor and therefore could not repair exact-current ancestry.The immediate cause was predecessor movement after this RED branch was cut. The technical root cause was that this head no longer descended from the exact current immediate predecessor. Force-pushing/rebasing this branch, merging a stale synthetic ref, or treating the historical base snapshot as current were rejected because they would destroy or misstate auditable ancestry and invalidate evidence.
Replacement
Draft PR #143 (
repair/durable-job-lease-worker-75265fa) was created from exact predecessor75265fae9fcf5faac6f8b9cab63247ac86f23dd4and replays the same eight test-only RED paths through ordinary commits. Fresh comparison proves #143 head436b13fe9f5321ba7714475cb860550b83ed57a9is exactly 8 commits ahead, 0 behind, with the exact predecessor as its merge base.Fresh #143 exact-head evidence is intentionally RED: CI
31287944859fails on Ubuntu/macOS/Windows only after exact-source checkout and exact-SHA verification, reaching the deliberately missing durable-worker production types and method boundary. Dependency Review31287944868, CycloneDX SBOM31287944840, and CodeRabbit exact-head status succeed. That evidence belongs only to #143 and is not merge authorization.Evidence preservation
This branch and its fail-first history remain intact for auditability. No check, review, approval, comment, status, base snapshot, or synthetic-merge evidence from this PR transfers to #143. No force-push or destructive history rewrite was used.
Future GREEN implementation and validation must proceed only on the exact live replacement ancestry after fresh revalidation.