diff options
author | Juan Antonio Osorio Robles <jaosorior@redhat.com> | 2021-01-12 15:54:32 +0200 |
---|---|---|
committer | Juan Antonio Osorio Robles <jaosorior@redhat.com> | 2021-01-12 16:10:17 +0200 |
commit | 020abbfeab3b3f3bc3b82edf1b9374b90d79ae91 (patch) | |
tree | 83cdea752a30c8de623a6fe3798ca59db0525be2 /pkg/systemd/generate/pods.go | |
parent | 64b86d004ebd7db0b2bc352475505050be0f8591 (diff) | |
download | podman-020abbfeab3b3f3bc3b82edf1b9374b90d79ae91.tar.gz podman-020abbfeab3b3f3bc3b82edf1b9374b90d79ae91.tar.bz2 podman-020abbfeab3b3f3bc3b82edf1b9374b90d79ae91.zip |
Expose security attribute errors with their own messages
This creates error objects for runtime errors that might come from the
runtime. Thus, indicating to users that the place to debug should be in
the security attributes of the container.
When creating a container with a SELinux label that doesn't exist, we
get a fairly cryptic error message:
```
$ podman run --security-opt label=type:my_container.process -it fedora bash
Error: OCI runtime error: write file `/proc/thread-self/attr/exec`: Invalid argument
```
This instead handles any errors coming from LSM's `/proc` API and
enhances the error message with a relevant indicator that it's related
to the container's security attributes.
A sample run looks as follows:
```
$ bin/podman run --security-opt label=type:my_container.process -it fedora bash
Error: `/proc/thread-self/attr/exec`: OCI runtime error: unable to assign security attribute
```
With `debug` log level enabled it would be:
```
Error: write file `/proc/thread-self/attr/exec`: Invalid argument: OCI runtime error: unable to assign security attribute
```
Note that these errors wrap ErrOCIRuntime, so it's still possible to to
compare these errors with `errors.Is/errors.As`.
One advantage of this approach is that we could start handling these
errors in a more efficient manner in the future.
e.g. If a SELinux label doesn't exist (yet), we could retry until it
becomes available.
Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
Diffstat (limited to 'pkg/systemd/generate/pods.go')
0 files changed, 0 insertions, 0 deletions