"juju status-history" doesn't include the concept of progress messages

Bug #1557918 reported by John A Meinel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Won't Fix
High
Unassigned

Bug Description

There are types of status messages that really only exist to give you information about something progressing. In a recent patch on Master, I added the ability to see the download progress of LXD containers. This is really nice for users that want to see what is taking so long for their instance to come up.
However, it now creates:
 downloading image: 1%
 downloading image: 2%
 downolading image: 3%
...
 downloading image: 100%

so 100 messages that are just telling you the same thing.
It would be better if we had some way to flag a Machine.SetInstanceStatus message as updating an earlier one. That would also prevent "update-status" messages from swamping the "juju status-history" output.

Revision history for this message
Eric Snow (ericsnowcurrently) wrote :

tl;dr good luck :)

FWIW, we ran into a similar situation with resources, tracking download progress of resources from the controller to the units. We considered using the status machinery as you've described because it would mostly re-use the existing framework. However, we opted to not go that route for exactly the reason you've described. Solving the problem of status-history spam was too heavy a lift for us given our schedule. (Ultimately we moved resource download progress over to "juju list-resources", but that's beside the point.) Instead we are tracking the download progress (of resources) separately, which is manageable since we can keep it all server-side.

All that said, I imagine it's a different story for LXD images (unless we are caching them on the controller like we do for resources).

Also note that if you are doing this client-side (e.g. in the uniter) then you are producing an API call for each of those status messages, right?

Revision history for this message
Eric Snow (ericsnowcurrently) wrote :

Ah, I see we're already considering caching images on the controller: https://github.com/juju/juju/pull/4755

tags: added: 2.0-count
Changed in juju-core:
milestone: 2.0-beta4 → 2.1.0
affects: juju-core → juju
Changed in juju:
milestone: 2.1.0 → none
milestone: none → 2.1.0
Revision history for this message
Anastasia (anastasia-macmood) wrote :

Removing 2.1 milestone as we will not be addressing this issue in 2.1.

Changed in juju:
milestone: 2.1.0 → none
Revision history for this message
Heather Lanigan (hmlanigan) wrote :

I'm not sure which command "status-history" has been replaced with in juju 2.x.

If you use `juju status`, it's a one time output. Commonly `watch juju status` is used, or `juju status --watch <duration>`, and machine status messages are replaced.

Changed in juju:
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.