Skip to content

Commit

Permalink
DO NOT MERGE: buildah vendor treadmill
Browse files Browse the repository at this point in the history
As you run --sync, please update this commit message with your
actual changes.

Changes since 2024-11-02:

  * skip_if_remote new mount-secret-env test (buildah 5185):
    it uses environment variables, which cannot work remote.
    This test is only run rootless, so we never see failures
    in podman PRs.

  * skip_if_remote a buildah-prune-cache test. Used to work
    fine, but stopped working after #5804 when Ed added
    an explicit TMPDIR= for enabling tests in parallel

  * (minor) one-line change to buildah-tests.diff , adjusting
    for new INET helper

Signed-off-by: Ed Santiago <[email protected]>
  • Loading branch information
edsantiago committed Nov 11, 2024
1 parent ea641b1 commit 471b6c1
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 10 deletions.
6 changes: 5 additions & 1 deletion test/buildah-bud/apply-podman-deltas
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,8 @@ skip_if_remote "--output option not implemented in podman-remote" \
skip_if_remote "logfile not implemented on remote" "bud-logfile-with-split-logfile-by-platform"

skip_if_remote "envariables do not automatically work with -remote." \
"build proxy"
"build proxy" \
"build-test --mount=type=secret test from env with chroot isolation"

# 2022-07-04 this is a new test in buildah; it's failing in treadmill
skip_if_remote "FIXME FIXME FIXME: does this test make sense in remote?" \
Expand All @@ -260,6 +261,9 @@ skip_if_remote "--events-backend does not work with podman-remote" \
skip_if_remote "--cert-dir option not working via remote and retry warnings are printed on the server" \
"build add https retry ca"

skip_if_remote "Weird. This used to work remote, until Ed set TMPDIR in #5804" \
"bud-with-mount-cache-like-buildkit with buildah prune should clear the cache"

###############################################################################
# BEGIN tests which are skipped due to actual podman or podman-remote bugs.

Expand Down
18 changes: 9 additions & 9 deletions test/buildah-bud/buildah-tests.diff
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
From e39e00a6d92acb83283705ea66c26261ca6a0170 Mon Sep 17 00:00:00 2001
From d7839d93860915b2a43c486d0fed89fee7313ec0 Mon Sep 17 00:00:00 2001
From: Ed Santiago <[email protected]>
Date: Thu, 6 Oct 2022 17:32:59 -0600
Subject: [PATCH] tweaks for running buildah tests under podman
Expand All @@ -9,10 +9,10 @@ Signed-off-by: Ed Santiago <[email protected]>
1 file changed, 115 insertions(+), 4 deletions(-)

diff --git a/tests/helpers.bash b/tests/helpers.bash
index 20f0787ff..c356500ae 100644
index f8ab624a8..0d8f5ce69 100644
--- a/tests/helpers.bash
+++ b/tests/helpers.bash
@@ -79,6 +79,38 @@ EOF
@@ -80,6 +80,38 @@ EOF
BUILDAH_REGISTRY_OPTS="${regconfopt} ${regconfdir} --short-name-alias-conf ${TEST_SCRATCH_DIR}/cache/shortnames.conf"
COPY_REGISTRY_OPTS="${BUILDAH_REGISTRY_OPTS}"
PODMAN_REGISTRY_OPTS="${regconfopt}"
Expand Down Expand Up @@ -51,7 +51,7 @@ index 20f0787ff..c356500ae 100644
}

function starthttpd() { # directory [working-directory-or-"" [certfile, keyfile]]
@@ -143,6 +175,32 @@ function teardown_tests() {
@@ -144,6 +176,32 @@ function teardown_tests() {
stop_git_daemon
stop_registry

Expand Down Expand Up @@ -84,7 +84,7 @@ index 20f0787ff..c356500ae 100644
# Workaround for #1991 - buildah + overlayfs leaks mount points.
# Many tests leave behind /var/tmp/.../root/overlay and sub-mounts;
# let's find those and clean them up, otherwise 'rm -rf' fails.
@@ -232,7 +290,12 @@ function copy() {
@@ -252,7 +310,12 @@ function copy() {
}

function podman() {
Expand All @@ -98,7 +98,7 @@ index 20f0787ff..c356500ae 100644
}

# There are various scenarios where we would like to execute `tests` as rootless user, however certain commands like `buildah mount`
@@ -296,8 +359,36 @@ function run_buildah() {
@@ -316,8 +379,36 @@ function run_buildah() {
--retry) retry=3; shift;; # retry network flakes
esac

Expand Down Expand Up @@ -136,7 +136,7 @@ index 20f0787ff..c356500ae 100644

# If session is rootless and `buildah mount` is invoked, perform unshare,
# since normal user cannot mount a filesystem unless they're in a user namespace along with its own mount namespace.
@@ -311,8 +402,8 @@ function run_buildah() {
@@ -331,8 +422,8 @@ function run_buildah() {
retry=$(( retry - 1 ))

# stdout is only emitted upon error; this echo is to help a debugger
Expand All @@ -147,7 +147,7 @@ index 20f0787ff..c356500ae 100644
# without "quotes", multiple lines are glommed together into one
if [ -n "$output" ]; then
echo "$output"
@@ -673,6 +764,26 @@ function skip_if_no_unshare() {
@@ -693,6 +784,26 @@ function skip_if_no_unshare() {
fi
}

Expand Down Expand Up @@ -175,5 +175,5 @@ index 20f0787ff..c356500ae 100644
daemondir=${TEST_SCRATCH_DIR}/git-daemon
mkdir -p ${daemondir}/repo
--
2.45.2
2.47.0

0 comments on commit 471b6c1

Please sign in to comment.