r/service - add validations + import checks for all tests

[DrFaust92]

Description

Acceptance tests

• Have you added an acceptance test for the functionality being added?
• Have you run the acceptance tests on this branch?

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccKubernetesService_'
--- PASS: TestAccKubernetesService_basic (6.44s)
--- PASS: TestAccKubernetesService_nodePort (4.89s)
--- PASS: TestAccKubernetesService_externalName (2.83s)
--- PASS: TestAccKubernetesService_externalName_toClusterIp (6.58s)
--- PASS: TestAccKubernetesService_generatedName (2.84s)
--- PASS: TestAccKubernetesService_regression (201.65s)

Release Note

Release note for CHANGELOG:

resource_kubernetes_service - add plan time validations for `spec.cluster_ip`, `spec.external_ips`, `spec.load_balancer_ip`,`spec.load_balancer_source_ranges`, `spec.port.node_port`, `spec.port.port`, `spec.health_check_node_port`
...

References

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[jrhouston]

Thanks for tidying this up @DrFaust92 😄

[benoittoulme]

If cluster_ip can be set to None, then
ValidateFunc: validation.IsIPAddress
does not work I believe.
( This breaks my deployment ;) )

[arohter]

Error: expected spec.0.cluster_ip to contain a valid IP, got: None

... in resource "kubernetes_service" "svc":
        cluster_ip = "None"

[jrhouston]

@DrFaust92 just FYI: #1292

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.