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

Do not require space tokens for SRM #574

Closed
bbockelm opened this issue Feb 1, 2018 · 3 comments
Closed

Do not require space tokens for SRM #574

bbockelm opened this issue Feb 1, 2018 · 3 comments

Comments

@bbockelm
Copy link
Contributor

bbockelm commented Feb 1, 2018

At many CMS sites, space tokens are not used even when SRM is utilized. However, it's not currently possible to skip this setting.

It may be as simple as setting the space token to the empty string (``), which could be done automatically if not specified.

However, there might be hidden dependencies that want a non-empty-string space token. Needs investigation.

@nataliaratnikova
Copy link
Contributor

Currently Rucio unconditionally passes spacetoken json parameter to FTS job. No matter what value is passed, this causes fts error on CMS sites which do not support spacetokens.
The desired behavior is: if space_token is not in the 'extended_attributes' , do not add spacetoken to fts json.

@bari12 bari12 added this to the 1.14.9 milestone Feb 2, 2018
bari12 added a commit that referenced this issue Feb 2, 2018
…pacetokens

Conveyor: Handle empty spacetokens correctly. Fix #574
@bari12 bari12 closed this as completed in 5b4c95a Feb 2, 2018
bari12 added a commit that referenced this issue Feb 2, 2018
…pacetokens

Conveyor: Handle empty spacetokens correctly. Fix #574
@nataliaratnikova
Copy link
Contributor

nataliaratnikova commented Feb 2, 2018

Hi @bbockelm , how do I now define an RSE so that it would not pass the spacetoken to FTS?
Adding 'srm' protocol still requires a non-empty spacetoken argument :
# rucio-admin rse add-protocol --hostname 'stormfe1.pi.infn.it' --scheme 'srm' --prefix '/cms/store/test/rucio' --web-service-path '/srm/managerv2?SFN=' --port '8444' --impl 'rucio.rse.protocols.gfalv2.Default' --domain-json '{"wan": {"read": 1, "write": 1, "delete": 1, "third_party_copy": 1}}' --space-token '' T2_IT_Pisa
Error: space-token and web-service-path must be provided for SRM endpoints.

bbockelm added a commit to bbockelm/rucio that referenced this issue Feb 13, 2018
Outside ATLAS (particularly, for CMS), space tokens are optional at
some SRM endpoints.  Previously, for rucio#574, we fixed the conveyor
to handle empty space tokens.  Additionally, the fix for the client-side
of rucio#575 got mistakenly wrapped up in rucio#570 (pluggable LFN2PFN) and put
into `next`.

This is the missing piece for `master`.
yuyiguo pushed a commit to yuyiguo/rucio that referenced this issue Sep 20, 2023
…_tapes

update static usage for tape rses
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants