Merge pull request #23 from andyxning/remove_magic_string

fanminshi · web-flow · commit aed6c3b4ef7a · 2018-05-09T10:10:46.000-07:00
remove magic string about protocol
diff --git a/vault-operator/pkg/vault/deploy_vault.go b/vault-operator/pkg/vault/deploy_vault.go
@@ -270,11 +270,11 @@ func statsdExporterContainer() v1.Container {
 		Ports: []v1.ContainerPort{{
 			Name:          "statsd",
 			ContainerPort: exporterStatsdPort,
-			Protocol:      "UDP",
+			Protocol:      v1.ProtocolUDP,
 		}, {
 			Name:          "prometheus",
 			ContainerPort: exporterPromPort,
-			Protocol:      "TCP",
+			Protocol:      v1.ProtocolTCP,
 		}},
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -270,11 +270,11 @@ func statsdExporterContainer() v1.Container {`
`270`	`270`	`Ports: []v1.ContainerPort{{`
`271`	`271`	`Name: "statsd",`
`272`	`272`	`ContainerPort: exporterStatsdPort,`
`273`		`- Protocol: "UDP",`
	`273`	`+ Protocol: v1.ProtocolUDP,`
`274`	`274`	`}, {`
`275`	`275`	`Name: "prometheus",`
`276`	`276`	`ContainerPort: exporterPromPort,`
`277`		`- Protocol: "TCP",`
	`277`	`+ Protocol: v1.ProtocolTCP,`
`278`	`278`	`}},`
`279`	`279`	`}`
`280`	`280`	`}`