Skip to content

ci: retire the moqx-000 legacy alias#484

Merged
gmarzot merged 1 commit into
mainfrom
chore/retire-moqx-000-alias
Jul 3, 2026
Merged

ci: retire the moqx-000 legacy alias#484
gmarzot merged 1 commit into
mainfrom
chore/retire-moqx-000-alias

Conversation

@gmarzot

@gmarzot gmarzot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

The interop runner no longer targets moqx-000, so ci-main's auto-deploy no longer needs to keep the alias alive.

ci-main.yml deploy job — remove the DOMAIN_ALIAS wiring:

  • the second Route53 A-record UPSERT (alias → runner IP)
  • the alias-SAN check in the cert step
  • the extra -d "$DOMAIN_ALIAS" (and now-unneeded --expand) on certbot certonly

The relay identity stays moqx-main.ci.openmoq.org.

docs/ci-architecture.md — it still called moqx-000 the auto-deploy target; updated to moqx-main.

Sequencing / safety

  • Interop runner: already off moqx-000 (confirmed).
  • moqx-analytics: repointed to moqx-main in gmarzot/moqx-analytics#1 — merge that + redeploy the log pipeline before/around this, since dropping the alias means its cert SAN and DNS stop being refreshed.
  • This does not delete the existing moqx-000 Route53 A record or force-shrink the current cert — those persist until they age out; moqx-000 TLS simply drops at the next renewal. Manual DNS record deletion can follow later.

Separate from the docker stats/jemalloc PR (#480).


This change is Reviewable

The interop runner no longer targets moqx-000, so ci-main's auto-deploy no
longer needs to maintain it. Drop the DOMAIN_ALIAS wiring from the deploy job:
the second Route53 A-record UPSERT, the alias-SAN check, and the extra
`-d $DOMAIN_ALIAS` on certbot. The relay's identity stays moqx-main.

Also refresh docs/ci-architecture.md, which still called moqx-000 the
auto-deploy target (it's moqx-main).

Note: the existing moqx-000 A record and its presence in the current cert SAN
persist until they age out — this only stops actively maintaining them. moqx-000
TLS drops off at the next cert renewal. moqx-analytics is repointed separately
(gmarzot/moqx-analytics#1); land that + redeploy the pipeline first.

@afrind afrind left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@afrind reviewed 2 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on gmarzot).

@gmarzot
gmarzot merged commit a18b11c into main Jul 3, 2026
16 checks passed
@gmarzot
gmarzot deleted the chore/retire-moqx-000-alias branch July 3, 2026 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants