feat(powerbi): Report to Dashboard lineage

[sgomezvillamor]

PowerBI Report objects (mapped as DataHub Dashboards) can contain PowerBI Dashboard objects (also DataHub Dashboards).

Important

Depends on PR #12433

Checklist

• The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• Links to related issues (if applicable)
• Tests for the changes have been added/updated (if applicable)
• Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
• For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Files with missing lines	Coverage Δ
...ion/src/datahub/ingestion/source/powerbi/config.py	98.45% <100.00%> (+<0.01%)	⬆️
...on/src/datahub/ingestion/source/powerbi/powerbi.py	94.64% <100.00%> (+0.28%)	⬆️
...on/source/powerbi/rest_api_wrapper/data_classes.py	92.99% <100.00%> (+0.03%)	⬆️
...n/source/powerbi/rest_api_wrapper/data_resolver.py	83.60% <ø> (+<0.01%)	⬆️
...ion/source/powerbi/rest_api_wrapper/powerbi_api.py	90.72% <100.00%> (+0.12%)	⬆️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ac13f25...bf21f7d. Read the comment docs.

[sgomezvillamor]

Tested locally with the generated golden file

datahub ingest mcps /Users/sergio/workspace/github/datahub-project/datahub/metadata-ingestion/tests/integration/powerbi/golden_test_container.json

[hsheth2]

thanks for cleaning this up

are there any entries in Constant that can be removed now?

[hsheth2]

are report_id and report set at the same time? if not, it might be worth putting a comment that says the latter is filled in at a later time

[sgomezvillamor]

ids are set on a first traversal and objects in a later one
I will add a comment

[hsheth2]

seems worrying that these lines weren't covered by the tests? as per the codecov report

also, to make sure I understand - the lineage is tile -> dashboard -> report?

[sgomezvillamor]

seems worrying that these lines weren't covered by the tests? as per the codecov report

this may be a sort of false positive, because my update on default_mock_response.json was specifically tailored to force this code path

also, to make sure I understand - the lineage is tile -> dashboard -> report?

Lineage is represented with a contains relationship. Any PowerBI Dashboard having a PowerBI Tile which references a parent PowerBI Report is modelled as follows:

PowerBI Report (DataHub Dashboard) -- contains --> PowerBI Dashboard (DataHub Dashboard) // this is the new addtion
PowerBI Dashboard (DataHub Dashboard) --contains --> PowerBI Tile (DataHub Chart) // this one was already supported

This was confirmed with users to match their expectations; this is how is modelled in PowerBI Lineage.

[sgomezvillamor]

merging; CI error is unrelated