Skip to content

Add restart policy to dlockss-node service#724

Merged
physikerwelt merged 1 commit intomainfrom
physikerwelt-patch-1
Mar 2, 2026
Merged

Add restart policy to dlockss-node service#724
physikerwelt merged 1 commit intomainfrom
physikerwelt-patch-1

Conversation

@physikerwelt
Copy link
Copy Markdown
Member

@physikerwelt physikerwelt commented Mar 2, 2026

restart dlockss (required if IPFS node is still starting as it does not have a proper health check yet)

Summary by CodeRabbit

  • Chores
    • Enhanced service resilience by enabling automatic restart on failure.

restart dlockss (required if IPFS node is still starting as it does not have a proper health check yet)
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 856ab7a and a9c6f32.

📒 Files selected for processing (1)
  • docker-compose-extra.yml

📝 Walkthrough

Walkthrough

A restart policy configuration was added to the dlockss-node service in the Docker Compose extra file, setting it to restart automatically unless explicitly stopped.

Changes

Cohort / File(s) Summary
Docker Compose Configuration
docker-compose-extra.yml
Added restart: unless-stopped policy to the dlockss-node service for automatic restart behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A service that rests, but won't stay down,
Restart policy keeps it safe and sound,
Unless you stop it with care,
The dlockss node runs everywhere! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately summarizes the main change: adding a restart policy to the dlockss-node service.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch physikerwelt-patch-1

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@physikerwelt physikerwelt merged commit 2fac7ed into main Mar 2, 2026
1 check passed
@physikerwelt physikerwelt deleted the physikerwelt-patch-1 branch March 2, 2026 15:26
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