Skip to content

Enable multi-entity and set default Dataflow template to 1.0.1#137

Closed
clincoln8 wants to merge 2 commits into
datacommonsorg:mainfrom
clincoln8:enable-multi-entity
Closed

Enable multi-entity and set default Dataflow template to 1.0.1#137
clincoln8 wants to merge 2 commits into
datacommonsorg:mainfrom
clincoln8:enable-multi-entity

Conversation

@clincoln8

@clincoln8 clincoln8 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

This PR:

  • Adds enableMultiEntity: 'true' to the Dataflow launch parameters in the Cloud Workflow definition. This enables support for multi-entity schemas during ingestion.
  • Updates the default Dataflow template version to ingestion-1.0.1.json in the workflow module variables.

@clincoln8 clincoln8 requested a review from dwnoble June 22, 2026 22:44

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the ingestion workflow module by adding the enableMultiEntity: 'true' parameter and updating the default Dataflow template path to version 1.0.1. The reviewer suggests exposing enableMultiEntity as a configurable variable instead of hardcoding it to maintain flexibility and backward compatibility with older template versions.

Comment thread infra/dcp/modules/ingestion/workflow/main.tf
… 1.0.1

- Added enableMultiEntity: 'true' to the Dataflow launch parameters in the Cloud Workflow definition.
- Updated the default Dataflow template version to ingestion-1.0.1.json in the workflow module variables.
- Removed isBaseDc: 'false' from the workflow launch parameters to ensure compatibility with the ingestion-1.0.1.json template (which does not support this parameter).

TAG=agy
CONV=22ae774d-bd33-45c6-b311-3f67e552a618
@clincoln8 clincoln8 force-pushed the enable-multi-entity branch from 34ce0f7 to ce2dee8 Compare June 22, 2026 23:17
Comment thread infra/dcp/modules/ingestion/workflow/main.tf
@clincoln8

Copy link
Copy Markdown
Contributor Author

no longer needed

@clincoln8 clincoln8 closed this Jun 23, 2026
@clincoln8 clincoln8 deleted the enable-multi-entity branch June 23, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant