Skip to content
Snippets Groups Projects

Instance streaming audit events - add FF and tab

All threads resolved!

What does this MR do and why?

This MR adds the feature flag instance_streaming_audit_events and when it is enabled it shows the Streams tab in Instance level Audit events page

Screenshots or screen recordings

Audit-Events--Admin-Area--GitLab

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Enable the feature flag
    Feature.enable(:instance_streaming_audit_events)
  2. Navigate to Admin menu and go to Monitoring -> Audit Events
  3. Ensure that the Streams tab is visible.
  4. Note: Clicking on this will cause an error due to backend APIs not setup correctly. This will be solved in subsequent MRs

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 #398107 (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
8 8 #js-audit-log-app{ data: { form_path: admin_audit_logs_path,
9 9 events: @events.to_json,
10 10 is_last_page: @is_last_page.to_json,
11 group_path: 'instance',
  • Nate Rosandich added 1 commit

    added 1 commit

    • 4ba6d913 - Add feature flag and show new tab

    Compare with previous version

  • Jay Montal
  • Nate Rosandich mentioned in commit b86b31a2

    mentioned in commit b86b31a2

  • Nate Rosandich added 1 commit

    added 1 commit

    • b86b31a2 - Add feature flag and show new tab

    Compare with previous version

  • mentioned in issue #409409 (closed)

  • Jay Montal approved this merge request

    approved this merge request

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

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

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

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

    For more info, please refer to the following links:

  • Jay Montal requested review from @f_caplette

    requested review from @f_caplette

  • Frédéric Caplette removed review request for @f_caplette

    removed review request for @f_caplette

  • Thong Kuah removed review request for @tkuah

    removed review request for @tkuah

  • Nate Rosandich added 1 commit

    added 1 commit

    • 83860ce5 - Move data injection to helper

    Compare with previous version

  • Nate Rosandich requested review from @f_caplette

    requested review from @f_caplette

  • Nate Rosandich requested review from @tkuah

    requested review from @tkuah

  • Nate Rosandich added 1 commit

    added 1 commit

    Compare with previous version

  • Thong Kuah approved this merge request

    approved this merge request

  • Thong Kuah removed review request for @tkuah

    removed review request for @tkuah

  • Frédéric Caplette approved this merge request

    approved this merge request

  • Frédéric Caplette resolved all threads

    resolved all threads

  • Frédéric Caplette requested review from @jmontal

    requested review from @jmontal

  • Jay Montal approved this merge request

    approved this merge request

  • Frédéric Caplette resolved all threads

    resolved all threads

  • Frédéric Caplette enabled an automatic merge when the pipeline for 5614ab99 succeeds

    enabled an automatic merge when the pipeline for 5614ab99 succeeds

  • Jay Montal removed review request for @jmontal

    removed review request for @jmontal

  • Nate Rosandich mentioned in commit 7c8abf3a

    mentioned in commit 7c8abf3a

  • Nate Rosandich aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • Nate Rosandich added 1640 commits

    added 1640 commits

    Compare with previous version

  • Thong Kuah enabled an automatic merge when the pipeline for 51d0ff9d succeeds

    enabled an automatic merge when the pipeline for 51d0ff9d succeeds

  • merged

  • Thong Kuah mentioned in commit 1f66473e

    mentioned in commit 1f66473e

  • Nate Rosandich mentioned in commit 86c62c1e

    mentioned in commit 86c62c1e

  • Nate Rosandich mentioned in merge request !121102 (merged)

    mentioned in merge request !121102 (merged)

  • Nate Rosandich mentioned in merge request !131851 (merged)

    mentioned in merge request !131851 (merged)

  • Please register or sign in to reply
    Loading