Skip to content

Fix ENV variable declaration in the test container, update fedora version to 42, tweak CI#342

Merged
rhusar merged 3 commits intomodcluster:mainfrom
jajik:httpd
Aug 5, 2025
Merged

Fix ENV variable declaration in the test container, update fedora version to 42, tweak CI#342
rhusar merged 3 commits intomodcluster:mainfrom
jajik:httpd

Conversation

@jajik
Copy link
Copy Markdown
Member

@jajik jajik commented Aug 4, 2025

The issue with the env variable (in the Containerfile) does not manifest in the testsuite because we're using a default in httpd_start, but still.

@jajik jajik added enhancement bug dependencies Pull requests that update a dependency file and removed enhancement labels Aug 4, 2025
@jajik
Copy link
Copy Markdown
Member Author

jajik commented Aug 5, 2025

Added a fix for our CI workflows, let's use only apt-get and not mix it with apt. Additionally, I removed duplicates in packages list and an unnecessary flag.

@jajik jajik changed the title Fix ENV variable declaration in the test container, update fedora version to 42 Fix ENV variable declaration in the test container, update fedora version to 42, tweak CI Aug 5, 2025
Copy link
Copy Markdown
Member

@rhusar rhusar left a comment

Choose a reason for hiding this comment

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

LGTM

@rhusar rhusar merged commit c21b822 into modcluster:main Aug 5, 2025
15 checks passed
@rhusar
Copy link
Copy Markdown
Member

rhusar commented Aug 5, 2025

Thanks @jajik !

@jajik jajik deleted the httpd branch November 13, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug dependencies Pull requests that update a dependency file enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants