Skip to content

Upgrade to UBI 10#36

Merged
Fryguy merged 3 commits into
ManageIQ:masterfrom
bdunne:c10s
May 19, 2026
Merged

Upgrade to UBI 10#36
Fryguy merged 3 commits into
ManageIQ:masterfrom
bdunne:c10s

Conversation

@bdunne
Copy link
Copy Markdown
Member

@bdunne bdunne commented Apr 22, 2026

@bdunne bdunne requested a review from Fryguy as a code owner April 22, 2026 17:57
@Fryguy
Copy link
Copy Markdown
Member

Fryguy commented Apr 22, 2026

Should this target an el10 branch?

Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
RUN ARCH=$(uname -m) && \
dnf -y --disableplugin=subscription-manager --setopt=tsflags=nodocs install \
RUN --mount=type=bind,from=quay.io/manageiq/build_tools:el10,source=/tools,target=/usr/local/bin \
ubi_2_stream_10 && \
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The name is killing me LOL

How about the following to align with enable_epel

Suggested change
ubi_2_stream_10 && \
enable_centos_stream && \

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

:trollface: I knew I'd get your attention 😆

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I also thought of prefixing all of these scripts with miq_ or something to make it more obvious that they're ours.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh yeah I like the prefix. miq or miq-tools or perhaps have it in a subdir, so miq-tools/

Comment thread Dockerfile Outdated
@bdunne
Copy link
Copy Markdown
Member Author

bdunne commented Apr 23, 2026

Should this target an el10 branch?

Maybe? Currently we only have a master branch here. Should we tie this to a release of the dbus_api_service or something?

@bdunne
Copy link
Copy Markdown
Member Author

bdunne commented Apr 23, 2026

Related: ManageIQ/container-postgresql#76

@bdunne bdunne mentioned this pull request Apr 23, 2026
14 tasks
@Fryguy Fryguy self-assigned this May 7, 2026
@Fryguy Fryguy added enhancement New feature or request dependencies labels May 7, 2026
@Fryguy
Copy link
Copy Markdown
Member

Fryguy commented May 11, 2026

Needs a rebase for the new script names.

@miq-bot
Copy link
Copy Markdown
Member

miq-bot commented May 18, 2026

Checked commits bdunne/container-httpd-init@6bf6143~...8ca9414 with ruby 3.3.10, rubocop 1.86.0, haml-lint 0.73.0, and yamllint 1.37.1
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@Fryguy Fryguy merged commit 314596f into ManageIQ:master May 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants