Skip to content
This repository has been archived by the owner on Mar 30, 2023. It is now read-only.

Enable accelerated networking #230

Merged
merged 6 commits into from
Oct 4, 2018
Merged

Conversation

russcam
Copy link
Contributor

@russcam russcam commented Oct 3, 2018

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

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
@russcam
Copy link
Contributor Author

russcam commented Oct 3, 2018

Need to run

npm run links -- --branch master

before merging

Copy link
Member

@Mpdreamz Mpdreamz left a 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": [
Copy link
Member

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.

Copy link
Contributor Author

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

@russcam russcam merged commit a3962ac into master Oct 4, 2018
@russcam russcam deleted the feature/accelerated-networking branch October 16, 2018 04:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants