summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #5038 from mheon/fix_release_scriptOpenShift Merge Robot2020-01-31
|\ | | | | [CI:DOCS] Fix a syntax error in hack/release.sh
| * Fix a syntax error in hack/release.shMatthew Heon2020-01-31
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | Merge pull request #5036 from vrothberg/fix-5034OpenShift Merge Robot2020-01-31
|\ \ | | | | | | sigproxy: return after closing the channel
| * | sigproxy: return after closing the channelValentin Rothberg2020-01-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When stopping signal handling (e.g., to properly handle ^C) we are also closing the signal channel. We should really return from the go-routine instead of continuing and risking double-closing the channel which leads to a panic. Fixes: #5034 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #5030 from baude/apiv2longnameOpenShift Merge Robot2020-01-31
|\ \ \ | |_|/ |/| | fix longname handling for bindings
| * | fix longname handling for bindingsBrent Baude2020-01-31
| |/ | | | | | | | | | | the api needs to account for image input where the image is encoded as a fqd image name. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | Merge pull request #5037 from mheon/release_notes_1.8.0OpenShift Merge Robot2020-01-31
|\ \ | |/ |/| [CI:DOCS] Minor update to release notes
| * Minor update to release notesMatthew Heon2020-01-31
|/ | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* Merge pull request #5032 from mheon/release_notes_1.8.0OpenShift Merge Robot2020-01-30
|\ | | | | Update release notes for v1.8.0
| * Update release notes for v1.8.0Matthew Heon2020-01-30
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | Merge pull request #5028 from baude/commitimageOpenShift Merge Robot2020-01-30
|\ \ | | | | | | make image reference for commit optional
| * | make image reference for commit optionalBrent Baude2020-01-30
| |/ | | | | | | | | | | | | | | to match docker compat, the image tag should be optional. Fixes: #5027 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | Merge pull request #5020 from baude/bindingtestrootlessOpenShift Merge Robot2020-01-30
|\ \ | | | | | | Bindingtestrootless
| * | rootless: enable shortcut only for podmanGiuseppe Scrivano2020-01-29
| | | | | | | | | | | | | | | | | | | | | disable joining automatically the user namespace if the process is not podman. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | test: honor TEMPDIR variableGiuseppe Scrivano2020-01-29
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | Merge pull request #5022 from ssbarnea/fix/validateOpenShift Merge Robot2020-01-30
|\ \ \ | | | | | | | | Assure validate includes lint
| * | | Assure validate includes lintSorin Sbarnea2020-01-30
| | |/ | |/| | | | | | | | | | | | | Fixed issue where lint was not run on CI, so we missed to prevent regressions. Signed-off-by: Sorin Sbarnea <ssbarnea@redhat.com>
* | | Merge pull request #5024 from rpjday/markdown/asterisksOpenShift Merge Robot2020-01-30
|\ \ \ | | | | | | | | markdown: fix erroneous asterisk markup for options
| * | | markdown: fix erroneous asterisk markup for optionsRobert P. J. Day2020-01-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix option markup in a number of man pages so it renders properly when viewing online. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | Merge pull request #5031 from baude/apiv2dockercontainerslashOpenShift Merge Robot2020-01-30
|\ \ \ \ | |_|/ / |/| | | compat container names begin with /
| * | | compat container names begin with /Brent Baude2020-01-30
|/ / / | | | | | | | | | | | | | | | when using the apiv2, containers names are preceeded with a /. fixing this for inspect and listcontainers. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | Merge pull request #5025 from delight/feature/ubuntu_doc_adjustmentOpenShift Merge Robot2020-01-30
|\ \ \ | | | | | | | | Feature/ubuntu doc adjustment
| * | | adjusts install.md (Ubuntu): replaces ${NAME} with hard-encoded Ubuntu to ↵Konstantinos Kostarellis2020-01-30
| | | | | | | | | | | | | | | | | | | | | | | | support all *buntu flavors - combines downloading and registering of the apt-secure key into a piped oneliner
| * | | adjusts install.md (Ubuntu): registries.conf setup is now in ↵Konstantinos Kostarellis2020-01-30
| | |/ | |/| | | | | | | containers-image package
* | | 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.