Skip to content

Revert "fix(docker): Move all Centos base images to Fedora base images" - #2114

Merged
hung-nguyen-hoang merged 1 commit into
masterfrom
GRIF-1221-prepare
Oct 30, 2026
Merged

hung-nguyen-hoang merged 1 commit into
masterfrom
GRIF-1221-prepare

Conversation

@hung-nguyen-hoang

@hung-nguyen-hoang hung-nguyen-hoang commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Jira: GRIF-1221

We are still using Centos image on Production So We need temporary revert the coding to prepare an urgent fixing on ticket GRIF-1221 for customers.

Summary by CodeRabbit

  • Chores
    • Updated the application’s container runtime to a newer Java 11 environment based on CentOS 9.
    • No application features or user-facing behavior changed in this update.

@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Walkthrough

Both Dockerfiles switch to the Java 11 JRE CentOS 9 image, update the parent_image label, and remove packages from their install lists.

Changes: Docker image update

Layer / File(s) Summary
Base image and package lists
Dockerfile, DockerfileOldImage
Both files use the Java 11 JRE CentOS 9 image and update the parent_image label. The package-install lists remove gawk and bzip2; DockerfileOldImage also removes gnupg.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Possibly related issues

  • gooddata/gooddata-ruby#2107 tracks Docker image dependencies from both Dockerfiles, which this change updates.

Possibly related PRs

Merge Risk: Minimal · up to 5293c

No verified merge-blocking issue remains in the Dockerfile changes.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title identifies the revert that restores CentOS base images, which matches the Dockerfile changes.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit checks the image tag,
Then updates labels in a careful zigzag.
Two package lists grow more spare,
As CentOS nine takes its place there.
The Dockerfiles match, neat as a pair.

Comment @coderabbitai help to get the list of available commands.

Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants