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

Publish source-facebook-marketing with stream status messages #25559

Merged

Conversation

jdpgrailsdev
Copy link
Contributor

What

  • Publish new version of the connector that supports stream status messages

How

  • Bump the CDK version used by the connector
  • Bump the version of the connector
  • Update the connector changelog

Recommended reading order

  1. setup.py
  2. Dockerfile
  3. facebook-marketing.md

🚨 User Impact 🚨

None

@jdpgrailsdev jdpgrailsdev requested a review from girarda April 26, 2023 14:55
@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/facebook-marketing labels Apr 26, 2023
@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 26, 2023

/test connector=connectors/source-facebook-marketing local_cdk=1

🕑 connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/4810182768
✅ connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/4810182768
Python tests coverage:

Name                                                        Stmts   Miss  Cover
-------------------------------------------------------------------------------
source_facebook_marketing/utils.py                             26      0   100%
source_facebook_marketing/streams/common.py                    44      0   100%
source_facebook_marketing/streams/async_job.py                217      0   100%
source_facebook_marketing/streams/__init__.py                   2      0   100%
source_facebook_marketing/spec.py                              42      0   100%
source_facebook_marketing/__init__.py                           5      0   100%
source_facebook_marketing/streams/async_job_manager.py         78      3    96%
source_facebook_marketing/streams/patches.py                   22      2    91%
source_facebook_marketing/api.py                               96      9    91%
source_facebook_marketing/source.py                            61      6    90%
source_facebook_marketing/streams/base_insight_streams.py     145     16    89%
source_facebook_marketing/streams/base_streams.py             143     22    85%
source_facebook_marketing/streams/streams.py                  125     29    77%
-------------------------------------------------------------------------------
TOTAL                                                        1006     87    91%
Name                                                        Stmts   Miss  Cover
-------------------------------------------------------------------------------
source_facebook_marketing/streams/__init__.py                   2      0   100%
source_facebook_marketing/__init__.py                           5      0   100%
source_facebook_marketing/spec.py                              42      1    98%
source_facebook_marketing/api.py                               96     12    88%
source_facebook_marketing/streams/patches.py                   22      3    86%
source_facebook_marketing/streams/base_streams.py             143     28    80%
source_facebook_marketing/streams/common.py                    44     16    64%
source_facebook_marketing/utils.py                             26     10    62%
source_facebook_marketing/source.py                            61     24    61%
source_facebook_marketing/streams/streams.py                  125     50    60%
source_facebook_marketing/streams/base_insight_streams.py     145     64    56%
source_facebook_marketing/streams/async_job.py                217    136    37%
source_facebook_marketing/streams/async_job_manager.py         78     60    23%
-------------------------------------------------------------------------------
TOTAL                                                        1006    404    60%

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 4116.55s (1:08:36) ==================

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 26, 2023

/publish connector=connectors/source-facebook-marketing

🕑 Publishing the following connectors:
connectors/source-facebook-marketing
https://github.com/airbytehq/airbyte/actions/runs/4812288870


Connector Version Did it publish? Were definitions generated?
connectors/source-facebook-marketing 0.3.5

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

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 26, 2023

/publish connector=connectors/source-facebook-marketing

🕑 Publishing the following connectors:
connectors/source-facebook-marketing
https://github.com/airbytehq/airbyte/actions/runs/4813323635


Connector Version Did it publish? Were definitions generated?
connectors/source-facebook-marketing 0.3.5

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

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 27, 2023

/publish connector=connectors/source-facebook-marketing

🕑 Publishing the following connectors:
connectors/source-facebook-marketing
https://github.com/airbytehq/airbyte/actions/runs/4820294182


Connector Version Did it publish? Were definitions generated?
connectors/source-facebook-marketing 0.3.5

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

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 27, 2023

/test connector=connectors/source-facebook-marketing

🕑 connectors/source-facebook-marketing https://github.com/airbytehq/airbyte/actions/runs/4820727133

@bazarnov
Copy link
Collaborator

@jdpgrailsdev We are currently off the rate limits for this connector, please hold for some time, until it's restored. Thank you.

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 27, 2023

/publish connector=connectors/source-facebook-marketing --run-tests=false

@jdpgrailsdev We are currently off the rate limits for this connector, please hold for some time, until it's restored. Thank you.

@bazarnov Are you asking me to wait to publish the connector? If so, do you know what the ETA on when I would be able to do that?

Error: Unexpected inputs provided: ["--run-tests"]

@bazarnov
Copy link
Collaborator

/publish connector=connectors/source-facebook-marketing --run-tests=false

@jdpgrailsdev We are currently off the rate limits for this connector, please hold for some time, until it's restored. Thank you.

@bazarnov Are you asking me to wait to publish the connector? If so, do you know what the ETA on when I would be able to do that?

Error: Unexpected inputs provided: ["--run-tests"]

Without the tests - you can, otherwise, you're blocked, no ETA when the limits will be restored, probably a couple of hours o so.

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 27, 2023

/publish connector=connectors/source-facebook-marketing --run-tests=false

Error: Unexpected inputs provided: ["--run-tests"]

@jdpgrailsdev
Copy link
Contributor Author

jdpgrailsdev commented Apr 27, 2023

/publish connector=connectors/source-facebook-marketing run-tests=false

🕑 Publishing the following connectors:
connectors/source-facebook-marketing
https://github.com/airbytehq/airbyte/actions/runs/4821141643


Connector Version Did it publish? Were definitions generated?
connectors/source-facebook-marketing 0.3.5

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

@jdpgrailsdev
Copy link
Contributor Author

/approve-and-merge reason="PR approved by Alex"

@octavia-approvington
Copy link
Contributor

It's time
fine lets go

@octavia-approvington octavia-approvington merged commit 8a01d05 into master Apr 27, 2023
@octavia-approvington octavia-approvington deleted the jonathan/publish-source-facebook-marketing branch April 27, 2023 15:33
marcosmarxm pushed a commit to natalia-miinto/airbyte that referenced this pull request Jun 8, 2023
…ehq#25559)

* Prepare to publish source-facebook-marketing

* Bump required CDK version

* 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
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/facebook-marketing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants