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

create grpc smoke tests #73

Closed
Tracked by #3
emilyashley opened this issue Feb 1, 2023 · 1 comment · Fixed by #102
Closed
Tracked by #3

create grpc smoke tests #73

emilyashley opened this issue Feb 1, 2023 · 1 comment · Fixed by #102
Assignees
Milestone

Comments

@emilyashley
Copy link
Contributor

No description provided.

@vreynolds vreynolds added this to the Beta milestone Feb 1, 2023
@pkanal
Copy link
Contributor

pkanal commented Feb 13, 2023

waiting for #72

@emilyashley emilyashley self-assigned this Feb 27, 2023
JamieDanielson added a commit that referenced this issue Feb 28, 2023
## Which problem is this PR solving?

- updates #73 
- updates #74 

## Short description of the changes

- expands on the regular python example app 
- adds http and grpc smoke tests for regular python example app
- adds notes around local testing and running the apps in docker
emilyashley added a commit that referenced this issue Mar 2, 2023
<!--
Thank you for contributing to the project! 💜
Please see our [OSS process
document](https://github.com/honeycombio/home/blob/main/honeycomb-oss-lifecycle-and-practices.md#)
to get an idea of how we operate.
-->

## Which problem is this PR solving?
- Closes #73 
- Closes #75 

## Short description of the changes
Adds bats smoke tests for the Flask App using grpc to send traces and
metrics to the OpenTelemetry Collector.

## How to verify that this has the expected result
see happy smoke tests in CI!

---------

Co-authored-by: JamieDanielson <jamieedanielson@gmail.com>
Co-authored-by: Robb Kidd <robbkidd@honeycomb.io>
Co-authored-by: Mike Goldsmith <goldsmith.mike@gmail.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 a pull request may close this issue.

3 participants