Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1146)

Delta Between Two Patch Sets: worker/provisioner/provisioner.go

Issue 87560043: errors: Improve tests, unify interfaces, wrapping (Closed)
Left Patch Set: Created 10 years, 12 months ago
Right Patch Set: errors: Improve tests, unify interfaces, wrapping Created 10 years, 12 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « worker/provisioner/lxc-broker_test.go ('k') | worker/provisioner/provisioner_test.go » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(Both sides are equal)
1 // Copyright 2012, 2013 Canonical Ltd. 1 // Copyright 2012, 2013 Canonical Ltd.
2 // Licensed under the AGPLv3, see LICENCE file for details. 2 // Licensed under the AGPLv3, see LICENCE file for details.
3 3
4 package provisioner 4 package provisioner
5 5
6 import ( 6 import (
7 "sync" 7 "sync"
8 8
9 "github.com/juju/loggo" 9 "github.com/juju/loggo"
10 "launchpad.net/tomb" 10 "launchpad.net/tomb"
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
261 machine, err := p.getMachine() 261 machine, err := p.getMachine()
262 if err != nil { 262 if err != nil {
263 return nil, err 263 return nil, err
264 } 264 }
265 return machine.WatchContainers(p.containerType) 265 return machine.WatchContainers(p.containerType)
266 } 266 }
267 267
268 func (p *containerProvisioner) getRetryWatcher() (apiwatcher.NotifyWatcher, erro r) { 268 func (p *containerProvisioner) getRetryWatcher() (apiwatcher.NotifyWatcher, erro r) {
269 return nil, errors.NotImplementedf("getRetryWatcher") 269 return nil, errors.NotImplementedf("getRetryWatcher")
270 } 270 }
LEFTRIGHT

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b