Skip to content

feat: add boot service status command#79

Merged
synackd merged 3 commits into
mainfrom
feat/boot-service-health
May 13, 2026
Merged

feat: add boot service status command#79
synackd merged 3 commits into
mainfrom
feat/boot-service-health

Conversation

@synackd
Copy link
Copy Markdown
Collaborator

@synackd synackd commented May 13, 2026

Pull Request Template

Thank you for your contribution! Please ensure the following before submitting:

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Description

boot-service v0.1.5 exposes the /health endpoint. This PR adds the boot service status command to query it from the CLI.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

For more info, see Contributing Guidelines.

synackd added 3 commits May 13, 2026 12:37
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@davidallendj davidallendj left a comment

Choose a reason for hiding this comment

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

LGTM. Tested local version with ochami boot service status and haproxy and worked without issues like expected.

@synackd synackd merged commit f07bc1e into main May 13, 2026
12 checks passed
@synackd synackd deleted the feat/boot-service-health branch May 13, 2026 22:14
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