Sync audit events to new tables
What does this MR do and why?
Sync audit events to new tables
This commit enables syncing of audit events to new tables using AuditEventService
EE: true
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.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
- Login to local gdk
- Check
AuditEvent.last
andAuditEvents::UserAuditEvent.last
are both generated.
Related to #470481 (closed)
Merge request reports
Activity
added devopsgovern feature flag groupcompliance sectionsec labels
assigned to @harsimarsandhu
added pipelinetier-1 label
- Resolved by 🤖 GitLab Bot 🤖
Proper labels assigned to this merge request. Please ignore me.
@harsimarsandhu
- please see the following guidance and update this merge request.1 Error Please add typebug typefeature, or typemaintenance label to this merge request. Edited by 🤖 GitLab Bot 🤖
added typefeature label
- A deleted user
added backend label
3 Warnings e898d030: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines. 83515eeb: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines. This merge request does not refer to an existing milestone. 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 @olaoluro
(UTC+1, 4.5 hours behind author)
@tkuah
(UTC+12, 6.5 hours ahead of author)
groupcompliance Reviewer review is optional for groupcompliance @nradina
(UTC+2, 3.5 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
DangerEdited by Ghost User- Resolved by Hitesh Raghuvanshi
Hey @hraghuvanshi could you please review this change?
requested review from @hraghuvanshi
- Resolved by Harsimar Sandhu
- Resolved by Hitesh Raghuvanshi
- Resolved by Hitesh Raghuvanshi
- Resolved by Harsimar Sandhu
- Resolved by Hitesh Raghuvanshi
added pipeline:mr-approved label
added pipelinetier-3 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 @georgekoltsov
E2E Test Result Summary
allure-report-publisher
generated test report!e2e-test-on-gdk:
test report for e898d030expand test summary
+------------------------------------------------------------------+ | suites summary | +-------------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +-------------+--------+--------+---------+-------+-------+--------+ | Package | 16 | 0 | 15 | 0 | 31 | ✅ | | Verify | 43 | 0 | 2 | 0 | 45 | ✅ | | Plan | 70 | 0 | 0 | 0 | 70 | ✅ | | Govern | 71 | 0 | 0 | 0 | 71 | ✅ | | Create | 127 | 0 | 12 | 0 | 139 | ✅ | | Manage | 0 | 0 | 1 | 0 | 1 | ➖ | | Release | 5 | 0 | 0 | 0 | 5 | ✅ | | Data Stores | 30 | 0 | 1 | 0 | 31 | ✅ | | Analytics | 2 | 0 | 0 | 0 | 2 | ✅ | | Monitor | 8 | 0 | 0 | 0 | 8 | ✅ | | Fulfillment | 1 | 0 | 0 | 0 | 1 | ✅ | | Secure | 2 | 0 | 0 | 0 | 2 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+ | Total | 375 | 0 | 31 | 0 | 406 | ✅ | +-------------+--------+--------+---------+-------+-------+--------+
e2e-package-and-test:
test report for e898d030expand test summary
+-------------------------------------------------------------+ | suites summary | +--------+--------+--------+---------+-------+-------+--------+ | | passed | failed | skipped | flaky | total | result | +--------+--------+--------+---------+-------+-------+--------+ | Govern | 105 | 0 | 8 | 2 | 113 | ✅ | | Create | 270 | 0 | 34 | 0 | 304 | ✅ | +--------+--------+--------+---------+-------+-------+--------+ | Total | 375 | 0 | 42 | 2 | 417 | ✅ | +--------+--------+--------+---------+-------+-------+--------+
Edited by Ghost Usermentioned in incident gitlab-org/quality/engineering-productivity/approved-mr-pipeline-incidents#941 (closed)
reset approvals from @hraghuvanshi by pushing to the branch
- Resolved by Hitesh Raghuvanshi
requested review from @georgekoltsov
mentioned in commit 9b5f44c7
added workflowstaging-canary label
added workflowcanary label and removed workflowstaging-canary label
added workflowstaging label and removed workflowcanary label
added workflowproduction label and removed workflowstaging label
changed milestone to %17.3
added workflowpost-deploy-db-production label and removed workflowproduction label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label