danielestevez commented on this pull request.


> +   /**
+    * @see InstanceType#T3_MEDIUM
+    */
+   public static EC2HardwareBuilder t3_medium() {
+      return new EC2HardwareBuilder(InstanceType.T3_MEDIUM).t3()
+              .ram(4096)
+              .processors(ImmutableList.of(new Processor(1.0, 
0.4))).rootDeviceType(RootDeviceType.EBS);
+   }
+
+   /**
+    * @see InstanceType#T3_LARGE
+    */
+   public static EC2HardwareBuilder t3_large() {
+      return new EC2HardwareBuilder(InstanceType.T3_LARGE).t3()
+              .ram(8192)
+              .processors(ImmutableList.of(new Processor(1.0, 
0.4))).rootDeviceType(RootDeviceType.EBS);

T3 instances has number of vCPU is wrong 
https://aws.amazon.com/it/ec2/instance-types/t3/
The javadoc is ok at ```InstanceType```, though

Could you please review them?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1245#pullrequestreview-165828852

Reply via email to