summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #12905 from cevich/re-enable_osxOpenShift Merge Robot2022-01-18
|\ | | | | [CI:DOCS] Restore OSX cross-build task
| * Revert "Cirrus: Temporarily disable OSX Cross task"Chris Evich2022-01-18
|/ | | | | | This reverts commit 860463d97eb3a006f2c608165d9e657be86080ec. Signed-off-by: Chris Evich <cevich@redhat.com>
* Merge pull request #12897 from vrothberg/bump-moduleOpenShift Merge Robot2022-01-18
|\ | | | | bump go module to version 4
| * bump go module to version 4Valentin Rothberg2022-01-18
|/ | | | | | | | | | | | | Automated for .go files via gomove [1]: `gomove github.com/containers/podman/v3 github.com/containers/podman/v4` Remaining files via vgrep [2]: `vgrep github.com/containers/podman/v3` [1] https://github.com/KSubedi/gomove [2] https://github.com/vrothberg/vgrep Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* Merge pull request #12684 from mheon/remap_statesOpenShift Merge Robot2022-01-18
|\ | | | | Revamp Libpod state strings for Docker compat
| * Revamp Libpod state strings for Docker compatMatthew Heon2022-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve our compatibility with Docker by better handling the state strings that we print in `podman ps`. Docker capitalizes all states in `ps` (we do not) - fix this in our PS code. Also, stop normalizing ContainerStateConfigured to the "Created" state, and instead make it always be Created, with the existing Created state becoming Initialized. I didn't rename the actual states because I'm somewhat reticent to make such a large change a day before we leave for break. It's somewhat confusing that ContainerStateConfigured now returns Created, but internally and externally we're still consistent. [NO NEW TESTS NEEDED] existing tests should catch anything that broke. I also consider this a breaking change. I will flag appropriately on Github. Fixes RHBZ#2010432 and RHBZ#2032561 Signed-off-by: Matthew Heon <mheon@redhat.com>
* | Merge pull request #12870 from rhatdan/userns1OpenShift Merge Robot2022-01-18
|\ \ | | | | | | Use PODMAN_USERNS environment variable when running as a service
| * | Use PODMAN_USERNS environment variable when running as a serviceDaniel J Walsh2022-01-17
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/11350#issuecomment-1011562526 Also add inspect information about the idmappings if they exists. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #12891 from lsm5/copr-fix-2OpenShift Merge Robot2022-01-17
|\ \ \ | | | | | | | | [NO NEW TESTS NEEDED] add builddeps to copr template
| * | | [NO NEW TESTS NEEDED] add builddeps to copr templateLokesh Mandvekar2022-01-17
| | |/ | |/| | | | | | | Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | | Merge pull request #12893 from cevich/tmp_dis_osxOpenShift Merge Robot2022-01-17
|\ \ \ | | | | | | | | [CI:DOCS] Cirrus: Temporarily disable OSX Cross task
| * | | Cirrus: Temporarily disable OSX Cross taskChris Evich2022-01-17
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | At the time of this commit, something is broken within Cirrus-CI or dependent infrastructure. This appears to be causing all OSX tasks to hang in the scheduling queue indefinitely. Workaround this by disabling the task to allow development work to proceed while a fix is realized. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #12892 from Luap99/commonOpenShift Merge Robot2022-01-17
|\ \ \ | |/ / |/| | update c/common to latest
| * | update c/common to latestPaul Holzinger2022-01-17
|/ / | | | | | | | | | | This contains changes that are needed to enable netavark e2e testing. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* | Merge pull request #12860 from rhatdan/cgroupsOpenShift Merge Robot2022-01-17
|\ \ | | | | | | Use CONTAINERS_CONF cgroups flag for remote API.
| * | Fix cgroup mode handling in api serverDaniel J Walsh2022-01-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | Also change code to globably be consistent when refering to capatilized Cgroup. Fixed: https://github.com/containers/podman/issues/12550 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
| * | Standardize on capatalized CgroupsDaniel J Walsh2022-01-14
| | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #12882 from jmguzik/unify-filters-cmdOpenShift Merge Robot2022-01-17
|\ \ \ | |_|/ |/| | Unify the method of parsing filters in cmd
| * | Unify the method of parsing filters in cmdJakub Guzik2022-01-16
| | | | | | | | | | | | | | | | | | | | | | | | This commit unifies the method of filters parsing in cmd. It removes also the function redundancy. [NO NEW TESTS NEEDED] Signed-off-by: Jakub Guzik <jguzik@redhat.com>
* | | Merge pull request #12880 from ↵OpenShift Merge Robot2022-01-17
|\ \ \ | | | | | | | | | | | | | | | | eriksjolund/replace_master_branch_links_with_main_branch_links_in_docs [CI:DOCS] fix default branch links
| * | | [CI:DOCS] fix default branch linksErik Sjölund2022-01-15
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | * Replace https://github.com/containers/podman/blob/master with https://github.com/containers/podman/blob/main to match the new default branch "main". Previously the default branch was "master". The substitutions were made in the documentation but not the code. Signed-off-by: Erik Sjölund <erik.sjolund@gmail.com>
* | | Merge pull request #12881 from ↵OpenShift Merge Robot2022-01-17
|\ \ \ | | | | | | | | | | | | | | | | eriksjolund/replace_master_branch_links_with_main_branch_links_in_code fix default branch links
| * | | fix default branch linksErik Sjölund2022-01-15
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Replace https://github.com/containers/podman/blob/master with https://github.com/containers/podman/blob/main to match the new default branch "main". Previously the default branch was "master". This is the only occurence found in the code. * Replace https://raw.githubusercontent.com/containers/libpod/master with https://raw.githubusercontent.com/containers/podman/main Signed-off-by: Erik Sjölund <erik.sjolund@gmail.com>
* | | Merge pull request #12876 from ↵OpenShift Merge Robot2022-01-16
|\ \ \ | | | | | | | | | | | | | | | | eriksjolund/fix_doc_rootless_md_native_overlayfs_support [CI:DOCS] Unprivileged native overlayfs is now supported
| * | | [CI:DOCS] Unprivileged native overlayfs is now supportedErik Sjölund2022-01-15
| | | | | | | | | | | | | | | | Signed-off-by: Erik Sjölund <erik.sjolund@gmail.com>
* | | | Merge pull request #12868 from lsm5/copr-fixOpenShift Merge Robot2022-01-16
|\ \ \ \ | |_|/ / |/| | | Add rpkg template for COPR autobuild
| * | | Add rpkg template for COPR autobuildLokesh Mandvekar2022-01-14
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit removes .autocopr/podman.spec in favor of podman.spec.rpkg to account for new recommendations for COPR builds. The build process now enables debug packages and also uses `%gobuild` instead of `make` or `go build` to get the packages as close as possible to Fedora koji builds. https://lists.fedoraproject.org/archives/list/copr-devel@lists.fedorahosted.org/message/3JORYRKDWFMJSR35Z4LIKDEXH2T5263H/ [NO NEW TESTS NEEDED] Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
* | | Merge pull request #12572 from rhatdan/imageOpenShift Merge Robot2022-01-15
|\ \ \ | | | | | | | | Remove two GetImages functions from API
| * | | Remove two GetImages functions from APIDaniel J Walsh2022-01-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] This is just code cleanup. The remote API has three different GetImages functions, which I believe can be handled by just one function. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #12861 from aklajnert/recursively_copy_certsOpenShift Merge Robot2022-01-15
|\ \ \ \ | | | | | | | | | | ignition: recursively copy cert files
| * | | | Recursively copy cert files.Andrzej Klajnert2022-01-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] Signed-off-by: Andrzej Klajnert <github@aklajnert.pl>
* | | | | Merge pull request #12527 from jwhonce/wip/manifest4OpenShift Merge Robot2022-01-15
|\ \ \ \ \ | | | | | | | | | | | | Refactor manifest list operations
| * | | | | Refactor manifest list operationsJhon Honce2022-01-14
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Update method/function signatures use the manifest list name and images associated with the operation explicitly, in general func f(ctx context.Context, manifestListName string, ImageNames []string, options *fOptions) * Leverage gorilla/mux Subrouters to support API v3.x and v4.x for manifests * Make manifest API endpoints more RESTful * Add PUT /manifest/{id} to update existing manifests * Add manifests.Annotate to go bindings, uncommented unit test * Add DELETE /manifest/{Id} to remove existing manifest list, use PUT /manifest/{id} to remove images from a list * Deprecated POST /manifest/{id}/add and /manifest/{id}/remove, use PUT /manifest/{id} instead * Corrected swagger godoc and updated to cover API changes * Update podman manifest commands to use registry.Context() * Expose utils.GetVar() to obtain query parameters by name * Unexpose server.registerSwaggerHandlers, not sure why this was ever exposed. * Refactored code to use http.Header instead of map[string]string when operating on HTTP headers. * Add API-Version header support in bindings to allow calling explicate versions of the API. Header is _NOT_ forwarded to the API service. Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | | Merge pull request #12875 from eriksjolund/fix_typo_env_podman_runOpenShift Merge Robot2022-01-15
|\ \ \ \ \ | |_|_|_|/ |/| | | | [CI:DOCS] Fix typo in --env
| * | | | [CI:DOCS] Fix typo in --envErik Sjölund2022-01-15
|/ / / / | | | | | | | | | | | | Signed-off-by: Erik Sjölund <erik.sjolund@gmail.com>
* | | | Merge pull request #12858 from rhatdan/hostsOpenShift Merge Robot2022-01-14
|\ \ \ \ | |/ / / |/| | | test/system: podman run update /etc/hosts
| * | | test/system: podman run update /etc/hostsDaniel J Walsh2022-01-14
| | |/ | |/| | | | | | | | | | | | | | | | This test case is used for covering rhbz#1902979. Signed-off-by: Alex Jia <ajia@redhat.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #12856 from fredr/issue-template/fix-linksOpenShift Merge Robot2022-01-14
|\ \ \ | | | | | | | | [CI:DOCS] troubleshooting links to main branch
| * | | troubleshooting links to main branchfredr2022-01-14
| | | | | | | | | | | | | | | | Signed-off-by: fredr <fredrik@enestad.com>
* | | | Merge pull request #12825 from elezar/update-cdiOpenShift Merge Robot2022-01-14
|\ \ \ \ | |_|/ / |/| | | Update use of CDI API
| * | | Use fully-qualified device name in CDI testEvan Lezar2022-01-14
| | | | | | | | | | | | | | | | Signed-off-by: Evan Lezar <elezar@nvidia.com>
| * | | Use new CDI APIEvan Lezar2022-01-14
| |/ / | | | | | | | | | | | | | | | | | | This change updates the CDI API to commit 46367ec063fda9da931d050b308ccd768e824364 which addresses some inconistencies in the previous implementation. Signed-off-by: Evan Lezar <elezar@nvidia.com>
* | | Merge pull request #12853 from cdoern/buildRelativeOpenShift Merge Robot2022-01-14
|\ \ \ | | | | | | | | Podman Build use absolute filepath
| * | | Podman Build use absolute filepathcdoern2022-01-13
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | podman build always finds the abs path but was never using it for the containerfile path. This was causing the remote client to be given a relative path that does not exist. Switch to evaluating and using absolute paths only. resolves #12841 Signed-off-by: cdoern <cdoern@redhat.com>
* | | Merge pull request #12849 from cdoern/podProhibitOpenShift Merge Robot2022-01-14
|\ \ \ | |/ / |/| | Prohibit --uid/gid map and --pod for container create/run
| * | Prohibit --uid/gid map and --pod for container create/runcdoern2022-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | add a check in namespaceOptions() that ensures the user is not setting a new uid/gid map if entering or creating a pod that has an infra container resolves #12669 Signed-off-by: cdoern <cdoern@redhat.com>
* | | Merge pull request #12828 from giuseppe/drop-checkOpenShift Merge Robot2022-01-13
|\ \ \ | | | | | | | | libpod: drop check for empty pod cgroup
| * | | libpod: refine check for empty pod cgroupGiuseppe Scrivano2022-01-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rootless containers do not use cgroups on cgroupv1 or if using cgroupfs, so improve the check to account for such configuration. Closes: https://github.com/containers/podman/issues/10800 Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2028243 [NO NEW TESTS NEEDED] it requires rebooting and the rundir on a non tmpfs file system. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #12826 from vrothberg/force-rm-podOpenShift Merge Robot2022-01-13
|\ \ \ \ | | | | | | | | | | podman container rm: remove pod
| * | | | podman container rm: remove podValentin Rothberg2022-01-13
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Support removing the entire pod when --depend is used on an infra container. --all now implies --depend to properly support removing all containers and not error out when hitting infra containers. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>