Skip to content

[deckhouse-cli] Add d8 CLI version to status report#408

Merged
ldmonster merged 4 commits into
mainfrom
feat/add-d8-version-to-d8-status-report
Jul 20, 2026
Merged

[deckhouse-cli] Add d8 CLI version to status report#408
ldmonster merged 4 commits into
mainfrom
feat/add-d8-version-to-d8-status-report

Conversation

@Glitchy-Sheep

@Glitchy-Sheep Glitchy-Sheep commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

The report gives no hint about which d8 binary produced it.
This adds the running CLI version so a reader can tell the version at a glance.

How it works

  • New status section cliversion, printed first in the report (above the actual report).
  • It reads version.Version, the same source as d8 --version.
  • No cluster access: the version is a property of the local binary.

Before / After

Before: d8 status shows cluster sections only; the d8 version is not in the output.

2026-07-16 AT 03 45@2x

After: the report starts with a Deckhouse CLI Version section showing the running version (e.g. v0.32.0).

2026-07-20 AT 10 52@2x

Print the running d8 CLI version at the top of `d8 status`.

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@Glitchy-Sheep Glitchy-Sheep self-assigned this Jul 16, 2026
@Glitchy-Sheep Glitchy-Sheep added the enhancement New feature or request label Jul 16, 2026
Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@Glitchy-Sheep
Glitchy-Sheep marked this pull request as ready for review July 16, 2026 00:55
@Glitchy-Sheep
Glitchy-Sheep requested a review from ldmonster as a code owner July 16, 2026 00:55
ldmonster
ldmonster previously approved these changes Jul 20, 2026
- `cliBanner` prints a faint "Generated by Deckhouse CLI <version>" line right above the report header.
- The version describes the local binary, not cluster state, so it sits outside the status sections as report metadata.
- `TestCLIBanner` checks the byline contains the label and the version.

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@ldmonster
ldmonster merged commit 80f752b into main Jul 20, 2026
5 checks passed
@ldmonster
ldmonster deleted the feat/add-d8-version-to-d8-status-report branch July 20, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants