summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* enable integration tests for restartBrent Baude2020-04-27
| | | | | | enable integrations tests for container restart. fixed bug where --running was not being honored. Signed-off-by: Brent Baude <bbaude@redhat.com>
* Merge pull request #5971 from giuseppe/fix-userns-testsOpenShift Merge Robot2020-04-24
|\ | | | | v2, podman: fix and enable all run_userns_test.go tests
| * podman: assume user namespace if there are mappingsGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | | | if some mappings are specified, assume there is a private user namespace. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * podman: implement userns=keep-idGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | | | add missing implementation for userns=keep-id and enable the user namespaces tests. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #5977 from QiWang19/commit-testOpenShift Merge Robot2020-04-24
|\ \ | | | | | | V2 tests: enables commit tests
| * | V2 tests: enables commit testsQi Wang2020-04-24
| | | | | | | | | | | | | | | | | | V2 tests enables commit tests Signed-off-by: Qi Wang <qiwan@redhat.com>
* | | Merge pull request #5974 from giuseppe/fix-initOpenShift Merge Robot2020-04-24
|\ \ \ | |/ / |/| | [v2] containers, init: skip invalid state errors with --all
| * | containers, init: skip invalid state errors with --allGiuseppe Scrivano2020-04-24
| |/ | | | | | | | | | | reintroduce the same check that exists in v1.9. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #5967 from giuseppe/run-test-fixesOpenShift Merge Robot2020-04-24
|\ \ | | | | | | v2, tests: fix various run_test.go failures
| * | podman: fix --log-opt=path=%sGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | podman: fix --http-proxyGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | podman: fix podman --group-addGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | test: fix --host-env testGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | podman: fix --cgroups=disabledGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | test: enable some run_test.go tests that pass nowGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | podman: add support for --rootfsGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | specgen: fix error messageGiuseppe Scrivano2020-04-24
| | | | | | | | | | | | | | | | | | the check is correct but the error message was stating the opposite. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | create: move validate after setting default nsGiuseppe Scrivano2020-04-24
| |/ | | | | | | | | | | validate the configuration only after we set the default namespaces. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #5972 from mheon/fix_noinfraOpenShift Merge Robot2020-04-24
|\ \ | | | | | | Do not join pod namespaces without an infra ctr
| * | Do not join pod namespaces without an infra ctrMatthew Heon2020-04-24
|/ / | | | | | | | | | | | | | | | | | | | | We do not want to join pod namespaces if no infra container is present. A pod may claim it shares namespaces without an infra container (I'll take an action item to fix that - it really should not be allowed), which was tripping up our default namespace code and forcing us to try and join the namespaces of the nonexistant infra container. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | Merge pull request #5968 from giuseppe/fix-cgroup-testsOpenShift Merge Robot2020-04-24
|\ \ | | | | | | v2, test: enable cgroup parent tests
| * | test: enable cgroup parent testsGiuseppe Scrivano2020-04-24
| |/ | | | | | | | | | | cgroups tests should pass now Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #5965 from ↵OpenShift Merge Robot2020-04-24
|\ \ | | | | | | | | | | | | containers/dependabot/go_modules/github.com/containers/common-0.9.5 Bump github.com/containers/common from 0.9.4 to 0.9.5
| * | Bump github.com/containers/common from 0.9.4 to 0.9.5dependabot-preview[bot]2020-04-24
| |/ | | | | | | | | | | | | | | | | Bumps [github.com/containers/common](https://github.com/containers/common) from 0.9.4 to 0.9.5. - [Release notes](https://github.com/containers/common/releases) - [Commits](https://github.com/containers/common/compare/v0.9.4...v0.9.5) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #5962 from rhatdan/selinuxOpenShift Merge Robot2020-04-24
|\ \ | |/ |/| Fix SELinux functions names to not be repetitive
| * Fix SELinux functions names to not be repetitiveDaniel J Walsh2020-04-23
| | | | | | | | | | | | | | Since functions are now in an selinux subpackage, they should not start with SELinux Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #5957 from giuseppe/drop-fooOpenShift Merge Robot2020-04-24
|\ \ | | | | | | foo: delete spurious file
| * | foo: delete spurious fileGiuseppe Scrivano2020-04-23
| |/ | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #5963 from akerl-unpriv/fix/tmpdir-for-pullOpenShift Merge Robot2020-04-24
|\ \ | |/ |/| set bigfilestemporarydir for pull
| * remove blank lineLes Aker2020-04-23
| | | | | | | | Signed-off-by: Les Aker <me@lesaker.org>
| * set bigfilestemporarydir for pullLes Aker2020-04-23
|/ | | | Signed-off-by: Les Aker <me@lesaker.org>
* Merge pull request #5933 from giuseppe/fix-namespacesOpenShift Merge Robot2020-04-23
|\ | | | | podman, v2: handle namespaces specified on the CLI
| * podman: handle namespaces specified on the CLIGiuseppe Scrivano2020-04-23
| | | | | | | | | | | | | | and handle differently the user namespace as it supports additional options. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * specgen: do not always set shmsizeGiuseppe Scrivano2020-04-23
| | | | | | | | | | | | if it wasn't overriden on the CLI, leave it undefined. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * pkg: fix shmsize error messageGiuseppe Scrivano2020-04-23
| | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | Merge pull request #5946 from jwhonce/wip/shutdownOpenShift Merge Robot2020-04-23
|\ \ | |/ |/| V2 restore libpod.Shutdown() when exiting podman commands
| * V2 restore libpod.Shutdown() when exiting podman commandsJhon Honce2020-04-22
| | | | | | | | Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | Merge pull request #5843 from QiWang19/manifest_createOpenShift Merge Robot2020-04-23
|\ \ | | | | | | manifest create,add,inspect
| * | manifest create,add,inspectQi Wang2020-04-22
| | | | | | | | | | | | | | | | | | Implememts manifest subcommands create, add, inspect. Signed-off-by: Qi Wang <qiwan@redhat.com>
* | | Merge pull request #5952 from rhatdan/vOpenShift Merge Robot2020-04-23
|\ \ \ | | | | | | | | Stop wrapping pull messages
| * | | Stop wrapping pull messagesDaniel J Walsh2020-04-23
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | The length and size of our error messages on failure to pull is huge. This patch at least eliminates some of the wrapping. But I think eventually we need to look at containers/image and see if we can modify the error messages to something a little more human friendly. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #5948 from jwhonce/jira/826OpenShift Merge Robot2020-04-23
|\ \ \ | | | | | | | | V2 Restore rmi tests
| * | | V2 Restore rmi testsJhon Honce2020-04-22
|/ / / | | | | | | | | | | | | | | | | | | * Introduced define.ErrImageInUse to assist in determining the exit code without resorting string searches. Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | Merge pull request #5942 from mheon/volume_flagsOpenShift Merge Robot2020-04-22
|\ \ \ | |_|/ |/| | Enable basic volumes support in Podmanv2
| * | Enable basic volumes support in Podmanv2Matthew Heon2020-04-22
| | | | | | | | | | | | | | | | | | | | | | | | This enables the --volume, --mount, and --tmpfs flags in Podmanv2. It does not enable init-related flags, image volumes, and --volumes-from. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | Merge pull request #5947 from rhatdan/vOpenShift Merge Robot2020-04-22
|\ \ \ | | | | | | | | Turn on version.go except for -v check
| * | | Turn on version.go except for -v checkDaniel J Walsh2020-04-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently podman -v does not work but the other version checks all pass. enabling tests to that we can get more tests running in CI. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #5936 from rhatdan/selinux1OpenShift Merge Robot2020-04-22
|\ \ \ \ | |_|_|/ |/| | | Move selinux labeling support from pkg/util to pkg/selinux
| * | | Move selinux labeling support from pkg/util to pkg/selinuxDaniel J Walsh2020-04-22
| | |/ | |/| | | | | | | | | | | | | | | | The goal here is to make the package less heavy and not overload the pkg/util. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #5930 from jwhonce/issues/5765OpenShift Merge Robot2020-04-22
|\ \ \ | | | | | | | | Instrumentation to answer #5765