summaryrefslogtreecommitdiff
path: root/cmd/podman
Commit message (Collapse)AuthorAge
* Exit with errors not just logging errorDaniel J Walsh2019-02-22
| | | | | | | Several commands were logging errors but exiting with a 0 exit code. This patch cleans these up. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* Merge pull request #2386 from baude/cobravalidationOpenShift Merge Robot2019-02-22
|\ | | | | Improve command line validation
| * hide --latest on the remote-clientbaude2019-02-22
| | | | | | | | | | | | | | | | in the case of the remote-client, it was decided to hide the latest flag to avoid confusion for end-users on what the "last" container, volume, or pod are. Signed-off-by: baude <bbaude@redhat.com>
| * Improve command line validationbaude2019-02-22
| | | | | | | | | | | | | | Use the checkallandlatest function to validate flag usage as part of the cobra command args validation. Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #2402 from baude/remotepodinspectOpenShift Merge Robot2019-02-22
|\ \ | |/ |/| podman-remote pod inspect|exists
| * podman-remote pod inspect|existsbaude2019-02-22
| | | | | | | | | | | | | | | | enable the remote client to be able to inspect a pod. also, bonus of enabling the podman pod exists command which returns a 0 or 1 depending on whether the given pod exists. Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #2389 from baude/issue2388OpenShift Merge Robot2019-02-22
|\ \ | | | | | | add newline to images output
| * | update: remove duplicate newlineEd Santiago2019-02-21
| | | | | | | | | | | | Signed-off-by: Ed Santiago <santiago@redhat.com>
| * | add newline to images outputbaude2019-02-20
| | | | | | | | | | | | | | | | | | | | | | | | ensure a final newline is always added to images output. fixes #2388 Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #2350 from mheon/lock_renumberOpenShift Merge Robot2019-02-21
|\ \ \ | |_|/ |/| | Add lock renumbering
| * | Fix typo in commentMatthew Heon2019-02-21
| | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | Do not make renumber shut down the runtimeMatthew Heon2019-02-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original intent behind the requirement was to ensure that, if two SHM lock structs were open at the same time, we should not make such a runtime available to the user, and should clean it up instead. It turns out that we don't even need to open a second SHM lock struct - if we get an error mapping the first one due to a lock count mismatch, we can just delete it, and it cleans itself up when it errors. So there's no reason not to return a valid runtime. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | Add podman system renumber commandMatthew Heon2019-02-21
| | | | | | | | | | | | | | | | | | | | | | | | This command allows for renumbering Podman locks after an upgrade to Podman with SHM locks from a 1.0 or earlier branch, or after the number of locks was changed. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | Add ability to get a runtime that renumbersMatthew Heon2019-02-21
| | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | podman-remote load imagebaude2019-02-21
|/ / | | | | | | | | | | | | enable the ability to load an image into remote storage using the remote client. Signed-off-by: baude <bbaude@redhat.com>
* | enable podman-remote pod rmbaude2019-02-21
| | | | | | | | | | | | add the ability to delete a pod from the remote client. Signed-off-by: baude <bbaude@redhat.com>
* | podman-remote save [image]baude2019-02-20
|/ | | | | | | Add the ability to save an image from the remote-host to the remote-client. Signed-off-by: baude <bbaude@redhat.com>
* Merge pull request #2377 from vrothberg/kill-squaredOpenShift Merge Robot2019-02-20
|\ | | | | remove duplicate kill from `podman --help`
| * remove duplicate kill from `podman --help`Valentin Rothberg2019-02-20
| | | | | | | | | | | | | | | | Remove the duplicate kill command and only keep it in the `mainCommands` containing commands that are implemented by the native client and the remote one. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | Merge pull request #2360 from vrothberg/parallel-searchOpenShift Merge Robot2019-02-20
|\ \ | | | | | | podman-search: run in parallel
| * | iopodman.SearchImages: add ImageSearchFilter to Varlink APIValentin Rothberg2019-02-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add some argument checks to the Varlink function to avoid referencing nil pointers, and complement the API.md descriptions. The varlink endpoint can be tested via varlink CLI: $ varlink call -m unix:/run/podman/io.podman/io.podman.SearchImages \ '{"query": "ruby", "limit": 0, "tlsVerify": false, "filter": {}}' Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | image.SearchImages: use SearchFilter typeValentin Rothberg2019-02-20
| | | | | | | | | | | | | | | | | | | | | Use an `image.SearchFilter` instead of a `[]string` in the SearchImages API. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | SearchImages: extend API with filter parameterValentin Rothberg2019-02-20
| | | | | | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | podman-search: refactor code to libpod/image/search.goValentin Rothberg2019-02-20
| | | | | | | | | | | | | | | | | | | | | | | | Refactor the image-search logic from cmd/podman/search.go to libpod/image/search.go and update podman-search and the Varlink API to use it. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | podman-search: run in parallelValentin Rothberg2019-02-20
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Spin up a goroutine for each registry to make podman-search run in parallel. This has considerable speed improvements. For instance, a `podman search ruby` drops from 11 to 2 seconds when using the following search registries: ```toml [registries.search] registries = ['docker.io', 'registry.fedoraproject.org', 'quay.io', 'registry.access.redhat.com', 'registry.centos.org'] ``` The number of parallel goroutines is limited to 6 to play nice with local resources and the registries. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | Merge pull request #2374 from mheon/rootless_recursive_startOpenShift Merge Robot2019-02-20
|\ \ | |/ |/| Ensure that userns is created for stopped rootless pods
| * Ensure that userns is created for stopped rootless podsMatthew Heon2019-02-19
| | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | Podman pod create now errors on receiving CLI argsMatthew Heon2019-02-19
|/ | | | | | | It has never accepted arguments, so we should error when passed args we will never use. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* Merge pull request #2363 from TomSweeneyRedHat/dev/tsweeney/cobraex5OpenShift Merge Robot2019-02-19
|\ | | | | Fifth chunk of Cobra Examples
| * Fifth chunk of Cobra ExamplesTomSweeneyRedHat2019-02-18
| | | | | | | | | | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> The fifth and final chunk of examples for the Cobra examples in the CLI help output. Also includes a few man page touchups. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | podman-remote pullbaude2019-02-19
|/ | | | | | Add status for remote users and podman remote-client pull. Signed-off-by: baude <bbaude@redhat.com>
* Merge pull request #2361 from TomSweeneyRedHat/dev/tsweeney/cobraex4OpenShift Merge Robot2019-02-18
|\ | | | | Add 4th chunk of Cobra Examples
| * Add 4th chunk of Cobra ExamplesTomSweeneyRedHat2019-02-18
| | | | | | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> Adds the 4th chunk of Cobra Examples to the CLI help. One more chunk of 10 to complete. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | OpenTracing support added to start, stop, run, create, pull, and psSebastian Jug2019-02-18
|/ | | | | | Drop context.Context field from cli.Context Signed-off-by: Sebastian Jug <sejug@redhat.com>
* Merge pull request #2354 from rhatdan/varlinkOpenShift Merge Robot2019-02-18
|\ | | | | Add registry name to fields returned by varlink image search
| * Add registry name to fields returned by varlink image searchDaniel J Walsh2019-02-15
| | | | | | | | | | | | | | | | | | | | | | Cockpit team wants to list the registry name where the image was found. Also fix up SearchImages code to check if the user specified a registry in his call to use that rather then all the registries, This matches podman search command. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #2357 from TomSweeneyRedHat/dev/tsweeney/cobrahelp3OpenShift Merge Robot2019-02-17
|\ \ | | | | | | Add 3rd chunk of Cobra examples
| * | Add 3rd chunk of Cobra examplesTomSweeneyRedHat2019-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> Adds the third chunk of Cobra examples to the cli help. As were putting together a release tomorrow, tried to hit the heavy commands with this PR. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | Merge pull request #2295 from haircommander/recursive-startOpenShift Merge Robot2019-02-17
|\ \ \ | |/ / |/| | Recursively start a pod if a container is run in it
| * | pod infra container is started before a container in a pod is run, started, ↵Peter Hunt2019-02-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | or attached. Prior, a pod would have to be started immediately when created, leading to confusion about what a pod state should be immediately after creation. The problem was podman run --pod ... would error out if the infra container wasn't started (as it is a dependency). Fix this by allowing for recursive start, where each of the container's dependencies are started prior to the new container. This is only applied to the case where a new container is attached to a pod. Also rework container_api Start, StartAndAttach, and Init functions, as there was some duplicated code, which made addressing the problem easier to fix. Signed-off-by: Peter Hunt <pehunt@redhat.com>
* | | Merge pull request #2340 from QiWang19/login_lyingOpenShift Merge Robot2019-02-17
|\ \ \ | | | | | | | | Fix `podman login` lying problem
| * | | Fix `podman login` lying problemQi Wang2019-02-14
| | | | | | | | | | | | | | | | Signed-off-by: Qi Wang <qiwan@redhat.com>
* | | | Merge pull request #2341 from TomSweeneyRedHat/dev/tsweeney/helpexamples2OpenShift Merge Robot2019-02-16
|\ \ \ \ | |_|/ / |/| | | Second chunk of Cobra help
| * | | Second chunk of Cobra helpTomSweeneyRedHat2019-02-15
| | |/ | |/| | | | | | | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> Adds examples to Cobra help for a second chunk of commands. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | Merge pull request #2335 from mheon/enable_detach_rmOpenShift Merge Robot2019-02-15
|\ \ \ | |/ / |/| | Enable --rm with --detach
| * | Fix error code retrieval for podman start --attachMatthew Heon2019-02-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we start a container with 'podman run' and it exits and is removed before we get the code, we grab its exit code from the Conmon exit file. Podman start --attach wants to do the same, but was missing the logic. Fix that here. Also, remove some --rm handling leftover in start. Don't need it anymore, we have the cleanup process now. Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | Enable --rm with --detachMatthew Heon2019-02-14
| | | | | | | | | | | | | | | | | | | | | We can now safely remove detached containers as --rm handling has moved into the cleanup process. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | podman: honor --storage-opt againGiuseppe Scrivano2019-02-15
| | | | | | | | | | | | | | | | | | possibly a regression after we moved to Cobra Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | Merge pull request #2188 from TomSweeneyRedHat/dev/tsweeney/pullallOpenShift Merge Robot2019-02-15
|\ \ \ | | | | | | | | Add --all-tags to pull command
| * | | Add --all-tags to pull commandTomSweeneyRedHat2019-02-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> Add --all-tags for the `podman pull` command so all tags of an image will be pulled, not just ':latest'. Emulates the change in Buildah https://github.com/containers/buildah/pull/1263 Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>