Skip to content

chore(source-intercom): increase HTTP connection pool size via dev CDK from #926#76122

Merged
Anatolii Yatsuk (tolik0) merged 5 commits intomasterfrom
devin/1775575855-source-intercom-dev-cdk
Apr 8, 2026
Merged

chore(source-intercom): increase HTTP connection pool size via dev CDK from #926#76122
Anatolii Yatsuk (tolik0) merged 5 commits intomasterfrom
devin/1775575855-source-intercom-dev-cdk

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot commented Apr 7, 2026

What

Updates source-intercom to use a prerelease version of the CDK built from airbyte-python-cdk#926, which increases MAX_CONNECTION_POOL_SIZE from 20 to 40. This aligns the HTTP connection pool size with connectors that use max_concurrency up to 40, eliminating urllib3 "Connection pool is full, discarding connection" warnings.

How

  • Updated connectorBuildOptions.baseImage in metadata.yaml from source-declarative-manifest:7.15.0 to the prerelease source-declarative-manifest:7.17.0.post4.dev24089021469 (including pinned sha256 digest).
  • Bumped dockerImageTag from 0.13.18 to 0.13.19.
  • Added 0.13.19 changelog entry in connector docs.

Review guide

  1. airbyte-integrations/connectors/source-intercom/metadata.yaml — base image tag + digest update, version bump.
  2. docs/integrations/sources/intercom.md — changelog entry for 0.13.19.

User Impact

No user impact — this is a test PR to validate the CDK change from airbyte-python-cdk#926 against the Intercom connector.

⚠️ This PR uses a dev/prerelease CDK image and should not be merged to production as-is. The base image should be reverted to a stable release once testing is complete.

Can this PR be safely reverted and rolled back?

  • YES 💚

Link to Devin session: https://app.devin.ai/sessions/d6a9f278fbce4ac3a0d8b527de67927f

Co-Authored-By: gl_anatolii.yatsuk <gl_anatolii.yatsuk@airbyte.io>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@octavia-bot octavia-bot bot marked this pull request as draft April 7, 2026 15:32
@octavia-bot
Copy link
Copy Markdown
Contributor

octavia-bot bot commented Apr 7, 2026

Note

📝 PR Converted to Draft

More info...

Thank you for creating this PR. As a policy to protect our engineers' time, Airbyte requires all PRs to be created first in draft status. Your PR has been automatically converted to draft status in respect for this policy.

As soon as your PR is ready for formal review, you can proceed to convert the PR to "ready for review" status by clicking the "Ready for review" button at the bottom of the PR page.

To skip draft status in future PRs, please include [ready] in your PR title or add the skip-draft-status label when creating your PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
      • Bump types: patch (default), minor, major, major_rc, rc, promote.
      • The rc type is a smart default: applies minor_rc if stable, or bumps the RC number if already RC.
      • The promote type strips the RC suffix to finalize a release.
      • Example: /bump-version type=rc or /bump-version type=minor
    • /bump-progressive-rollout-version - Alias for /bump-version type=rc. Bumps with an RC suffix and enables progressive rollout.
  • ❇️ AI Testing and Review (internal link: AI-SDLC Docs):
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
  • 🐍 Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

Co-Authored-By: gl_anatolii.yatsuk <gl_anatolii.yatsuk@airbyte.io>
@devin-ai-integration devin-ai-integration bot changed the title chore(source-intercom): use dev CDK version from airbyte-python-cdk#926 chore(source-intercom): increase HTTP connection pool size via dev CDK from #926 Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

source-intercom Connector Test Results

13 tests   9 ✅  13s ⏱️
 2 suites  4 💤
 2 files    0 ❌

Results for commit 0969cc1.

♻️ This comment has been updated with latest results.

Co-Authored-By: gl_anatolii.yatsuk <gl_anatolii.yatsuk@airbyte.io>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-pg6f36yow-airbyte-growth.vercel.app

Built with commit 0969cc1.
This pull request is being automatically deployed with vercel-action

devin-ai-integration bot and others added 2 commits April 7, 2026 15:57
…elog

Co-Authored-By: gl_anatolii.yatsuk <gl_anatolii.yatsuk@airbyte.io>
Co-Authored-By: gl_anatolii.yatsuk <gl_anatolii.yatsuk@airbyte.io>
@tolik0 Anatolii Yatsuk (tolik0) merged commit c3b1e12 into master Apr 8, 2026
43 checks passed
@tolik0 Anatolii Yatsuk (tolik0) deleted the devin/1775575855-source-intercom-dev-cdk branch April 8, 2026 14:00
devin-ai-integration bot added a commit that referenced this pull request Apr 8, 2026
PR #76122 was merged on 2026-04-08, not 2026-04-07. The /bump-version
command set the date to when it ran rather than the actual merge date.

Co-Authored-By: bot_apk <apk@cognition.ai>
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.

3 participants