pyjuju

Error messages are logged twice in the command line

Bug #994855 reported by Thomas Herve
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pyjuju
Fix Released
Undecided
Unassigned

Bug Description

For example, when using config set:

$ > juju set service key=invalid
Invalid value for key: 'invalid'
2012-05-04 14:50:14,712 ERROR Invalid value for key: 'invalid'

Related branches

lp:~therve/pyjuju/remove-duplicated-error-message
Kapil Thangavelu (community): Approve
Diff: 592 lines (+56/-63)
20 files modified
juju/control/command.py (+0/-2)
juju/control/tests/test_add_relation.py (+6/-6)
juju/control/tests/test_add_unit.py (+2/-2)
juju/control/tests/test_bootstrap.py (+3/-3)
juju/control/tests/test_config_get.py (+2/-2)
juju/control/tests/test_config_set.py (+3/-2)
juju/control/tests/test_constraints_set.py (+2/-2)
juju/control/tests/test_control.py (+5/-6)
juju/control/tests/test_debug_hooks.py (+3/-3)
juju/control/tests/test_deploy.py (+9/-9)
juju/control/tests/test_destroy_environment.py (+2/-2)
juju/control/tests/test_destroy_service.py (+1/-2)
juju/control/tests/test_expose.py (+2/-2)
juju/control/tests/test_remove_relation.py (+4/-4)
juju/control/tests/test_remove_unit.py (+4/-4)
juju/control/tests/test_resolved.py (+2/-2)
juju/control/tests/test_ssh.py (+2/-6)
juju/control/tests/test_terminate_machine.py (+1/-1)
juju/control/tests/test_unexpose.py (+2/-2)
juju/control/tests/test_upgrade_charm.py (+1/-1)
Revision history for this message
Thomas Herve (therve) wrote :

The following diff fixes it:

=== modified file 'juju/control/command.py'
--- juju/control/command.py 2011-10-18 15:42:24 +0000
+++ juju/control/command.py 2012-05-04 21:53:29 +0000
@@ -62,8 +62,6 @@
                 stream.write(tracebackIO.getvalue())
                 self.options.log.error(tracebackIO.getvalue())

- # be a bit noisy on failure
- print >>stream, result.getErrorMessage()
             self.options.log.error(result.getErrorMessage())

             if self.exit_code == 0:

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

This was merged and released as part of 0.6

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

Other bug subscribers

Remote bug watches

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