Skip to content
Snippets Groups Projects

Expose `vulnerability_events` field in REST integrations API endpoints

All threads resolved!

What does this MR do and why?

The vulnerability events have been around since 14.3 49e788b9 but was previously missing from integration API responses. This change exposes the field in the integrations API.

How to set up and validate locally

Visit the integrations API endpoints:

  • http://gdk.test:3000/api/v4/projects/<project_id>/integrations?private_token=<private_token>
  • http://gdk.test:3000/api/v4/projects/<project_id>/integrations/jenkins?private_token=<private_token> (or any individual integration endpoint)

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 Luke Duncalfe

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
  • Luke Duncalfe requested review from @eread

    requested review from @eread

  • Contributor

    @.luke Some end-to-end (E2E) tests should run based on the stage label.

    Please start the trigger-omnibus-and-follow-up-e2e job in the qa stage and ensure tests in the follow-up-e2e:package-and-test-ee pipeline pass before this MR is merged. (E2E tests are computationally intensive and don't run automatically for every push/rebase, so we ask you to run this job manually at least once.)

    To run all E2E tests, apply the pipeline:run-all-e2e label and run a new pipeline.

    E2E test jobs are allowed to fail due to flakiness. See current failures at the latest pipeline triage issue.

    Once done, apply the :white_check_mark: emoji on this comment.

    Team members only: for any questions or help, reach out on the internal #quality Slack channel.

  • Evan Read removed review request for @eread

    removed review request for @eread

  • Contributor

    Allure report

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :exclamation: test report for 223ba34c

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Data Stores | 20     | 0      | 0       | 1     | 20    | ❗     |
    | Manage      | 13     | 0      | 1       | 0     | 14    | ✅     |
    | Plan        | 51     | 0      | 0       | 0     | 51    | ✅     |
    | Create      | 38     | 0      | 0       | 0     | 38    | ✅     |
    | Verify      | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Govern      | 34     | 0      | 0       | 0     | 34    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 164    | 0      | 1       | 1     | 165   | ❗     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :exclamation: test report for 774209f4

    expand test summary
    +-----------------------------------------------------------------------+
    |                            suites summary                             |
    +------------------+--------+--------+---------+-------+-------+--------+
    |                  | passed | failed | skipped | flaky | total | result |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Manage           | 158    | 0      | 13      | 3     | 171   | ❗     |
    | Create           | 151    | 0      | 35      | 1     | 186   | ❗     |
    | Configure        | 1      | 0      | 0       | 0     | 1     | ✅     |
    | Package          | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Plan             | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Monitor          | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Data Stores      | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Govern           | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Framework sanity | 0      | 0      | 2       | 0     | 2     | ➖     |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Total            | 334    | 0      | 52      | 4     | 386   | ❗     |
    +------------------+--------+--------+---------+-------+-------+--------+
  • Luke Duncalfe added 1 commit

    added 1 commit

    • 774209f4 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Luke Duncalfe requested review from @eread

    requested review from @eread

  • Luke Duncalfe requested review from @bmarjanovic

    requested review from @bmarjanovic

  • Evan Read approved this merge request

    approved this merge request

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

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

    For more info, please refer to the following links:

  • Evan Read resolved all threads

    resolved all threads

  • Bojan Marjanovic approved this merge request

    approved this merge request

  • Bojan Marjanovic requested review from @reprazent and removed review request for @bmarjanovic

    requested review from @reprazent and removed review request for @bmarjanovic

  • Evan Read removed review request for @eread

    removed review request for @eread

  • Luke Duncalfe changed the description

    changed the description

  • Bob Van Landuyt removed review request for @reprazent

    removed review request for @reprazent

  • Luke Duncalfe added 1 commit

    added 1 commit

    • 223ba34c - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Luke Duncalfe changed milestone to %16.5

    changed milestone to %16.5

  • Luke Duncalfe requested review from @reprazent

    requested review from @reprazent

  • Evan Read approved this merge request

    approved this merge request

  • Bob Van Landuyt resolved all threads

    resolved all threads

  • Bob Van Landuyt approved this merge request

    approved this merge request

  • Bob Van Landuyt enabled an automatic merge when the pipeline for a69c26ab succeeds

    enabled an automatic merge when the pipeline for a69c26ab succeeds

  • Bob Van Landuyt mentioned in commit 9ced950c

    mentioned in commit 9ced950c

  • added workflowstaging label and removed workflowcanary label

  • Luke Duncalfe changed milestone to %16.4

    changed milestone to %16.4

  • Luke Duncalfe mentioned in merge request !132306 (merged)

    mentioned in merge request !132306 (merged)

  • Please register or sign in to reply
    Loading