Skip to content

feat: add /_floci/health endpoint#1

Open
Meemaw wants to merge 6 commits intomainfrom
devin/1774943892-health-endpoint-and-persistence
Open

feat: add /_floci/health endpoint#1
Meemaw wants to merge 6 commits intomainfrom
devin/1774943892-health-endpoint-and-persistence

Conversation

@Meemaw
Copy link
Copy Markdown
Member

@Meemaw Meemaw commented Mar 31, 2026

Summary

Adds a GET /_floci/health endpoint that returns the Floci version, edition, and status of all enabled services. Modeled after LocalStack's /_localstack/health, this gives users and tooling a standard readiness probe.

Example response:

{
  "version": "1.0.11",
  "edition": "community",
  "services": {
    "ssm": "available",
    "sqs": "available",
    "s3": "available",
    "dynamodb": "available"
  }
}

Type of change

  • New feature (feat:)

AWS Compatibility

N/A — /_floci/health is a Floci-internal endpoint, not an AWS API.

Key review points

  • Version resolution: Uses System.getenv("FLOCI_VERSION") (set as a Docker build arg in all Dockerfiles) rather than Quarkus config. Falls back to "dev" when unset.
  • Service status: All enabled services are reported as "available" — there is no per-service health probing since all in-process services are available if Floci is running.
  • Auth: This endpoint has no AWS auth headers — verify it isn't caught by any request-signing validation middleware.
  • Test coverage: Single integration test covers the happy path (all services enabled). No test for disabled-service filtering or version env var resolution.

Checklist

  • ./mvnw test passes locally
  • New or updated integration test added
  • Commit messages follow Conventional Commits

Confidence: 🟢 HIGH — Straightforward JAX-RS endpoint with integration test passing.

Link to Devin session: https://app.devin.ai/sessions/82322c243f164f09bc376a26a72a3ada
Requested by: @Meemaw


Open with Devin

Co-Authored-By: Matej Snuderl <ematej.snuderl@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown

Original prompt from Matej

Can you add persistence to floci?

https://docs.localstack.cloud/aws/capabilities/state-management/persistence/

Basically the idea is that we would build our docker image with setup built in (e.g bake in S3 buckets, sqs queues etc so we wouldn't have to pay the overhead for it each time we start it).

@devin-ai-integration
Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration bot marked this pull request as ready for review March 31, 2026 08:08
…int only

Co-Authored-By: Matej Snuderl <ematej.snuderl@gmail.com>
@devin-ai-integration devin-ai-integration bot changed the title feat: add /_floci/health endpoint and Dockerfile.seed for persistence feat: add /_floci/health endpoint Mar 31, 2026
devin-ai-integration bot and others added 2 commits April 2, 2026 07:18
- Change edition from 'community' to 'floci-open-source'
- Show all services in health response (enabled='running', disabled='available')
- Support both /_floci/health and /_localstack/health paths for backward compatibility

Co-Authored-By: Matej Snuderl <ematej.snuderl@gmail.com>
Co-Authored-By: Matej Snuderl <ematej.snuderl@gmail.com>
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

devin-ai-integration bot and others added 2 commits April 2, 2026 07:25
Co-Authored-By: Matej Snuderl <ematej.snuderl@gmail.com>
Co-Authored-By: Matej Snuderl <ematej.snuderl@gmail.com>
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.

1 participant