Skip to content

Add IPs to the first ENI on startup#648

Merged
mogren merged 1 commit intoaws:masterfrom
mogren:ensure-eni0-has-ips
Oct 9, 2019
Merged

Add IPs to the first ENI on startup#648
mogren merged 1 commit intoaws:masterfrom
mogren:ensure-eni0-has-ips

Conversation

@mogren
Copy link
Copy Markdown
Contributor

@mogren mogren commented Oct 8, 2019

Description of changes:

  • Make sure we have IPs available before making the node ready
  • This change will add a 400 - 600ms delay before marking a node as Ready
  • For new nodes, this saves at least 2.5 seconds before IPs are available

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mogren mogren added this to the v1.6 milestone Oct 8, 2019
@mogren mogren requested review from nckturner and vsiddharth October 8, 2019 22:50
@mogren mogren force-pushed the ensure-eni0-has-ips branch 2 times, most recently from f87f732 to 778db90 Compare October 9, 2019 03:10
@jaypipes
Copy link
Copy Markdown
Contributor

jaypipes commented Oct 9, 2019

Is there an issue this is resolving/addresssing?

@mogren mogren force-pushed the ensure-eni0-has-ips branch from 778db90 to 98ae1e7 Compare October 9, 2019 20:48
@mogren mogren force-pushed the ensure-eni0-has-ips branch from 98ae1e7 to 2652c36 Compare October 9, 2019 22:04
Copy link
Copy Markdown
Contributor

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

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

👍

@mogren mogren merged commit 8f40e8e into aws:master Oct 9, 2019
@mogren mogren deleted the ensure-eni0-has-ips branch November 5, 2019 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants