Merged
Conversation
- Create ovn-kubernetes-components directory for component extraction - Add Dockerfile based on Dockerfile.ovn-kubernetes with OVN-K8s binaries removed - Currently builds only ovnkubernetesresourceinjector - ipallocator and dpucniprovisioner builds commented out for incremental extraction - Removed OVN packages, kubectl, and ovn-kubernetes binaries/scripts - Base image: Ubuntu 24.04 with minimal dependencies (openvswitch, netplan, systemd, dnsmasq)
- Copy cmd/ovnkubernetesresourceinjector/ with main.go - Copy internal/ovnkubernetesresourceinjector/webhooks/ package - Add go.mod and go.sum with minimal dependencies - Update import paths from doca-platform to ovn-kubernetes-components - Add .gitignore to exclude compiled binary - Dockerfile successfully builds 57MB binary - Tests included: webhook_test.go
- Add cmd/ipallocator/main.go with main entry point - Add internal/readyz/readyz.go utility for readiness probes - Update go.mod to include CNI and doca-platform dependencies - Uncomment ipallocator build in Dockerfile - Update .gitignore to ignore binaries but not source directories - Dockerfile successfully builds 4.1MB ipallocator binary - Uses github.com/nvidia/doca-platform/pkg/ipallocator as library
- Add cmd/dpucniprovisioner/main.go with Linux build constraint - Add internal/cniprovisioner/dpu/ with provisioner logic - Add internal/utils/ovsclient/ with OVS client wrapper - Add internal/constants/ for shared constants - Update import paths to ovn-kubernetes-components
- Add docker-build-dpf-utils target to build the dpf-utils container image - Add docker-push-dpf-utils target to push the dpf-utils image - Add lint target to run golangci-lint on dpf-utils code - Add test target to run tests with coverage for dpf-utils - Add golangci-lint to tool dependencies - Targets work with: ovnkubernetesresourceinjector, ipallocator, and dpucniprovisioner
killianmuldoon
force-pushed
the
pr-add-utils
branch
from
43622b6 to
ff45938
Compare
- Add build-dpf-utils job to build-images workflow for multi-arch container builds - Add test-dpf-utils workflow to run lint and test on dpf-utils code - Update Makefile with lint and test targets for dpf-utils - Update Makefile to build dpf-utils Docker image - Configure helm chart to use dpf-utils image for: - ovn-kubernetes-resource-injector webhook - dpuManifests ipallocator and cniprovisioner containers - Remove unused nodeWithoutDPUManifests.imagedpf from helm values
killianmuldoon
force-pushed
the
pr-add-utils
branch
from
ff45938 to
304685b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.