Skip to content

[draft] Pilot manager legacy adaptor - #8696

Closed
fstagni wants to merge 2 commits into
DIRACGrid:integrationfrom
fstagni:pilotManager_legacyAdaptor
Closed

fstagni wants to merge 2 commits into
DIRACGrid:integrationfrom
fstagni:pilotManager_legacyAdaptor

Conversation

@fstagni

@fstagni fstagni commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

BEGINRELEASENOTES
Thank you for writing the text to appear in the release notes. It will show up
exactly as it appears between the two bold lines

Please follow the template:
*Subsystem
NEW/CHANGE/FIX: explanation

For examples look into release.notes

ENDRELEASENOTES

@fstagni
fstagni force-pushed the pilotManager_legacyAdaptor branch from 806dd7b to d947f2e Compare July 10, 2026 16:02
@fstagni
fstagni force-pushed the pilotManager_legacyAdaptor branch from d947f2e to 1f9f94c Compare July 14, 2026 13:21
@aldbr aldbr linked an issue Sep 3, 2026 that may be closed by this pull request
@aldbr aldbr closed this Sep 3, 2026
@fstagni fstagni reopened this Sep 3, 2026
@aldbr

aldbr commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

@AcquaDiGiorgio you can have a look at this one too

@aldbr aldbr closed this Sep 24, 2026
@AcquaDiGiorgio

Copy link
Copy Markdown

Okay so I based my PR on the changes made by Robin at #8233, which seem to be based on this one, so in turn I've already used these changes. In general, my PR is quite similar to this one, with getPilots implemented and a couple of other minor changes.

In addition, this PR seems to have the same problem as mine about obtaining the jobs of a pilot. selectPilots calls self.getJobsForPilotByStamp(pilotStamp) but this function is not implemented. I guess this was the skeleton for a later implementation that never came. Maybe for the same reason?

Just one question, is adding the pilotStamp to the DB, Agent and Service required for the simplification part of the migration? Shouldn't we maintain the current functionality and try to adpat to DiracX?

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.

Create a legacy adaptor for PilotManager

3 participants