Commit graph

25936 commits

Author SHA1 Message Date
Paul Holzinger
39750faab3
test/system: fix test race in exec leak check
On very slow systems it can be that it takes over 5s after the sleep
process was started and until the find_exec_pid_files function finds the
file. This was observed on a ppc64le machine by Red Hat QE.

Just making the sleep longer should fix that problem and it doesn't
really effect the total test time because we stop the container
afterwards so there is no extra delay added with this either.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-16 14:02:38 +02:00
openshift-merge-bot[bot]
9ac8845575
Merge pull request #27077 from mohanboddu/first_contrib_cert_update_action
Update github action to use pull_request_target
2025-09-16 11:49:31 +00:00
openshift-merge-bot[bot]
3381205ccb
Merge pull request #27061 from Luap99/bindings-upgrade
pkg/bindings: use HTTP 101 upgrade request for attach
2025-09-16 08:40:54 +00:00
openshift-merge-bot[bot]
e95fd4f38e
Merge pull request #27039 from Luap99/golangci-cache
cirrus: fix golangci-lint cache leak + update freebsd version
2025-09-16 07:30:48 +00:00
Mohan Boddu
b8258a53dd Update github action to use pull_request_target
pull_request doesn't have access to the secret, pull_request_target
does have access to the secrets

Signed-off-by: Mohan Boddu <mboddu@redhat.com>
2025-09-15 17:12:10 -04:00
openshift-merge-bot[bot]
2e15251436
Merge pull request #27072 from Honny1/update-doc
docs: remove remote limitation note for --build-context option
2025-09-15 14:26:59 +00:00
Jan Rodák
ab5e400a9a
docs: remove remote limitation note for --build-context option
- Removal of a note in the build-context documentation about remote Podman client limitations
- Removal of skip statements for build-context tests in the test suite

Pull request #26628 adds support for --build-context for the remote client.

Signed-off-by: Jan Rodák <hony.com@seznam.cz>
2025-09-15 15:50:02 +02:00
openshift-merge-bot[bot]
27c4fb6323
Merge pull request #27063 from vanou/fix_typo_in_chrootdirs_option_doc
fix: Correct typo in chrootdirs option
2025-09-15 07:07:18 +00:00
Vanou Ishii
8d06a9e9f7 fix: Correct typo in chrootdirs option
In explanation of chrootdirs option, leading / is dropped
from podman managed file path (/etc/hostname). So this PR
adds leading /.

Signed-off-by: Vanou Ishii <ishii.vanou@fujitsu.com>
2025-09-13 14:09:04 +09:00
Paul Holzinger
2702156bd7
pkg/bindings: on terminal attach always wait for stdout to be done
Do not exit early just because stdin copy was done.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-12 19:13:47 +02:00
Paul Holzinger
7ff07b6e0c
pkg/bindings: use HTTP 101 upgrade request for attach
For exec and attach use an upgrade request which the server responds
with HTTP 101 status. Since go 1.12 the Body can be casted to an
io.Writer and then use that to write to the server.

This does however not allow us to skip the ugly hack of overwriting
the default dialContext() because the ReadWriterCloser on the body
does not allow us to call CloseWrite() which is critical to correctly
close the stdin side. So we still have to extract the underlying
net.Conn for that.

Using the cast response.Body is important because the underlying http
lib reads from the socket to parse the header and response code of
course and it is possible that it read more content than needed that is
kept in its buffer but because we then only directly read from the
connection it was possible we miss the first few bytes of the message.

This should fix the issue and hopefully also some long standing
podman-remote missing output flakes in CI.

Fixes: #26951

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-12 19:13:34 +02:00
openshift-merge-bot[bot]
098f1eb534
Merge pull request #27055 from Luap99/revert-quadlet-docs
Revert quadlet docs rewrite
2025-09-11 17:33:50 +00:00
Paul Holzinger
070d7c3ad3
Revert "Rewrite the Quadlet documentation."
This reverts commit c12b1b32bc.

