Skip to content
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

feature(cli): adding support for the all flag when listing resources #2501

Merged
merged 2 commits into from
May 10, 2023
Merged

feature(cli): adding support for the all flag when listing resources #2501

merged 2 commits into from
May 10, 2023

Conversation

xoscar
Copy link
Collaborator

@xoscar xoscar commented May 7, 2023 •
edited
Loading

This PR enables the --all flag for the list verb for every resource.

Use tracetest list <resource> --all to get all resources at once, it can be combined with the different output types: json, yaml and pretty

Changes

  • Enables all flag from the CLI
  • Adds list endpoint for config, datastores and polling
  • Updates OpenAPI with the new list resource specs

Fixes

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@xoscar xoscar linked an issue May 7, 2023 that may be closed by this pull request
[CLI Improvements] Support all resource #2304
Closed
@xoscar xoscar self-assigned this May 9, 2023
@xoscar xoscar marked this pull request as ready for review May 9, 2023 04:59
jorgeepc
jorgeepc approved these changes May 9, 2023
View reviewed changes
Copy link
Contributor

@jorgeepc jorgeepc left a comment

Choose a reason for hiding this comment

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

💣

@xoscar xoscar merged commit ea3c7f1 into main May 10, 2023
29 checks passed
@xoscar xoscar deleted the 2304-cli-improvements-support-all-resource branch May 10, 2023 12:54
schoren pushed a commit that referenced this pull request Jun 5, 2023
@xoscar @schoren
feature(cli): adding support for the all flag when listing resources (#…
6ac60de 
…2501)

* feature(cli): adding support for the all flag when listing resources

* fixing unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jorgeepc jorgeepc jorgeepc approved these changes

@danielbdias danielbdias Awaiting requested review from danielbdias

@mathnogueira mathnogueira Awaiting requested review from mathnogueira

@schoren schoren Awaiting requested review from schoren

Assignees

@xoscar xoscar

Labels
backend CLI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CLI Improvements] Support all resource
2 participants
@xoscar @jorgeepc