Skip to content

Conversation

@parkervcp
Copy link
Contributor

I forgot to mkdir for plugins

I forgot to mkdir for plugins
@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

📝 Walkthrough

Walkthrough

The entrypoint script is updated to include the /pelican-data/plugins directory in the list of volume directories created during container initialization. This ensures the plugins directory exists with proper ownership and permissions on container startup.

Changes

Cohort / File(s) Summary
Docker entrypoint configuration
docker/entrypoint.sh
Added /pelican-data/plugins directory to the volume directory creation list during container initialization.
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix plugins in entrypoint' directly relates to the changeset, which adds the /pelican-data/plugins directory to the entrypoint script.
Description check ✅ Passed The description 'I forgot to mkdir for plugins' accurately describes the change made in the pull request, explaining the forgotten directory creation.
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.


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.

@HttpRafa
Copy link

@parkervcp Will this cause any issues when using the Docker image? I’m worried about bricking the panel by installing plugins.

@parkervcp parkervcp merged commit 8e8ce3b into pelican-dev:main Jan 19, 2026
32 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants