Skip to content

chore(source-typeform): bump SDM base image for memory monitor (CDK PR #962) - #75945

Closed
devin-ai-integration[bot] wants to merge 2 commits into
masterfrom
devin/1774998580-sdm-dev-typeform
Closed

devin-ai-integration[bot] wants to merge 2 commits into
masterfrom
devin/1774998580-sdm-dev-typeform

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Mar 31, 2026 •

Copy link
Copy Markdown
Contributor

What

Updates source-typeform to use the dev build of source-declarative-manifest from airbyte-python-cdk#962, which introduces the dual-condition memory monitor with fail-fast shutdown on memory threshold.

How

Review guide

  1. airbyte-integrations/connectors/source-typeform/metadata.yaml
    • ⚠️ Note: the base image is a dev build (7.13.0.post13.dev...), not a released version — verify the @sha256:dc8f1d05... digest matches the published dev image
    • Version jumps from an RC (1.4.7-rc.2) to a full release (1.4.8) — confirm this is intended
  2. docs/integrations/sources/typeform.md — changelog entry for 1.4.8

User Impact

Connectors built from this PR will use the dev SDM image with the new memory monitor. If the memory monitor has issues, connectors could fail-fast on memory pressure that would previously have been tolerated.

Can this PR be safely reverted and rolled back?

  • YES 💚

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

…tor test)

Co-Authored-By: patrick.nilan@airbyte.io <patrick.nilan@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 March 31, 2026 23:16
@octavia-bot

octavia-bot Bot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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

👋 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: patrick.nilan@airbyte.io <patrick.nilan@airbyte.io>
@github-actions

Copy link
Copy Markdown
Contributor

source-typeform Connector Test Results

7 tests   3 ✅  11s ⏱️
1 suites  4 💤
1 files    0 ❌

Results for commit f9895f6.

@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

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

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

@pnilan

Copy link
Copy Markdown
Member

Closing as typeform is in rc version

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.

2 participants