Skip to content

Fail-stop multiple origin config for globus (for now)#3170

Merged
jhiemstrawisc merged 1 commit intoPelicanPlatform:mainfrom
whwjiang:fix-3163
Feb 27, 2026
Merged

Fail-stop multiple origin config for globus (for now)#3170
jhiemstrawisc merged 1 commit intoPelicanPlatform:mainfrom
whwjiang:fix-3163

Conversation

@whwjiang
Copy link
Contributor

Fixes #3163

We validate that the export doesn't configure multiple origins. I also inverted the "multiple origin export for Globus" unit test

Copy link
Member

@jhiemstrawisc jhiemstrawisc left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!!

@jhiemstrawisc
Copy link
Member

The failing test is unrelated to this diff and looks like it might be a new flaky test. Pinging @bbockelm to take a look at https://github.com/PelicanPlatform/pelican/actions/runs/22501985335/job/65191127749?pr=3170

@jhiemstrawisc jhiemstrawisc merged commit 4f77c6a into PelicanPlatform:main Feb 27, 2026
27 of 29 checks passed
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.

Globus Origins currently only support one export, but don't assert more aren't given

2 participants