Skip to content

[WIP][?.?] support "remoteonly" value for the Access protocol parameter #8291

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: integration
Choose a base branch
from
Open

[WIP][?.?] support "remoteonly" value for the Access protocol parameter #8291

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b2fe13a
feat: add "remoteonly" value to the SE Access option
Sep 1, 2025
d33ee73
fix: update documentation with the new "remoteonly" Access option
Sep 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion docs/source/AdministratorGuide/Resources/storage.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ DIRAC provides an abstraction of a SE interface that allows to access different
# The name of the DIRAC Plugin module to be used for implementation
# of the access protocol
PluginName = SRM2
# Flag specifying the access type (local/remote)
# Flag specifying the access type (local/remote/remoteonly)
Access = remote
# Protocol name
Protocol = srm
Expand Down Expand Up @@ -50,6 +50,7 @@ Configuration options are:
* ``UseCatalogURL``: default ``False``. If ``True``, use the url stored in the catalog instead of regenerating it
* ``ChecksumType``: default ``ADLER32``. NOT ACTIVE !
* ``Alias``: when set to the name of another storage element, it instanciates the other SE instead.
* ``Access``: Can be ``local``, ``remote`` or ``remoteonly``. Options specify that the protocol can be used in local (e.g. upload from a WN to local SE), remote+local or remote context.
* ``ReadAccess``: default ``True``. Allowed for Read if no RSS enabled (:ref:`activateRSS`)
* ``WriteAccess``: default ``True``. Allowed for Write if no RSS enabled
* ``CheckAccess``: default ``True``. Allowed for Check if no RSS enabled
Expand Down
4 changes: 4 additions & 0 deletions src/DIRAC/Resources/Storage/StorageElement.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1167,6 +1167,10 @@ def __filterPlugins(self, methodName, protocols=None, inputProtocol=None):
log.debug(f"Plugin {protocolSection} not allowed for {methodName}.")
continue

# If protocol is remote only and the context is local, skip it.
if pluginParameters["Access"].lower() == "remoteonly" and localSE:
continue

# If we are attempting a putFile and we know the inputProtocol
if methodName == "putFile" and inputProtocol:
if inputProtocol not in pluginParameters["InputProtocols"]:
Expand Down
4 changes: 2 additions & 2 deletions src/DIRAC/Resources/Storage/StorageFactory.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -333,14 +333,14 @@ def _getConfigStorageProtocols(self, storageName, derivedStorageName=None, seCon
for protocolSectionName, protocolDict in self.protocols.items():
# Now update the local and remote protocol lists.
# A warning will be given if the Access option is not set to local or remote.
if protocolDict["Access"].lower() == "remote":
if protocolDict["Access"].lower() in ("remote", "remoteonly"):
self.remoteProtocolSections.append(protocolSectionName)
elif protocolDict["Access"].lower() == "local":
self.localProtocolSections.append(protocolSectionName)
else:
errStr = (
"StorageFactory.__getProtocolDetails: The 'Access' option \
for %s:%s is neither 'local' or 'remote'."
for %s:%s is not 'local', 'remote' or 'remoteonly'."
% (storageName, protocolSectionName)
)
gLogger.warn(errStr)
Expand Down
Loading