Source Slack: retry timeout error #21321

artem1205 · 2023-01-12T15:05:34Z

What

Resolving https://github.com/airbytehq/oncall/issues/1292

How

retry timeout error

Pre-merge Checklist

Expand the relevant checklist and delete the others.

Updating a connector

Community member or Airbyter

Grant edit access to maintainers (instructions)
Secrets in the connector's spec are annotated with airbyte_secret
Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
Code reviews completed
Documentation updated
- Connector's README.md
- Connector's bootstrap.md. See description and examples
- Changelog updated in docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
PR name follows PR naming conventions

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

Create a non-forked branch based on this PR and test the below items on it
Build is successful
If new credentials are required for use in CI, add them to GSM. Instructions.
/test connector=connectors/<name> command is passing
New Connector version released on Dockerhub and connector version bumped by running the /publish command described here

artem1205 · 2023-01-12T15:06:46Z

/test connector=connectors/source-slack

🕑 connectors/source-slack https://github.com/airbytehq/airbyte/actions/runs/3903483885
❌ connectors/source-slack https://github.com/airbytehq/airbyte/actions/runs/3903483885
🐛 https://gradle.com/s/zgatzavbd2qdg

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]
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/tests/test_core.py:94: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/tests/test_core.py:377: The previous and actual discovered catalogs are identical.
====== 2 failed, 29 passed, 2 skipped, 31 warnings in 1052.52s (0:17:32) =======

> Task :airbyte-integrations:connectors:source-slack:sourceAcceptanceTest FAILED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.6/userguide/command_line_interface.html#sec:command_line_warnings

Execution optimizations have been disabled for 1 invalid unit(s) of work during this build to ensure correctness.
Please consult deprecation warnings for more details.
48 actionable tasks: 23 executed, 25 up-to-date

Publishing build scan...
https://gradle.com/s/zgatzavbd2qdg

The remote build cache was disabled during the build due to errors.
S3 cache writes: 1, elapsed: 676ms, sent to cache: 450 B

artem1205 · 2023-01-12T18:36:07Z

/test connector=connectors/source-slack

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

Name                       Stmts   Miss  Cover
----------------------------------------------
source_slack/__init__.py       2      0   100%
source_slack/source.py       218     18    92%
----------------------------------------------
TOTAL                        220     18    92%
	 Name                                                 Stmts   Miss  Cover   Missing
	 ----------------------------------------------------------------------------------
	 source_acceptance_test/base.py                          12      4    67%   16-19
	 source_acceptance_test/config.py                       141      5    96%   87, 93, 239, 243-244
	 source_acceptance_test/conftest.py                     211     95    55%   36, 42-44, 49, 54, 77, 83, 89-91, 110, 115-117, 123-125, 131-132, 137-138, 143, 149, 158-167, 173-178, 193, 217, 248, 254, 262-267, 275-285, 293-306, 311-317, 324-335, 342-358
	 source_acceptance_test/plugin.py                        69     25    64%   22-23, 31, 36, 120-140, 144-148
	 source_acceptance_test/tests/test_core.py              402    115    71%   53, 58, 93-104, 109-116, 120-121, 125-126, 308, 346-363, 376-387, 391-396, 402, 435-440, 478-485, 528-530, 533, 598-606, 618-621, 626, 682-683, 689, 692, 728-738, 751-776
	 source_acceptance_test/tests/test_incremental.py       160     14    91%   56-63, 68-81, 244
	 source_acceptance_test/utils/asserts.py                 39      2    95%   62-63
	 source_acceptance_test/utils/common.py                  94     10    89%   16-17, 32-38, 72, 75
	 source_acceptance_test/utils/compare.py                 62     23    63%   21-51, 68, 97-99
	 source_acceptance_test/utils/connector_runner.py       133     33    75%   24-27, 46-47, 50-54, 57-58, 73-75, 78-80, 83-85, 88-90, 93-95, 124-125, 159-161, 208
	 source_acceptance_test/utils/json_schema_helper.py     107     13    88%   30-31, 38, 41, 65-68, 96, 120, 192-194
	 ----------------------------------------------------------------------------------
	 TOTAL                                                 1609    339    79%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/tests/test_core.py:94: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/source_acceptance_test/tests/test_core.py:377: The previous and actual discovered catalogs are identical.
