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
[Merged by Bors] - Add progress bar to cluster local setup #1574
Conversation
6d47e03
to
0907769
Compare
0907769
to
41da8fa
Compare
Trait for rendered text Stop loop in case of failure Add spinner to progress bar Update changelog
05a8681
to
e265b8e
Compare
Still reviewing, but right off the bat I'm seeing warnings for unused Results. |
let mut progress = cluster_checker | ||
.run_and_fix_with_progress() | ||
.map(|check| Ok(LocalSetupProgressMessage::Check(check))); | ||
sender | ||
.send_all(&mut progress) | ||
.await | ||
.map_err(LocalInstallError::ChannelSendError)?; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The key difference is here. progress
is a stream and sender
is a sink, so we can plug them together with send_all
instead of needing to loop to push events through
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be good for cluster start --setup
, let's merge and then work on cluster start
fdcb197
to
7530f31
Compare
7530f31
to
f95515d
Compare
bors r + |
bors ping |
pong |
bors r+ |
Update to progress indicator for this command: `fluvio cluster start --local --setup` Co-authored-by: Nick Mosher <nicholastmosher@gmail.com>
Pull request successfully merged into master. Build succeeded: |
Update to progress indicator for this command:
fluvio cluster start --local --setup