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

Installing the OSDF Cache by Container document #225

Conversation

matyasselmeci
Copy link
Contributor

I took most of the language from the RPM doc. I was using the docs from Pelican main as a reference, though none of the knobs I used so far require 7.12.

I haven't written docs on tuning the cache parameters (e.g. high/low water mark), nor how to migrate from XCache; migrating a 'multiple data, multiple meta' config will require parameters from 7.12, though.

Copy link
Contributor

@brianhlin brianhlin left a comment

Choose a reason for hiding this comment

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

Some minor fixups in this PR matyasselmeci#3

Some comments in-line and I'm not sure I'm in live with the sections / ordering but I don't have any concrete suggestions atm. Otherwise I think this is a good first pass.

docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
The host is superfluous and apparently confusing whether the word "single"
refers to the host, or refers to the partition.
Copy link
Contributor Author

@matyasselmeci matyasselmeci left a comment

Choose a reason for hiding this comment

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

Add updates from review comments

docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
docs/data/osdf/install-cache-container.md Outdated Show resolved Hide resolved
@matyasselmeci matyasselmeci merged commit d762f75 into osg-htc:master Jan 9, 2025
1 check failed
@matyasselmeci matyasselmeci deleted the pr/osdf-cache-container.SOFTWARE-6013 branch January 9, 2025 15:25
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.

2 participants