=========== 31 passed, 2 skipped, 31 warnings in 1535.31s (0:25:35) ============

artem1205 · 2023-01-13T11:21:24Z

/publish connector=connectors/source-slack

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

Connector	Did it publish?	Were definitions generated?
connectors/source-slack	❌	❌

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

artem1205 · 2023-01-13T13:12:35Z

/publish connector=connectors/source-slack

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

Connector	Did it publish?	Were definitions generated?
connectors/source-slack	✅	✅

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

* Source Slack: retry timeout error * Source Slack: update docs * Source Slack: update retry * auto-bump connector version Co-authored-by: Octavia Squidington III <[email protected]>

Source Slack: retry timeout error

3896b83

octavia-squidington-iv added area/connectors Connector related issues connectors/source/slack labels Jan 12, 2023

Source Slack: update docs

b2f9b1e

octavia-squidington-iv added the area/documentation Improvements or additions to documentation label Jan 12, 2023

Source Slack: update retry

44d7b3f

artem1205 requested review from arsenlosenko and bazarnov January 12, 2023 20:28

bazarnov approved these changes Jan 13, 2023

View reviewed changes

arsenlosenko approved these changes Jan 13, 2023

View reviewed changes

auto-bump connector version

dd97456

octavia-squidington-iii temporarily deployed to more-secrets January 13, 2023 14:46 — with GitHub Actions Inactive

octavia-squidington-iii temporarily deployed to more-secrets January 13, 2023 16:10 — with GitHub Actions Inactive

artem1205 enabled auto-merge (squash) January 13, 2023 16:31

Merge branch 'master' into artem1205/source-slack-OC-1292

3eb6cc9

artem1205 temporarily deployed to more-secrets January 13, 2023 16:52 — with GitHub Actions Inactive

artem1205 temporarily deployed to more-secrets January 13, 2023 16:53 — with GitHub Actions Inactive

artem1205 merged commit 19b5375 into master Jan 13, 2023

artem1205 deleted the artem1205/source-slack-OC-1292 branch January 13, 2023 17:32

octavia-squidington-iii mentioned this pull request Jan 24, 2023

Bump Airbyte version from 0.40.28 to 0.40.29 #21767

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Source Slack: retry timeout error #21321

Source Slack: retry timeout error #21321

artem1205 commented Jan 12, 2023

artem1205 commented Jan 12, 2023 •

edited by github-actions bot

Loading

artem1205 commented Jan 12, 2023 •

edited by github-actions bot

Loading

artem1205 commented Jan 13, 2023 •

edited by github-actions bot

Loading

artem1205 commented Jan 13, 2023 •

edited by github-actions bot

Loading

Source Slack: retry timeout error #21321

Source Slack: retry timeout error #21321

Conversation

artem1205 commented Jan 12, 2023

What

How

Recommended reading order

Pre-merge Checklist

Community member or Airbyter

Airbyter

artem1205 commented Jan 12, 2023 • edited by github-actions bot Loading

Build Failed

artem1205 commented Jan 12, 2023 • edited by github-actions bot Loading

Build Passed

artem1205 commented Jan 13, 2023 • edited by github-actions bot Loading

artem1205 commented Jan 13, 2023 • edited by github-actions bot Loading

artem1205 commented Jan 12, 2023 •

edited by github-actions bot

Loading

artem1205 commented Jan 12, 2023 •

edited by github-actions bot

Loading

artem1205 commented Jan 13, 2023 •

edited by github-actions bot

Loading

artem1205 commented Jan 13, 2023 •

edited by github-actions bot

Loading