atlassian
Adding GostatsD Lambda Extension
Starting this off as a draft since it is mostly complete.
There is a few things I want to double check such as:
- [x] Forwarder will flush on context.Done()
- [x] release assets to github
- [x] The use if a virtual clock
- [ ] updated docs on how to use the extension
- [ ] a local integration test to validate work
However, since the logic is done. There is nothing stopping the current work from being reviewed.
Forwarder to flush all metrics within a time interval is done here https://github.com/atlassian/gostatsd/pull/422/files
👋 , I'm gonna be taking over this branch and pushing this through. I'll start by resolving the comments then move onto resolving merge conflicts.
Gonna mark as ready for review for now, got a few increment PRs incoming for:
- Subscribing to the telemetry API
- Supporting manual flushes in HTTP forwarder
Ideally we can address any major issues with this PR in the incoming ones. For now the lambda extension is in a state where it works when the lambda is continuously invoked. If this is not occuring then the metrics from an invocation may not flush until the next invocation of the lambda.
The PR increments mentioned above will resolve this issue and add support for flushing per invocation.