Project

General

Profile

Feature #3104

Changing the order of config file lookup for glidecondor_addDN

Added by Igor Sfiligoi about 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Igor Sfiligoi
Category:
-
Target version:
Start date:
11/02/2012
Due date:
% Done:

0%

Estimated time:
Stakeholders:
Duration:

Description

glidecondor_addDN when used in -daemon mode,
currently first looks up for LOCAL_CONFIG,
and only if that cannot be found, it will try the config.d files.

I am proposing we change this logic;
it should first try to put the changes into a dedicated config.d file,
and only if that does not work, modify LOCAL_CONFIG.

History

#1 Updated by Igor Sfiligoi about 8 years ago

  • Status changed from New to Feedback
  • Assignee changed from Igor Sfiligoi to Douglas Strain

Comitted to
branch_v2plus_igor_3104

Doug, can you have a look?

#2 Updated by Douglas Strain about 8 years ago

  • Assignee changed from Douglas Strain to Igor Sfiligoi

Please change 'condorExe.iexe_cmd("condor_config_val LOCAL_CONFIG_DIR")[0][:-1]' to use strip() instead. I think we've decided in a different ticket that was a better method.

Other than that, the fix looks fine to me. Feel free to merge after you have changed that.

#3 Updated by Igor Sfiligoi about 8 years ago

I am trying to minimize the changes... just moving blocks of code around.

Will address the strip issue in #3054.

Merged in v2plus and master.

#4 Updated by Parag Mhashilkar almost 8 years ago

  • Target version set to v2_7_x

#5 Updated by Parag Mhashilkar almost 8 years ago

  • Target version changed from v2_7_x to 293

#6 Updated by Parag Mhashilkar almost 8 years ago

  • Status changed from Feedback to Resolved

#7 Updated by Parag Mhashilkar over 7 years ago

  • Target version changed from 293 to v2_7

#8 Updated by Parag Mhashilkar over 7 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF