Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Grab credentials from profiles.yml #491

Merged

Conversation

pik94
Copy link
Contributor

@pik94 pik94 commented Apr 11, 2023

Provide an opportunity to users to extract database credentials from dbt profiles.yml when creating a new data source in the Datafold cloud.

When the user reaches the step of entering database credentials when creating a new data source in the Datafold cloud, he is prompted to use the credentials from profiles.yml. If he gives his consent to this, we collect the necessary data from the file.

If for some reason we were unable to extract this data from the file, a corresponding message is displayed in the terminal and it is suggested to enter this data manually.

@pik94 pik94 added enhancement New feature or request cloud Issues related to data-diff <> Datafold Cloud interaction --dbt Issues/features related to the dbt integration labels Apr 11, 2023
@pik94 pik94 self-assigned this Apr 11, 2023
@pik94 pik94 force-pushed the ilia-dx-564-grab-credentials-from-profilesyml branch from 6570989 to c85b83a Compare April 11, 2023 11:29
@pik94 pik94 changed the title Ilia dx 564 grab credentials from profilesyml Grab credentials from profiles.yml Apr 11, 2023
@pik94 pik94 force-pushed the ilia-dx-564-grab-credentials-from-profilesyml branch from 14518fa to d5fcdaa Compare April 11, 2023 14:46
@pik94 pik94 force-pushed the ilia-dx-564-grab-credentials-from-profilesyml branch from d5fcdaa to ae862c6 Compare April 12, 2023 12:35
@pik94 pik94 force-pushed the ilia-dx-564-grab-credentials-from-profilesyml branch from ae862c6 to e391403 Compare April 12, 2023 12:37
@pik94 pik94 merged commit 4cf56db into datafold:master Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
--dbt Issues/features related to the dbt integration cloud Issues related to data-diff <> Datafold Cloud interaction enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants