Skip to content

[7655] Upgrade GithubImport issue events mechanism

Rostyslav Safonov requested to merge rs-7655-migrate-issue-events-upgrade into master

What does this MR do and why?

It upgrades Github import issue events mechanism.

Firstly, we tried to import issue events via request per each issue. It consumed to much time to be done although imported all events. Current upgrade adding a way of using single events endpoint per repository what decreases number of requests. But it unfortunately also exclude specifically cross-reference events.

So, we keep ability to choose between feature flags:

  • github_importer_single_endpoint_issue_events_import (that is expensive but full)
  • github_importer_issue_events_import (that is light but excludes cross-referenced events - should become as default after verification)

See details: &7655 (closed) #365977 (comment 1022018304)

Screenshots or screen recordings

How to set up and validate locally

Perform import from Github repository which has issues with all event types in epic providen above. After import finish make sure that all issue events are present depends on feature flag you turned on.

  • github_importer_single_endpoint_issue_events_import flag should import all event types
  • github_importer_issue_events_import flag should import all event types expect cross-referenced events

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 Rostyslav Safonov

Merge request reports