You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The group migration integration tests are a common culprit for failing sporadically because accounts groups pile up over time causing tests to timeout due to longer running API calls (e.g. #3706 and #3702).
Proposed Solution
Scope relevant groups using include_group_ids in the GroupsCrawler and dependent object so that we do not need to list all account groups in the integration tests bu that we limit the calls to GET for the include_group_ids
Additional Context
A include_group_names parameter already exists, which should be kept for backwards compatibility of the config file. We need the group ids to switch from LIST to GET calls.
Also #3718 requests for a clean up of the account groups
The text was updated successfully, but these errors were encountered:
Is there an existing issue for this?
Problem statement
The group migration integration tests are a common culprit for failing sporadically because accounts groups pile up over time causing tests to timeout due to longer running API calls (e.g. #3706 and #3702).
Proposed Solution
Scope relevant groups using
include_group_ids
in theGroupsCrawler
and dependent object so that we do not need to list all account groups in the integration tests bu that we limit the calls toGET
for theinclude_group_ids
Additional Context
A
include_group_names
parameter already exists, which should be kept for backwards compatibility of the config file. We need the group ids to switch fromLIST
toGET
calls.Also #3718 requests for a clean up of the account groups
The text was updated successfully, but these errors were encountered: