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

"Unsupported OS detected: Centos 8", but documentation says Centos 8 is supported #1057

Closed
chrismartino opened this issue Jun 10, 2024 · 2 comments · Fixed by #1053
Closed

Comments

@chrismartino
Copy link

Versions

  • Pi-hole: v5.18.2
  • AdminLTE: v5.21
  • FTL: v5.25.1

Platform

  • OS and version: CentOS Stream release 8
  • Platform: VM

Expected behavior

pihole -up to update pi-hole.

Actual behavior / bug

pihole -up reports:

 [i] Checking for / installing Required dependencies for OS Check...
  [✓] Checking for grep
  [✓] Checking for bind-utils

  [✗] Unsupported OS detected: Centos 8

Dig version check says CentOS 8 is not supported:

$ dig -t txt versions.pi-hole.net

; <<>> DiG 9.11.36-RedHat-9.11.36-11.el8 <<>> -t txt versions.pi-hole.net
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 14716
;; flags: qr rd ra; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 1232
;; QUESTION SECTION:
;versions.pi-hole.net.          IN      TXT

;; ANSWER SECTION:
versions.pi-hole.net.   5       IN      TXT     "Raspbian=10,11,12 Ubuntu=20,22,23,24 Debian=10,11,12 Fedora=39,40 CentOS=9"

But the documentation says CentOS 8 is supported:
image

Steps to reproduce

Steps to reproduce the behavior:

  1. Login to CentOS 8 Stream system
  2. Run pihole -up.
  3. Fail
@yubiuser
Copy link
Member

This is only because this PR is not merged yet: #1053.

CentOS 8 is EOL and our test suite failed permanently because relevant repo mirrors have been removed. We decided to drop CentOS 8 from the list of supported OS.

@yubiuser yubiuser transferred this issue from pi-hole/pi-hole Jun 10, 2024
@yubiuser yubiuser linked a pull request Jun 10, 2024 that will close this issue
1 task
@PromoFaux
Copy link
Member

Now merged!

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 a pull request may close this issue.

3 participants