summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* set interactive mode with compat create endpointbaude2020-09-23
| | | | | | when creating a container using the compat endpoint, the interactive bool was being hard set to false and ignoring the user's input. Signed-off-by: baude <bbaude@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
| * | | | | Install bats as rootJordan Christiansen2020-09-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Installing bats to /usr/local requires root privileges. Without this, `make install.tools` fails. However, if I do `sudo make install.tools`, then all of the other dependencies and git clones in the current directory end up owned by root. This limits root privileges to the part that needs it. Signed-off-by: Jordan Christiansen <xordspar0@gmail.com>
* | | | | | Merge pull request #7629 from cevich/fix_dbuildOpenShift Merge Robot2020-09-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Makefile: Remove broken targets
| * | | | | | Makefile: Fix broken libpodimage targetsChris Evich2020-09-18
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The configuration for this container has moved to the `containers/automation_images` repository. Leverage the image built from the new location to guarantee synchronization with images used in CI. Also, remove several other targets that haven't worked for a really long time. Upon discussion, they seem to apply to a minority of users. Remove them to clean up the `Makefile` and reduce maintenance burden. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | | Merge pull request #7694 from mheon/fix_exec_supplemental_groupsOpenShift Merge Robot2020-09-19
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Preserve groups in exec sessions in ctrs with --user
| * | | | | Preserve groups in exec sessions in ctrs with --userMatthew Heon2020-09-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Podman wants to guarantee that exec sessions retain the groups of the container they are started in, unless explicitly overridden by the user. This guarantee was broken for containers where the `--user` flag was specified; this patch resolves that. Somewhere in the Exec rewrite for APIv2, I changed the location where the container's User is passed into the exec session (similar to groups, we also want to preserve user unless overridden). The lower-level Exec APIs already handled setting user and group appropriately if not specified when the exec session was created, but I added duplicate code to handle this higher in the stack - and that code only handled setting user, not supplemental groups, breaking support in that specific case. Two things conspired to make this one hard to track down: first, things were only broken if the container explicitly set a user; otherwise, the container user would still appear to be unset to the lower-level code, which would properly set supplemental groups (this tricked our existing test into passing). Also, the `crun` OCI runtime will add the groups without prompting, which further masked the problem there. I debated making `runc` do the same, but in the end it's better to fix this in Podman - it's better to be explicit about what we want done so we will work with all OCI runtimes. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | | | Merge pull request #7660 from ashley-cui/logsOpenShift Merge Robot2020-09-18
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix remote logs
| * | | | | | WIP: Fix remote logsAshley Cui2020-09-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docker compatibility - logs endpoint does not write stream headers if container has a tty Signed-off-by: Ashley Cui <acui@redhat.com>
* | | | | | | Merge pull request #7594 from alvistack/master-linux-amd64OpenShift Merge Robot2020-09-18
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | Update nix pin with `make nixpkgs`