Skip to content

Commit

Permalink
V1.0.0 (#350)
Browse files Browse the repository at this point in the history
* Switch to using OpenTelemetry for both tracing and metrics

This commit replaces the `metrics` and `trace` packages with a new `clue` package that leverages OpenTelemetry for instrumenting metrics and traces.

See the README for instructions on how to upgrade.

* Add tests

* Remove coupling on Goa middleware

* Use GRPC stat.handler for tracing in gRPC (#349)

This replaces the deprecated use of interceptors.
See open-telemetry/opentelemetry-go-contrib#3002.

Remove use of Goa request ID middleware. This is superseded by logging span IDs.

* Add example `tester` service to show API Integration Testing of a Goa System (#331)

* initial commit

* Empty-Commit

* add readme file (still need to fill out)

* tester readme update

* Trying out a workflow to run integration tests against local binaries

* Add proto install to /example/weather/scripts/setup

* fix integration.yaml

* disable mono on ubuntu

* move mono stop

* Try to find out what's running on 8084

* Trying to disable mono?

* trying to stop mono 2

* add a sleep after stop/disable?

* add some echo

* see if stop/disable mono fails

* add back in || true, change front ports to see if this even works at all

* oops forgot to change int test port

* try no send to dev null?

* sleep before testing

* take out netstat

* sleep longer? give time for docker to do work to get grafana?

* remove dev>null again... :\

* sleep 60 :(

* Implement review comment changes

* Fix runTests check for filtering if

* Empty-Commit

* try starting each service individually, check fo rmono

* just KILL mono?

* back to 8084, check all ports

* fix typo

* add flag to turn off grafana for services (for CI testing)

* clean up integration.yaml script

* review fixes

* Get Stack Trace to Err Log on Panic

* small fixes for style

* protoc -> 25.1, better wait in integration.yaml

* Add synchronous testing feature

* Make one of the synchronous as an example

* move func maps to service definition

* Remove use of legacy flag

---------

Co-authored-by: Jason Borneman <49321691+jasondborneman@users.noreply.github.com>
  • Loading branch information
raphael and jasondborneman authored Jan 8, 2024
1 parent 15fc752 commit fcfcc9c
Show file tree
Hide file tree
Showing 143 changed files with 1,916 additions and 9,547 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,10 @@ jobs:
echo "run setup script"
./scripts/setup
echo "run server"
./bin/forecaster -monitoring-enabled=false &
./bin/locator -monitoring-enabled=false &
./bin/tester -monitoring-enabled=false &
./bin/front -monitoring-enabled=false &
./bin/forecaster &
./bin/locator &
./bin/tester &
./bin/front &
check_service_health "http://localhost:8081/healthz" &
check_service_health "http://localhost:8083/healthz" &
check_service_health "http://localhost:8091/healthz" &
Expand Down
8 changes: 4 additions & 4 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
.vscode/
example/weather/services/forecaster/cmd/forecaster/forecaster
example/weather/services/locator/cmd/locator/locator
example/weather/services/front/cmd/front/front
example/weather/.overmind.sock
example/weather/bin/*
example/weather/services/forecaster/cmd/forecaster/forecaster
example/weather/services/front/cmd/front/front
example/weather/services/locator/cmd/locator/locator
example/weather/signoz
coverage.out
go.work
2 changes: 1 addition & 1 deletion LICENSE
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
The MIT License (MIT)

Copyright (c) 2022 Raphael Simon and Clue Contributors
Copyright (c) 2024 Raphael Simon and Clue Contributors

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
Expand Down
Loading

0 comments on commit fcfcc9c

Please sign in to comment.