Skip to content

fix: add skip_local_config_file variable to avoid auth errors at CERN…#8637

Open
aldbr wants to merge 1 commit into
DIRACGrid:integrationfrom
aldbr:fix-htcondor-resource-usage
Open

fix: add skip_local_config_file variable to avoid auth errors at CERN…#8637
aldbr wants to merge 1 commit into
DIRACGrid:integrationfrom
aldbr:fix-htcondor-resource-usage

Conversation

@aldbr

@aldbr aldbr commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

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

BEGINRELEASENOTES
*Resources
FIX: add an environment variable to get time left in allocations at CERN
ENDRELEASENOTES

@aldbr aldbr requested review from atsareg and fstagni as code owners June 22, 2026 07:11
@aldbr aldbr force-pushed the fix-htcondor-resource-usage branch from 7623fdd to 45a8bb2 Compare June 22, 2026 07:12
@fstagni

fstagni commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

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

@sfayer

sfayer commented Jun 22, 2026

Copy link
Copy Markdown
Member

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

aldbr commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

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!

@aldbr aldbr force-pushed the fix-htcondor-resource-usage branch from 45a8bb2 to 8f34524 Compare June 22, 2026 13:41
@aldbr aldbr force-pushed the fix-htcondor-resource-usage branch from 8f34524 to e34ef92 Compare June 22, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants