Skip to content

Fall back to system config location for non-root users if user config is not provided#2990

Merged
turetske merged 9 commits into
PelicanPlatform:mainfrom
jhiemstrawisc:issue-2954
Jan 14, 2026
Merged

Fall back to system config location for non-root users if user config is not provided#2990
turetske merged 9 commits into
PelicanPlatform:mainfrom
jhiemstrawisc:issue-2954

Conversation

@jhiemstrawisc

Copy link
Copy Markdown
Member

I thought this was a feature previously, but it turns out it probably wasn't.

In any case, Pelican now falls back to its standard system config location '/etc/pelican' in the event that a non-root user doesn't supply their own config.

This behavior is needed for site-local caches because HTCondor jobs don't run as root, and system admins need a way to configure which Cache an EP will use without pre-generating config for each individual user of the EP.

@jhiemstrawisc

jhiemstrawisc commented Jan 14, 2026

Copy link
Copy Markdown
Member Author

Note that I decided to pull on a thread here a bit, thus the explosion in size -- most of the noise comes from the last commit, which is where I ran a markdown formatter on our docs to enforce the "one sentence per line" standard some subset of "we" agreed on some time ago but never enforced.

I will take this PR back out of draft mode once I double check that docs build/render correctly.

@jhiemstrawisc jhiemstrawisc marked this pull request as draft January 14, 2026 17:35

@turetske turetske left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A few very minor comments. I didn't bother checking every change your markdown formatting did.

Obviously make sure the linter check succeeds and the docs look good and ping me again.

Comment thread config/config_test.go Outdated
Comment thread docs/app/parameters/page.mdx Outdated
… not provided

I thought this was a feature previously, but it turns out it probably wasn't.

In any case, Pelican now falls back to its standard system config location
'/etc/pelican' in the event that a non-root user doesn't supply their own
config.

This behavior is needed for site-local caches because HTCondor jobs don't
run as root, and system admins need a way to configure which Cache an EP
will use without pre-generating config for each individual user of the EP.
This is a standard some subset of the Royal "We" agreed on a while ago,
but a standard uncommunicated and unvalidated is no standard at all.

While my eyes are on this page, I'll fix it up. Soon to come -- a linter
rule!
I hadn't realized we already run pre-commit in CI
Sigh, looks like mdformat works for .md but not .mdx
@jhiemstrawisc

Copy link
Copy Markdown
Member Author

I gave up on trying to format .mdx files -- it turns out mdformat was breaking a lot of the JSX integration for our docs, so I had to remove it.

@jhiemstrawisc jhiemstrawisc marked this pull request as ready for review January 14, 2026 19:38

@turetske turetske left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

configuration critical High priority for next release

Projects

None yet

2 participants