Skip to content
Snippets Groups Projects

Change the unit primitive for troubleshoot job

Merged Allison Browne requested to merge ab-rca-unit-primitive into master
All threads resolved!

What does this MR do and why?

This adds the unit primitive for the /troubleshoot_job slash command tool.

Similar too: !157606 (merged)

Part of making this available to Enterprise Duo customers only. Issue: #457573 (closed)

Edited by Allison Browne

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
  • changed milestone to %17.3

  • added 1 commit

    Compare with previous version

  • Allison Browne added 699 commits

    added 699 commits

    Compare with previous version

  • Thanks @mksionek, sending this back over to you. :ping_pong:

    • Resolved by Gosia Ksionek

      @jivanvl, I've added some additional gating of the feature on the backend based on the duo enterprise license. I've pushed the ability to the front end, like so:

                push_frontend_ability(ability: :troubleshoot_job_with_ai, resource: @build, user: @current_user)
      

      Can we gate the feature on the front end side using this check in rootCauseAnalysisDuoIsAvailable and aiRootCauseAnalysisAvailable?

      We only want to show rootCauseAnalysisDuoIsAvailable if troubleshoot_job_with_ai is true.

      Do you want to commit directly to this branch? otherwise we could do it in 3 MRs:

      1. introduce backend ability and push_frontend_ability
      2. Use that on the frontend
      3. Use that on the backend

      To avoid ending up in a state where the frontend has access but the backend does not.

      Edited by Allison Browne
  • added 1 commit

    • f27fc747 - Push the new troubleshoot_job_with_ai ability.

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 6c27594d - Add ability check to the Frontend for job logs

    Compare with previous version

  • A deleted user added frontend label

    added frontend label

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 88ee8797 - Delete EE auth specs from CE

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 93ca5bdd - Fix the feature specs for button visibility

    Compare with previous version

  • added 1 commit

    • b4016c44 - Fix troubleshoot job executor mocking

    Compare with previous version

  • Allison Browne marked this merge request as ready

    marked this merge request as ready

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

  • Gosia Ksionek
  • added 1 commit

    • 5367e2e1 - Fix troubleshoot job executor specs

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • mentioned in issue #466262 (closed)

  • Gosia Ksionek approved this merge request

    approved this merge request

  • Gosia Ksionek resolved all threads

    resolved all threads

  • Gosia Ksionek enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • @mfanGitLab, this one needs verify approval too. Can you take a look?

  • Allison Browne requested review from @mfanGitLab

    requested review from @mfanGitLab

  • Max Fan approved this merge request

    approved this merge request

  • Hey @mokhax ! :wave: Are you able to help review this MR from an authorization point of view? Would be great if we could get this in sooner rather than later :pray:

  • Max Fan requested review from @mokhax

    requested review from @mokhax

  • mo khan approved this merge request

    approved this merge request

  • Gosia Ksionek started a merge train

    started a merge train

  • Gosia Ksionek removed this merge request from the merge train because pipeline did not succeed. Learn more.

    removed this merge request from the merge train because pipeline did not succeed. Learn more.

  • Gosia Ksionek started a merge train

    started a merge train

  • mentioned in issue #473802 (closed)

  • added workflowstaging label and removed workflowcanary label

  • mentioned in issue #473481 (closed)

  • Max Fan mentioned in merge request !160438 (merged)

    mentioned in merge request !160438 (merged)

  • Kev Kloss mentioned in merge request !161878 (merged)

    mentioned in merge request !161878 (merged)

  • Please register or sign in to reply
    Loading