Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make docker packaging test more resilient #111205

Merged

Conversation

mark-vieira
Copy link
Contributor

Wrap check for container shutdown log message in an assertBusy() to deal with race conditions.

Closes #111132

@mark-vieira mark-vieira added >test Issues or PRs that are addressing/adding tests :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts auto-backport-and-merge v7.17.23 v8.15.1 v8.14.4 labels Jul 23, 2024
@elasticsearchmachine elasticsearchmachine added Team:Delivery Meta label for Delivery team v8.16.0 labels Jul 23, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@mark-vieira mark-vieira added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jul 23, 2024
@elasticsearchmachine elasticsearchmachine merged commit ce3bc5d into elastic:main Jul 23, 2024
18 checks passed
@mark-vieira mark-vieira deleted the fix-docker-interrupt-test branch July 23, 2024 18:41
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
7.17 Commit could not be cherrypicked due to conflicts
8.15 Commit could not be cherrypicked due to conflicts
8.14 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 111205

mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jul 23, 2024
Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes elastic#111132
# Conflicts:
#	muted-tests.yml
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jul 23, 2024
Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes elastic#111132
# Conflicts:
#	muted-tests.yml
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jul 23, 2024
Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes elastic#111132
# Conflicts:
#	muted-tests.yml
#	qa/os/src/test/java/org/elasticsearch/packaging/test/DockerTests.java
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jul 23, 2024
Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes elastic#111132
# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this pull request Jul 23, 2024
* Make docker packaging test more resilient (#111205)

Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes #111132
# Conflicts:
#	muted-tests.yml
#	qa/os/src/test/java/org/elasticsearch/packaging/test/DockerTests.java

* Fix compile issue
elasticsearchmachine pushed a commit that referenced this pull request Jul 23, 2024
Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes #111132
# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this pull request Jul 23, 2024
Wrap check for container shutdown log message in an `assertBusy()` to
deal with race conditions.

Closes #111132
# Conflicts:
#	muted-tests.yml
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!) :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests v7.17.23 v8.14.4 v8.15.1 v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] DockerTests test600Interrupt failing
2 participants