microsoft / coe-starter-kit

Other
737 stars 216 forks source link

[CoE Starter Kit - BUG] SYNC HELPER - Get Security Role Users #6140

Closed nasilvae closed 1 year ago

nasilvae commented 1 year ago

Does this bug already exist in our backlog?

Describe the issue

Flow is exhibiting an error where is not finding Security Roles image

Expected Behavior

Flow should account for all security roles

What solution are you experiencing the issue with?

Core

What solution version are you using?

July Release

What app or flow are you having the issue with?

Sync Helper - Get Security Role Users

What method are you using to get inventory and telemetry?

Data Export

Steps To Reproduce

Run Flow

Anything else?

No response

Jenefer-Monroe commented 1 year ago

Please see if it passes on rerun

annsarP commented 1 year ago

Please see if it passes on rerun

Hello. We are also experiencing this error in our environment. After re-run it passes. How to catch this error and avoid flow's run failure reported in CoE Admin Command Center app?

Jenefer-Monroe commented 1 year ago

Likely this was an issue because you had multiple instances of the flow running at once. Can you please tell me

  1. how many envts you have in your tenant
  2. how many times a day this flow runs (you can get this from the analytics page)
nasilvae commented 1 year ago

Likely this was an issue because you had multiple instances of the flow running at once. Can you please tell me

  1. how many envts you have in your tenant
  2. how many times a day this flow runs (you can get this from the analytics page)

Hi @Jenefer-Monroe the flow runs successfully if reran. We have about 803 environments and below is the bar graph from admin center.

image

Jenefer-Monroe commented 1 year ago

Can you please share how many times this flow is running per day? image

annsarP commented 1 year ago

Can you please share how many times this flow is running per day? image

For Admin | Sync Template v3 (Environment Properties) flow in my environment it's around 160 - 200 runs per day: image

Here's also analytics for SYNC HELPER - Get Security Role Users flow: image

Jenefer-Monroe commented 1 year ago

Good that is all in alignment with what I expected. Please share a screenshot of the flow runs themselves so I can see how long they are taking to execute. image

annsarP commented 1 year ago

Good that is all in alignment with what I expected. Please share a screenshot of the flow runs themselves so I can see how long they are taking to execute. image

For latest runs: image

For failed runs: image

nasilvae commented 1 year ago

Can you please share how many times this flow is running per day? image

image

nasilvae commented 1 year ago

Good that is all in alignment with what I expected. Please share a screenshot of the flow runs themselves so I can see how long they are taking to execute. image

image image

Jenefer-Monroe commented 1 year ago

thanks or your help. You and a few others assisted in finding the fix here. Please see https://github.com/microsoft/coe-starter-kit/issues/6095 for details. Local fixed for August.

CoEStarterKitBot commented 1 year ago

@nasilvae This has been fixed in the latest release. Please install the latest version of the toolkit following the instructions for installing updates. Note that if you do not remove the unmanaged layers as described there you will not receive updates from us.