Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DLPX-87656 Enable SFTP for Internal Variants #464

Merged

Conversation

dbshah12
Copy link
Contributor

@dbshah12 dbshah12 commented Sep 7, 2023

Problem

  • As part of https://github.com/delphix/dlpx-app-gate/pull/1190 of dlpx-app-gate, the file transfer protocol of sshj changed from SCP to SFTP but sftp is disabled on DE so to test SFTP working on DE as part of dxoTest and internal-variants uncomment below line from sshd_config:
    Subsystem sftp /usr/lib/openssh/sftp-server

Solution

  • Added a condition in the ansible code of commenting above the line, Only comment for the external variants.

Testing

  • Took ab-pre-push image and tested sftp, sftp is working on the qa-internal image.

@dbshah12 dbshah12 force-pushed the dlpx/pr/dbshah12/e7023bba-6682-4cbe-808f-a69c30d440a5 branch from 0b0df74 to 33c2ada Compare September 7, 2023 13:49
@dbshah12 dbshah12 force-pushed the dlpx/pr/dbshah12/e7023bba-6682-4cbe-808f-a69c30d440a5 branch from 33c2ada to f4c9c7a Compare September 7, 2023 13:51
@dbshah12 dbshah12 merged commit 6319767 into develop Sep 8, 2023
13 checks passed
@dbshah12 dbshah12 deleted the dlpx/pr/dbshah12/e7023bba-6682-4cbe-808f-a69c30d440a5 branch September 8, 2023 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants