aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #3883 from baude/varlinkbuildcleanupOpenShift Merge Robot2019-08-26
|\ | | | | clean up after remote build
| * clean up after remote buildbaude2019-08-25
| | | | | | | | | | | | | | | | | | | | when performing an image build over a varlink connection, we should clean up tmp files that are a result of sending the file to the host and untarring it for the build. Fixes: #3869 Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #3824 from baude/varlinkendpointtestOpenShift Merge Robot2019-08-26
|\ \ | | | | | | Create framework for varlink endpoint integration tests
| * | Create framework for varlink endpoint integration testsbaude2019-08-16
| | | | | | | | | | | | | | | | | | | | | add the ability to write integration tests similar to our e2e tests for the varlink endpoints. Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #3755 from mheon/fix_cninameOpenShift Merge Robot2019-08-26
|\ \ \ | |_|/ |/| | Adjust name of Podman CNI network bridge
| * | Adjust name of Podman CNI network bridgeMatthew Heon2019-08-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both Podman and CRI-O set up CNI bridges with the name 'cni0'. If both our CNI conflist and the CRI-O conflist are installed, whoever runs first will win - that is, they will configure the bridge, and everyone will use it. Problem: the CRI-O CNI config conflicts with ours and results in containers with no networking. Solution: rename our bridge so we don't conflict with CRI-O. At the same time, hit our IPAM configuration. The current version was an older format in danger of deprecation. The new format is documented at [1]. Finally, fix indentation for the entire file. [1] https://github.com/containernetworking/plugins/tree/master/plugins/ipam/host-local#example-configurations Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | Merge pull request #3877 from TomSweeneyRedHat/dev/tsweeney/cnifixOpenShift Merge Robot2019-08-25
|\ \ \ | |/ / |/| | Update cni config instructions
| * | Update cni config instructionsTomSweeneyRedHat2019-08-22
|/ / | | | | | | | | | | | | | | | | Update the CNI configuration instructions to line up with the changes introduced in #3868. Also do a bit less documentation of the configuration and point to the GitHub project so we won't get out of sync in the future. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | Merge pull request #3845 from chrahunt/patch-2OpenShift Merge Robot2019-08-22
|\ \ | | | | | | Fix minor typos in podman-run docs.
| * | Fix minor typos in podman-run docs.Christopher Hunt2019-08-21
| | | | | | | | | | | | Signed-off-by: Chris Hunt <chrahunt@gmail.com>
* | | Merge pull request #3844 from chrahunt/patch-1OpenShift Merge Robot2019-08-22
|\ \ \ | | | | | | | | Fix link format in rootless_tutorial.md.
| * | | Fix link format in rootless_tutorial.md.Christopher Hunt2019-08-21
| |/ / | | | | | | | | | | | | | | | From `[link](url)` to [`link`](url) Signed-off-by: Chris Hunt <chrahunt@gmail.com>
* | | Merge pull request #3800 from vrothberg/generate-podOpenShift Merge Robot2019-08-22
|\ \ \ | | | | | | | | generate systemd pod
| * | | generate systemd: support pods and geneartig filesValentin Rothberg2019-08-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support generating systemd unit files for a pod. Podman generates one unit file for the pod including the PID file for the infra container's conmon process and one unit file for each container (excluding the infra container). Note that this change implies refactorings in the `pkg/systemdgen` API. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | generate systemd: drop support for remote clientsValentin Rothberg2019-08-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Drop the support for remote clients to generate systemd-service files. The generated files are machine-dependent and hence relate only to the a local machine. Furthermore, a proper service management when using a remote-client is not possible as systemd has no access to a process. Dropping the support will also reduce the risk of making users believe that the generated services are usable in a remote scenario. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #3873 from rhatdan/migrateOpenShift Merge Robot2019-08-22
|\ \ \ \ | | | | | | | | | | Need to include command name in error message
| * | | | Need to include command name in error messageDaniel J Walsh2019-08-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I hit this error and it told be to system migrate` as opposed to `podman system migrate` Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #3872 from baude/3861OpenShift Merge Robot2019-08-22
|\ \ \ \ \ | | | | | | | | | | | | podman-remote: cp crashes
| * | | | | podman-remote: cp crashesbaude2019-08-21
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | prune unwanted messages when running a container remotely. also, cp is not remote-enabled yet and as such should not be available on the remote client. Fixes: #3861 Signed-off-by: baude <bbaude@redhat.com>
* | | | | Merge pull request #3868 from giuseppe/dockerfile-cni-rpmOpenShift Merge Robot2019-08-22
|\ \ \ \ \ | | | | | | | | | | | | Dockerfile.fedora: install cni plugins package
| * | | | | Dockerfile.fedora: install cni plugins packageGiuseppe Scrivano2019-08-21
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the version available on Fedora is updated, so we don't need to build from source. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #3863 from TomSweeneyRedHat/dev/tsweeney/digestfile2OpenShift Merge Robot2019-08-21
|\ \ \ \ \ | |_|/ / / |/| | | | Add --digestfile option to push
| * | | | Add --digestfile option to pushTomSweeneyRedHat2019-08-21
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the digestfile option to the push command so the digest can be stored away in a file when requested by the user. Also have added a debug statement to show the completion of the push. Emulates Buildah's https://github.com/containers/buildah/pull/1799/files Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | Merge pull request #2940 from giuseppe/drop-firewallOpenShift Merge Robot2019-08-21
|\ \ \ \ | |/ / / |/| | | networking: use firewall plugin
| * | | networking: use firewall pluginGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | drop the pkg/firewall module and start using the firewall CNI plugin. It requires an updated package for CNI plugins. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #3856 from giuseppe/fix-dockerfile-cniOpenShift Merge Robot2019-08-20
|\ \ \ \ | | | | | | | | | | Dockerfile*: fix build for CNI plugins
| * | | | Dockerfile*: fix build for CNI pluginsGiuseppe Scrivano2019-08-20
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #3855 from TomSweeneyRedHat/dev/tsweeney/readmetouchOpenShift Merge Robot2019-08-20
|\ \ \ \ \ | | | | | | | | | | | | Touchup README with Buildah build usage
| * | | | | Touchup README with Buildah build usageTomSweeneyRedHat2019-08-20
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding the verbiage from the Buildah README.MD to Podmans. Original PR by @vrothberg at: https://github.com/containers/buildah/pull/1801/files Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | | Merge pull request #3858 from haircommander/exec-userOpenShift Merge Robot2019-08-20
|\ \ \ \ \ | | | | | | | | | | | | exec: run with user specified on container start
| * | | | | exec: run with user specified on container startPeter Hunt2019-08-20
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before, if the container was run with a specified user that wasn't root, exec would fail because it always set to root unless respecified by user. instead, inherit the user from the container start. Signed-off-by: Peter Hunt <pehunt@redhat.com>
* | | | | Merge pull request #3569 from rhatdan/cgroupsv1OpenShift Merge Robot2019-08-20
|\ \ \ \ \ | |/ / / / |/| | | | Fix error message on podman stats on cgroups v1 rootless environments
| * | | | Fix error message on podman stats on cgroups v1 rootless environmentsDaniel J Walsh2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | podman stats does not work in rootless environments with cgroups V1. Fix error message and document this fact. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #3853 from giuseppe/update-cniOpenShift Merge Robot2019-08-20
|\ \ \ \ \ | | | | | | | | | | | | Dockerfile.*: bump CNI plugins commit
| * | | | | Dockerfile.*: bump CNI plugins commitGiuseppe Scrivano2019-08-19
| | |/ / / | |/| | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #3760 from rhatdan/authOpenShift Merge Robot2019-08-19
|\ \ \ \ \ | | | | | | | | | | | | Use GetRuntimeDir to setup auth.json for login
| * | | | | Use GetRuntimeDir to setup auth.json for loginDaniel J Walsh2019-08-12
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #3852 from edsantiago/batsOpenShift Merge Robot2019-08-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Flake fix: build test timeout
| * | | | | | Flake fix: build test timeoutEd Santiago2019-08-19
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The priv test added to the build test in June runs an 'apk' command which, unavoidably, has to fetch stuff from the net. This is slow and unreliable, and periodically leads to timeout failures. Worse, when this happens, some sort of invisible buildah-only container gets left behind that leads to failures in subsequent tests when trying to reset to known state. Imperfect workaround: try a 240-second timeout (up from 60) when running apk. As backup, add a custom teardown() which attempts to force-remove all containers and any new images. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | | Merge pull request #3476 from ashley-cui/remotehealthcheckOpenShift Merge Robot2019-08-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Implement healthcheck for remote client
| * | | | | | Implement healthcheck for remote clientAshley Cui2019-08-19
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously unimplemented. Works the same way the local one does, except its remote. Signed-off-by: Ashley Cui <ashleycui16@gmail.com>
* | | | | | Merge pull request #3849 from openSUSE/directory-pullOpenShift Merge Robot2019-08-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix directory pull image name for OCI images
| * | | | | | Fix directory pull image name for OCI imagesSascha Grunert2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a breaking change and modifies the resulting image name when pulling from an directory via `oci:...`. Without this patch, the image names pulled via a local directory got processed incorrectly, like this: ``` > podman pull oci:alpine > podman images REPOSITORY TAG IMAGE ID CREATED SIZE localhost/oci alpine 4fa153a82426 5 weeks ago 5.85 MB ``` We now use the same approach as in the corresponding [buildah fix][1] to adapt the behavior for correct `localhost/` prefixing. [1]: https://github.com/containers/buildah/pull/1800 After applying the patch the same OCI image pull looks like this: ``` > ./bin/podman pull oci:alpine > podman images REPOSITORY TAG IMAGE ID CREATED SIZE localhost/alpine latest 4fa153a82426 5 weeks ago 5.85 MB ``` End-to-end tests have been adapted as well to cover the added scenario. Relates to: https://github.com/containers/buildah/issues/1797 Signed-off-by: Sascha Grunert <sgrunert@suse.com>
* | | | | | | Merge pull request #3848 from giuseppe/enable-all-tests-crunOpenShift Merge Robot2019-08-19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | tests: enable all tests for crun
| * | | | | | | test: enable all tests for crunGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | | | | | test: fix return code check for missing workdirGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | | | | | .cirrus.yml: use crun from git masterGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | | | | | libpod, pkg: lookup also for crun failuresGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | | | | | libpod.conf: add crun to runtime_supports_jsonGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | | | | | containers, create: debug message on failed deletionGiuseppe Scrivano2019-08-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if the container failed to be created, don't assume it is still known to the OCI runtime. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>