Skip to content

Commit

Permalink
Merge pull request #73 from Luap99/5.5-main
Browse files Browse the repository at this point in the history
bump main to 5.5
  • Loading branch information
openshift-merge-bot[bot] authored Jan 30, 2025
2 parents 8d11f97 + 4c4c3ef commit dfc8ddd
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 18 deletions.
2 changes: 1 addition & 1 deletion .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ env:
# Vars used for the macos and windows testing
MACHINE_IMAGE_BASE_URL: "https://api.cirrus-ci.com/v1/artifact/build/${CIRRUS_BUILD_ID}/image_build/image/"
# podman version used by windows/macos verify suite
PODMAN_INSTALL_VERSION: 5.3.1
PODMAN_INSTALL_VERSION: 5.4.0-rc2

gcp_credentials: ENCRYPTED[b06ef3490b73469d9da1402568d6f3e46a852955a4ab0807689d50352ecf2852cb5903e8d3b7603eaab9d1c7c7d851a5]

Expand Down
2 changes: 1 addition & 1 deletion podman-rpm-info-vars.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ export PODMAN_RPM_TYPE="dev"
# PODMAN_VERSION is used for fetching the right rpm when PODMAN_RPM_TYPE is set to release.
# However it is always used to derive the machine-os image tag (x.y) so this must be valid
# at any given time.
export PODMAN_VERSION="5.4.0-dev"
export PODMAN_VERSION="5.5.0-dev"
export PODMAN_RPM_RELEASE="1"
33 changes: 17 additions & 16 deletions verify/basic_test.go
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
package verify

import (
"os"
"strings"

. "github.com/onsi/ginkgo/v2"
. "github.com/onsi/gomega"
. "github.com/onsi/gomega/gexec"
Expand Down Expand Up @@ -100,19 +97,23 @@ var _ = Describe("run basic podman commands", func() {
Expect(sshSession.outputToString()).To(And(ContainSubstring("ip_tables"), ContainSubstring("ip6_tables")))

// set by podman-rpm-info-vars.sh
if version := os.Getenv("PODMAN_VERSION"); version != "" {
// version is x.y.z while image uses x.y, remove .z so we can match
index := strings.LastIndex(version, ".")
if index >= 0 {
version = version[:index]
}
// verify the rpm-ostree image inside uses the proper podman image reference
sshSession, err = mb.setCmd([]string{"machine", "ssh", machineName, "sudo rpm-ostree status --json | jq -r '.deployments[0].\"container-image-reference\"'"}).run()
Expect(err).ToNot(HaveOccurred())
Expect(sshSession).To(Exit(0))
Expect(sshSession.outputToString()).
To(Equal("ostree-remote-image:fedora:docker://quay.io/podman/machine-os:" + version))
}

// TODO: there is no 5.5 in the copr yet as podman main would need to be bumped.
// But in order to do that it needs working machine images, catch-22.
// Skip this check for now, we should consider only doing this check on release branches.
// if version := os.Getenv("PODMAN_VERSION"); version != "" {
// // version is x.y.z while image uses x.y, remove .z so we can match
// index := strings.LastIndex(version, ".")
// if index >= 0 {
// version = version[:index]
// }
// // verify the rpm-ostree image inside uses the proper podman image reference
// sshSession, err = mb.setCmd([]string{"machine", "ssh", machineName, "sudo rpm-ostree status --json | jq -r '.deployments[0].\"container-image-reference\"'"}).run()
// Expect(err).ToNot(HaveOccurred())
// Expect(sshSession).To(Exit(0))
// Expect(sshSession.outputToString()).
// To(Equal("ostree-remote-image:fedora:docker://quay.io/podman/machine-os:" + version))
// }
})

It("machine stop/start cycle", func() {
Expand Down

0 comments on commit dfc8ddd

Please sign in to comment.