Skip to content

Modify dockerfile deps#1532

Open
rsun19 wants to merge 1 commit intored-hat-data-services:mainfrom
rsun19:modify-dockerfile-deps
Open

Modify dockerfile deps#1532
rsun19 wants to merge 1 commit intored-hat-data-services:mainfrom
rsun19:modify-dockerfile-deps

Conversation

@rsun19
Copy link

@rsun19 rsun19 commented Mar 18, 2026

https://redhat.atlassian.net/browse/RHOAIENG-48527

Description

How Has This Been Tested?

Test Impact

Request review criteria:

Self checklist (all need to be checked):

  • The developer has manually tested the changes and verified that the changes work
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)
  • The code follows our Best Practices (React coding standards, PatternFly usage, performance considerations)

If you have UI changes:

  • Included any necessary screenshots or gifs if it was a UI change.
  • Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

  • The developer has tested their solution on a cluster by using the image produced by the PR to main

@rsun19 rsun19 closed this Mar 18, 2026
@rsun19 rsun19 reopened this Mar 18, 2026
@rsun19 rsun19 force-pushed the modify-dockerfile-deps branch from 344efbc to 088e811 Compare March 18, 2026 13:14
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Copy link

@lucferbux lucferbux left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@ederign
Copy link

ederign commented Mar 18, 2026

/lgtm
/approve

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.

3 participants