Skip to content

gRPC AOT scenarios #1761

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

Closed
wants to merge 1 commit into from
Closed

gRPC AOT scenarios #1761

wants to merge 1 commit into from

Conversation

JamesNK
Copy link
Member

@JamesNK JamesNK commented Oct 22, 2022

πŸ”₯ πŸš€ πŸ’£ 🌢️

Depends on grpc/grpc-dotnet#1924

@sebastienros
Copy link
Member

Need to check it will work fist since we reached the limit of the yaml size on Friday and had to remove some benchmark (memory consumption when loading the whole processed file is limited to 10MB)
Will probably have to split the pipeline in two first before accepting this one. Another option would be to move some predefined strings in other crank config files so we don't have to include their content in the azdo yaml files.

@sebastienros
Copy link
Member

Since then the CI has been fixed. But now we have them in the NativeAOT page, can we close this PR?

@JamesNK
Copy link
Member Author

JamesNK commented Feb 9, 2023

Might do this in the future, but the current AOT test is fine for now.

@JamesNK JamesNK closed this Feb 9, 2023
@sebastienros sebastienros deleted the JamesNK-patch-1 branch August 15, 2023 17:32
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