update the base image to red hat ubi 9.7#12663
Merged
kopernic-pl merged 3 commits into3.0.0from Jan 12, 2026
Merged
Conversation
cleanup dockerfiles a bit as well
There was a problem hiding this comment.
Pull request overview
This pull request updates the Dockerfiles for the swagger-generator module by upgrading the Red Hat UBI base image from version 9.6 to 9.7 and consolidating multiple RUN commands into single statements to improve build efficiency and reduce image layers.
Key changes:
- Base image upgraded from
redhat/ubi9-minimal:9.6toredhat/ubi9-minimal:9.7 - Multiple sequential RUN commands consolidated into single multi-line statements
- Package installation commands merged in the main Dockerfile for better build efficiency
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| modules/swagger-generator/Dockerfile_root | Upgraded base image to 9.7 and consolidated directory/permission operations into a single RUN command |
| modules/swagger-generator/Dockerfile | Upgraded base image to 9.7, consolidated package installation commands, and merged user/group/ownership operations |
Comments suppressed due to low confidence (1)
modules/swagger-generator/Dockerfile_root:5
- The package installation commands should be consolidated into a single
microdnf installstatement (similar to the main Dockerfile) and includemicrodnf clean allat the end to reduce image size.
RUN microdnf -y install which && \
microdnf -y install glibc-langpack-en && \
microdnf -y install tar && \
microdnf -y install gzip
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
daniel-kmiecik
previously approved these changes
Jan 9, 2026
…t/ubi9-minimal:9.7
daniel-kmiecik
approved these changes
Jan 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR checklist
./bin/to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.shand./bin/security/{LANG}-petstore.shif updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\.3.0.0branch for changes related to OpenAPI spec 3.0. Default:master.Description of the PR
This pull request updates the Dockerfiles used for the swagger-generator module to streamline the build process and keep the base images up to date. The main changes include upgrading the Red Hat UBI base image version, consolidating package installation commands, and simplifying multi-step RUN commands for efficiency and maintainability.
Base image and package installation updates:
redhat/ubi9-minimal:9.6toredhat/ubi9-minimal:9.7in bothDockerfileandDockerfile_rootto ensure the latest security patches and features are used. [1] [2]microdnf installcommands into a single command inDockerfilefor improved build efficiency.Build process simplification:
RUNcommands (such as directory creation, permission changes, and file removal) into singleRUNstatements in both Dockerfiles, making the Dockerfiles cleaner and reducing the number of layers in the resulting images. [1] [2]RUNcommand inDockerfilefor better readability and efficiency.