Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

cmd/swarm: add cli UI for tag progress on pushsync #1814

Merged
merged 2 commits into from
Sep 30, 2019
Merged

cmd/swarm: add cli UI for tag progress on pushsync #1814

merged 2 commits into from
Sep 30, 2019

Conversation

acud
Copy link
Member

@acud acud commented Sep 25, 2019

This PR adds some progress bars that are shown on the console when uploading files with the --track option.
We should also consider that right now there is no distinction between pushsync uploads and pullsync uploads, so if a user does not have pushsync enabled and uploads with this flag, the CLI will hang indefinitely because the associated tag is not updated.

We should consider integrating tags into pull syncing as well.

@acud acud added enhancement cli Swarm command line interface labels Sep 25, 2019
@acud acud added this to the 0.5.0 milestone Sep 25, 2019
@acud acud requested review from nonsense, janos and zelig September 25, 2019 12:41
@acud acud self-assigned this Sep 25, 2019
@acud acud force-pushed the tag-cli-ui branch 2 times, most recently from 6eac88e to c044c9d Compare September 25, 2019 13:03
@zelig
Copy link
Member

zelig commented Sep 25, 2019

@acud acud force-pushed the tag-cli-ui branch 2 times, most recently from e00b2bd to b872629 Compare September 26, 2019 11:08
@acud acud force-pushed the tag-cli-ui branch 2 times, most recently from 8d471b6 to a30e735 Compare September 27, 2019 16:36
api, cmd: fixes after rebase master

cmd/swarm: make progressbars show up by default

change for loop

cmd/swarm: add chunk count to progress bars

cmd/swarm: fix tests with new flag

cmd/swarm: remove number of chunks

cmd/swarm: add debug mode for progress bars
Copy link
Contributor

@nonsense nonsense left a comment

Choose a reason for hiding this comment

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

I think this can be simplified further as discussed, but good enough for a first version in my opinion.

I see you already did that. Thanks!

cmd/swarm/upload.go Outdated Show resolved Hide resolved
cmd/swarm/upload.go Outdated Show resolved Hide resolved
@acud acud merged commit 98fdadd into master Sep 30, 2019
@acud acud deleted the tag-cli-ui branch September 30, 2019 09:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cli Swarm command line interface enhancement ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants