Skip to content

Commit

Permalink
Exclude .img images from upgrades (#160)
Browse files Browse the repository at this point in the history
and fix renamed Earthly ARG

Fixes kairos-io/kairos#598

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>

Signed-off-by: Dimitris Karakasilis <dimitris@karakasilis.me>
  • Loading branch information
jimmykarily committed Jan 3, 2023
1 parent b5bf46a commit 0d636d2
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 6 deletions.
6 changes: 3 additions & 3 deletions Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -272,9 +272,9 @@ linux-bench-scan:
# Generic targets
# usage e.g. ./earthly.sh +datasource-iso --CLOUD_CONFIG=tests/assets/qrcode.yaml
datasource-iso:
ARG ELEMENTAL_IMAGE
ARG OSBUILDER_IMAGE
ARG CLOUD_CONFIG
FROM $ELEMENTAL_IMAGE
FROM $OSBUILDER_IMAGE
RUN zypper in -y mkisofs
WORKDIR /build
RUN touch meta-data
Expand Down Expand Up @@ -358,4 +358,4 @@ run-proxmox-tests:

COPY +edgevpn/edgevpn /usr/bin/edgevpn
COPY . .
RUN PATH=$PATH:$GOPATH/bin ginkgo --label-filter "$TEST_SUITE" --fail-fast -r ./tests/e2e/
RUN PATH=$PATH:$GOPATH/bin ginkgo --label-filter "$TEST_SUITE" --fail-fast -r ./tests/e2e/
2 changes: 1 addition & 1 deletion internal/provider/upgrade.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ func ListVersions(e *pluggable.Event) pluggable.EventResponse {
displayTags := []string{}

for _, t := range tags {
if strings.Contains(t, "k3s") {
if strings.Contains(t, "k3s") && !strings.Contains(t, ".img") {
displayTags = append(displayTags, t)
}
}
Expand Down
8 changes: 6 additions & 2 deletions tests/provider_upgrade_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ package mos_test
import (
"encoding/json"

. "github.com/spectrocloud/peg/matcher"
"github.com/mudler/go-pluggable"
. "github.com/onsi/ginkgo/v2"
. "github.com/onsi/gomega"
. "github.com/spectrocloud/peg/matcher"
"golang.org/x/mod/semver"
)

Expand All @@ -23,7 +23,7 @@ var _ = Describe("provider upgrade test", Label("provider-upgrade"), func() {
})

Context("agent.available_releases event", func() {
It("returns the available versions ordered", func() {
It("returns the available versions ordered, excluding '.img' tags", func() {
resultStr, _ := Sudo(`echo '{}' | /system/providers/agent-provider-kairos agent.available_releases`)

var result pluggable.EventResponse
Expand All @@ -41,6 +41,10 @@ var _ = Describe("provider upgrade test", Label("provider-upgrade"), func() {

semver.Sort(sorted)

for _, t := range sorted {
Expect(t).ToNot(ContainSubstring(".img"))
}

Expect(sorted).To(Equal(versions))
})
})
Expand Down

0 comments on commit 0d636d2

Please sign in to comment.