Skip to content

Commit

Permalink
test: remove workarounds for fixed bugs
Browse files Browse the repository at this point in the history
Bugs have been fixed, workarounds are not needed anymore.
  • Loading branch information
henrywang authored and jrusz committed Feb 7, 2024
1 parent 232bce9 commit fc81b89
Showing 1 changed file with 3 additions and 18 deletions.
21 changes: 3 additions & 18 deletions tools/provision.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,25 +36,10 @@ if [[ $ID == rhel || $ID == centos ]] && ! rpm -q epel-release; then
fi

# RHEL 8.6+ and CentOS 9 require different handling for ansible
ge86=$(echo "${VERSION_ID}" | awk '{print $1 >= 8.6}') # do a numerical comparison for the version
echo -n "${ID}=${VERSION_ID} "
if [[ "${ID}" == "rhel" || "${ID}" == "centos" ]] && (( ge86 )); then
sudo dnf install -y ansible-core koji
else
if [[ "$VERSION_ID" == "8.4" ]]; then
sudo dnf install -y ansible koji
fi

# workaround for bug https://bugzilla.redhat.com/show_bug.cgi?id=2162815
if [[ "${ID}" == "fedora" && "$VERSION_ID" == "37" ]]; then
sudo dnf install -y nghttp2
fi

# workaround for bug https://bugzilla.redhat.com/show_bug.cgi?id=2057769
if [[ "$VERSION_ID" == "9.0" || "$VERSION_ID" == "9" ]]; then
if [[ -f "/usr/share/qemu/firmware/50-edk2-ovmf-amdsev.json" ]]; then
jq '.mapping += {"nvram-template": {"filename": "/usr/share/edk2/ovmf/OVMF_VARS.fd","format": "raw"}}' /usr/share/qemu/firmware/50-edk2-ovmf-amdsev.json | sudo tee /tmp/50-edk2-ovmf-amdsev.json
sudo mv /tmp/50-edk2-ovmf-amdsev.json /usr/share/qemu/firmware/50-edk2-ovmf-amdsev.json
fi
else
sudo dnf install -y ansible-core koji
fi

sudo mkdir -p /etc/osbuild-composer
Expand Down

1 comment on commit fc81b89

@major
Copy link
Contributor

@major major commented on fc81b89 Feb 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WHOA, RELEASE #100! That's awesome! 🎉 Congratulations, everyone.

I know @gicmo would be super proud to see this, too. 😉

Please sign in to comment.