Support pipeline job context in Duo Chat
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
- From Rails console enable feature flags:
Feature.enable(:ai_build_reader_for_chat)
Feature.enable(:v2_chat_agent_integration)
- Run GDK.
- In AI gateway repo checkout the
468461-support-pipeline-context-in-duo-chat
branch and run AI gateway. - Visit a pipeline job and ask a question to Duo Chat about the pipeline (What is this pipeline about?)
- You should receive a details about the build.
Related to #468461 (closed)
Merge request reports
Activity
changed milestone to %17.4
assigned to @evakadlecova
added pipelinetier-1 label
- A deleted user
added backend feature flag labels
- Resolved by Eva Kadlecová
4 Warnings This merge request is quite big (627 lines changed), please consider splitting it into multiple merge requests. bb00b437: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines. c4c009d9: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines. 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 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
(UTC+2, same timezone as author)
@harsimarsandhu
(UTC+5.5, 3.5 hours ahead of author)
~"Verify" Reviewer review is optional for ~"Verify" @stanhu
(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
danger-review
job that generated this comment.Generated by
Dangeradded 507 commits
-
c8cf88f5...1cab5da7 - 506 commits from branch
master
- 2e859032 - Support pipeline job context in Duo Chat
-
c8cf88f5...1cab5da7 - 506 commits from branch
added 9 commits
-
2e859032...fbd06fc2 - 8 commits from branch
master
- 477b3845 - Support pipeline job context in Duo Chat
-
2e859032...fbd06fc2 - 8 commits from branch
changed milestone to %17.5
added missed:17.4 label
added 2732 commits
-
ee27a407...43a3d0d8 - 2731 commits from branch
master
- 6ca78d13 - Support pipeline job context in Duo Chat
-
ee27a407...43a3d0d8 - 2731 commits from branch
added 80 commits
-
8ff796cb...9524e105 - 79 commits from branch
master
- 4eb650af - Support pipeline job context in Duo Chat
-
8ff796cb...9524e105 - 79 commits from branch
added 163 commits
-
8265dea3...9889e08c - 162 commits from branch
master
- a50183e2 - Support pipeline job context in Duo Chat
-
8265dea3...9889e08c - 162 commits from branch
added 435 commits
-
a50183e2...e93fc117 - 434 commits from branch
master
- 3761fe60 - Support pipeline job context in Duo Chat
-
a50183e2...e93fc117 - 434 commits from branch
added 61 commits
-
3761fe60...56d8d2cc - 60 commits from branch
master
- e8e27c57 - Support pipeline job context in Duo Chat
-
3761fe60...56d8d2cc - 60 commits from branch
added 35 commits
-
e8e27c57...ae775944 - 34 commits from branch
master
- 1e5dd07c - Support pipeline job context in Duo Chat
-
e8e27c57...ae775944 - 34 commits from branch
added 386 commits
-
1e5dd07c...9f829ba5 - 385 commits from branch
master
- cda4ce73 - Support pipeline job context in Duo Chat
-
1e5dd07c...9f829ba5 - 385 commits from branch
mentioned in merge request gitlab-org/modelops/applied-ml/code-suggestions/ai-assist!1379 (merged)
added 33 commits
-
cda4ce73...9f4a2d68 - 32 commits from branch
master
- f11751b5 - Support pipeline job context in Duo Chat
-
cda4ce73...9f4a2d68 - 32 commits from branch
- Resolved by Mark Chao
@brytannia Can you please review this MR?
requested review from @brytannia
- Resolved by Mark Chao
requested review from @lesley-r and removed review request for @brytannia
- Resolved by Lesley Razzaghian
This is really cool!!
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!
added pipeline:mr-approved label
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.
requested review from @brytannia and removed review request for @lesley-r
requested review from @furkanayhan
- Resolved by Furkan Ayhan
@furkanayhan Can you do the Verify review please? Thank you!
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for bb00b437expand 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:
test report for 57880e0aexpand 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 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
- Resolved by Furkan Ayhan
- Resolved by Eva Kadlecová
- Resolved by Furkan Ayhan
- Resolved by Eva Kadlecová
added 1 commit
- 057edb50 - Add scenario to test lines limit in JobAiEntity spec
removed review request for @brytannia
reset approvals from @brytannia by pushing to the branch
- Resolved by Mark Chao
@furkanayhan I addressed your comments, can you please have another look?
Edited by Eva Kadlecová
requested review from @furkanayhan
added verify-reviewimpacted label
- Resolved by Furkan Ayhan
mentioned in issue #498199 (closed)
removed review request for @furkanayhan
added pipelinetier-3 pipeline:run-e2e-omnibus-once labels and removed pipelinetier-2 label
removed pipeline:run-e2e-omnibus-once label
@harsimarsandhu Can you please do the maintainer review?
requested review from @harsimarsandhu
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
Toggle commit list-
57880e0a...2694340c - 2230 commits from branch
reset approvals from @furkanayhan by pushing to the branch
- Resolved by Furkan Ayhan
Hi @furkanayhan could we ask you to approve again due to rebasing? Thanks.
requested review from @furkanayhan
started a merge train
mentioned in commit 8513b692
mentioned in issue #498931
added workflowstaging-canary label and removed workflowin dev label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
added workflowpost-deploy-db-staging label and removed workflowproduction label
added workflowpost-deploy-db-production label and removed workflowpost-deploy-db-staging label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label
mentioned in issue gitlab-org/quality/triage-reports#20610 (closed)