Skip to content

Commit b24220b

Browse files
committed
Revert "test/e2e: try debug potential pasta issue"
This reverts commit f517e52. The issue #24219 has been fixed a long time ago and this no longer flakes so we do not need to run with debug logs all the time. Signed-off-by: Paul Holzinger <[email protected]>
1 parent 27c4fb6 commit b24220b

File tree

2 files changed

+2
-50
lines changed

2 files changed

+2
-50
lines changed

test/e2e/pull_test.go

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ package integration
44

55
import (
66
"fmt"
7-
"io"
87
"os"
98
"path/filepath"
109
"runtime"
@@ -668,33 +667,13 @@ var _ = Describe("Podman pull", func() {
668667

669668
podmanTest.AddImageToRWStore(ALPINE)
670669

671-
success := false
672-
registryArgs := []string{"run", "-d", "--name", "registry", "-p", "5012:5000"}
673670
if isRootless() {
674671
err := podmanTest.RestoreArtifact(REGISTRY_IMAGE)
675672
Expect(err).ToNot(HaveOccurred())
676-
677-
// Debug code for https://github.com/containers/podman/issues/24219
678-
logFile := filepath.Join(podmanTest.TempDir, "pasta.log")
679-
registryArgs = append(registryArgs, "--network", "pasta:--trace,--log-file,"+logFile)
680-
defer func() {
681-
if success {
682-
// only print the log on errors otherwise it will clutter CI logs way to much
683-
return
684-
}
685-
686-
f, err := os.Open(logFile)
687-
Expect(err).ToNot(HaveOccurred())
688-
defer f.Close()
689-
GinkgoWriter.Println("pasta trace log:")
690-
_, err = io.Copy(GinkgoWriter, f)
691-
Expect(err).ToNot(HaveOccurred())
692-
}()
693673
}
694-
registryArgs = append(registryArgs, REGISTRY_IMAGE, "/entrypoint.sh", "/etc/docker/registry/config.yml")
695674
lock := GetPortLock("5012")
696675
defer lock.Unlock()
697-
session := podmanTest.Podman(registryArgs)
676+
session := podmanTest.Podman([]string{"run", "-d", "--name", "registry", "-p", "5012:5000", REGISTRY_IMAGE, "/entrypoint.sh", "/etc/docker/registry/config.yml"})
698677
session.WaitWithDefaultTimeout()
699678
Expect(session).Should(ExitCleanly())
700679

@@ -707,8 +686,6 @@ var _ = Describe("Podman pull", func() {
707686
session = decryptionTestHelper(imgPath)
708687

709688
Expect(session.LineInOutputContainsTag(imgPath, "latest")).To(BeTrue())
710-
711-
success = true
712689
})
713690
})
714691

test/e2e/run_test.go

Lines changed: 1 addition & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ package integration
44

55
import (
66
"fmt"
7-
"io"
87
"net"
98
"os"
109
"path/filepath"
@@ -2354,31 +2353,9 @@ WORKDIR /madethis`, BB)
23542353

23552354
podmanTest.AddImageToRWStore(ALPINE)
23562355

2357-
success := false
2358-
registryArgs := []string{"run", "-d", "--name", "registry", "-p", "5006:5000"}
2359-
if isRootless() {
2360-
// Debug code for https://github.com/containers/podman/issues/24219
2361-
logFile := filepath.Join(podmanTest.TempDir, "pasta.log")
2362-
registryArgs = append(registryArgs, "--network", "pasta:--trace,--log-file,"+logFile)
2363-
defer func() {
2364-
if success {
2365-
// only print the log on errors otherwise it will clutter CI logs way to much
2366-
return
2367-
}
2368-
2369-
f, err := os.Open(logFile)
2370-
Expect(err).ToNot(HaveOccurred())
2371-
defer f.Close()
2372-
GinkgoWriter.Println("pasta trace log:")
2373-
_, err = io.Copy(GinkgoWriter, f)
2374-
Expect(err).ToNot(HaveOccurred())
2375-
}()
2376-
}
2377-
registryArgs = append(registryArgs, REGISTRY_IMAGE, "/entrypoint.sh", "/etc/docker/registry/config.yml")
2378-
23792356
lock := GetPortLock("5006")
23802357
defer lock.Unlock()
2381-
session := podmanTest.Podman(registryArgs)
2358+
session := podmanTest.Podman([]string{"run", "-d", "--name", "registry", "-p", "5006:5000", REGISTRY_IMAGE, "/entrypoint.sh", "/etc/docker/registry/config.yml"})
23822359
session.WaitWithDefaultTimeout()
23832360
Expect(session).Should(ExitCleanly())
23842361

@@ -2412,8 +2389,6 @@ WORKDIR /madethis`, BB)
24122389
session.WaitWithDefaultTimeout()
24132390
Expect(session).Should(Exit(0))
24142391
Expect(session.ErrorToString()).To(ContainSubstring("Trying to pull " + imgPath))
2415-
2416-
success = true
24172392
})
24182393

24192394
It("podman run --shm-size-systemd", func() {

0 commit comments

Comments
 (0)