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

featureflagservice: expose grpc service on port FEATURE_FLAG_GRPC_SERVICE_PORT #250

Merged

Conversation

tsloughter
Copy link
Member

This makes it easier to use the featureflag service from outside of the docker-compose network with like grpcurl for testing.

@tsloughter tsloughter requested a review from a team August 2, 2022 22:35
@mic-max
Copy link
Contributor

mic-max commented Aug 2, 2022

@tsloughter If you plan on testing the featureflag service, could you add it to /test/test.js so we can make them once and run them forever :)
It's the service that I forgot to add to my e2e testing

@tsloughter
Copy link
Member Author

@mic-max I wasn't planning on adding tests, I just meant for manual testing purposes. But I can plan to add some in test.js eventually.

@cartersocha cartersocha merged commit 0ca979d into open-telemetry:main Aug 3, 2022
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
…VICE_PORT (open-telemetry#250)

Co-authored-by: Carter Socha <43380952+cartersocha@users.noreply.github.com>
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.

5 participants