Skip to content

[9.2] Fix tests and logging for Docker-based tests and unmute the tests (#141098)#141294

Merged
elasticsearchmachine merged 1 commit intoelastic:9.2from
jozala:backport/9.2/pr-141098
Jan 26, 2026
Merged

[9.2] Fix tests and logging for Docker-based tests and unmute the tests (#141098)#141294
elasticsearchmachine merged 1 commit intoelastic:9.2from
jozala:backport/9.2/pr-141098

Conversation

@jozala
Copy link
Contributor

@jozala jozala commented Jan 26, 2026

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

…astic#141098)

* Unmute DockerTests in muted-tests.yml

Removed muting for multiple Docker-related integration tests in
`muted-tests.yml`, as they are likely resolved in elastic#140701 and ready for
re-evaluation.

* Refactor authentication retry logic in ServerUtils

Removed unnecessary boolean flag for retrying on authentication failure

* Fix Docker log dumps

Fixed the problem with running `docker logs null` after test failure.
The problem was caused by calling the `removeContainer()` before
`dumpDebug()` because `dumpDebug()` is called from the @rule on the
base class and `removeContainer()` from @after method on
the child class.

* Fix conditional logic in ServerUtils for cluster health check

Adjusted logic to properly set the `started` flag only after a successful cluster health API response.

* Make the logging less verbose

This warning log with exception was very loud and sometimes hide the
real issue.
It has been changed to only log the exception message.

(cherry picked from commit bb91f1e)

# Conflicts:
#	muted-tests.yml
@jozala jozala added >non-issue :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) and removed v9.2.5 labels Jan 26, 2026
@elasticsearchmachine elasticsearchmachine merged commit c7123da into elastic:9.2 Jan 26, 2026
35 checks passed
@jozala jozala deleted the backport/9.2/pr-141098 branch January 26, 2026 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >non-issue

2 participants