-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
✨ Source Pipedrive : Migrate Python CDK to Low-code CDK #29293
Changes from all commits
43715e1
5cf197f
9be5217
15e6dcd
880f28d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,16 +1,38 @@ | ||
FROM python:3.9-slim | ||
FROM python:3.9.11-alpine3.15 as base | ||
|
||
# build and load all requirements | ||
FROM base as builder | ||
WORKDIR /airbyte/integration_code | ||
|
||
# upgrade pip to the latest version | ||
RUN apk --no-cache upgrade \ | ||
&& pip install --upgrade pip \ | ||
&& apk --no-cache add tzdata build-base | ||
|
||
# Bash is installed for more convenient debugging. | ||
RUN apt-get update && apt-get install -y bash && rm -rf /var/lib/apt/lists/* | ||
|
||
COPY setup.py ./ | ||
# install necessary packages to a temporary folder | ||
RUN pip install --prefix=/install . | ||
|
||
# build a clean environment | ||
FROM base | ||
WORKDIR /airbyte/integration_code | ||
COPY source_pipedrive ./source_pipedrive | ||
|
||
# copy all loaded and built libraries to a pure basic image | ||
COPY --from=builder /install /usr/local | ||
# add default timezone settings | ||
COPY --from=builder /usr/share/zoneinfo/Etc/UTC /etc/localtime | ||
RUN echo "Etc/UTC" > /etc/timezone | ||
|
||
# bash is installed for more convenient debugging. | ||
RUN apk --no-cache add bash | ||
|
||
# copy payload code only | ||
COPY main.py ./ | ||
COPY setup.py ./ | ||
RUN pip install . | ||
COPY source_pipedrive ./source_pipedrive | ||
|
||
ENV AIRBYTE_ENTRYPOINT "python /airbyte/integration_code/main.py" | ||
ENTRYPOINT ["python", "/airbyte/integration_code/main.py"] | ||
|
||
LABEL io.airbyte.version=1.0.0 | ||
LABEL io.airbyte.version=2.0.0 | ||
LABEL io.airbyte.name=airbyte/source-pipedrive |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,3 @@ | ||
# | ||
# Copyright (c) 2023 Airbyte, Inc., all rights reserved. | ||
# | ||
|
||
|
||
def test_example(): | ||
assert True |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -1,72 +1,45 @@ | ||||||||||||||||||||||
# See [Connector Acceptance Tests](https://docs.airbyte.com/connector-development/testing-connectors/connector-acceptance-tests-reference) | ||||||||||||||||||||||
# for more information about how to configure these tests | ||||||||||||||||||||||
connector_image: airbyte/source-pipedrive:dev | ||||||||||||||||||||||
test_strictness_level: "high" | ||||||||||||||||||||||
acceptance_tests: | ||||||||||||||||||||||
spec: | ||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- spec_path: "source_pipedrive/spec.json" | ||||||||||||||||||||||
- spec_path: "source_pipedrive/spec.yaml" | ||||||||||||||||||||||
backward_compatibility_tests_config: | ||||||||||||||||||||||
disable_for_version: 1.0.0 | ||||||||||||||||||||||
connection: | ||||||||||||||||||||||
Comment on lines
+8
to
10
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please keep the same config as before. |
||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- config_path: "secrets/config.json" | ||||||||||||||||||||||
status: "succeed" | ||||||||||||||||||||||
- config_path: "secrets/old_config.json" | ||||||||||||||||||||||
status: "succeed" | ||||||||||||||||||||||
- config_path: "integration_tests/invalid_config.json" | ||||||||||||||||||||||
status: "failed" | ||||||||||||||||||||||
discovery: | ||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- config_path: "secrets/config.json" | ||||||||||||||||||||||
backward_compatibility_tests_config: | ||||||||||||||||||||||
disable_for_version: 0.1.19 | ||||||||||||||||||||||
disable_for_version: 1.0.0 | ||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There are not reason to change to breaking change. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Due to schema changes, I have done There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It shouldn't be the case. For migrating a Beta/GA the schema should be broke right now. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should I remove disable for version There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, please remove those lines. |
||||||||||||||||||||||
basic_read: | ||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- config_path: "secrets/config.json" | ||||||||||||||||||||||
expect_records: | ||||||||||||||||||||||
path: "integration_tests/expected_records.jsonl" | ||||||||||||||||||||||
ignored_fields: | ||||||||||||||||||||||
users: | ||||||||||||||||||||||
- name: modified | ||||||||||||||||||||||
bypass_reason: "constantly increasing date-time field" | ||||||||||||||||||||||
Comment on lines
-21
to
-28
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please keep the same. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. expected_records.jsonl this file was old that's why I removed it and it will lead to failures There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @avirajsingh7 that files is used to compare records when running integration tests. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So for that I need to fix expected_records.jsonl file also according to latest schemas or I can keep old one? |
||||||||||||||||||||||
- name: last_login | ||||||||||||||||||||||
bypass_reason: "constantly increasing date-time field" | ||||||||||||||||||||||
deal_fields: | ||||||||||||||||||||||
- name: show_in_pipelines | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: important_flag | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: pipeline_ids | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: update_time | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: last_updated_by_user_id | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
organization_fields: | ||||||||||||||||||||||
- name: update_time | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: important_flag | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: last_updated_by_user_id | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
person_fields: | ||||||||||||||||||||||
- name: update_time | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: important_flag | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: last_updated_by_user_id | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
product_fields: | ||||||||||||||||||||||
- name: update_time | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: important_flag | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
- name: last_updated_by_user_id | ||||||||||||||||||||||
bypass_reason: "Unstable data" | ||||||||||||||||||||||
fail_on_extra_columns: false | ||||||||||||||||||||||
incremental: | ||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- config_path: "secrets/config.json" | ||||||||||||||||||||||
configured_catalog_path: "integration_tests/configured_catalog.json" | ||||||||||||||||||||||
future_state: | ||||||||||||||||||||||
future_state_path: "integration_tests/abnormal_state.json" | ||||||||||||||||||||||
fail_on_extra_columns: false | ||||||||||||||||||||||
empty_streams: | ||||||||||||||||||||||
- name: files | ||||||||||||||||||||||
- name: filters | ||||||||||||||||||||||
- name: leads | ||||||||||||||||||||||
- name: notes | ||||||||||||||||||||||
- name: activities | ||||||||||||||||||||||
- name: pipelines | ||||||||||||||||||||||
- name: products | ||||||||||||||||||||||
- name: stages | ||||||||||||||||||||||
- name: deal_products | ||||||||||||||||||||||
|
||||||||||||||||||||||
incremental: | ||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- config_path: "secrets/config.json" | ||||||||||||||||||||||
configured_catalog_path: "integration_tests/configured_catalog.json" | ||||||||||||||||||||||
future_state: | ||||||||||||||||||||||
future_state_path: "integration_tests/abnormal_state.json" | ||||||||||||||||||||||
Comment on lines
+38
to
+42
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||
full_refresh: | ||||||||||||||||||||||
tests: | ||||||||||||||||||||||
- config_path: "secrets/config.json" | ||||||||||||||||||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,3 @@ | ||
#!/usr/bin/env sh | ||
|
||
source "$(git rev-parse --show-toplevel)/airbyte-integrations/bases/connector-acceptance-test/acceptance-test-docker.sh" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
# | ||
# Copyright (c) 2023 Airbyte, Inc., all rights reserved. | ||
# |
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.