refactor: adapt ResourceStatus/SiteStatus consumers to binary model - #8738
refactor: adapt ResourceStatus/SiteStatus consumers to binary model#8738HeloiseJoffe wants to merge 2 commits into
Conversation
a080078 to
873af57
Compare
|
Hi, this one LGTM. At the same time there are some failing tests in the backward compatibility: https://github.com/DIRACGrid/DIRAC/actions/runs/31604149176/job/94138661798?pr=8738#step:13:359 : and https://github.com/DIRACGrid/DIRAC/actions/runs/31604149176/job/94138661798?pr=8738#step:13:491 These are testing that a client of DIRAC version v9.0.X can talk with a server with version v9.1.X (this version). |
aldbr
left a comment
There was a problem hiding this comment.
There are other places that would need to be checked:
- The
if,elifblocks should likely disappear (we should not gate the writing part anymore):DIRAC/src/DIRAC/DataManagementSystem/scripts/dirac_admin_ban_se.py
Lines 116 to 125 in 36806a1
DIRAC/src/DIRAC/DataManagementSystem/scripts/dirac_admin_ban_se.py
Lines 135 to 144 in 36806a1
DIRAC/src/DIRAC/DataManagementSystem/scripts/dirac_admin_ban_se.py
Lines 155 to 165 in 36806a1
- Same here:
DIRAC/src/DIRAC/DataManagementSystem/scripts/dirac_admin_allow_se.py
Lines 125 to 136 in 36806a1
873af57 to
c209b8a
Compare
c209b8a to
640e1fd
Compare
closes #8686
BEGINRELEASENOTES
*ResourceStatusSystem
CHANGE: adapt ResourceStatus/SiteStatus consumers to binary model
ENDRELEASENOTES