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

Address Elasticsearch-related integration test failures caused by timeouts #35366

Merged
merged 2 commits into from
Aug 16, 2023

Conversation

yrodiere
Copy link
Member

I'm not 100% sure about the Gelf tests (they may rely on some of the disabled features), so let's see what CI has to say about it.

cc @gsmet

yrodiere and others added 2 commits August 16, 2023 10:12
…tegration tests

Some settings seem to have only limited effect on what gets disabled on
startup:

* cluster.deprecation_indexing.enabled doesn't seem to disable index templates
* stack.templates.enabled doesn't seem to disable ingest pipelines

And some features don't seem to have any settings to disable them, in
particular the "Fleet" module.

But this is as far as we can go.

Co-Authored-By: marko-bekhta <[email protected]>
…search-java-client

Seems like we had raised the timeout everywhere else, but forgot to do
it there.
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 16, 2023

/cc @gsmet (elasticsearch,hibernate-search), @loicmathieu (elasticsearch)

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Cool, thanks. Let's wait for CI!

@yrodiere yrodiere added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Aug 16, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 16, 2023

Failing Jobs - Building bbccb84

Status Name Step Failures Logs Raw logs
✔️ Gradle Tests - JDK 11
Gradle Tests - JDK 11 Windows Build Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

io.quarkus.gradle.devmode.KotlinProjectWithCompilerArgsDevModeTest.main line 17 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeClient was not fulfilled within 2 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

@gsmet gsmet merged commit e17e766 into quarkusio:main Aug 16, 2023
23 of 24 checks passed
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Aug 16, 2023
@quarkus-bot quarkus-bot bot added this to the 3.4 - main milestone Aug 16, 2023
@gsmet
Copy link
Member

gsmet commented Aug 16, 2023

CI looks happy, let's merge.

@gsmet gsmet modified the milestones: 3.4 - main, 3.3.1 Aug 26, 2023
@yrodiere yrodiere deleted the elasticsearch-timeout branch January 29, 2024 11:25
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.

2 participants