Skip to content

fix: changing the error log to work with an unoptimized image#49384

Merged
shawkins merged 1 commit into
keycloak:mainfrom
shawkins:iss49383
May 28, 2026
Merged

fix: changing the error log to work with an unoptimized image#49384
shawkins merged 1 commit into
keycloak:mainfrom
shawkins:iss49383

Conversation

@shawkins

Copy link
Copy Markdown
Contributor

closes: #49383

closes: keycloak#49383

Signed-off-by: Steve Hawkins <shawkins@redhat.com>

@michalvavrik michalvavrik left a comment

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.

Changes itself looks fine, thanks.

As far as reason for change, why now and what has changed:

  • there is not whole lot of information, my reasoning is that the test is only passing when custom image is used which happens in the Github CI via -Dtest.operator.custom.image=custom-keycloak, but locally I may not set it and still want tests to be passing. Put it that way - LGTM.

Thanks

@shawkins

Copy link
Copy Markdown
Contributor Author

Changes itself looks fine, thanks.

As far as reason for change, why now and what has changed:

  • there is not whole lot of information, my reasoning is that the test is only passing when custom image is used which happens in the Github CI via -Dtest.operator.custom.image=custom-keycloak, but locally I may not set it and still want tests to be passing. Put it that way - LGTM.

Thanks

Yes, sorry that is effectively in the issue, but not directly stated on the PR. The further backstory is that productization was running this test without a custom image in a pipeline and noticed it didn't pass.

@shawkins
shawkins marked this pull request as ready for review May 28, 2026 11:37
@shawkins
shawkins requested review from a team as code owners May 28, 2026 11:37

@Pepo48 Pepo48 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks Steve!

@shawkins
shawkins merged commit 4934e2c into keycloak:main May 28, 2026
140 of 142 checks passed
vmuzikar pushed a commit that referenced this pull request May 29, 2026
#49480)

closes: #49383


(cherry picked from commit 4934e2c)

Signed-off-by: Steve Hawkins <shawkins@redhat.com>
@stianst stianst mentioned this pull request Jun 1, 2026
sashyo pushed a commit to tide-foundation/keycloak-IGA that referenced this pull request Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Operator KeycloakDeploymentTest.testConfigErrorLog fails without custom image

4 participants