summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Cleanup podman spec to not show git checkout is dirtyDaniel J Walsh2018-11-12
| | | | | | | Currently we modify the git checkout which ends up showing that the checkout is dirty. This patch sets the PYTHON_VERSION so that python code will handle it correctly without having to modify the actual code. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* Merge pull request #1803 from QiWang19/issuebuildah1051OpenShift Merge Robot2018-11-12
|\ | | | | Add space between num & unit in images output
| * Add space between num & unit in images outputQi Wang2018-11-12
| | | | | | | | Signed-off-by: Qi Wang <qiwan@redhat.com>
* | Merge pull request #1802 from mheon/troubleshooting_rootlessOpenShift Merge Robot2018-11-12
|\ \ | | | | | | Update troubleshooting guide to deal with rootless path
| * | Update troubleshooting guide to deal with rootless pathMatthew Heon2018-11-12
| | | | | | | | | | | | | | | | | | | | | | | | We had a breaking change to rootless containers' temporary files directories in Podman 0.11.1. Document a workaround to make the change no longer working. Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | | Merge pull request #1779 from giuseppe/troubleshoot-unprivileged-pingOpenShift Merge Robot2018-11-12
|\ \ \ | | | | | | | | troubleshooting.md: add a recipe for rootless ping
| * | | troubleshooting.md: add a recipe for rootless pingGiuseppe Scrivano2018-11-12
| | | | | | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #1794 from jmtd/patch-1OpenShift Merge Robot2018-11-12
|\ \ \ \ | | | | | | | | | | remove $-prefix from (most) shell examples
| * | | | remove $-prefix from (most) shell examplesJonathan Dowland2018-11-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For the shell examples related to building and installing podman, remove the shell prompt indicator character '$'. This makes copying and pasting lines much easier. Retain the prompt indicator for the post-install shell examples, since they (often) mix shell commands and shell command output, so it serves a useful purpose to distinguish them. Signed-off-by: Jonathan Dowland <jdowland@redhat.com>
* | | | | Merge pull request #1799 from xakraz/fixes/doc_containersOpenShift Merge Robot2018-11-12
|\ \ \ \ \ | | | | | | | | | | | | docs: Fix duplicated entry for pod-container-unmount
| * | | | | docs: Fix duplicated entry for pod-container-unmountXavier Krantz2018-11-11
| | |_|_|/ | |/| | | | | | | | | | | | | Signed-off-by: Xavier Krantz <xakraz@gmail.com>
* | | | | Merge pull request #1795 from rhatdan/userOpenShift Merge Robot2018-11-12
|\ \ \ \ \ | |/ / / / |/| | | | Better document rootless containers
| * | | | Better document rootless containersDaniel J Walsh2018-11-10
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Need to return an error pointing user in right direction if rootless podman fails, because of no /etc/subuid or /etc/subgid files. Also fix up man pages to better describe rootless podman. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #1772 from baude/rmdelpausedOpenShift Merge Robot2018-11-09
|\ \ \ \ | | | | | | | | | | rm -f now removes a paused container
| * | | | rm -f now removes a paused containerbaude2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We now can remove a paused container by sending it a kill signal while it is paused. We then unpause the container and it is immediately killed. Also, reworked how the parallelWorker results are handled to provide a more consistent approach to how each subcommand implements it. It also fixes a bug where if one container errors, the error message is duplicated when printed out. Signed-off-by: baude <bbaude@redhat.com>
* | | | | Merge pull request #1789 from mheon/fix_add_hosts_testOpenShift Merge Robot2018-11-09
|\ \ \ \ \ | | | | | | | | | | | | Accurately update state if prepare() partially fails
| * | | | | Accurately update state if prepare() partially failsMatthew Heon2018-11-08
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are seeing some issues where, when part of prepare() fails (originally noticed due to a bad static IP), the other half does not successfully clean up, and the state can be left in a bad place (not knowing about an active SHM mount for example). Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | Merge pull request #1726 from giuseppe/default-to-fuse-overlayfsOpenShift Merge Robot2018-11-09
|\ \ \ \ \ | |_|/ / / |/| | | | rootless: default to fuse-overlayfs when available
| * | | | rootless: default to fuse-overlayfs when availableGiuseppe Scrivano2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If fuse-overlayfs is present, rootless containers default to use it. This can still be overriden either via the command line with --storage-driver or in the ~/.config/containers/storage.conf configuration file. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | | Merge pull request #1793 from vrothberg/rootless-infoOpenShift Merge Robot2018-11-09
|\ \ \ \ \ | | | | | | | | | | | | info: add rootless field
| * | | | | info: add rootless fieldValentin Rothberg2018-11-09
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a rootless field to the info data (e.g., `podman info`) to indicate if the executing user is root or not. In most cases, this can be guessed but now it is clear and may aid in debugging, reporting and understanding certain issues. Signed-off-by: Valentin Rothberg <vrothberg@suse.com>
* | | | | Merge pull request #1786 from vbatts/docsOpenShift Merge Robot2018-11-08
|\ \ \ \ \ | | | | | | | | | | | | podman_tutorial: cni build path has changed
| * | | | | podman_tutorial: cni build path has changedVincent Batts2018-11-08
| | |_|_|/ | |/| | | | | | | | | | | | | Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
* | | | | Merge pull request #1784 from cevich/cirrus_ignore_ircOpenShift Merge Robot2018-11-08
|\ \ \ \ \ | | | | | | | | | | | | Cirrus: Ignore any error from the IRC messenger
| * | | | | Cirrus: Ignore any error from the IRC messengerChris Evich2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | | Merge pull request #1787 from baude/netstatusreworkOpenShift Merge Robot2018-11-08
|\ \ \ \ \ \ | | | | | | | | | | | | | | correct assignment of networkStatus
| * | | | | | correct assignment of networkStatusbaude2018-11-08
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | once we changed configureNetNS to return a result beyond an error, we need to make sure that we used locals instead of ctr attributes when determining networks. Resolves #1752 Signed-off-by: baude <bbaude@redhat.com>
* | | | | | Merge pull request #1790 from isimluk/beware-of-goroutine-localsOpenShift Merge Robot2018-11-08
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Do not hide errors when creating container with UserNSRoot
| * | | | | Do not hide errors when creating container with UserNSRootŠimon Lukašík2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This one is tricky. By using `:=` operator we have made err variable to be local in the gorutine and different from `err` variable in the surrounding function. And thus `createContainer` function returned always nil, even in cases when some error occurred in the gorutine. Signed-off-by: Šimon Lukašík <slukasik@redhat.com>
* | | | | | Merge pull request #1785 from mheon/bump-0.11.1OpenShift Merge Robot2018-11-08
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Bump to v0.11.1
| * | | | | Bump gitvalidation epochMatthew Heon2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | | | | Bump to v0.11.2-devMatthew Heon2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | | | | Bump to v0.11.1v0.11.1Matthew Heon2018-11-08
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | | | | Merge pull request #1781 from mheon/release_notes_0.11.1OpenShift Merge Robot2018-11-08
|\ \ \ \ \ | |_|/ / / |/| | | | Update release notes for 0.11.1
| * | | | Update release notes for 0.11.1Matthew Heon2018-11-08
|/ / / / | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | | | Merge pull request #1778 from vrothberg/update-seccompOpenShift Merge Robot2018-11-08
|\ \ \ \ | |_|/ / |/| | | update seccomp.json
| * | | update seccomp.jsonValentin Rothberg2018-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge the following changes from the upstream Moby seccomp profile: * commit b2a907c8cab6 ("Whitelist statx syscall for libseccomp-2.3.3 onward") * commit 47dfff68e436 ("Whitelist syscalls linked to CAP_SYS_NICE in default seccomp profile") * commit ccd22ffcc8b5 ("Move the syslog syscall to be gated by CAP_SYS_ADMIN or CAP_SYSLOG") Signed-off-by: Valentin Rothberg <vrothberg@suse.com>
* | | | Merge pull request #1773 from TomSweeneyRedHat/dev/tsweeney/logoptfix1OpenShift Merge Robot2018-11-07
|\ \ \ \ | | | | | | | | | | Touch up --log* options and daemons in man pages
| * | | | Touch up --log* options and daemons in man pagesTomSweeneyRedHat2018-11-07
|/ / / / | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | Merge pull request #1774 from mheon/fix_add_hosts_testOpenShift Merge Robot2018-11-07
|\ \ \ \ | | | | | | | | | | Fix run --hostname test that started failing post-merge
| * | | | Fix run --hostname test that started failing post-mergeMatthew Heon2018-11-07
|/ / / / | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | Merge pull request #1764 from rhatdan/nopasswdOpenShift Merge Robot2018-11-07
|\ \ \ \ | | | | | | | | | | Don't fail if /etc/passwd or /etc/group does not exists
| * | | | Don't fail if /etc/passwd or /etc/group does not existsDaniel J Walsh2018-11-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Container images can be created without passwd or group file, currently if one of these containers gets run with a --user flag the container blows up complaining about t a missing /etc/passwd file. We just need to check if the error on read is ENOEXIST then allow the read to return, not fail. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #1771 from baude/prepareOpenShift Merge Robot2018-11-07
|\ \ \ \ \ | | | | | | | | | | | | move defer'd function declaration ahead of prepare error return
| * | | | | move defer'd function declaration ahead of prepare error returnbaude2018-11-07
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | | | | Merge pull request #1689 from mheon/add_runc_timeoutOpenShift Merge Robot2018-11-07
|\ \ \ \ \ | | | | | | | | | | | | Do not call out to runc for sync
| * | | | | Print error status code if we fail to parse itMatthew Heon2018-11-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we read the conmon error status file, if Atoi fails to parse the string we read from the file as an int, print the string as part of the error message so we know what might have gone wrong. Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | | Properly set Running state when starting containersMatthew Heon2018-11-07
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | | | | Fix misspellingMatthew Heon2018-11-07
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | | | | Retrieve container PID from conmonMatthew Heon2018-11-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of running a full sync after starting a container to pick up its PID, grab it from Conmon instead. Signed-off-by: Matthew Heon <matthew.heon@gmail.com>