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

WIP: Boilerplatey stuff #2

Merged
merged 22 commits into from Jan 11, 2023
Merged

WIP: Boilerplatey stuff #2

merged 22 commits into from Jan 11, 2023

Conversation

tcmal
Copy link
Contributor

@tcmal tcmal commented Jan 9, 2023

  • Add tracing and some other related crates for logging and profiling.
  • Add config for loading config from environment and file (mostly convenience)
  • Add docker-compose.dev.yml file and nix configuration, incl building docker image using nix
  • Switch to async redis/deadpool_redis and sqlx instead of sync
  • Add opentelemetry exporter so trace information can be viewed in grafana
  • Add a RequestError type that can be used to represent internal/external errors instead of panicking.
  • Add instructions on how to set up proxy for development
  • Fix some clippy lints

The resulting telemetry (in debug mode) looks like this in grafana:

image
image

@tcmal tcmal self-assigned this Jan 9, 2023
@tcmal tcmal marked this pull request as draft January 9, 2023 21:49
@mayacopeland
Copy link
Contributor

Everything looks fine, there'll be some changes I'll need to make afterwards with redis anyway, atleast for player buffers in order to not lose packets because someone wrote something to the buffer as we're flushing it.

@mayacopeland mayacopeland marked this pull request as ready for review January 10, 2023 15:00
@mayacopeland mayacopeland marked this pull request as draft January 10, 2023 15:02
@mayacopeland mayacopeland marked this pull request as ready for review January 11, 2023 16:38
@mayacopeland mayacopeland merged commit ef084bc into trunk Jan 11, 2023
@mayacopeland mayacopeland deleted the boilerplate branch January 11, 2023 21:31
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.

None yet

2 participants