summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Fix typo in the remove network api docPaul Holzinger2020-09-07
| | | | Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* APIv2 Add network list filteringPaul Holzinger2020-09-07
| | | | | | | | | | Add the filter option to the libpod endpoint. Add support for the name filter on the docker endpoint. Add apiv2 tests for the network list endpoints. Enable podman network integration tests for remote. Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* Merge pull request #7519 from jjzmajic/fix/vestigial-mkdirOpenShift Merge Robot2020-09-02
|\ | | | | Don't create ~/.config after removing storage.conf
| * Don't create ~/.config after removing storage.confjjzmajic2020-09-02
| | | | | | | | | | | | | | | | Fixes #7509. There is no need to create a ~/.config directory now that ~/.config/containers/storage.conf is not created automatically. Podman has no use for it if it does not exist already. Signed-off-by: jjzmajic <uros.m.perisic@gmail.com>
* | Merge pull request #7426 from Edward5hen/apiv2-containers-testOpenShift Merge Robot2020-09-02
|\ \ | | | | | | APIv2 test: add more tests for containers
| * | APIv2 test: add more tests for containersEdward Shen2020-09-01
| |/ | | | | | | Signed-off-by: Edward Shen <weshen@redhat.com>
* | Merge pull request #7528 from mheon/update_master_206_realOpenShift Merge Robot2020-09-02
|\ \ | | | | | | [CI:DOCS] Update master with release notes for v2.0.6
| * | Update master with release notes for v2.0.6Matthew Heon2020-09-01
|/ / | | | | | | | | | | Also update README to reflect the new release. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | Merge pull request #7517 from baude/crioruncOpenShift Merge Robot2020-09-01
|\ \ | | | | | | use crio runc on CICID ubuntu
| * | use crio runc on CICID ubuntuBrent Baude2020-08-31
| | | | | | | | | | | | | | | | | | | | | when running CICD on Ubuntu where no cgroups v2, we need to use a newer runc for things like seccomp and the default ubuntu runc is not new enough. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | Merge pull request #7470 from containers/dependabot/go_modules/k8s.io/api-0.19.0OpenShift Merge Robot2020-09-01
|\ \ \ | | | | | | | | Bump k8s.io/api from 0.18.8 to 0.19.0
| * | | Bump k8s.io/api from 0.18.8 to 0.19.0Daniel J Walsh2020-08-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [k8s.io/api](https://github.com/kubernetes/api) from 0.18.8 to 0.19.0. - [Release notes](https://github.com/kubernetes/api/releases) - [Commits](https://github.com/kubernetes/api/compare/v0.18.8...v0.19.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #7516 from mheon/handle_no_passwd_fileOpenShift Merge Robot2020-09-01
|\ \ \ \ | |_|_|/ |/| | | Ensure rootless containers without a passwd can start
| * | | Ensure rootless containers without a passwd can startMatthew Heon2020-08-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to modify /etc/passwd to add an entry for the user in question, but at the same time we don't want to require the container provide a /etc/passwd (a container with a single, statically linked binary and nothing else is perfectly fine and should be allowed, for example). We could create the passwd file if it does not exist, but if the container doesn't provide one, it's probably better not to make one at all. Gate changes to /etc/passwd behind a stat() of the file in the container returning cleanly. Fixes #7515 Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | Merge pull request #7514 from edsantiago/reenable_flaky_bindings_testOpenShift Merge Robot2020-08-31
|\ \ \ \ | |/ / / |/| | | bindings: reenable flaky(?) pause/unpause test
| * | | bindings: reenable flaky(?) pause/unpause testEd Santiago2020-08-31
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reference: #6518, a very-frequently-flaking CI test, disabled a month ago (#7143) because it was triggering so often in CI. Unfortunately, that seems to have simply swept the problem under the rug. AFAICT nobody has bothered to look at the root bug, so let's just reenable. If the problem persists, I'll let annoyed developers squeaky-wheel 6158 so there's some incentive to fix it. If the problem has miraculously gone away in the last month, that's a win too. (This test failure does not reproduce on my laptop, nor does it lend itself to devising a simple reproducer on a test VM.) Also: since #5325 appears to have been closed as fixed, remove a 'Skip' that references it. Unfortunately this also requires removing a lot of other cruft. This was an incidental oh-by-the-way addition that I thought would be trivial but ended up causing a much larger diff. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | Merge pull request #7496 from zhangguanzhang/play-kube-handle-hostAliasesOpenShift Merge Robot2020-08-31
|\ \ \ | |/ / |/| | handle play kube with pod.spec.hostAliases
| * | handle play kube with pod.spec.hostAliaseszhangguanzhang2020-08-31
|/ / | | | | | | Signed-off-by: zhangguanzhang <zhangguanzhang@qq.com>
* | Merge pull request #7507 from ↵OpenShift Merge Robot2020-08-31
|\ \ | | | | | | | | | | | | containers/dependabot/go_modules/github.com/google/uuid-1.1.2 Bump github.com/google/uuid from 1.1.1 to 1.1.2
| * | Bump github.com/google/uuid from 1.1.1 to 1.1.2dependabot-preview[bot]2020-08-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/google/uuid](https://github.com/google/uuid) from 1.1.1 to 1.1.2. - [Release notes](https://github.com/google/uuid/releases) - [Commits](https://github.com/google/uuid/compare/v1.1.1...v1.1.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #7494 from haircommander/play-kube-socketOpenShift Merge Robot2020-08-31
|\ \ \ | |/ / |/| | play kube: handle Socket HostPath type
| * | play kube: handle Socket HostPath typePeter Hunt2020-08-28
| | | | | | | | | | | | | | | | | | as well as add test cases for it and the other HostPath types we currently support Signed-off-by: Peter Hunt <pehunt@redhat.com>
* | | Merge pull request #7132 from ↵OpenShift Merge Robot2020-08-29
|\ \ \ | |_|/ |/| | | | | | | | containers/dependabot/go_modules/github.com/containers/conmon-2.0.20incompatible Bump github.com/containers/conmon from 2.0.19+incompatible to 2.0.20+incompatible
| * | Bump github.com/containers/conmonDaniel J Walsh2020-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/containers/conmon](https://github.com/containers/conmon) from 2.0.19+incompatible to 2.0.20+incompatible. - [Release notes](https://github.com/containers/conmon/releases) - [Changelog](https://github.com/containers/conmon/blob/master/changelog.txt) - [Commits](containers/conmon@v2.0.19...v2.0.20) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #7469 from zhangguanzhang/generate-kube-with-ExtraHostsOpenShift Merge Robot2020-08-28
|\ \ \ | | | | | | | | fix podman generate kube with HostAliases
| * | | fix podman generate kube with HostAliaseszhangguanzhang2020-08-27
| | | | | | | | | | | | | | | | Signed-off-by: zhangguanzhang <zhangguanzhang@qq.com>
* | | | Merge pull request #7436 from rhatdan/variantOpenShift Merge Robot2020-08-28
|\ \ \ \ | |_|_|/ |/| | | Add support for image pull overrides
| * | | Add support for variant when pulling imagesDaniel J Walsh2020-08-28
| | | | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
| * | | Document override-arch and override-osDaniel J Walsh2020-08-28
| | | | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #7481 from Luap99/keep-confOpenShift Merge Robot2020-08-28
|\ \ \ \ | | | | | | | | | | Don't remove config files with podman system reset
| * | | | Don't remove config files with podman system resetPaul Holzinger2020-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check if storage.conf exists and display a message that this file should be removed if it has not been modified. Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* | | | | Merge pull request #7431 from mheon/update_master_206OpenShift Merge Robot2020-08-28
|\ \ \ \ \ | | | | | | | | | | | | [CI:DOCS] Update Master to reflect v2.0.5 release
| * | | | | Small tweaks to readme scope sectionMatthew Heon2020-08-28
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | | | | Update Master to reflect v2.0.5 releaseMatthew Heon2020-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds release notes and updates current version in README. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | | Merge pull request #7448 from baude/issue7444OpenShift Merge Robot2020-08-28
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix panic when checking len on nil object
| * | | | | | fix panic when checking len on nil objectBrent Baude2020-08-28
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | issue #7444 describes a problem where an image does not have a manifest file and cannot be processed by our library correctly. the origin of the panic is because we are checking the len of a nil object's attribute. this is a temporary fix to protect from the panic in the future. the origin of the problem is more interesting and requires more work when the code author returns from pto. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | | Merge pull request #7355 from rhatdan/envOpenShift Merge Robot2020-08-28
|\ \ \ \ \ \ | | | | | | | | | | | | | | Use environment from containers.conf
| * | | | | | Use environment from containers.confDaniel J Walsh2020-08-28
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | podman needs to use the environment settings in containers.conf when setting up the containers. Also host environment variables should be relative to server side not the client. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #7429 from lsm5/localunit-test-fixOpenShift Merge Robot2020-08-28
|\ \ \ \ \ \ | | | | | | | | | | | | | | delete prior /dev/shm/*
| * | | | | | Delete prior /dev/shm/*Lokesh Mandvekar2020-08-28
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, subsequent runs of `make localunit` fail and complain about prior existing /dev/shm/libpod_test and /dev/shm/test1. This commit deletes these files if existing already, prior to running the tests. Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | | | | | Merge pull request #7428 from mheon/apiv2_container_pod_errorsOpenShift Merge Robot2020-08-28
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Ensure pod REST API endpoints include ctr errors
| * | | | | Ensure pod REST API endpoints include ctr errorsMatthew Heon2020-08-27
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The APIv2 pod endpoints that operate on multiple containers, such as Start, Kill, Pause, Unpause, do not report errors encountered by individual containers, because they incorrectly assume that any error is fatal. The documentation for the Libpod API calls notes, however, that ErrPodPartialFail will *always* be returned if any container failed; so we need to ignore that error and continue to collating and returning container errors. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | Merge pull request #7416 from Luap99/fix-swagger-docOpenShift Merge Robot2020-08-28
|\ \ \ \ \ | |_|/ / / |/| | | | [CI:DOCS] fix swagger api docs
| * | | | [CI:DOCS] fix swagger api docsPaul Holzinger2020-08-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Separate the volume endpoints into compat and libpod, as it is done for the other endpoints. Move the libpod image push endpoint to images. Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* | | | | Merge pull request #7464 from edsantiago/batsOpenShift Merge Robot2020-08-28
|\ \ \ \ \ | | | | | | | | | | | | BATS: fix corner case in --userns=keep-id test
| * | | | | BATS: fix corner case in --userns=keep-id testEd Santiago2020-08-27
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test that does 'adduser' in a keep-id container had a really dumb bug: if the user running the test has UID 1000, then podman itself (via keep-id) will add the "1000" passwd entry, and the in-container "adduser" will allocate 1001, making our test fail. This triggered in f31/f32 podman gating tests, but (?!?) never in rawhide gating tests. Solution: explicitly feed a UID to adduser. Make sure that it's not the same as the UID of the current user. Also (unrelated): fix a ridiculous "run mkdir || die". At the time I wrote that I probably had no idea how BATS works. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | Merge pull request #7471 from ↵OpenShift Merge Robot2020-08-28
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/containernetworking/plugins-0.8.7 Bump github.com/containernetworking/plugins from 0.8.6 to 0.8.7
| * | | | | Bump github.com/containernetworking/plugins from 0.8.6 to 0.8.7dependabot-preview[bot]2020-08-28
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/containernetworking/plugins](https://github.com/containernetworking/plugins) from 0.8.6 to 0.8.7. - [Release notes](https://github.com/containernetworking/plugins/releases) - [Commits](https://github.com/containernetworking/plugins/compare/v0.8.6...v0.8.7) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #7449 from openSUSE/log-levelOpenShift Merge Robot2020-08-28
|\ \ \ \ \ | | | | | | | | | | | | Fix log level case regression
| * | | | | Fix log level case regressionSascha Grunert2020-08-28
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With previous versions of Podman (like v1.9.2) it was always possible to specify the log level in any case, for example `INFO`. This behavior has silently changed, where the `--log-level` flag only accepts lower case levels. This commit re-enables the old behavior and adds an e2e test for it. Signed-off-by: Sascha Grunert <sgrunert@suse.com>