Fallback to /etc/condor/condor_config

[mapsacosta]

The Schedd class is only capable of using a condor_config file present in the same folder. In order to make this code friendlier with people running their own variations of condor, the line:

lpcjobqueue/src/lpcjobqueue/schedd.py

Line 7 in b08e3c7

os.environ["CONDOR_CONFIG"] = os.path.join(os.path.dirname(__file__), "condor_config")

Should fallback to the default directory /etc/condor/condor_config and then fail.

The location of the condor_config could also be a configurable parameter

[nsmith-]

I think then we would want a separate docker image on top of docker-coffea-dask that adds this file, for use at LPC, right?

[mapsacosta]

Yep, in our case we just pull the file (and other condor configs) from Git with an initContainer.
I could also see this as either a configurable parameter or an environment variable, especially if we're trying to extend to other places (such as lxplus). I am working on a quick MR to implement falling back to the default condor installation file and we can re-iterate if needed.

[nsmith-]

Sounds good. Then, what to do about the schedd picking algorithm? It currently is somewhat LPC-specific, idk what it would do at other sites.

[nsmith-]

As of #21 we now just always load the path

lpcjobqueue/src/lpcjobqueue/schedd.py

Line 8 in 6713349

os.environ["CONDOR_CONFIG"] = os.path.join("/srv/.condor_config")

Does that work for you, or should we do a if os.path.exists and then fallback to /etc/condor/condor_config? (actually if we just don't set the env variable, that is the default right?