-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
remove Dockerfile and .dockerignore for Java connectors #31769
remove Dockerfile and .dockerignore for Java connectors #31769
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Current dependencies on/for this PR: This comment was auto-generated by Graphite. |
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
@@ -16,6 +16,7 @@ Note that the `secrets` directory is git-ignored by default, so there is no dang | |||
|
|||
#### Build | |||
Build the connector image via Gradle: | |||
You need a local install of [airbyte-ci](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md) to run this command. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't. Gradle will provision a python venv and do the poetry song-and-dance to install airbyte-ci from source in the .venv folder in the repo root.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes, I forgot your wizardries!
/approve-and-merge reason="Deleting unused files + can't realistically run the CI on all the modified connectors" |
What
Closes #30708
A clumsy merge reintroduced some java connnectors Dockerfile.
And we also forgot to remove
.dockerignore
filesHow
Remove
Dockerfile
and.dockerignore
files for Java connectors.Replace
airbyteDocker
in README withbuildConnectorImage
Java connectors docker images are 💯 built via
airbyte-ci
and don't require Dockerfile anymore.