summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #4912 from jwhonce/wip/swaggerOpenShift Merge Robot2020-01-22
|\ | | | | [CI:DOCS] Update build images
| * Update build imagesJhon Honce2020-01-21
| | | | | | | | | | | | | | | | | | | | | | | | | | * Add swagger annotations for all the query and response parameters for buildimages * Improve populating the BuildOptions struct * Improve swagger.json generation, removing tags.xml and move tag definiation into the swagger:meta block * Update Makefile to be more robust, added target for validation * TODO once validation passes add that step to the generation step Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | Merge pull request #4861 from giuseppe/add-cgroups-disabled-conmonOpenShift Merge Robot2020-01-22
|\ \ | | | | | | oci_conmon: do not create a cgroup under systemd
| * | oci_conmon: do not create a cgroup under systemdGiuseppe Scrivano2020-01-16
| | | | | | | | | | | | | | | | | | | | | | | | Detect whether we are running under systemd (if the INVOCATION_ID is set). If Podman is running under a systemd service, we do not need to create a cgroup for conmon. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | systemdgen: specify --cgroups=disabled-conmon for --newGiuseppe Scrivano2020-01-16
| | | | | | | | | | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | podman: add new option --cgroups=no-conmonGiuseppe Scrivano2020-01-16
| | | | | | | | | | | | | | | | | | | | | | | | it allows to disable cgroups creation only for the conmon process. A new cgroup is created for the container payload. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | Merge pull request #4928 from mheon/release_notes_171OpenShift Merge Robot2020-01-22
|\ \ \ | | | | | | | | Update RELEASE_NOTES for v1.7.1
| * | | Update RELEASE_NOTES for v1.7.1Matthew Heon2020-01-22
| | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | Merge pull request #4931 from rpjday/docs/rmi/backquotesOpenShift Merge Robot2020-01-22
|\ \ \ \ | | | | | | | | | | markdown: remove extraneous backquote from "podman rmi"
| * | | | markdown: remove extraneous backquote from "podman rmi"Robert P. J. Day2020-01-22
| |/ / / | | | | | | | | | | | | | | | | | | | | Extraneous backquote messes up rendering of "man podman-rmi". Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | Merge pull request #4911 from giuseppe/rootless-set-containers-envOpenShift Merge Robot2020-01-22
|\ \ \ \ | |/ / / |/| | | rootless: set C variables also on shortcut
| * | | rootless: set C variables also on shortcutGiuseppe Scrivano2020-01-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | make sure the rootless env variables are set also when we are joining directly the user+mount namespace without creating a new process. It is required by pkg/unshare in containers/common. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #4919 from vrothberg/fix-modOpenShift Merge Robot2020-01-22
|\ \ \ \ | | | | | | | | | | go.mod: fix parse error
| * | | | go.mod: fix parse errorValentin Rothberg2020-01-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the go.mod parsing error caused by updating openshift/api. Fixes: #4728 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | Merge pull request #4899 from baude/apiv2serviceOpenShift Merge Robot2020-01-22
|\ \ \ \ \ | | | | | | | | | | | | Apiv2service
| * | | | | Add service endpointbaude2020-01-21
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add service endpoint for the new API. Also supports the varlink implementation. Signed-off-by: baude <bbaude@redhat.com> Refactor to allow developer more control of API server * Add api.NewServerWithSettings() to create an API server with custom settings * Add api.ListenUnix() to create a UDS net.Listener and setup UDS Signed-off-by: Jhon Honce <jhonce@redhat.com> Signed-off-by: baude <bbaude@redhat.com> More service completion Add podman service command that allows users to run either a RESTful or varlink protocol API service. Addition of docs and RESTful listening. Signed-off-by: baude <bbaude@redhat.com> Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | Merge pull request #4925 from cevich/fix_docs_no_snapOpenShift Merge Robot2020-01-21
|\ \ \ \ \ | | | | | | | | | | | | [CI:DOCS] Cirrus: Fix logic typo
| * | | | | Cirrus: Fix logic typoChris Evich2020-01-21
| | |_|_|/ | |/| | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #4915 from baude/reviewcorrectionsOpenShift Merge Robot2020-01-21
|\ \ \ \ \ | |/ / / / |/| | | | [CI:DOCS]First pass at review comments
| * | | | [CI:DOCS]First pass at review commentsbaude2020-01-21
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Tackling the first comments in the review pass. More to come. Signed-off-by: baude <bbaude@redhat.com> Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | Merge pull request #4923 from cevich/docs_no_snapOpenShift Merge Robot2020-01-21
|\ \ \ \ | |/ / / |/| | | [CI:DOCS] Cirrus: No upload snap for docs job
| * | | Cirrus: No upload snap for docs jobChris Evich2020-01-21
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Previously, the upload-snap job was running during post-merge testing when the magic ci-docs string was present in the PR description. This fixes that and makes all the `only_if`'s more consistent. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #4914 from baude/apidevelOpenShift Merge Robot2020-01-21
|\ \ \ | |/ / |/| | [CI:DOCS] Add logo and dev statement
| * | Disable go mods on varlink builds in specBrent Baude2020-01-21
| | | | | | | | | | | | | | | | | | Fixes test images test failure. Signed-off-by: Brent Baude <bbaude@redhat.com>
| * | [CI:DOCS] Add logo and dev statementBrent Baude2020-01-20
|/ / | | | | | | | | | | Add statement to the info section stating this is experimental only for now. Also add the podman logo to the right-hand navigation. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | Merge pull request #4898 from baude/rtdapi3OpenShift Merge Robot2020-01-18
|\ \ | | | | | | [CI:DOCS]static files end up in _static on rtd
| * | [CI:DOCS]static files end up in _static on rtdbaude2020-01-17
|/ / | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #4897 from baude/rtdapi2OpenShift Merge Robot2020-01-17
|\ \ | | | | | | [CI:DOCS] Correct link syntax
| * | [CI:DOCS] Correct link syntaxbaude2020-01-17
|/ / | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #4896 from baude/rtdapiOpenShift Merge Robot2020-01-17
|\ \ | | | | | | [CI:DOCS]Connect API docs and RTD
| * | [CI:DOCS]Connect API docs and RTDbaude2020-01-17
|/ / | | | | | | Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #4895 from cevich/swagpublishOpenShift Merge Robot2020-01-17
|\ \ | | | | | | [CI:DOCS] Post-process swagger yaml and publish
| * | post-process swagger yaml for publishbaude2020-01-17
|/ / | | | | | | | | Signed-off-by: baude <bbaude@redhat.com> Signed-off-by: Chris Evich <cevich@redhat.com>
* | Merge pull request #4892 from cevich/fix_upload_imageOpenShift Merge Robot2020-01-17
|\ \ | | | | | | Minor: Bugfix in upload image
| * | Minor: Bugfix in upload imageChris Evich2020-01-17
| | | | | | | | | | | | | | | | | | | | | | | | The release upload process always involves two filenames, however the second filename might (someday) be optional. The code allowed for this, however input validation did not. This change fixes the validation. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #4721 from openSUSE/tag-docsOpenShift Merge Robot2020-01-17
|\ \ \ | |/ / |/| | Update `tag` documentation regarding 'alias' usage
| * | Update `tag` documentation regarding 'alias' usageSascha Grunert2020-01-17
| | | | | | | | | | | | | | | | | | | | | | | | The word `alias` is not very common when speaking about image names and tags. So we just refer to image name as the overall identifier of an image. Signed-off-by: Sascha Grunert <sgrunert@suse.com>
* | | Merge pull request #4873 from lsm5/debian-install-updateOpenShift Merge Robot2020-01-17
|\ \ \ | |/ / |/| | update install instructions for Debian, Raspbian and Ubuntu
| * | update install instructions for Debian, Raspbian and UbuntuLokesh Mandvekar2020-01-17
|/ / | | | | | | Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | Merge pull request #4837 from mheon/rework_attachOpenShift Merge Robot2020-01-16
|\ \ | | | | | | Add an API for Attach over HTTP API
| * | Add an API for Attach over HTTP APIMatthew Heon2020-01-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new APIv2 branch provides an HTTP-based remote API to Podman. The requirements of this are, unfortunately, incompatible with the existing Attach API. For non-terminal attach, we need append a header to what was copied from the container, to multiplex STDOUT and STDERR; to do this with the old API, we'd need to copy into an intermediate buffer first, to handle the headers. To avoid this, provide a new API to handle all aspects of terminal and non-terminal attach, including closing the hijacked HTTP connection. This might be a bit too specific, but for now, it seems to be the simplest approach. At the same time, add a Resize endpoint. This needs to be a separate endpoint, so our existing channel approach does not work here. I wanted to rework the rest of attach at the same time (some parts of it, particularly how we start the Attach session and how we do resizing, are (in my opinion) handled much better here. That may still be on the table, but I wanted to avoid breaking existing APIs in this already massive change. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | Merge pull request #4879 from vrothberg/install-golangci-updatesOpenShift Merge Robot2020-01-16
|\ \ \ | | | | | | | | hack/install_golangci.sh: check env vars
| * | | hack/install_golangci.sh: check env varsValentin Rothberg2020-01-16
| | | | | | | | | | | | | | | | | | | | | | | | Also make it executable and `set -eo pipefail`. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #4887 from vrothberg/fix-4886OpenShift Merge Robot2020-01-16
|\ \ \ \ | |_|_|/ |/| | | e2e/run_signal_test.go: make it more robust
| * | | e2e/run_signal_test.go: make it more robustValentin Rothberg2020-01-16
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Make the signal test more robust by just checking that the container's exit code is non-zero. There are two possible exit codes (i.e., 130 and 137) depending on how the container is being killed, which is likely responsible for CI flakes. Fixes: #4886 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #4884 from vrothberg/systemd-ignoreOpenShift Merge Robot2020-01-16
|\ \ \ | | | | | | | | systemdgen: add --ignore flag to generic services
| * | | systemdgen: add --ignore flag to generic servicesValentin Rothberg2020-01-16
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The --ignore flag lets Podman ignore errors when a specified container does not exist (anymore). That's a nice addition to generic services generated via the --new flag. Those services create new containers and can hence allows user to manually remove a container; may it only be by accident. The important part of using the --ignore flag is that Podman will exit 0 which plays nicer with most restart policies; a non-zero exit may yield systemd to restart the entire service which is arguably wrong if the user manually deletes the container. If desired, users can still alter the generated files. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | Merge pull request #4869 from rhatdan/networkOpenShift Merge Robot2020-01-16
|\ \ \ | |/ / |/| | Remove c.String(net)
| * | Remove c.String(net)Daniel J Walsh2020-01-16
| |/ | | | | | | | | | | | | We have a lot of cludgy code trying to make --net and --network equivalent. This will allow --net to still exists but will eliminate the help and confusion. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #4881 from vrothberg/terminate-serviceOpenShift Merge Robot2020-01-16
|\ \ | |/ |/| v2: don't block sigterm and add the service to `make binaries`