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.

Sign up for GitHub

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

Jump to bottom

podman system service: what should .Host.RemoteSocket.Path be? #12023

Closed

edsantiago opened this issue Oct 18, 2021 · 9 comments · Fixed by #14037

Assignees

Labels

kind/bug locked - please file new issue/PR stale-issue

Member

edsantiago commented Oct 18, 2021

Start service with a nonstandard socket:

$ bin/podman system service --timeout=0 tcp:localhost:12345

Run info against it:

$ bin/podman-remote --url="tcp://localhost:12345" info --format '{{.Host.RemoteSocket.Path}}'
/run/user/14904/podman/podman.sock

This surprised me. I expected tcp://<something>:12345.

There is no obvious way to find the socket info: podman-remote info | grep 12345 returns no output.

The text was updated successfully, but these errors were encountered:

Member

rhatdan commented Oct 18, 2021

This looks like a bug.

vrothberg added the kind/bug label

Member

vrothberg commented Nov 17, 2021

@jwhonce could you tackle it? I had a quick look and it seems that the notion of it being a path (in contrast to a URI) is very embedded.

vrothberg assigned jwhonce

github-actions bot commented Dec 18, 2021

A friendly reminder that this issue had no activity for 30 days.

github-actions bot added the stale-issue label

rhatdan removed the stale-issue label

Member

rhatdan commented Dec 18, 2021

@jwhonce did you ever get a chance to look at this?

jwhonce mentioned this issue

Report correct RemoteURI #12895

Closed

github-actions bot commented Jan 18, 2022

A friendly reminder that this issue had no activity for 30 days.

github-actions bot added the stale-issue label

edsantiago removed the stale-issue label

Member Author

edsantiago commented Jan 18, 2022

It's being worked on, @github-actions! Like, in the last two hours! #ReadingComprehensionFail #ThankGoodness #WeHaveSomeTimeBeforeTheRobotApocalypse

github-actions bot commented Feb 18, 2022

A friendly reminder that this issue had no activity for 30 days.

github-actions bot added the stale-issue label

edsantiago removed the stale-issue label

github-actions bot commented Mar 24, 2022

A friendly reminder that this issue had no activity for 30 days.

github-actions bot added the stale-issue label

edsantiago removed the stale-issue label

jwhonce added a commit to jwhonce/podman that referenced this issue


          Report correct RemoteURI

336354a

Rather than assuming a filesystem path, the API service URI is recorded
in the libpod runtime configuration and then reported as requested.

Note: All schemes other than "unix" are hard-coded to report URI exists.

Fixes containers#12023

Signed-off-by: Jhon Honce <[email protected]>

github-actions bot commented Apr 24, 2022

A friendly reminder that this issue had no activity for 30 days.

github-actions bot added the stale-issue label

rhatdan mentioned this issue

Report correct RemoteURI #14037

Merged

rhatdan added a commit to rhatdan/podman that referenced this issue


          Report correct RemoteURI

5fa6f68

Rather than assuming a filesystem path, the API service URI is recorded
in the libpod runtime configuration and then reported as requested.

Note: All schemes other than "unix" are hard-coded to report URI exists.

Fixes containers#12023

Signed-off-by: Jhon Honce <[email protected]>
Signed-off-by: Daniel J Walsh <[email protected]>

openshift-merge-robot closed this as completed in #14037

openshift-cherrypick-robot pushed a commit to openshift-cherrypick-robot/podman that referenced this issue


          Report correct RemoteURI

6ea122f

Rather than assuming a filesystem path, the API service URI is recorded
in the libpod runtime configuration and then reported as requested.

Note: All schemes other than "unix" are hard-coded to report URI exists.

Fixes containers#12023

Signed-off-by: Jhon Honce <[email protected]>
Signed-off-by: Daniel J Walsh <[email protected]>

github-actions bot added the locked - please file new issue/PR label

github-actions bot locked as resolved and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.