Skip to content

Remove RHEL7 and scl_enable #624

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

Jump to bottom
Merged
merged 4 commits into from
Oct 20, 2025
Merged

Remove RHEL7 and scl_enable #624

merged 4 commits into from
Oct 20, 2025

Conversation

pkhartsk
Copy link
Contributor

@pkhartsk pkhartsk commented Oct 10, 2025 •
edited by github-actions bot
Loading

This PR removes mentions of and files for RHEL7. Also removes scl_enable and updates READMEs and scripts to not contain mentions of non-Stream CentOS, dead links, small typos, etc.

@github-actions GitHub Actions
Copy link

Pull Request validation

Failed

🔴 Review - Missing review from a member (1 required)

Success

🟢 CI - All checks have passed

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 10, 2025 •
edited
Loading

Pull Request validation

Failed

🔴 Failed or pending statuses:

  • Testing Farm - CentOS Stream 10 - 16[error]
  • Testing Farm - CentOS Stream 9 - 13[error]
  • Testing Farm - Fedora - 16[error]
  • Testing Farm - Fedora - 15[error]
  • Testing Farm - CentOS Stream 9 - 15[error]
  • Testing Farm - CentOS Stream 9 - 16[error]
  • Testing Farm - RHEL8 - 13[pending]
  • Testing Farm - RHEL9 - 16[pending]
  • Testing Farm - RHEL9 - 13[pending]
  • Testing Farm - RHEL8 - 12[pending]
  • Testing Farm - RHEL10 - 16[pending]
  • Testing Farm - RHEL8 - 15[pending]
  • Testing Farm - RHEL8 - 16[pending]
  • Testing Farm - RHEL9 - 15[pending]

Success

🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved

@pkhartsk
Copy link
Contributor Author

[test]

@pkhartsk
Copy link
Contributor Author

[test]

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 13, 2025 •
edited
Loading

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
Fedora - 15Fedora-latestx86_64✅ passed20.10.2025 10:00:319min 46stest pipeline
CentOS Stream 9 - 13CentOS-Stream-9x86_64✅ passed20.10.2025 10:01:3311min 36stest pipeline
Fedora - 16Fedora-latestx86_64✅ passed20.10.2025 10:00:3510min 10stest pipeline
CentOS Stream 9 - 16CentOS-Stream-9x86_64✅ passed20.10.2025 10:00:4412min 18stest pipeline
CentOS Stream 9 - 15CentOS-Stream-9x86_64✅ passed20.10.2025 10:01:3011min 40stest pipeline
RHEL10 - 16RHEL-10-Nightlyx86_64✅ passed20.10.2025 08:15:122h 52min 36stest pipeline
RHEL8 - 16RHEL-8.10.0-Nightlyx86_64✅ passed20.10.2025 08:15:132h 48min 17stest pipeline
RHEL9 - 15RHEL-9.6.0-Nightlyx86_64✅ passed20.10.2025 08:15:153h 55min 31stest pipeline
CentOS Stream 10 - 16CentOS-Stream-10x86_64✅ passed20.10.2025 10:00:2010min 31stest pipeline
RHEL8 - 15RHEL-8.10.0-Nightlyx86_64✅ passed20.10.2025 08:15:124h 23min 12stest pipeline
RHEL8 - 13RHEL-8.10.0-Nightlyx86_64✅ passed20.10.2025 10:00:142h 37min 12stest pipeline
RHEL9 - 16RHEL-9.6.0-Nightlyx86_64✅ passed20.10.2025 10:00:291h 28min 13stest pipeline
RHEL8 - 12RHEL-8.10.0-Nightlyx86_64✅ passed20.10.2025 08:15:103h 17min 30stest pipeline
RHEL9 - 13RHEL-9.6.0-Nightlyx86_64✅ passed20.10.2025 10:00:2443min 39stest pipeline

@pkhartsk pkhartsk requested a review from phracek October 13, 2025 15:20
phracek
phracek requested changes Oct 20, 2025
View reviewed changes
Copy link
Member

@phracek phracek left a comment

Choose a reason for hiding this comment

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

All looks good.

But I would prefere also remove this file completelly including
the dir. It does not make sense to have it there https://github.com/sclorg/postgresql-container/blob/master/hooks/pre_build_distgen

phracek
phracek approved these changes Oct 20, 2025
View reviewed changes
Copy link
Member

@phracek phracek 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 removing RHEL\7 and scl_enable

@phracek
Copy link
Member

phracek commented Oct 20, 2025

[test]

@pkhartsk
Copy link
Contributor Author

[test]

@pkhartsk pkhartsk merged commit 8ae0579 into master Oct 20, 2025
16 checks passed
@pkhartsk pkhartsk deleted the remove_sclenable branch October 20, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phracek phracek phracek approved these changes

Assignees

No one assigned

Labels

pr/failing-ci pr/needs-manual-merge READY-to-MERGE

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pkhartsk @phracek