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.

Sign up for GitHub

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

Jump to bottom

Release connectors with namespace change. #2990

Merged
merged 10 commits into from
Apr 21, 2021
Merged

Release connectors with namespace change. #2990

merged 10 commits into from
Apr 21, 2021

Conversation

davinchia
Copy link
Contributor

@davinchia davinchia commented Apr 20, 2021 •
edited
Loading

What

Release all connectors affected by namespace change. Includes all JDBC sources and destinations.

Also add documentation for normalisation. Prerequisite to actually releasing.

How

Bump all related connectors to 0.3.0

Pre-merge Checklist

  • Run integration tests
  • Publish Docker images

Recommended reading order

  1. namespaces.md
  2. various docker files and the source/destination definition list.

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/source-jdbc

🕑 connectors/source-jdbc https://github.com/airbytehq/airbyte/actions/runs/767072118
✅ connectors/source-jdbc https://github.com/airbytehq/airbyte/actions/runs/767072118

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/source-postgres

🕑 connectors/source-postgres https://github.com/airbytehq/airbyte/actions/runs/767134698
✅ connectors/source-postgres https://github.com/airbytehq/airbyte/actions/runs/767134698

@davinchia davinchia requested a review from cgardens April 20, 2021 13:26
@davinchia davinchia marked this pull request as ready for review April 20, 2021 13:27
@davinchia davinchia removed the request for review from michel-tricot April 20, 2021 13:27
@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/source-mssql

🕑 connectors/source-mssql https://github.com/airbytehq/airbyte/actions/runs/767289189
✅ connectors/source-mssql https://github.com/airbytehq/airbyte/actions/runs/767289189

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/source-mysql

🕑 connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/767290409
✅ connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/767290409

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/source-oracle

🕑 connectors/source-oracle https://github.com/airbytehq/airbyte/actions/runs/767290619
✅ connectors/source-oracle https://github.com/airbytehq/airbyte/actions/runs/767290619

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/source-redshift

🕑 connectors/source-redshift https://github.com/airbytehq/airbyte/actions/runs/767291141
✅ connectors/source-redshift https://github.com/airbytehq/airbyte/actions/runs/767291141

@jrhizor
Copy link
Contributor

jrhizor commented Apr 20, 2021 •
edited
Loading

This has the changes in #2961 for destination-postgres. If namespacing isn't in the release we still need to release a version of destination-postgres before release to pick up the changes.

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited
Loading

@jrhizor yeap. it will be in the release. I'll remove the acceptance test block and do the actual release in the follow up PR after we release the related postgres connectors in this one.

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/destination-postgres

🕑 connectors/destination-postgres https://github.com/airbytehq/airbyte/actions/runs/768907299
✅ connectors/destination-postgres https://github.com/airbytehq/airbyte/actions/runs/768907299

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/destination-jdbc

🕑 connectors/destination-jdbc https://github.com/airbytehq/airbyte/actions/runs/768907449
✅ connectors/destination-jdbc https://github.com/airbytehq/airbyte/actions/runs/768907449

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/destination-bigquery

🕑 connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/768907642
✅ connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/768907642

@davinchia
Copy link
Contributor Author

davinchia commented Apr 20, 2021 •
edited by github-actions bot
Loading

/test connector=connectors/destination-snowflake

🕑 connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/768907916
✅ connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/768907916

cgardens
cgardens approved these changes Apr 21, 2021
View reviewed changes
docs/architecture/namespaces.md Outdated Show resolved Hide resolved
@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-postgres

🕑 connectors/source-postgres https://github.com/airbytehq/airbyte/actions/runs/769117213
✅ connectors/source-postgres https://github.com/airbytehq/airbyte/actions/runs/769117213

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-mssql

🕑 connectors/source-mssql https://github.com/airbytehq/airbyte/actions/runs/769118114
✅ connectors/source-mssql https://github.com/airbytehq/airbyte/actions/runs/769118114

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-mysql

🕑 connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/769119505
✅ connectors/source-mysql https://github.com/airbytehq/airbyte/actions/runs/769119505

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-oracle

🕑 connectors/source-oracle https://github.com/airbytehq/airbyte/actions/runs/769119796
✅ connectors/source-oracle https://github.com/airbytehq/airbyte/actions/runs/769119796

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/source-redshift

🕑 connectors/source-redshift https://github.com/airbytehq/airbyte/actions/runs/769120111
✅ connectors/source-redshift https://github.com/airbytehq/airbyte/actions/runs/769120111

@davinchia @cgardens
Update docs/architecture/namespaces.md
d25d906 
Co-authored-by: Charles <giardina.charles@gmail.com>
@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/destination-postgres

🕑 connectors/destination-postgres https://github.com/airbytehq/airbyte/actions/runs/769169817
✅ connectors/destination-postgres https://github.com/airbytehq/airbyte/actions/runs/769169817

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/destination-bigquery

🕑 connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/769169913
✅ connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/769169913

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/destination-snowflake

🕑 connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/769170012
✅ connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/769170012

@davinchia
Copy link
Contributor Author

davinchia commented Apr 21, 2021 •
edited by github-actions bot
Loading

/publish connector=connectors/destination-redshift

🕑 connectors/destination-redshift https://github.com/airbytehq/airbyte/actions/runs/769171065
✅ connectors/destination-redshift https://github.com/airbytehq/airbyte/actions/runs/769171065

@davinchia davinchia merged commit 42686ad into master Apr 21, 2021
@davinchia davinchia deleted the davinchia/release-connectors-namespace branch April 21, 2021 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cgardens cgardens cgardens approved these changes

@ChristopheDuong ChristopheDuong Awaiting requested review from ChristopheDuong

Assignees
No one assigned
Labels
connectors/destination/bigquery connectors/destination/snowflake connectors/destinations-warehouse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@davinchia @jrhizor @cgardens @karinakuz