Skip to content

skupper site status ignores -o flag #2049

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
hash-d opened this issue Apr 6, 2025 · 2 comments · May be fixed by #2062
Open

skupper site status ignores -o flag #2049

hash-d opened this issue Apr 6, 2025 · 2 comments · May be fixed by #2062
Assignees
@lynnemorrison
Labels
cli Issues associated with the CLI

Comments

@hash-d
Copy link
Member

hash-d commented Apr 6, 2025

Describe the bug

Exactly what it says on the tin

How To Reproduce

$ skupper site status -o json
NAME    STATUS  MESSAGE
asdf    Ready   OK
$ skupper site status -o yaml
NAME    STATUS  MESSAGE
asdf    Ready   OK

Expected behavior

An YAML or JSON output

Environment details

  • Skupper CLI: 2.0 and main
  • Skupper Operator (if applicable): Not applicable
  • Platform: kubernetes, openshift

Additional context

Not applicable
@hash-d
Copy link
Member Author

hash-d commented Apr 6, 2025

Additionally, on site status:

$ skupper site status 
NAME    STATUS  MESSAGE
west    Ready   OK
$ oc get sites.skupper.io 
NAME   STATUS   SITES IN NETWORK   MESSAGE
west   Ready    2                  OK

It does not relay the SITES IN NETWORK column — is that intentional?

@lynnemorrison
Copy link
Collaborator

lynnemorrison commented Apr 7, 2025 •
edited
Loading

@hash-d "sites in network" is not specified in refdog, so that is why it is not displayed in the cli output. https://skupperproject.github.io/refdog/commands/site/status.html

But it could be added if @ssorj agrees

@noe The site command does appear to be missing the -o flag processing. I can add it.

@lynnemorrison lynnemorrison self-assigned this Apr 7, 2025
@lynnemorrison lynnemorrison added the cli Issues associated with the CLI label Apr 7, 2025
@lynnemorrison lynnemorrison linked a pull request Apr 9, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lynnemorrison lynnemorrison

Labels
cli Issues associated with the CLI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add output flag support for site command lynnemorrison/skupper
2 participants
@hash-d @lynnemorrison