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.

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 Mixpanel, Source Pinterest, Source Freshdesk: fix builds #25915

Merged
merged 10 commits into from
May 11, 2023

Conversation

davydov-d
Copy link
Collaborator

@davydov-d davydov-d commented May 9, 2023

What

Fix builds for:

  • Source Mixpanel. Expected Records Mismatch: data went missing in one of the streams - Export.
  • Source Pipedrive. Expected Records Mismatch: stream emits records that contain extra fields (neither exposed in schema nor documented in the api reference).
  • Source Freshdesk. Expected Records Mismatch: stream TicketFields - new item added to the choices field in one of the records: "Chat - Internal Task": 14
  • Source Pinterest. Expected Records Mismatch: the schema used to be missing a field. Stream BoardPins now has one more field: creative_type

How

  • Mixpanel: refill the stream and update expected records
  • Pipedrive: Do not emit undocumented fields as part of a record
  • Freshdesk: update expected records
  • Pinterest: update expected records + expose new field in the stream schema, bump version

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2023

Before Merging a Connector Pull Request

Wow! 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:

  • PR name follows PR naming conventions
  • Breaking changes are considered. If a Breaking Change is being introduced, ensure an Airbyte engineer has created a Breaking Change Plan and you've followed all steps in the Breaking Changes Checklist
  • Connector version has been incremented in the Dockerfile and metadata.yaml according to our Semantic Versioning for Connectors guidelines
  • Secrets in the connector's spec are annotated with airbyte_secret
  • All documentation files are up to date. (README.md, bootstrap.md, docs.md, etc...)
  • Changelog updated in docs/integrations/<source or destination>/<name>.md with an entry for the new version. See changelog example
  • You, or an Airbyter, have run /test successfully on this PR - or on a non-forked branch
  • You, or an Airbyter, have run /publish successfully on this PR - or on a non-forked branch
  • You've updated the connector's metadata.yaml file (new!)
  • The Octavia bot updated the source_definitions.yaml or destination_definitions.yaml, or you ran processResources manually (deprecated)

If the checklist is complete, but the CI check is failing,

  1. Check for hidden checklists in your PR description

  2. Toggle the github label checklist-action-run on/off to re-run the checklist CI.

@davydov-d davydov-d changed the title #connector health: fix builds for pipedrive, mixpanel, freshdesk, pin… 🐛 Source Mixpanel, Source Pipedrive, Source Pinterest, Source Freshdesk: fix builds May 9, 2023
@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-freshdesk

🕑 connectors/source-freshdesk https://github.com/airbytehq/airbyte/actions/runs/4925927190
✅ connectors/source-freshdesk https://github.com/airbytehq/airbyte/actions/runs/4925927190
Python tests coverage:

Name                                        Stmts   Miss  Cover
---------------------------------------------------------------
source_freshdesk/source.py                     22      0   100%
source_freshdesk/availability_strategy.py      10      0   100%
source_freshdesk/__init__.py                    2      0   100%
source_freshdesk/streams.py                   224      4    98%
source_freshdesk/utils.py                      20      1    95%
---------------------------------------------------------------
TOTAL                                         278      5    98%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:578: The previous and actual discovered catalogs are identical.
================== 38 passed, 2 skipped in 508.18s (0:08:28) ===================

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-mixpanel

🕑 connectors/source-mixpanel https://github.com/airbytehq/airbyte/actions/runs/4925928108

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-pipedrive

🕑 connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4925927980
❌ connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4925927980
🐛 https://gradle.com/s/vh6x5tgod2rgu

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contai...
FAILED test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
FAILED test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:578: The previous and actual discovered catalogs are identical.
=================== 5 failed, 34 passed, 2 skipped in 49.36s ===================

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-pinterest

🕑 connectors/source-pinterest https://github.com/airbytehq/airbyte/actions/runs/4925928928
✅ connectors/source-pinterest https://github.com/airbytehq/airbyte/actions/runs/4925928928
Python tests coverage:

