summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #6044 from cevich/update_f32OpenShift Merge Robot2020-05-01
|\ | | | | Cirrus: Update to Fedora 32 proper
| * Cirrus: Utilize new cache imagesChris Evich2020-04-30
| | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
| * Cirrus: Utilize new base imagesChris Evich2020-04-30
| | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
| * cirrus: Update to Fedora 32 properChris Evich2020-04-30
| | | | | | | | | | | | | | | | Now that it's officially released, update to it from the beta. Also (and significant), adjust the SELinux context of the GCP metadata service. Add a comment to the code explaining why this is necessary. Signed-off-by: Chris Evich <cevich@redhat.com>
* | Merge pull request #6055 from baude/v2argsOpenShift Merge Robot2020-04-30
|\ \ | | | | | | fix commands without input
| * | fix commands without inputbaude2020-04-30
| | | | | | | | | | | | | | | | | | in cases where commands require input and we dont provide it, we often would segv. This can be attributed in many cases to the subcommand not picked up the cobra Args attribute or neither had them. Signed-off-by: baude <bbaude@redhat.com>
| * | System tests: help messages: check required-argEd Santiago2020-04-30
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a usage message is of the form '... [flags] ARGNAME', where ARGNAME is all-caps and not in brackets, it must be a required argument. Try running podman subcommand without ARGNAME, and make sure that podman bails out with an informative message. (Since this message is freeform in each subcommand, not Cobra-generated, we have a lot of possible variations to check for). Fix podman login/logout Use messages to indicate that REGISTRY is now optional (as of #5233). This test has actually been in place for over a year but due to a typo on my part -- a missing space -- it was not being run. "For want of a space, much testing was lost". Signed-off-by: Ed Santiago <santiago@redhat.com>
* | Merge pull request #6056 from jwhonce/wip/rootlessOpenShift Merge Robot2020-04-30
|\ \ | | | | | | V2 Commands that require ParentNS (rootful) are report error
| * | V2 Commands that require ParentNS (rootful) are report errorJhon Honce2020-04-30
| | | | | | | | | | | | | | | | | | * Updated unmount to require ParentNS Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | Merge pull request #5861 from ↵OpenShift Merge Robot2020-04-30
|\ \ \ | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/containers/psgo-1.5.0 Bump github.com/containers/psgo from 1.4.0 to 1.5.0
| * | | Bump github.com/containers/psgo from 1.4.0 to 1.5.0dependabot-preview[bot]2020-04-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/containers/psgo](https://github.com/containers/psgo) from 1.4.0 to 1.5.0. - [Release notes](https://github.com/containers/psgo/releases) - [Commits](https://github.com/containers/psgo/compare/v1.4.0...v1.5.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #6052 from sujil02/enable-prune-testOpenShift Merge Robot2020-04-30
|\ \ \ \ | |_|/ / |/| | | Enable prune integration test. Fixes container prune
| * | | Enable prune integration test. Fixes container prune.Sujil022020-04-30
| | |/ | |/| | | | | | | | | | | | | | | | Fixes container prune to prune created and configured containers. Disables couple of system prune test as not yet in with v2. Signed-off-by: Sujil02 <sushah@redhat.com>
* | | Merge pull request #6046 from jwhonce/jira/822OpenShift Merge Robot2020-04-30
|\ \ \ | |/ / |/| | V2 enable ps tests
| * | V2 enable ps testsJhon Honce2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | * Combine cobra.Command helper functions into validate package from registry and common packages * Introduce ChoiceValue for flags Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | Merge pull request #6047 from QiWang19/attach-testOpenShift Merge Robot2020-04-30
|\ \ \ | |/ / |/| | testv2: enable attach test
| * | testv2: enable attach testQi Wang2020-04-29
|/ / | | | | | | | | | | testv2: enable attach test Signed-off-by: Qi Wang <qiwan@redhat.com>
* | Merge pull request #5956 from jwhonce/wip/versionOpenShift Merge Robot2020-04-29
|\ \ | | | | | | V2 restore podman -v command
| * | V2 restore podman -v commandJhon Honce2020-04-29
| | | | | | | | | | | | | | | | | | | | | * Removed extra spaces and improved error message * Updated tests to use gomega matchers Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | Merge pull request #6042 from baude/systemtestfinalOpenShift Merge Robot2020-04-29
|\ \ \ | | | | | | | | enable final system test
| * | | enable final system testbaude2020-04-29
| |/ / | | | | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #5949 from jwhonce/jira/796OpenShift Merge Robot2020-04-29
|\ \ \ | |/ / |/| | V2 Restore images list tests
| * | V2 Restore images list testsJhon Honce2020-04-29
|/ / | | | | | | | | | | | | | | * Fix history --quiet formatting * Fix image inspect --format=json * Fix image list --sort Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | Merge pull request #6039 from vrothberg/enable-search-testsOpenShift Merge Robot2020-04-29
|\ \ | | | | | | Enable search tests
| * | enable search testsValentin Rothberg2020-04-29
| | | | | | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | pull/search options: tls verify -> skipValentin Rothberg2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | Change the logic in the options from tls-verify to skipping verification. It require a constant brain yoga to translate from doing verification (CLI logic) to skipping it (c/image logic). As the code is using c/image, let's make it consistent. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #6010 from TomSweeneyRedHat/dev/tsweeney/ctrimagefixesOpenShift Merge Robot2020-04-29
|\ \ \ | | | | | | | | Update podmanimage files to adjust perms on containers.conf for rootless
| * | | Update podmanimage files to adjust perms on containers.conf for rootlessTomSweeneyRedHat2020-04-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding the changes to the Podman image Docker/Containerfiles similar to @rhatdan 's changes in https://github.com/containers/buildah/pull/2332 In short it changes the perms on containers.conf so it can be used by a rootless user. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | Merge pull request #5853 from cevich/unify_in_podman_buildOpenShift Merge Robot2020-04-29
|\ \ \ \ | | | | | | | | | | Unify in_podman container packaging & VM packaging
| * | | | Cirrus: Utilize new VM imagesChris Evich2020-04-24
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
| * | | | Cirrus: Unify package installationChris Evich2020-04-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, test-build critical container images depended upon for CI-purposes. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #6038 from giuseppe/v2-enable-copyOpenShift Merge Robot2020-04-29
|\ \ \ \ \ | |_|_|/ / |/| | | | test: enable cp tests
| * | | | test: enable cp testsGiuseppe Scrivano2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and fix them to use a distinct directory. I've seen a test failing as it was using the same directory as another test, that is not related to v2 itself. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #6037 from vrothberg/enable-push-testsOpenShift Merge Robot2020-04-29
|\ \ \ \ \ | | | | | | | | | | | | Enable push tests
| * | | | | login system test: enable "push ok"Valentin Rothberg2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | | | enable the push e2e testsValentin Rothberg2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the recent changes, push is working now. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | | | push: fix --tls-verifyValentin Rothberg2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix --tls-verify parsing and make the associated options reflect the correct logic. Other commands are affected as well but will be fixed later. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | | | push: simplify cmdValentin Rothberg2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The indirection via a 2nd variable isn't needed. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | | | push: fix push with one argumentValentin Rothberg2020-04-29
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When doing a `podman push $IMG`, $IMG acts as the source and the destination. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | Merge pull request #6036 from giuseppe/fix-rootlessport-panicOpenShift Merge Robot2020-04-29
|\ \ \ \ \ | | | | | | | | | | | | rootlessport: use two different channels
| * | | | | rootlessport: use two different channelsGiuseppe Scrivano2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The same channel is written to by two different goroutines. Use a different channel for each of them so to avoid writing to a closed channel. Closes: https://github.com/containers/libpod/issues/6018 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | | | specgen: honor slirp4netnsGiuseppe Scrivano2020-04-29
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #6031 from mheon/release_191_in_masterOpenShift Merge Robot2020-04-29
|\ \ \ \ \ | | | | | | | | | | | | Update release notes and README for 1.9.1 release
| * | | | | Update release notes and README for 1.9.1 releaseMatthew Heon2020-04-28
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | | Merge pull request #6035 from giuseppe/move-rootless-open-before-forkOpenShift Merge Robot2020-04-29
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | rootless: move ns open before fork
| * | | | | rootless: move ns open before forkGiuseppe Scrivano2020-04-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | commit 788fdc685b00dee5ccb594bef845204250c4c123 introduced a race where the target process dies before the child process opens the namespace files. Move the open before the fork so if it fails the parent process can attempt to join a different container instead of failing. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | | Merge pull request #6022 from vrothberg/enable-inspect-testsOpenShift Merge Robot2020-04-29
|\ \ \ \ \ \ | | | | | | | | | | | | | | enable inspect tests
| * | | | | | enable inspect testsValentin Rothberg2020-04-29
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A surprisingly big change. A core problem was that `podman inspect` allows for passing containers AND images with the default `--type=all`. This only worked partially as the data was processed in isolation which caused various issues (e.g., two separate outputs instead of one) but it also caused issues regarding error handling. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | Merge pull request #5998 from vrothberg/generate-systemdOpenShift Merge Robot2020-04-29
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | generate systemd
| * | | | | generate systemdValentin Rothberg2020-04-29
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement `podman generate systemd` for Podman v2 and enable associated tests. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>