terraform-provider-kubernetes
terraform-provider-kubernetes copied to clipboard
chore: Default Timeout for KubernetesServiceAccount has been extended to 1 minute.
Description
All resources except KubernetesServiceAccount are more than 1 minute. Only KubernetesServiceAccount is treated unequally, with a timeout of 30 seconds. If the network is slowing down and you are applying to an environment that is physically far away, 30 seconds may cause a timeout.
Translated with www.DeepL.com/Translator (free version)

Q: But why don't I just extend the timeout by myself? A: Yes, but I don't understand why only KubernetesServiceAccount is given 30 seconds while all other resources except KubernetesServiceAccount are given more than 1 minute. KubernetesServiceAccount should be treated equally.
Acceptance tests
- [x] Have you added an acceptance test for the functionality being added?
- [x] Have you run the acceptance tests on this branch?
Output from acceptance testing:
$ make testacc TESTARGS='-run=TestAccXXX'
...
Release Note
Release note for CHANGELOG:
Default Timeout for KubernetesServiceAccount has been extended to 1 minute.
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 Could you please review it? 👀
Hi @kahirokunn,
Thank you for opening this PR. However, I will close it without merging it into our main branch.
The creation of a service account is a relatively fast operation and the timeout that we have in place was added to cover a case when we are waiting for a service account token to be created for this new service account. Kubernetes does not create service account tokens for new service accounts starting from the v1.24. For short time, the provider didn't handle this case and the creation of a new SA ended up with a timeout. This was fixed in v2.13.0.
30 seconds is enough for such a small task as this one and we will keep it as it is. If in your case it is not enough, please increase it per resource as described here.
Thank you!
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.