Skip to content

Add version table generation script and makefile entry #409

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 5 commits into from
Nov 4, 2025
Merged

Add version table generation script and makefile entry #409

merged 5 commits into from
Nov 4, 2025

Conversation

@pkhartsk
Copy link
Contributor

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

To test, edit README.md in any -container repo to contain

<!--
Table start
-->
<!--
Table end
-->

, replace the common directory with this branch, and then run:
$ make version-table
This will generate a supported version table between the commented tags along with expandable container registry addresses for each version.

To see how it works in practice with github actions, see example PRs in my fork at https://github.com/pkhartsk/httpd-container/pulls.

@github-actions
Copy link

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

Pull Request validation

Success

🟢 CI - All checks have passed
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved

Auto Merge

Failed

🔴 Pull Request has unsupported target branch master, expected branches are: ''

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟢 Pull Request meet requirements, mergeable_state is clean

@pkhartsk pkhartsk requested review from hhorak and phracek October 29, 2025 11:51
phracek
phracek requested changes Oct 31, 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.

Please have a look on my proposals. Thanks for bringing this improvement, though. Good work.

phracek
phracek approved these changes Nov 3, 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.

Let's try first round.

@phracek
Copy link
Member

phracek commented Nov 3, 2025

[test]

phracek
phracek requested changes Nov 3, 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.

There are some issues with BASH? Please address it. 'Sanity Check' especially.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025 •
edited
Loading

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
Fedora - postgresql-containerFedora-latestx86_64✅ passed03.11.2025 15:23:3129min 42stest pipeline
CentOS Stream 10 - s2i-base-containerCentOS-Stream-10x86_64✅ passed03.11.2025 15:39:1422min 50stest pipeline
Fedora - nginx-containerFedora-latestx86_64✅ passed03.11.2025 15:23:5925min 48stest pipeline
Fedora - s2i-python-containerFedora-latestx86_64✅ passed03.11.2025 15:33:1250min 46stest pipeline
Fedora - s2i-base-containerFedora-latestx86_64✅ passed03.11.2025 16:03:5226min 46stest pipeline
CentOS Stream 9 - s2i-perl-containerCentOS-Stream-9x86_64✅ passed03.11.2025 16:02:4635min 55stest pipeline
CentOS Stream 9 - s2i-base-containerCentOS-Stream-9x86_64✅ passed03.11.2025 15:35:3026min 33stest pipeline
CentOS Stream 9 - nginx-containerCentOS-Stream-9x86_64✅ passed03.11.2025 15:28:2132min 24stest pipeline
RHEL8 - nginx-containerRHEL-8.10.0-Nightlyx86_64✅ passed03.11.2025 16:00:1621min 43stest pipeline
RHEL8 - s2i-base-containerRHEL-8.10.0-Nightlyx86_64✅ passed03.11.2025 16:04:3820min 24stest pipeline
RHEL10 - postgresql-containerRHEL-10-Nightlyx86_64✅ passed03.11.2025 15:22:2219min 19stest pipeline
RHEL10 - s2i-python-containerRHEL-10-Nightlyx86_64✅ passed03.11.2025 16:04:2228min 32stest pipeline
RHEL8 - postgresql-containerRHEL-8.10.0-Nightlyx86_64✅ passed03.11.2025 15:33:3542min 27stest pipeline
RHEL8 - s2i-perl-containerRHEL-8.10.0-Nightlyx86_64✅ passed03.11.2025 15:50:2327min 11stest pipeline
RHEL9 - s2i-python-containerRHEL-9.6.0-Nightlyx86_64✅ passed03.11.2025 15:42:421h 19min 28stest pipeline
Fedora - s2i-perl-containerFedora-latestx86_64✅ passed03.11.2025 15:34:4751min 49stest pipeline
RHEL9 - postgresql-containerRHEL-9.6.0-Nightlyx86_64✅ passed03.11.2025 15:28:0435min 8stest pipeline
CentOS Stream 10 - s2i-python-containerCentOS-Stream-10x86_64✅ passed03.11.2025 15:56:0246min 7stest pipeline
CentOS Stream 9 - s2i-python-containerCentOS-Stream-9x86_64✅ passed03.11.2025 16:18:461h 29min 10stest pipeline
CentOS Stream 10 - nginx-containerCentOS-Stream-10x86_64✅ passed03.11.2025 15:42:2020min 56stest pipeline
CentOS Stream 10 - s2i-perl-containerCentOS-Stream-10x86_64✅ passed04.11.2025 08:36:0020min 53stest pipeline
RHEL9 - nginx-containerRHEL-9.6.0-Nightlyx86_64✅ passed03.11.2025 15:31:5229min 27stest pipeline
RHEL9 - s2i-base-containerRHEL-9.6.0-Nightlyx86_64✅ passed03.11.2025 16:10:1223min 40stest pipeline
RHEL10 - s2i-perl-containerRHEL-10-Nightlyx86_64✅ passed03.11.2025 16:09:5426min 15stest pipeline
RHEL10 - s2i-base-containerRHEL-10-Nightlyx86_64✅ passed03.11.2025 16:03:3218min 9stest pipeline
RHEL9 - s2i-perl-containerRHEL-9.6.0-Nightlyx86_64✅ passed03.11.2025 15:53:5531min 16stest pipeline
RHEL10 - nginx-containerRHEL-10-Nightlyx86_64✅ passed03.11.2025 15:23:5617min 6stest pipeline
CentOS Stream 10 - postgresql-containerCentOS-Stream-10x86_64✅ passed03.11.2025 15:53:5625min 2stest pipeline
CentOS Stream 9 - postgresql-containerCentOS-Stream-9x86_64✅ passed03.11.2025 16:20:1526min 56stest pipeline
RHEL8 - s2i-python-containerRHEL-8.10.0-Nightlyx86_64✅ passed03.11.2025 16:18:052h 4min 41stest pipeline

@pkhartsk
Copy link
Contributor Author

pkhartsk commented Nov 3, 2025

[test]

phracek
phracek approved these changes Nov 3, 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.

LGTM. Thanks

@pkhartsk pkhartsk merged commit baf7d8b into sclorg:master Nov 4, 2025
4 checks passed
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

@hhorak hhorak Awaiting requested review from hhorak

Assignees

No one assigned

Labels

pr/needs-manual-merge ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pkhartsk @phracek