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

[WIP] Enable test suit support for podman executed by non-privileged user #4544

Closed
wants to merge 1 commit into from

Conversation

matejak
Copy link
Member

@matejak matejak commented Jun 26, 2019

When podman run is executed by root, the newly created container gets it's IP address, and it listens on port 22. However, when executed by normal user, there is no IP assigned, and the containerized process listens on localhost. So it is possible to use the test suite container running sshd, but it has to listen on unprivileged port, as normal users can't listen on port 22.

This PR aims to extend the test suite, so it is possible to operate on entities that expose SSH service on unprivileged, and therefore non-standard ports.

@matejak matejak changed the title Enable test suit support for podman executed by non-privileged user [WIP] Enable test suit support for podman executed by non-privileged user Jun 26, 2019
@matejak matejak added the Python label Jun 26, 2019
@matejak matejak added this to the 0.1.45 milestone Jun 26, 2019
@adelton
Copy link
Collaborator

adelton commented Jul 19, 2019

Is there anything else needed in this commit?

@matejak
Copy link
Member Author

matejak commented Jul 19, 2019

There is way too many things that are needed.
The whole test suite assumes that it will be able to connect to the target on the port 22, so the code has to be parametrized to allow for different port numbers. There are functions and methods that use ssh or scp, and they need to get reorganized.

@matejak
Copy link
Member Author

matejak commented Sep 4, 2019

Superseded by #4789

@matejak matejak closed this Sep 4, 2019
@matejak matejak deleted the podman_normaluser branch October 9, 2019 11:45
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.

None yet

3 participants