/review

@TomerHeber Im not sure how exactly it resolves #982
dee-kryvenko suggested a workdaround

data "env0_teams" "all_teams" {}

data "env0_team" "compartment_admins" {
  for_each = toset([
    for group in data.env0_teams.all_teams.names : group
    if contains(local.okta_admin_groups, group)
  ])
  name = each.value
}

The downside is that env0_teams data source trying to get a list of all teams in the org, which is excessive and unnecessary. Additionally, it fails agains another issue described in #981

This PR's implementation filters the full backend response.

@TomerHeber Im not sure how exactly it resolves #982 dee-kryvenko suggested a workdaround

data "env0_teams" "all_teams" {}

data "env0_team" "compartment_admins" {
  for_each = toset([
    for group in data.env0_teams.all_teams.names : group
    if contains(local.okta_admin_groups, group)
  ])
  name = each.value
}

The downside is that env0_teams data source trying to get a list of all teams in the org, which is excessive and unnecessary. Additionally, it fails agains another issue described in #981

This PR's implementation filters the full backend response.

Hi @liranfarage89 :

Have you observed the discussion in:
#982

In addition we are now caching the API call. The full teams list will be stored in memory (for every terraform execution).

@TomerHeber I read it and still don't get why this filter is necessary as long as the user suggested a workaround that basically does the same

@TomerHeber I read it and still don't get why this filter is necessary as long as the user suggested a workaround that basically does the same

@liranfarage89

Got it!
Are you okay with closing this issue?