Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Host Network Attachment Definition #192

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add Host Network Attachment Definition #192

wants to merge 4 commits into from

Commits on Oct 21, 2024

  1. networkfn-nad-host/00.networkfun-nad.yaml: Add Host network attachmen… 

    …t definition

By including the Host nad yaml file we are able to centralize the NAD
setup wihtin the operator itself

Currently the Namespace follows the CDA implementation where it uses the
Default Namespace
    root committed Oct 21, 2024
    Configuration menu
    Copy the full SHA
    0cb3708 View commit details
    Browse the repository at this point in the history

  2. dpuoperatorconfig_controller.go: Add check within ensureNetworkFuncti… 

    …oNAD to include host setup as well

- The idea is to move the check from outside the nad and always
  reconcile the NAD and apply the appropriate manifest but currently the
auto mode can't be used to apply the Host NAD since it requires the
daemon so we keep a check to skip it.
- This implementation has problems with auto since this mode relies on
  the daemon, which opens a bigger discussion on whether managing the
NAD is the job of the daemon or the controller.
    root committed Oct 21, 2024
    Configuration menu
    Copy the full SHA
    c124cc0 View commit details
    Browse the repository at this point in the history

  3. dpuoperatorconfig_controller.go: Check if it's not auto then ensure NAD 

    We should skip deploying the NAD ourselves if the configuration spec
mode is `auto`, since this implementation can't auto detect the hardware in the
controller.
    root committed Oct 21, 2024
    Configuration menu
    Copy the full SHA
    c25e43a View commit details
    Browse the repository at this point in the history

  4. cda: Bump CDA with the removal of the Host DPU NAD creation 

    As part of bringing Host DPU NAD creation into the DPU-Operator,we need
to also remove the steps where the cda creates the host dpu nad.
    root committed Oct 21, 2024
    Configuration menu
    Copy the full SHA
    52b4f8e View commit details
    Browse the repository at this point in the history