r/kubernetes_service: Switch targetPort to string

[appilon]

Fix #134
switch targetPort to string in schema
new acceptance test
update docs to reflect targetPort is optional

TF_ACC=1 go test ./kubernetes -v -run=TestAccKubernetesService -timeout 120m
=== RUN   TestAccKubernetesServiceAccount_basic
--- PASS: TestAccKubernetesServiceAccount_basic (1.98s)
=== RUN   TestAccKubernetesServiceAccount_update
--- PASS: TestAccKubernetesServiceAccount_update (3.04s)
=== RUN   TestAccKubernetesServiceAccount_generatedName
--- PASS: TestAccKubernetesServiceAccount_generatedName (0.96s)
=== RUN   TestAccKubernetesService_basic
--- PASS: TestAccKubernetesService_basic (1.60s)
=== RUN   TestAccKubernetesService_loadBalancer
--- PASS: TestAccKubernetesService_loadBalancer (58.68s)
=== RUN   TestAccKubernetesService_nodePort
--- PASS: TestAccKubernetesService_nodePort (0.89s)
=== RUN   TestAccKubernetesService_noTargetPort
--- PASS: TestAccKubernetesService_noTargetPort (97.85s)
=== RUN   TestAccKubernetesService_stringTargetPort
--- PASS: TestAccKubernetesService_stringTargetPort (118.08s)
=== RUN   TestAccKubernetesService_externalName
--- PASS: TestAccKubernetesService_externalName (0.85s)
=== RUN   TestAccKubernetesService_importBasic
--- PASS: TestAccKubernetesService_importBasic (0.96s)
=== RUN   TestAccKubernetesService_generatedName
--- PASS: TestAccKubernetesService_generatedName (0.85s)
=== RUN   TestAccKubernetesService_importGeneratedName
--- PASS: TestAccKubernetesService_importGeneratedName (0.94s)
PASS
ok  	github.com/terraform-providers/terraform-provider-kubernetes/kubernetes	286.771s

Fortunately HCL appears to be flexible enough that it accepts both target_port: 80 or target_port: "80". Kubernetes has its own IntOrString type that will parse the stringified integer to an int. Long story short there should be no breaking changes in anyones configs

[catsby]

👍

[appilon]

I broke the data source woops, that will teach me for not running the full suite of tests:

=== RUN   TestAccKubernetesDataSourceService_basic
--- FAIL: TestAccKubernetesDataSourceService_basic (0.89s)
	testing.go:518: Step 0 error: Error applying: 1 error(s) occurred:

		* data.kubernetes_service.test: data.kubernetes_service.test: spec.0.port.0.target_port: '' expected type 'int', got unconvertible type 'string'
	testing.go:579: Error destroying resource! WARNING: Dangling resources
		may exist. The full state and error is shown below.

		Error: Error refreshing: 1 error(s) occurred:

		* data.kubernetes_service.test: 1 error(s) occurred:

		* data.kubernetes_service.test: data.kubernetes_service.test: spec.0.port.0.target_port: '' expected type 'int', got unconvertible type 'string'

		State: <nil>