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 Hubspot, Gitlab, Salesloft, Pinterest: fix builds #26188

Merged
merged 17 commits into from May 17, 2023

Conversation

davydov-d
Copy link
Collaborator

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

What

Fix builds for:

  • Source Hubspot. Expected and actual records mismatch of the Deals stream - new property added - hs_closed_won_count, value of the property hs_time_in_appointmentscheduled updated.
  • Source Gitlab. Expected and actual records mismatch of the Jobs stream - value of the property runner_id updated.
  • Source Salesloft. Expected and actual records mismatch of the Cadences stream - new property added - latest_active_date.
  • Source Pinterest. Expected and actual records mismatch of the BoardPins stream - new property added - product_tags.

How

  • Source Hubspot: update expected records. No need to update stream schema as it is generated dynamically.
  • Source Gitlab: update expected records.
  • Source Salesloft: udate expected records, update stream schema, bump version, update changelog.
  • Source Pinterest: udate expected records, update stream schema, bump version, update changelog.

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/test connector=connectors/source-gitlab

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

Name                        Stmts   Miss  Cover
-----------------------------------------------
source_gitlab/__init__.py       2      0   100%
source_gitlab/source.py        61      3    95%
source_gitlab/streams.py      279     15    95%
-----------------------------------------------
TOTAL                         342     18    95%

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.
================== 43 passed, 2 skipped in 571.04s (0:09:31) ===================

@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels May 17, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 17, 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!

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
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/test connector=connectors/source-hubspot

🕑 connectors/source-hubspot https://github.com/airbytehq/airbyte/actions/runs/5003877980
❌ connectors/source-hubspot https://github.com/airbytehq/airbyte/actions/runs/5003877980
🐛 https://gradle.com/s/khzq7f74x52hg

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, 39 passed, 2 skipped in 439.27s (0:07:19) ==============

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/test connector=connectors/source-salesloft

🕑 connectors/source-salesloft https://github.com/airbytehq/airbyte/actions/runs/5003878953
❌ connectors/source-salesloft https://github.com/airbytehq/airbyte/actions/runs/5003878953
🐛 https://gradle.com/s/igbyusgy7hwu4

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - Failed: Please check...
FAILED test_core.py::TestBasicRead::test_read[inputs1] - Failed: Please check...
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
============= 2 failed, 49 passed, 1 skipped in 172.42s (0:02:52) ==============

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/test connector=connectors/source-pinterest

🕑 connectors/source-pinterest https://github.com/airbytehq/airbyte/actions/runs/5003879618
✅ connectors/source-pinterest https://github.com/airbytehq/airbyte/actions/runs/5003879618
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 216.99s (0:03:36) ===================

@davydov-d davydov-d changed the title Ddavydov/connector health 2023 05 17 🐛 Source Hubspot, Gitlab, Salesloft, Pinterest: fix builds May 17, 2023
@davydov-d
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/test connector=connectors/source-salesloft

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

Name                           Stmts   Miss  Cover
--------------------------------------------------
source_salesloft/__init__.py       2      0   100%
source_salesloft/source.py       158      2    99%
--------------------------------------------------
TOTAL                            160      2    99%

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.
================== 51 passed, 1 skipped in 170.69s (0:02:50) ===================

@davydov-d
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/test connector=connectors/source-hubspot

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

Name                          Stmts   Miss  Cover
-------------------------------------------------
source_hubspot/errors.py         10      0   100%
source_hubspot/constants.py       2      0   100%
source_hubspot/__init__.py        2      0   100%
source_hubspot/helpers.py        70      3    96%
source_hubspot/streams.py       897     77    91%
source_hubspot/source.py         60     19    68%
-------------------------------------------------
TOTAL                          1041     99    90%
Name                          Stmts   Miss  Cover
-------------------------------------------------
source_hubspot/constants.py       2      0   100%
source_hubspot/__init__.py        2      0   100%
source_hubspot/helpers.py        70      7    90%
source_hubspot/errors.py         10      1    90%
source_hubspot/source.py         60     14    77%
source_hubspot/streams.py       897    217    76%
-------------------------------------------------
TOTAL                          1041    239    77%

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.
================== 40 passed, 2 skipped in 406.60s (0:06:46) ===================

@davydov-d davydov-d marked this pull request as ready for review May 17, 2023 14:30
@davydov-d
Copy link
Collaborator Author

davydov-d commented May 17, 2023

/publish connector=connectors/source-pinterest

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


| Connector | Version | Did it publish? |
| --- | --- | --- | --- |
| connectors/source-pinterest | 0.5.0 | ✅ |

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 17, 2023

/publish connector=connectors/source-salesloft

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

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

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

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

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 17, 2023

/publish connector=connectors/source-salesloft

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


| Connector | Version | Did it publish? |
| --- | --- | --- | --- |
| connectors/source-salesloft | 1.1.0 | ✅ |

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

@davydov-d davydov-d merged commit 5575d86 into master May 17, 2023
23 of 26 checks passed
@davydov-d davydov-d deleted the ddavydov/connector-health-2023-05-17 branch May 17, 2023 17:03
sajarin pushed a commit that referenced this pull request May 17, 2023
* Connector health: source hubspot, gitlab, snapchat-marketing: fix builds

* connector health: fix builds for gitlab, salesloft, pinterest, hubspot

* conenctor health: update changelog

* Automated Change

* connector health - fix prop type

* connector health: upd expected records

* Automated Change

* empty commit to trigger ci

---------

Co-authored-by: davydov-d <davydov-d@users.noreply.github.com>
marcosmarxm pushed a commit to natalia-miinto/airbyte that referenced this pull request Jun 8, 2023
…#26188)

* Connector health: source hubspot, gitlab, snapchat-marketing: fix builds

* connector health: fix builds for gitlab, salesloft, pinterest, hubspot

* conenctor health: update changelog

* Automated Change

* connector health - fix prop type

* connector health: upd expected records

* Automated Change

* empty commit to trigger ci

---------

Co-authored-by: davydov-d <davydov-d@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

3 participants