Viewing all changes in revision 1129.
- Committer: Gary Poster
- Date: 2013-10-11 14:44:27 UTC
- mfrom: (1128.1.1 fixGoSandboxWatcher2)
- Revision ID: gary.poster@canonical.com-20131011144427-icyufqma6757tryb
Fix Go sandbox AllWatcher initialization
The Go sandbox AllWatcher was trying to send deltas immediately, which does not match the juju core implementation. This fixes the behavior, with a test.
It also fixes a user-visible bug. To qa, turn off the simulator in the config file, and then deploy a service in the default go sandbox. Before my branch, the unit would never appear, because the delta for this change was lost. After my branch, the unit appears correctly.
R=matthew.scott
CC=
https://codereview.appspot.com/14548050
The Go sandbox AllWatcher was trying to send deltas immediately, which does not match the juju core implementation. This fixes the behavior, with a test.
It also fixes a user-visible bug. To qa, turn off the simulator in the config file, and then deploy a service in the default go sandbox. Before my branch, the unit would never appear, because the delta for this change was lost. After my branch, the unit appears correctly.
R=matthew.scott
CC=
https://codereview.appspot.com/14548050
- files modified:
- app/store/env/sandbox.js
expand all
collapse all
added
removed
