Skip to content

Commit

Permalink
Core: Fix ATLAS permission for declaring bad PFNs rucio#4265
Browse files Browse the repository at this point in the history
This is a stopgap solution. It should be followed up in issue rucio#4269.
  • Loading branch information
dchristidis committed Jan 25, 2021
1 parent aa1a64a commit 76bdf47
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/rucio/core/permission/atlas.py
Expand Up @@ -1139,10 +1139,10 @@ def perm_add_bad_pfns(issuer, kwargs):
:param kwargs: List of arguments for the action.
:returns: True if account is allowed, otherwise False
"""
if kwargs['state'] in [str(BadPFNStatus.BAD), str(BadPFNStatus.TEMPORARY_UNAVAILABLE)]:
if kwargs['state'] in [BadPFNStatus.BAD.name, BadPFNStatus.TEMPORARY_UNAVAILABLE.name]:
is_cloud_admin = bool([acc_attr for acc_attr in list_account_attributes(account=issuer) if (acc_attr['key'].startswith('cloud-')) and (acc_attr['value'] == 'admin')])
return _is_root(issuer) or has_account_attribute(account=issuer, key='admin') or is_cloud_admin
elif kwargs['state'] == str(BadPFNStatus.SUSPICIOUS):
elif kwargs['state'] == BadPFNStatus.SUSPICIOUS.name:
return True
return _is_root(issuer)

Expand Down

0 comments on commit 76bdf47

Please sign in to comment.