Skip to content

dev: Add docker compose SQL health check - #1272

Open
addshore wants to merge 1 commit into
wbstack:mainfrom
addshore:20260925-sql-healthcheck
Open

addshore wants to merge 1 commit into
wbstack:mainfrom
addshore:20260925-sql-healthcheck

Conversation

@addshore

Copy link
Copy Markdown
Collaborator

Wait for MariaDB to accept connections before the API starts. This avoids setup commands racing database startup.

@outdooracorn
outdooracorn changed the base branch from main to ollie-skip-mattermost-notifications September 28, 2026 19:50
@outdooracorn
outdooracorn force-pushed the 20260925-sql-healthcheck branch from 90050b8 to 012a278 Compare September 28, 2026 19:52
@outdooracorn
outdooracorn deleted the branch wbstack:main September 28, 2026 20:25
@addshore

Copy link
Copy Markdown
Collaborator Author

Not sure what happened with this one, but it shows as closed? and force pushed? but also not merged... and I cant reopen it?

@outdooracorn

outdooracorn commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Ugh! This was me trying to stack it with #1281 - I forgot your can't use the GitHub Stacked PRs across repos :(

The force push was me trying to add it to the stack, the closing was GitHub being stupid!

Sorry, you might have to recreate this PR.

@addshore addshore reopened this Sep 28, 2026
@addshore

Copy link
Copy Markdown
Collaborator Author

Interestingly, after you commented, the Reopen button enabled for me!
So this is open again now!

@addshore
addshore changed the base branch from ollie-skip-mattermost-notifications to main September 28, 2026 23:55
@outdooracorn

Copy link
Copy Markdown
Member

Interestingly, after you commented, the Reopen button enabled for me! So this is open again now!

I think it was because I created the ollie-skip-mattermost-notifications branch again that this PR was based on.

Wait for MariaDB to accept connections before the API starts. This avoids setup commands racing database startup.
@outdooracorn
outdooracorn force-pushed the 20260925-sql-healthcheck branch from 012a278 to ce89525 Compare September 29, 2026 06:32
@addshore addshore changed the title Add SQL health check Add docker compose development SQL health check Sep 29, 2026
@addshore addshore changed the title Add docker compose development SQL health check dev: Add docker compose SQL health check Sep 29, 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.

2 participants