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.

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

add a hostNetwork setting to the Driver and ControllerPlugin Specs #194

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

obnoxxx
Copy link
Collaborator

@obnoxxx obnoxxx commented Jan 13, 2025

This bool setting can be added to the controllerPlugin section of the Driver Spec.

It will be propagated to all controller plugin pods.

This implements the following design:

https://github.com/ceph/ceph-csi-operator/blob/main/docs/design/hostNetwork.md

see issue #157 for background and context.

Describe what this PR does

This work in progress is a draft PR for the implementation of host Networking for controller plugin pods as described in https://github.com/ceph/ceph-csi-operator/blob/main/docs/design/hostNetwork.md

Is there anything that requires special attention

This needs thorough testing.

Is the change backward compatible?

should be but this needs testing.

Are there concerns around backward compatibility?

none right now

For example:

Related issues

Fixes: #157

Future concerns

none.

Checklist:

  • Commit Message Formatting: Commit titles and messages follow
    guidelines in the developer
    guide
    .
  • Reviewed the developer guide on Submitting a Pull
    Request
  • Pending release
    notes

    updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

@obnoxxx
Copy link
Collaborator Author

obnoxxx commented Jan 13, 2025

@nb-ohad, @Madhu-1 , @rohan47, this is a replacement for the closed stale PR #176
this is a replacement

This bool setting can be added to the controllerPlugin section of the
Driver Spec.

It will be propagated to all controller plugin pods.

This implements the following design:

https://github.com/ceph/ceph-csi-operator/blob/main/docs/design/hostNetwork.md

Signed-off-by: Michael Adam <[email protected]>
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.

Enforce host networking for Ceph CSI controller plugin pods
1 participant