mirror of https://github.com/containers/podman.git
Mac PM test: Require pre-installed rosetta
Previously, the mac podman-machine tests installed rosetta before executing any tests. As a best-practice (and because the Macs in CI are shared) tests should never permanently modify the system. As of this commit, the system setup script used for the CI Macs does the rosetta installation. Remove the test setup code that installed rosetta and add a CI-level confirmation that it's been pre-installed. Signed-off-by: Chris Evich <cevich@redhat.com>
This commit is contained in:
parent
e53b96cb25
commit
74e8f98772
|
|
@ -7,6 +7,14 @@
|
|||
|
||||
set -euo pipefail
|
||||
|
||||
# Confirm rosetta is installed/enabled and working
|
||||
if ! arch -arch x86_64 /usr/bin/uname -m; then
|
||||
# This likely means whatever script used to prepare this mac failed
|
||||
# and/or did not execute `sudo softwareupdate --install-rosetta --agree-to-license`
|
||||
echo "Rosetta doesn't appear to be installed, or is non-functional."
|
||||
exit 1
|
||||
fi
|
||||
|
||||
# The otherwise standard `/etc/ci_environment` file cannot be used in this
|
||||
# context, because the system is shared for multiple tasks. Instead, persist
|
||||
# env. vars required during /subsequent/ testing steps via a "magic" Cirrus-CI
|
||||
|
|
|
|||
|
|
@ -5,7 +5,6 @@ import (
|
|||
"fmt"
|
||||
"io"
|
||||
"os"
|
||||
"os/exec"
|
||||
"path/filepath"
|
||||
"runtime"
|
||||
"strings"
|
||||
|
|
@ -70,13 +69,6 @@ var _ = BeforeSuite(func() {
|
|||
if pullError != nil {
|
||||
Fail(fmt.Sprintf("failed to pull wsl disk: %q", pullError))
|
||||
}
|
||||
if testProvider.VMType() == define.AppleHvVirt {
|
||||
cmd := exec.Command("softwareupdate", "--install-rosetta", "--agree-to-license")
|
||||
err := cmd.Run()
|
||||
if err != nil {
|
||||
Fail(fmt.Sprintf("Command failed with error: %q", err))
|
||||
}
|
||||
}
|
||||
})
|
||||
|
||||
var _ = SynchronizedAfterSuite(func() {}, func() {})
|
||||
|
|
|
|||
Loading…
Reference in New Issue