Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

al2gpu: Remove epel repo after installing dkms and nvidia open modules #191

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

sparrc
Copy link
Contributor

@sparrc sparrc commented Jan 18, 2024

Summary

After installing dkms and nvidia open modules, remove and cleanup the epel repo.

The reason for this is that the epel repo is a public fedora repo. The URLs of this repo require access to the open internet. Customers using isolated subnets will not have internet access so all package updates and installs will fail when yum tries talking to the epel repo.

Removing it ensures that the AMI by default only points to internal Amazon Linux repos which are accessible in isolated subnets.

Testing

al2gpu AMI built, ran /var/lib/ecs/scripts/install-nvidia-open-kmod.sh script

New tests cover the changes:

Description for the changelog

Bugfix: Fix al2gpu AMI package installs and updates when running in isolated subnets

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sudo yum-config-manager --disable epel
sudo rm /etc/yum.repos.d/epel.repo
sudo rm /etc/yum.repos.d/epel-testing.repo
sudo amazon-linux-extras disable epel
Copy link
Member

@fierlion fierlion Jan 18, 2024

Choose a reason for hiding this comment

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

good catch -- I had to go validate this myself to see that the epel topic was auto-enabled by the install.

 24  epel=latest              enabled      [ =7.11  =stable ]

It was.

@sparrc sparrc merged commit da10872 into main Jan 23, 2024
3 checks passed
@sparrc sparrc deleted the gpu-rm-epel-repo branch January 23, 2024 22:59
@Yiyuanzzz Yiyuanzzz mentioned this pull request Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants