Skip to content

feat: add workflow-uuid label to batch, dask, and session pods#668

Open
iamakamen wants to merge 2 commits into
reanahub:masterfrom
iamakamen:fix-667-universal-labels
Open

feat: add workflow-uuid label to batch, dask, and session pods#668
iamakamen wants to merge 2 commits into
reanahub:masterfrom
iamakamen:fix-667-universal-labels

Conversation

@iamakamen
Copy link
Copy Markdown

@iamakamen iamakamen commented Dec 17, 2025

This PR harmonises the metadata labeling by adding the workflow-uuid label to all runtime workloads produced by the Workflow Controller.

Closes #667

This change is part of a cross-repository update. Please merge in conjunction with:

Copilot AI review requested due to automatic review settings December 17, 2025 23:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a standardized workflow-uuid label to all REANA runtime workloads (Batch jobs, Dask clusters, and Interactive Sessions) to enable consistent identification and querying of resources across the platform.

  • Adds workflow-uuid label to batch workflow Job pods
  • Adds workflow-uuid label to interactive session Deployment pods
  • Adds workflow-uuid label to Dask cluster, autoscaler, and dashboard ingress resources

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
reana_workflow_controller/workflow_run_manager.py Adds workflow-uuid label to batch Job metadata, paralleling the existing reana-run-batch-workflow-uuid label
reana_workflow_controller/k8s.py Adds workflow-uuid label to interactive session Deployment pod template, alongside existing reana-run-session-workflow-uuid label
reana_workflow_controller/dask.py Adds workflow-uuid label to Dask cluster metadata, autoscaler metadata, dashboard ingress middleware, and dashboard ingress resources, complementing existing reana-run-dask-workflow-uuid labels

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@iamakamen
Copy link
Copy Markdown
Author

iamakamen commented Apr 30, 2026

Hi @tiborsimko @jirikuncar @mr-c @dinosk, I wanted to follow up on this PR. Happy to refine or adjust based on feedback.

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.

add workflow-uuid labels for user runtime pods

3 participants