summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* io.podman.socket: drop Also=multi-user.targetJonathan Lebon2019-10-08
| | | | | | | | | | | | | | | | | | | | | | Using `Also=` means that the target unit will also be installed/uninstalled together with our unit. Doing `Also=multi-user.target` essentially says: disable `multi-user.target` if `io.podman.socket` is disabled, which sounds... not at all like what we want. In practice, systemd thankfully ignores this (likely because it's the default target). I think having `Also=io.podman.socket` in the `io.podman.service` already does what we want here: it gets installed under `sockets.target` whenever the service is. (And the fact that systemd ignored this means that it wasn't actually playing a role in resolving #3998.) This was causing `systemctl preset-all` to dump core in Fedora CoreOS: https://github.com/coreos/fedora-coreos-tracker/issues/290 (Likely there's a systemd bug around here too.) Signed-off-by: Jonathan Lebon <jonathan@jlebon.com>
* Merge pull request #4032 from rhatdan/pids-limitOpenShift Merge Robot2019-10-07
|\ | | | | Setup a reasonable default for pids-limit 4096
| * Setup a reasonable default for pids-limit 4096Daniel J Walsh2019-10-04
| | | | | | | | | | | | | | | | | | | | | | CRI-O defaults to 1024 for the maximum pids in a container. Podman should have a similar limit. Once we have a containers.conf, we can set the limit in this file, and have it easily customizable. Currently the documentation says that -1 sets pids-limit=max, but -1 fails. This patch allows -1, but also indicates that 0 also sets the max pids limit. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #4196 from mheon/normal_remove_on_evictOpenShift Merge Robot2019-10-07
|\ \ | | | | | | When evicting containers, perform a normal remove first
| * | When evicting containers, perform a normal remove firstMatthew Heon2019-10-04
| | | | | | | | | | | | | | | | | | | | | | | | This ensures that containers that didn't require an evict will be dealt with normally, and we only break out evict for containers that refuse to be removed by normal means. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | Merge pull request #4165 from mtrmac/c-image-4OpenShift Merge Robot2019-10-04
|\ \ \ | |_|/ |/| | Update c/image to v4.0.0 + Buildah to 1.11.3
| * | Update c/image to v4.0.1 and buildah to 1.11.3Miloslav Trmač2019-10-04
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | This requires updating all import paths throughout, and a matching buildah update to interoperate. I can't figure out the reason for go.mod tracking github.com/containers/image v3.0.2+incompatible // indirect ((go mod graph) lists it as a direct dependency of libpod, but (go list -json -m all) lists it as an indirect dependency), but at least looking at the vendor subdirectory, it doesn't seem to be actually used in the built binaries. Signed-off-by: Miloslav Trmač <mitr@redhat.com>
* | Merge pull request #4180 from cevich/update_download_linksOpenShift Merge Robot2019-10-04
|\ \ | | | | | | Docs: Update links, add links to latest
| * | Docs: Update links, add links to latestChris Evich2019-10-03
| | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
| * | Cirrus: Fix log URIs & add optional $ALSO_FILENAMEChris Evich2019-10-03
| | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #4194 from ↵OpenShift Merge Robot2019-10-04
|\ \ \ | | | | | | | | | | | | | | | | containers/dependabot/go_modules/gopkg.in/yaml.v2-2.2.4 Bump gopkg.in/yaml.v2 from 2.2.3 to 2.2.4
| * | | Bump gopkg.in/yaml.v2 from 2.2.3 to 2.2.4dependabot-preview[bot]2019-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [gopkg.in/yaml.v2](https://github.com/go-yaml/yaml) from 2.2.3 to 2.2.4. - [Release notes](https://github.com/go-yaml/yaml/releases) - [Commits](https://github.com/go-yaml/yaml/compare/v2.2.3...v2.2.4) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #4186 from cevich/log_runc_crunOpenShift Merge Robot2019-10-04
|\ \ \ \ | |_|_|/ |/| | | Cirrus: Simplify package NVR logging
| * | | Cirrus: Simplify package NVR loggingChris Evich2019-10-03
| | |/ | |/| | | | | | | | | | | | | Also include runc/crun version for Fedora Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #3549 from marcov/evict-containerOpenShift Merge Robot2019-10-04
|\ \ \ | |_|/ |/| | Add ability to evict a container
| * | rm: add containers eviction with `rm --force`Marco Vedovati2019-09-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | Add ability to evict a container when it becomes unusable. This may happen when the host setup changes after a container creation, making it impossible for that container to be used or removed. Evicting a container is done using the `rm --force` command. Signed-off-by: Marco Vedovati <mvedovati@suse.com>
* | | Merge pull request #4188 from Mrigank11/validate_network_nameOpenShift Merge Robot2019-10-04
|\ \ \ | | | | | | | | podman network create: validate user input
| * | | podman network create: validate user inputMrigank Krishan2019-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Disallow invalid/confusing names such as '../bar' or 'foo ' Closes #4184 Signed-off-by: Mrigank Krishan <mrigankkrishan@gmail.com>
* | | | Merge pull request #4161 from afbjorklund/machine-envOpenShift Merge Robot2019-10-03
|\ \ \ \ | |/ / / |/| | | Handle environment variables from podman-machine
| * | | Apply changes also to the windows implementationAnders F Björklund2019-10-02
| | | | | | | | | | | | | | | | Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
| * | | Allow setting default parameters with env varsAnders F Björklund2019-10-01
| | | | | | | | | | | | | | | | Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
| * | | Avoid hard-coding path to varlink and podmanAnders F Björklund2019-10-01
| | | | | | | | | | | | | | | | Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
| * | | Allow changing IdentityFile and to IgnoreHostsAnders F Björklund2019-10-01
| | | | | | | | | | | | | | | | Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
* | | | Merge pull request #4174 from cevich/use_bash_not_shOpenShift Merge Robot2019-10-03
|\ \ \ \ | | | | | | | | | | System-tests: Use bash explicitly
| * | | | System-tests: Use bash explicitlyChris Evich2019-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Ubuntu, /bin/sh != /bin/bash. Update system-tests to only use bash for testing consistency across platforms. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #4173 from mheon/bump_release_in_readmeOpenShift Merge Robot2019-10-03
|\ \ \ \ \ | | | | | | | | | | | | Podman 1.6.0 has been released, update the README
| * | | | | Podman 1.6.0 has been released, update the READMEMatthew Heon2019-10-02
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | | | Merge pull request #4163 from ↵OpenShift Merge Robot2019-10-03
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/gopkg.in/yaml.v2-2.2.3 Bump gopkg.in/yaml.v2 from 2.2.2 to 2.2.3
| * | | | | | Bump gopkg.in/yaml.v2 from 2.2.2 to 2.2.3dependabot-preview[bot]2019-10-02
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [gopkg.in/yaml.v2](https://github.com/go-yaml/yaml) from 2.2.2 to 2.2.3. - [Release notes](https://github.com/go-yaml/yaml/releases) - [Commits](https://github.com/go-yaml/yaml/compare/v2.2.2...v2.2.3) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Matthew Heon <mheon@redhat.com> Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | Merge pull request #4179 from edsantiago/batsOpenShift Merge Robot2019-10-03
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | system tests: info: deal with hyphen in username
| * | | | | system tests: info: deal with hyphen in usernameEd Santiago2019-10-03
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ...e.g. cloud-user. 9822f54ac was intended to fix this, but it doesn't. Simple and standard solution is to move the dash to the end of the character class. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | Merge pull request #4169 from TomSweeneyRedHat/dev/tsweeney/tutreadfix2OpenShift Merge Robot2019-10-03
|\ \ \ \ \ | | | | | | | | | | | | Add api link to tutorials
| * | | | | Add api link to tutorialsTomSweeneyRedHat2019-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We recently moved the "How to use libpod for custom/derivative projects" page to the docs/tutorials directory. This adds a link to the README.md there so it can be more easily found and adds a logo to the tutorial itself. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | | | Merge pull request #4177 from mheon/bump-1.6.1OpenShift Merge Robot2019-10-02
|\ \ \ \ \ \ | | | | | | | | | | | | | | Bump to v1.6.1
| * | | | | | Bump gitvalidation epochMatthew Heon2019-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | | | Bump to v1.6.2-devMatthew Heon2019-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | | | Bump to v1.6.1v1.6.1Matthew Heon2019-10-02
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | | Merge pull request #4176 from mheon/release_notes_1.6.1OpenShift Merge Robot2019-10-02
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Update release notes for v1.6.1
| * | | | | Update release notes for v1.6.1Matthew Heon2019-10-02
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | | Merge pull request #4170 from mheon/bump-1.6.1-rc1OpenShift Merge Robot2019-10-02
|\ \ \ \ \ | |_|_|/ / |/| | | | Bump to v1.6.1-RC1
| * | | | Bump gitvalidation epochMatthew Heon2019-10-02
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | Bump to v1.6.1-devMatthew Heon2019-10-02
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | Bump to v1.6.1-rc1v1.6.1-rc1Matthew Heon2019-10-02
| |/ / / | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | Merge pull request #4171 from giuseppe/allow-cgroupfs-on-cgroupOpenShift Merge Robot2019-10-02
|\ \ \ \ | | | | | | | | | | rootless: allow cgroupfs manager on cgroups v2
| * | | | rootless: allow cgroupfs manager on cgroups v2Giuseppe Scrivano2019-10-02
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | if there are no resources specified, make sure the OCI resources block is empty so that the OCI runtime won't complain. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #4168 from edsantiago/batsOpenShift Merge Robot2019-10-02
|\ \ \ \ | |/ / / |/| | | system tests: reenable skipped tests
| * | | system tests: reenable skipped testsEd Santiago2019-10-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Issue #3829 (cp symlinks) has been fixed: enable tests for it And, it looks like podman-remote is now handling exit status of a force-rm'ed container. Enable that test too. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #4150 from ↵OpenShift Merge Robot2019-10-02
|\ \ \ \ | |/ / / |/| | | | | | | | | | | containers/dependabot/go_modules/github.com/onsi/gomega-1.7.0 Bump github.com/onsi/gomega from 1.5.0 to 1.7.0
| * | | Bump github.com/onsi/gomega from 1.5.0 to 1.7.0dependabot-preview[bot]2019-10-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.5.0 to 1.7.0. - [Release notes](https://github.com/onsi/gomega/releases) - [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/gomega/compare/v1.5.0...v1.7.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #4166 from giuseppe/set-dbus-session-if-not-setOpenShift Merge Robot2019-10-02
|\ \ \ \ | | | | | | | | | | rootless: set DBUS_SESSION_BUS_ADDRESS if it is not set