Skip to content

destination-snowflake: Use airbyte/java-connector-base:1.0.0 - #49913

Merged
Augustin (alafanechere) merged 4 commits into
masterfrom
destination-snowflake/base-image-migration
Jan 9, 2025
Merged

Augustin (alafanechere) merged 4 commits into
masterfrom
destination-snowflake/base-image-migration

Conversation

@alafanechere

Copy link
Copy Markdown
Contributor

Make the connector use our official base image for java connectors

@alafanechere
Augustin (alafanechere) requested a review from a team as a code owner December 18, 2024 14:58
@vercel

vercel Bot commented Dec 18, 2024 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 7, 2025 2:08pm

@alafanechere
Augustin (alafanechere) force-pushed the destination-snowflake/base-image-migration branch from 6cd7eed to 8f40776 Compare December 18, 2024 14:58
@alafanechere

Augustin (alafanechere) commented Dec 18, 2024 •

Copy link
Copy Markdown
Contributor Author

/format-fix

Format-fix job started... Check job output.

✅ Changes applied successfully. (3af0285)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Augustin (@alafanechere) why is this file changing so much if the main changeset is the base image?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's because the addition of the base image address was automated and serializing the metadata file back to yaml changed the original order. Nothing is really changing except the connectorBuildOptions. It's indeed tricky to review and trust. I'll make the change manually for certified connectors.

@alafanechere
Augustin (alafanechere) merged commit 7ddc6d6 into master Jan 9, 2025
@alafanechere
Augustin (alafanechere) deleted the destination-snowflake/base-image-migration branch January 9, 2025 15:18

This branch was successfully deployed

1 active deployment
Preview — 5343c721 Deployed Jan 7, 2025 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation base-image-migration connectors/destination/snowflake

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants