Skip to content

driver: set correct DNSPolicy based on HostNetwork for ctrl plugins#445

Open
rohan47 wants to merge 1 commit intoceph:mainfrom
rohan47:dns_policy_ctrlplugin
Open

driver: set correct DNSPolicy based on HostNetwork for ctrl plugins#445
rohan47 wants to merge 1 commit intoceph:mainfrom
rohan47:dns_policy_ctrlplugin

Conversation

@rohan47
Copy link

@rohan47 rohan47 commented Mar 26, 2026

Describe what this PR does

In case of data replication separation, Controller Plugins needs to run on hostNetwork and the DNS policy should be set to ClusterFirstWithHostNetwork

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.

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.

@rohan47 LGTM, please update the PR description and commit message to be more descriptive when we are setting what

In case of data replication separation, Controller Plugins needs to run
on hostNetwork and the DNS policy should be set to
ClusterFirstWithHostNetwork

Signed-off-by: Rohan Gupta <rohgupta@redhat.com>
@rohan47 rohan47 force-pushed the dns_policy_ctrlplugin branch from ab1b1e0 to 26a2600 Compare March 26, 2026 08:38
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.

2 participants