Skip to content

Fixup for version table script shebang #414

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
phracek merged 1 commit into from
Dec 16, 2025
Merged

Fixup for version table script shebang #414

phracek merged 1 commit into from
Dec 16, 2025

Conversation

@pkhartsk
Copy link
Contributor

@pkhartsk pkhartsk commented Dec 15, 2025 •
edited by github-actions bot
Loading

/usr/bin/env python3 is more universal than /usr/bin/python3 and supports switchable python versions

@pkhartsk pkhartsk requested a review from phracek December 15, 2025 11:36
@github-actions
Copy link

github-actions bot commented Dec 15, 2025 •
edited
Loading

Pull Request validation

Failed

🔴 Failed or pending statuses:

  • Testing Farm - RHEL10 - s2i-python-container[error]
  • Testing Farm - RHEL10 - s2i-perl-container[pending]
  • Testing Farm - Fedora - s2i-python-container[pending]
  • Testing Farm - RHEL10 - nginx-container[pending]
  • Testing Farm - RHEL10 - s2i-base-container[pending]
  • Testing Farm - RHEL9 - s2i-perl-container[pending]
  • Testing Farm - RHEL10 - postgresql-container[pending]

Success

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

phracek
phracek approved these changes Dec 15, 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. We use this Python SHEBANG in all other scripts.

@phracek
Copy link
Member

phracek commented Dec 15, 2025

[test]

@github-actions
Copy link

github-actions bot commented Dec 15, 2025 •
edited
Loading

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
RHEL10 - postgresql-containerRHEL-10-Nightlyx86_64❌ error16.12.2025 08:28:1020min 32stest pipeline
Fedora - nginx-containerFedora-latestx86_64✅ passed16.12.2025 08:28:0617min 15stest pipeline
Fedora - s2i-python-containerFedora-latestx86_64✅ passed16.12.2025 08:28:0740min 45stest pipeline
CentOS Stream 10 - nginx-containerCentOS-Stream-10x86_64✅ passed15.12.2025 12:26:1313min 50stest pipeline
RHEL8 - s2i-base-containerRHEL-8.10.0-Nightlyx86_64✅ passed15.12.2025 12:26:3318min 19stest pipeline
RHEL10 - nginx-containerRHEL-10-Nightlyx86_64❌ error16.12.2025 08:28:0820min 20stest pipeline
RHEL9 - nginx-containerRHEL-9.6.0-Nightlyx86_64✅ passed15.12.2025 12:18:2629min 32stest pipeline
CentOS Stream 9 - nginx-containerCentOS-Stream-9x86_64✅ passed15.12.2025 12:31:2117min 45stest pipeline
CentOS Stream 9 - s2i-base-containerCentOS-Stream-9x86_64✅ passed15.12.2025 12:35:2214min 46stest pipeline
CentOS Stream 10 - s2i-perl-containerCentOS-Stream-10x86_64✅ passed15.12.2025 12:35:0921min 28stest pipeline
RHEL8 - nginx-containerRHEL-8.10.0-Nightlyx86_64✅ passed15.12.2025 12:37:2821min 13stest pipeline
RHEL10 - s2i-perl-containerRHEL-10-Nightlyx86_64❌ error16.12.2025 08:28:0721min 9stest pipeline
RHEL10 - s2i-python-containerRHEL-10-Nightlyx86_64❌ error16.12.2025 08:28:0819min 53stest pipeline
Fedora - postgresql-containerFedora-latestx86_64✅ passed15.12.2025 12:40:0222min 43stest pipeline
CentOS Stream 9 - postgresql-containerCentOS-Stream-9x86_64✅ passed15.12.2025 12:35:5626min 23stest pipeline
CentOS Stream 9 - s2i-perl-containerCentOS-Stream-9x86_64✅ passed15.12.2025 12:38:4924min 4stest pipeline
RHEL9 - s2i-perl-containerRHEL-9.6.0-Nightlyx86_64✅ passed16.12.2025 08:28:0830min 13stest pipeline
RHEL10 - s2i-base-containerRHEL-10-Nightlyx86_64❌ error16.12.2025 08:28:0821min 9stest pipeline
RHEL9 - s2i-base-containerRHEL-9.6.0-Nightlyx86_64✅ passed15.12.2025 12:48:4923min 9stest pipeline
CentOS Stream 10 - postgresql-containerCentOS-Stream-10x86_64✅ passed15.12.2025 12:49:4722min 47stest pipeline
CentOS Stream 10 - s2i-base-containerCentOS-Stream-10x86_64✅ passed15.12.2025 12:46:5726min 30stest pipeline
Fedora - s2i-perl-containerFedora-latestx86_64✅ passed15.12.2025 12:37:4138min 37stest pipeline
RHEL9 - postgresql-containerRHEL-9.6.0-Nightlyx86_64✅ passed15.12.2025 12:40:3236min 13stest pipeline
Fedora - s2i-base-containerFedora-latestx86_64✅ passed15.12.2025 12:58:5219min 19stest pipeline
RHEL8 - s2i-perl-containerRHEL-8.10.0-Nightlyx86_64✅ passed15.12.2025 12:47:5731min 20stest pipeline
RHEL8 - postgresql-containerRHEL-8.10.0-Nightlyx86_64✅ passed15.12.2025 12:43:4440min 12stest pipeline
RHEL9 - s2i-python-containerRHEL-9.6.0-Nightlyx86_64✅ passed15.12.2025 12:34:121h 19min 50stest pipeline
CentOS Stream 10 - s2i-python-containerCentOS-Stream-10x86_64✅ passed15.12.2025 12:49:441h 7min 55stest pipeline
CentOS Stream 9 - s2i-python-containerCentOS-Stream-9x86_64✅ passed15.12.2025 12:34:231h 26min 43stest pipeline
RHEL8 - s2i-python-containerRHEL-8.10.0-Nightlyx86_64✅ passed15.12.2025 12:28:582h 4min 11stest pipeline

@pkhartsk
Copy link
Contributor Author

pkhartsk commented Dec 16, 2025

RHEL10 container failures expected as #413 hasn't been merged yet.

phracek
phracek approved these changes Dec 16, 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

@phracek phracek merged commit 01d0fe7 into master Dec 16, 2025
29 of 34 checks passed
@phracek phracek deleted the version-script-fix branch December 16, 2025 09:55
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 ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pkhartsk @phracek