Skip to content

systemvm: correct sftp subsystem path of debian wheezy#104

Closed
resmo wants to merge 1 commit into
apache:masterfrom
resmo:fix/debian-sshd-config
Closed

systemvm: correct sftp subsystem path of debian wheezy#104
resmo wants to merge 1 commit into
apache:masterfrom
resmo:fix/debian-sshd-config

Conversation

@resmo
Copy link
Copy Markdown
Member

@resmo resmo commented Mar 9, 2015

The sftp subsystem path is /usr/lib/openssh/sftp-server according https://packages.debian.org/wheezy/amd64/openssh-server/filelist and checked with plain wheezy install.

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Mar 10, 2015

Confirmed location using 4.5 systemvms. LGTM, merging.

@asfgit asfgit closed this in 47de4a1 Mar 10, 2015
asfgit pushed a commit that referenced this pull request Mar 10, 2015
This closes #104

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
(cherry picked from commit 47de4a1)
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@karuturi
Copy link
Copy Markdown
Member

@resmo May I know how you discovered this?
AFAIK, scp and sftp are working fine from ssvm and we do not need sftp server on ssvm
Can we remove this subsystem?

@resmo
Copy link
Copy Markdown
Member Author

resmo commented Mar 10, 2015

@karuturi we use Ansible to manage the routers ad-hoc (config fixes, package updates, security fixes, etc). Ansible uses SFTP for file transfer per default (but it can be configured to use SCP as fallback). So we discovered this issue while debugging.

I can not confirm SFTP works without this patch. This patch fixed the SFTP functionality for us. I see no reason why the SFTP subsystem should be removed.

On the other hand, SFTP has less overhead then SCP (aka faster) and in a big environment of hundreds of routers, this is performance gain.

So voting against removing the subsystem.

@resmo resmo deleted the fix/debian-sshd-config branch March 10, 2015 09:17
@karuturi
Copy link
Copy Markdown
Member

@resmo Thanks for the explanation.
+1 for the sftp-server if it makes it faster :)

gauravaradhye pushed a commit to gauravaradhye/cloudstack that referenced this pull request Mar 10, 2015
This closes apache#104

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
llambiel pushed a commit to exoscale/cloudstack that referenced this pull request Mar 11, 2015
This closes apache#104

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
yadvr added a commit that referenced this pull request Jan 20, 2021
Adds order column in zone/template/offerings list view tables.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Co-authored-by: Rohit Yadav <rohit@apache.org>
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.

3 participants