-
Notifications
You must be signed in to change notification settings - Fork 162
Conversation
This commit enables accelerated networking by default for VM SKUs that support it, for both Elasticsearch nodes and the Kibana VM. Exposes the option to override accelerated networking if necessary. This is exposed in the case that new VMs are introduced that support accelerated networking, or existing VMs are updated to enable accelerated networking, giving users the ability to control accelerated networking behaviour. Closes #224
Ubuntu version is the same for both 5.x and 6.x
Need to run
before merging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change itself LGTM 👍 It'd be interesting to put a number to the actual performance increase.
Now we're running under the calculated risk accelerated
is in fact accelerated. It's a safe assumption but still 😸
@@ -12,6 +12,87 @@ | |||
"numberOfDataNodes" : 50, | |||
"numberOfClientNodes" : 20, | |||
"diskSizes" : ["Small", "Medium", "Large", "XLarge", "XXLarge"], | |||
"vmAcceleratedNetworking": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's sad we need to maintain these compat lists rather than ARM
exposing this in a apply or noop fashion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed 😢 ours can't be only the template that does this
This PR enables accelerated networking by default for VM SKUs that support it,
for both Elasticsearch nodes and the Kibana VM.
Exposes the option to override accelerated networking if necessary. This is exposed in
the case that new VMs are introduced that support accelerated networking,
or existing VMs are updated to enable accelerated networking, giving users the ability to
control accelerated networking behaviour.
Closes #224