Skip to content

Commit

Permalink
Merge pull request #581 from ColeBollig/CE_V6-HTCONDOR-1968-spam-job-…
Browse files Browse the repository at this point in the history
…router-syntax-deprecation-warning-branch

HTCONDOR-1968: Warn about old job router configuration syntax depreca…
  • Loading branch information
ColeBollig authored Sep 8, 2023
2 parents d2c7592 + c975ac7 commit 5f40db7
Show file tree
Hide file tree
Showing 16 changed files with 130 additions and 2 deletions.
8 changes: 8 additions & 0 deletions config/01-ce-router-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,14 @@ SCHEDD_ATTRS = $(SCHEDD_ATTRS) HTCondorCEVersion grid_resource
# JOB_ROUTER_DEFAULTS = $(JOB_ROUTER_DEFAULTS) [set_foo = 1;]
#
MERGE_JOB_ROUTER_DEFAULT_ADS=True
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_DEFAULTS = $(JOB_ROUTER_DEFAULTS_GENERATED)

# Use JOB_ROUTER_DEFAULTS + JOB_ROUTER_ENTRIES by default instead of
Expand Down
8 changes: 8 additions & 0 deletions config/01-ce-router.conf
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,14 @@ ROUTED_JOB_MAX_TIME = 4320
# JOB_ROUTER_DEFAULTS = $(JOB_ROUTER_DEFAULTS) [set_foo = 1;]
#
MERGE_JOB_ROUTER_DEFAULT_ADS=True
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_DEFAULTS = $(JOB_ROUTER_DEFAULTS_GENERATED)


Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-bosco-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,14 @@

# Basic route for submitting to BOSCO
# Use osg-configure to set BOSCO_RMS and BOSCO_ENDPOINT
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES = \
[ \
GridResource = "batch $(BOSCO_RMS) $(BOSCO_ENDPOINT)"; \
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-bosco.conf
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,14 @@

# Basic route for submitting to BOSCO
# Use osg-configure to set BOSCO_RMS and BOSCO_ENDPOINT
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES = \
[ \
GridResource = "batch $(BOSCO_RMS) $(BOSCO_ENDPOINT)"; \
Expand Down
9 changes: 8 additions & 1 deletion config/02-ce-condor-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,14 @@
###############################################################################

# Submit the job to the site Condor

#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
TargetUniverse = 5;
Expand Down
9 changes: 8 additions & 1 deletion config/02-ce-condor.conf
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,14 @@
###############################################################################

# Submit the job to the site Condor

#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
TargetUniverse = 5;
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-lsf-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,14 @@
###############################################################################

# Basic route for submitting to LSF
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch lsf";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-lsf.conf
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,14 @@
###############################################################################

# Basic route for submitting to LSF
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch lsf";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-pbs-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,14 @@
###############################################################################

# Basic route for submitting to PBS
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch pbs";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-pbs.conf
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,14 @@
###############################################################################

# Basic route for submitting to PBS
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch pbs";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-sge-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,14 @@
###############################################################################

# Basic route for submitting to SGE
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch sge";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-sge.conf
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,14 @@
###############################################################################

# Basic route for submitting to SGE
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch sge";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-slurm-defaults.conf
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,14 @@
###############################################################################

# Basic route for submitting to Slurm
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch slurm";
Expand Down
8 changes: 8 additions & 0 deletions config/02-ce-slurm.conf
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,14 @@
###############################################################################

# Basic route for submitting to Slurm
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES @=jre
[
GridResource = "batch slurm";
Expand Down
8 changes: 8 additions & 0 deletions contrib/puppet/osg_ce_condor/files/config.d/02-ce-pbs.conf
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,14 @@
###############################################################################

# Basic route for submitting to PBS
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
# Warning: JOB_ROUTER_DEFAULTS, JOB_ROUTER_ENTRIES, JOB_ROUTER_ENTRIES_CMD, and
# JOB_ROUTER_ENTRIES_FILE are deprecated and will be removed for V24 of
# the HTCondor Software Suite. New configuration syntax for the job router is defined using
# JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For new syntax example visit:
# https://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration
# Note: The removal will occur during the lifetime of the HTCondor V23 feature series.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
JOB_ROUTER_ENTRIES = \
[ \
GridResource = "batch pbs"; \
Expand Down
10 changes: 10 additions & 0 deletions src/verify_ce_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,16 @@ def main():

# If no JOB_ROUTER_ROUTE_<name> rules exist, verify JOB_ROUTER_DEFAULTS and JOB_ROUTER_ENTRIES
else:
used_deprecated_knobs = []
for knob in ['JOB_ROUTER_DEFAULTS', 'JOB_ROUTER_ENTRIES', 'JOB_ROUTER_ENTRIES_CMD','JOB_ROUTER_ENTRIES_FILE']:
if knob in htcondor.param:
used_deprecated_knobs.append(knob)
if len(used_deprecated_knobs) > 0:
warn(f"{', '.join(used_deprecated_knobs)} are deprecated and will be removed for V24 of the HTCondor Software Suite. New configuration"
+ " syntax for the job router is defined using JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>. For example"
+ " use of new syntax visit:\nhttps://htcondor.readthedocs.io/en/latest/grid-computing/job-router.html#an-example-configuration\n\n"
+ " Note: The removal will occur during the lifetime of the HTCondor V23 feature series.\n")

for attr in ['JOB_ROUTER_DEFAULTS', 'JOB_ROUTER_ENTRIES']:
try:
config_val = htcondor.param[attr]
Expand Down

0 comments on commit 5f40db7

Please sign in to comment.