Skip to content

feat: Job metadata is also registered as labels#62

Merged
TrevorBasinger merged 5 commits intomainfrom
tb/job-system-labels
Apr 17, 2026
Merged

feat: Job metadata is also registered as labels#62
TrevorBasinger merged 5 commits intomainfrom
tb/job-system-labels

Conversation

@TrevorBasinger
Copy link
Copy Markdown
Member

@TrevorBasinger TrevorBasinger commented Apr 17, 2026

Summary

This PR adds additive, reserved job system labels under roar.*, derived from existing local job metadata while keeping jobs.metadata intact
for downstream compatibility.

What changed

  • derive roar.* system labels for local jobs from existing metadata
  • refresh those labels on job creation/update paths, including:
    • local run/get/put jobs
    • OSMO host execution updates
    • Ray/OSMO fragment reconstitution
    • post-reconstitution composite materialization
  • reserve roar.* from manual label set/copy operations
  • hide roar.* from default show/DAG rendering to avoid noisy user-facing output
  • sync current job system labels to GLaaS, including publish-time put jobs
  • update unit/integration/live label expectations accordingly

Notes

  • existing metadata blobs are preserved unchanged
  • existing artifact system labels under dataset.* are unchanged
  • this is an additive migration step toward label-backed job metadata

Testing

  • ruff check .
  • mypy roar
  • pytest -m "not live_glaas and not ebpf" -q

@TrevorBasinger TrevorBasinger marked this pull request as ready for review April 17, 2026 17:09
Copy link
Copy Markdown
Member

@christophergeyer christophergeyer left a comment

Choose a reason for hiding this comment

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

🚢 it.

@TrevorBasinger TrevorBasinger merged commit 5e2b8f7 into main Apr 17, 2026
12 checks passed
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.

2 participants