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

[9.0] PilotAgentsDB: move from OwnerGroup to VO #7396

Merged
merged 3 commits into from
Jan 18, 2024

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Jan 16, 2024

  • Add changes in wiki, including populating VO field

closes #7286

BEGINRELEASENOTES

*WMS
CHANGE: PilotAgentsDB: move from OwnerGroup to VO

ENDRELEASENOTES

@fstagni fstagni force-pushed the 90_pilots_OwnerGroup2VO branch 2 times, most recently from 8956fa5 to 4220c5d Compare January 17, 2024 11:51
@fstagni fstagni marked this pull request as ready for review January 17, 2024 13:24
@fstagni fstagni requested a review from atsareg as a code owner January 17, 2024 13:24
@@ -77,9 +74,9 @@ def export_getCurrentPilotCounters(cls, attrDict={}):
types_addPilotReferences = [list, str]

@classmethod
def export_addPilotReferences(cls, pilotRef, ownerGroup, gridType="DIRAC", pilotStampDict={}):
def export_addPilotReferences(cls, pilotRef, VO, gridType="DIRAC", pilotStampDict={}):
Copy link
Member

Choose a reason for hiding this comment

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

Will this be a problem during the update for old pilots to connect to new servers?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At the moment this is only called server side (by SiteDirector). There has to be anyway a DB update (I mentioned in the PR description).

@chrisburr chrisburr merged commit 85ed4e5 into DIRACGrid:integration Jan 18, 2024
24 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep:ignore Prevent sweeping from being ran for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PilotAgentsDB: add VO?
3 participants