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

config.MkdirAll does not return an error if the requested directory already exists but does not have the requested permissions #1830

Open
brianaydemir opened this issue Dec 14, 2024 · 0 comments
Assignees
Labels
bug Something isn't working cache Issue relating to the cache component critical High priority for next release
Milestone

Comments

@brianaydemir
Copy link
Contributor

Pretty much what the title says.

Pelican can be configured to read/write data from many locations on the filesystem. Some of these locations are passed on to other applications, such as XRootD. Especially for these other applications, it is not sufficient to check that the directory merely exists. The directory must exist with the correct permissions — which Pelican knows, but does not check for if the directory already exists.

This issue came to light where OSDF caches running v7.11.8 crashed on start up, but not on earlier versions. To make a long story short, the caches crashed on start-up because an upgraded version of XRootD performed additional sanity checks on its configuration. Pelican is responsible for providing XRootD it's configuration in the first place, so Pelican should be performing those same, or at least substantively similar, sanity checks.

@brianaydemir brianaydemir added bug Something isn't working critical High priority for next release cache Issue relating to the cache component labels Dec 14, 2024
@brianaydemir brianaydemir added this to the v7.13.0 milestone Dec 14, 2024
@brianaydemir brianaydemir self-assigned this Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cache Issue relating to the cache component critical High priority for next release
Projects
None yet
Development

No branches or pull requests

1 participant