Skip to content
Snippets Groups Projects

[devel] Add option to dirac-dms-add-transformation

Merged Philippe Charpentier requested to merge charpent/LHCbDIRAC:devel-PhC-190308 into devel

In order to find out from the production request DB which processing passes are active, and thus try and launch only the relevant replications, add an option --MCVersion that can take the following values:

  • "All": all numerical values of the MCVersion (i.e. the ConfigVersion of the BK) are considered
  • Any numerical value or comma-separated list of values (e.g. --MCVersion 2011,2012)

It forces the option --Unique in order to not re-launch an already existing replication

The other options can be used like --Start, --ListProcessingPasses or --Test

===========

Fix getSEStorageSpace() as the DB content was changed and token is now seName

Edited by Philippe Charpentier

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Christophe Haen approved this merge request

    approved this merge request

  • Philippe Charpentier resolved all discussions

    resolved all discussions

  • Federico Stagni added Review OK label and removed To be reviewed label

    added Review OK label and removed To be reviewed label

  • added 5 commits

    Compare with previous version

  • Added a fix to getSEStorageSpace() This MUST be merged before releasing v9r3

  • added To be reviewed label and removed Review OK label

  • Philippe Charpentier changed the description

    changed the description

  • 406 406
    407 407 """
    408 408
    409 # Given a SE, we need to find its endpoint
    410 endpoint = CSHelpers.getStorageElementEndpoint(seName)
    411 if not endpoint['OK']:
    412 return endpoint
    413 endpoint = endpoint['Value']
    414
    415 spaceToken = CSHelpers.getSEToken(seName)
    416 if not spaceToken['OK']:
    417 return spaceToken
    418 spaceToken = spaceToken['Value']
    419
    420 res = self.selectSpaceTokenOccupancyCache(endpoint, spaceToken)
    409 # FIXME: The DB contains the SE name as token... This is dirty but mandatory
    • Thanks.

      Actually, the only client of this method is in PluginUtilities.py, that could maybe directly call self.selectSpaceTokenOccupancyCache(token=seName) , and this method (getSEStorageSpace) can then be deleted.

    • Philippe is AFK now. Let's merge it this way now, but I agree with you. Let's keep it for a later PR

    • Please register or sign in to reply
  • If the change is for everybody in the DIRAC DB, I would suggest to change the DIRAC method with a selectSEOccupancyCache(seName=seName) rather than token=seName ;-)

  • I prefer having the ugly stuff in the RSS code than in the TS ;-)

  • Philippe Charpentier changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading