Skip to content

Add ARCHITECTURE and DRIVER_VERSION to MV_WORKER_NODE_GPU_SUMMARY#132

Open
EdwardKaravakis wants to merge 1 commit into
mainfrom
feat/gpu-brokerage-mv-architecture-driver
Open

Add ARCHITECTURE and DRIVER_VERSION to MV_WORKER_NODE_GPU_SUMMARY#132
EdwardKaravakis wants to merge 1 commit into
mainfrom
feat/gpu-brokerage-mv-architecture-driver

Conversation

@EdwardKaravakis
Copy link
Copy Markdown
Member

Summary

  • Adds ARCHITECTURE and DRIVER_VERSION columns to MV_WORKER_NODE_GPU_SUMMARY (both Oracle and PostgreSQL) — these fields exist in the source worker_node_gpus table but were not projected into the MV
  • Adds the MV definition to pg_PANDA_SCHEDULER_JOBS.sql (base schema) so fresh PostgreSQL installations include it
  • New PostgreSQL patch 0.1.4.patch.sql to upgrade existing deployments

Motivation

Required by ATLASPANDA-1684 (GPU brokerage enhancements). The brokerage in panda-server will use these fields to filter queues by GPU microarchitecture generation (Volta, Ampere, Hopper...) and NVIDIA driver version / CUDA compatibility. The companion panda-server PR uses CRIC as a GPU capability gate and the WN GPU monitoring MV as the source for all attribute checks.

Oracle upgrade script

See the DB-upgrade-scripts wiki page (WiP entry for DB version 0.1.4).

Test plan

  • Apply 0.1.4.patch.sql to DOMA PostgreSQL and verify mv_worker_node_gpu_summary has the new columns populated
  • Verify Oracle MV can be dropped and recreated with the new definition
  • Confirm GRANT SELECT to ATLAS_PANDA_READER is in place

Add two columns missing from the GPU summary materialized view that are
available in the source worker_node_gpus table but were not projected:
- ARCHITECTURE (e.g. Volta, Ampere, Hopper, Ada Lovelace)
- DRIVER_VERSION (NVIDIA driver version string)

These are needed to support VRAM, GPU architecture, and CUDA version
filtering in the PanDA brokerage (ATLASPANDA-1684).

Also adds the MV definition to the base postgres schema file
(pg_PANDA_SCHEDULER_JOBS.sql) so fresh installations include it.
@EdwardKaravakis EdwardKaravakis marked this pull request as ready for review May 11, 2026 13:49
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.

1 participant