aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | Merge pull request #1989 from baude/deletecontainerfailstartOpenShift Merge Robot2018-12-13
|\ \ \ \ \ | | | | | | | | | | | | failed containers with --rm should remove themselves
| * | | | | failed containers with --rm should remove themselvesbaude2018-12-12
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when starting or running a container that has --rm, if the starting container fails (like due to an invalid command), the container should get removed. Resolves: #1985 Signed-off-by: baude <bbaude@redhat.com>
* | | | | Merge pull request #1990 from kunalkushwaha/bugfix-NamedVolumeCheckOpenShift Merge Robot2018-12-13
|\ \ \ \ \ | | | | | | | | | | | | condition fixed for adding volume to boltdb.
| * | | | | condition fixed for adding volume to boltdb.Kunal Kushwaha2018-12-13
| | |_|/ / | |/| | | | | | | | | | | | | Signed-off-by: Kunal Kushwaha <kushwaha_kunal_v7@lab.ntt.co.jp>
* | | | | Merge pull request #1982 from cevich/ci_docs_linkOpenShift Merge Robot2018-12-13
|\ \ \ \ \ | | | | | | | | | | | | Cirrus: Add/fix documentation links
| * | | | | Fix documentation links and flowChris Evich2018-12-12
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #1987 from EmilienM/systemd/ciOpenShift Merge Robot2018-12-12
|\ \ \ \ \ | | | | | | | | | | | | e2e: add tests for systemd
| * | | | | e2e: add tests for systemdEmilien Macchi2018-12-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add functional tests to start a container from systemd. This patch will: - create a systemd unit file to start redis container - create the container with `podman create` - enable the service - start the container with systemd - check that the service is actually running Signed-off-by: Emilien Macchi <emilien@redhat.com>
* | | | | | Merge pull request #2001 from mheon/fix_make_shellOpenShift Merge Robot2018-12-12
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Makefile tweaks to fix make shell
| * | | | | Makefile tweaks to fix make shellMatthew Heon2018-12-12
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, bump the Dockerfile to use the latest Golang image, as most of our testing is now done on 1.11 Fixes: #1999 Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | | | | Merge pull request #1962 from rhatdan/criuOpenShift Merge Robot2018-12-12
|\ \ \ \ \ | |/ / / / |/| | | | Set Socket label for contianer
| * | | | Set Socket label for contianerDaniel J Walsh2018-12-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow container processes to write to the CRIU socket that gets injected into the container. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #1983 from baude/kubespellOpenShift Merge Robot2018-12-11
|\ \ \ \ \ | |_|_|/ / |/| | | | fix typo in kubernetes
| * | | | fix typo in kubernetesbaude2018-12-11
| | |/ / | |/| | | | | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | | | Merge pull request #1978 from mheon/only_one_execuserOpenShift Merge Robot2018-12-11
|\ \ \ \ | |/ / / |/| | | Prevent a second lookup of user for image volumes
| * | | Prevent a second lookup of user for image volumesMatthew Heon2018-12-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of forcing another user lookup when mounting image volumes, just use the information we looked up when we started generating the spec. This may resolve #1817 Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | Merge pull request #1975 from giuseppe/fix-rootless-restartOpenShift Merge Robot2018-12-11
|\ \ \ \ | | | | | | | | | | rootless: fix restart when using fuse-overlayfs
| * | | | rootless: fix restart when using fuse-overlayfsGiuseppe Scrivano2018-12-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With rootless containers we cannot really restart an existing container as we would need to join the mount namespace as well to be able to reuse the storage, so ensure the container is stopped first. Closes: https://github.com/containers/libpod/issues/1965 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #1977 from vdemeester/no--i-go-buildOpenShift Merge Robot2018-12-11
|\ \ \ \ \ | | | | | | | | | | | | No need to use `-i` in go build (with go 1.10 and above)
| * | | | | No need to use `-i` in go build (with go 1.10 and above)Vincent Demeester2018-12-11
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | > The go build command now maintains a cache of recently built packages, separate from the installed packages in $GOROOT/pkg or $GOPATH/pkg. The effect of the cache should be to speed builds that do not explicitly install packages or when switching between different copies of source code (for example, when changing back and forth between different branches in a version control system). The old advice to add the -i flag for speed, as in go build -i or go test -i, is no longer necessary: builds run just as fast without -i. This should also fix podman builds for NixOS, snap-installed go, … Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
* | | | | Merge pull request #1966 from mheon/ensure_storage_opts_initOpenShift Merge Robot2018-12-11
|\ \ \ \ \ | | | | | | | | | | | | Ensure storage options are properly initialized
| * | | | | Ensure storage options are properly initializedMatthew Heon2018-12-10
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If one of storage GraphRoot or RunRoot are specified, but the other is not, c/storage will not use the default, and will throw an error instead. Ensure that in cases where this would happen, we populate the fields with the c/storage defaults ourselves. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | Merge pull request #1970 from baude/kubecapaOpenShift Merge Robot2018-12-11
|\ \ \ \ \ | | | | | | | | | | | | Add capabilities to generate kube
| * | | | | Add capabilities to generate kubebaude2018-12-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using the default capabilities, we can determine which caps were added and dropped. Now added them to the security context structure. Signed-off-by: baude <bbaude@redhat.com>
* | | | | | Merge pull request #1971 from cevich/base_image_docsOpenShift Merge Robot2018-12-11
|\ \ \ \ \ \ | | | | | | | | | | | | | | Cirrus: Update base-image build docs
| * | | | | | Cirrus: Update base-image build docsChris Evich2018-12-10
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mostly minor tweaks and clarifications. Though there was one missing (required) make value I fixed. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | | Merge pull request #1857 from mtrmac/blob-info-cachingOpenShift Merge Robot2018-12-11
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Vendor c/image + Buildah after merging blob-info-caching in both
| * | | | | Update for API changeMiloslav Trmač2018-12-08
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Miloslav Trmač <mitr@redhat.com>
| * | | | | Vendor buildah after merging mtrmac/blob-info-caching-on-top-of-contents-cachingMiloslav Trmač2018-12-08
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Miloslav Trmač <mitr@redhat.com>
| * | | | | Vendor c/image after merging c/image#536Miloslav Trmač2018-12-08
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | ... which adds blob info caching Signed-off-by: Miloslav Trmač <mitr@redhat.com>
* | | | | Merge pull request #1968 from baude/disablepaprf29OpenShift Merge Robot2018-12-10
|\ \ \ \ \ | |_|/ / / |/| | | | disable F29 tests on PAPR
| * | | | disable F29 tests on PAPRbaude2018-12-10
|/ / / / | | | | | | | | | | | | | | | | | | | | We now test F29 Cloud in cirrus Signed-off-by: baude <bbaude@redhat.com>
* | | | Merge pull request #1958 from baude/morevarlinkdocsOpenShift Merge Robot2018-12-10
|\ \ \ \ | |/ / / |/| | | add more example usage to varlink endpoints
| * | | add more example usage to varlink endpointsbaude2018-12-09
| |/ / | | | | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #1957 from mheon/bump-0.12.1.1Matthew Heon2018-12-07
|\ \ \ | | | | | | | | Bump to v0.12.1.1
| * | | Bump gitvalidation epochMatthew Heon2018-12-07
| | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | | Bump to v0.12.2-devMatthew Heon2018-12-07
| | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | | Bump to v0.12.1.1v0.12.1.1Matthew Heon2018-12-07
|/ / / | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | | Merge pull request #1956 from mheon/release_notes_0.12.1.1Matthew Heon2018-12-07
|\ \ \ | |/ / |/| | Update release notes for v0.12.1.1
| * | Update release notes for v0.12.1.1Matthew Heon2018-12-07
|/ / | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | Merge pull request #1955 from mheon/fix_hooks_not_existOpenShift Merge Robot2018-12-07
|\ \ | | | | | | Fix errors where OCI hooks directory does not exist
| * | Fix errors where OCI hooks directory does not existMatthew Heon2018-12-07
| | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | Merge pull request #1953 from baude/podstoptimeoutOpenShift Merge Robot2018-12-07
|\ \ \ | | | | | | | | add timeout to pod stop
| * | | add timeout to pod stopbaude2018-12-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | like podman stop of containers, we should allow the user to specify a timeout override when stopping pods; otherwise they have to wait the full timeout time specified during the pod/container creation. Signed-off-by: baude <bbaude@redhat.com>
* | | | Merge pull request #1893 from jwhonce/bug/1869OpenShift Merge Robot2018-12-07
|\ \ \ \ | | | | | | | | | | Refactor CLI booleans to be consistent and defined behavior
| * | | | Refactor BooleanAction to mimic golang interfaceJhon Honce2018-12-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Change all store_true or store_false to use store_bool. New behavior documented in BooleanAction docstring. * Remove any extraneous code identified by pylint in files from above. Fixes #1869 Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | | Merge pull request #1928 from baude/podtokubeOpenShift Merge Robot2018-12-07
|\ \ \ \ \ | | | | | | | | | | | | generate kube
| * | | | | generate kubebaude2018-12-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add the ability to generate kubernetes pod and service yaml representations of libpod containers and pods. Signed-off-by: baude <bbaude@redhat.com>
* | | | | | Merge pull request #1788 from cevich/cirrus_base_imagesOpenShift Merge Robot2018-12-07
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Codify and document base-image production + Enable testing with Fedora
| * | | | | Cirrus: Document and codify base-image productionChris Evich2018-12-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A number of images required for future testing are not present in GCE. Importing them is a long proscribed process prone to errors and complications. Improve this situation by documenting, and encoding the majority of the steps required. Due to the required complexity, these are clearly identified as 'semi-automated'. This means a discerning eye is sometimes needed to address unforeseen problems (networking issues, format or packaging changes, etc). Nevertheless, having these steps in writing, will reduce current and future maintenance burden while supporting future testing needs of RHEL, Fedora and Fedora Atomic Host. Also: * Add necessary configuration, scripts, and Makefile updates needed to prepare RHEL, Fedora, & FAH cloud images for use in GCE. This is a complex, multi-step process where the cloud image is booted un a local user-mod qemu-kvm instance, where it can be modified. From there, it's converted into a specific format, and imported into GCE. Lastly, the imported raw disk data is made available as a GCE VM image. Note: As of this commit, the RHEL base-image builds (CentOS has native image), however neither RHEL or CentOS cache-images build correctly. * Left testing on FAH disabled, the GCE/Cirrus integration needs needs more work. Specifically, the python3-based google startup script service throws a permission-denied (as root) when trying to create a temp. directory. Did not investigate further, though manually running the startup script does allow the libpod tests to start running. * Enabled Fedora 29 image to execute tests and general use. * Utilize the standardized F28-based container image for gating of more the intensive unit and integration testing. Update documentation to reflect this as the standard platform for these checks. Rename tasks with shorter names and to better reflect their purpose. * Cirrus: Trim unnecessary env vars before testing since the vast majority are only required for orchestration purposes. Since most are defined within `.cirrus.yml`, it's a good place to store the list of undesirables. Since each of the cirrus-scripts runs in it's own shell, unsetting these near the end will have no consequence. Also trim down the number of calls to show_env_vars() Signed-off-by: Chris Evich <cevich@redhat.com>