Skip to content

chore: DO NOT MERGE - source-shopify with CDK dev 7.17.3.post4.dev24792917452 (memory monitor test)#76915

Closed
devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/1776879823-source-shopify-cdk-dev-test
Closed

chore: DO NOT MERGE - source-shopify with CDK dev 7.17.3.post4.dev24792917452 (memory monitor test)#76915
devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/1776879823-source-shopify-cdk-dev-test

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

⚠️ DO NOT MERGE

This PR pins source-shopify to a dev/prerelease build of airbyte-cdk for validation purposes only. It is not intended for release. The dev CDK version will be yanked from PyPI once testing concludes.

What

Bumps source-shopify from airbyte-cdk = "^7" (currently resolving to 7.17.0) to the exact prerelease pin airbyte-cdk = "7.17.3.post4.dev24792917452", which contains the memory-monitor threshold + formatting + instantiation-log changes from airbytehq/airbyte-python-cdk#995.

Used to validate the CDK changes against a real connector before the CDK PR is merged and a proper release is cut.

How

  • pyproject.toml: pin airbyte-cdk to 7.17.3.post4.dev24792917452; bump connector version 3.3.03.3.1
  • poetry.lock: regenerated via poetry lock --no-update
  • metadata.yaml: bump dockerImageTag 3.3.03.3.1
  • docs/integrations/sources/shopify.md: changelog entry for 3.3.1 (PR link is TODO; will not be filled since this PR is not meant to merge)

Review guide

  1. airbyte-integrations/connectors/source-shopify/pyproject.toml
  2. airbyte-integrations/connectors/source-shopify/poetry.lock
  3. airbyte-integrations/connectors/source-shopify/metadata.yaml
  4. docs/integrations/sources/shopify.md

User Impact

None — this PR will not be merged. Its only purpose is to exercise the new memory-monitor fail-fast thresholds (_CRITICAL_THRESHOLD 0.98 → 0.95, _HIGH_PRESSURE_THRESHOLD 0.95 → 0.90), humanized error byte formatting, and one-shot MemoryMonitor instantiated with critical threshold: … INFO log in a real connector context.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Related

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

…92917452 (memory monitor test)

Co-Authored-By: patrick.nilan@airbyte.io <patrick.nilan@airbyte.io>
@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.

@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

@pnilan
Copy link
Copy Markdown
Contributor

Patrick Nilan (pnilan) commented Apr 22, 2026

/publish-connectors-prerelease

Pre-release Connector Publish Started

Publishing pre-release build for connector source-shopify.
PR: #76915

Pre-release versions will be tagged as {version}-preview.8c51c50
and are available for version pinning via the scoped_configuration API.

View workflow run
Pre-release Publish: SUCCESS

Docker image (pre-release):
airbyte/source-shopify:3.3.1-preview.8c51c50

Docker Hub: https://hub.docker.com/layers/airbyte/source-shopify/3.3.1-preview.8c51c50

Registry JSON:

@github-actions
Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

Project:airbyte-docs
Status: ✅  Deploy successful!
Preview URL:https://airbyte-docs-9y5618kkj-airbyte-growth.vercel.app
Latest Commit:8c51c50

Deployed with vercel-action

@github-actions
Copy link
Copy Markdown
Contributor

source-shopify Connector Test Results

211 tests   201 ✅  1m 59s ⏱️
  2 suites   10 💤
  2 files      0 ❌

Results for commit 8c51c50.

@pnilan
Copy link
Copy Markdown
Contributor

approve

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