Skip to content

Using the SSHCmdHelper instead of a SSH Session.#210

Closed
wilderrodrigues wants to merge 1 commit into
apache:masterfrom
schubergphilis:fix/citrix_create_directories
Closed

Using the SSHCmdHelper instead of a SSH Session.#210
wilderrodrigues wants to merge 1 commit into
apache:masterfrom
schubergphilis:fix/citrix_create_directories

Conversation

@wilderrodrigues
Copy link
Copy Markdown
Contributor

The triled ssh2 session was throwing an IOException when there was no directory present.

We found problems, for example, when SCP the id_rsa into /root/.ssh, since the .ssh did not exist and was not being created. Due to that, we could not connect the Host to the Hypervisor.

I replaced the Session by the SSHCmdHelper.sshExecuteCmd and everything worked fine.

Tested with deployDatacenter.py and XenServer 6.2

@asfbot
Copy link
Copy Markdown

asfbot commented Apr 29, 2015

cloudstack-pull-requests #67 SUCCESS
This pull request looks good

@wilderrodrigues
Copy link
Copy Markdown
Contributor Author

Travis is happy and green; SSVM and CPVM created and connected successfully; Isolated networks, VPCs and VMs also created successfully.

Closing this PR

shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Mar 13, 2023
* Create table to store available console sessions

* Handle the new table in Java (VO and DAO)

* Manage console sessions via database

* Fix cherry-pick: verify in database if the session exist

* Address reviews: rename table to console_session and rename java objects according to the table name

* Redesign console_session to store more data

* Remove unnecessary constructor

* Use create table syntax previous to mariadb 10.5

* Add console session cleanup task

* Address review

* Add missing config keys

* Fix sonar cloud reports

* In progress fix console load report

* Fix remove console when session ends

* Improve setting description

---------

Co-authored-by: GutoVeronezi <daniel@scclouds.com.br>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants