-
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
Zendesk airbyte entrypoint #4469
Conversation
/test connector=connectors/source-zendesk-support-singer
|
@@ -5,11 +5,12 @@ RUN apt-get update && apt-get install -y bash && rm -rf /var/lib/apt/lists/* | |||
ENV CODE_PATH="source_zendesk_support_singer" | |||
ENV AIRBYTE_IMPL_MODULE="source_zendesk_support_singer" | |||
ENV AIRBYTE_IMPL_PATH="SourceZendeskSupportSinger" | |||
ENV AIRBYTE_ENTRYPOINT "python /airbyte/integration_code/source.py" |
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.
ENV AIRBYTE_ENTRYPOINT "python /airbyte/integration_code/source.py" | |
ENV AIRBYTE_ENTRYPOINT "/airbyte/base.sh" |
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.
docker inspect airbyte/source-zendesk-support-singer:0.2.2
shows /airbyte/base.sh
as the existing entrypoint.
/test connector=connectors/source-zendesk-support-singer
|
looks like this probably has to be pushed with skipping tests with |
/publish connector=connectors/source-zendesk-support-singer run-tests=false
|
I dont understand the CI error :( @sherifnada any idea why CI is failing to publish zendesk support and is calling zoom? |
not sure about zoom but previous failed runs of this workflow have been because the schema of the configured catalog used in testing is out of date, requires destination_sync_mode |
/test connector=connectors/source-zendesk-support-singer
|
/test connector=connectors/source-zendesk-support-singer
|
/publish connector=connectors/source-zendesk-support-singer
|
/publish connector=connectors/source-zendesk-support-singer
|
/publish connector=connectors/source-zendesk-support-singer
|
/publish connector=connectors/source-zendesk-support-singer
|
/publish connector=connectors/source-zendesk-support-singer
|
What
Describe what the change is solving
It helps to add screenshots if it affects the frontend.
How
Describe the solution
Recommended reading order
x.java
y.python
Pre-merge Checklist
Expand the checklist which is relevant for this PR.
Connector checklist
airbyte_secret
in the connector's spec./gradlew :airbyte-integrations:connectors:<name>:integrationTest
./test connector=connectors/<name>
command as documented here is passing.README.md
docs/SUMMARY.md
if it's a new connectordocs/integrations/<source or destination>/<name>
.docs/integrations/...
. See changelog exampledocs/integrations/README.md
contains a reference to the new connector/publish
command described hereConnector Generator checklist
-scaffold
in their name) have been updated with the latest scaffold by running./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates
then checking in your changes