summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Bump github.com/containers/storage from 1.16.2 to 1.16.3dependabot-preview[bot]2020-03-12
| | | | | | | | | | Bumps [github.com/containers/storage](https://github.com/containers/storage) from 1.16.2 to 1.16.3. - [Release notes](https://github.com/containers/storage/releases) - [Changelog](https://github.com/containers/storage/blob/master/docs/containers-storage-changes.md) - [Commits](https://github.com/containers/storage/compare/v1.16.2...v1.16.3) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* Merge pull request #5458 from baude/apiv2eventslibpodOpenShift Merge Robot2020-03-11
|\ | | | | [CI:DOCS]Add libpod event endpoint
| * [CI:DOCS]Add libpod event endpointBrent Baude2020-03-11
| | | | | | | | | | | | add an endpoint for events on the libpod side of the house. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | Merge pull request #5460 from edsantiago/man_page_xrefOpenShift Merge Robot2020-03-11
|\ \ | | | | | | man page cross-reference fixes: part 2
| * | man page cross-reference fixes: part 2Ed Santiago2020-03-11
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The other direction: fix or clean up elements documented in man pages but which did/do not exist in actual podman: * runlabel: add missing "-n" alias for --name And, remove man page entries for nonexistent options: * podman commit: --iidfile * podman container runlabel: --rootfs, --storage * podman create: --cpu-count There are two problems I don't know how to deal with. Both are related to main_local.go:rootCmd.PersistentFlags() : 1) podman-build.1.md documents --cni-config-dir and --runtime options, but these are not actually options under podman build; they are global options. The documentation in this man page differs from that under podman-build. 2) podman ps implements a binary --namespace option, but this option does not (cannot?) appear in --help because there's a global --namespace string option and Cobra somehow gets confused about this. Do we really intend for global options to be parsed on the right-hand side of subcommands? This strikes me as unintuitive and potentially confusing, although the fact that it has taken me this long to discover it suggests that it's not _that_ confusing. Suggestions welcome. I can file issues for 1/2 above, or simply teach my script to special-case ignore them. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | Merge pull request #5372 from sujil02/podstatusOpenShift Merge Robot2020-03-11
|\ \ | | | | | | Update start stop api to use pod status function.
| * | Update start stop api to use pod status function.Sujil022020-03-10
| | | | | | | | | | | | | | | | | | | | | Update the strat and stop api to validate pod status based on status functions rather than iterating over all the containers in the pod. (Addressing the TO-DO) Signed-off-by: Sujil02 <sushah@redhat.com>
* | | Merge pull request #5459 from mheon/bump_readme_181OpenShift Merge Robot2020-03-11
|\ \ \ | | | | | | | | [CI:DOCS] Update version in README to v1.8.1
| * | | Update version in README to v1.8.1Matthew Heon2020-03-11
|/ / / | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | Merge pull request #5457 from mheon/bump-1.8.1OpenShift Merge Robot2020-03-11
|\ \ \ | |_|/ |/| | [CI:DOCS]Bump to v1.8.1
| * | Bump to v1.8.2-devMatthew Heon2020-03-11
| | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | Bump to v1.8.1v1.8.1Matthew Heon2020-03-11
|/ / | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | Merge pull request #5453 from edsantiago/man_page_xrefOpenShift Merge Robot2020-03-10
|\ \ | | | | | | man pages: fix inconsistencies
| * | man pages: fix inconsistenciesEd Santiago2020-03-10
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I wrote a script to cross-reference podman --help against man pages. It found a bunch of inconsistencies fix them: * options missing from man pages * options misspelled or misformatted in man pages (usually misplaced asterisks or missing dashes, but see --dns-opt) * one spurious comma in the actual source file --help This is a fix in which I iterate over 'podman CMD --help' and check for presence in man pages. The other way around (look for flags in man pages, check podman CMD --help) is probably impossible: there are too many special cases Signed-off-by: Ed Santiago <santiago@redhat.com>
* | Merge pull request #5402 from vrothberg/syslogOpenShift Merge Robot2020-03-10
|\ \ | | | | | | docs: clarify bools in `podman --help`
| * | hide --trace flagValentin Rothberg2020-03-10
| | | | | | | | | | | | | | | | | | The tracing flag is not really something users should worry about. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | podman --help: mention defaults of boolsValentin Rothberg2020-03-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | While string variables are shown to expect a string argument, boolean variables are not. This easily tricks users into believing they are simple switches without arguments. Hence, explicitly mention the default to clarify it a bit more. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | docs: clarify that --syslog expects an argumentValentin Rothberg2020-03-10
| | | | | | | | | | | | | | | | | | | | | Clarify in the man page that --syslog expexts an argument to prevent users from believing it's a switch. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #5450 from baude/armOpenShift Merge Robot2020-03-10
|\ \ \ | | | | | | | | build for amd64|arm|ppc64le
| * | | build for amd64|arm|ppc64leBrent Baude2020-03-10
| | | | | | | | | | | | | | | | | | | | | | | | to protect against architecture specific build regressions, we cross compile for multiple architectures Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | Merge pull request #5452 from mheon/release_notes_181OpenShift Merge Robot2020-03-10
|\ \ \ \ | |_|_|/ |/| | | [CI:DOCS] Update release notes for v1.8.1 final release
| * | | Update release notes for v1.8.1 final releaseMatthew Heon2020-03-10
|/ / / | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | Merge pull request #5448 from vrothberg/update-systemdOpenShift Merge Robot2020-03-10
|\ \ \ | |/ / |/| | update systemd & dbus dependencies
| * | update systemd & dbus dependenciesValentin Rothberg2020-03-10
| |/ | | | | | | | | | | | | | | Update the outdated systemd and dbus dependencies which are now provided as go modules. This will further tighten our dependencies and releases and pave the way for the upcoming auto-update feature. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | Merge pull request #5432 from jwhonce/wip/compat_refactorOpenShift Merge Robot2020-03-10
|\ \ | | | | | | Refactor handler packages
| * | Refactor handler packagesJhon Honce2020-03-10
|/ / | | | | | | | | | | | | To help with packaging, the handlers in pkg/api/handlers are now found in pkg/api/handler/compat. Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | Merge pull request #5437 from coandco/runlabel_set_doc_updateOpenShift Merge Robot2020-03-10
|\ \ | | | | | | Remove nonexistent --set arg from runlabel documentation
| * | Remove nonexistent --set arg from runlabel documentationClint Olson2020-03-10
| |/ | | | | | | Signed-off-by: Clint Olson <clint@populi.co>
* | Merge pull request #5353 from cpanato/GH-4414OpenShift Merge Robot2020-03-10
|\ \ | | | | | | commands: rename file and add likns to readthedocs
| * | commands: rename file and add likns to readthedocsCarlos Panato2020-03-09
| | | | | | | | | | | | Signed-off-by: Carlos Panato <ctadeu@gmail.com>
* | | Merge pull request #5435 from mheon/bump-1.8.1-rc4OpenShift Merge Robot2020-03-10
|\ \ \ | |_|/ |/| | [CI:DOCS] Bump to v1.8.1-RC4
| * | Bump to v1.8.1-devMatthew Heon2020-03-09
| | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | Bump to v1.8.1-rc4v1.8.1-rc4Matthew Heon2020-03-09
|/ / | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | Merge pull request #5307 from QiWang19/security-opt-genkubeOpenShift Merge Robot2020-03-09
|\ \ | | | | | | fix security-opt generate kube
| * | fix security-opt generate kubeQi Wang2020-03-03
| | | | | | | | | | | | | | | | | | | | | fix #4950 add selinux options from --security-opt of the container to generate kube result Signed-off-by: Qi Wang <qiwan@redhat.com>
* | | Merge pull request #5426 from ↵OpenShift Merge Robot2020-03-09
|\ \ \ | |_|/ |/| | | | | | | | containers/dependabot/go_modules/github.com/containers/storage-1.16.2 Bump github.com/containers/storage from 1.16.1 to 1.16.2
| * | Bump github.com/containers/storage from 1.16.1 to 1.16.2dependabot-preview[bot]2020-03-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/containers/storage](https://github.com/containers/storage) from 1.16.1 to 1.16.2. - [Release notes](https://github.com/containers/storage/releases) - [Changelog](https://github.com/containers/storage/blob/master/docs/containers-storage-changes.md) - [Commits](https://github.com/containers/storage/compare/v1.16.1...v1.16.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #5429 from mheon/revert_exec_changesMatthew Heon2020-03-09
|\ \ \ | | | | | | | | Revert exec changes
| * | | Revert "exec: get the exit code from sync pipe instead of file"Matthew Heon2020-03-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4b72f9e4013411208751df2a92ab9f322d4da5b2. Continues what began with revert of d3d97a25e8c87cf741b2e24ac01ef84962137106 in previous commit. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | | Revert "Exec: use ErrorConmonRead"Matthew Heon2020-03-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit d3d97a25e8c87cf741b2e24ac01ef84962137106. This does not resolve the issues we expected it would, and has some unexpected side effects with the upcoming exec rework. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | | Revert "exec: fix error code when conmon fails"Matthew Heon2020-03-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4632b81c81a73025a960e339f40bc805f8a6c70a. We are reverting #5373 as well, which lays the foundation for this commit, so it has to go as well. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | Merge pull request #5427 from containers/systemd-defaultMatthew Heon2020-03-09
|\ \ \ \ | |/ / / |/| | | generate systemd: add `default.target` to INSTALL
| * | | rootles tutorial: remove systemd unit exampleValentin Rothberg2020-03-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The example was not entirely correct. Users should use `podman generate systemd` and use the output either directly or as a template for further adjustments to their needs. Keeping an example in the rootless tutorial is a maintenance burdon and can easily suggest incorrect usage patterns to users. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | generate systemd: add `default.target` to INSTALLValentin Rothberg2020-03-09
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When enabling a systemd service we can specify which target will start it by specifying it in the `[INSTALL]` section. In case of root, this is commonly set to `multi-user.target` which is used to start other essential system services such as the network manager, D-BUS and more. However, the `multi-user.target` is not enough on all systems, especially when running rootless and enabling user services. Multiple users have reported issues that there isn't even an attempt to start the service. Setting the INSTALL target to `default.target` will fix the rootless case. However, `default.target` may vary among systems. Fedora Workstation, for instance, sets the `default.target` to the graphical target (i.e., runlevel 5) while Fedora Server sets it to `multi-user.target` which is on runlevel 2 and hence way earlier in the startup sequence. As INSTALL allows for specifying multiple INSTALL targets, we can set it to `multi-user.target` to continue supporting existing workloads AND to `default.target` which MAY redundantly attempt to start it at a later point; effectively a NOP for the root case and essential for rootless. Fixes: #5423 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #5424 from vrothberg/homeOpenShift Merge Robot2020-03-09
|\ \ \ | |/ / |/| | use c/storage instead of docker/docker
| * | use storage/pkg/ioutilsValentin Rothberg2020-03-09
| | | | | | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | use storage/pkg/homedirValentin Rothberg2020-03-09
|/ / | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | Merge pull request #5412 from rhatdan/tmpdirOpenShift Merge Robot2020-03-08
|\ \ | | | | | | Allow users to set TMPDIR environment
| * | Allow users to set TMPDIR environmentDaniel J Walsh2020-03-06
| | | | | | | | | | | | | | | | | | | | | Some users have small /var/tmp directories and need to be able to specify a different location for temporary files, which includes more space. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #5395 from cevich/fix_fedora_minimalOpenShift Merge Robot2020-03-08
|\ \ \ | | | | | | | | Cirrus: Fix fedora-minimal mirroring