Skip to content

[int] Parameterise SQL in WMS PilotAgents#8617

Merged
fstagni merged 1 commit into
DIRACGrid:integrationfrom
sfayer:sql_wms_padb_param
Jul 8, 2026
Merged

[int] Parameterise SQL in WMS PilotAgents#8617
fstagni merged 1 commit into
DIRACGrid:integrationfrom
sfayer:sql_wms_padb_param

Conversation

@sfayer

@sfayer sfayer commented Jun 15, 2026

Copy link
Copy Markdown
Member

Part of #8547.
This one was already partially done for earlier patch, but this is the remainder.

BEGINRELEASENOTES
*WorkloadManagement
FIX: Parameterise SQL in WMS PilotAgents
ENDRELEASENOTES

@sfayer sfayer requested review from atsareg and fstagni as code owners June 15, 2026 15:40
@sfayer sfayer force-pushed the sql_wms_padb_param branch from a35f8df to 8f561e2 Compare June 16, 2026 08:49
@fstagni

fstagni commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This will be evaluated possibly after #8597 is (at least partially) resolved.

@sfayer sfayer force-pushed the sql_wms_padb_param branch from 8f561e2 to 1a9ea7d Compare June 17, 2026 21:15
@sfayer sfayer force-pushed the sql_wms_padb_param branch from 1a9ea7d to 9c00b3a Compare July 8, 2026 07:58
@sfayer

sfayer commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

Hi @fstagni,

This one has been on the queue for a few weeks now and it seems to me like #8957 is blocked on various other fairly big tasks... Any chance this one could go ahead of it?

Regards,
Simon

@fstagni

fstagni commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

You can rebase now that #8624 is merged

@fstagni

fstagni commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

You can rebase now that #8624 is merged

Actually, no need. Merging.

@fstagni fstagni merged commit 65facc4 into DIRACGrid:integration Jul 8, 2026
23 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Jul 8, 2026
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.

3 participants