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

ssh wrapper for PROBE #19

Merged
merged 14 commits into from
Nov 8, 2024
Merged

ssh wrapper for PROBE #19

merged 14 commits into from
Nov 8, 2024

Conversation

Acesif
Copy link
Contributor

@Acesif Acesif commented Jun 19, 2024

The script was tested using openssh docker image.
You can run the script by:
env__PROBE_LIB=libprobe/build python -m probe_py.manual.cli ssh -- [ssh_args] <remote@host> [remote_cmd]

@charmoniumQ
Copy link
Owner

It may be easier to test this with an OpenSSH docker image. But then you have to do at least two steps: one to upload the keys and one to test. Read their README carefully. It's up to you. Please document whatever route you choose.

@charmoniumQ
Copy link
Owner

This implementation looks reasonable; let me know when you can run tests.

@Ex-32 Ex-32 mentioned this pull request Jun 24, 2024
3 tasks
@Acesif Acesif requested a review from charmoniumQ July 9, 2024 15:31
probe_src/probe_py/cli.py Outdated Show resolved Hide resolved
probe_src/probe_py/cli.py Outdated Show resolved Hide resolved
@Acesif Acesif changed the title feat: draft ssh subcommand ssh wrapper for PROBE Aug 7, 2024
@Acesif Acesif requested a review from charmoniumQ August 7, 2024 12:13
probe_src/probe_py/cli.py Outdated Show resolved Hide resolved
probe_src/probe_py/cli.py Outdated Show resolved Hide resolved
probe_src/probe_py/cli.py Outdated Show resolved Hide resolved
probe_src/probe_py/cli.py Outdated Show resolved Hide resolved
@Acesif Acesif requested a review from charmoniumQ August 13, 2024 21:56
@charmoniumQ charmoniumQ marked this pull request as draft August 14, 2024 19:11
@charmoniumQ charmoniumQ mentioned this pull request Aug 20, 2024
@Acesif Acesif marked this pull request as ready for review August 27, 2024 18:41
@Acesif Acesif requested a review from charmoniumQ September 9, 2024 19:48
Copy link
Owner

@charmoniumQ charmoniumQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you commit the script you were using to test this in Docker to tests?

probe_src/python/probe_py/manual/cli.py Outdated Show resolved Hide resolved
@charmoniumQ charmoniumQ marked this pull request as draft September 13, 2024 16:25
@Acesif Acesif marked this pull request as ready for review September 16, 2024 10:16
@Acesif
Copy link
Contributor Author

Acesif commented Sep 16, 2024

For the Docker, you would need to generate the rsa key pair in order for the ssh wrapper to work

@Acesif Acesif requested a review from charmoniumQ September 20, 2024 09:26
@charmoniumQ charmoniumQ merged commit e8d0e9f into charmoniumQ:main Nov 8, 2024
1 check passed
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