Remove github_import_extended_events feature flag
What does this MR do and why?
This change removes the github_import_extended_events
feature flag and all the no longer used code.
The feature flag was enabled by default on GitLab 16.9.
Workers that are no longer used were updated to not perform any tasks following the Sidekiq best-practices. A follow-up MR will delete the files.
Related to: #435089 (closed)
Context
For context, on #433536 (closed), GitHub Import was updated to import the resources notes, merged_by, pull request reviews, and pull request review requests via events. The change is behind the github_import_extended_events
feature flag. So, now that we are removing the feature flag, we can remove all code that imports such resources via the previous original approach.
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
- Trigger a GitHub Import
- Go to + Project
- Import Project
- Select GitHub Import
- Provide the access token
- Select the project to be imported
- Check if no error occurs and resources were imported
- Check the import stats in the UI or via API by requesting
GET /api/v4/projects/<PROJECT_ID>/import/
- Navigate the imported project and check if issues, merge requests, notes, and events were imported
- Check the import stats in the UI or via API by requesting