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.

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-remote: history: results inconsistent with non-remote podman #7122

Closed
edsantiago opened this issue Jul 28, 2020 · 1 comment · Fixed by #7247
Closed

podman-remote: history: results inconsistent with non-remote podman #7122

edsantiago opened this issue Jul 28, 2020 · 1 comment · Fixed by #7247
Assignees
@baude
Labels
HTTP API Bug is in RESTful API kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. remote Problem is in podman-remote

Comments

@edsantiago
Copy link
Member

In window 1:

$ ./bin/podman system service --timeout=0

In window 2:

$ podman pull alpine     (just to make sure you have the image)
$ ./bin/podman-remote history alpine
ID            CREATED        CREATED BY                                     SIZE    COMMENT
a24bb4013296  292 years ago                                                 0 B
<missing>     292 years ago  /bin/sh -c #(nop) ADD file:c92c248239f8c7b...  845 kB

Expected:

ID            CREATED      CREATED BY                                     SIZE     COMMENT
a24bb4013296  8 weeks ago  /bin/sh -c #(nop)  CMD ["/bin/sh"]             0 B
<missing>     8 weeks ago  /bin/sh -c #(nop) ADD file:c92c248239f8c7b...  5.84 MB

Note the missing CMD in the podman-remote output. Note also the probably-inaccurate dates.

master @ 288ebec, root & rootless.
@mheon mheon added kind/bug Categorizes issue or PR as related to a bug. remote Problem is in podman-remote HTTP API Bug is in RESTful API labels Jul 28, 2020
@mheon
Copy link
Member

mheon commented Jul 28, 2020

I'm going to guess this is in the API and throw the APIv2 label on.

@edsantiago edsantiago mentioned this issue Aug 3, 2020
Merged
@baude baude mentioned this issue Aug 6, 2020
Merged
@baude baude self-assigned this Aug 6, 2020
baude added a commit to baude/podman that referenced this issue Aug 11, 2020
@baude
Replace deepcopy on history results
be41c58 
the deepcopy in the remote history code path was throwing an uncaught error on a type mismatch.  we now manually do the conversion and fix the type mismatch on the fly.

Fixes: containers#7122

Signed-off-by: Brent Baude <bbaude@redhat.com>
mheon pushed a commit to mheon/libpod that referenced this issue Aug 17, 2020
@baude @mheon
Replace deepcopy on history results
c3b7cc2 
the deepcopy in the remote history code path was throwing an uncaught error on a type mismatch.  we now manually do the conversion and fix the type mismatch on the fly.

Fixes: containers#7122

Signed-off-by: Brent Baude <bbaude@redhat.com>
mheon pushed a commit to mheon/libpod that referenced this issue Aug 20, 2020
@baude @mheon
Replace deepcopy on history results
c539091 
the deepcopy in the remote history code path was throwing an uncaught error on a type mismatch.  we now manually do the conversion and fix the type mismatch on the fly.

Fixes: containers#7122

Signed-off-by: Brent Baude <bbaude@redhat.com>
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@baude baude

Labels
HTTP API Bug is in RESTful API kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. remote Problem is in podman-remote
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace deepcopy on history results baude/podman
3 participants
@edsantiago @mheon @baude