summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Show API doc for several versionsPaul Holzinger2022-02-09
| | | | | | | | | | | Right now it is not possible to look at the API version for a specific version. docs.podman.io always show the latest version from the main branch. This is not want many users want so they now have the ability to select a different version. Fixes #12796 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* Merge pull request #13142 from tmds/ImageCreate_200_schemaOpenShift Merge Robot2022-02-09
|\ | | | | [CI:DOCS] Add schema for ImageCreate 200 response.
| * [NO NEW TEST NEEDED] Add schema for ImageCreate 200 response.Tom Deseyn2022-02-09
| | | | | | | | Signed-off-by: Tom Deseyn <tom.deseyn@gmail.com>
* | Merge pull request #13170 from rhatdan/idmapOpenShift Merge Robot2022-02-09
|\ \ | | | | | | idmap should be able to be specified along with other options
| * | idmap should be able to be specified along with other optionsDaniel J Walsh2022-02-08
|/ / | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] crun is not available everywhere to test idmap. Kernel might not be recent enough and not all file systems support idmap option. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #13164 from ↵OpenShift Merge Robot2022-02-08
|\ \ | | | | | | | | | | | | containers/dependabot/go_modules/github.com/docker/distribution-2.8.0incompatible Bump github.com/docker/distribution from 2.7.1+incompatible to 2.8.0+incompatible
| * | Bump github.com/docker/distributiondependabot[bot]2022-02-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/docker/distribution](https://github.com/docker/distribution) from 2.7.1+incompatible to 2.8.0+incompatible. - [Release notes](https://github.com/docker/distribution/releases) - [Commits](https://github.com/docker/distribution/compare/v2.7.1...v2.8.0) --- updated-dependencies: - dependency-name: github.com/docker/distribution dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
* | | Merge pull request #13163 from myml/myml/fix-durationOpenShift Merge Robot2022-02-08
|\ \ \ | | | | | | | | fix: Multiplication of durations
| * | | fix: Multiplication of durationsmyml2022-02-08
| |/ / | | | | | | | | | | | | | | | | | | 'killContainerTimeout' is already 5 second [NO NEW TESTS NEEDED] Signed-off-by: myml <wurongjie1@gmail.com>
* | | Merge pull request #13159 from Luap99/slirp4-scopeOpenShift Merge Robot2022-02-08
|\ \ \ | |/ / |/| | move rootless netns slirp4netns process to systemd user.slice
| * | move rootless netns slirp4netns process to systemd user.slicePaul Holzinger2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When running podman inside systemd user units, it is possible that systemd kills the rootless netns slirp4netns process because it was started in the default unit cgroup. When the unit is stopped all processes in that cgroup are killed. Since the slirp4netns process is run once for all containers it should not be killed. To make sure systemd will not kill the process we move it to the user.slice. Fixes #13153 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* | | Merge pull request #13156 from flouthoc/fix-compat-build-response-headerOpenShift Merge Robot2022-02-07
|\ \ \ | | | | | | | | compat: endpoint `/build` must set header `content type` as `application/json` in response header.
| * | | compat: endpoint /build must set header content type as application/json in ↵Aditya R2022-02-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | reponse Lot of clients are expecting proper `Content-type: application/json` configured in response headers of `/build` compat api. Following commit fixes that. Fixes issues where code is setting header field after writing header which is wrong. We must set `content-type` before we write and flush http header. Signed-off-by: Aditya R <arajan@redhat.com>
* | | | Merge pull request #13158 from edsantiago/batsOpenShift Merge Robot2022-02-07
|\ \ \ \ | | | | | | | | | | Cleanup: remove obsolete/misleading bug workaround
| * | | | Cleanup: remove obsolete/misleading bug workaroundEd Santiago2022-02-07
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Followup to #13129: remove a no-longer-necessary workaround for a healthcheck bug. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #13129 from flouthoc/healthcheck-session-read-from-pipeOpenShift Merge Robot2022-02-07
|\ \ \ \ | |_|/ / |/| | | healthcheck, libpod: Read healthcheck event output from os pipe
| * | | tests: retrofit healthcheck system testsAditya R2022-02-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All the healthcheck return output now but systems tests is written to expect empty output which seems wrong. Modify jq output to contain newline character rather than actual newline Signed-off-by: Aditya R <arajan@redhat.com>
| * | | healthcheck, libpod: Read healthcheck event output from os pipeAditya R2022-02-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems we are ignoring output from healthcheck session. Open a valid pipe to healthcheck session in order read its output. Use common pipe for both `stdout/stderr` since that was the previous behviour as well. Signed-off-by: Aditya R <arajan@redhat.com>
* | | | Merge pull request #13155 from myml/mainOpenShift Merge Robot2022-02-07
|\ \ \ \ | |_|/ / |/| | | Fix: Do not print error when parsing journald log fails
| * | | Fix: Do not print error when parsing journald log failsmyml2022-02-07
|/ / / | | | | | | | | | | | | | | | | | | foramtError was written as err [NO NEW TESTS NEEDED] Signed-off-by: myml <wurongjie1@gmail.com>
* | | Merge pull request #13147 from cevich/more_netavarkOpenShift Merge Robot2022-02-04
|\ \ \ | | | | | | | | Cirrus: Expand netavark testing to include rootless
| * | | Cirrus: Expand netavark testing to include rootlessChris Evich2022-02-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The list of netavark/aardvark-dns tasks is likely to be ever-growing for the near-term. Consolidate them into a single task entry with a matrix attribute to make future expansions less YAML-duplicative. Also, re-arrange it to fall in-between the CNI integration and system task sections. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | Merge pull request #13146 from cevich/dependabot_gotermOpenShift Merge Robot2022-02-04
|\ \ \ \ | | | | | | | | | | Bump github.com/buger/goterm from 1.0.1 to 1.0.4
| * | | | Bump github.com/buger/goterm from 1.0.1 to 1.0.4dependabot[bot]2022-02-04
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/buger/goterm](https://github.com/buger/goterm) from 1.0.1 to 1.0.4. - [Release notes](https://github.com/buger/goterm/releases) - [Commits](https://github.com/buger/goterm/compare/v1.0.1...v1.0.4) --- updated-dependencies: - dependency-name: github.com/buger/goterm dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | | Merge pull request #13143 from Luap99/dnsOpenShift Merge Robot2022-02-04
|\ \ \ \ | |_|/ / |/| | | append podman dns search domain
| * | | append podman dns search domainPaul Holzinger2022-02-04
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Append the podman dns seach domain to the host search domains when we use the dnsname/aardvark server. Previously it would only use podman seach domains and discard the host domains. Fixes #13103 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* | | Merge pull request #12930 from cdoern/podCgroupOpenShift Merge Robot2022-02-04
|\ \ \ | | | | | | | | Podman pod create --share-parent vs --share=cgroup
| * | | Podman pod create --share-parent vs --share=cgroupcdoern2022-02-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | separated cgroupNS sharing from setting the pod as the cgroup parent, made a new flag --share-parent which sets the pod as the cgroup parent for all containers entering the pod remove cgroup from the default kernel namespaces since we want the same default behavior as before which is just the cgroup parent. resolves #12765 Signed-off-by: cdoern <cdoern@redhat.com> Signed-off-by: cdoern <cbdoer23@g.holycross.edu> Signed-off-by: cdoern <cdoern@redhat.com>
* | | | Merge pull request #13134 from edsantiago/batsOpenShift Merge Robot2022-02-04
|\ \ \ \ | | | | | | | | | | System tests: revert emergency skip of checkpoint tests
| * | | | System tests: revert emergency skip of checkpoint testsEd Santiago2022-02-03
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert #13049. criu-3.16.1-6.fc36 fixes the problem and is now in fc36 stable: https://bodhi.fedoraproject.org/updates/FEDORA-2022-183b337712 (Yes, I confirmed that tests pass on a rawhide vm) Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #13136 from rhatdan/VENDOROpenShift Merge Robot2022-02-04
|\ \ \ \ | |/ / / |/| | | Update containers/buildah v1.24.1
| * | | Update containers/buildah v1.24.1Daniel J Walsh2022-02-03
|/ / / | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #13024 from cevich/netavark_systemOpenShift Merge Robot2022-02-03
|\ \ \ | | | | | | | | Cirrus: Add netavark/aardvark system test task
| * | | Cirrus: Add netavark/aardvark system test taskChris Evich2022-02-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add a system-test that verifies netavark driver is in use when magic env. var. is set. Signed-off-by: Chris Evich <cevich@redhat.com>
| * | | Cirrus: Log netavark/aardvark binary build info.Chris Evich2022-02-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enabled by: * https://github.com/containers/netavark/pull/191 * https://github.com/containers/aardvark-dns/pull/36 Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | Merge pull request #13131 from ↵Daniel J Walsh2022-02-03
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/containers/common-0.47.3 Bump github.com/containers/common from 0.47.2 to 0.47.3
| * | | | Bump github.com/containers/common from 0.47.2 to 0.47.3dependabot[bot]2022-02-03
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/containers/common](https://github.com/containers/common) from 0.47.2 to 0.47.3. - [Release notes](https://github.com/containers/common/releases) - [Commits](https://github.com/containers/common/compare/v0.47.2...v0.47.3) --- updated-dependencies: - dependency-name: github.com/containers/common dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | | Merge pull request #13127 from ↵Daniel J Walsh2022-02-03
|\ \ \ \ | |/ / / |/| | | | | | | | | | | containers/dependabot/go_modules/github.com/containers/image/v5-5.19.1 Bump github.com/containers/image/v5 from 5.19.0 to 5.19.1
| * | | Bump github.com/containers/image/v5 from 5.19.0 to 5.19.1dependabot[bot]2022-02-03
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/containers/image/v5](https://github.com/containers/image) from 5.19.0 to 5.19.1. - [Release notes](https://github.com/containers/image/releases) - [Commits](https://github.com/containers/image/compare/v5.19.0...v5.19.1) --- updated-dependencies: - dependency-name: github.com/containers/image/v5 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | Merge pull request #13125 from nalind/url-docsOpenShift Merge Robot2022-02-03
|\ \ \ | | | | | | | | [CI:DOCS] Document `schema` values in the `--url` flag
| * | | Document `schema` values in the `--url` flagNalin Dahyabhai2022-02-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Document the recognized `schema` types that can be used in a value passed to the `--url` command line flag. [CI:DOCS] Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
* | | | Merge pull request #13047 from cdoern/scpSyntaxOpenShift Merge Robot2022-02-03
|\ \ \ \ | | | | | | | | | | podman image scp syntax correction
| * | | | podman image scp syntax correctioncdoern2022-01-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] image scp was reading the localhost syntax too loosely causing some errors with domains or hosts containing the word localhost. Fixed that and added a few lines to make sure the pure localhost connections do not touch sshd resolves #13021 Signed-off-by: cdoern <cdoern@redhat.com>
* | | | | Merge pull request #13117 from Luap99/resolvOpenShift Merge Robot2022-02-03
|\ \ \ \ \ | | | | | | | | | | | | Revert "Move each search dns to its own line"
| * | | | | Revert "Move each search dns to its own line"Paul Holzinger2022-02-02
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit a1bc8cb52cefd49e8cc54ae14d1864b8a1ec216e. Please see resolv.conf(5) search domains must be on the same line. If you use multiple seach key words only the last one is used. I tested this with alpine and it works correctly when they are on the same line so I am not sure what issues Dan had with it but this is not correct. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* | | | | Merge pull request #13121 from ydayagi/resourcefieldrefOpenShift Merge Robot2022-02-03
|\ \ \ \ \ | | | | | | | | | | | | play kube envVar.valueFrom.resourceFieldRef
| * | | | | play kube envVar.valueFrom.resourceFieldRefYaron Dayagi2022-02-02
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add support for env vars values from container resources see https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.19/#envvarsource-v1-core relates to issue https://github.com/containers/podman/issues/12756 Signed-off-by: Yaron Dayagi <ydayagi@redhat.com>
* | | | | Merge pull request #13116 from Luap99/reset-networksOpenShift Merge Robot2022-02-03
|\ \ \ \ \ | |_|_|_|/ |/| | | | system prune: remove all networks
| * | | | system prune: remove all networksPaul Holzinger2022-02-03
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | podman system prune should also remove all networks. When we want to users to migrate to the new network stack we recommend to run podman system reset. However this did not remove networks and if there were still networks around we would continue to use cni since this was considered an old system. There is one exception for the default network. It should not be removed since this could cause other issues when it no longer exists. The network backend detection logic ignores the default network so this is fine. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* | | | Merge pull request #13120 from nalind/slashify-dockerfileOpenShift Merge Robot2022-02-03
|\ \ \ \ | | | | | | | | | | pkg/bindings/images.Build(): slashify "dockerfile" values, too