Name                           Stmts   Miss  Cover
--------------------------------------------------
source_pinterest/utils.py          8      0   100%
source_pinterest/__init__.py       2      0   100%
source_pinterest/source.py       224     17    92%
--------------------------------------------------
TOTAL                            234     17    93%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
================== 49 passed, 1 skipped in 215.28s (0:03:35) ===================

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-pipedrive

🕑 connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4926723395
❌ connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4926723395
🐛 https://gradle.com/s/jop5lyvmrcdys

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - Failed: Stream activ...
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:578: The previous and actual discovered catalogs are identical.
============= 1 failed, 38 passed, 2 skipped in 133.90s (0:02:13) ==============

@davydov-d davydov-d requested a review from erohmensing May 9, 2023 14:45
@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-pipedrive

🕑 connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4927228400
❌ connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4927228400
🐛 https://gradle.com/s/pdgryr4fhigv2

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - Failed: Stream activ...
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:578: The previous and actual discovered catalogs are identical.
============= 1 failed, 38 passed, 2 skipped in 138.81s (0:02:18) ==============

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 9, 2023

/test connector=connectors/source-pipedrive

🕑 connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4928132377
❌ connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4928132377
🐛 https://gradle.com/s/aszxt6ygwataq

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - Failed: Stream deals...
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:578: The previous and actual discovered catalogs are identical.
============= 1 failed, 38 passed, 2 skipped in 134.64s (0:02:14) ==============

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 10, 2023

/test connector=connectors/source-pipedrive

🕑 connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4934569994
✅ connectors/source-pipedrive https://github.com/airbytehq/airbyte/actions/runs/4934569994
Python tests coverage:

Name                           Stmts   Miss  Cover
--------------------------------------------------
source_pipedrive/auth.py           8      0   100%
source_pipedrive/__init__.py       2      0   100%
source_pipedrive/source.py        38      2    95%
source_pipedrive/streams.py      101     15    85%
--------------------------------------------------
TOTAL                            149     17    89%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:578: The previous and actual discovered catalogs are identical.
================== 39 passed, 2 skipped in 220.30s (0:03:40) ===================

@davydov-d
Copy link
Collaborator Author

@erohmensing I've excluded Pipedrive changes from this PR as we talked before

@erohmensing
Copy link
Contributor

Thanks! Other changes look good

@erohmensing erohmensing changed the title 🐛 Source Mixpanel, Source Pipedrive, Source Pinterest, Source Freshdesk: fix builds 🐛 Source Mixpanel, Source Pinterest, Source Freshdesk: fix builds May 10, 2023
@davydov-d
Copy link
Collaborator Author

davydov-d commented May 11, 2023

/publish connector=connectors/pinterest

🕑 Publishing the following connectors:
connectors/pinterest
https://github.com/airbytehq/airbyte/actions/runs/4943897693


Connector Version Did it publish? Were definitions generated?
connectors/pinterest

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 11, 2023

/publish connector=connectors/source-pinterest

🕑 Publishing the following connectors:
connectors/source-pinterest
https://github.com/airbytehq/airbyte/actions/runs/4943941320


Connector Version Did it publish? Were definitions generated?
connectors/source-pinterest 0.3.0

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@davydov-d davydov-d merged commit ac154b1 into master May 11, 2023
@davydov-d davydov-d deleted the ddavydov/connector-health branch May 11, 2023 05:17
marcosmarxm pushed a commit to natalia-miinto/airbyte that referenced this pull request Jun 8, 2023
…rbytehq#25915)

* #connector health: fix builds for pipedrive, mixpanel, freshdesk, pinterest

* Source Pipedrive: another fix

* Source Pipedrive: another fix

* Source Pipedrive: another fix

* Source Pipedrive: update expected records

* connector health: upd changelogs

* connector health: exclude pipedrive changes for now

* connector health: exclude pipedrive changes for now

* connector health: exclude pipedrive changes for now

* auto-bump connector version

---------

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants