Skip to content

Conversation

phracek
Copy link
Member

@phracek phracek commented Sep 10, 2025

Remove usage .version and replace it with $dir variable.

The variable has format like:
1.24, 2.4-micro, 13.

So it can be used for tagging and testing.

As well as for test.sh it can be used

The variable has format like:
1.24, 2.4-micro, 13.

So it can be used for tagging and testing.

As well as for test.sh it can be used

Signed-off-by: Petr "Stone" Hracek <[email protected]>
Copy link

github-actions bot commented Sep 10, 2025

Pull Request validation

Failed

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

Success

🟢 CI - All checks have passed

@phracek
Copy link
Member Author

phracek commented Sep 10, 2025

[test]

Copy link

github-actions bot commented Sep 10, 2025

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
CentOS Stream 9 - s2i-perl-containerCentOS-Stream-9x86_64✅ passed10.09.2025 11:22:2421min 55stest pipeline
RHEL8 - nginx-containerRHEL-8.10.0-Nightlyx86_64✅ passed10.09.2025 11:22:0323min 18stest pipeline
CentOS Stream 10 - postgresql-containerCentOS-Stream-10x86_64✅ passed10.09.2025 11:30:1814min 21stest pipeline
CentOS Stream 10 - nginx-containerCentOS-Stream-10x86_64✅ passed10.09.2025 11:33:3713min 12stest pipeline
Fedora - nginx-containerFedora-latestx86_64✅ passed10.09.2025 11:33:4312min 26stest pipeline
RHEL10 - nginx-containerRHEL-10-Nightlyx86_64✅ passed30.09.2025 10:52:2917min 55stest pipeline
RHEL9 - postgresql-containerRHEL-9.6.0-Nightlyx86_64✅ passed10.09.2025 11:22:1136min 36stest pipeline
RHEL10 - s2i-perl-containerRHEL-10-Nightlyx86_64✅ passed10.09.2025 11:33:3826min 56stest pipeline
Fedora - s2i-perl-containerFedora-latestx86_64✅ passed10.09.2025 11:22:1238min 50stest pipeline
CentOS Stream 9 - nginx-containerCentOS-Stream-9x86_64✅ passed10.09.2025 11:44:2216min 32stest pipeline
CentOS Stream 10 - s2i-base-containerCentOS-Stream-10x86_64✅ passed10.09.2025 11:52:0113min 5stest pipeline
Fedora - postgresql-containerFedora-latestx86_64✅ passed10.09.2025 11:46:0719min 8stest pipeline
CentOS Stream 9 - s2i-base-containerCentOS-Stream-9x86_64✅ passed10.09.2025 11:50:4014min 9stest pipeline
RHEL8 - s2i-base-containerRHEL-8.10.0-Nightlyx86_64✅ passed10.09.2025 11:48:0819min 52stest pipeline
RHEL9 - s2i-perl-containerRHEL-9.6.0-Nightlyx86_64✅ passed10.09.2025 11:48:1429min 31stest pipeline
RHEL8 - postgresql-containerRHEL-8.10.0-Nightlyx86_64✅ passed10.09.2025 11:32:1645min 5stest pipeline
CentOS Stream 10 - s2i-python-containerCentOS-Stream-10x86_64✅ passed10.09.2025 11:48:0234min 3stest pipeline
RHEL10 - s2i-base-containerRHEL-10-Nightlyx86_64✅ passed10.09.2025 12:06:5218min 25stest pipeline
CentOS Stream 9 - postgresql-containerCentOS-Stream-9x86_64✅ passed10.09.2025 12:02:4124min 4stest pipeline
RHEL10 - postgresql-containerRHEL-10-Nightlyx86_64✅ passed10.09.2025 12:09:0219min 41stest pipeline
Fedora - s2i-base-containerFedora-latestx86_64✅ passed10.09.2025 12:16:4912min 20stest pipeline
RHEL9 - nginx-containerRHEL-9.6.0-Nightlyx86_64✅ passed10.09.2025 12:06:2427min 58stest pipeline
RHEL10 - s2i-python-containerRHEL-10-Nightlyx86_64✅ passed10.09.2025 12:07:0327min 27stest pipeline
CentOS Stream 10 - s2i-perl-containerCentOS-Stream-10x86_64✅ passed10.09.2025 12:14:5120min 17stest pipeline
RHEL9 - s2i-base-containerRHEL-9.6.0-Nightlyx86_64✅ passed10.09.2025 12:18:3422min 54stest pipeline
RHEL8 - s2i-perl-containerRHEL-8.10.0-Nightlyx86_64✅ passed10.09.2025 12:14:5929min 48stest pipeline
CentOS Stream 9 - s2i-python-containerCentOS-Stream-9x86_64✅ passed10.09.2025 11:33:391h 26min 25stest pipeline
Fedora - s2i-python-containerFedora-latestx86_64✅ passed10.09.2025 12:14:3352min 7stest pipeline
RHEL8 - s2i-python-containerRHEL-8.10.0-Nightlyx86_64✅ passed10.09.2025 11:32:061h 51min 30stest pipeline
RHEL9 - s2i-python-containerRHEL-9.6.0-Nightlyx86_64✅ passed10.09.2025 12:08:261h 18min 18stest pipeline

Copy link
Member

@frenzymadness frenzymadness left a comment

Choose a reason for hiding this comment

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

The changes look good to me, and the tests in sclorg/s2i-python-container#755 proved that they work.

@phracek phracek merged commit 303f17b into master Oct 8, 2025
34 checks passed
@phracek phracek deleted the remove_version_usage branch October 8, 2025 12:22
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.

2 participants