summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* [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 #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
| * | | Cirrus: Add [CI:BUILD] magic that only buildsChris Evich2022-02-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is intended for cases where no code changes were made in this repo. but something did change which could affect the toolchain. For example, `Makefile` or packaging changes. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | Merge pull request #13105 from lsm5/copr-fix-dependenciesOpenShift Merge Robot2022-02-01
|\ \ \ \ | | | | | | | | | | COPR: fix dependencies
| * | | | COPR: fix dependenciesLokesh Mandvekar2022-02-01
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | containers-common rpm now `Recommends: netavark` and `Provides: container-network-stack` which are actually provided by both cni-plugins and netavark. Netavark has a `Recommends: aardvark-dns` already. So, we should only depend on the containers-common package and let it handle everything. Also, dnsname no longer needs to be recommended if we want new users to use netavark / aardvark-dns. [NO NEW TESTS NEEDED] Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | | | Merge pull request #13066 from rhatdan/manOpenShift Merge Robot2022-02-01
|\ \ \ \ | | | | | | | | | | Clarify remote client means Mac and Windows
| * | | | Clarify remote client means Mac and WindowsDaniel J Walsh2022-01-28
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #13085 from ydayagi/mainOpenShift Merge Robot2022-02-01
|\ \ \ \ \ | |_|_|/ / |/| | | | play kube envVar.valueFrom.fieldRef
| * | | | play kube envVar.valueFrom.fieldRefYaron Dayagi2022-01-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add support for env vars values from pod spec fields 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 #12814 from cevich/netavarkOpenShift Merge Robot2022-02-01
|\ \ \ \ \ | |_|_|/ / |/| | | | Cirrus: Add e2e task w/ upstream netavark
| * | | | Cirrus: Also download aardvark-dns binaryChris Evich2022-01-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This involves a minor code-change so the download/install can run in a loop for the two different repositories and binaries. Given everything is exactly the same except the URLs and names. Signed-off-by: Chris Evich <cevich@redhat.com>
| * | | | Cirrus: Add e2e task w/ upstream netavarkChris Evich2022-01-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR adds the CI mechanisms to obtain the latest upstream netavark binary, and set a magic env-var to indicate e2e tests should execute podman with `--network-driver=netavark`. A future commit implement this functionality within the e2e tests. Due to the way the new environment is enabled, the standard task name is too long for github to display without adding ellipsis. Force the custom task name `Netavark Integration` to workaround this. At some future point, when netavark is more mainstream/widely supported, this custom task and upstream binary install can simply be removed - i.e. netavark will simply be used by default in the normal e2e tasks. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #13101 from giuseppe/report-slirp4netns-statsOpenShift Merge Robot2022-02-01
|\ \ \ \ \ | | | | | | | | | | | | libpod: report slirp4netns network stats
| * | | | | libpod: report slirp4netns network statsGiuseppe Scrivano2022-02-01
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | by default slirp4netns uses the tap0 device. When slirp4netns is used, use that device by default instead of eth0. Closes: https://github.com/containers/podman/issues/11695 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>