chore: move updates from create to sdkUpdate #141
Conversation
Currently, we attempt to make certain update calls in sdkCreate, right after we populate the resource from createOutput. These calls usually fail as the Table may still be in `Creating` status. Another issue that kept happening was a throttling error during sdkFind for the DescribeTTL call. With these changes, we will be checking the Table status and reconciling if needed, and only retrieving TTL when the table has `available` status.
|
@michaelhtm: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: a-hilaly, michaelhtm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description of changes:
Currently, we attempt to make certain update calls in sdkCreate, right
after we populate the resource from createOutput. These calls usually
fail as the Table may still be in
Creatingstatus.Another issue that kept happening was a throttling error during sdkFind
for the DescribeTTL call. With these changes, we will be checking the
Table status and reconciling if needed, and only retrieving TTL when the
table has
availablestatus.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.