Skip to content

Commit

Permalink
kola: Rename qemu-unpriv to qemu
Browse files Browse the repository at this point in the history
There is only one qemu platform now and it is unprivileged.
  • Loading branch information
travier committed Jan 27, 2023
1 parent 25c0f2e commit 5b5525a
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion mantle/cmd/kola/kola.go
Original file line number Diff line number Diff line change
Expand Up @@ -583,7 +583,7 @@ func syncFindParentImageOptions() error {
// Here we handle the --fetch-parent-image --> platform-specific options
// based on its cosa build metadata
switch kolaPlatform {
case "qemu-unpriv":
case "qemu":
if qemuImageDir == "" {
if qemuImageDir, err = ioutil.TempDir("/var/tmp", "kola-run-upgrade"); err != nil {
return err
Expand Down
8 changes: 4 additions & 4 deletions mantle/cmd/kola/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -177,14 +177,14 @@ func syncOptionsImpl(useCosa bool) error {
if kola.QEMUIsoOptions.IsoPath != "" {
kolaPlatform = "qemu-iso"
} else {
kolaPlatform = "qemu-unpriv"
kolaPlatform = "qemu"
}
}

// There used to be a "privileged" qemu path, it is no longer supported.
// Alias qemu to qemu-unpriv.
if kolaPlatform == "qemu" {
kolaPlatform = "qemu-unpriv"
// Alias qemu-unpriv to qemu.
if kolaPlatform == "qemu-unpriv" {
kolaPlatform = "qemu"
} else if kolaPlatform == "iso" {
kolaPlatform = "qemu-iso"
}
Expand Down
8 changes: 4 additions & 4 deletions mantle/kola/harness.go
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ func NewFlight(pltfrm string) (flight platform.Flight, err error) {
flight, err = openstack.NewFlight(&OpenStackOptions)
case "packet":
flight, err = packet.NewFlight(&PacketOptions)
case "qemu-unpriv":
case "qemu":
flight, err = unprivqemu.NewFlight(&QEMUOptions)
case "qemu-iso":
flight, err = qemuiso.NewFlight(&QEMUIsoOptions)
Expand Down Expand Up @@ -458,8 +458,8 @@ func filterTests(tests map[string]*register.Test, patterns []string, pltfrm stri

checkPlatforms := []string{pltfrm}

// qemu-unpriv has the same restrictions as QEMU but might also want additional restrictions due to the lack of a Local cluster
if pltfrm == "qemu-unpriv" {
// qemu has additional restrictions due to the lack of a Local cluster
if pltfrm == "qemu" {
checkPlatforms = append(checkPlatforms, "qemu")
}

Expand Down Expand Up @@ -616,7 +616,7 @@ func filterTests(tests map[string]*register.Test, patterns []string, pltfrm stri
if allowed, excluded := isAllowed(Options.Distribution, t.Distros, t.ExcludeDistros); !allowed || excluded {
continue
}
if pltfrm == "qemu-unpriv" {
if pltfrm == "qemu" {
if allowed, excluded := isAllowed(QEMUOptions.Firmware, t.Firmwares, t.ExcludeFirmwares); !allowed || excluded {
continue
}
Expand Down

0 comments on commit 5b5525a

Please sign in to comment.