Correct MB/GB constants
authorTim Van Steenburgh <tvansteenburgh@gmail.com>
Wed, 22 Feb 2017 16:05:32 +0000 (11:05 -0500)
committerTim Van Steenburgh <tvansteenburgh@gmail.com>
Wed, 22 Feb 2017 16:05:32 +0000 (11:05 -0500)
examples/add_machine.py
juju/constraints.py

index 34f7869..c5f9a6c 100755 (executable)
@@ -13,8 +13,8 @@ import logging
 from juju import loop
 from juju.model import Model
 
-MB = 1024 * 1024
-GB = MB * 1024
+MB = 1
+GB = 1024
 
 
 async def main():
index 97529e4..c551883 100644 (file)
@@ -21,6 +21,7 @@ import re
 MEM = re.compile('^[1-9][0-9]*[MGTP]$')
 
 # Multiplication factors to get Megabytes
+# https://github.com/juju/juju/blob/master/constraints/constraints.go#L666
 FACTORS = {
     "M": 1,
     "G": 1024,
@@ -31,6 +32,7 @@ FACTORS = {
 SNAKE1 = re.compile(r'(.)([A-Z][a-z]+)')
 SNAKE2 = re.compile('([a-z0-9])([A-Z])')
 
+
 def parse(constraints):
     """
     Constraints must be expressed as a string containing only spaces