Skip to content
Snippets Groups Projects

Support pipeline job context in Duo Chat

Merged Eva Kadlecová requested to merge 468461-support-pipeline-context-in-duo-chat into master
All threads resolved!

What does this MR do and why?

Allow Duo Chat users to ask questions about pipeline jobs.

AI Gateway MR: gitlab-org/modelops/applied-ml/code-suggestions/ai-assist!1379 (merged)

Evaluation results

Results for the master branch: master evaluation.

Results for this branch: MR evaluation.

Metric Master branch This branch
Correctness 3.64 3.62
Readability 3.66 3.67
Comprehensiveness 3.58 3.57

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

How to set up and validate locally

  1. From Rails console enable feature flags:
Feature.enable(:ai_build_reader_for_chat)
Feature.enable(:v2_chat_agent_integration)
  1. Run GDK.
  2. In AI gateway repo checkout the 468461-support-pipeline-context-in-duo-chat branch and run AI gateway.
  3. Visit a pipeline job and ask a question to Duo Chat about the pipeline (What is this pipeline about?)
  4. You should receive a details about the build.

Related to #468461 (closed)

Edited by Eva Kadlecová

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
  • 4 Warnings
    :warning: This merge request is quite big (627 lines changed), please consider splitting it into multiple merge requests.
    :warning: bb00b437: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
    :warning: c4c009d9: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
    :warning: c4c009d9: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
    1 Message
    :book: CHANGELOG missing:

    If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog.

    If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

    Reviewer roulette

    Category Reviewer Maintainer
    backend @ahegyi profile link current availability (UTC+2, same timezone as author) @harsimarsandhu profile link current availability (UTC+5.5, 3.5 hours ahead of author)
    ~"Verify" Reviewer review is optional for ~"Verify" @stanhu profile link current availability (UTC-4, 6 hours behind author)

    Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

  • added 1 commit

    • cd4dc195 - Support pipeline job context in Duo Chat

    Compare with previous version

  • added 1 commit

    • c8cf88f5 - Support pipeline job context in Duo Chat

    Compare with previous version

  • Eva Kadlecová resolved all threads

    resolved all threads

  • Eva Kadlecová added 507 commits

    added 507 commits

    Compare with previous version

  • Eva Kadlecová added 9 commits

    added 9 commits

    Compare with previous version

  • Eva Kadlecová changed the description

    changed the description

  • added 1 commit

    • a9b7284f - Support pipeline job context in Duo Chat

    Compare with previous version

  • added 1 commit

    • ee27a407 - Support pipeline job context in Duo Chat

    Compare with previous version

  • changed milestone to %17.5

  • Eva Kadlecová added 2732 commits

    added 2732 commits

    Compare with previous version

  • added 1 commit

    • 8ff796cb - Support pipeline job context in Duo Chat

    Compare with previous version

  • Eva Kadlecová added 80 commits

    added 80 commits

    Compare with previous version

  • added 1 commit

    • 60aaa5d5 - Support pipeline job context in Duo Chat

    Compare with previous version

  • added 1 commit

    • 8265dea3 - Support pipeline job context in Duo Chat

    Compare with previous version

  • Eva Kadlecová added 163 commits

    added 163 commits

    Compare with previous version

  • Eva Kadlecová added 435 commits

    added 435 commits

    Compare with previous version

  • Eva Kadlecová added 61 commits

    added 61 commits

    Compare with previous version

  • Eva Kadlecová added 35 commits

    added 35 commits

    Compare with previous version

  • Eva Kadlecová changed the description

    changed the description

  • Eva Kadlecová added 386 commits

    added 386 commits

    Compare with previous version

  • Eva Kadlecová changed the description

    changed the description

  • Eva Kadlecová added 33 commits

    added 33 commits

    Compare with previous version

  • Eva Kadlecová requested review from @brytannia

    requested review from @brytannia

  • Tetiana Chupryna requested review from @lesley-r and removed review request for @brytannia

    requested review from @lesley-r and removed review request for @brytannia

  • Lesley Razzaghian
  • This is really cool!! :fire: :rocket: :fire:

    I couldn't find anything to suggest, it looks great to me. :)

    @brytannia Would you like to do the maintainer review, or should we also have another person review it? Thank you!

  • Lesley Razzaghian approved this merge request

    approved this merge request

  • added pipelinetier-2 label and removed pipelinetier-1 label

  • Before you set this MR to auto-merge

    This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

    Before you set this MR to auto-merge, please check the following:

    • You are the last maintainer of this merge request
    • The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
    • This pipeline is recent enough (created in the last 8 hours)

    If all the criteria above apply, please set auto-merge for this merge request.

    See pipeline tiers and merging a merge request for more details.

  • Lesley Razzaghian requested review from @brytannia and removed review request for @lesley-r

    requested review from @brytannia and removed review request for @lesley-r

  • requested review from @furkanayhan

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for bb00b437

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Create      | 129    | 0      | 18      | 0     | 147   | ✅     |
    | Govern      | 73     | 0      | 0       | 0     | 73    | ✅     |
    | Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
    | Verify      | 45     | 0      | 2       | 0     | 47    | ✅     |
    | Package     | 24     | 0      | 11      | 0     | 35    | ✅     |
    | Plan        | 76     | 0      | 0       | 0     | 76    | ✅     |
    | Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Data Stores | 33     | 0      | 1       | 0     | 34    | ✅     |
    | Fulfillment | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Secure      | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Manage      | 1      | 0      | 1       | 0     | 2     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 402    | 0      | 33      | 0     | 435   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-omnibus: :white_check_mark: test report for 57880e0a

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Plan        | 142    | 0      | 16      | 0     | 158   | ✅     |
    | Create      | 222    | 0      | 30      | 0     | 252   | ✅     |
    | Data Stores | 58     | 0      | 16      | 0     | 74    | ✅     |
    | Analytics   | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Package     | 44     | 0      | 20      | 0     | 64    | ✅     |
    | Verify      | 70     | 0      | 24      | 0     | 94    | ✅     |
    | Govern      | 136    | 0      | 8       | 0     | 144   | ✅     |
    | Monitor     | 4      | 0      | 20      | 0     | 24    | ✅     |
    | Secure      | 6      | 0      | 4       | 0     | 10    | ✅     |
    | Release     | 8      | 0      | 2       | 0     | 10    | ✅     |
    | Manage      | 2      | 0      | 14      | 0     | 16    | ✅     |
    | Configure   | 0      | 0      | 2       | 0     | 2     | ➖     |
    | ModelOps    | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Fulfillment | 4      | 0      | 14      | 0     | 18    | ✅     |
    | Growth      | 0      | 0      | 4       | 0     | 4     | ➖     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 700    | 0      | 178     | 0     | 878   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Furkan Ayhan
  • Furkan Ayhan
  • Furkan Ayhan requested changes

    requested changes

  • added 1 commit

    • 86d62dcd - Use ask_build unit primitive

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 057edb50 - Add scenario to test lines limit in JobAiEntity spec

    Compare with previous version

  • added 1 commit

    • c9e726d9 - Move extract_resource method to private

    Compare with previous version

  • Tetiana Chupryna removed review request for @brytannia

    removed review request for @brytannia

  • Tetiana Chupryna approved this merge request

    approved this merge request

  • added 1 commit

    • bd21e085 - Add scenarios for build URL identification

    Compare with previous version

  • Eva Kadlecová reset approvals from @brytannia by pushing to the branch

    reset approvals from @brytannia by pushing to the branch

  • added 1 commit

    • 34560f3d - Add scenarios for build URL identification

    Compare with previous version

  • requested review from @furkanayhan

  • mentioned in issue #498199 (closed)

  • Eva Kadlecová added 2 commits

    added 2 commits

    • ce183e26 - Remove represent method from JobSerializer
    • 57880e0a - Use build_reader unit primitive in BuildReader Executor

    Compare with previous version

  • Furkan Ayhan approved this merge request

    approved this merge request

  • Furkan Ayhan removed review request for @furkanayhan

    removed review request for @furkanayhan

  • @harsimarsandhu Can you please do the maintainer review?

  • requested review from @harsimarsandhu

  • Eva Kadlecová added 2240 commits

    added 2240 commits

    • 57880e0a...2694340c - 2230 commits from branch master
    • 239bbe59 - Support pipeline job context in Duo Chat
    • 1f21bb9d - Use ask_build unit primitive
    • c4c009d9 - Remove EE::JobSerializer
    • bac5de79 - Add scenario to test lines limit in JobAiEntity spec
    • a28069c1 - Move extract_resource method to private
    • 2982df5d - Add scenarios for build URL identification
    • 0171367b - Remove represent method from JobSerializer
    • df2ebd5d - Use build_reader unit primitive in BuildReader Executor
    • 1bb28ea2 - Keep represent method in JobSerializer
    • bb00b437 - Fix chat_spec.rb

    Compare with previous version

  • Eva Kadlecová reset approvals from @furkanayhan by pushing to the branch

    reset approvals from @furkanayhan by pushing to the branch

  • Mark Chao approved this merge request

    approved this merge request

  • Mark Chao enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Mark Chao resolved all threads

    resolved all threads

  • Mark Chao requested review from @furkanayhan

    requested review from @furkanayhan

  • Furkan Ayhan approved this merge request

    approved this merge request

  • Furkan Ayhan resolved all threads

    resolved all threads

  • merged

  • Mark Chao mentioned in commit 8513b692

    mentioned in commit 8513b692

  • Eva Kadlecová resolved all threads

    resolved all threads

  • Eva Kadlecová mentioned in issue #498931

    mentioned in issue #498931

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading