Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/926#discussion_r160126361
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2609,7 +2609,7 @@ protected LoginCredentials waitForSshable(
protected LoginCredentials waitForSshable(
HostAndPort hostAndPort, Iterable<LoginCredentials>
credentialsToTry, ConfigBag setup) {
String waitForSshable = setup.get(WAIT_FOR_SSHABLE);
- checkArgument(!"false".equalsIgnoreCase(waitForSshable),
"waitForSshable called despite waitForSshable=%s for %s", waitForSshable,
hostAndPort);
+// checkArgument(!"false".equalsIgnoreCase(waitForSshable),
"waitForSshable called despite waitForSshable=%s for %s", waitForSshable,
hostAndPort);
--- End diff --
Please avoid committing commented-out lines of code. Is this line supposed
to be deleted, or was it commented out to help debugging?
---
