aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #14999 from sstosh/restart-optionOpenShift Merge Robot2022-08-17
|\ | | | | Add restart --cidfile, --filter
| * Update cmd/podman/containers/restart.goTom Sweeney2022-08-12
| | | | | | | | | | Co-authored-by: Tom Sweeney <tsweeney@redhat.com> Signed-off-by: Toshiki Sonoda <sonoda.toshiki@fujitsu.com>
| * Add restart --cidfile, --filterToshiki Sonoda2022-08-12
| | | | | | | | | | | | | | --cidfile : Read container ID from the specified file and restart the container. --filter : restart the filtered container. Signed-off-by: Toshiki Sonoda <sonoda.toshiki@fujitsu.com>
* | Merge pull request #15350 from nalind/manifest-amendOpenShift Merge Robot2022-08-17
|\ \ | | | | | | podman manifest create: accept --amend and --insecure flags
| * | podman manifest create: accept --amend and --insecure flagsNalin Dahyabhai2022-08-16
| | | | | | | | | | | | | | | | | | | | | | | | Accept a --amend flag in `podman manifest create`, and treat `--insecure` as we would `--tls-verify=false` in `podman manifest`'s "add", "create", and "push" subcommands. Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
* | | Merge pull request #15324 from dfr/freebsd-buildOpenShift Merge Robot2022-08-17
|\ \ \ | |/ / |/| | utils: Add missing symbols for FreeBSD
| * | utils: Add missing symbols for FreeBSDDoug Rabson2022-08-16
| | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] Signed-off-by: Doug Rabson <dfr@rabson.org>
* | | Merge pull request #15345 from edsantiago/runc_fixOpenShift Merge Robot2022-08-16
|\ \ \ | | | | | | | | system tests: fix broken runc test
| * | | system tests: fix broken runc testEd Santiago2022-08-16
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Followup to #14613, which was never actually tested until this week in RHEL8 gating tests (see issue #15337). * add missing backslash in '|' expression * allow extra text after error (e.g., "invalid argument") No way to test this until it makes its way into RHEL8, so, fingers crossed. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | Merge pull request #15317 from rhatdan/debugOpenShift Merge Robot2022-08-16
|\ \ \ | | | | | | | | Add compatibility support for --debug flag from docker
| * | | Add compatibility support for --debug -D flag from dockerDaniel J Walsh2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | This is another fix for https://github.com/containers/podman/issues/14917 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #15339 from edsantiago/docs_dedup_annotationOpenShift Merge Robot2022-08-16
|\ \ \ \ | | | | | | | | | | Man pages: refactor common options: --annotation
| * | | | Man pages: refactor common options: --annotationEd Santiago2022-08-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactor the --annotation option, but only between podman create, kube play, and run. This does not include: * podman build: - usage is in terms of images, not containers/pods * manifest add, manifest annotate: - usage is in terms of images, not containers/pods - also, wording is slightly different Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | Merge pull request #15270 from nicrowe00/kubegenerateOpenShift Merge Robot2022-08-16
|\ \ \ \ \ | |/ / / / |/| | | | Add "podman kube generate" command
| * | | | Add "podman kube generate" commandNiall Crowe2022-08-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "podman kube generate" creates Kubernetes YAML from Podman containers, pods or volumes. Users will still be able to use "podman generate kube" as an alias of "kube generate". Signed-off-by: Niall Crowe <nicrowe@redhat.com>
* | | | | Merge pull request #15331 from edsantiago/docs_dedup_piecemealOpenShift Merge Robot2022-08-16
|\ \ \ \ \ | |_|_|/ / |/| | | | Man pages: refactor common options: arch
| * | | | Man pages: refactor common options: archEd Santiago2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Smaller, more reviewable chunks. This is just one option, --arch. Future PRs may, if the reviewing is easy, include multiple options. This one includes fixes to the preprocessor script, though: * big oops, I was not handling '<<something pod|something>>' where 'pod' appears other than the beginning of the string. * I was also not handling 'container<<| or pod>>', where one side was empty. * Behavior change: <<subcommand>>, on podman-pod-foo, becomes just 'foo' (not 'pod foo'). This will be useful in a future PR where we refactor --pod-id-file. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | Merge pull request #15328 from cevich/machine_fixupsOpenShift Merge Robot2022-08-15
|\ \ \ \ \ | | | | | | | | | | | | [CI:DOCS] Cirrus: Update podman-machine comment
| * | | | | Cirrus: Update podman-machine commentChris Evich2022-08-15
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Replace TODO comment with helpful hint for future maintainers. Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #15318 from rhatdan/dns-optOpenShift Merge Robot2022-08-15
|\ \ \ \ \ | | | | | | | | | | | | Add Docker compatibility for --dns-option to --dns-opt
| * | | | | Add Docker compatibility for --dns-option to --dns-optDaniel J Walsh2022-08-15
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | Fixes one of the issues found in https://github.com/containers/podman/issues/14917 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #15327 from rhatdan/man1OpenShift Merge Robot2022-08-15
|\ \ \ \ \ | | | | | | | | | | | | [CI:DOCS] podman images and friends can take one image as argument
| * | | | | podman images and friends can take one image as argumentDaniel J Walsh2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/15309 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #15323 from rhatdan/warnOpenShift Merge Robot2022-08-15
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Don't warn on '/' not being shared while in a container
| * | | | | Don't warn on '/' not being shared while in a containerDaniel J Walsh2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://github.com/containers/podman/issues/15295 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | | Merge pull request #15325 from ashley-cui/dstoreOpenShift Merge Robot2022-08-15
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | [CI:DOCS] Add .DS_Store to gitignore
| * | | | | [CI:DOCS] Add .DS_Store to gitignoreAshley Cui2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ignore .DS_Store, a MacOS metadata file created by the OS. Signed-off-by: Ashley Cui <acui@redhat.com>
* | | | | | Merge pull request #15301 from chenxiaolong/restart_neverOpenShift Merge Robot2022-08-15
|\ \ \ \ \ \ | |/ / / / / |/| | | | | podman-kube@.service.in: Remove Restart=never option with typo
| * | | | | podman-kube@.service.in: Remove Restart=never option with typoAndrew Gunnerson2022-08-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | systemd expects the value of the option to be `no` instead, but this is already the default behavior. This fixes the following warning when running `systemctl status` on the unit: Failed to parse service restart specifier, ignoring: never Signed-off-by: Andrew Gunnerson <chillermillerlong@hotmail.com>
* | | | | | Merge pull request #15319 from ↵OpenShift Merge Robot2022-08-15
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/mattn/go-isatty-0.0.16 build(deps): bump github.com/mattn/go-isatty from 0.0.14 to 0.0.16
| * | | | | build(deps): bump github.com/mattn/go-isatty from 0.0.14 to 0.0.16dependabot[bot]2022-08-15
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/mattn/go-isatty](https://github.com/mattn/go-isatty) from 0.0.14 to 0.0.16. - [Release notes](https://github.com/mattn/go-isatty/releases) - [Commits](https://github.com/mattn/go-isatty/compare/v0.0.14...v0.0.16) --- updated-dependencies: - dependency-name: github.com/mattn/go-isatty dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | | | Merge pull request #15316 from dfr/freebsd-buildOpenShift Merge Robot2022-08-15
|\ \ \ \ \ | |/ / / / |/| | | | Add non-linux build stubs for pkg/domain and pkg/specgen
| * | | | pkg/domain: Add stubs for non-linux buildsDoug Rabson2022-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] Signed-off-by: Doug Rabson <dfr@rabson.org>
| * | | | pkg/specgen: Add stubs for non-linux buildsDoug Rabson2022-08-15
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces a local type rlimT which is used to convert runtime-spec POSIXRlimit to platform-specific Rlimit structures - on FreeBSD rlimit members are signed integers. [NO NEW TESTS NEEDED] Signed-off-by: Doug Rabson <dfr@rabson.org>
* | | | Merge pull request #15290 from dfr/freebsd-buildOpenShift Merge Robot2022-08-13
|\ \ \ \ | |/ / / |/| | | FreeBSD build fixes for pkg/util and pkg/machine
| * | | pkg: Build pkg/machine on FreeBSDDoug Rabson2022-08-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing symbols for freebsd and freebsd/amd64. [NO NEW TESTS NEEDED] Signed-off-by: Doug Rabson <dfr@rabson.org>
| * | | pkg: Build pkg/util on FreeBSDDoug Rabson2022-08-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove freebsd from the unsupported list and add missing symbols. [NO NEW TESTS NEEDED] Signed-off-by: Doug Rabson <dfr@rabson.org>
| * | | Add default value for $RMDoug Rabson2022-08-13
|/ / / | | | | | | | | | | | | | | | | | | | | | For some reason GNU make doesn't define this on FreeBSD. [NO NEW TESTS NEEDED] Signed-off-by: Doug Rabson <dfr@rabson.org>
* | | Merge pull request #15253 from ↵OpenShift Merge Robot2022-08-12
|\ \ \ | |_|/ |/| | | | | | | | fulldecent/fix-broken-link-to-install-instructions [CI:DOCS] fix updated links
| * | Fix updated link to install instructionsWilliam Entriken2022-08-11
|/ / | | | | | | Signed-off-by: William Entriken <github.com@phor.net>
* | Merge pull request #15286 from mheon/bump_420_mainOpenShift Merge Robot2022-08-11
|\ \ | | | | | | [CI:DOCS] Update release notes on main branch for v4.2.0 release
| * | Update release notes for v4.2.0Matthew Heon2022-08-11
| | | | | | | | | | | | | | | | | | Also add Podman Desktop to our README. Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | Merge pull request #15289 from flouthoc/update-build-docsOpenShift Merge Robot2022-08-11
|\ \ \ | | | | | | | | [CI:DOCS] docs: specify `git` protocol is not supported for github hosted repo
| * | | docs: specify git protocol is not supported for github hosted repoAditya R2022-08-11
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Build from URL does not supports `git://` is source is hosted on Github. Reason: https://github.blog/2021-09-01-improving-git-protocol-security-github/ [CI:DOCS] [NO NEW TESTS NEEDED] [NO TESTS NEEDED] Similar to: https://github.com/containers/buildah/pull/4179 Signed-off-by: Aditya R <arajan@redhat.com>
* | | Merge pull request #15285 from YoitoFes/fix-15210OpenShift Merge Robot2022-08-11
|\ \ \ | | | | | | | | remote manifest push: show copy progress
| * | | remote manifest push: show copy progressNaoto Kobayashi2022-08-11
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | `podman-remote manifest push` has shown absolutely no progress at all. Fix that by doing the same as the remote-push code does. Like remote-push, `quiet` parameter is true by default for backwards compatibility. Signed-off-by: Naoto Kobayashi <naoto.kobayashi4c@gmail.com>
* | | Merge pull request #15157 from cdoern/apiLogOpenShift Merge Robot2022-08-11
|\ \ \ | |_|/ |/| | fix LogConfig type for libpod API
| * | fix LogConfig type for libpod APICharlie Doern2022-08-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [NO NEW TESTS NEEDED] our native API was consuming the docker compat type for the API since the two have the exact same name. Fix this by renaming LogConfig to LogConfigLibpod resolves #15138 Signed-off-by: Charlie Doern <cdoern@redhat.com>
* | | Merge pull request #15248 from vrothberg/RUN-1606OpenShift Merge Robot2022-08-11
|\ \ \ | | | | | | | | kube play: sd-notify integration
| * | | kube play: sd-notify integrationValentin Rothberg2022-08-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Integrate sd-notify policies into `kube play`. The policies can be configured for all contianers via the `io.containers.sdnotify` annotation or for indidivual containers via the `io.containers.sdnotify/$name` annotation. The `kube play` process will wait for all containers to be ready by waiting for the individual `READY=1` messages which are received via the `pkg/systemd/notifyproxy` proxy mechanism. Also update the simple "container" sd-notify test as it did not fully test the expected behavior which became obvious when adding the new tests. Signed-off-by: Valentin Rothberg <vrothberg@redhat.com>