-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add random errors in AdService #694
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What would be the reason for creating this error in the normal flow? Should this logic be behind a feature flag (disabled by default)? I’d expect the demo app to run without errors by default unless specifically breaking it.
I agree with @mviitane, if we want to add errors to Don't get me wrong @jack-berg I think the PR is valid and it adds a scenario to the demo. You can either add this to the current PR, or if you prefer, we can close this one and you can send a new PR with the implementation. Here is an example of how that is handled in the |
Adding this as a feature flag sounds reasonable to me. I tried to make progress on this today but was having some trouble running the featureflagservice locally, so I'm blocked for the moment. |
…y-demo into adservice-errors
@jack-berg the featureflagservice was fixed yesterday 🥳 |
src/featureflagservice/priv/repo/migrations/20220524172636_create_featureflags.exs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Running fine on my docker environment. Added few minor comments.
src/featureflagservice/priv/repo/migrations/20220524172636_create_featureflags.exs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changelog conflict needs to be resolved. Otherwise, LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Fixes open-telemetry#696, Downgrades OTP (open-telemetry#698) * Use different base images for frauddetectionservice (open-telemetry#691) * Use different docker base images for frauddetection service Signed-off-by: svrnm <[email protected]> * Add entry in CHANGELOG Signed-off-by: svrnm <[email protected]> * update changelog Signed-off-by: svrnm <[email protected]> * Remove extra blank line Signed-off-by: svrnm <[email protected]> Co-authored-by: Juliano Costa <[email protected]> Co-authored-by: Carter Socha <[email protected]> * fix(paymentservice): respect temporality env var (open-telemetry#693) * fix(paymentservice): update dependencies so that temporality env var is respected * fix(changelog): add changelog entry * Add healthcheck to featureflagservice (open-telemetry#661) * Add healthcheck to featureflagservice In order to assess the health of this central service, compose should perform a check on the service. * Add changelog entry Co-authored-by: Juliano Costa <[email protected]> Co-authored-by: Carter Socha <[email protected]> * bot is broken (open-telemetry#699) Co-authored-by: Juliano Costa <[email protected]> * Fix link to the docs (open-telemetry#701) * reduce quoteservice spans (open-telemetry#702) * fixing container name nit - quote service (open-telemetry#706) * [recommendationservice] - base image, licensing and diagram update (open-telemetry#700) * update service metric requirement table (open-telemetry#707) * update service metric requirement table * add feature table * Update emailservice Dockerfile to use alpine and multistage build (open-telemetry#703) * Update emailservice Dockerfile Signed-off-by: svrnm <[email protected]> * Use alpine Signed-off-by: svrnm <[email protected]> * Add changelog Signed-off-by: svrnm <[email protected]> * newline Signed-off-by: svrnm <[email protected]> * Update CHANGELOG.md * Update .dockerignore Signed-off-by: svrnm <[email protected]> Co-authored-by: Juliano Costa <[email protected]> * Update dockerfile for adservice to use different base images (open-telemetry#705) * Enable exemplar export/display support (open-telemetry#704) * Enable exemplar export/display support Ensure the collector's prometheus endpoint produces exemplars Ensure that prometheus collects and stores exemplar data Ensure that grafana will display links to exemplars from dashboards/the explore tab * Enable exemplars for latency histogram in demo dashboard * update contributing guide (open-telemetry#711) * update contributing guide * fix lint Co-authored-by: Carter Socha <[email protected]> * Add random errors in AdService (open-telemetry#694) * feature(frontend): adding instrumentation metrics (open-telemetry#632) * feature(frontend): adding instrumentation metrics * feature(frontend): adding instrumentation metrics * updating package-lock.json * Update docker-compose.yml Co-authored-by: Marc Pichler <[email protected]> --------- Co-authored-by: Marc Pichler <[email protected]> * Update CONTRIBUTING.md (open-telemetry#720) * fix metrics import (open-telemetry#721) Signed-off-by: Pierre Tessier <[email protected]> * Update .env (open-telemetry#722) * Updated demo link for Sumo Logic (open-telemetry#718) * Add build for arm64 arch (open-telemetry#657) * Add build for arm64 arch * Add change to changelog * enable build on PR * remove build on pr --------- Co-authored-by: Juliano Costa <[email protected]> Co-authored-by: Carter Socha <[email protected]> Co-authored-by: Austin Parker <[email protected]> * [chore] - fix image version (open-telemetry#726) Remove the `v` prefix from the image version. We now publish images without this prefix. * Adds caveats to current exemplar support in the demo architecture (open-telemetry#725) Also, updates collector version: exemplars for the latency histogram produced by the span metrics processor are now getting exported for all services and endpoints, not just the first one in the metric family. Co-authored-by: Pierre Tessier <[email protected]> * Add Sentry (sentry.io) demo to README (open-telemetry#724) * Add Sentry (sentry.io) demo to README * use alpha order --------- Co-authored-by: Pierre Tessier <[email protected]> * Drop docs folder and adjust links (open-telemetry#730) * Drop docs folder and adjust links * Update PULL_REQUEST_TEMPLATE.md * Add entry to CHANGELOG.md * README: drop unused logo reference * add reflection for grpcurl (open-telemetry#735) * Add cross-compilation for shipping service (open-telemetry#715) * Add cross-compilation for shipping service Signed-off-by: svrnm <[email protected]> * Update CHANGELOG.md * Update Dockerfile --------- Signed-off-by: svrnm <[email protected]> * Ensure all http.server.duration metrics have the same description (open-telemetry#738) * Add OTel logo to README (open-telemetry#744) * rename proto package from hipstershop to oteldemo (open-telemetry#740) * prometheus: honor_labels to have the right job and instance (open-telemetry#743) Co-authored-by: Carter Socha <[email protected]> Co-authored-by: Juliano Costa <[email protected]> * Nit fix contributing guide (open-telemetry#747) --------- Signed-off-by: svrnm <[email protected]> Signed-off-by: Pierre Tessier <[email protected]> Co-authored-by: Josh Lee <[email protected]> Co-authored-by: Severin Neumann <[email protected]> Co-authored-by: Juliano Costa <[email protected]> Co-authored-by: Carter Socha <[email protected]> Co-authored-by: Marc Pichler <[email protected]> Co-authored-by: Cedric Ziel <[email protected]> Co-authored-by: Brett McBride <[email protected]> Co-authored-by: James Lawrie <[email protected]> Co-authored-by: Austin Parker <[email protected]> Co-authored-by: jack-berg <[email protected]> Co-authored-by: Oscar Reyes <[email protected]> Co-authored-by: Marc Pichler <[email protected]> Co-authored-by: Pierre Tessier <[email protected]> Co-authored-by: Pawel Brzoska <[email protected]> Co-authored-by: Abhijeet Prasad <[email protected]> Co-authored-by: Patrice Chalin <[email protected]> Co-authored-by: Jared Tan <[email protected]> Co-authored-by: Alex Boten <[email protected]> Co-authored-by: Goutham Veeramachaneni <[email protected]> Co-authored-by: Cijo Thomas <[email protected]>
Generate random errors that exercise error handling login in the AdService. Currently, the error handling logic which adds an event and sets the span status isn't reachable, as far as I can tell.