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

contrib/aws: add coverage for Amazon Linux 2023 #10273

Closed
wants to merge 1 commit into from

Conversation

wenduwan
Copy link
Contributor

@wenduwan wenduwan commented Aug 3, 2024

Amazon Linux 2023 has 6.x kernel which is not covered in CI.

As a result we will consume more hpc6a capacity. To avoid capacity issues, switch tcp/socket tests to a more abundant instance type, e.g. c6gn.16xlarge.

@wenduwan wenduwan requested a review from a team August 3, 2024 00:47
@shijin-aws
Copy link
Contributor

@wenduwan can we move tcp/sockets to non-efa instances? they don't need it, right?

@wenduwan
Copy link
Contributor Author

wenduwan commented Aug 5, 2024

@shijin-aws I forgot but did we have a special reason to use hpc6a before? I don't think we need EFA either.

@shijin-aws
Copy link
Contributor

shijin-aws commented Aug 5, 2024

I don't think there is a special reason, probably because the yaml config enables EFA by default and if you use an non-efa instance you need to pass additional cmd line args

Amazon Linux 2023 has 6.x kernel which is not covered in CI.

Also test tcp/socket providers on non-EFA platforms to conserve capacity.

Signed-off-by: Wenduo Wang <[email protected]>
@wenduwan
Copy link
Contributor Author

wenduwan commented Aug 5, 2024

@shijin-aws Cool. Switched to c6g instead. Could you kick off CI? I don't have permission to run this change.

@shijin-aws
Copy link
Contributor

I don't have permission to run this change.

I don't understand, shouldn't your PR trigger the new test as is?

@wenduwan
Copy link
Contributor Author

wenduwan commented Aug 5, 2024

It does not.

@shijin-aws
Copy link
Contributor

Then how your earlier push go through AWS CI?

@shijin-aws
Copy link
Contributor

bot:aws:retest

2 similar comments
@wenduwan
Copy link
Contributor Author

wenduwan commented Aug 5, 2024

bot:aws:retest

@wenduwan
Copy link
Contributor Author

wenduwan commented Aug 5, 2024

bot:aws:retest

@wenduwan wenduwan closed this Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants