pyjuju

juju-log and relation-set outputs {}

Bug #915506 reported by Nick Barcet on 2012-01-12

This bug affects 1 person

	Status	Importance	Assigned to	Milestone
pyjuju	Fix Released	Low	Benjamin Saller	pyjuju galapagos
juju (Ubuntu)	Fix Released	Undecided	Unassigned
Precise	Fix Released	Undecided	Unassigned

Bug Description

== SRU Justification ==

=== Impact ===

This causes some charms to have to add complex output control logic for calling juju-log.

=== Test Case ===

1. juju bootstrap
2. juju deploy --repository /usr/share/doc/juju/examples local:mysql
3. juju debug-hooks mysql/0
4. upon seeing the install window open, type 'juju-log foo'. Affected versions will show '{}', unaffected versions will return without printing anything.

=== Dev Fix ===

This was fixed in upstream commit 534

=== Regression Potential ===

Low. Charms *MIGHT* have been written in a way where not printing {} would cause issues, but this is very unlikely and is easy to work around.

== Original Bug Description ==

using juju with lxc provider on oneiric.

Enter debugging session on a unit with 'juju debug-hooks' and catch an event.
At the console type 'juju-log "test"'
The standard output receives a {}

This is really anoying if you called juju-log within a function which standard output is important to you...

See original description

Tags:

Related branches

lp:~bcsaller/pyjuju/log-output

Merged into lp:pyjuju at revision 534

Juju Engineering: Pending requested 2012-04-26

lp:~clint-fewbar/ubuntu/precise/juju/testing-time

lp:ubuntu/precise-proposed/juju

Revision history for this message

Nick Barcet (nijaba) wrote on 2012-01-12:

strace.log Edit (555.4 KiB, text/plain)

strace of the call to juju-log on the lxc container

Revision history for this message

Kapil Thangavelu (hazmat) wrote on 2012-01-27:

nothing to do with lxc, also applies to relation-set

Changed in juju:
status:	New → Confirmed
summary:	- juju-log outputs {} on lxc + juju-log and relation-set outputs {}

Revision history for this message

Kapil Thangavelu (hazmat) wrote on 2012-01-27:

the hook cli return values are hooked up directly to a renderer, a no value magic output should resolve.

Changed in juju:
importance:	Undecided → Low

Kapil Thangavelu (hazmat) on 2012-01-27

Changed in juju:
milestone:	none → florence

Kapil Thangavelu (hazmat) on 2012-03-17

Changed in juju:
milestone:	florence → galapagos

Kapil Thangavelu (hazmat) on 2012-04-26

Changed in juju:
assignee:	nobody → Benjamin Saller (bcsaller)

Benjamin Saller (bcsaller) on 2012-04-26

Changed in juju:
milestone:	galapagos → honolulu
status:	Confirmed → In Progress

Benjamin Saller (bcsaller) on 2012-05-03

Changed in juju:
status:	In Progress → Fix Released

Revision history for this message

Clint Byrum (clint-fewbar) wrote on 2012-05-03:

Re-targetting to galapagos so we can track all the bugs that get merged before 12.04.1

Changed in juju:
milestone:	honolulu → galapagos

Clint Byrum (clint-fewbar) on 2012-05-15

description:	updated
Changed in juju (Ubuntu):
status:	New → Fix Released
Changed in juju (Ubuntu Precise):
status:	New → Triaged

Revision history for this message

Chris Halse Rogers (raof) wrote on 2012-06-01: Please test proposed package

Hello Nick, or anyone else affected,

Accepted juju into precise-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in juju (Ubuntu Precise):
status:	Triaged → Fix Committed
tags:	added: verification-needed

Revision history for this message

Clint Byrum (clint-fewbar) wrote on 2012-06-01:

Verified working in precise-proposed

root@ip-10-244-15-92:/var/lib/juju/units/mysql-0/charm# apt-cache policy juju
juju:
  Installed: 0.5+bzr531-0ubuntu1.1
  Candidate: 0.5+bzr531-0ubuntu1.1
  Version table:
*** 0.5+bzr531-0ubuntu1.1 0
        500 http://us-west-2.ec2.archive.ubuntu.com/ubuntu/ precise-proposed/universe amd64 Packages
        100 /var/lib/dpkg/status
     0.5+bzr531-0ubuntu1 0
        500 http://us-west-2.ec2.archive.ubuntu.com/ubuntu/ precise/universe amd64 Packages
root@ip-10-244-15-92:/var/lib/juju/units/mysql-0/charm# juju-log foo
root@ip-10-244-15-92:/var/lib/juju/units/mysql-0/charm#

tags:

added: verification-done
removed: verification-needed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2012-07-09:

This bug was fixed in the package juju - 0.5+bzr531-0ubuntu1.2

---------------
juju (0.5+bzr531-0ubuntu1.2) precise-proposed; urgency=low

* d/p/upstream-541-542.patch: fix shell script to make proposed work
for local provider. (LP: #1007657)

juju (0.5+bzr531-0ubuntu1.1) precise-proposed; urgency=low

  * d/p/upstream-532.patch: use proper whitespace separation in
    relation-ids. (LP: #988065)
  * d/p/upstream-533.patch: workaround to suppress Zookeeper logging
    which goes out of control in local provider on Reboot.(LP: #958312)
  * d/p/upstream-534.patch: fix to stop juju-log and relation-set commands
    outputting "{}" to stdout, which can break some programs and makes it
    difficult to use them in a debug-hooks session. (LP: #915506)
  * d/p/upstream-537.patch: make HTTPS apt sources usable inside local
    provider. (LP: #993034)
  * d/p/upstream-538.patch: Add support to install juju from proposed
    pocket to test proposed SRU's. (LP: #926550)
-- Clint Byrum <email address hidden> Mon, 18 Jun 2012 14:00:56 -0700

Changed in juju (Ubuntu Precise):
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Bug attachments

strace.log Edit

Add attachment

Remote bug watches

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