jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrea Turli <notificati...@github.com>
Subject Re: [jclouds/jclouds-labs] [JCLOUDS-1430] Aliyun ECS (#443)
Date Thu, 09 Aug 2018 10:02:42 GMT
andreaturli commented on this pull request.



> +      checkTagsInNodeEquals(node, tags);
+
+      getAnonymousLogger().info(
+              format("<< available node(%s) os(%s) in %ss", node.getId(), node.getOperatingSystem(),
createSeconds));
+
+      watch.reset().start();
+
+      client.runScriptOnNode(nodeId, AdminAccess.builder().adminUsername("web").build(),
nameTask("admin-web"));
+
+      long configureSeconds = watch.elapsed(TimeUnit.SECONDS);
+
+      getAnonymousLogger().info(
+              format(
+                      "<< configured node(%s) in %ss",
+                      nodeId,
+                      configureSeconds));

I actually like the idea of using `python -m SimpleHttpServer` instead of installing `java`
and `jetty` as it takes much longer

If we agree on this I think i can open a PR for jclouds/jclouds and change the ComputeServiceLiveTest
for all of the providers

-- 
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/443#discussion_r208873439
Mime
View raw message