aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Fix errors found in coverity scanDaniel J Walsh2020-05-01
| | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* Merge pull request #6057 from baude/v2networkingOpenShift Merge Robot2020-05-01
|\ | | | | v2networking enable commands
| * add provided cni networks to spec genbaude2020-05-01
| | | | | | | | | | | | enable final integration tests for networking. Signed-off-by: baude <bbaude@redhat.com>
| * v2networking enable commandsbaude2020-04-30
| | | | | | | | | | | | Enable the networking commands for v2. Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #6004 from rhatdan/ulimitsOpenShift Merge Robot2020-05-01
|\ \ | | | | | | Set up ulimits for rootless containers.
| * | Set up ulimits for rootless containers.Daniel J Walsh2020-04-28
| | | | | | | | | | | | | | | | | | | | | Currently we are setting the maximum limits for rootful podman containers, no reason not to set them by default for rootless users as well Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #6016 from giuseppe/fix-createOpenShift Merge Robot2020-05-01
|\ \ \ | | | | | | | | v2, podman: fix create and entrypoint tests
| * | | test: enable start testsGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | | | | | | | | | `docker start` doesn't resolve partial ID -> full ID, adapt test. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | podman, start: propagate back the raw inputGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | this is necessary as we expect "podman start $ID_NAME" to print the same arguments the user passed in instead of the full ID. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | test: enable remaining run testsGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | test: enable entrypoint testsGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | test: enable create testsGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | cmd, podman: do not override entrypoint if unsetGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | cmd, podman: use String instead of variable+StringVarGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | cmd, podman: handle --pod new:PODGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | create: propagate override-arch and override-osGiuseppe Scrivano2020-04-30
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | 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