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

Enhancement: Complete porting of remaining probes to SQLAlchemy syntax #812

Closed
3 tasks
dynamic-entropy opened this issue Jun 19, 2024 · 1 comment · Fixed by #816 or dmwm/rucio-flux#307
Closed
3 tasks
Assignees
Milestone

Comments

@dynamic-entropy
Copy link
Contributor

Enhancement Description

Three of the probes in our dockerfile need to be modernised:
PRs for two already seem to exist, but might need a review before merging and deploying.

Use Case

Fix them all!!!


I take this statement back (might have mixed up with some other tests):

# TODO: Most of these don't work either, each with a different error - different reasons than above

It's just one, reviewed the PR to suggest the necessary fix.

Possible Solution

No response

Related Issues

rucio/probes#89

@ericvaandering
Copy link
Member

I've linked the various PRs to this issue. Aside from the one in probes, they are done (and the one in probes isn't needed, we pull from the issue).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment