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

Allow for a larger timeout for the Logging-Gelf integration tests #33103

Merged
merged 1 commit into from May 4, 2023

Conversation

Sanne
Copy link
Member

@Sanne Sanne commented May 3, 2023

This fixed the build on podman for me locally.

Resolves #33090

@Sanne
Copy link
Member Author

Sanne commented May 3, 2023

@yrodiere the logstash container image is taking ages to boot, it seems like it needs to connect to ES and uses an image which has some similarities. Would you perhaps know of some way to speed it up like we did for the regular ES image?

@holly-cummins
Copy link
Contributor

Resolves #33090

@yrodiere
Copy link
Member

yrodiere commented May 3, 2023

@yrodiere the logstash container image is taking ages to boot, it seems like it needs to connect to ES and uses an image which has some similarities. Would you perhaps know of some way to speed it up like we did for the regular ES image?

I don't, sorry. I've only ever used logstash to run these integration tests. I suppose I can add optimizing this to my todo-list, but don't hold your breath... :)

@Sanne
Copy link
Member Author

Sanne commented May 3, 2023

@yrodiere no worries, don't spend time on it. It's often worth asking, but I only mean to know if you already happen to know ;)

@quarkus-bot
Copy link

quarkus-bot bot commented May 3, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@gsmet gsmet merged commit 998ff4c into quarkusio:main May 4, 2023
19 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.1 - main milestone May 4, 2023
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.

Test failure: quarkus-integration-test-logging-gelf fails with podman
4 participants