aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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
| * | | | pkg/bindings/images.Build(): slashify "dockerfile" values, tooNalin Dahyabhai2022-02-02
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the Dockerfile isn't in the root directory of the build context, the client supplies its pathname to the server, but it needs to do so using "/" as the path separator, not the client OS's path separator. CI can't test Windows clients, so [NO NEW TESTS NEEDED] Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
* | | | Merge pull request #12859 from baude/netavarke2eOpenShift Merge Robot2022-02-02
|\ \ \ \ | | | | | | | | | | Enable e2e tests with netavark
| * | | | netavark e2e testsBrent Baude2022-02-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | enabled e2e tests for netavark Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | Merge pull request #13118 from edsantiago/batsOpenShift Merge Robot2022-02-02
|\ \ \ \ \ | |_|/ / / |/| | | | hack/bats: fix broken usage message
| * | | | hack/bats: fix broken usage messageEd Santiago2022-02-02
|/ / / / | | | | | | | | | | | | | | | | | | | | I am so, so sorry. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #13089 from rhatdan/resolv.confOpenShift Merge Robot2022-02-02
|\ \ \ \ | | | | | | | | | | Only change network fields if they were actually changed by the user
| * | | | Move each search dns to its own lineDaniel J Walsh2022-02-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Alpine does not seem to use search correctly when there are multiple search domains on the same line. It only uses the first with the advent. When podman runs within a separate network we are appending on dns.podman as a search, if you add a search domain, then this causes the local search on network to fail. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
| * | | | Only change network fields if they were actually changed by the userDaniel J Walsh2022-02-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/13065 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #13114 from cevich/minor_cirrusOpenShift Merge Robot2022-02-02
|\ \ \ \ \ | | | | | | | | | | | | [CI:DOCS] Cirrus: Restrict special/limited-use task
| * | | | | Cirrus: Minor - limit release task applicabilityChris Evich2022-02-02
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This task/test is guaranteed to fail on non-release PRs. Upon reviewing actual practice and the release docs, this task should be limited to PRs with a title containing `release` or `bump` keywords. Also fix a minor comment typo. Ref: https://github.com/containers/podman/pull/13106#pullrequestreview-869855449 Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #13104 from giuseppe/fix-podman-stats-commentOpenShift Merge Robot2022-02-02
|\ \ \ \ \ | |/ / / / |/| | | | [CI:DOCS] docs: clarify rootless net stats
| * | | | docs: clarify rootless net statsGiuseppe Scrivano2022-02-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | follow-up for https://github.com/containers/podman/pull/13101 [CI:DOCS] Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #13079 from TomSweeneyRedHat/dev/tsweeney/hellowOpenShift Merge Robot2022-02-02
|\ \ \ \ \ | | | | | | | | | | | | [CI:DOCS] Add a hello world image to quay.io
| * | | | | [CI:DOCS] Add a hello world image to quay.iotomsweeneyredhat2022-02-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] This commit describes how to run the quay.io/podman/hello image. It also contains the files necessary to build that image localy, and a README.md explaining the image and how to build it. Signed-off-by: tomsweeneyredhat <tsweeney@redhat.com>
* | | | | | Merge pull request #13107 from rhatdan/shmOpenShift Merge Robot2022-02-02
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Fix size to match Docker selection
| * | | | | Fix size to match Docker selectionDaniel J Walsh2022-02-01
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/13096 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #13110 from giuseppe/enforce-dev-shm-with-noexec-nosuid-nodevOpenShift Merge Robot2022-02-02
|\ \ \ \ \ | | | | | | | | | | | | libpod: enforce noexec,nosuid,nodev for /dev/shm
| * | | | | libpod: enforce noexec,nosuid,nodev for /dev/shmGiuseppe Scrivano2022-02-02
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | these mount flags are already used for the /dev/shm mount on the host, but they are not set for the bind mount itself. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #13106 from cevich/not_buildOpenShift Merge Robot2022-02-02
|\ \ \ \ \ | |/ / / / |/| | | | Cirrus: Add [CI:BUILD] magic that only builds