22400 Commits

Author SHA1 Message Date
5040c6e390 Makefile: remove useless HACK variable in e2e test
This doesn't do anything and I was unable to find anything in the git
history when or why it was added. The HACK var is just adding another
directory to the ginkgo argument list. As the only arg used was the
"./hack" dir which does not contain any go files it does nothing
besides confusing me about its purpose.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-26 14:36:20 +02:00
537c21a49a test/e2e: fix volumes and suid/dev/exec options
When the source dir is already mounted noexec, nodev or nosuid then a
rootless user cannot mount the dir into the container without these
options for obvious reasons.

So in order to run the test we must ensure the dir is mounted with these
options first, if they are simply skip as the test will fail otherwise.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-26 14:36:20 +02:00
31034a1b6f test/e2e: volumes and suid/dev/exec options works remote
--volume works fine with podman-remote you just need to know that the
path is server side.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-26 14:36:20 +02:00
cc6e999f2a test/e2e: fix limits test
When you run locally with a higher oom_score_adj then the one used in
the test podman will print a warning and not set the oom lower then the
current value. Thus use 999 as value which should only cause problems
for users with oom_score_adj value of 1000 (max value) which seems
unlikely.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-26 14:36:19 +02:00
ebc48503a8 Merge pull request #22512 from containers/renovate/github.com-rootless-containers-rootlesskit-v2-2.x
Update module github.com/rootless-containers/rootlesskit/v2 to v2.1.0
2024-04-26 12:24:15 +00:00
3927a52ece Merge pull request #22509 from sanmai-NL/Clarify-network-options-docs
Correct network option name `ip` -> `ip6`
2024-04-26 12:21:30 +00:00
195d10011c Merge pull request #22481 from findnature/main
chore: fix function names in comment
2024-04-26 12:18:46 +00:00
abbfdb3622 Merge branch 'containers:main' into main 2024-04-26 10:07:05 +08:00
c2d7b5415f Update module github.com/rootless-containers/rootlesskit/v2 to v2.1.0
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2024-04-26 02:06:10 +00:00
3c1c6a44ef Correct option name ip -> ip6
Also, properly capitalize.

Signed-off-by: Sander Maijers <3374183+sanmai-NL@users.noreply.github.com>
2024-04-25 22:17:06 +02:00
15cc8862c1 Merge pull request #22410 from mheon/automount_images_k8s
Add the ability to automount images as volumes via play
2024-04-25 19:26:47 +00:00
30e2c923d6 Add the ability to automount images as volumes via play
Effectively, this is an ability to take an image already pulled
to the system, and automatically mount it into one or more
containers defined in Kubernetes YAML accepted by `podman play`.

Requirements:
- The image must already exist in storage.
- The image must have at least 1 volume directive.
- The path given by the volume directive will be mounted from the
  image into the container. For example, an image with a volume
  at `/test/test_dir` will have `/test/test_dir` in the image
  mounted to `/test/test_dir` in the container.
- Multiple images can be specified. If multiple images have a
  volume at a specific path, the last image specified trumps.
- The images are always mounted read-only.
- Images to mount are defined in the annotation
  "io.podman.annotations.kube.image.automount/$ctrname" as a
  semicolon-separated list. They are mounted into a single
  container in the pod, not the whole pod.

As we're using a nonstandard annotation, this is Podman only, any
Kubernetes install will just ignore this.

Underneath, this compiles down to an image volume
(`podman run --mount type=image,...`) with subpaths to specify
what bits we want to mount into the container.

Signed-off-by: Matt Heon <mheon@redhat.com>
2024-04-25 14:12:27 -04:00
693ae0ebc6 Add support for image volume subpaths
Image volumes (the `--mount type=image,...` kind, not the
`podman volume create --driver image ...` kind - it's strange
that we have two) are needed for our automount scheme, but the
request is that we mount only specific subpaths from the image
into the container. To do that, we need image volume subpath
support. Not that difficult code-wise, mostly just plumbing.

Also, add support to the CLI; not strictly necessary, but it
doesn't hurt anything and will make testing easier.

