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

docs: add design for enforcing host network. #170

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

obnoxxx
Copy link
Collaborator

@obnoxxx obnoxxx commented Nov 10, 2024

Describe what this PR does

This PR adds a design document for a way to enable the use of host networking on Ceph-CSI controller plugin pods.

Is there anything that requires special attention

Is the change backward compatible?

yes as it is only adding a document.

Are there concerns around backward compatibility?

none

Related issues

Mention any github issues relevant to this PR. Adding below line
will help to auto close the issue once the PR is merged.

It is a first step toward a solution for issue #157

but not an implementation and hence not a complete solution yet.

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 obnoxxx marked this pull request as draft November 10, 2024 18:14
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
@obnoxxx obnoxxx changed the title docs: add desigfn for enforcing host network. docs: add design for enforcing host network. Nov 11, 2024
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from 89df060 to 2e71b04 Compare November 11, 2024 10:07
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch 2 times, most recently from 324a6aa to 7e68bc4 Compare November 11, 2024 12:02
@obnoxxx obnoxxx requested review from rohan47 and Madhu-1 November 11, 2024 12:03
@obnoxxx obnoxxx marked this pull request as ready for review November 11, 2024 12:03
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch 4 times, most recently from 7fa11c7 to e141e75 Compare November 11, 2024 12:21
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from e141e75 to 690a2a7 Compare November 11, 2024 18:38
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@Madhu-1 Madhu-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the file name to hostNetwork.md?

docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
docs/design/enforceHostNetwork.md Outdated Show resolved Hide resolved
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from 477f5d5 to 440d18f Compare November 12, 2024 16:05
@obnoxxx
Copy link
Collaborator Author

obnoxxx commented Nov 12, 2024

@Madhu-1 wrote:

Please update the file name to hostNetwork.md?

done.

@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from 440d18f to 14d64b3 Compare November 12, 2024 16:19
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from 5662954 to 9bfab5c Compare November 13, 2024 09:09
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch 2 times, most recently from f5b7a9a to 16b476e Compare November 13, 2024 09:37
@obnoxxx obnoxxx requested a review from Madhu-1 November 13, 2024 09:37
docs/design/hostNetwork.md Show resolved Hide resolved
docs/design/hostNetwork.md Show resolved Hide resolved
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
docs/design/hostNetwork.md Show resolved Hide resolved
docs/design/hostNetwork.md Show resolved Hide resolved
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Nov 13, 2024

@nb-ohad PTAL as you have requested changes

@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch 5 times, most recently from c82931a to 2da9b5f Compare November 13, 2024 10:33
@obnoxxx obnoxxx requested a review from Madhu-1 November 13, 2024 10:34
Copy link
Collaborator

@Madhu-1 Madhu-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nits.

docs/design/hostNetwork.md Outdated Show resolved Hide resolved
docs/design/hostNetwork.md Outdated Show resolved Hide resolved
docs/design/hostNetwork.md Show resolved Hide resolved
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from fe838b2 to c111ce0 Compare November 13, 2024 11:05
@obnoxxx obnoxxx requested a review from Madhu-1 November 13, 2024 11:05
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from c111ce0 to b3ceca6 Compare November 13, 2024 11:21
Signed-off-by: Michael Adam <obnox@samba.org>
@obnoxxx obnoxxx force-pushed the design-enforce_host_net branch from 1bf399d to fa2df3e Compare November 13, 2024 11:27
@nb-ohad nb-ohad merged commit f325f74 into ceph:main Nov 14, 2024
12 checks passed
@obnoxxx obnoxxx deleted the design-enforce_host_net branch November 14, 2024 18:12
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.

6 participants