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 self-assigned this Sep 25, 2019
@acud acud added this to Backlog in Swarm Core - Sprint planning via automation 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

@zelig zelig moved this from Backlog to In progress in Swarm Core - Sprint planning Sep 26, 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
Swarm Core - Sprint planning automation moved this from In progress to Done 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
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants