Skip to content

Conversation

@andrasbacsai
Copy link
Member

This pull request introduces significant improvements to the build, deployment, and documentation processes for the Sentinel project. The main highlights are a unified and modernized Docker build workflow supporting multiple architectures, enhanced documentation for onboarding and usage, and various code and dependency updates for maintainability and transparency.

Build and CI/CD Workflow Modernization:

  • Refactored GitHub Actions workflows (release.yaml and release-next.yaml) to use a unified matrix-based build-push job for both amd64 and aarch64 (ARM64) architectures, removing duplicated job definitions and enabling easier maintenance. The manifest creation step was updated to use the correct registries and tagging conventions, and the workflows now use the latest action versions and runners. [1] [2] [3] [4] [5] [6]
  • Updated the Docker build process to dynamically set the target architecture using the GOARCH environment variable, enabling a single Dockerfile to build images for multiple platforms. The now-redundant Dockerfile.arm64 has been removed. [1] [2]
  • Added a local build helper script build.sh for quick and customizable Docker image builds, improving developer experience.

Documentation Enhancements:

  • Overhauled the README.md to provide detailed feature descriptions, quick start guides, configuration instructions, API reference, architecture overview, development and deployment examples, and support information, making it easier for new users and contributors to get started.

Code and Dependency Updates:

  • Updated the version constant in pkg/config/config.go to 0.0.17, reflecting the new release.
  • Minor logging improvement: the application now logs its version at startup for better traceability.
  • Updated Go module dependencies to their latest versions, ensuring security and compatibility. [1] [2]
  • Changed the default container health status from "unhealthy" to "unknown" in the push service, providing more accurate reporting when health information is unavailable.

dependabot bot and others added 11 commits November 20, 2025 08:42
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.42.0 to 0.45.0.
- [Commits](golang/crypto@v0.42.0...v0.45.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-version: 0.45.0
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
…rg/x/crypto-0.45.0

chore(deps): bump golang.org/x/crypto from 0.42.0 to 0.45.0
Refactor release workflow and consolidate Dockerfiles
refactor: streamline GitHub Actions workflows and remove obsolete files
…check

Previously, containers without a healthcheck definition were incorrectly
reported as "unhealthy". This caused confusion in the Coolify UI, showing
containers as unhealthy when they were actually functioning correctly.

The Docker API returns no Health field for containers without healthchecks.
When this field is nil, Sentinel should report the health status as "unknown"
instead of "unhealthy".

This change updates pkg/push/push.go:154 to default to "unknown" instead of
"unhealthy", matching the behavior of Coolify's SSH-based status checks.

Expected behavior after fix:
- Containers WITH healthcheck: Report actual status (healthy/unhealthy/starting)
- Containers WITHOUT healthcheck: Report "unknown"
- Containers with nil State: Continue to report "unknown" (edge case)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
- Update version from 0.0.15 to 0.0.17
- Add startup log message showing Sentinel version on startup

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
fix: default to unknown health status for containers without healthcheck
- Create API.md with detailed endpoint documentation, examples, and error responses
- Add openapi.yaml OpenAPI 3.0 specification for interactive API docs
- Enhance README.md with quick start, configuration, and deployment examples

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Add comprehensive API documentation
@andrasbacsai andrasbacsai changed the title Next v0.0.17 Nov 20, 2025
@andrasbacsai andrasbacsai merged commit 96289d8 into main Nov 20, 2025
4 checks passed
@andrasbacsai andrasbacsai deleted the next branch November 20, 2025 10:03
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