-
Notifications
You must be signed in to change notification settings - Fork 1.9k
docs(external docs): Revise Azure instance type recommendations #24147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Updated Azure recommendations for instance types.
| | --- | --- | | ||
| | AWS | c6i.2xlarge (recommended) or c6g.2xlarge | | ||
| | Azure | f8 | | ||
| | Azure | f8 or D8plds_v6 | |
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.
Need to please the spell checker somehow.
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.
@zapdos26 you can add D8plds_v6 to .github/actions/spelling/expect.txt
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.
Added :) Apologies, was out for the past couple of weeks
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.
Looks like that fails another section checking the spell check expect.txt
Added D8plds_v6 to spelling expect.txt
Summary
Updated Azure recommendations for instance types to specify Azure's ARM64 platform and update the recommendation to match.
Vector configuration
N/A
How did you test this PR?
N/A
Change Type
Is this a breaking change?
Does this PR include user facing changes?
no-changeloglabel to this PR.References
https://learn.microsoft.com/en-us/azure/virtual-machines/sizes/cobalt-overview
Notes
@vectordotdev/vectorto reach out to us regarding this PR.pre-pushhook, please see this template.make fmtmake check-clippy(if there are failures it's possible some of them can be fixed withmake clippy-fix)make testgit merge origin masterandgit push.Cargo.lock), pleaserun
make build-licensesto regenerate the license inventory and commit the changes (if any). More details here.