-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
🎉 New Source: Genesys [python cdk] #17559
🎉 New Source: Genesys [python cdk] #17559
Conversation
Thanks @YiyangLi ill get back to you soon witj the changes requested 👍 |
Integration Vetting Webhook-based? (no/partially/yes) is there an SDK available? Can we connect with HTTP requests directly? Creating an account Will it require some work through the UI? If so, what is required? Has Sandbox environment? Available streams for sync
Integration supports incremental sync? Other information/blockers |
airbyte-integrations/connectors/source-genesys/source_genesys/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-genesys/source_genesys/source.py
Outdated
Show resolved
Hide resolved
…yte into koconder-connector-genesys
@YiyangLi thanks for your review comments, I have pushed all my changes. Gradle results are also passing for intergration and acceptance tests:
|
/publish connector=connectors/source-genesys run-tests=false
if you have connectors that successfully published but failed definition generation, follow step 4 here |
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.
Resolved merge conflicts, ready to merge @marcosmarxm @sajarin @YiyangLi |
/test connector=connectors/source-genesys
Build FailedTest summary info:
|
What
This change adds an initial release of a source connector for the Genesys cloud platform.
How
The initial release of the connector extracts information about the following:
Closes: airbytehq/connector-contest#48
Recommended reading order
x.java
y.python
🚨 User Impact 🚨
Are there any breaking changes? What is the end result perceived by the user? If yes, please merge this PR with the 🚨🚨 emoji so changelog authors can further highlight this if needed.
Pre-merge Checklist
Expand the relevant checklist and delete the others.
New Connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampledocs/integrations/README.md
airbyte-integrations/builds.md
Airbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described hereUpdating a connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampleAirbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described hereConnector Generator
-scaffold
in their name) have been updated with the latest scaffold by running./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates
then checking in your changesTests
Unit
========================================= test session starts ==========================================
platform darwin -- Python 3.9.10, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- /Users/vincentkoc/Git/_PERSO/airbyte/.venv3/bin/python
cachedir: .pytest_cache
rootdir: /Users/vincentkoc/Git/_PERSO/airbyte, configfile: pytest.ini
plugins: mock-3.6.1
collected 10 items
unit_tests/test_source.py::test_check_connection PASSED
unit_tests/test_source.py::test_streams PASSED
unit_tests/test_streams.py::test_request_params PASSED
unit_tests/test_streams.py::test_request_headers PASSED
unit_tests/test_streams.py::test_http_method PASSED
unit_tests/test_streams.py::test_should_retry[HTTPStatus.OK-False] PASSED
unit_tests/test_streams.py::test_should_retry[HTTPStatus.BAD_REQUEST-False] PASSED
unit_tests/test_streams.py::test_should_retry[HTTPStatus.TOO_MANY_REQUESTS-True] PASSED
unit_tests/test_streams.py::test_should_retry[HTTPStatus.INTERNAL_SERVER_ERROR-True] PASSED
unit_tests/test_streams.py::test_backoff_time PASSED
=========================================== warnings summary ===========================================
airbyte-integrations/connectors/source-genesys/unit_tests/test_source.py: 16 warnings
airbyte-integrations/connectors/source-genesys/unit_tests/test_streams.py: 8 warnings
/Users/vincentkoc/Git/_PERSO/airbyte/.venv3/lib/python3.9/site-packages/airbyte_cdk/sources/streams/http/http.py:41: DeprecationWarning: Call to deprecated class NoAuth. (Set
authenticator=None
instead) -- Deprecated since version 0.1.20.self._authenticator: HttpAuthenticator = NoAuth()
airbyte-integrations/connectors/source-genesys/unit_tests/test_source.py: 16 warnings
airbyte-integrations/connectors/source-genesys/unit_tests/test_streams.py: 8 warnings
/Users/vincentkoc/Git/_PERSO/airbyte/.venv3/lib/python3.9/site-packages/deprecated/classic.py:173: DeprecationWarning: Call to deprecated class HttpAuthenticator. (Use requests.auth.AuthBase instead) -- Deprecated since version 0.1.20.
return old_new1(cls, *args, **kwargs)
-- Docs: https://docs.pytest.org/en/stable/warnings.html
=================================== 10 passed, 48 warnings in 4.79s ====================================
Integration
========================================= test session starts ==========================================
platform darwin -- Python 3.9.10, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- /Users/vincentkoc/Git/_PERSO/airbyte/.venv3/bin/python
cachedir: .pytest_cache
rootdir: /Users/vincentkoc/Git/_PERSO/airbyte, configfile: pytest.ini
plugins: mock-3.6.1
collected 0 items
======================================== no tests ran in 0.01s =========================================
Acceptance
Put your acceptance tests output here.