Skip to content

fix: use ClusterFirstWithHostNet when hostNetwork is enabled#10

Merged
thxCode merged 1 commit intogpustack:mainfrom
aiwantaozi:fix/5133
Apr 28, 2026
Merged

fix: use ClusterFirstWithHostNet when hostNetwork is enabled#10
thxCode merged 1 commit intogpustack:mainfrom
aiwantaozi:fix/5133

Conversation

@aiwantaozi
Copy link
Copy Markdown
Contributor

Signed-off-by: michelia <michelia@seal.io>
Copilot AI review requested due to automatic review settings April 28, 2026 02:34
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Kubernetes pod specification to set the dns_policy to ClusterFirstWithHostNet when host_network is enabled. A review comment correctly identifies that the equal_pods comparison logic must also be updated to include the dns_policy field, otherwise existing workloads will not be updated to reflect this change.

Comment thread gpustack_runtime/deployer/kuberentes.py
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes Kubernetes pod DNS behavior when hostNetwork is enabled by ensuring the correct dnsPolicy is applied (per issue #5133).

Changes:

  • Set V1PodSpec.dns_policy to ClusterFirstWithHostNet when workload.host_network is enabled.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread gpustack_runtime/deployer/kuberentes.py
Copy link
Copy Markdown
Contributor

@thxCode thxCode left a comment

Choose a reason for hiding this comment

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

LGTM

@thxCode thxCode merged commit 514f12f into gpustack:main Apr 28, 2026
11 checks passed
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.

3 participants