Code review comment for lp:~axwalk/juju-core/lp1203935-ec2-octal-prices

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Reviewers: mp+176315_code.launchpad.net,

Message:
Please take a look.

Description:
Correct EC2 instance pricing (octal->decimal)

Some prices were entered in octal. This fixes that.

https://code.launchpad.net/~axwalk/juju-core/lp1203935-ec2-octal-prices/+merge/176315

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/11419046/

Affected files:
   A [revision details]
   M environs/ec2/instancetype.go

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: tarmac-20130723031013-rutwe89s8u7anok1
+New revision: <email address hidden>

Index: environs/ec2/instancetype.go
=== modified file 'environs/ec2/instancetype.go'
--- environs/ec2/instancetype.go 2013-05-15 02:06:51 +0000
+++ environs/ec2/instancetype.go 2013-07-23 03:47:45 +0000
@@ -191,7 +191,7 @@
    "m1.xlarge": 640,
    "m3.xlarge": 700,
    "m3.2xlarge": 1400,
- "t1.micro": 020,
+ "t1.micro": 20,
    "m2.xlarge": 495,
    "m2.2xlarge": 990,
    "m2.4xlarge": 1980,
@@ -205,7 +205,7 @@
    "m1.xlarge": 640,
    "m3.xlarge": 700,
    "m3.2xlarge": 1400,
- "t1.micro": 020,
+ "t1.micro": 20,
    "m2.xlarge": 495,
    "m2.2xlarge": 990,
    "m2.4xlarge": 1980,
@@ -219,7 +219,7 @@
    "m1.xlarge": 520,
    "m3.xlarge": 550,
    "m3.2xlarge": 1100,
- "t1.micro": 020,
+ "t1.micro": 20,
    "m2.xlarge": 460,
    "m2.2xlarge": 920,
    "m2.4xlarge": 1840,
@@ -234,7 +234,7 @@
    "m1.medium": 160,
    "m1.large": 320,
    "m1.xlarge": 640,
- "t1.micro": 027,
+ "t1.micro": 27,
    "m2.xlarge": 540,
    "m2.2xlarge": 1080,
    "m2.4xlarge": 2160,
@@ -282,7 +282,7 @@
    "m1.xlarge": 480,
    "m3.xlarge": 500,
    "m3.2xlarge": 1000,
- "t1.micro": 020,
+ "t1.micro": 20,
    "m2.xlarge": 410,
    "m2.2xlarge": 820,
    "m2.4xlarge": 1640,

« Back to merge proposal