Skip to content

Commit

Permalink
Remove P8 and P9 hack so tests pass for Power10
Browse files Browse the repository at this point in the history
008e04c and a6d7bbb added and used the libguestfs-ppc64le-wrapper.sh in order to address some failing tests on ppc64le. Using a Power10 machine with KVM enabled, tests are passing with this script removed. There was a successful test run with: cosa kola run --tag '!reprovision'

Signed-off-by: Mick Tarsel <[email protected]>
  • Loading branch information
mtarsel authored and ravanelli committed Oct 22, 2024
1 parent 4155975 commit 9692c44
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 25 deletions.
15 changes: 0 additions & 15 deletions mantle/platform/qemu.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,6 @@ import (
"github.com/coreos/coreos-assembler/mantle/system"
"github.com/coreos/coreos-assembler/mantle/system/exec"
"github.com/pkg/errors"

"golang.org/x/sys/unix"
)

var (
Expand Down Expand Up @@ -859,19 +857,6 @@ func newGuestfish(arch, diskImagePath string, diskSectorSize int) (*coreosGuestf
cmd := exec.Command("guestfish", guestfishArgs...)
cmd.Env = append(os.Environ(), "LIBGUESTFS_BACKEND=direct")

// Hack to run with a wrapper on older P8 hardware running RHEL7
switch arch {
case "ppc64le":
u := unix.Utsname{}
if err := unix.Uname(&u); err != nil {
return nil, errors.Wrapf(err, "detecting kernel information")
}
if strings.Contains(fmt.Sprintf("%s", u.Release), "el7") {
plog.Infof("Detected el7. Running using libguestfs-ppc64le-wrapper.sh")
cmd.Env = append(cmd.Env, "LIBGUESTFS_HV=/usr/lib/coreos-assembler/libguestfs-ppc64le-wrapper.sh")
}
}

// make sure it inherits stderr so we see any error message
cmd.Stderr = os.Stderr
stdout, err := cmd.StdoutPipe()
Expand Down
8 changes: 0 additions & 8 deletions src/libguestfish.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,6 @@ export LIBGUESTFS_BACKEND=direct

arch=$(uname -m)


# Hack to run with a wrapper on older P8 hardware running RHEL7
if [ "$arch" = "ppc64le" ] ; then
if [[ "$(uname -r)" =~ "el7" ]]; then
export LIBGUESTFS_HV="/usr/lib/coreos-assembler/libguestfs-ppc64le-wrapper.sh"
fi
fi

# Hack to give ppc64le more memory inside the libguestfs VM.
# The compiled in default I see when running `guestfish get-memsize`
# is 1280. We need this because we are seeing issues from
Expand Down
2 changes: 0 additions & 2 deletions src/libguestfs-ppc64le-wrapper.sh

This file was deleted.

0 comments on commit 9692c44

Please sign in to comment.