Skip to content
Snippets Groups Projects

Add new llm logger for debugging AI features

Merged Nicolas Dular requested to merge nd/add-ai-logging into master
All threads resolved!

What does this MR do and why?

Adds a new logger and debugging statements for AI features.

Issue: https://gitlab.com/gitlab-org/gitlab/-/issues/409977

Screenshots or screen recordings

logging

How to set up and validate locally

  1. tail -f log/llm.log
  2. Set up your local instance for AI features
  3. Test an AI feature like summarize comments
    1. Active it by enabling the feature flag: Feature.enable(:summarize_comments)
    2. Go to an issue with comments and press the "Summarize Comments" button (see video)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Nicolas Dular

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Nicolas Dular requested review from @mksionek

    requested review from @mksionek

  • Gosia Ksionek
  • Gosia Ksionek approved this merge request

    approved this merge request

  • LGTM @nicolasdular, just one question!

  • Gosia Ksionek requested review from @jprovaznik and removed review request for @mksionek

    requested review from @jprovaznik and removed review request for @mksionek

  • :wave: @mksionek, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • A deleted user added documentation label

    added documentation label

  • Nicolas Dular changed the description

    changed the description

  • Nicolas Dular added 1 commit

    added 1 commit

    Compare with previous version

  • Nicolas Dular added 255 commits

    added 255 commits

    Compare with previous version

  • Jan Provaznik approved this merge request

    approved this merge request

  • Jan Provaznik unapproved this merge request

    unapproved this merge request

  • Jan Provaznik removed review request for @jprovaznik

    removed review request for @jprovaznik

  • Nicolas Dular added 51 commits

    added 51 commits

    Compare with previous version

  • Nicolas Dular requested review from @jprovaznik and @sselhorn

    requested review from @jprovaznik and @sselhorn

  • Jan Provaznik approved this merge request

    approved this merge request

  • Suzanne Selhorn approved this merge request

    approved this merge request

  • Nicolas Dular added 1 commit

    added 1 commit

    Compare with previous version

  • Suzanne Selhorn resolved all threads

    resolved all threads

  • Suzanne Selhorn approved this merge request

    approved this merge request

  • Suzanne Selhorn removed review request for @sselhorn

    removed review request for @sselhorn

  • Author Maintainer

    As we have all approvals, @jprovaznik would you please go ahead and merge the MR? Thanks!

  • Jan Provaznik approved this merge request

    approved this merge request

  • Jan Provaznik enabled an automatic merge when the pipeline for cb84e753 succeeds

    enabled an automatic merge when the pipeline for cb84e753 succeeds

  • Thanks @nicolasdular, MWPS set :rocket:

  • merged

  • Jan Provaznik mentioned in commit d743e701

    mentioned in commit d743e701

  • added workflowstaging label and removed workflowcanary label

  • mentioned in merge request !161814 (merged)

  • Please register or sign in to reply
    Loading