Skip to content

Add a default /etc/pelican/pelican.yaml file to the image#2481

Merged
matyasselmeci merged 1 commit into
PelicanPlatform:mainfrom
matyasselmeci:pr/use-config-d.2149
Jul 15, 2025
Merged

Add a default /etc/pelican/pelican.yaml file to the image#2481
matyasselmeci merged 1 commit into
PelicanPlatform:mainfrom
matyasselmeci:pr/use-config-d.2149

Conversation

@matyasselmeci

Copy link
Copy Markdown
Contributor

which just loads all the config from /usr/share/pelican/config.d and /etc/pelican/config.d. Create those directories when building the image.

which just loads all the config from `/usr/share/pelican/config.d` and `/etc/pelican/config.d`.
Create those directories when building the image.
@matyasselmeci matyasselmeci added this to the v7.18 milestone Jul 10, 2025
@matyasselmeci matyasselmeci linked an issue Jul 10, 2025 that may be closed by this pull request
7 tasks
@jhiemstrawisc jhiemstrawisc requested review from jhiemstrawisc and removed request for jhiemstrawisc July 10, 2025 14:09
@patrickbrophy patrickbrophy requested a review from brianhlin July 10, 2025 20:33

@brianhlin brianhlin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This LGTM but was there really no default pelican.yaml config before this? Interesting

@matyasselmeci

Copy link
Copy Markdown
Contributor Author

This LGTM but was there really no default pelican.yaml config before this? Interesting

I guess strictly speaking you don't need one if you're doing all the config by env var.

@matyasselmeci matyasselmeci merged commit 507fdcd into PelicanPlatform:main Jul 15, 2025
25 of 28 checks passed
@matyasselmeci matyasselmeci deleted the pr/use-config-d.2149 branch July 15, 2025 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set default config dir locations

2 participants