mirror of https://github.com/k3s-io/k3s
fix reuse nodePort issue
parent
265c7df39e
commit
16960d3ad2
|
@ -1784,7 +1784,7 @@ type LoadBalancerIngress struct {
|
|||
type ServiceSpec struct {
|
||||
// The list of ports that are exposed by this service.
|
||||
// More info: http://releases.k8s.io/HEAD/docs/user-guide/services.md#virtual-ips-and-service-proxies
|
||||
Ports []ServicePort `json:"ports"`
|
||||
Ports []ServicePort `json:"ports" patchStrategy:"merge" patchMergeKey:"port"`
|
||||
|
||||
// This service will route traffic to pods having labels matching this selector.
|
||||
// Label keys and values that must match in order to receive traffic for this service.
|
||||
|
|
|
@ -555,6 +555,25 @@ var _ = framework.KubeDescribe("Kubectl client", func() {
|
|||
By("checking the result")
|
||||
forEachReplicationController(c, ns, "app", "redis", validateReplicationControllerConfiguration)
|
||||
})
|
||||
It("should reuse nodePort when apply to an existing SVC", func() {
|
||||
mkpath := func(file string) string {
|
||||
return filepath.Join(framework.TestContext.RepoRoot, "examples/guestbook-go", file)
|
||||
}
|
||||
serviceJson := mkpath("redis-master-service.json")
|
||||
nsFlag := fmt.Sprintf("--namespace=%v", ns)
|
||||
By("creating Redis SVC")
|
||||
framework.RunKubectlOrDie("create", "-f", serviceJson, nsFlag)
|
||||
By("getting the original nodePort")
|
||||
originalNodePort := framework.RunKubectlOrDie("get", "service", "redis-master", nsFlag, "-o", "jsonpath={.spec.ports[0].nodePort}")
|
||||
By("applying the same configuration")
|
||||
framework.RunKubectlOrDie("apply", "-f", serviceJson, nsFlag)
|
||||
By("getting the nodePort after applying configuration")
|
||||
currentNodePort := framework.RunKubectlOrDie("get", "service", "redis-master", nsFlag, "-o", "jsonpath={.spec.ports[0].nodePort}")
|
||||
By("checking the result")
|
||||
if originalNodePort != currentNodePort {
|
||||
framework.Failf("nodePort should keep the same")
|
||||
}
|
||||
})
|
||||
})
|
||||
|
||||
framework.KubeDescribe("Kubectl cluster-info", func() {
|
||||
|
|
Loading…
Reference in New Issue