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.

Sign up for GitHub

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

Jump to bottom

hw10-observability submission #8

Merged

vladyslavpavlenko merged 22 commits into main from hw10-observability

Aug 3, 2024

Collaborator

vladyslavpavlenko commented Jul 19, 2024 •

edited

Loading

A detailed overview of this pull request, as well as the comments on alerts, can be found in hw10-observability.md.

vladyslavpavlenko added 6 commits

July 19, 2024 18:17


          build: add multi-stage-build, update docker-compose.yml & makefile

2df39ea


          chore: remove unnecessary golangci-config.yml from root

6ba792f


          feat: implement zap logger

9a53337


          fix, refactor: fix unique constraint issue, use zap logger

0e1ad72


          refactor: minor improvements to app & cmd packages, use zap logger

54ba902


          refactor: use zap logger

vladpistun reviewed

View reviewed changes

.dockerignore Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

cmd/api/main.go Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

cmd/api/main.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/app/run.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/email/consumer/kafka_consumer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/email/consumer/kafka_consumer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/email/consumer/kafka_consumer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/email/consumer/kafka_consumer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/outbox/producer/kafka_producer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/outbox/producer/kafka_producer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/outbox/producer/kafka_producer.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/rateapi/logger.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

internal/storage/gormstorage/driver.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

pkg/logger/logger.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

pkg/logger/logger.go Outdated Show resolved Hide resolved

vladpistun reviewed

View reviewed changes

pkg/logger/logger.go Outdated Show resolved Hide resolved


          fix: minor improvements to logging

b86bba3

vladyslavpavlenko self-assigned this

vladyslavpavlenko added the enhancement label

vladyslavpavlenko added 5 commits

July 21, 2024 00:45


          feat: add vm metrics to handlers

22b211b


          build: change postgres volume mirror

a095bdd


          refactor: remove global repo from handlers

f6478aa


          fix: fix golangci-lint trigger

b585d54


          feat: separate metrics server, add volume mirroring

vladyslavpavlenko added 6 commits

July 21, 2024 21:57


          fix: update scrape target

e818d36


          feat: add more neat logs to handlers (with request id)

0b3fe26


          feat: add metrics to mailer

39725ca


          feat: add metrics to rateapi

43e813f


          chore: improve metrics naming consistency

1de84e8


          docs: add hw10-observability review

dc49a00

vladyslavpavlenko changed the title ~~hw10-observability~~ hw10-observability submission

vladyslavpavlenko added the documentation label

vladyslavpavlenko requested a review from agritsyk

July 23, 2024 07:24

agritsyk reviewed

View reviewed changes

internal/storage/gormstorage/driver.go Show resolved Hide resolved

agritsyk approved these changes

View reviewed changes

vladyslavpavlenko added 4 commits

July 29, 2024 12:44


          refactor: restructure logger package

4bebf5c


          chore: add taskfile

b49d708


          chore: update lint github action

1efd742


          chore: fix lint github action

488679e

vladyslavpavlenko merged commit 75d592b into main

1 check passed

vladyslavpavlenko deleted the hw10-observability branch

August 3, 2024 08:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation enhancement