Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Writebackhost never set for Pelican origins outside of topology #507

Closed
joereuss12 opened this issue Dec 18, 2023 · 2 comments · Fixed by #622
Closed

Writebackhost never set for Pelican origins outside of topology #507

joereuss12 opened this issue Dec 18, 2023 · 2 comments · Fixed by #622
Assignees
Labels
bug Something isn't working enhancement New feature or request
Milestone

Comments

@joereuss12
Copy link
Contributor

As of now, writing is impossible to pelican origins outside of topology because writebackhost is never set. Starting a new Pelican origin does not tell the director how to write back to that origin, in addition the director doesn't have any way to tell the client how to write back either.

@joereuss12 joereuss12 added this to the v7.5.0 milestone Dec 18, 2023
@joereuss12 joereuss12 self-assigned this Dec 19, 2023
@joereuss12
Copy link
Contributor Author

Update: a temporary fix is in place that the writebackhost is automatically set to the origin url. This should become configurable/set up with a flag when the user runs pelican origin serve

@joereuss12 joereuss12 added bug Something isn't working enhancement New feature or request labels Dec 19, 2023
@joereuss12 joereuss12 linked a pull request Dec 20, 2023 that will close this issue
@joereuss12 joereuss12 linked a pull request Jan 8, 2024 that will close this issue
@joereuss12
Copy link
Contributor Author

This is fixed with #622, Origins now have a boolean indicating whether they are writeable or not. In addition, instead of looking for the writebackhost for PUTs, we now get the origin_url directly and check if that origin is writeable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
1 participant