Skip to content

Conversation

@jiechen0826
Copy link
Contributor

@jiechen0826 jiechen0826 commented Jun 17, 2023

Issue #, if available:

Description of changes:

  • Do not resync pool for InsufficientCidrBlocks error as it results in unnecessary calls of DescribeInstances.
  • Change retry wait time forInsufficientCidrBlocks error to 2 min.
  • Add a new field in pool to indicate whether prefix is available in the subnet, so that it can inform pod controller of the error and its corresponding wait time.
  • Send pod event when InsufficientCidrBlocks happens and pod needs to retry after 2 min.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jiechen0826 jiechen0826 requested a review from a team as a code owner June 17, 2023 09:09
@jiechen0826 jiechen0826 changed the title do not resync pool for InsufficientCidrBlocks error do not resync or reconcile pool for InsufficientCidrBlocks error and add longer wait time for retry Jun 18, 2023
Copy link
Contributor

@haouc haouc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@haouc haouc merged commit 6589feb into aws:master Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants