Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CT-3005] Support dictionary of additional configs in dbt_project.yml #8438

Closed
1 task done
Tracked by #8600
jtcohen6 opened this issue Aug 17, 2023 · 1 comment · Fixed by #8527
Closed
1 task done
Tracked by #8600

[CT-3005] Support dictionary of additional configs in dbt_project.yml #8438

jtcohen6 opened this issue Aug 17, 2023 · 1 comment · Fixed by #8527

Comments

@jtcohen6
Copy link
Contributor

jtcohen6 commented Aug 17, 2023

Housekeeping

  • I am a maintainer of dbt-core

Short description

Allow setting a dictionary of arbitrary configs in dbt_project.yml, which are to be read & used by dbt Cloud.

We don't need to validate the keys in this dictionary, nor store them in the project config class.

Let's call this dictionary dbt-cloud:

# dbt_project.yml
dbt-cloud:
  project-id: 1234
  ...

Acceptance criteria

It is possible to set a dictionary of arbitrary configs in dbt_project.yml, as metadata for dbt Cloud.

Impact to Other Teams

Cloud CLI impacted (see jira ink below), ensure this is completed within this sprint

Context

internal slack thread
internal jira ticket

@github-actions github-actions bot changed the title Support dictionary of additional configs in dbt_project.yml [CT-3005] Support dictionary of additional configs in dbt_project.yml Aug 17, 2023
@graciegoheen
Copy link
Contributor

graciegoheen commented Aug 21, 2023

Open question:

  • how will they access these keys? are they intending on parsing these themselves?
    • via reading the dbt_project.yml file
    • yes, they would parse these themselves
  • what should we name this?
    • let's go with cloud dbt-cloud

response discussed here and here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants