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

Delta Between Two Patch Sets: environs/ec2/ec2.go

Issue 9128047: Use multiple base urls for simplestreams (Closed)
Left Patch Set: Use multiple base urls for simplestreams Created 11 years, 11 months ago
Right Patch Set: Use multiple base urls for simplestreams Created 11 years, 11 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 | « cmd/jujud/upgrade.go ('k') | environs/ec2/export_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
1 // Copyright 2011, 2012, 2013 Canonical Ltd.
2 // Licensed under the AGPLv3, see LICENCE file for details.
3
1 package ec2 4 package ec2
2 5
3 import ( 6 import (
4 "fmt" 7 "fmt"
5 "io/ioutil" 8 "io/ioutil"
6 "launchpad.net/goamz/aws" 9 "launchpad.net/goamz/aws"
7 "launchpad.net/goamz/ec2" 10 "launchpad.net/goamz/ec2"
8 "launchpad.net/goamz/s3" 11 "launchpad.net/goamz/s3"
9 "launchpad.net/juju-core/constraints" 12 "launchpad.net/juju-core/constraints"
10 "launchpad.net/juju-core/environs" 13 "launchpad.net/juju-core/environs"
(...skipping 240 matching lines...) Expand 10 before | Expand all | Expand 10 after
251 var err error 254 var err error
252 for a := shortAttempt.Start(); a.Next(); { 255 for a := shortAttempt.Start(); a.Next(); {
253 _, err = e.loadState() 256 _, err = e.loadState()
254 if err != nil { 257 if err != nil {
255 break 258 break
256 } 259 }
257 } 260 }
258 if err == nil { 261 if err == nil {
259 return fmt.Errorf("environment is already bootstrapped") 262 return fmt.Errorf("environment is already bootstrapped")
260 } 263 }
261 » if _, notFound := err.(*environs.NotFoundError); !notFound { 264 » if !environs.IsNotFoundError(err) {
262 return fmt.Errorf("cannot query old bootstrap state: %v", err) 265 return fmt.Errorf("cannot query old bootstrap state: %v", err)
263 } 266 }
264 267
265 possibleTools, err := environs.FindBootstrapTools(e, cons) 268 possibleTools, err := environs.FindBootstrapTools(e, cons)
266 if err != nil { 269 if err != nil {
267 return err 270 return err
268 } 271 }
269 inst, err := e.startInstance(&startInstanceParams{ 272 inst, err := e.startInstance(&startInstanceParams{
270 machineId: "0", 273 machineId: "0",
271 machineNonce: state.BootstrapNonce, 274 machineNonce: state.BootstrapNonce,
(...skipping 758 matching lines...) Expand 10 before | Expand all | Expand 10 after
1030 } 1033 }
1031 var data []byte 1034 var data []byte
1032 data, err = ioutil.ReadAll(resp.Body) 1035 data, err = ioutil.ReadAll(resp.Body)
1033 if err != nil { 1036 if err != nil {
1034 continue 1037 continue
1035 } 1038 }
1036 return strings.TrimSpace(string(data)), nil 1039 return strings.TrimSpace(string(data)), nil
1037 } 1040 }
1038 return 1041 return
1039 } 1042 }
LEFTRIGHT

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