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

Schedd.xquery() has been deprecated in HTCondor 10.7.0 #12027

Closed
amaltaro opened this issue Jun 26, 2024 · 0 comments · Fixed by #12158
Closed

Schedd.xquery() has been deprecated in HTCondor 10.7.0 #12027

amaltaro opened this issue Jun 26, 2024 · 0 comments · Fixed by #12158
Assignees
Labels
Feature change Technical Debt Used to track issues that address technical needs internal to WM team

Comments

@amaltaro
Copy link
Contributor

amaltaro commented Jun 26, 2024

Impact of the new feature
WMAgent

Is your feature request related to a problem? Please describe.
Yes, this method has been deprecated in HTCondor 10.7.0:

/usr/lib64/python3.9/site-packages/htcondor/_deprecation.py:41: FutureWarning: Schedd.xquery() was deprecated in version 10.7.0 and will be removed in a future release.
  warnings.warn(message, FutureWarning)

so we better migrate (back?) to Schedd.query(), before we have to patch things out.

Describe the solution you'd like
There are a few places in WMAgent codebase that we rely on the Schedd.xquery() method. We need to adopt Schedd.query(), which is supposed to be exactly the same as the former, but instead of returning a generator, it will expand everything in memory.

Describe alternatives you've considered
We are already in a dangerous situation, given that we are now running HTCondor python bindings 10.9.0. So it has to be done.

Additional context
Documentation: https://htcondor.readthedocs.io/en/latest/apis/python-bindings/api/htcondor.html#interacting-with-schedulers
IF needed, please talk to the CRAB folks, has they have already performed this migration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature change Technical Debt Used to track issues that address technical needs internal to WM team
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants