Skip to content

Show zone server addrs#654

Merged
tertsdiepraam merged 2 commits into
mainfrom
show-zone-server-addrs
May 12, 2026
Merged

Show zone server addrs#654
tertsdiepraam merged 2 commits into
mainfrom
show-zone-server-addrs

Conversation

@tertsdiepraam
Copy link
Copy Markdown
Contributor

@tertsdiepraam tertsdiepraam commented May 12, 2026

Adds the addresses for the zone servers in two places:

  • Show the relevant review server in cascade zone status we are at a manual review
  • Show all addresses in cascade status

We now also show multiple addresses under "Published zone available at" in zone status.

❯ cascade zone status example.com
zone:   example.com
policy: foo
source: /home/terts/data/zones/example.txt

review
  loaded: manual
  signed: manual

last published
  <no versions published yet>

status: waiting for loaded review
  ✔ load (serial: 2001062501)
  ■ review loaded zone
  |   zone must be reviewed manually
  |   loaded zone is available at:
  |     - 127.0.0.1:4540
  |     - [::1]:4540
  |   possible actions:
  |     cascade zone approve --unsigned example.com 2001062501
  |     cascade zone reject --unsigned example.com 2001062501
  |
  ○ sign
  ○ review signed zone
  ○ publish

❯ cascade status
Serving zones at the following addresses:
  Loaded review:
    - 127.0.0.1:4540
    - [::1]:4540
  Signed review:
    - 127.0.0.1:4541
    - [::1]:4541
  Server:
    - 127.0.0.1:4542
    - [::1]:4542

Signing queue:
  The signing queue is currently empty.

  • If you are changing Rust code or integration tests (Cargo.*, crates/, etc/, integration-tests/, src/):
    • Did you run the integration tests with act through the act-wrapper (as described in TESTING.md)?

@tertsdiepraam tertsdiepraam requested a review from ximon18 May 12, 2026 14:25
@tertsdiepraam tertsdiepraam force-pushed the show-zone-server-addrs branch from 3aed012 to b32c62d Compare May 12, 2026 14:39
Copy link
Copy Markdown
Member

@ximon18 ximon18 left a comment

Choose a reason for hiding this comment

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

LGTM!

@tertsdiepraam tertsdiepraam merged commit d5b810c into main May 12, 2026
9 checks passed
@tertsdiepraam tertsdiepraam deleted the show-zone-server-addrs branch May 12, 2026 20:01
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