Skip to content

[cardano-testnet] - Status command - #6741

Merged
palas merged 15 commits into
masterfrom
status-command
Oct 28, 2026
Merged

palas merged 15 commits into
masterfrom
status-command

Conversation

@palas

@palas palas commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Description

This PR adds a status command for cardano-testnet tool, that checks the status of a testnet generated by cardano-testnet. It takes a testnet configuration dir path, and an optional --json flag that makes the output format be JSON following the schema also defined in this PR.

If the --json format is not specified, the output format is human readable, and includes an ASCII table that shows the information about each of the nodes in the testnet.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. These may include:
    • golden tests
    • property tests
    • roundtrip tests
    • integration tests
      See Running tests for more details
  • Any changes are noted in the CHANGELOG.md for affected package
    • cardano-node-chairman, cardano-submit-api and cardano-testnet instead need a
      changelog fragment in <package>/.changes/, because their CHANGELOG.md is generated
      from fragments at release time. Copy _TEMPLATE.yml from that directory, or run
      nix run github:input-output-hk/cardano-dev#herald -- new
  • The version bounds in .cabal files are updated
  • CI passes. See note on CI. The following CI checks are required:
    • Code is linted with hlint. See .github/workflows/check-hlint.yml to get the hlint version
    • Code is formatted with stylish-haskell. See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
    • Code builds on Linux, MacOS and Windows for ghc-9.6 and ghc-9.12
  • Self-reviewed the diff

@palas palas self-assigned this Oct 20, 2026
@palas
palas requested review from a team as code owners October 20, 2026 19:37
@palas palas linked an issue Oct 20, 2026 that may be closed by this pull request
Comment thread cardano-testnet/src/Testnet/Status/Run.hs Outdated
@Jimbo4350
Jimbo4350 self-requested a review October 22, 2026 13:11

@carbolymer carbolymer 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.

Exception handling needs improving.

We could improve error handling code by not using ExceptT.

Comment thread cardano-testnet/src/Testnet/Status/Run.hs
Comment thread cardano-testnet/src/Testnet/Status/Run.hs Outdated
Comment thread cardano-testnet/src/Testnet/Status/Run.hs
Comment thread cardano-testnet/README.md
Comment thread cardano-testnet/src/Testnet/Status/Types.hs
Comment thread cardano-testnet/src/Testnet/Status/Types.hs Outdated
Comment thread cardano-testnet/src/Testnet/Status/Run.hs Outdated
Comment thread cardano-testnet/src/Testnet/Status/Types.hs
Comment thread cardano-testnet/src/Testnet/Status/Run.hs
Comment thread cardano-testnet/src/Testnet/Status/Run.hs Outdated
Comment thread cardano-testnet/src/Testnet/Status/Run.hs
Comment thread cardano-testnet/test/cardano-testnet-test/Cardano/Testnet/Test/Status.hs Outdated
Comment thread cardano-testnet/src/Testnet/Status/Run.hs
Comment thread cardano-testnet/src/Testnet/Status/Run.hs
Comment thread cardano-testnet/test/cardano-testnet-golden/Cardano/Testnet/Test/Golden/Status.hs Outdated
@Jimbo4350
Jimbo4350 self-requested a review October 27, 2026 14:02

@Jimbo4350 Jimbo4350 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.

LGTM

@palas
palas enabled auto-merge October 27, 2026 19:56
@palas
palas added this pull request to the merge queue Oct 27, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Oct 27, 2026
@palas
palas enabled auto-merge October 27, 2026 21:50
@palas
palas added this pull request to the merge queue Oct 28, 2026
Merged via the queue into master with commit cf70fb6 Oct 28, 2026
38 of 40 checks passed
@palas
palas deleted the status-command branch October 28, 2026 02:30
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.

[FR] cardano-testnet: add a status command

3 participants