Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTCONDOR-1968: Warn about old job router configuration syntax depreca… #581

Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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
# HTCondor. 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.
#!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
ColeBollig marked this conversation as resolved.
Show resolved Hide resolved
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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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
# HTCondor. 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("%s are deprecated and will be removed for V24 of HTCondor. New configuration"
% ", ".join(used_deprecated_knobs)
ColeBollig marked this conversation as resolved.
Show resolved Hide resolved
+ " syntax for the job router is defined using JOB_ROUTER_ROUTE_NAMES and JOB_ROUTER_ROUTE_<name>."
+ " Note: The removal will occur during the lifetime of the HTCondor V23 feature series.\n")
ColeBollig marked this conversation as resolved.
Show resolved Hide resolved

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