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

Update 02-installation.md #10317

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

legna-namor
Copy link

https://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm has been moved to archives.

New Link:
https://archives.fedoraproject.org/pub/archive/epel/7/x86_64/Packages/e/epel-release-7-14.noarch.rpm

Added --nogpgcheck flag to prevent following error:

Downloading packages:
warning: /home/user/epel-release-7-14.noarch.rpm: Header V4 RSA/SHA256 Signature, key ID 352c64e5: NOKEY

Public key for epel-release-7-14.noarch.rpm is not installed

@cla-bot cla-bot bot added the cla/signed label Jan 22, 2025
Copy link
Member

@oxzi oxzi left a comment

Choose a reason for hiding this comment

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

Thanks for your Pull Request!

I thought we have already removed all references to RHEL 7 as it went end-of-life last June (except Extended Life Cycle Support). But it seems this was not the case for Icinga 2 itself.

And as people are still having a need, I guess this PR is valid until we decide to drop it here as well.

@legna-namor
Copy link
Author

Thanks for your Pull Request!

I thought we have already removed all references to RHEL 7 as it went end-of-life last June (except Extended Life Cycle Support). But it seems this was not the case for Icinga 2 itself.

And as people are still having a need, I guess this PR is valid until we decide to drop it here as well.

No worries, glad to help!

yum install https://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm
yum install --nogpgcheck https://archives.fedoraproject.org/pub/archive/epel/7/x86_64/Packages/e/epel-release-7-14.noarch.rpm
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is just an archived version, shouldn't it be signed exactly the same as before?

Added --nogpgcheck flag to prevent following error:

Downloading packages:
warning: /home/user/epel-release-7-14.noarch.rpm: Header V4 RSA/SHA256 Signature, key ID 352c64e5: NOKEY

Public key for epel-release-7-14.noarch.rpm is not installed

Sounds more like a step to import the corresponding key was missing. However, it's questionable if it's worth putting effort into this:

I thought we have already removed all references to RHEL 7 as it went end-of-life last June (except Extended Life Cycle Support). But it seems this was not the case for Icinga 2 itself.

And as people are still having a need, I guess this PR is valid until we decide to drop it here as well.

Indeed, we don't build packages for RHEL 7 anymore1, like yesterday's Icinga 2.14.4 release wasn't even built for RHEL 7 anymore. So I'd say it would be better to just remove the instructions for RHEL 7 instead of giving false hopes.

Footnotes

  1. Except in very special circumstances like the 2.14.3 security release.

Copy link
Author

Choose a reason for hiding this comment

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

Lets nuke it!

I just needed it for my systems that have not been migrated yet. The latest version i have working is:
r2.14.3-1

Want me to update the commit and remove RHEL7 from the Docs?

Copy link
Member

Choose a reason for hiding this comment

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

Valid point.

Want me to update the commit and remove RHEL7 from the Docs?

Feel free to alter this PR :)

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

Successfully merging this pull request may close these issues.

3 participants