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

[v8.0] fix: ComputingElement.initializeParameters() concatenate tags instead of overriding them #7087

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Jul 3, 2023

Aims at fixing #7086

BEGINRELEASENOTES
*Resources
FIX: concatenate tags instead of overriding them in ComputingElement.initializeParameters()
ENDRELEASENOTES

@aldbr aldbr requested review from fstagni and atsareg as code owners July 3, 2023 13:39
@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Jul 3, 2023
@aldbr aldbr force-pushed the rel-v8r0_FIX_ComputingElementinitalizeParameters branch from d21330f to b605a2d Compare July 3, 2023 14:16
@fstagni
Copy link
Contributor

fstagni commented Jul 3, 2023

I think this anyway would deserve to be tested at least in https://jenkins-dirac.web.cern.ch/view/DIRAC/job/Pilot3_CVM4_pipeline/

@fstagni fstagni closed this Jul 5, 2023
@fstagni fstagni reopened this Jul 5, 2023
@chrisburr chrisburr merged commit 5e5b1af into DIRACGrid:rel-v8r0 Jul 5, 2023
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Jul 5, 2023
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Jul 5, 2023
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/5468485715

Successful:

  • integration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pilot tags are ignored if /Resources/Sites/{grid_type}/{site}/CEs/{ce_name}/Queues/Tag is set
4 participants