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

The installer scripts fails on fedora - config-manager broken arguments #3014

Open
ricardo-st opened this issue Dec 10, 2024 · 6 comments
Open

Comments

@ricardo-st
Copy link

Describe the problem
The install script does not finalize in fedora.

It returns:
Unknown argument "--add-repo" for command "config-manager". Add "--help" for more information about the arguments.

The fix is easy, change:

        ${SUDO} dnf config-manager --add-repo /etc/yum.repos.d/netbird.repo

for

        ${SUDO} dnf config-manager addrepo --from-repofile=/etc/yum.repos.d/netbird.repo

To Reproduce

Steps to reproduce the behavior:
Run curl -fsSL https://pkgs.netbird.io/install.sh | sh in a clean fedora install.

Expected behavior

The script installs.

Are you using NetBird Cloud?

Self-Hosted

NetBird version

0.34.1

NetBird status -dA output:

N/A

Do you face any (non-mobile) client issues?

N/A

Screenshots

N/A

Additional context

N/A

@mohamed-essam
Copy link
Contributor

mohamed-essam commented Dec 11, 2024

I have a fresh Fedora 40 installation and the script worked normally, and in my dnf config-manager --help it shows:

Config-manager command-specific options:
  --save                save the current options (useful with --setopt)
  --add-repo URL        add (and enable) the repo from the specified file or
                        url

According to this page this is a new format for Fedora 41+, so the script will need to detect which version it is to change the command accordingly to maintain backward compatibility.

@mohamed-essam
Copy link
Contributor

@ricardo-st Could you test this install script to ensure it works on DNF v5+ (I tested on dnf v4 and it works correctly) 🙏
#3026

@orchard0
Copy link

@mohamed-essam Salam. Your commit fixes the issue with the dnf v5 argument changes. Another issue is that OpenPGP checks for packages from the repo are being skipped. Is there a key issue?

@mohamed-essam
Copy link
Contributor

@orchard0 im not sure, does that only happen with dnf v5?

@orchard0
Copy link

orchard0 commented Dec 14, 2024

Thank God I notice that gpgcheck=0 on line 124 in the install script. If I manually change that to gpgcheck=1 then I get the following error:

OpenPGP check for package "netbird-ui-0.34.1-1.x86_64" (/var/cache/libdnf5/NetBird-3f1333a6e9119c03/packages/netbird-ui_0.34.1_linux_amd64.rpm) from repo "NetBird" has failed: The package is not signed.

I get the same message for the main package:

OpenPGP check for package "netbird-0.34.1-1.x86_64" (/var/cache/libdnf5/NetBird-3f1333a6e9119c03/packages/netbird_0.34.1_linux_amd64.rpm) from repo "NetBird" has failed: The package is not signed.

Are the packages not signed?

@orchard0
Copy link

The key issue has been reported before earlier this year: #1698

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants