summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #5017 from vrothberg/fix-4829OpenShift Merge Robot2020-01-30
|\ | | | | Makefile cleanups
| * speed up MakefileValentin Rothberg2020-01-30
| | | | | | | | | | | | | | | | | | | | Speed up the Makefile by removing variable references from the release targets. Now, the variables will only be (lazily) evaluated when they are actually needed and not for each invocation of the Makefile which has it down considerably. Fixes: #4829 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * Makefile: systemd: echo instead of warnValentin Rothberg2020-01-30
| | | | | | | | | | | | Just echo the message instead of warning to not impact the exit code. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * Makefile: remove redundant BUILDFLAGSValentin Rothberg2020-01-30
| | | | | | | | | | | | We don't set it, so there's no need to keep it. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * Makefile: consistent PHONY useValentin Rothberg2020-01-30
| | | | | | | | | | | | | | Add a .PHONY line over each target instead of mixing this notation with a separate but incomplete single list. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * Makefile: remove dead vagrant targetValentin Rothberg2020-01-30
| | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * Makefile: move systemd buildtag checkValentin Rothberg2020-01-30
| | | | | | | | | | | | | | | | Move the systemd-buildtag check into the `bin/podman` target. No need to execute the check for all invocations of the Makefile. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | Merge pull request #4951 from rpjday/docs/unshareOpenShift Merge Robot2020-01-30
|\ \ | | | | | | markdown: fix grammar/formatting, standardize on markdown
| * | markdown: fix grammar/formatting, standardize on markdownRobert P. J. Day2020-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While fixing grammar and list formatting issues, standardize on markdown as follows: - commands are marked by '**' - files are marked by backquotes - list items are marked with leading '-' Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | Merge pull request #5019 from cevich/fix_gate_epochOpenShift Merge Robot2020-01-30
|\ \ \ | |_|/ |/| | Cirrus: Set EPOCH_TEST_COMMIT during gate task
| * | Cirrus: Set EPOCH_TEST_COMMIT during gate taskChris Evich2020-01-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For whatever reason, this required variable is not set when `make` is doing validation within automation. Fix this by establishing the value based on data provided from the CI system. Since this data is specific to automation executing against a PR, limit this specific job to not run during testing of branches (post-merge). This is safe since we can assume the PR would not have been merged, if the basics checked during gating did not pass. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #5018 from mheon/new_anon_fieldOpenShift Merge Robot2020-01-29
|\ \ \ | | | | | | | | Deprecate & remove IsCtrSpecific in favor of IsAnon
| * | | Deprecate & remove IsCtrSpecific in favor of IsAnonMatthew Heon2020-01-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Podman 1.6.3, we added support for anonymous volumes - fixing our old, broken support for named volumes that were created with containers. Unfortunately, this reused the database field we used for the old implementation, and toggled volume removal on for `podman run --rm` - so now, we were removing *named* volumes created with older versions of Podman. We can't modify these old volumes in the DB, so the next-safest thing to do is swap to a new field to indicate volumes should be removed. Problem: Volumes created with 1.6.3 and up until this lands, even anonymous volumes, will not be removed. However, this is safer than removing too many volumes, as we were doing before. Fixes #5009 Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | Merge pull request #5015 from baude/bindingtestfixesOpenShift Merge Robot2020-01-29
|\ \ \ \ | |/ / / |/| | | apiv2 binding test fixes
| * | | apiv2 binding test fixesBrent Baude2020-01-29
| | | | | | | | | | | | | | | | | | | | | | | | a recent refactor in the bindings broke the tests. quick fixes to get them working again. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | Merge pull request #5013 from vrothberg/fix-4916OpenShift Merge Robot2020-01-29
|\ \ \ \ | | | | | | | | | | history: fix size computing
| * | | | history: fix size computingValentin Rothberg2020-01-29
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Get the layer's size whether it relates to the first history entry or not. This fixes issues where the first entry would always be shown to be of size 0. Fixes: #4916 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #5011 from vrothberg/fix-4814OpenShift Merge Robot2020-01-29
|\ \ \ \ | |/ / / |/| | | run `varlink_generate` on Linux only
| * | | run `varlink_generate` on Linux onlyValentin Rothberg2020-01-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Running the `varlink_generate` make target on non-Linux machines is not supported, so restrict it to Linux only. Fixes: #4814 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #5008 from baude/badconfOpenShift Merge Robot2020-01-28
|\ \ \ \ | | | | | | | | | | display file name of bad cni conf
| * | | | display file name of bad cni confBrent Baude2020-01-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if one of the cni conf files is badly formatted or cannot be loaded, we now display the error as well as the filename. Fixes: #2909 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | Merge pull request #4973 from rhatdan/sortOpenShift Merge Robot2020-01-28
|\ \ \ \ \ | |_|_|/ / |/| | | | Throw error on invalid sort value
| * | | | Throw error on invalid sort valueDaniel J Walsh2020-01-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We define the valid sort values, so we should throw an error on invalid sort values. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #5007 from baude/disableloginOpenShift Merge Robot2020-01-28
|\ \ \ \ \ | |/ / / / |/| | | | rootless login/logout tests fail
| * | | | rootless login/logout tests failBrent Baude2020-01-28
| |/ / / | | | | | | | | | | | | | | | | | | | | On F31 CI tests, we have uncovered several failing tests as rootless that need to be fixed. For the interim, we are going to disable those tests. Issue #5006 has been created to track and complete this. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | Merge pull request #4997 from jwhonce/issues/4364OpenShift Merge Robot2020-01-28
|\ \ \ \ | |/ / / |/| | | Update remote client bridge documentation.
| * | | Update remote client bridge documentation.Jhon Honce2020-01-28
| | | | | | | | | | | | | | | | | | | | | | | | * Add PODMAN_VARLINK_BRIDGE examples. Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | Merge pull request #4993 from baude/playkubepullOpenShift Merge Robot2020-01-28
|\ \ \ \ | | | | | | | | | | honor pull policy in play kube
| * | | | honor pull policy in play kubeBrent Baude2020-01-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a container specification has a pull policy, we should honor it when recreating the pods/containers from yaml. furthermore, ini kube, if a tag is :latest, then the always pull policy is automatically instituted. Fixes: #4880 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | Merge pull request #4984 from baude/pullarchoverrideOpenShift Merge Robot2020-01-28
|\ \ \ \ \ | | | | | | | | | | | | expose --arch-override option for pull
| * | | | | expose --arch-override option for pullBrent Baude2020-01-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We no longer wish to hide the --arch-override from the cli on pulls. we now expose it. docs updated. tests already exist. Fixes: #4849 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | | Merge pull request #5005 from rpjday/docs/tildes_HOMEOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ | | | | | | | | | | | | | | docs: replace '~' with $HOME in markdown as '~' isn't rendered properly
| * | | | | | docs: replace '~' with $HOME in markdown as '~' isn't rendered properlyRobert P. J. Day2020-01-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Apparently, a tilde is currently rendered as a single space, making a mess of command examples that use it, so use $HOME instead. https://github.com/mattermost/mattermost-server/issues/8228 Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | | | | Merge pull request #4989 from ↵OpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/opencontainers/selinux-1.3.1 build(deps): bump github.com/opencontainers/selinux from 1.3.0 to 1.3.1
| * | | | | | | build(deps): bump github.com/opencontainers/selinux from 1.3.0 to 1.3.1dependabot-preview[bot]2020-01-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/opencontainers/selinux](https://github.com/opencontainers/selinux) from 1.3.0 to 1.3.1. - [Release notes](https://github.com/opencontainers/selinux/releases) - [Commits](https://github.com/opencontainers/selinux/compare/1.3.0...v1.3.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | | | Merge pull request #4964 from openSUSE/apparmor-signalsOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | apparmor: allow receiving of signals from 'podman kill'
| * | | | | | | apparmor: allow receiving of signals from 'podman kill'Sascha Grunert2020-01-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In newer kernels, AppArmor will reject attempts to send signals to a container because the signal originated from outside of that AppArmor profile. Correct this by allowing all unconfined signals to be received. Signed-off-by: Sascha Grunert <sgrunert@suse.com>
* | | | | | | | Merge pull request #4988 from ↵OpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/rootless-containers/rootlesskit-0.7.2 build(deps): bump github.com/rootless-containers/rootlesskit from 0.7.1 to 0.7.2
| * | | | | | | | build(deps): bump github.com/rootless-containers/rootlesskitdependabot-preview[bot]2020-01-28
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/rootless-containers/rootlesskit](https://github.com/rootless-containers/rootlesskit) from 0.7.1 to 0.7.2. - [Release notes](https://github.com/rootless-containers/rootlesskit/releases) - [Commits](https://github.com/rootless-containers/rootlesskit/compare/v0.7.1...v0.7.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | | | Merge pull request #5003 from lsm5/debian-doc-updateOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | install.md: registries.conf setup in containers-image package
| * | | | | | | | install.md: registries.conf setup in containers-image packageLokesh Mandvekar2020-01-28
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No need for separate setup steps with podman 1.7.0~5. Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | | | | | | | Merge pull request #4980 from baude/bindingdocsOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | [CI:DOCS]Binding overhauls
| * | | | | | | [CI:DOCS]Binding overhaulsBrent Baude2020-01-28
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add binding for networks and begin documentation for binding methods for godoc. Also, add major functions to their own subpackages so reduce the amount of of method confusion. So instead of: bindings.ListImages(), we now do a [bindings].images.List(). Also, the connection is passed to each binding method via a context to allow for future growth. Lastly, add first set of tests. There are a couple of things to work out for rootless tests yet. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | | | Merge pull request #4999 from rpjday/docs/network_rstOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | docs: fix incomplete heading underlining in network.rst
| * | | | | | | docs: fix incomplete heading underlining in network.rstRobert P. J. Day2020-01-28
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lack of proper underlining generates processing diagnostic. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | | | | Merge pull request #4996 from baude/rootlessnetnsOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [CI:DOCS]rootless exec cannot join root namespace
| * | | | | | | [CI:DOCS]rootless exec cannot join root namespaceBrent Baude2020-01-27
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add a quick note in the rootless.md about usage of --net=host as rootless and subsequent podman execs. Fixes: #4473 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | | | Merge pull request #4995 from rpjday/docs/podman_stopOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | docs: add missing hyphen for '-t' option, command '$' prompts
| * | | | | | | docs: add missing hyphen for '-t' option, command '$' promptsRobert P. J. Day2020-01-28
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | | | | Merge pull request #4974 from rhatdan/manOpenShift Merge Robot2020-01-28
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Cleanup man pages exit code descriptions