Skip to content

fix: installation script compatibility with fedora 41 and later, fedora headless unclear errors #5239

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

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

Bloodis94
Copy link
Contributor

@Bloodis94 Bloodis94 commented Apr 23, 2025

Description

This PR improves the installation script by addressing compatibility and clarity issues:

  1. Renamed VERSION to LOCALAI_VERSION to prevent conflicts with system variables.
  2. Enhanced NVIDIA and CUDA repository setup for DNF5 compatibility.
  3. Revised Fedora version handling for CUDA installations.
  4. Standardized the use of LOCALAI_VERSION for Docker image tags.
  5. Improved logging for repository and binary downloads.
  6. Added a temporary bypass for nvidia-smi installation on Fedora Cloud Edition.

This PR fixes #
Compatibility with Fedora 41 and later (handling of newer dnf5 instead of dnf4)
Unclear errors when launching the installation script on a headless Fedora setup.

Notes for Reviewers

This is a WIP that atleast dosn't fail with an unclear error anymore on a Fedora Cloud Edition setup, it just fail to find a GPU for the docker run command.

The only missing step is to find a way to install nvidia-smi on this type of setup without pulling the entire x11 dependency tree(with akmod it pull x11, pipewire, nouveau, etc).

Signed commits

  • Yes, I signed my commits.

- Renamed VERSION to LOCALAI_VERSION to avoid conflicts with system variables.
- Enhanced NVIDIA and CUDA repository installation for DNF5 compatibility.
- Adjusted default Fedora version handling for CUDA installation.
- Updated Docker image tag handling to use LOCALAI_VERSION consistently.
- Improved logging messages for repository and LocalAI binary downloads.
- Added a temporary bypass for nvidia-smi installation on Fedora Cloud Edition.
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for localai ready!

Name Link
🔨 Latest commit f00029d
🔍 Latest deploy log https://app.netlify.com/sites/localai/deploys/68097ad657212600081c248d
😎 Deploy Preview https://deploy-preview-5239--localai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mudler mudler requested a review from Copilot April 24, 2025 07:33
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • docs/static/install.sh: Language not supported

Copy link
Owner

@mudler mudler left a comment

Choose a reason for hiding this comment

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

I don't have a fedora install to try it out - changes looks good. Thanks!

@mudler mudler merged commit 1ae0b89 into mudler:master Apr 24, 2025
10 of 11 checks passed
@mudler mudler added the bug Something isn't working label May 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants