Feature flag rollout jira_connect_full_namespace_sync
What
Remove the :jira_connect_full_namespace_sync
feature flag. The feature flag was added as a kill switch in case we see performance problems with the feature.
Owners
- Team:~"group::ecosystem"
- Most appropriate slack channel to reach out to:
#g_ecosystem
- Best individual to reach out to: @Andysoiron
Expectations
What are we expecting to happen?
The feature loads all projects in the given namespace and schedules a sidekiq job for each project that loads up to 400 merge requests. There is a potential performance problem when the feature is used for namespaces with many projects that have many merge requests. However we carefully considered those problems and addressed them in !43880 (merged)
We are planning to include more data in the sync so it would be useful to keep the feature flag in place until #263240 (closed) is done.
What can we monitor to detect problems with this?
- JiraConnect::SyncMergeRequestWorker: https://log.gprd.gitlab.net/goto/309f97d4a5c3e918e2c07754fefc94ee
- JiraConnect::SynctBranchWorker: https://log.gprd.gitlab.net/goto/96364e957898896c4dc7e9ee5534b6de
- JiraConnect::SyncProjectWorker: https://log.gprd.gitlab.net/goto/5f0e03847ddc1b074d6346199c8bc4d2
Roll Out Steps
-
Enable on staging ( /chatops run feature set jira_connect_full_namespace_sync true --staging
) -
Test on staging -
Ensure that documentation has been updated -
Enable on GitLab.com by running chatops command in #production
(/chatops run feature set jira_connect_full_namespace_sync true
) -
Announce on the issue that the flag has been enabled -
Monitor feature until next release (13.7) -
Add synchronization of branches and commits -
Monitor feature until next release (13.8) -
Remove feature flag and add changelog entry -
After the flag removal is deployed, clean up the feature flag by running chatops command in #production
channel
Rollback Steps
-
This feature can be disabled by running the following Chatops command:
/chatops run feature set jira_connect_full_namespace_sync false
Edited by Andy Schoenen