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

Add a basic logging to the service #31

Merged
merged 1 commit into from
Apr 21, 2023

Conversation

ochaplashkin
Copy link

@ochaplashkin ochaplashkin commented Mar 7, 2023

Description

We are adding logging to the service.
Now, the activity records will be linked via a cross-service unique ID that github sends: X-GitHub-Delivery.

Logs are written in the container's stdout as JSON and are available by the docker logs command.

Usage

  1. You tagged the bot and nothing happened;
  2. Go to the Settings > Webhooks > docbot > Recent Deliveries;
  3. Choose your webhook;
  4. Copy X-GitHub-Delivery;
  5. Get logs from a DocBot using any tool (docker logs, kibana, etc.);
  6. Do grep/search/etc by X-GitHub-Delivery.

E.g result:

[Mar 7, 2023 @16:06:31.893] <X-GitHub-Delivery> Comment has been sent successfully
[Mar 7, 2023 @16:06:30.640] <X-GitHub-Delivery> Request is processed ok
[Mar 7, 2023 @16:06:30.631] <X-GitHub-Delivery> Start <issue_comment> event processing
[Mar 7, 2023 @16:06:30.626] <X-GitHub-Delivery> Accept webhook from GitHub service

Resolves #28

@ochaplashkin ochaplashkin requested a review from ylobankov March 7, 2023 12:46
@ochaplashkin ochaplashkin force-pushed the add-basic-logs branch 2 times, most recently from 63593f8 to 57a5c11 Compare April 20, 2023 12:52
Added a basic logging and configuration of loggers.
Request processing is related to the cross-service `X-GitHub-Delivery`.

Resolves tarantool#28
@ylobankov ylobankov merged commit d1fac91 into tarantool:master Apr 21, 2023
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.

Configure logging and integrate it with Kibana
2 participants