trevorflanagan commented on this pull request.
> + @Test
+ public void testDeployAndStartServer() {
+ Boolean started = Boolean.TRUE;
+ NetworkInfo networkInfo = NetworkInfo
+ .create(NETWORK_DOMAIN_ID, NIC.builder().vlanId(VLAN_ID).build(),
Lists.<NIC>newArrayList());
+ List<Disk> disks =
ImmutableList.of(Disk.builder().scsiId(0).speed("STANDARD").build());
+ serverId = api().deployServer(deployedServerName, IMAGE_ID, started,
networkInfo, "P$$ssWwrrdGoDd!", disks, null);
+ assertNotNull(serverId);
+ waitForServerStatus(api(), serverId, true, true, 30 * 60 * 1000,
"Error");
+ waitForServerState(api(), serverId, State.NORMAL, 30 * 60 * 1000,
"Error");
+ }
+
+ @Test(dependsOnMethods = "testDeployAndStartServer")
+ public void testReconfigureServer() {
+ api().reconfigureServer(serverId, 4, CpuSpeed.HIGHPERFORMANCE.name(), 1);
+ waitForServerState(api(), serverId, State.PENDING_CHANGE, 30 * 60 *
1000, "Error");
I was finding some inconsistencies when running the test class. The extra check
for PENDING_CHANGE state meant that the test was correctly validated. I have
run the tests again without this check and it seems to be fine. I will remove,
--
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-labs/pull/400#discussion_r152038371
