Skip to content

feat(dpf operator): dpu service charts#414

Open
FrankSpitulski wants to merge 2 commits intoNVIDIA:mainfrom
FrankSpitulski:feat/dpf-operator/dpu-service-charts
Open

feat(dpf operator): dpu service charts#414
FrankSpitulski wants to merge 2 commits intoNVIDIA:mainfrom
FrankSpitulski:feat/dpf-operator/dpu-service-charts

Conversation

@FrankSpitulski
Copy link
Contributor

@FrankSpitulski FrankSpitulski commented Feb 28, 2026

Description

part 3, adding in dpu service charts

I kept these separate from the carbide charts since these are not deployed as part of carbide, rather they are deployed in the dpu cluster.

these charts follow the dpu service reqs to respond to dpf operator triggered helm values.

this is a lift from the binaries directly into a "default" container+chart to give us a place to start. I expect these will need to be wired together in future commits.

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.)

ran the make targets to build the charts and containers

Additional Notes

@copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 28, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@FrankSpitulski FrankSpitulski force-pushed the feat/dpf-operator/dpu-service-charts branch from 65759db to df829e9 Compare February 28, 2026 00:33
@FrankSpitulski FrankSpitulski marked this pull request as ready for review February 28, 2026 00:33
@FrankSpitulski FrankSpitulski requested a review from a team as a code owner February 28, 2026 00:33
@ajf
Copy link
Collaborator

ajf commented Mar 2, 2026

It'd be better if we could use a distroess container; but probably wouldn't block on this. cc @ianderson-nvidia who did that work for the control plane stuff.

@FrankSpitulski
Copy link
Contributor Author

It'd be better if we could use a distroess container; but probably wouldn't block on this. cc @ianderson-nvidia who did that work for the control plane stuff.

made the regular packages distroless. otel uses shell scripts so I left that one. if satisfied please merge, as I lack permissions to merge even when the PR is approved.

@FrankSpitulski
Copy link
Contributor Author

the CI appears to be failing on a flaky test. "monitor_ib_status_and_fix_incorrect_pkey_associations"

Signed-off-by: fspitulski <fspitulski@nvidia.com>
Signed-off-by: fspitulski <fspitulski@nvidia.com>
@FrankSpitulski FrankSpitulski force-pushed the feat/dpf-operator/dpu-service-charts branch from 740e99f to b2fdd7c Compare March 3, 2026 23:46
@FrankSpitulski FrankSpitulski requested a review from chet March 3, 2026 23:46
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.

4 participants