-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Errors are not reported when starting a pod from the REST API #8865
Labels
HTTP API
Bug is in RESTful API
In Progress
This issue is actively being worked by the assignee, please do not work on this at this time.
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.
Comments
openshift-ci-robot
added
the
kind/bug
Categorizes issue or PR as related to a bug.
label
Jan 2, 2021
@jwhonce PTAL |
baude
added
In Progress
This issue is actively being worked by the assignee, please do not work on this at this time.
and removed
In Progress
This issue is actively being worked by the assignee, please do not work on this at this time.
labels
Jan 4, 2021
we are getting a partial fail error back in this case. but then we still return an OK. That is easy to fix, but it also looks like the encoder is eating/throwing away the slice of errs. i need @jwhonce to advise here and then fix should be easy. |
jwhonce
added
HTTP API
Bug is in RESTful API
In Progress
This issue is actively being worked by the assignee, please do not work on this at this time.
labels
Jan 13, 2021
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 1, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 1, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 1, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
jwhonce
added a commit
to jwhonce/podman
that referenced
this issue
Feb 2, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods - Fix test_resize() result check Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com>
mheon
pushed a commit
to mheon/libpod
that referenced
this issue
Feb 4, 2021
- When one or more containers in the Pod reports an error on an operation report StatusConflict and report the error(s) - jsoniter type encoding used to marshal error as string using error.Error() - Update test framework to allow setting any flag when creating pods - Fix test_resize() result check Fixes containers#8865 Signed-off-by: Jhon Honce <jhonce@redhat.com> <MH: Fix cherry-pick conflicts> Signed-off-by: Matthew Heon <matthew.heon@pm.me>
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 22, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
HTTP API
Bug is in RESTful API
In Progress
This issue is actively being worked by the assignee, please do not work on this at this time.
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.
Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
Errors are not reported when starting a pod from the REST API.
Steps to reproduce the issue:
$ podman system service -t 0 tcp:127.0.0.1:1234
Describe the results you received:
The
Errs
property contains empty objects instead of the expected string (as described in the reference).Output of
podman version
:Podman running on Fedora 33.
Have you tested with the latest version of Podman and have you checked the Podman Troubleshooting Guide?
No
The text was updated successfully, but these errors were encountered: