Skip to content
Snippets Groups Projects

Stream audit events to GCP logging for instance level UI

All threads resolved!

What does this MR do and why?

This adds a form to add the required fields for GCP Logging streaming APIs that was added here API to stream audit events to GCP logging for i... (&11303 - closed)

Screenshots or screen recordings

Audit-Events--Admin-Area--GitLab

How to set up and validate locally

  1. Go to any Admin Area
  2. Go to "Monitoring" > "Audit events"
  3. Click "Streams" tab
  4. Create a new streaming destination by clicking "Add streaming destination"
  5. Add, edit and delete both HTTP and GCP Logging destinations
    1. https://docs.gitlab.com/ee/administration/audit_event_streaming/
  6. Ensure group level streaming has not been effected
    1. https://docs.gitlab.com/ee/administration/audit_event_streaming/#top-level-group-streaming-destinations
  7. Ensure private key is not visible after first adding destination, but can be updated

MR acceptance checklist

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

Related to #415731 (closed)

Edited by Nate Rosandich

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
  • Evan Read
  • Evan Read
  • Evan Read
  • Evan Read removed review request for @eread

    removed review request for @eread

  • Nate Rosandich added 1 commit

    added 1 commit

    • 0572a03b - Apply 4 suggestion(s) to 1 file(s)

    Compare with previous version

  • Nate Rosandich requested review from @eread

    requested review from @eread

  • Nate Rosandich marked this merge request as ready

    marked this merge request as ready

  • Nate Rosandich added 638 commits

    added 638 commits

    Compare with previous version

  • 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 removed review request for @eread

    removed review request for @eread

  • Evan Read resolved all threads

    resolved all threads

  • Contributor

    @nrosandich 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.

  • Nate Rosandich added 214 commits

    added 214 commits

    Compare with previous version

  • Nate Rosandich added 1 commit

    added 1 commit

    • 24bafbb2 - Add instance GCP logging streaming destinations

    Compare with previous version

  • Nate Rosandich requested review from @cam.x

    requested review from @cam.x

  • Nate Rosandich requested review from @jmontal

    requested review from @jmontal

  • mentioned in issue #415731 (closed)

  • Evan Read approved this merge request

    approved this merge request

  • Jay Montal
  • Jay Montal
  • Jay Montal
  • Jay Montal approved this merge request

    approved this merge request

  • Jay Montal requested review from @zcuddy and removed review request for @jmontal

    requested review from @zcuddy and removed review request for @jmontal

  • mentioned in issue #428113 (closed)

  • Camellia X Yang approved this merge request

    approved this merge request

  • Zack Cuddy
  • Zack Cuddy resolved all threads

    resolved all threads

  • Zack Cuddy approved this merge request

    approved this merge request

  • Zack Cuddy enabled an automatic merge when the pipeline for 4e2c6c27 succeeds

    enabled an automatic merge when the pipeline for 4e2c6c27 succeeds

  • merged

  • Zack Cuddy mentioned in commit 03843c4b

    mentioned in commit 03843c4b

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading