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

Unified Diff: environs/cloudinit.go

Issue 59930043: Add errgo as a dep, and start using it.
Patch Set: Created 11 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: environs/cloudinit.go
=== modified file 'environs/cloudinit.go'
--- environs/cloudinit.go 2014-01-23 05:45:58 +0000
+++ environs/cloudinit.go 2014-02-04 06:12:44 +0000
@@ -6,6 +6,8 @@
import (
"fmt"
+ "github.com/errgo/errgo"
+
"launchpad.net/juju-core/agent"
coreCloudinit "launchpad.net/juju-core/cloudinit"
"launchpad.net/juju-core/constraints"
@@ -138,7 +140,7 @@
// These really are directly relevant to running a state server.
cert, key, err := cfg.GenerateStateServerCertAndKey()
if err != nil {
- return fmt.Errorf("cannot generate state server certificate: %v", err)
+ return errgo.Annotate(err, "cannot generate state server certificate")
}
mcfg.StateServerCert = cert
mcfg.StateServerKey = key

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