line / armeria

Your go-to microservice framework for any situation, from the creator of Netty et al. You can build any type of microservice leveraging your favorite technologies, including gRPC, Thrift, Kotlin, Retrofit, Reactive Streams, Spring Boot and Dropwizard.
https://armeria.dev
Apache License 2.0
4.73k stars 899 forks source link

Add gRPC reverse proxy server example #5722

Closed eottabom closed 1 week ago

eottabom commented 1 month ago

Motivation:

Add gRPC reverse proxy server example

Modifications:

Add gRPC reverse proxy server example with testcontainers.

Result:

CLAassistant commented 1 month ago

CLA assistant check
All committers have signed the CLA.

github-actions[bot] commented 1 month ago

๐Ÿ” Build Scanยฎ (commit: 0c8ecb410a9b4548ec4924af83d7763ac7383d8c)

Job name Status Build Scanยฎ
build-windows-latest-jdk-21 โœ… https://ge.armeria.dev/s/cl6pxeh6pzi2a
build-self-hosted-unsafe-jdk-8 โœ… https://ge.armeria.dev/s/r2gsxbrrvhw2c
build-self-hosted-unsafe-jdk-21-snapshot-blockhound โœ… https://ge.armeria.dev/s/l3wmgpewtgzh2
build-self-hosted-unsafe-jdk-17-min-java-17-coverage โœ… https://ge.armeria.dev/s/sf4jpb56eq4ec
build-self-hosted-unsafe-jdk-17-min-java-11 โœ… https://ge.armeria.dev/s/senoeher2ztfm
build-self-hosted-unsafe-jdk-17-leak โœ… https://ge.armeria.dev/s/ovnvnpjdy3kx4
build-self-hosted-unsafe-jdk-11 โœ… https://ge.armeria.dev/s/s6keqs42qlb5e
build-macos-12-jdk-21 โŒ (failure) https://ge.armeria.dev/s/bvai7u6u62aqc
trustin commented 1 month ago

Could you check the build failures and make sure the tests are not flaky?

eottabom commented 3 weeks ago

It does not appear to be a problem caused by overlapping ports. Let me find the cause again.

eottabom commented 3 weeks ago

build-self-hosted-unsafe-jdk-21-snapshot-blockhound

I don't think there's a docker on the ci server

java.util.concurrent.CompletionException: java.lang.IllegalStateException: Could not find a valid Docker environment. Please see logs and check configuration

jrhee17 commented 3 weeks ago

Pushed some small commits ๐Ÿ™‡ Let me know if any changes don't make sense

eottabom commented 2 weeks ago

Looks great, thanks! Could you also run this script to see if everything works correctly? https://github.com/line/armeria-examples/blob/main/update-examples.sh You might run this command: (I supposed armeria and armeria-examples are in the same parent directory) ./update-examples.sh 1.29.0 ../armeria

This script is not working properly.

The following error occurs Is there a dependency I need to add? image

minwoox commented 2 weeks ago

This script is not working properly.

Let me see what happened. ๐Ÿ˜‰

minwoox commented 2 weeks ago

Let me see what happened. ๐Ÿ˜‰

It works correctly. The problem was just that we needed to convert the libs.testcontainers.junit.jupiter to the actual names in the script like: -pe "s/libs.testcontainers.junit.jupiter/'org.testcontainers:junit-jupiter:$TESTCONTAINER_VERSION'/g;" \

It will be handled when we release the next version so you don't have to worry about it. ๐Ÿ˜‰

eottabom commented 2 weeks ago

Rather, I think learned a lot. Thank you. ๐Ÿ˜„