juju wait --workload fails because juju show-status-log is not tab delimited

Bug #1745423 reported by Jason Hobbs
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Canonical Juju
Won't Fix
Undecided
Unassigned
Juju Wait Plugin
Fix Released
Undecided
Jason Hobbs

Bug Description

get_error_repeat_count() tries to split up show-status-log output based on tabs, but it doesn't use tabs, at least in juju 2.4.

http://paste.ubuntu.com/26459197/

In juju 2.1.2, it does seem to be tab delimited, so maybe this is due to a change in juju.

Related branches

description: updated
Changed in juju-wait:
assignee: nobody → Jason Hobbs (jason-hobbs)
status: New → In Progress
Revision history for this message
Stuart Bishop (stub) wrote :

I'm adding juju to this bug report, as the output of show-status-log does not appear correctly parseable without effort and making assumptions. Adding the standard --format=yaml and --format=json flags should make this future proof, avoiding things like locale issues rendering the timestamps differently.

Revision history for this message
John A Meinel (jameinel) wrote : Re: [Bug 1745423] Re: juju wait --workload fails because juju show-status-log is not tab delimited

Should we add a new bug on Juju that we should support "juju
show-status-log --format" ? I certainly support the idea of having
structured output for something like that.

On Mon, Jan 29, 2018 at 12:55 PM, Stuart Bishop <email address hidden>
wrote:

> I'm adding juju to this bug report, as the output of show-status-log
> does not appear correctly parseable without effort and making
> assumptions. Adding the standard --format=yaml and --format=json flags
> should make this future proof, avoiding things like locale issues
> rendering the timestamps differently.
>
> --
> You received this bug notification because you are subscribed to juju.
> Matching subscriptions: juju bugs
> https://bugs.launchpad.net/bugs/1745423
>
> Title:
> juju wait --workload fails because juju show-status-log is not tab
> delimited
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju/+bug/1745423/+subscriptions
>

Revision history for this message
Stuart Bishop (stub) wrote :

I think adding show-status-log --format is the preferred way of addressing this in Juju.

Changed in juju-wait:
status: In Progress → Fix Released
Revision history for this message
Anastasia (anastasia-macmood) wrote :

I am marking this as Won't Fix in juju in favor of the new bug # 1746390 created suggested in comment # 3.

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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