[TK-1957] Update container.resources schema for Pod, DaemonSet and Deployment

[arybolovlev]

Description

This PR does the following changes:

• 🐛 fix an issue when empty values of spec.container.resources.limits or spec.container.resources.requests produce continuous diff output during plan although no real changes were made. Affected resources: kubernetes_pod, kubernetes_pod_v1, kubernetes_daemonset, kubernetes_daemon_set_v1, kubernetes_deployment, kubernetes_deployment_v1.
• 🐛 fix an issue when changing values of spec.container.resources.limits or spec.container.resources.requests does not update appropriate Kubernetes resources. Affected resources: kubernetes_pod, kubernetes_pod_v1.
• ✨ changing values of spec.container.resources.limits or spec.container.resources.requests will force resource recreation. Affected resources: kubernetes_pod, kubernetes_pod_v1.

Related documentation changes are in the different PR: #1882

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="-count 1 -run ^TestAccKubernetesDaemonSet_with_resource_requirements"
...
=== RUN   TestAccKubernetesDaemonSet_with_resource_requirements
--- PASS: TestAccKubernetesDaemonSet_with_resource_requirements (6.57s)

$ make testacc TESTARGS="-count 1 -run ^TestAccKubernetesDeployment_with_resource_requirements"
...
=== RUN   TestAccKubernetesDeployment_with_resource_requirements
--- PASS: TestAccKubernetesDeployment_with_resource_requirements (6.15s)

$ make testacc TESTARGS="-count 1 -run ^TestAccKubernetesPod_with_resource_requirements"
...
=== RUN   TestAccKubernetesPod_with_resource_requirements
--- PASS: TestAccKubernetesPod_with_resource_requirements (19.99s)

Release Note

Release note for CHANGELOG:

ENHANCEMENT:

* `r/kubernetes_pod_v1`: changing values of `spec.container.resources.limits` or `spec.container.resources.requests` will force resource recreation.
* `r/kubernetes_pod`: changing values of `spec.container.resources.limits` or `spec.container.resources.requests` will force resource recreation.

BUG FIXES:

* Fix an issue when changing values of `spec.container.resources.limits` or `spec.container.resources.requests` does not update appropriate Kubernetes resources. Affected resources: `kubernetes_pod`, `kubernetes_pod_v1`.
* Fix an issue when empty values of `spec.container.resources.limits` or `spec.container.resources.requests` produce continuous diff output during `plan` although no real changes were made. Affected resources: `kubernetes_pod`, `kubernetes_pod_v1`, `kubernetes_daemonset`, `kubernetes_daemon_set_v1`, `kubernetes_deployment`, `kubernetes_deployment_v1`.

References

Fix: #1880

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

[arybolovlev]

Remove the error return from this function since we never return any errors, just nil.

[arybolovlev]

Remove length validation here and accept an empty map from flattenResourceList to store empty values instead of null in the Terraform state file.

[arybolovlev]

Not sure what exactly has changed here, this is the result of 'make website-lint-fix'.

[jrhouston]

It's just trailing whitespace being remove

[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.

[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.