aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* libpod: Remove 100msec delay during shutdownAlexander Larsson2022-10-07
| | | | | | | | | | | | | | | | | | | | | | | When shutting down the image engine we always wait for the image even goroutine to finish writing any outstanding events. However, the loop for that always waits 100msec every iteration. This means that (depending on the phase) shutdown is always delayed up to 100msec. This is delaying "podman run" extra much because podman is run twice (once for the run and once as cleanup via a conmon callback). Changing the image loop to exit immediately when a libimageEventsShutdown (but first checking for any outstanding events to write) improves podman run times by about 100msec on average. Note: We can't just block on the event loop reading the shutdown event anymore, we need to wait until it read and processed any outstanding events, so we now send the shutdown event and then block waiting for the channel to be closed by the event loop. [NO NEW TESTS NEEDED] Signed-off-by: Alexander Larsson <alexl@redhat.com>
* Merge pull request #16034 from rhatdan/VENDOROpenShift Merge Robot2022-10-06
|\ | | | | Update vendor of containers/buildah v1.28.0
| * Update vendor of containers/buildah v1.28.0Daniel J Walsh2022-10-03
| | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #15910 from edsantiago/cron_treadmillOpenShift Merge Robot2022-10-06
|\ \ | | | | | | Proof of concept: nightly dependency treadmill
| * | Proof of concept: nightly dependency treadmillEd Santiago2022-09-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed in f2f: this is the cleanest, simplest mechanism I can think of to auto-test the Big Three dependencies: simply run go mod edit immediately after git checkout, then run the entire CI test suite. This differs significantly from the buildah treadmill, in that buildah is almost impossible to re-vendor without manual intervention. (In practice, so are these, but let's dream of a world in which this will run and pass every night). (I want a pony too). Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | Merge pull request #16061 from edsantiago/docs_dedup_timeOpenShift Merge Robot2022-10-06
|\ \ \ | | | | | | | | [CI:DOCS] Man pages: refactor common options: --time
| * | | Man pages: refactor common options: --timeEd Santiago2022-10-05
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Only in container/pod stop/rm/restart man pages; the others (volume-rm, network-rm, system-service) are too different to refactor. Mostly an easy one, no manual reconciliation needed apart from the pod-vs-container difference. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | Merge pull request #16055 from edsantiago/docs_dedup_podOpenShift Merge Robot2022-10-05
|\ \ \ | | | | | | | | [CI:DOCS] Man pages: refactor common options: --pod
| * | | Man pages: refactor common options: --podEd Santiago2022-10-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only between podman-create and -run; the other meanings of --pod are too different. This almost didn't feel worth refactoring, except the podman-run version fixed a word and added a possibly important note about infra containers. I went with the podman-run version. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #16029 from andrei-n-cosma/kube-default-networkOpenShift Merge Robot2022-10-05
|\ \ \ \ | |/ / / |/| | | Add pods created by play kube to a default network
| * | | Add pods created by kube play to a default networkAndrei Natanael Cosma2022-10-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to allow pods to reach other pods (as in Kubernetes) they all need to be added to the same network. A network is created (if it doesn't exist) and pods created by play-kube are added to that network. When network options are passed to kube command the pods are not attached to the default kube network. Signed-off-by: Andrei Natanael Cosma <andrei@intersect.ro>
* | | | Merge pull request #16049 from edsantiago/docs_dedup_featuresOpenShift Merge Robot2022-10-05
|\ \ \ \ | | | | | | | | | | [CI:DOCS] Man pages: refactor common options: --features
| * | | | Man pages: refactor common options: --featuresEd Santiago2022-10-04
| |/ / / | | | | | | | | | | | | | | | | | | | | Easy one: text was already identical between both files. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #16050 from baude/bz2131836OpenShift Merge Robot2022-10-05
|\ \ \ \ | |/ / / |/| | | Prevent nil pointer deref in GetImage
| * | | Prevent nil pointer deref in GetImageBrent Baude2022-10-04
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Trying to print the image id on a failed inspect will result in a nil pointer panic because the image will be nil. Replace image.id with the image name which is defined as a string without the use of inspect. Fixes: bz#2131836 [NO NEW TESTS NEEDED] Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | Merge pull request #16032 from edsantiago/docs_dedup_variantOpenShift Merge Robot2022-10-04
|\ \ \ | | | | | | | | [CI:DOCS] Man pages: refactor common options: --variant
| * | | Man pages: refactor common options: --variantEd Santiago2022-10-03
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Two different texts, split into two .md files. Nontrivial, but still easy to review because the text is unchanged. I was unable to reconcile either version with podman-build, so that file remains with a separate version. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | Merge pull request #16045 from ↵OpenShift Merge Robot2022-10-04
|\ \ \ | | | | | | | | | | | | | | | | containers/dependabot/go_modules/github.com/opencontainers/image-spec-1.1.0-rc2 build(deps): bump github.com/opencontainers/image-spec from 1.1.0-rc1 to 1.1.0-rc2
| * | | build(deps): bump github.com/opencontainers/image-specdependabot[bot]2022-10-04
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [github.com/opencontainers/image-spec](https://github.com/opencontainers/image-spec) from 1.1.0-rc1 to 1.1.0-rc2. - [Release notes](https://github.com/opencontainers/image-spec/releases) - [Changelog](https://github.com/opencontainers/image-spec/blob/main/RELEASES.md) - [Commits](https://github.com/opencontainers/image-spec/compare/v1.1.0-rc1...v1.1.0-rc2) --- updated-dependencies: - dependency-name: github.com/opencontainers/image-spec dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
* | | Merge pull request #16037 from cevich/fix_branch_go_checkOpenShift Merge Robot2022-10-03
|\ \ \ | |/ / |/| | [CI:BUILD] Cirrus: Fix tag & branch go checks failing
| * | Cirrus: Fix tag & branch go checks failingChris Evich2022-10-03
|/ / | | | | | | | | | | | | | | When running on a branch or tag, `req_env_vars()` will call `exit(1)` because `$CIRRUS_PR` is empty (as expected). The original intention was to simply skip language checks on non-PRs. Fix the condition to match. Signed-off-by: Chris Evich <cevich@redhat.com>
* | Merge pull request #16028 from edsantiago/docs_dedup_compression-formatOpenShift Merge Robot2022-10-03
|\ \ | | | | | | [CI:DOCS] Man pages: refactor common options: --compression-format
| * | Man pages: refactor common options: --compression-formatEd Santiago2022-10-02
| | | | | | | | | | | | | | | | | | Easy one: text was already identical across both files. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | Merge pull request #15846 from jakecorrenti/pod-logs-latestOpenShift Merge Robot2022-10-03
|\ \ \ | |/ / |/| | `podman pod logs -l` no longer panics
| * | `podman pod logs -l` no longer panicsJake Correnti2022-09-29
| | | | | | | | | | | | | | | | | | | | | Fixed issue where executing the command `podman pod logs -l` would panic because it was indexing into an empty arguments array. Signed-off-by: Jake Correnti <jcorrenti13@gmail.com>
* | | Merge pull request #16016 from Luap99/netns-cleanupOpenShift Merge Robot2022-10-01
|\ \ \ | | | | | | | | cleanup: always delete netns mount
| * | | cleanup: always delete netns mountPaul Holzinger2022-09-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should not keep the netns if there was a cleanup problem. Deleting the netns will also delete the virtual links inside and thus make the IPs available again for the next use. context: https://github.com/containers/netavark/issues/302 [NO NEW TESTS NEEDED] This is very hard to trigger reliable and it would need to work with cni and netavark. This mostly happens because of specic bugs but those will be fixed and then this test would fail. Signed-off-by: Paul Holzinger <pholzing@redhat.com>
* | | | Merge pull request #15978 from iamkirkbater/add-brew-to-issue-templateOpenShift Merge Robot2022-10-01
|\ \ \ \ | | | | | | | | | | [CI:DOCS] Adds brew info podman to issue template.
| * | | | Adds brew info podman to issue template.Kirk Bater2022-09-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just a quick little addition to provide the command to get the package info from brew for those who might not know. Signed-off-by: Kirk Bater <kirk.bater@gmail.com>
* | | | | Merge pull request #16015 from mtrmac/pgzip-updateOpenShift Merge Robot2022-10-01
|\ \ \ \ \ | |_|/ / / |/| | | | Update for https://github.com/klauspost/pgzip/pull/50
| * | | | Update for https://github.com/klauspost/pgzip/pull/50Miloslav Trmač2022-09-30
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... to fix reads of compressed data by docker-archive: > go get github.com/klauspost/pgzip@master > make vendor Signed-off-by: Miloslav Trmač <mitr@redhat.com>
* | | | Merge pull request #16010 from vrothberg/container-inspectOpenShift Merge Robot2022-09-30
|\ \ \ \ | | | | | | | | | | container inspect: include image digest
| * | | | container inspect: include image digestValentin Rothberg2022-09-30
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include the digest of the image in `podman container inspect`. The image digest is a key information for auditing as it defines the identify of an image. This way, it can be determined whether a container used an image with a given CVE etc. Signed-off-by: Valentin Rothberg <vrothberg@redhat.com>
* | | | Merge pull request #16008 from rhatdan/VENDOROpenShift Merge Robot2022-09-30
|\ \ \ \ | | | | | | | | | | Update vendor containers/(common,image)
| * | | | Update vendor containers/(common,image)Daniel J Walsh2022-09-30
| |/ / / | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #15999 from edsantiago/docs_dedup_sign-passphrase-fileOpenShift Merge Robot2022-09-30
|\ \ \ \ | | | | | | | | | | [CI:DOCS] Man pages: Refactor common options: --sign-passphrase-file
| * | | | Man pages: Refactor common options: --sign-passphrase-fileEd Santiago2022-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Trivial one: no human intervention needed, the man page text was already identical between both files. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | | Merge pull request #16001 from cevich/image_readmeOpenShift Merge Robot2022-09-30
|\ \ \ \ \ | |_|/ / / |/| | | | [CI:DOCS] Add quay-description update reminder
| * | | | [CI:DOCS] Add quay-description update reminderChris Evich2022-09-29
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Chris Evich <cevich@redhat.com>
* | | | | Merge pull request #16005 from sstosh/fix-typoOpenShift Merge Robot2022-09-30
|\ \ \ \ \ | | | | | | | | | | | | Fix typo about e2e tests name
| * | | | | Fix typo about e2e tests nameToshiki Sonoda2022-09-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * podman stop --all -> podman kill --all * podman pause --filter -> podman restart --filter [NO NEW TESTS NEEDED] [NO TESTS NEEDED] Signed-off-by: Toshiki Sonoda <sonoda.toshiki@fujitsu.com>
* | | | | | Merge pull request #15868 from rst0git/podman-run-checkpoint-imgOpenShift Merge Robot2022-09-30
|\ \ \ \ \ \ | |/ / / / / |/| | | | | cmd/podman: add support for checkpoint images
| * | | | | Add test for podman run with checkpoint imageRadostin Stoyanov2022-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `podman run` command has been extended with support for checkpoint images. A checkpoint image contains image files generated by criu that allow to restore the runtime state of containerized applications. This patch adds a test case for this functionality. Signed-off-by: Radostin Stoyanov <radostin@redhat.com>
| * | | | | cmd/podman: add support for checkpoint imagesRadostin Stoyanov2022-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch extends the podman run command with support for checkpoint images. When `podman run` is invoked with an image that contains a checkpoint, it would restore the container from that checkpoint. Example: podman run -d --name looper busybox /bin/sh -c \ 'i=0; while true; do echo $i; i=$(expr $i + 1); sleep 1; done' podman container checkpoint --create-image checkpoint-image-1 looper podman run checkpoint-image-1 Signed-off-by: Radostin Stoyanov <radostin@redhat.com>
| * | | | | restore: make IsCheckpointImage reusableRadostin Stoyanov2022-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Podman allows to store a container checkpoints as an images. This patch makes the check that is used to recognise such checkpoint images reusable by moving it in utils. This functionality will be reused in a subsequent patch to extend the `podman run` command with support for checkpoint images. Signed-off-by: Radostin Stoyanov <radostin@redhat.com>
* | | | | | Merge pull request #15909 from cdoern/restoreOpenShift Merge Robot2022-09-29
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix restore API endpoint
| * | | | | | fix restore API endpointcdoern2022-09-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | restore endpoint was totally ignoring --pod, it was missing from the schema and from query handling on the api handlers side. add support for it here. resolves #15018 Signed-off-by: cdoern <cbdoer23@g.holycross.edu>
* | | | | | | Merge pull request #15782 from cevich/bundle_build_tasksOpenShift Merge Robot2022-09-29
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Cirrus: Task consolidation
| * | | | | | | Cirrus: Combine build and code consistency tasksChris Evich2022-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's conceivable for CI to spend a lot of time testing code which otherwise should be rejected due to quality problems. Previously this was validated in a dedicated task, however a failure would still fail the CI run. Simplify the number of CI tasks by combining the consistency check at the tail-end of the build task. Signed-off-by: Chris Evich <cevich@redhat.com>
| * | | | | | | Cirrus: Combine pre-test checks into build taskChris Evich2022-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, two tasks always ran first, prior to anything else. One to verify network and external-service connectivity. Another to verify certain important `.cirrus.yml` standards are met. However, as the total number of tasks continues to grow, the need to keep these basic checks as dedicated prerequisites is of decreasing value/importance. Fold these two checks into a new `pretesting_script` component of the Fedora `build` task, on both `x86_64` and `aarch64`. Signed-off-by: Chris Evich <cevich@redhat.com>
generated by cgit v1.2.3-54-g00ecf (git 2.39.0) at 2024-10-17 06:27:11 +0000