Skip to content

Add DLOCKSS_IDENTITY_PATH to environment variables#725

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

Add DLOCKSS_IDENTITY_PATH to environment variables#725
physikerwelt merged 1 commit intomainfrom
physikerwelt-patch-1

Conversation

@physikerwelt
Copy link
Copy Markdown
Member

@physikerwelt physikerwelt commented Mar 2, 2026

use the same peer_id for ipfs and dlockss

Summary by CodeRabbit

  • Chores
    • Updated distributed storage service configuration for improved data persistence and management capabilities.

use the same peer_id for ipfs and dlockss
@physikerwelt physikerwelt merged commit d3c163a into main Mar 2, 2026
1 check was pending
@physikerwelt physikerwelt deleted the physikerwelt-patch-1 branch March 2, 2026 15:56
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 2, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2fac7ed and 6d8d64d.

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

📝 Walkthrough

Walkthrough

The dlockss-node service in docker-compose-extra.yml was updated with a new environment variable DLOCKSS_IDENTITY_PATH pointing to "/ipfs/keystore" and a read-only volume mount for ipfs-data at /ipfs/.

Changes

Cohort / File(s) Summary
DLOCKSS Node Configuration
docker-compose-extra.yml
Added environment variable DLOCKSS_IDENTITY_PATH: "/ipfs/keystore" and read-only volume mount ipfs-data:/ipfs/:ro to the dlockss-node service.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • PR #722: Adds DLOCKSS_IPFS_NODE environment variable and ./dlockss-files:/data:ro mount to dlockss-node service, expanding IPFS configuration.
  • PR #723: Adds DLOCKSS_NODE_NAME environment variable to dlockss-node service, continuing service configuration enhancements.

Poem

🐰 A hopping good change to the Docker compose,
With IPFS paths now set, oh what a dose!
Identity keystore locked in place,
Read-only volumes keep the pace! 🔐

✨ 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

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.

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