microsoft / coe-starter-kit

Other
751 stars 224 forks source link

[CoE Starter Kit - BUG] Data Export - all ModelDriven Apps showing as owned by SYSTEM #7181

Open albertocastro365 opened 11 months ago

albertocastro365 commented 11 months ago

Does this bug already exist in our backlog?

Describe the issue

In PowerApps App table I haven't data of usage from Model Driven Apps. Also de App Owner of Model Driven apps is SYSTEM: image

Instead in Power BI Model I have the data usage and de App Owner correct: image

Expected Behavior

No response

What solution are you experiencing the issue with?

Core

What solution version are you using?

October 2023

What app or flow are you having the issue with?

PowerApps App table

What method are you using to get inventory and telemetry?

Data Export

Steps To Reproduce

No response

Anything else?

No response

AB#1516

Jenefer-Monroe commented 11 months ago

We do not have data usage for Model Driven apps unfortunately so that is expected

Jenefer-Monroe commented 11 months ago

For app owner, if you are using data export then the table should be written to by the dataflow as you show. Can you please tell me, directly after you do a dataflow refresh does it show correctly? Wondering if some other process is changing it?

albertocastro365 commented 11 months ago

We do not have data usage for Model Driven apps unfortunately so that is expected

But by any reason? I mean, the data exists in json files. It would be very helpfull have it in PowerApps App table.

For app owner, if you are using data export then the table should be written to by the dataflow as you show. Can you please tell me, directly after you do a dataflow refresh does it show correctly? Wondering if some other process is changing it?

No, we haven't any custom process in my CoESK Environment. I have this behaviour in DEV and PROD Environments.

Jenefer-Monroe commented 11 months ago

Sorry I meant any of our processes. Trying to see when they get flagged. Right after the dataflows run, is it correct? And then gets incorrect later? Or is it wrong right when the dataflows run.

Jenefer-Monroe commented 11 months ago

Regarding MDA usage, in the past they did not include the AppID for MDA's only for canvas. Looks like they ahve resolved this now! Will get this on the backlog

Jenefer-Monroe commented 11 months ago

ok I repro the issue with the SYSTEM user as well.

albertocastro365 commented 11 months ago

Sorry I meant any of our processes. Trying to see when they get flagged. Right after the dataflows run, is it correct? And then gets incorrect later? Or is it wrong right when the dataflows run.

The dataflow write wrong the information. But I am reviewing the CoE BYODL Apps datalflow and I see errors like this issue https://github.com/microsoft/coe-starter-kit/issues/7182#issuecomment-1828181181

Jenefer-Monroe commented 11 months ago

that is in relation to canvas apps yes?

albertocastro365 commented 11 months ago

Yes, in this other issue I am talking about of canvas. It seems a change in the structure of json files that affects to apps dataflows.

Jenefer-Monroe commented 11 months ago

OK it appears they have stopped writing the creatorid, I'm getting nulls, casn you check to see if yours is coming through as nulls as well?

Image

Jenefer-Monroe commented 11 months ago

looks like this bug https://github.com/microsoft/coe-starter-kit/issues/6658 which used to be just in this persons tenant has now spread. I'll dupe to that issue and we can track with it

albertocastro365 commented 11 months ago

OK it appears they have stopped writing the creatorid, I'm getting nulls, casn you check to see if yours is coming through as nulls as well?

Image

Me too

Jenefer-Monroe commented 11 months ago

Can you create a support ticket? We are pressing as well but that will be helpful

albertocastro365 commented 11 months ago

Done! Case 2312010050004046

albertocastro365 commented 11 months ago

Feedback from support: "I got feedback from the product team that for model apps, CreatedPrincipalId is set to null, this is a known issue. They are working on the enhancement of this feature to include createdby value."

Jenefer-Monroe commented 11 months ago

thank you so much for the update!