summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* libpod: avoid polling container statusGiuseppe Scrivano2019-09-04
| | | | | | | | | | | | | | | | | | | | | | | use the inotify backend to be notified on the container exit instead of polling continuosly the runtime. Polling the runtime slowns significantly down the podman execution time for short lived processes: $ time bin/podman run --rm -ti fedora true real 0m0.324s user 0m0.088s sys 0m0.064s from: $ time podman run --rm -ti fedora true real 0m4.199s user 0m5.339s sys 0m0.344s Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* Merge pull request #3928 from edsantiago/enable_man_page_checker_in_ciOpenShift Merge Robot2019-09-04
|\ | | | | Enable hack/man-page-checker in CI
| * Enable hack/man-page-checker in CIEd Santiago2019-09-03
| | | | | | | | | | | | | | With huge thanks to @rwha for #3915. All man pages are clean and consistent now - let's keep them that way. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | Merge pull request #3930 from cevich/update_irc_tableOpenShift Merge Robot2019-09-04
|\ \ | | | | | | Cirrus: Update e-mail -> IRC Nick table
| * | Cirrus: Update e-mail -> IRC Nick tableChris Evich2019-09-03
|/ / | | | | | | | | | | | | | | Also add fixes to help prevent 'fatal: Invalid revision range' error. Should obtaining all authors from the range still fail, only grab the HEAD commit author as a fallback. Signed-off-by: Chris Evich <cevich@redhat.com>
* | Merge pull request #3926 from giuseppe/add-warning-mismatch-configurationOpenShift Merge Robot2019-09-03
|\ \ | | | | | | rootless: detect user namespace configuration changes
| * | pkg/util: use rootless function to read additional usersGiuseppe Scrivano2019-09-03
| | | | | | | | | | | | | | | | | | | | | make pkg/rootless.GetConfiguredMappings public so that it can be used from pkg/util. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | rootless: detect user namespace configuration changesGiuseppe Scrivano2019-09-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | detect if the current user namespace doesn't match the configuration in the /etc/subuid and /etc/subgid files. If there is a mismatch, raise a warning and suggest the user to recreate the user namespace with "system migrate", that also restarts the containers. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | Merge pull request #3904 from cevich/fix_img_pruneOpenShift Merge Robot2019-09-03
|\ \ \ | |_|/ |/| | Cirrus: imgprune fixes
| * | Cirrus: Load base-image names indirectlyChris Evich2019-08-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than hard-coding all four base-image env. var name, load the values based on the shared variable name suffix. Thanks to Ed Santiago <santiago@redhat.com> for the suggestion. Signed-off-by: Chris Evich <cevich@redhat.com>
| * | Cirrus: Remove image_prune YAML-alias workaroundChris Evich2019-08-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally when written, the image_prune_task was passing incorrect credentials into it's container. This was traced back to mishandling of a YAML-alias, and worked-around (including a comment). However, as of this commit, it appear YAML-alias substitution problem has been addressed. Restore the (correct) use of a YAML-alias so the correct credentials are passed into the prune container. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #3915 from rwha/fix_man_page_testsOpenShift Merge Robot2019-09-03
|\ \ \ | | | | | | | | Fix formatting and enable hack/man-page-checker
| * | | Add command aliases to SYNOPSIS sectionRyan Whalen2019-08-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The files under docs/links reference another man page, e.g. `man podman-container-list` displays `podman-ps(1)`. This adds the alias to the in the displayed page's SYNOPSIS section. Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
| * | | Exclude podman-remoteRyan Whalen2019-08-31
| | | | | | | | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
| * | | Fix table spacingRyan Whalen2019-08-30
| | | | | | | | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
| * | | Revert the descriptive text for podman-remoteRyan Whalen2019-08-30
| | | | | | | | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
| * | | WIP - ignore man pages for commands besides podmanRyan Whalen2019-08-30
| | | | | | | | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
| * | | podman-remote is not a subcommandRyan Whalen2019-08-30
| | | | | | | | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
| * | | Fix formatting and enable hack/man-page-checkerRyan Whalen2019-08-30
| | | | | | | | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
* | | | Merge pull request #3925 from MatMaul/doc-rootless-systemdOpenShift Merge Robot2019-09-03
|\ \ \ \ | |_|_|/ |/| | | rootless.md: add systemd unit example
| * | | rootless.md: add systemd unit exampleMathieu Velten2019-09-03
|/ / / | | | | | | | | | Signed-off-by: Mathieu Velten <matmaul@gmail.com>
* | | Merge pull request #3909 from giuseppe/rootless-bind-mount-devOpenShift Merge Robot2019-09-02
|\ \ \ | | | | | | | | rootless: bind mount devices instead of creating them
| * | | spec: provide custom implementation for getDevicesGiuseppe Scrivano2019-09-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | provide an implementation for getDevices that skip unreadable directories for the current user. Based on the implementation from runc/libcontainer. Closes: https://github.com/containers/libpod/issues/3919 Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org> Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| * | | spec: do not set devices cgroup when rootlessGiuseppe Scrivano2019-09-02
| | | | | | | | | | | | | | | | | | | | | | | | eBPF requires to be root in the init namespace. Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
| * | | rootless: bind mount devices instead of creating themGiuseppe Scrivano2019-09-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when running in rootless mode, --device creates a bind mount from the host instead of specifying the device in the OCI configuration. This is required as an unprivileged user cannot use mknod, even when root in a user namespace. Closes: https://github.com/containers/libpod/issues/3905 Signed-off-by: Giuseppe Scrivano <giuseppe@scrivano.org>
* | | | Merge pull request #3922 from giuseppe/add-note-failing-systemd-cgroups-v1OpenShift Merge Robot2019-09-02
|\ \ \ \ | |/ / / |/| | | docs: add note about failing rhel7 systemd on cgroups v2
| * | | docs: add note about failing rhel7 systemd on cgroups v2Giuseppe Scrivano2019-09-02
| | | | | | | | | | | | | | | | | | | | | | | | Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1747933 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #3916 from cevich/personalize_successOpenShift Merge Robot2019-09-01
|\ \ \ \ | |_|/ / |/| | | Personalize CI success messages to IRC
| * | | Cirrus: On success, add IRC nick mention to msgChris Evich2019-08-30
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than spamming the podman channel with impersonal success messages referring to PR numbers, mention the author by nick name and include the PR title and link. Also avoid needless logging of all bot-script interactions with IRC when there is no error detected. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #3911 from vrothberg/go-proxyOpenShift Merge Robot2019-08-30
|\ \ \ | | | | | | | | Makefile: use go proxy
| * | | Makefile: use go proxyValentin Rothberg2019-08-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use GOPROXY=https://proxy.golang.org to speed up fetching dependencies. Setting it makes `make vendor` three times faster in my local env. For details please refer to https://proxy.golang.org/. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | Merge pull request #3913 from rwha/fix/manpage-linksOpenShift Merge Robot2019-08-30
|\ \ \ \ | |_|_|/ |/| | | Fix links to manpages
| * | | Fix links to manpagesRyan Whalen2019-08-30
|/ / / | | | | | | | | | Signed-off-by: Ryan Whalen <rj.whalen@gmail.com>
* | | Merge pull request #3908 from TomasTomecek/man-ev-bkndOpenShift Merge Robot2019-08-30
|\ \ \ | |/ / |/| | man: events-logger → events-backend
| * | man: events-logger → events-backendTomas Tomecek2019-08-29
| | | | | | | | | | | | Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
* | | Merge pull request #3907 from baude/commitcapsOpenShift Merge Robot2019-08-29
|\ \ \ | |/ / |/| | dont panic when using varlink commit and uppercase image names
| * | dont panic when using varlink commit and uppercase image namesbaude2019-08-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | when using an upper case image name for container commit, we observed panics due to a channel closing early. Fixes: #3897 Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #3710 from cevich/release_redoOpenShift Merge Robot2019-08-29
|\ \ \ | | | | | | | | Release redo
| * | | Cirrus: Reimplement release archive + uploadChris Evich2019-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initial implementation was far more complicated than necessary. Strip out the complexities in favor of a simpler and more direct approach. Signed-off-by: Chris Evich <cevich@redhat.com>
| * | | Readme: Links for automatic binary releasesChris Evich2019-08-28
| | | | | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | Merge pull request #3833 from QiWang19/cert-dirOpenShift Merge Robot2019-08-28
|\ \ \ \ | | | | | | | | | | add --cert-dir image sign
| * | | | add --cert-dir image signQi Wang2019-08-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Requirement from #2726 Add --cert-dir for `podman image sign`. Signed-off-by: Qi Wang <qiwan@redhat.com>
* | | | | Merge pull request #3892 from cevich/google_vpcOpenShift Merge Robot2019-08-28
|\ \ \ \ \ | | | | | | | | | | | | Cirrus: Block CNI use of google VPCs
| * | | | | Cirrus: Block CNI use of google VPCsChris Evich2019-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Specifically pertaining to executing tests in google cloud, there are default, pre-allocated class-a subnetworks for each region (data-center). Each includes a gateway using a `.1` LSB and all are routable from other regions in google cloud via these gateways. Because the default CNI configuration also utilizes class-a subnetworks, this creates the possibility for IPv4 address-space clashes. Since the default regional cloud subnets are pre-defined/known, preventing clashes can be accomplished by seeding these subnets in a dummy CNI configuration. The default behavior of podman is to grab the highest priority CNI configuration. Name the dummy config. appropriate so it always loads last. Also name the bridge itself with an obvious name `do-not-use`, such that any related testing errors should be easier to debug. Also: * Minor cleanup of `install_test_configs()` * Move install_test_configs in `setup_environment.sh` to after possible run of `remove_packaged_podman_files()` because that also strips out `/etc/cni/net.d/87-podman-bridge.conflist`. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | | Merge pull request #3893 from mheon/readd_volume_locksOpenShift Merge Robot2019-08-28
|\ \ \ \ \ \ | | | | | | | | | | | | | | Re-add volume locks
| * | | | | | Re-add locks to volumes.Matthew Heon2019-08-28
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will require a 'podman system renumber' after being applied to get lock numbers for existing volumes. Add the DB backend code for rewriting volume configs and use it for updating lock numbers as part of 'system renumber'. Signed-off-by: Matthew Heon <matthew.heon@pm.me>
* | | | | | Merge pull request #3728 from mheon/systemd_container_testOpenShift Merge Robot2019-08-28
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | Add an integration test for systemd in a container
| * | | | | Temporarily disable systemd test for CGroups V2Matthew Heon2019-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert this one CGroups V2 support for systemd containers is added. Signed-off-by: Matthew Heon <mheon@redhat.com>
| * | | | | Add an integration test for systemd in a containerMatthew Heon2019-08-28
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@pm.me>
| * | | | | clean up after remote buildbaude2019-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when performing an image build over a varlink connection, we should clean up tmp files that are a result of sending the file to the host and untarring it for the build. Fixes: #3869 Signed-off-by: baude <bbaude@redhat.com>