summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Updated based on Jhonce commentsParker Van Roy2021-02-24
| | | | | | | | | Added ID to default removed outdated imports slight refactor Test Network Headers added to network_test.go Signed-off-by: Parker Van Roy <pvanroy@redhat.com>
* updated common to 0.35.0Parker Van Roy2021-02-24
| | | | Signed-off-by: Parker Van Roy <pvanroy@redhat.com>
* Refactored fileParker Van Roy2021-02-24
| | | | | | | | moved --format to templateOut() Rm parse package except JSON Signed-off-by: Parker Van Roy <pvanroy@redhat.com>
* Merge pull request #9504 from kierdavis/fix-manpageOpenShift Merge Robot2021-02-24
|\ | | | | [CI:DOCS] podman-system-connection.1.md: fix copy/paste error
| * podman-system-connection.1.md: fix copy/paste errorKier Davis2021-02-24
|/ | | | Signed-off-by: Kier Davis <me@kierdavis.com>
* Merge pull request #9494 from mheon/sort_capsOpenShift Merge Robot2021-02-24
|\ | | | | Sort CapDrop in inspect to guarantee order
| * Sort CapDrop in inspect to guarantee orderMatthew Heon2021-02-23
| | | | | | | | | | | | | | | | | | | | The order of CapAdd when inspecting containers is deterministic. However, the order of CapDrop is not (for unclear reasons). Add a quick sort on the final array to guarantee a consistent order. Fixes #9490 Signed-off-by: Matthew Heon <mheon@redhat.com>
* | Merge pull request #9501 from Luap99/dns-search-cniOpenShift Merge Robot2021-02-24
|\ \ | | | | | | Add dns search domains from cni response to resolv.conf
| * | Add dns search domains from cni response to resolv.confPaul Holzinger2021-02-24
|/ / | | | | | | | | | | | | This fixes slow local host name lookups. see containers/dnsname#57 Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* | Merge pull request #9497 from Luap99/fix-9496OpenShift Merge Robot2021-02-23
|\ \ | | | | | | Network connect error if net mode is not bridge
| * | Network connect error if net mode is not bridgePaul Holzinger2021-02-23
|/ / | | | | | | | | | | | | | | | | | | Only the the network mode bridge supports cni networks. Other network modes cannot use network connect/disconnect so we should throw a error. Fixes #9496 Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* | Merge pull request #9485 from vrothberg/fix-9479OpenShift Merge Robot2021-02-23
|\ \ | |/ |/| container removal: handle already removed containers
| * container removal: handle already removed containersValentin Rothberg2021-02-23
| | | | | | | | | | | | | | | | | | | | | | | | Since commit d54478d8eaec, a container's lock is released before attempting to stop it via the OCI runtime. This opened the window for various kinds of race conditions. One of them led to #9479 where the removal+cleanup sequences of a `run --rm` session overlapped with `rm -af`. Make both execution paths more robust by handling the case of an already removed container. Fixes: #9479 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | Merge pull request #9483 from ↵OpenShift Merge Robot2021-02-23
|\ \ | | | | | | | | | | | | containers/dependabot/go_modules/github.com/rootless-containers/rootlesskit-0.13.2 Bump github.com/rootless-containers/rootlesskit from 0.13.1 to 0.13.2
| * | Bump github.com/rootless-containers/rootlesskit from 0.13.1 to 0.13.2dependabot-preview[bot]2021-02-23
| |/ | | | | | | | | | | | | | | | | Bumps [github.com/rootless-containers/rootlesskit](https://github.com/rootless-containers/rootlesskit) from 0.13.1 to 0.13.2. - [Release notes](https://github.com/rootless-containers/rootlesskit/releases) - [Commits](https://github.com/rootless-containers/rootlesskit/compare/v0.13.1...v0.13.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #9463 from ↵OpenShift Merge Robot2021-02-23
|\ \ | | | | | | | | | | | | containers/dependabot/go_modules/k8s.io/apimachinery-0.20.4 Bump k8s.io/apimachinery from 0.20.3 to 0.20.4
| * | Bump k8s.io/apimachinery from 0.20.3 to 0.20.4dependabot-preview[bot]2021-02-23
| |/ | | | | | | | | | | | | | | | | Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.20.3 to 0.20.4. - [Release notes](https://github.com/kubernetes/apimachinery/releases) - [Commits](https://github.com/kubernetes/apimachinery/compare/v0.20.3...v0.20.4) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #8349 from EduardoVega/7778-chowning-based-on-uidOpenShift Merge Robot2021-02-23
|\ \ | |/ |/| Add U volume flag to chown source volumes
| * Add U volume flag to chown source volumesEduardo Vega2021-02-22
|/ | | | Signed-off-by: Eduardo Vega <edvegavalerio@gmail.com>
* Merge pull request #9480 from jmguzik/replace-nil-with-braces-network-resourceOpenShift Merge Robot2021-02-22
|\ | | | | [NO TESTS NEEDED] Replace Labels and Options nulls with {} in NetworkResource
| * Replace Labels and Options nulls with {} in NetworkResourceJakub Guzik2021-02-23
| | | | | | | | Signed-off-by: Jakub Guzik <jakubmguzik@gmail.com>
* | Merge pull request #9474 from cevich/disable_f32OpenShift Merge Robot2021-02-22
|\ \ | | | | | | Cirrus: Temp. disable prior-fedora (F32) testing
| * | Cirrus: Temp. disable prior-fedora (F32) testingChris Evich2021-02-22
| | | | | | | | | | | | | | | | | | | | | | | | In anticipation of F34beta support, preemptively disable testing on "prior-Fedora". This will allow development to move forward without a soon-to-be-EOL distro. holding anything back. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | Merge pull request #9455 from Luap99/fix-network-idsOpenShift Merge Robot2021-02-22
|\ \ \ | | | | | | | | Fix podman network IDs handling
| * | | Fix podman network IDs handlingPaul Holzinger2021-02-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The libpod network logic knows about networks IDs but OCICNI does not. We cannot pass the network ID to OCICNI. Instead we need to make sure we only use network names internally. This is also important for libpod since we also only store the network names in the state. If we would add a ID there the same networks could accidentally be added twice. Fixes #9451 Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
* | | | Merge pull request #9464 from giuseppe/fix-cgroupv1-statsOpenShift Merge Robot2021-02-22
|\ \ \ \ | | | | | | | | | | cgroup: change cgroup deletion logic on v1
| * | | | cgroup: change cgroup deletion logic on v1Giuseppe Scrivano2021-02-22
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | do not raise an error if the cgroup exists at least on one controller. Previously it expected the cgroup to exists under all the controllers. [NO TESTS NEEDED] Closes: https://github.com/containers/podman/issues/9252 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* | | | Merge pull request #9469 from vrothberg/cp-stdoutOpenShift Merge Robot2021-02-22
|\ \ \ \ | | | | | | | | | | podman cp: /dev/std{in,out} fixes
| * | | | podman cp: test /dev/stdin correctlyValentin Rothberg2021-02-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | /dev/stdin should not be treated as "-" to remain compatible with Docker and to have a more consistent and idiomatic interface. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | | podman cp: treat /dev/stdout correctlyValentin Rothberg2021-02-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | /dev/stdout should not be treated as "-" to remain compatible with Docker and to have a more consistent and idiomatic interface. Fixes: #9362 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | Merge pull request #9456 from matejvasek/make_internalOpenShift Merge Robot2021-02-22
|\ \ \ \ \ | |_|_|_|/ |/| | | | Make binding util internal
| * | | | [NO TESTS NEEDED] Make binding util internalMatej Vasek2021-02-22
| |/ / / | | | | | | | | | | | | | | | | | | | | The functions are supposed to be called only from generated code Signed-off-by: Matej Vasek <mvasek@redhat.com>
* | | | Merge pull request #9445 from jmguzik/no-header-info-for-systemd-generationOpenShift Merge Robot2021-02-22
|\ \ \ \ | | | | | | | | | | No header info for systemd generation
| * | | | --no-header flag implementation for generate systemdJakub Guzik2021-02-22
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Jakub Guzik <jakubmguzik@gmail.com>
* | | | | Merge pull request #9468 from edsantiago/new_test_checkerOpenShift Merge Robot2021-02-22
|\ \ \ \ \ | | | | | | | | | | | | pr-should-include-tests: recognized "renamed" tests
| * | | | | pr-should-include-tests: recognized "renamed" testsEd Santiago2021-02-22
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | git tries to recognize renamed files. This isn't always as helpful as intended. Turn it off, so we'll always see files as 'A'dded. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | Merge pull request #9117 from tmds/swagger_missing_schema_propertiesOpenShift Merge Robot2021-02-22
|\ \ \ \ \ | |/ / / / |/| | | | [NO TESTS NEEDED] swagger: add missing schema properties
| * | | | swagger: add missing schema propertiesTom Deseyn2021-02-22
| | |_|/ | |/| | | | | | | | | | | | | | | | | | NO_TESTS_NEEDED Signed-off-by: Tom Deseyn <tom.deseyn@gmail.com>
* | | | Merge pull request #9420 from rhatdan/kubeOpenShift Merge Robot2021-02-22
|\ \ \ \ | | | | | | | | | | [NO TESTS NEEDED] Allow podman play kube to read yaml file from stdin
| * | | | Allow podman play kube to read yaml file from stdinDaniel J Walsh2021-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/8996 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #9335 from rlukin/patch-1OpenShift Merge Robot2021-02-22
|\ \ \ \ \ | |_|_|_|/ |/| | | | [ci:docs] Update rootless_tutorial.md
| * | | | Fix 'storage.options' indentRoman2021-02-21
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Roman Lukin <me@rlukin.ru>
* | | | | Merge pull request #9275 from rhatdan/buildOpenShift Merge Robot2021-02-22
|\ \ \ \ \ | | | | | | | | | | | | Add missing params for podman-remote build
| * | | | | Add missing params for podman-remote buildDaniel J Walsh2021-02-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/9290 Currently we still have hard coded --isolation=chroot for podman-remote build. Implement missing arguments for podman build Implements --jobs, --disable-compression, --excludes Fixes: MaxPullPushRetries RetryDuration Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #9448 from vrothberg/bump-module-versionOpenShift Merge Robot2021-02-22
|\ \ \ \ \ \ | | | | | | | | | | | | | | bump go module to v3
| * | | | | | bump go module to v3Valentin Rothberg2021-02-22
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We missed bumping the go module, so let's do it now :) * Automated go code with github.com/sirkon/go-imports-rename * Manually via `vgrep podman/v2` the rest Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | Merge pull request #9458 from jmguzik/podman-systemd-generate-new-e2eOpenShift Merge Robot2021-02-22
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Two usage variations of --new flag added to e2e
| * | | | | Two variations of --new flag added to e2eJakub Guzik2021-02-22
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Jakub Guzik <jakubmguzik@gmail.com>
* | | | | Merge pull request #9436 from jonas-schievink/patch-1OpenShift Merge Robot2021-02-21
|\ \ \ \ \ | | | | | | | | | | | | [NO TESTS NEEDED] Log working dir when chdir fails
| * | | | | Log working dir when chdir failsJonas Schievink2021-02-19
| | |_|/ / | |/| | | | | | | | | | | | | Signed-off-by: Jonas Schievink <jonasschievink@gmail.com>