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.

Sign up for GitHub

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 2 commits into
base: master
Choose a base branch
from
Open

Update 02-installation.md #10317

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3ffbf3b
Update 02-installation.md
legna-namor Jan 22, 2025
eb1effc
Update 02-installation.md
legna-namor Jan 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion doc/02-installation.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ dnf install https://dl.fedoraproject.org/pub/epel/epel-release-latest-${OSVER}.n
```bash
subscription-manager repos --enable rhel-7-server-optional-rpms

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 :)

```
<!-- {% endif %} -->

Expand Down