Skip to content

fix: split NMX-C partition IDs from NMX-M IDs#1950

Open
tmcroberts97 wants to merge 1 commit into
NVIDIA:mainfrom
tmcroberts97:fix/nmx-c-db
Open

fix: split NMX-C partition IDs from NMX-M IDs#1950
tmcroberts97 wants to merge 1 commit into
NVIDIA:mainfrom
tmcroberts97:fix/nmx-c-db

Conversation

@tmcroberts97
Copy link
Copy Markdown
Contributor

Add a dedicated nmx_c_partition_id column for physical NVLink partitions, and remove the NOT NULL constraint on the nmx_m_id column. NMX-C-created partitions will no longer make use of the legacy column.

Also, update the monitor to build the DB partition map based on the new NMX-C ID column instead of parsing nmx_m_id. When a partition is successfully created or updated, delete any legacy partition rows in the DB from the same logical partition and nvlink domain.

Description

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Related Issues (Optional)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

Add a dedicated nmx_c_partition_id column for physical NVLink
partitions, and remove the NOT NULL constraint on the nmx_m_id column.
NMX-C-created partitions will no longer make use of the legacy column.

Also, update the monitor to build the DB partition map based on the new
NMX-C ID column instead of parsing nmx_m_id. When a partition is
successfully created or updated, delete any legacy partition rows in the
DB from the same logical partition and nvlink domain.

Signed-off-by: Thomas McRoberts <tmcroberts@nvidia.com>
@tmcroberts97 tmcroberts97 requested a review from a team as a code owner May 27, 2026 14:59
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