Error messages are logged twice in the command line
Bug #994855 reported by
Thomas Herve
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 modifiedjuju/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)
To post a comment you must log in.
The following diff fixes it:
=== modified file 'juju/control/ command. py' command. py 2011-10-18 15:42:24 +0000 command. py 2012-05-04 21:53:29 +0000
stream. write(traceback IO.getvalue( ))
self. options. log.error( tracebackIO. getvalue( ))
--- juju/control/
+++ juju/control/
@@ -62,8 +62,6 @@
- # be a bit noisy on failure getErrorMessage ()
self. options. log.error( result. getErrorMessage ())
- print >>stream, result.
if self.exit_code == 0: