Skip to content

Ensure OpenCloud doesn't start until Tika is ready.#300

Merged
micbar merged 1 commit into
opencloud-eu:mainfrom
jgoclawski:add_tika_healthcheck
Jun 25, 2026
Merged

Ensure OpenCloud doesn't start until Tika is ready.#300
micbar merged 1 commit into
opencloud-eu:mainfrom
jgoclawski:add_tika_healthcheck

Conversation

@jgoclawski

Copy link
Copy Markdown
Contributor

Should fix opencloud-eu/opencloud#2881
I am aware that there's a bigger PR opened (#209), but mine is more oriented on fixing a race condition I believe I've met. And this PR also adds the crucial depends_on part.

@rhafer rhafer requested a review from micbar June 8, 2026 07:21
@jgoclawski

Copy link
Copy Markdown
Contributor Author

@micbar hey, just wanted to check if you could take a look at this PR anytime soon?

I tried to keep it small and easy to review. Unfortunately, wget and curl are not available inside tika, so a less readable solution had to be applied. A bit similar to what you guys had here fe3497e but hopefully without the issues it had. Also based on this: paperless-ngx/paperless-ngx#11672 (comment)

@micbar

micbar commented Jun 25, 2026

Copy link
Copy Markdown
Member

@jgoclawski Thanks! Nice improvement.

@micbar micbar merged commit d965660 into opencloud-eu:main Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

failed to acquire migration lock; skipping migrations

2 participants