Skip to content

ADAP-1128: Migrate dbt-postgres into dbt-adapters#615

Merged
mikealfare merged 120 commits intomainfrom
monorepo/migrate-dbt-postgres
Jan 24, 2025
Merged

ADAP-1128: Migrate dbt-postgres into dbt-adapters#615
mikealfare merged 120 commits intomainfrom
monorepo/migrate-dbt-postgres

Conversation

@mikealfare
Copy link
Contributor

Migrate dbt-postgres into this repo using the following shell script and call:

repo=$1
source_branch=$2
target_branch=$3

# create a remote for the legacy adapter repo and fetch the latest commits
git remote remove old || true
git remote add old https://github.com/dbt-labs/$repo.git
git fetch old

# merge the updated branch from the legacy repo into the dbt-adapters repo
git checkout $target_branch
git merge old/$source_branch --allow-unrelated-histories

# remove the remote that was created by this process
git remote remove old || true
source ./scripts/migrate-adapter.sh dbt-postgres monorepo/stage monorepo/migrate-dbt-postgres

Update workflows to include dbt-postgres.

Make updates related with code quality checks due to slightly different config.

Merge this PR as a regular merge instead of a squash merge in order to retain git blame history.

@mikealfare
Copy link
Contributor Author

@mikealfare mikealfare marked this pull request as ready for review January 23, 2025 01:39
@mikealfare mikealfare requested a review from a team as a code owner January 23, 2025 01:39
@mikealfare
Copy link
Contributor Author

The dbt-redshift failures are all the IAM Role env variable issue and the dbt-athena failures are the iceberg retries issue. These are known issues and unrelated to dbt-postgres.

@mikealfare mikealfare merged commit 8b303eb into main Jan 24, 2025
211 of 218 checks passed
@mikealfare mikealfare deleted the monorepo/migrate-dbt-postgres branch January 24, 2025 18:16
colin-rogers-dbt pushed a commit that referenced this pull request Feb 3, 2025
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jérémy Guiselin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla:yes The PR author has signed the CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.