The content contains incorrect information and misses a lot of details
from the previous page that must be restored.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-11 19:00:19 +02:00
Paul Holzinger
bb422c8372
Revert "Change the syntax to not depend on jinja2."
This reverts commit 9de737bf29.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-11 19:00:17 +02:00
Paul Holzinger
fd60d63bf4
Revert "Deduplicate more options."
This reverts commit 6756eb3412.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-11 19:00:02 +02:00
Paul Holzinger
4ae8e386ef
Revert "docs: restore podman-systemd.unit.5"
This reverts commit cab3c6de6d.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-11 18:59:11 +02:00
openshift-merge-bot[bot]
998c9d8f6a
Merge pull request #27014 from SquallATF/patch-1
fix(emulation): avoid nil pointer dereference in registeredBinfmtMisc
2025-09-11 10:29:48 +00:00
openshift-merge-bot[bot]
1ea1958a2d
Merge pull request #27048 from containers/renovate/actions-setup-node-5.x
[skip-ci] Update actions/setup-node action to v5
2025-09-11 10:10:39 +00:00
openshift-merge-bot[bot]
84cbe0bbb6
Merge pull request #27047 from containers/renovate/actions-github-script-8.x
[skip-ci] Update actions/github-script action to v8
2025-09-11 10:07:50 +00:00
Peiyuan Song
04af9ae3fc fix(emulation): handle fs.ErrNotExist in registeredBinfmtMisc
When `/proc/sys/fs/binfmt_misc` is not mounted, filepath.WalkDir may return
fs.ErrNotExist errors. These should be handled gracefully and return nil
instead of causing a panic.

Signed-off-by: Peiyuan Song <squallatf@gmail.com>
2025-09-11 16:56:13 +08:00
renovate[bot]
80f1d48d44
[skip-ci] Update actions/setup-node action to v5
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2025-09-10 21:03:12 +00:00
openshift-merge-bot[bot]
da88a5b097
Merge pull request #27046 from eriksjolund/troubleshooting-lsetxattr
troubleshooting: document lsetxattr error
2025-09-10 19:45:49 +00:00
renovate[bot]
aa8bbbb1ff
[skip-ci] Update actions/github-script action to v8
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2025-09-10 18:36:42 +00:00
Erik Sjölund
796be1e4bf
troubleshooting: document lsetxattr error
Signed-off-by: Erik Sjölund <erik.sjolund@gmail.com>
2025-09-10 20:32:34 +02:00
openshift-merge-bot[bot]
05ff9a561f
Merge pull request #27045 from containers/renovate/actions-checkout-5.x
[skip-ci] Update actions/checkout action to v5
2025-09-10 17:46:43 +00:00
renovate[bot]
e1c1961d48
[skip-ci] Update actions/checkout action to v5
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2025-09-10 17:04:48 +00:00
openshift-merge-bot[bot]
9f7c6c3821
Merge pull request #27017 from Luap99/go-1.24
Update to go 1.24
2025-09-10 16:52:16 +00:00
openshift-merge-bot[bot]
fa10748df8
Merge pull request #26916 from mohanboddu/first_contrib_cert
Adding github action to generate a badge for first time contributor
2025-09-10 14:58:56 +00:00
Paul Holzinger
43a294fa94
vendor: update golang.org/x packages
They require go 1.24 and now that we bumped it we can update them. Seem
easier to do it her eonce than having to wait for renovate to update
each individually.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:05 +02:00
Paul Holzinger
4417e6269c
use maps.Clone() over Copy() when possible
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:05 +02:00
Paul Holzinger
637de6022f
libpod/oci_conmon_linux.go: fix false postive in linter
staticcheck claims the var value is not used but we use it for the
mount.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:05 +02:00
Paul Holzinger
9e4fad8d06
pkg/systemd/quadletL silence one lint warning
t.Chdir() is not an exact replacement here

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:05 +02:00
Paul Holzinger
78e5a521b0
inline some conditionals
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:04 +02:00
Paul Holzinger
8631032556
run modernize -fix ./...
Using golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize

+ some manual cleanup in libpod/lock/shm/shm_lock_test.go as it
  generated an unused variable
+ restored one removed comment

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:04 +02:00
Paul Holzinger
dc5a791f58
use bytes.SplitSeq where possible
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:04 +02:00
Paul Holzinger
b97525a78d
use strings.SplitSeq where possible
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:04 +02:00
Paul Holzinger
4e3e9bfb70
update to go 1.24
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 16:17:03 +02:00
openshift-merge-bot[bot]
906f87b985
Merge pull request #27030 from Luap99/new-images
New VM images 2025-09-10
2025-09-10 14:14:58 +00:00
openshift-merge-bot[bot]
e03a5d2ca8
Merge pull request #26985 from mheon/add_design_doc_conmonv3
Add a directory for design documents
2025-09-10 13:38:39 +00:00
Paul Holzinger
8537afca66
test/e2e: fix podman run default mask test with crun 1.24
The ls call for the directory now fails with EACCES, see
https://github.com/containers/crun/issues/1876

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 14:52:09 +02:00
Paul Holzinger
3bb32d846b
test/system: skip noswap memory mounts correctly
On debian with an older we get this error instead:
fsconfig() failed: tmpfs: Unknown parameter 'noswap'

So handle that case as well to skip the test correctly.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 14:52:08 +02:00
Paul Holzinger
52fcdcf64c
test/system: skip idmapped mount correctly
The full error message is:
failed to create idmapped mount: mount_setattr /tmp/CI_SKPI/podman_bats.RX6gD9/rootfs: invalid argument

So in order to match this account for the variable part.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 14:52:08 +02:00
Paul Holzinger
bd7aada776
cirrus: update to freebsd 14.3
Freebsd 13.4 is EOL so update to the latest one.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 14:33:39 +02:00
Paul Holzinger
710e216390
cirrus: fix golangci-lint cache leak
Do not use reupload_on_changes, this will make the cache grow unbound
and I have seen the cache become so large then restoring it and
uploading it took several minutes thus making the task time worse
compared to no cache. I manually cleaned the cache a few times to fix
this but it need to properly be fixed here.

To not have a stale cache for to long also use date +%U which will
create a new cache once a week. This is in line with the offical
golangci-lint github action which invalidates the cache every 7 days by
default[1].

[1] https://github.com/golangci/golangci-lint-action/blob/main/README.md#cache-invalidation-interval

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 14:23:51 +02:00
Paul Holzinger
a720139dd8
New VM images 2025-09-10
https://github.com/containers/automation_images/pull/416

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
2025-09-10 12:55:11 +02:00
openshift-merge-bot[bot]
cf06546b7f
Merge pull request #26877 from Honny1/build-speedup
refactor: Modularize binding build functions
2025-09-10 10:43:47 +00:00
Matt Heon
5b10b51788 Add a directory for design documents
Add a new directory, which I'm currently dubbing "Features", in
which will live design documents - descriptions of Podman
features that will be implemented or are being implemented.
Add a README and template to this directory to make the purpose
clear and enable easy addition of new design documents.

Signed-off-by: Matt Heon <matthew.heon@pm.me>
2025-09-09 18:12:38 -04:00
openshift-merge-bot[bot]
737108ba04
Merge pull request #27031 from Luap99/makefile-helper-dir
Makefile: restore HELPER_BINARIES_DIR
2025-09-09 19:54:10 +00:00
openshift-merge-bot[bot]
c8854f290e
Merge pull request #27027 from ygalblum/quadlet-build-file-spec
Quadlet build - consider File path that starts with a systemd specifier as absolute
2025-09-09 19:09:35 +00:00
Ygal Blum
936a01e088 Quadlet build - consider File path that starts with a systemd specifier as absolute
Fixes https://github.com/containers/podman/issues/26746

Signed-off-by: Ygal Blum <ygal.blum@gmail.com>
2025-09-09 13:57:36 -04:00