datahub
datahub copied to clipboard
feat(ingest): powerbi # add powerbi workspaces to containers
Conditionally add PowerBI workspace as a container in ingestion, the implementation then adds all of the entities ingested from the workspace to the container.
The workspace name is not delivered outside from the ingestion phase, so it's not possible to achieve similar behavior with transformation steps.
Configuration defaults to not extracting containers.
Checklist
- [x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
- [x] Links to related issues (if applicable)
- [x] Tests for the changes have been added/updated (if applicable)
- [x] 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.
- [x] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub
Unit Test Results (metadata ingestion)
8 files 8 suites 57m 51s :stopwatch: 767 tests 765 :heavy_check_mark: 2 :zzz: 0 :x: 1 536 runs 1 531 :heavy_check_mark: 5 :zzz: 0 :x:
Results for commit 0abd0496.
:recycle: This comment has been updated with latest results.
Unit Test Results (build & test)
621 tests 617 :heavy_check_mark: 15m 37s :stopwatch: 157 suites 4 :zzz: 157 files 0 :x:
Results for commit 0abd0496.
:recycle: This comment has been updated with latest results.
@mohdsiddique Pinging for heads up, I've fixed your suggestions. The review may continue.
Oops - looks like there are conflicts. @mohdsiddique can help you sort any out in Slack before merging :)
Fixed, all of the conflicts were from my other PR.
Waiting on CI to do final merge. Cheers!