Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

cli: Respect DataSourceOverrides CLI flag #4319

Merged
merged 2 commits into from
Dec 13, 2022

Conversation

briancain
Copy link
Member

Prior to this commit, the job template submitted by the pipeline run CLI would ignore any data source overrides defined by the base CLI flag. This commit updates that to respect those overrides, if set.

Prior to this commit, the job template submitted by the pipeline run CLI
would ignore any data source overrides defined by the base CLI flag.
This commit updates that to respect those overrides, if set.
@briancain briancain added pr/no-changelog No automatic changelog entry required for this pull request backport/0.10.x labels Dec 12, 2022
@briancain briancain requested review from a team, catsby and izaaklauer December 12, 2022 23:29
@github-actions github-actions bot added the core label Dec 12, 2022
@briancain briancain removed the pr/no-changelog No automatic changelog entry required for this pull request label Dec 12, 2022
Copy link
Contributor

@catsby catsby left a comment

Choose a reason for hiding this comment

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

Should we add a changelog entry? It seems like a bug fix or at the least a change in behavior if users are using that flag and perhaps not noticing that it's not working as intended

@briancain
Copy link
Member Author

@catsby already added a changelog! I add that label when I open the PR so the commit doesn't get a PR check failure, and then remove it when I commit the changelog with the right PR number.

@briancain briancain merged commit c719a38 into main Dec 13, 2022
@briancain briancain deleted the cli/pipeline-run/use-datasource-overrides branch December 13, 2022 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants