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.

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

Add output streaming support to sensuctl #4238

Open
echlebek opened this issue Mar 25, 2021 · 2 comments
Open

Add output streaming support to sensuctl #4238

echlebek opened this issue Mar 25, 2021 · 2 comments
Labels
bug component:sensuctl Sensu CLI improvements

Comments

@echlebek
Copy link
Contributor

Expected Behavior

When listing resources with sensuctl and using the --chunk-size option, I expect to receive each chunk as it arrives.

Current Behavior

The chunks are collated until there are no more chunks to bad had, before being output.

Possible Solution

Add output streaming support to sensuctl. This will necessitate rewriting much of the output formatting code, which is due for an overhaul in any case.

Steps to Reproduce (for bugs)

  1. Try to output tens of thousands of resources with a small chunk size. (i.e. sensuctl event list --chunk-size 100)
  2. Observe that the operation takes a long time to return any information.
@echlebek echlebek added the bug label Mar 25, 2021
@ccressent
Copy link
Contributor

Some context: #2966

@calebhailey
Copy link

One of the comments in #2966 that caught my attention was RE: tabular output column widths, which is something I'd love to revisit. The way we render columns in certain sensuctl outputs today is quite cumbersome because we don't place any limits on the output (e.g. if an event has a very long output the columnar output of sensuctl event list is a mess). I'd love to see us adopt some sort of truncation, if not completely revisit the design of columnar list output.

@calebhailey calebhailey added the component:sensuctl Sensu CLI improvements label Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component:sensuctl Sensu CLI improvements
Projects
None yet
Development

No branches or pull requests

3 participants