Signed-off-by: Matt Heon <mheon@redhat.com>
2024-04-25 14:12:27 -04:00
273020160c Merge pull request #22504 from mheon/bump_buildah_fix_ci
Bump buildah to fix CI
2024-04-25 18:06:20 +00:00
0ccee4e51f Merge pull request #22489 from edsantiago/exitwitherror-yetmorelowhangingfruit
ExitWithError() - yet more low-hanging fruit
2024-04-25 18:03:36 +00:00
5f57571d65 Bump Buildah to latest main
Includes fixes from Ed's treadmill script.

Signed-off-by: Matt Heon <mheon@redhat.com>
2024-04-25 12:17:43 -04:00
e8a9c58125 Update Makefile to Go 1.22 for in-container
This unbreaks make vendor-in-container now that we have the
`toolchain` line in go.mod.

Signed-off-by: Matt Heon <mheon@redhat.com>
2024-04-25 12:12:48 -04:00
1526ebf095 Merge pull request #22461 from containers/renovate/github.com-docker-docker-26.x
fix(deps): update module github.com/docker/docker to v26.1.0+incompatible
2024-04-24 15:42:10 +00:00
e4c9910aec ExitWithError() - yet more low-hanging fruit
Followup to [1]#22270: wherever possible/practical, extend command
error checks to include explicit exit status codes and error strings.

Just trying to shrink down #22346 to a manageable, reviewable size.

Signed-off-by: Ed Santiago <santiago@redhat.com>
2024-04-24 09:04:50 -06:00
1a23451067 Merge pull request #22486 from edsantiago/exitwitherror-morelowhangingfruit
ExitWithError() - more low-hanging fruit
2024-04-24 14:58:22 +00:00
a7c61c31a3 Merge pull request #22465 from Luap99/go1.21
Update to go 1.21
2024-04-24 14:30:59 +00:00
3b212957d5 Merge pull request #22485 from edsantiago/exitwitherror-lowhangingfruit
ExitWithError() - low-hanging fruit
2024-04-24 13:27:27 +00:00
80c3a22440 ExitWithError() - more low-hanging fruit
Followup to #22270: wherever possible/practical, extend command
error checks to include explicit exit status codes and error strings.

More low-hanging fruit: small reviewable chunks

Signed-off-by: Ed Santiago <santiago@redhat.com>
2024-04-24 07:11:40 -06:00
4da5c5d3aa ExitWithError() - low-hanging fruit
Followup to #22270: wherever possible/practical, extend command
error checks to include explicit exit status codes and error strings.

Because #22346 is stalled, these are some trivial easy-to-review
changes that get us closer to the goal.

Signed-off-by: Ed Santiago <santiago@redhat.com>
2024-04-24 05:44:40 -06:00
c681df35c0 chore: fix function names in comment
Signed-off-by: findnature <cricis@aliyun.com>
2024-04-24 12:07:38 +08:00
f82c7cefef Merge pull request #22468 from grisu48/contributing
[CI:DOCS] Remove redundant Prerequisite before build section
2024-04-23 21:34:37 +00:00
5e1c3f58dc Merge pull request #22458 from ashley-cui/artifacts
[CI:DOCS] Fix artifact action
2024-04-23 20:45:08 +00:00
3f78c39c50 Remove redundant Prerequisite before build section
The contents of the "Prerequisite before buil"d section are given already
in a link in its "Prepare your environment" parent section and therefore
redundant. Let's remove it.

Signed-off-by: phoenix <felix.niederwanger@suse.com>
2024-04-23 14:48:46 +02:00
35c2c82df2 Merge pull request #22464 from grisu48/opensuse
[CI:DOCS] Add installation instructions for openSUSE
2024-04-23 12:35:36 +00:00
fa1fd1646e Remove PKG_CONFIG_PATH
Remove the export PKG_CONFIG_PATH which appears not necessary anymore.

Signed-off-by: phoenix <felix.niederwanger@suse.com>
2024-04-23 11:59:09 +02:00
3b28cdfb20 Add installation instructions for openSUSE
Adds the installation instructions for openSUSE, similar to the present
ones for Fedora and Debian/Ubuntu.

Tested on openSUSE Tumbleweed and Leap 15.6.

Signed-off-by: phoenix <felix.niederwanger@suse.com>
2024-04-23 11:57:57 +02:00
83dbbc3a51 Replace golang.org/x/exp/slices with slices from std
Use "slices" from the standard library, this package was added in go
1.21 so we can use it now.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-23 11:16:40 +02:00
c04d03ef8c Update to go 1.21
So that we can continue updating our dependencies.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-23 11:07:28 +02:00
6bab09c51d Merge pull request #22457 from lsm5/remove-el8
[skip-ci] Packit/rpm: remove el8 jobs and spec conditionals
2024-04-23 10:17:27 +02:00
c31728ca57 fix(deps): update module github.com/docker/docker to v26.1.0+incompatible
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2024-04-22 21:47:26 +00:00
c22eefb939 [CI:DOCS] Fix artifact action
Fix a bug where the check uploads section didn't actually mark the os/arch to be built.

Signed-off-by: Ashley Cui <acui@redhat.com>
2024-04-22 15:23:52 -04:00
333682132a [skip-ci] Packit/rpm: remove el8 jobs and spec conditionals
EL8 will go EOL soon and nothing on main will be shipped to el8.

Signed-off-by: Lokesh Mandvekar <lsm5@redhat.com>
2024-04-22 11:54:19 -04:00
7986eeaf6a Merge pull request #22429 from lsm5/centos10
[skip-ci] Packit: enable CentOS 10 Stream build jobs
2024-04-22 11:27:02 -04:00
318437f7b5 Merge pull request #22455 from edsantiago/e2e-stop-littering
e2e tests: stop littering
2024-04-22 14:31:09 +00:00
6d347927a6 Merge pull request #22443 from Luap99/podman-healthcheck-events
add containers.conf healthcheck_events support
2024-04-22 13:49:52 +00:00
96331d40d9 Merge pull request #22442 from Luap99/store-error
libpod: wrap store setup error message
2024-04-22 13:47:08 +00:00
ac04cb4ac5 e2e tests: stop littering
"tmpdir + string" does not do what you think it does.

Signed-off-by: Ed Santiago <santiago@redhat.com>
2024-04-22 06:43:19 -06:00
ac963a7036 Merge pull request #22453 from MHBauer/format-docs-example
[CI:DOCS] format podman-pull example as code
2024-04-22 10:16:48 +00:00
c2cadfb5c5 Merge pull request #22322 from mheon/update_the_config
Make `podman update` changes persistent
2024-04-22 07:50:48 +00:00
27174ff3fb [CI:DOCS] format podman-pull example as code
Signed-off-by: Morgan Bauer <bauer.morgan@gmail.com>
2024-04-21 19:46:39 -07:00
3c15aa3724 Merge pull request #22446 from ashley-cui/artifacts
[CI:DOCS] Build & upload release artifacts with GitHub Actions
2024-04-19 23:03:18 +00:00
e7f0c1947e [CI:DOCS] Build & upload release artifacts with GitHub Actions
Add a new GitHub Action that builds and uploads release artifacts. This action is triggered by publishing a release on GitHub. The action will only build if the specfic artifact is missing.

This action also triggers the Windows installer action, since the Windows installer action depends on an uploaded artifact.

Note that the action runs on ubuntu-22.04

Signed-off-by: Ashley Cui <acui@redhat.com>
2024-04-19 14:16:23 -04:00
2681ab23d1 libpod: getHealthCheckLog() remove unessesary check
Checking if the file exists before opening it anyway is really pointless
and needs a extra syscall and in theory is racy as the file might have
been changed between the two calls. We can simply ignore the ENOENT
error on the ReadFile call.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-19 17:32:55 +02:00
2ae6d0d4dd add containers.conf healthcheck_events support
When the field is set to false we should never log healthcheck events.

Fixes https://issues.redhat.com/browse/RHEL-18987

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-19 17:29:48 +02:00
33def2eea7 vendor latest c/common
Includes the healthcheck_events field.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2024-04-19 17:29:48 +02:00