apache/cloudstack
1
0
Fork 0
mirror of https://github.com/apache/cloudstack.git synced 2025-11-02 20:02:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

CLOUDSTACK-9843 : Performance improvement of deployVirtualMachine, createFirewallRule, createPortForwardingRule

Browse Source 
removed 1 sec sleep in SSHHelper.
This commit is contained in:
Sudhansu 2017-03-21 16:19:44 +05:30
parent 9cc3ae8a94
commit 28bb8ba1e4
2 changed files with 2 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
utils/src/main/java/com/cloud/utils/ssh/SshHelper.java
View File

@ -39,12 +39,6 @@ public class SshHelper {
private static final int DEFAULT_CONNECT_TIMEOUT = 180000;
private static final int DEFAULT_KEX_TIMEOUT = 60000;
/**
* Waiting time to check if the SSH session was successfully opened. This value (of 1000
* milliseconds) represents one (1) second.
*/
private static final long WAITING_OPEN_SSH_SESSION = 1000;
private static final Logger s_logger = Logger.getLogger(SshHelper.class);
public static Pair<Boolean, String> sshExecute(String host, int port, String user, File pemKeyFile, String password, String command) throws Exception {
@ -236,14 +230,9 @@ public class SshHelper {
}
}
/**
* It gets a {@link Session} from the given {@link Connection}; then, it waits
* {@value #WAITING_OPEN_SSH_SESSION} milliseconds before returning the session, given a time to
* ensure that the connection is open before proceeding the execution.
*/
protected static Session openConnectionSession(Connection conn) throws IOException, InterruptedException {
Session sess = conn.openSession();
Thread.sleep(WAITING_OPEN_SSH_SESSION);
return sess;
}

2
utils/src/test/java/com/cloud/utils/ssh/SshHelperTest.java
View File

@ -146,6 +146,6 @@ public class SshHelperTest {
Mockito.verify(conn).openSession();
PowerMockito.verifyStatic();
Thread.sleep(Mockito.anyLong());
}
}