Skip to content

Commit

Permalink
Merge pull request #16525 from Luap99/CI-update-image
Browse files Browse the repository at this point in the history
update CI images to include pasta
  • Loading branch information
openshift-merge-robot authored Jan 19, 2023
2 parents 29a90c3 + 7be8ff5 commit 7689e26
Show file tree
Hide file tree
Showing 7 changed files with 13 additions and 13 deletions.
5 changes: 2 additions & 3 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ env:
UBUNTU_NAME: "ubuntu-2204"

# Image identifiers
IMAGE_SUFFIX: "c5069932136759296"
IMAGE_SUFFIX: "c6447802205601792"
# EC2 images
FEDORA_AMI: "fedora-aws-${IMAGE_SUFFIX}"
FEDORA_AARCH64_AMI: "fedora-podman-aws-arm64-${IMAGE_SUFFIX}"
Expand All @@ -44,8 +44,7 @@ env:
# Container FQIN's
FEDORA_CONTAINER_FQIN: "quay.io/libpod/fedora_podman:${IMAGE_SUFFIX}"
PRIOR_FEDORA_CONTAINER_FQIN: "quay.io/libpod/prior-fedora_podman:${IMAGE_SUFFIX}"
# FIXME, replace override with common suffix once everything is in sync
WINDOWS_AMI: "win-server-wsl-c6447802205601792"
WINDOWS_AMI: "win-server-wsl-${IMAGE_SUFFIX}"
####
#### Control variables that determine what to run and how to run it.
#### N/B: Required ALL of these are set for every single task.
Expand Down
6 changes: 5 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -571,6 +571,10 @@ ginkgo:
ginkgo-remote:
$(MAKE) ginkgo-run TAGS="$(REMOTETAGS) remote_testing" HACK=

.PHONY: testbindings
testbindings: .install.ginkgo
ACK_GINKGO_RC=true $(GINKGO) -v $(TESTFLAGS) -tags "$(TAGS) remote" $(GINKGOTIMEOUT) -progress -trace -noColor -debug -timeout 30m -v -r ./pkg/bindings/test

.PHONY: localintegration
localintegration: test-binaries ginkgo

Expand Down Expand Up @@ -901,7 +905,7 @@ install.systemd:
endif

.PHONY: install.tools
install.tools: .install.golangci-lint .install.swagger ## Install needed tools
install.tools: .install.golangci-lint ## Install needed tools
$(MAKE) -C test/tools

.PHONY: .install.goimports
Expand Down
3 changes: 2 additions & 1 deletion contrib/cirrus/lib.sh
Original file line number Diff line number Diff line change
Expand Up @@ -219,7 +219,8 @@ use_cni() {
#
# FIXME FIXME FIXME: if/when we bring back Ubuntu (or use Debian),
# someone will have to conditionalize these rpm/dnf commands
rpm -e --nodeps netavark aardvark-dns
# Do not fail when netavark and aardvark-dns are not installed.
rpm -e --nodeps netavark aardvark-dns || true
msg "Installing default CNI configuration"
dnf install -y $PACKAGE_DOWNLOAD_DIR/podman-plugins*
cd $GOSRC || exit 1
Expand Down
1 change: 1 addition & 0 deletions contrib/cirrus/logcollector.sh
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ case $1 in
container-selinux
libseccomp
netavark
passt
)
;;
ubuntu)
Expand Down
7 changes: 2 additions & 5 deletions contrib/cirrus/runner.sh
Original file line number Diff line number Diff line change
Expand Up @@ -101,11 +101,8 @@ function _run_bindings() {
gitcommit_magic="/define.gitCommit=${GIT_COMMIT}"
fi

# Subshell needed so logformatter will write output in cwd; if it runs in
# the subdir, .cirrus.yml will not find the html'ized log
(cd pkg/bindings/test && \
echo "$gitcommit_magic" && \
ginkgo -progress -trace -noColor -debug -timeout 30m -r -v) |& logformatter
(echo "$gitcommit_magic" && \
make testbindings) |& logformatter
}

function _run_docker-py() {
Expand Down
3 changes: 0 additions & 3 deletions test/e2e/run_networking_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ var _ = Describe("Podman run networking", func() {
})

It("podman verify network scoped DNS server and also verify updating network dns server", func() {
// TODO: Unskip after https://github.com/containers/podman/pull/16525
Skip("TODO: unskip after https://github.com/containers/podman/pull/16525")
// Following test is only functional with netavark and aardvark
SkipIfCNI(podmanTest)
net := createNetworkName("IntTest")
Expand Down Expand Up @@ -90,7 +88,6 @@ var _ = Describe("Podman run networking", func() {
session.WaitWithDefaultTimeout()
Expect(session).Should(Exit(1))
Expect(session.OutputToString()).To(ContainSubstring(";; connection timed out; no servers could be reached"))

})

It("podman run network connection with default bridge", func() {
Expand Down
1 change: 1 addition & 0 deletions test/system/505-networking-pasta.bats
Original file line number Diff line number Diff line change
Expand Up @@ -335,6 +335,7 @@ function teardown() {
}

@test "podman networking with pasta(1) - Local forwarder, IPv4" {
skip "FIXME: #17074: some pasta dns problem"
skip_if_no_ipv4 "IPv4 not routable on the host"

run_podman run --dns 198.51.100.1 \
Expand Down

0 comments on commit 7689e26

Please sign in to comment.