[aldbr]

It's very specific but the whole HTCondorResourceUsage is already very specific anyway...

UPDATE: given the many recent changes with the time left computation, I think it's fine not to keep the CERN-specific HTCondorResourceUsage module (that anyway nobody can use currently). It's hard to maintain as we currently don't have any hint if CERN is changing its configuration (that's what happened here)

BEGINRELEASENOTES
*Resources
CHANGE: remove HTCondorResourceUsage
ENDRELEASENOTES

[fstagni]

I need an HTCondor expert to review this. Maybe @sfayer ?

[sfayer]

Hmm, I don't recognise the SKIP_LOCAL_CONFIG_FILE variable, is that a CERN specific gate they've put in their config? If it is, a more portable version might be CONDOR_CONFIG=/dev/null which should work everywhere, although would need testing.

I also note that the runCommand function probably doesn't use a shell, so including variables like that before the command may not work as expected: You might need to plumb through the "env" parameter so that you can set it that way).

An alternate approach could be to read the ads file directly from python: It's a simple "key = value" text-file format (and if this is running in a context with diracos2, the classad module is also likely available). If you take that approach note that "CurrentTime" isn't in the file and would need generating in python instead.

Regards,
Simon

[aldbr]

This variable (and the whole module actually) is HTCondor CERN-specific and the variable is a recommendation from the CERN HTCondor experts.

I also note that the runCommand function probably doesn't use a shell, so including variables like that before the command may not work as expected: You might need to plumb through the "env" parameter so that you can set it that way).

An alternate approach could be to read the ads file directly from python: It's a simple "key = value" text-file format (and if this is running in a context with diracos2, the classad module is also likely available). If you take that approach note that "CurrentTime" isn't in the file and would need generating in python instead.

Indeed, I am going to check what's the best approach, thanks!