Skip to content

Added support for using htcondor2 and classad2 libraries#664

Merged
shreyb merged 1 commit into
fermitools:masterfrom
shreyb:issue-663
Apr 27, 2026
Merged

Added support for using htcondor2 and classad2 libraries#664
shreyb merged 1 commit into
fermitools:masterfrom
shreyb:issue-663

Conversation

@shreyb
Copy link
Copy Markdown
Collaborator

@shreyb shreyb commented Apr 22, 2026

This PR adds conditional imports of the htcondor2 and classad2 Condor Python bindings. We want to still support both htcondor and classad, since some client machines are running Condor 23, which does not have the 2 versions of those libraries.

The plan is that in the upcoming release (1.13), we will support both htcondor/classad and the 2 versions of those, and then drop support for htcondor/classad in the 1.14 release.

This code has been exhaustively tested on the following condor combinations (each combination below is client --> schedd):

  1. Condor 23 --> Condor 25 (Current FNAL production)
  2. Condor 24 --> Condor 25
  3. Condor 25 --> Condor 25 (Near future FNAL production)
  4. Condor 25 --> Condor 24 (condor_vault_storer FAILS, but we never intend to use this combination, and it is probably independent of this change.)

Closes #663.

@shreyb shreyb added this to the 1.13 milestone Apr 22, 2026
@shreyb shreyb requested a review from vitodb April 22, 2026 15:29
Copy link
Copy Markdown
Contributor

@vitodb vitodb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!

@shreyb shreyb merged commit 41e79ab into fermitools:master Apr 27, 2026
3 of 4 checks passed
@shreyb shreyb mentioned this pull request Apr 27, 2026
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.

Add support for condor 25

2 participants