Skip to content

docs(Client): Description of multi-RI functionality - #861

Open
Twokubikmeter wants to merge 2 commits into
rucio:mainfrom
Twokubikmeter:multihost
Open

Twokubikmeter wants to merge 2 commits into
rucio:mainfrom
Twokubikmeter:multihost

Conversation

@Twokubikmeter

Copy link
Copy Markdown
Contributor

Documentation for the multi-RI functionality introduced in Rucio 39.

@bari12 bari12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The text seems wrong,

Comment thread docs/user/using_the_client.md Outdated

To enable the functionality a configuration is needed for each rucio server
being accessed. The primary config file needs to have values for `multi_host_commands`
and `multi_host_servers`. `multi_host_commands` should contain a comma separated file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

comma separated file?
I dont't think this is right.

Comment thread docs/user/using_the_client.md
Comment thread docs/user/using_the_client.md Outdated
auth_token_file_path = /path/to/token/file
```

## Multi-RI configuration

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think in the upstream repo we called this multi host configuration. It's also worth to explain what this is trying to achieve here.

Comment thread docs/user/using_the_client.md Outdated

Some setups may want to call two rucio servers in parallel. This could be two experiments wanting to share data stored in similair way.

To enable the functionality a configuration is needed for each rucio server

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
To enable the functionality a configuration is needed for each rucio server
To enable the functionality a configuration is needed for each Rucio server

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry about being pedantic, but all instances of Rucio should be capitalized


Some setups may want to call two rucio servers in parallel. This could be two experiments wanting to share data stored in similair way.

To enable the functionality a configuration is needed for each rucio server

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry about being pedantic, but all instances of Rucio should be capitalized

This branch has not been deployed

No deployments
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.

3 participants