-
Notifications
You must be signed in to change notification settings - Fork 8
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
[TASK] Support the PodNetworkType and NodeAffinity filed in the pod entity. #205
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #205 +/- ##
===========================================
+ Coverage 79.2% 81.66% +2.45%
===========================================
Files 49 49
Lines 2462 2825 +363
===========================================
+ Hits 1950 2307 +357
- Misses 388 390 +2
- Partials 124 128 +4
Continue to review full report at Codecov.
|
src/pod/pod.go
Outdated
} | ||
case entity.PodClusterNetwork: | ||
//For cluster network, we won't set the nodeAffinity and any netwokr options. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
network
22f901f
to
4dd40c9
Compare
src/pod/pod.go
Outdated
@@ -231,16 +231,26 @@ func CreatePod(sp *serviceprovider.Container, pod *entity.Pod) error { | |||
return err | |||
} | |||
|
|||
nodeNames := []string{} | |||
nodeNames := pod.NodeAffinity |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
要不要把nodeNames改成nodeAffinity?
In the pod entity, we use the PodNetworkType to choose what kind of the network the Pod use and use the
nodeAffinity
to indicate which nodes it wants to run.