summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* stats refactorValentin Rothberg2020-09-23
| | | | | | | | | | | | Refactor the entities' stats API to simplify using it and reduce the risk of running into concurrency issues at the call sites. Further simplify the stats code by de-spaghetti-ing the logic and reducing duplicate code. `ContainerStats` now returns a data channel and an error. If the error is nil, callers can read from the channel. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* Merge pull request #7734 from lsm5/fix-build-with-varlinkOpenShift Merge Robot2020-09-22
|\ | | | | fix build with varlink
| * fix build with varlinkLokesh Mandvekar2020-09-22
| | | | | | | | | | | | | | also add a cirrus task for building binaries with varlink. From: Chris Evich <cevich@redhat.com> Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | Merge pull request #7712 from baude/killreturnOpenShift Merge Robot2020-09-22
|\ \ | | | | | | add missing return for compat kill
| * | add missing return for compat killbaude2020-09-21
| | | | | | | | | | | | | | | | | | on an error condition in kill for the compatibility layer, we were missing a return. Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #7729 from mheon/info_cgroup_managerOpenShift Merge Robot2020-09-22
|\ \ \ | | | | | | | | Include cgroup manager in `podman info` output
| * | | Include cgroup manager in `podman info` outputMatthew Heon2020-09-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is very useful for debugging cgroups v2, especially on rootless - we need to ensure people are correctly using systemd cgroups in these cases. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | Merge pull request #7733 from mheon/bump_readme_210OpenShift Merge Robot2020-09-22
|\ \ \ \ | | | | | | | | | | [CI:DOCS] Bump version in README to v2.1.0
| * | | | Bump version in README to v2.1.0Matthew Heon2020-09-22
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | Merge pull request #7730 from jwhonce/wip/headersOpenShift Merge Robot2020-09-22
|\ \ \ \ \ | |_|_|_|/ |/| | | | Add Server header to API service responses
| * | | | Add Server header to API service responsesJhon Honce2020-09-22
| | |/ / | |/| | | | | | | | | | | | | | | | | | Aids in reading logs of different services Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | Merge pull request #7400 from cevich/fix_missing_gpg2OpenShift Merge Robot2020-09-22
|\ \ \ \ | | | | | | | | | | Cirrus: Provide gpg2 in Ubuntu images
| * | | | Cirrus: Add gpg2 to Ubuntu imagesChris Evich2020-09-17
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #7719 from vrothberg/fix-multi-image-saveOpenShift Merge Robot2020-09-22
|\ \ \ \ \ | |_|_|/ / |/| | | | podman save: fix redirect of multi-images
| * | | | podman save: fix redirect of multi-imagesValentin Rothberg2020-09-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix a bug when saving multi-image archives where the progress bars from copying images was accidentally written to the archive and hence corrupted it. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | Merge pull request #7728 from mheon/bump-2.1.0OpenShift Merge Robot2020-09-22
|\ \ \ \ \ | |_|_|/ / |/| | | | [CI:DOCS] Bump to v2.1.0
| * | | | Bump to v2.2.0-devMatthew Heon2020-09-22
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | Bump to v2.1.0v2.1.0Matthew Heon2020-09-22
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | Merge pull request #7724 from vrothberg/hooksOpenShift Merge Robot2020-09-22
|\ \ \ \ \ | | | | | | | | | | | | pkg/hooks: support all hooks
| * | | | | pkg/hooks: support all hooksValentin Rothberg2020-09-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add support for the missing hook types [1]: - createContainer - createRuntime - startContainer Otherwise, Podman won't inject them into the runtime config (and pass it on to runc/crun) but error out. [1] https://github.com/opencontainers/runtime-spec/blob/44341cdd36f6fee6ddd73e602f9e3eca1466052f/runtime.md#lifecycle Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | Merge pull request #7714 from rhatdan/makeOpenShift Merge Robot2020-09-22
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Print nice error message when python is not installed
| * | | | | Print nice error message when python is not installedDaniel J Walsh2020-09-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If user runs `make help` on a system without python, we should print a decent message. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #7725 from mheon/release_notes_210_finalOpenShift Merge Robot2020-09-22
|\ \ \ \ \ \ | | | | | | | | | | | | | | [CI:DOCS] Update release notes for v2.1.0 Final Release
| * | | | | | Update release notes for v2.1.0 Final ReleaseMatthew Heon2020-09-22
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | | Merge pull request #7720 from vrothberg/campfireOpenShift Merge Robot2020-09-22
|\ \ \ \ \ \ | | |/ / / / | |/| | | | update stale bot
| * | | | | update stale botValentin Rothberg2020-09-22
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the GitHub action to mark issues and PRs as stale. There are a couple of useful features, most importantly, the bot will remove the stale label from issues as soon as there's either an activity or a comment. This reduces some manual overhead: the stale bot will only drop a comment on issues and PRs that are not marked as stale. Hence, as we appreciated the reminders, we had to manually remove the label which should now turn into campfire tales. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | Merge pull request #7717 from rhatdan/attachOpenShift Merge Robot2020-09-22
|\ \ \ \ \ | | | | | | | | | | | | Fix up attach tests for podman remote
| * | | | | Fix up attach tests for podman remoteDaniel J Walsh2020-09-22
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we execute podman-remote attach, we were not checking if the container was in the correct state, this is leading to timeouts and we had turned off remote testing. Also added an IfRemote() function so we can turn on more tests when using the "-l" flag for local, but use container name for remote. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #7691 from rhatdan/mountOpenShift Merge Robot2020-09-22
|\ \ \ \ \ | |/ / / / |/| | | | Fix podman image unmount to only report images unmounted
| * | | | Fix podman image unmount to only report images unmountedDaniel J Walsh2020-09-21
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Currently `podman image unmount` report every image that is mounted when it unmounts them. We should only report unmounted actually mounted images. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #7715 from TomSweeneyRedHat/dev/tsweeney/contribOpenShift Merge Robot2020-09-21
|\ \ \ \ | | | | | | | | | | [CI:DOCS] Add 'In Progress' note to CONTRIBUTING.md
| * | | | [CI:DOCS] Add 'In Progress' note to CONTRIBUTING.mdTomSweeneyRedHat2020-09-21
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the CONTRIBUTING.md to explain the use of the new "In Progress" label in the Podman repository. This emulates the verbiage in Buildah. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | Merge pull request #7695 from jwhonce/issues/7686OpenShift Merge Robot2020-09-21
|\ \ \ \ | |_|/ / |/| | | Restore 'id' stanza in pull results
| * | | Restore 'id' stanza in pull resultsJhon Honce2020-09-21
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | id is the last image id from the set of id's returned via the images stanza. id may be deprecated in a future version of the API Created test_rest_v2_0_0.py to reflect the bump in the API Version. Fixes #7686 Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | Merge pull request #7698 from AkihiroSuda/rootless-cni-deflake-lo-upOpenShift Merge Robot2020-09-21
|\ \ \ | |/ / |/| | rootless-cni-infra: fix flakiness during bringing up lo interface
| * | libpod: bumps up rootless-cni-infra to 2Akihiro Suda2020-09-21
| | | | | | | | | | | | Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
| * | rootless-cni-infra: fix flakiness during bringing up lo interfaceAkihiro Suda2020-09-20
| | | | | | | | | | | | | | | | | | | | | | | | Fix #7618 This bumps up ROOTLESS_CNI_INFRA_VERSION to 2 Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
* | | Merge pull request #7688 from vrothberg/fix-7521OpenShift Merge Robot2020-09-21
|\ \ \ | | | | | | | | remote stats fixes
| * | | stats: log errors instead of sending 500Valentin Rothberg2020-09-21
| | | | | | | | | | | | | | | | | | | | | | | | As 200 is already out the door, we cannot send 500s anymore. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | stats: detect closed client connectionValentin Rothberg2020-09-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Detect closed client connections and stop streaming. Fixes: #7521 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | stats endpoint: write OK header onceValentin Rothberg2020-09-18
| | | | | | | | | | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #7697 from rhatdan/ignoreOpenShift Merge Robot2020-09-21
|\ \ \ \ | | | | | | | | | | Fix handling of podman-remote stop --ignore
| * | | | Fix handling of podman-remote stop --ignoreDaniel J Walsh2020-09-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch simplifies the hanlding of rhe --ignore flag, for podman stop, rm Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #7696 from jwhonce/wip/versionOpenShift Merge Robot2020-09-21
|\ \ \ \ \ | | | | | | | | | | | | Refactor version handling in cmd tree
| * | | | | Refactor version handling in cmd treeJhon Honce2020-09-18
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move from simple string to semver objects * Change client API Version from '1' to 2.0.0 Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | | Merge pull request #7665 from ↵OpenShift Merge Robot2020-09-21
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/rootless-containers/rootlesskit-0.10.1 Bump github.com/rootless-containers/rootlesskit from 0.10.0 to 0.10.1
| * | | | | Bump github.com/rootless-containers/rootlesskit from 0.10.0 to 0.10.1Daniel J Walsh2020-09-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/rootless-containers/rootlesskit](https://github.com/rootless-containers/rootlesskit) from 0.10.0 to 0.10.1. - [Release notes](https://github.com/rootless-containers/rootlesskit/releases) - [Commits](https://github.com/rootless-containers/rootlesskit/compare/v0.10.0...v0.10.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #7702 from Luap99/fix#7701OpenShift Merge Robot2020-09-21
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Fix incorrect parsing of create/run --volumes-from
| * | | | | Fix incorrect parsing of create/run --volumes-fromPaul Holzinger2020-09-20
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a bunch of tests to ensure that --volumes-from works as expected. Also align the podman create and run man page. Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* | | | | Merge pull request #7674 from xordspar0/bats-install-rootOpenShift Merge Robot2020-09-19
|\ \ \ \ \ | | | | | | | | | | | | Install bats as root