Snap install download progress neatening up

Bug #1584590 reported by Mark Shuttleworth on 2016-05-23
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
snapd
High
Unassigned

Bug Description

The snap download progress indicator could be neater. How about this:

  $ snap install firefox chrome telegram
  firefox installed
  thunderbird installed
  telegram 90% [=====================================] 11.3M / 12.2M

So it would be:

   developer/name %%%% [=== progress bar ] size / size

We would show all the downloads (because in future we much run several in parallel, yay HTTP/2) at the bottom of the list and above that the set of installs.

description: updated
Michael Vogt (mvo) on 2016-05-30
Changed in snappy:
status: New → Triaged
importance: Undecided → High
Mark Shuttleworth (sabdfl) wrote :

When it's done it would look just like this:

  $ snap install firefox chrome telegram
  firefox installed
  thunderbird installed
  telegram installed
  $

Michael Vogt (mvo) wrote :
Changed in snappy:
status: Triaged → In Progress
Michael Vogt (mvo) on 2016-09-23
Changed in snappy:
status: In Progress → Fix Committed
Michael Vogt (mvo) wrote :

This is progressing forward again, sorry for the slowness here, the reason was that we had to wait for vendoring of dependencies to land first because our progressbar implementation in the package lacked some of the features we needed.

https://github.com/snapcore/snapd/pull/1956 has landed (the bit that shows the snap name before the progress bar) and https://github.com/snapcore/snapd/pull/1957 for install/remove of multi snaps at the same time is under review. Once that is landed the output looks like this currently:

$ sudo ./snap install go-example-webserver krita hello
krita (stable) 3.0.1-snap14 from 'krita' installed
go-example-webserver (stable) 16.04-6 from 'canonical' installed
hello (stable) 2.10 from 'canonical' installed

Please let us know if this is too verbose and if we should omit the extra information like channel or version.

Michael Vogt (mvo) wrote :

This is in the 2.16 release that is currently in yakkety-proposed and xenial-proposed.

On 23/09/16 07:50, Michael Vogt wrote:
> $ sudo ./snap install go-example-webserver krita hello
> krita (stable) 3.0.1-snap14 from 'krita' installed
> go-example-webserver (stable) 16.04-6 from 'canonical' installed
> hello (stable) 2.10 from 'canonical' installed
>
> Please let us know if this is too verbose and if we should omit the
> extra information like channel or version.

Let's omit '(stable)' when the channel is stable. I would include any
non-stable channel as you have it here.

I think we can also omit the publisher name. Separately, we do need a
"snap info" command that tells you about stuff like that. I'll file a
separate bug to track that.

So we would end up with:

krita (beta) 3.2.2 installed
go-example-webserver 16.03 installed
hello (edge) 1.1.4a5 installed

That looks nice and tight :)

Mark

Michael Vogt (mvo) wrote :

With PR https://github.com/snapcore/snapd/pull/2374 it will output:

$ snap install krita --beta
krita (beta) 3.0.93 installed

$ snap install go-example-webserver
go-example-webserver 16.04-9 installed

$ snap install hello --edge
hello (edge) 2.10.42 installed

We also have snap info in snapd 2.18, so seening the publisher is also easy now:
$ snap info krita
name: krita
summary: "Krita: the free digital painting studio"
publisher: krita
tracking: beta
installed: 3.0.93 (11) -
channels:
  stable: 3.0.1.1-snap16 (8) -
  candidate: 3.0.92 (10) -
  beta: 3.0.93 (11) -
  edge: 3.0.93 (11) -

Changed in snappy:
status: Fix Committed → In Progress
Gustavo Niemeyer (niemeyer) wrote :

Per our prior debate around this command and its output, we cannot omit the publisher name from that output, otherwise for a Normal Person these snaps are all coming from Canonical and are equally trustable.

Zygmunt Krynicki (zyga) wrote :

I'm marking this bug as confirmed as I don't think it is in progress at this time and there is no developer actively working on it.

Changed in snappy:
status: In Progress → Confirmed
affects: snappy → snapd
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers