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.

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

Init logger #14

Closed
roma-glushko opened this issue Dec 9, 2023 · 2 comments · Fixed by #30
Closed

Init logger #14

roma-glushko opened this issue Dec 9, 2023 · 2 comments · Fixed by #30
Assignees
@roma-glushko
Labels
area:telemetry Logging, metrics, traces
Milestone
Glide: Private Pr...

Comments

@roma-glushko
Copy link
Member

roma-glushko commented Dec 9, 2023 •
edited
Loading

Investigate if the default slogs module added in go1.21 is a good option to rely for logging functionality-wise.

If not, lets configure zap.

Requirements:

  • it's possible to set log level/severity via configs(account for that since configs are still in TODO)
  • it should be possible to log records as colored plain text and as JSON
  • it's possible to log a message and additional fields
  • it's fine to just write logs to stdout
@roma-glushko roma-glushko self-assigned this Dec 9, 2023
@roma-glushko roma-glushko changed the title init logger Init logger Dec 9, 2023
@roma-glushko roma-glushko added the area:telemetry Logging, metrics, traces label Dec 10, 2023
@mkrueger12
Copy link
Contributor

You can decide on the direction we choose here.

@roma-glushko
Copy link
Member Author

@mkrueger12 do you want to take this one? I just need a bit of time to see slogs is a viable way to go. As of now, the description is the current plan around logging functionality.

@roma-glushko roma-glushko added this to the Sprint 1 milestone Dec 17, 2023
roma-glushko added a commit that referenced this issue Dec 17, 2023
@roma-glushko
#14 Inited Glide logging based on Zap
1c16e4b
@roma-glushko roma-glushko linked a pull request Dec 17, 2023 that will close this issue
#14 🛠️ Inited Glide Logging #30
Merged
roma-glushko added a commit that referenced this issue Dec 19, 2023
@roma-glushko
#14 introducing a new telemetry struct
f808c7c
roma-glushko added a commit that referenced this issue Dec 21, 2023
@roma-glushko
#14: Introduced a new telemetry struct to hold all observability comp…
d17a295 
…onents
roma-glushko added a commit that referenced this issue Dec 21, 2023
@roma-glushko
#14: Added the router struct
baabc11
roma-glushko added a commit that referenced this issue Dec 23, 2023
@roma-glushko
#14: Configured hertz logger to obey the general logging config
c524b4e
roma-glushko added a commit that referenced this issue Dec 23, 2023
@roma-glushko
#14 Added more static checks
8a1bd91
roma-glushko added a commit that referenced this issue Dec 23, 2023
@roma-glushko
#14 🛠️ Inited Glide Logging (#30)
23693c7 
* #14 Inited Glide logging based on Zap

* #14 introducing a new telemetry struct

* #14: Introduced a new telemetry struct to hold all observability components

* #14: Added the router struct

* #14: Configured hertz logger to obey the general logging config

* #14 Added more static checks
@roma-glushko roma-glushko mentioned this issue Jan 21, 2024
Merged
roma-glushko added a commit that referenced this issue Jan 21, 2024
@roma-glushko
📦 Release: v0.0.1-rc.1 (#93)
1c3e68a 
- ✨ [providers] Support for OpenAI Chat API #3 (@mkrueger12 )
- ✨ [API] #54 Unified Chat API  (@mkrueger12 )
- ✨ [providers] Support for Cohere Chat API #5 (@mkrueger12 )
- ✨ [providers] Support for Azure OpenAI Chat API #4  (@mkrueger12 )
- ✨ [providers] Support for OctoML Chat API #58 (@mkrueger12 )
- ✨ [routing] The Routing Mechanism, Adaptive Health Tracking, and Fallbacks #42 #43 #51 (@roma-glushko)
- ✨ [routing] Support for round robin routing strategy #44 (@roma-glushko)
- ✨ [routing] Support for the least latency routing strategy #46 (@roma-glushko)
- ✨ [routing] Support for weighted round robin routing strategy #45 (@roma-glushko)
- ✨ [providers] Support for Anthropic Chat API #60 (@mkrueger12 )
- ✨ [docs] OpenAPI specifications #22 (@roma-glushko )
- 🔧 [chores] Inited the project #6 (@roma-glushko)
- 🔊 [telemetry] Inited logging  #14 (@roma-glushko)
- 🔧 [chores] Inited Glide's CLI #12 (@roma-glushko)
- 👷 [chores] Setup CI workflows #8 (@roma-glushko)
- ⚙️ [config] Inited configs #11 (@roma-glushko)
-  🔧 [chores] Automatic coverage reports #39 (@roma-glushko)
- 👷 [build] Setup release workflows #9 (@roma-glushko)
roma-glushko added a commit that referenced this issue Feb 18, 2024
@roma-glushko
#14: Used JSON-based messages
23d8062
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roma-glushko roma-glushko

Labels
area:telemetry Logging, metrics, traces
Projects
None yet
Milestone
Glide: Private Preview (MVP)
Development

Successfully merging a pull request may close this issue.

#14 🛠️ Inited Glide Logging EinStack/glide
2 participants
@roma-glushko @mkrueger12