Skip to content

chore: certify destination-motherduck - #75642

Merged
Aaron ("AJ") Steers (aaronsteers) merged 1 commit into
masterfrom
devin/1774922954-certify-destination-motherduck
Mar 31, 2026
Merged

chore: certify destination-motherduck#75642
Aaron ("AJ") Steers (aaronsteers) merged 1 commit into
masterfrom
devin/1774922954-certify-destination-motherduck

Conversation

@aaronsteers

@aaronsteers Aaron ("AJ") Steers (aaronsteers) commented Mar 31, 2026

Copy link
Copy Markdown
Member

What

Upgrades destination-motherduck from community to certified support level.

How

Changed supportLevel: communitysupportLevel: certified in the connector's metadata.yaml.

Review guide

  1. airbyte-integrations/connectors/destination-motherduck/metadata.yaml

Reviewer checks:

  • The ab_internal.sl and ab_internal.ql fields are currently both 100. Should these be increased (e.g. to 200/300) to reflect certified status?
  • The releaseStage is still alpha. Is this compatible with supportLevel: certified, or should it be bumped to beta / generally_available?
  • remoteRegistries.pypi.enabled is false. Confirm this is acceptable for a certified connector.

User Impact

destination-motherduck will appear as "certified" instead of "community" in the Airbyte UI and connector registry. No functional changes.

Can this PR be safely reverted and rolled back?

  • YES 💚

Link to Devin session: https://app.devin.ai/sessions/0d4c25fa273147bba0ed9ed9d4f76a0b
Requested by: Aaron ("AJ") Steers (@aaronsteers)


Open with Devin

Upgrade destination-motherduck from community to certified support level.
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 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

@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.
  • ❇️ 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.
    • /bump-progressive-rollout-version - Bumps connector version with an RC suffix (2.16.10-rc.1) for progressive rollouts (enableProgressiveRollout: true).
      • Example: /bump-progressive-rollout-version changelog="Add new feature for progressive rollout"
  • ☕️ 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.

@github-actions

Copy link
Copy Markdown
Contributor

destination-motherduck Connector Test Results

50 tests   48 ✅  18s ⏱️
 2 suites   2 💤
 2 files     0 ❌

Results for commit 6389114.

@aaronsteers
Aaron ("AJ") Steers (aaronsteers) marked this pull request as ready for review March 31, 2026 02:14
Comment on lines 39 to 41
ab_internal:
sl: 100
ql: 100

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we not need to update sl or ql?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at existing certified destinations, sl and ql values vary widely — there's no single required value for certified status. Several certified destinations already use sl: 100, ql: 100:

  • destination-customer-io → sl: 100, ql: 100
  • destination-databricks → sl: 100, ql: 100
  • destination-hubspot → sl: 100, ql: 100
  • destination-mssql → sl: 100, ql: 100
  • destination-s3-data-lake → sl: 100, ql: 100

Others use higher values (e.g. BigQuery, Snowflake, S3 use sl: 300, ql: 300).

So the current sl: 100, ql: 100 is compatible with certified status. Would you like me to bump these to a specific value, or leave as-is?

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@aaronsteers
Aaron ("AJ") Steers (aaronsteers) merged commit d4a0815 into master Mar 31, 2026
58 of 59 checks passed
@aaronsteers
Aaron ("AJ") Steers (aaronsteers) deleted the devin/1774922954-certify-destination-motherduck branch March 31, 2026 02:16
dilanalex pushed a commit to dilanalex/airbyte that referenced this pull request Apr 6, 2026
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
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