Pegasus sweep 5/6: remove references to decommissioned pegasus infrastructure - #74757
Open
stephenliang wants to merge 1 commit into
Open
Pegasus sweep 5/6: remove references to decommissioned pegasus infrastructure#74757stephenliang wants to merge 1 commit into
stephenliang wants to merge 1 commit into
Conversation
Copy link
Copy Markdown
Member
Sorry, something went wrong.
stephenliang
marked this pull request as ready for review
August 20, 2026 16:44
stephenliang
force-pushed
the
stephen/pegasus-sweep-5-refs
branch
from
4b7ef0e to
cd15d6c
Compare
August 20, 2026 21:01
stephenliang
force-pushed
the
stephen/pegasus-sweep-5-refs
branch
from
cd15d6c to
698e849
Compare
August 21, 2026 16:29
…cture Remove repository references to infrastructure that no longer exists. None of these edits has a deploy surface. - lib/cdo/honeybadger.rb: drop the pegasus project (id 34365) from get_recent_issues; the project is retired. - lib/cdo/server_tools.rb: delete the ELB deregistration path. A read-only AWS check (2026-07-07) found zero classic ELBs; all three load balancers it targeted are gone. Delete the internal method, its caller-free wrapper deregister_frontends, and the call in deprovision_frontends, whose comment and prompt now describe only what it still does. Note: the whole ServerTools class has zero external references; deleting the rest is follow-up work. - lib/cdo/sequel.rb: drop a commented-out logging line that instructed readers to start bin/pegasus-server. - dashboard/lib/certificate_image.rb: drop a stale comment; the file never uses pegasus_dir. - aws/.gitignore and the dockerignore generated from it: drop the .pegasus-built and .pegasus-up-to-date markers; nothing writes them. Part 5/6 of the pegasus dead-code sweep (openspec change pegasus-dead-code-sweep, plan in #73702). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
stephenliang
force-pushed
the
stephen/pegasus-sweep-5-refs
branch
from
698e849 to
93e778f
Compare
August 21, 2026 17:08
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
1 participant
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.
Several files still reference pegasus infrastructure that no longer exists. None of these edits has a deploy surface; this PR is repo hygiene.
Part 5/6 of the pegasus dead-code sweep. Supersedes draft #74751.
Links
Testing story
Rails boots. ruby -c and rubocop pass on every edited file. Require smoke: cdo/honeybadger, cdo/server_tools, and cdo/sequel each load standalone. Grep gates: deregister_frontends and the marker names appear nowhere outside this diff; 34365 has no other reference.