PowerShell оптимизирует производительность

Есть несколько улучшений, которые вы можете внести, чтобы сделать ваш код более эффективным.

Ключевые моменты:

1. Избегайте Add-Member любой ценой, прикрепление свойств заметки к существующему объекту уже дорого, а этот командлет делает это еще дороже.
2. Уменьшите количество вызовов Graph. В настоящее время вы запрашиваете Graph, чтобы получить пользователей, соответствующих вашему условию -Filter, а затем для каждого пользователя вы делаете вызов Graph, чтобы проверить, являются ли они членами групп.
   Вместо этого сделайте один вызов Graph, получите пользователей, которые соответствуют вашему -Filter, а также получите их членство с помощью -ExpandProperty memberOf. Затем вы можете использовать это членство (свойство .MemberOf) для выполнения проверок.
3. Используйте -Select, чтобы уменьшить количество свойств, которые вы хотите вернуть с помощью Graph.

$tmSku = Get-MgSubscribedSku .....

$groups = [ordered]@{
    # key   = The property name in your result object
    # value = The Group Id (the GUID that identifies it)
    Group1 = 'xxxx-xxx-xxx-xxx-xxxxx'
    Group2 = 'xxxx-xxx-xxx-xxx-xxxxx'
    Group3 = 'xxxx-xxx-xxx-xxx-xxxxx'
}

$getMgUserSplat = @{
    Filter           = "assignedLicenses/any(x: x/skuId eq $($tmsku.SkuId))"
    ConsistencyLevel = 'eventual'
    CountVariable    = 'count'
    Select           = 'DisplayName', 'Id', 'Mail', 'UserPrincipalName', 'JobTitle', 'MemberOf'
    ExpandProperty   = 'memberOf'
    All              = $true
}

$users = Get-MgUser @getMgUserSplat

$result = foreach ($user in $users) {
    # use a `Hashset<T>` so we have the `.Contains` method from it
    # (fast stuff)
    [System.Collections.Generic.HashSet[guid]] $membership = @($user.MemberOf.Id)
    $outObject = [ordered]@{
        DisplayName       = $user.DisplayName
        Id                = $user.Id
        Mail              = $user.Mail
        UserPrincipalName = $user.UserPrincipalName
        JobTitle          = $user.JobTitle
    }

    foreach ($group in $groups.GetEnumerator()) {
        # `.Contains` here outputs a bool (true / false)
        # you can however add a condition here, if true then 'Yes', else 'No'
        $outObject[$group.Key] = $membership.Contains($group.Value)
    }

    [pscustomobject] $outObject
}

$result

Добавляю это как новый ответ, поскольку считаю, что предыдущий стоит сохранить для будущих читателей. Проблема с предыдущим ответом, как мы узнали с помощью OP (см. обширный раздел комментариев), tl;dr с использованием $expand=memberOf и $expand=transitiveMemberOf не дает полного членства пользователя, поэтому этот ответ подходит к проблеме по-другому, пытаясь сохранить его эффективность.

Основная идея, как объяснено в предыдущем ответе, состоит в том, чтобы попытаться максимально сократить количество вызовов Graph. Это основная причина, почему ваш текущий код работает медленно. Подход этого ответа состоит в том, чтобы запросить группы и получить их transitiveMembers , которые являются пользователями (transitiveMembers/microsoft.graph.user), где этим пользователям назначена целевая лицензия (assignedLicenses/any(e: e/skuId eq xxx-xx-xxx-xxx)), и для каждого пользователя проецировать свои Id ($select=id).

Затем мы применяем тот же подход, что и в вашем коде: получаем всех пользователей, имеющих назначенную лицензию, но для сравнения мы можем проверить, существует ли каждый пользователь в HashShet<T>, содержащем всех членов Id каждой группы.

Для справки: запрос членов группы можно выполнить с помощью Get-MgGroupMember, что должно сделать код менее обширным, но лично я не использую командлеты в модуле Graph, за исключением Invoke-MgGraphRequest, поэтому я, честно говоря, понятия не имею, возможно ли это и как это сделать с помощью этого командлета.

Add-Type -AssemblyName System.Web

# this function handles paging,
# see for details: https://learn.microsoft.com/en-us/graph/paging
function Get-GroupMembersId {
    [CmdletBinding()]
    param([string] $uri)

    do {
        $response = Invoke-MgGraphRequest GET $uri -Headers @{ ConsistencyLevel = 'eventual' }
        $uri = $response.'@odata.nextLink'
        if ($null -ne $response.value) {
            $response.value.Id
        }
    }
    while ($uri)
}

# the target Sku here
$tmSku = Get-MgSubscribedSku .....

# construct the Uri, the idea here will be to get all members of each group
# in `$groups.Keys` where the members of each group has the assigned license
# equal to `$tmsku.SkuId` and, for each user, project their Id
$query = [System.Web.HttpUtility]::ParseQueryString('')
$query['$count'] = 'true'
$query['$filter'] = 'assignedLicenses/any(e: e/skuId eq {0})' -f $tmsku.SkuId
$query['$select'] = 'id'
$uri = [System.UriBuilder] 'https://graph.microsoft.com'
$uri.Query = [System.Web.HttpUtility]::UrlDecode($query.ToString())

$groups = [ordered]@{
    # key   = The property name in your result object
    # value = The Group Id (the GUID that identifies it)
    Group1 = 'xxxx-xxx-xxx-xxx-xxxxx'
    Group2 = 'xxxx-xxx-xxx-xxx-xxxxx'
    Group3 = 'xxxx-xxx-xxx-xxx-xxxxx'
}

# this map will have:
# key   = The property name in your result object (group1, group2...)
# value = The Hashset<Guid> containing the user Ids members of each group
$map = [ordered]@{}
$groups.GetEnumerator() | ForEach-Object {
    # using `transitiveMembers` for recursive call (use just `members` for direct)
    $uri.Path = 'v1.0/groups/{0}/transitiveMembers/microsoft.graph.user' -f $_.Value
    $map[$_.Key] = [System.Collections.Generic.HashSet[guid]] @(Get-GroupMembersId $uri.Uri.ToString())
}

# now we continue with the same approach as in the previous answer but here
# we don't need to `ExpandProperty = 'memberOf'` because we already have all
# members of each group in `$map`
$getMgUserSplat = @{
    Filter           = "assignedLicenses/any(x: x/skuId eq $($tmsku.SkuId))"
    Select           = 'DisplayName', 'Id', 'Mail', 'UserPrincipalName', 'JobTitle'
    All              = $true
}

$users = Get-MgUser @getMgUserSplat

$result = foreach ($user in $users) {
    $outObject = [ordered]@{
        DisplayName       = $user.DisplayName
        Id                = $user.Id
        Mail              = $user.Mail
        UserPrincipalName = $user.UserPrincipalName
        JobTitle          = $user.JobTitle
    }

    foreach ($group in $map.GetEnumerator()) {
        # now here remember, `$group.Key` is the group name that
        # will later on be the Property Name of your object
        # and `$group.Value` is a HashSet<Guid> of all members of that group
        # so we can use its `.Contains` method for a super fast search
        $outObject[$group.Key] = $group.Value.Contains($user.Id)
    }

    [pscustomobject] $outObject
}

$result