From 94e0a015cd0073106807e16be4c9f750ab37aef4 Mon Sep 17 00:00:00 2001 From: Aled Sage Date: Tue, 10 Sep 2013 11:16:15 +0100 Subject: [PATCH] SoftLayer: report ram as MB rather than GB - fixes https://issues.apache.org/jira/browse/JCLOUDS-269 --- .../softlayer/compute/functions/ProductItemsToHardware.java | 7 +++---- .../softlayer/compute/SoftLayerTemplateBuilderLiveTest.java | 12 ++++++------ .../compute/functions/ProductItemsToHardwareTest.java | 4 ++-- 3 files changed, 11 insertions(+), 12 deletions(-) diff --git a/providers/softlayer/src/main/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardware.java b/providers/softlayer/src/main/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardware.java index 68aeea9..4adee6f 100644 --- a/providers/softlayer/src/main/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardware.java +++ b/providers/softlayer/src/main/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardware.java @@ -85,11 +85,10 @@ public Hardware apply(Iterable items) { double cores = ProductItems.capacity().apply(coresItem).doubleValue(); Matcher cpuMatcher = cpuDescriptionRegex.matcher(coresItem.getDescription()); double coreSpeed = (cpuMatcher.matches()) ? Double.parseDouble(cpuMatcher.group(cpuMatcher.groupCount())) : DEFAULT_CORE_SPEED; - int ram = ProductItems.capacity().apply(ramItem).intValue(); + int ram = ProductItems.capacity().apply(ramItem).intValue() * 1024; - return new HardwareBuilder().ids(hardwareId).processors(ImmutableList.of(new Processor(cores, coreSpeed))).ram( - ram) - .hypervisor("XenServer") + return new HardwareBuilder().ids(hardwareId).processors(ImmutableList.of(new Processor(cores, coreSpeed))) + .ram(ram).hypervisor("XenServer") .volumes( Iterables.transform(filter(items, categoryCodeMatches(diskCategoryRegex)), new Function() { diff --git a/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/SoftLayerTemplateBuilderLiveTest.java b/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/SoftLayerTemplateBuilderLiveTest.java index 03f2efb..31f593d 100644 --- a/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/SoftLayerTemplateBuilderLiveTest.java +++ b/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/SoftLayerTemplateBuilderLiveTest.java @@ -87,7 +87,7 @@ public void testDefaultTemplateBuilder() throws IOException { assertEquals(defaultTemplate.getImage().getOperatingSystem().is64Bit(), true); assertEquals(defaultTemplate.getImage().getOperatingSystem().getFamily(), OsFamily.UBUNTU); assertEquals(getCores(defaultTemplate.getHardware()), 1.0d); - assertEquals(defaultTemplate.getHardware().getRam(), 1); + assertEquals(defaultTemplate.getHardware().getRam(), 1*1024); assertEquals(getSpace(defaultTemplate.getHardware()), 25.0d); assertEquals(defaultTemplate.getHardware().getVolumes().get(0).getType(), Volume.Type.LOCAL); // test that we bound the correct templateoptions in guice @@ -141,7 +141,7 @@ public void testBiggestTemplateBuilderWhenBootIsSAN() throws IOException { Template template = context.getComputeService().templateBuilder().biggest().build(); assertEquals(getCores(template.getHardware()), 16.0d); - assertEquals(template.getHardware().getRam(), 16); + assertEquals(template.getHardware().getRam(), 16*1024); assertEquals(getSpace(template.getHardware()), 100.0d); assertEquals(template.getHardware().getVolumes().get(0).getType(), Volume.Type.SAN); } finally { @@ -161,7 +161,7 @@ public void testDefaultTemplateBuilderWhenPrivateNetwork() throws IOException { Template template = context.getComputeService().templateBuilder().build(); assertEquals(getCores(template.getHardware()), 1.0d); - assertEquals(template.getHardware().getRam(), 1); + assertEquals(template.getHardware().getRam(), 1*1024); assertEquals(getSpace(template.getHardware()), 25.0d); assertEquals(template.getHardware().getVolumes().get(0).getType(), Volume.Type.LOCAL); } finally { @@ -181,7 +181,7 @@ public void testBiggestTemplateBuilderWhenPrivateNetwork() throws IOException { Template template = context.getComputeService().templateBuilder().biggest().build(); assertEquals(getCores(template.getHardware()), 8.0d); - assertEquals(template.getHardware().getRam(), 16); + assertEquals(template.getHardware().getRam(), 16*1024); assertEquals(getSpace(template.getHardware()), 100.0d); assertEquals(template.getHardware().getVolumes().get(0).getType(), Volume.Type.LOCAL); } finally { @@ -194,7 +194,7 @@ public void testBiggestTemplateBuilderWhenPrivateNetwork() throws IOException { public void testFastestTemplateBuilder() throws IOException { Template template = view.getComputeService().templateBuilder().fastest().build(); assertEquals(getCores(template.getHardware()), 16.0d); - assertEquals(template.getHardware().getRam(), 1); + assertEquals(template.getHardware().getRam(), 1*1024); assertEquals(getSpace(template.getHardware()), 25.0d); assertEquals(template.getHardware().getVolumes().get(0).getType(), Volume.Type.LOCAL); } @@ -203,7 +203,7 @@ public void testFastestTemplateBuilder() throws IOException { public void testBiggestTemplateBuilder() throws IOException { Template template = view.getComputeService().templateBuilder().biggest().build(); assertEquals(getCores(template.getHardware()), 16.0d); - assertEquals(template.getHardware().getRam(), 16); + assertEquals(template.getHardware().getRam(), 16*1024); assertEquals(getSpace(template.getHardware()), 100.0d); assertEquals(template.getHardware().getVolumes().get(0).getType(), Volume.Type.LOCAL); } diff --git a/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardwareTest.java b/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardwareTest.java index f097c0d..cac6159 100644 --- a/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardwareTest.java +++ b/providers/softlayer/src/test/java/org/jclouds/softlayer/compute/functions/ProductItemsToHardwareTest.java @@ -101,7 +101,7 @@ public void testHardware() { assertEquals(1, processors.size()); assertEquals(2.0, processors.get(0).getCores()); - assertEquals(2, hardware.getRam()); + assertEquals(2048, hardware.getRam()); List volumes = hardware.getVolumes(); assertEquals(1, volumes.size()); @@ -127,7 +127,7 @@ public void testHardwareWithPrivateCore() { assertEquals(1, processors.size()); assertEquals(2.0, processors.get(0).getCores()); - assertEquals(2, hardware.getRam()); + assertEquals(2048, hardware.getRam()); List volumes = hardware.getVolumes(); assertEquals(1, volumes.size()); -- 1.8.4