microsoft / coe-starter-kit

Other
720 stars 211 forks source link

[CoE Starter Kit - BUG] Default Environment not listed in DLP Impact Tool #8461

Open nasilvae opened 2 weeks ago

nasilvae commented 2 weeks ago

Does this bug already exist in our backlog?

Describe the issue

Team,

can we include "Default" to the drop down environment type selector to the impact tool as this was the original ask?

https://github.com/microsoft/coe-starter-kit/issues/6555

Expected Behavior

Default environment should be available in the DLP Impact app

What solution are you experiencing the issue with?

Core

What solution version are you using?

4.31

What app or flow are you having the issue with?

DLP Impact Analysis

What method are you using to get inventory and telemetry?

Cloud flows

Steps To Reproduce

No response

Anything else?

No response

AB#3553

Jenefer-Monroe commented 2 weeks ago

Default is not excluded I do see it in mine, so perhaps the original issue of too many envts just wasnt correctly addressed. Sending to dev for investigation.

nasilvae commented 2 weeks ago

Default is not excluded I do see it in mine, so perhaps the original issue of too many envts just wasnt correctly addressed. Sending to dev for investigation.

I just replicated it in mines. I searched for the default environment but I don't see it as an option in the combo box to filter it.

image

The tenant experiencing the problem operates on a large scale. At present, we're manually embedding the environment into the application to make it functional. Our requirement is simply to include the Default environment SKU in the dropdown menu. This will allow us to select and operate with the default environment without the necessity for manual embedding. The tenant possesses more than 2800 environments, which we believe exceeds the capacity of the environment panel, leading to the current issue.

nasilvae commented 2 weeks ago

We just need the following change :) image

nasilvae commented 2 weeks ago

We just need the following change :) image

Issue seems to be more complex :(

image

nasilvae commented 2 weeks ago

I think we need the combo box to reset the query so it reloads the data. We also are seeing teams environments but they are not in the combo box.

Jenefer-Monroe commented 2 weeks ago

oh I see, I was on the wrong page entirely. Yes I dont see teams or default there either image

To dev for investigation.

nasilvae commented 2 weeks ago

We believe the issue to be with the collections. image

We are using one collection for all environment types and we are filtering from that collection. We need a collection for each environment type. This way we have more space for more environments.

nasilvae commented 2 weeks ago

We are modified the original collection to only have the default environment since that is the environment we are currently working on as a workaround but We believe that if we create a collection for the different environment types we should be good to go. As this will give the app more runway for tenants with thousands of environments.

col_environments, Filter(Environments, 'Environment Sku' = "Default")

Jenefer-Monroe commented 2 weeks ago

Does it not work to just search for default without the filter at all? image

nasilvae commented 2 weeks ago

Does it not work to just search for default without the filter at all? image

no, We believe that because we have so many environments we go over the limit of the collection.