Skip to content
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

[FEATURE] Add support for more instance types #27

Closed
dbwiddis opened this issue May 4, 2023 · 2 comments
Closed

[FEATURE] Add support for more instance types #27

dbwiddis opened this issue May 4, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@dbwiddis
Copy link
Member

dbwiddis commented May 4, 2023

Is your feature request related to a problem?

Selection of instance types is limited by the cpuArch context to a specific size of Compute-Optimized instance.

const ec2InstanceType = (props.cpuType === AmazonLinuxCpuType.X86_64)
? InstanceType.of(InstanceClass.C5, InstanceSize.XLARGE) : InstanceType.of(InstanceClass.C6G, InstanceSize.XLARGE);

Some plugins or extensions require other instance types, such as memory optimized for Anomaly Detection HCAD (See opensearch-project/opensearch-sdk-java#652 and https://aws.amazon.com/blogs/big-data/detect-anomalies-on-one-million-unique-entities-with-amazon-opensearch-service/)

What solution would you like?

Allow a user to specify the instance type for each node type.

What alternatives have you considered?

Manually changing the instance types in the infra-stack.ts code.

Do you have any additional context?

Using T instance for development work costs less money while figuring things out!

See also #6

@dbwiddis dbwiddis added enhancement New feature or request untriaged labels May 4, 2023
@jordarlu jordarlu removed the untriaged label May 9, 2023
@jordarlu
Copy link

jordarlu commented May 9, 2023

Thank you, @dbwiddis ..
@rishabh6788 will comment , and in the mean time, it will be great if you can also contribue on the changes in PR ..

@rishabh6788
Copy link
Collaborator

#30 PR to add more choices for data and ml nodes. @dbwiddis

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants