Skip to content

chore(source-zendesk-chat): bump SDM base image to stable 7.17.2 - #76846

Merged
Patrick Nilan (pnilan) merged 2 commits into
masterfrom
devin/1776782745-bump-sdm-stable-zendesk-chat
Apr 21, 2026
Merged

Patrick Nilan (pnilan) merged 2 commits into
masterfrom
devin/1776782745-bump-sdm-stable-zendesk-chat

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Apr 21, 2026 •

Copy link
Copy Markdown
Contributor

What

Moves source-zendesk-chat off the pre-release source-declarative-manifest:7.13.0.post13.dev23823332086 image and onto the current stable 7.17.2. Bumps connector version 1.3.9 → 1.3.10.

The dev image was introduced in #75942 to validate the memory-monitor change from airbyte-python-cdk #962; that change is now in the stable 7.17.2 SDM release so the dev pin is no longer needed.

How

  • metadata.yaml — baseImage → docker.io/airbyte/source-declarative-manifest:7.17.2@sha256:1bfcca53335b2669b02414180fe9ad96b0abd9c95d6cbf50089c1d881fc07801
  • metadata.yaml — dockerImageTag → 1.3.10
  • docs/integrations/sources/zendesk-chat.md — changelog entry added (follow-up commit)

Reference: #76632 for the same baseImage-bump pattern.

Review guide

  1. airbyte-integrations/connectors/source-zendesk-chat/metadata.yaml
  2. docs/integrations/sources/zendesk-chat.md

User Impact

  • Connector moves from a pre-release SDM image to the current stable 7.17.2 release.
  • No behavioral changes expected for end users.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

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


Open in Devin Review

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

@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

github-actions Bot commented Apr 21, 2026 •

Copy link
Copy Markdown
Contributor

source-zendesk-chat Connector Test Results

45 tests   41 ✅  32s ⏱️
 2 suites   4 💤
 2 files     0 ❌

Results for commit 1173ec9.

♻️ This comment has been updated with latest results.

@pnilan
Patrick Nilan (pnilan) marked this pull request as ready for review April 21, 2026 14:50
@pnilan
Patrick Nilan (pnilan) merged commit 45e496b into master Apr 21, 2026
51 of 52 checks passed
@pnilan
Patrick Nilan (pnilan) deleted the devin/1776782745-bump-sdm-stable-zendesk-chat branch April 21, 2026 14:54
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

Hi — I'm an AI assistant (Devin) following the connector-docs playbook to review user-facing documentation for changes in this PR.

This PR is a non-functional SDM base image bump from 7.13.0.post13.dev23823332086 to the stable 7.17.2 tag, with the changelog entry for version 1.3.10 already added in this PR.

I reviewed the existing documentation at docs/integrations/sources/zendesk-chat.md against the connector's manifest.yaml, metadata.yaml, and the Zendesk Live Chat API Getting Started / OAuth Authentication docs. I found that the existing documentation is already accurate and complete:

  • Changelog: The 1.3.10 entry has the correct date (2026-04-21) and the PR link ([76846]) matches the URL.
  • Prerequisites and setup: subdomain and start_date format match the spec in manifest.yaml.
  • OAuth scopes (read, chat) match the oauth_connector_input_specification.scopes in the manifest.
  • Supported streams (12 streams) exactly match the streams declared in manifest.yaml: accounts, agents, agent_timeline, bans, chats, departments, goals, roles, routing_settings, shortcuts, skills, triggers.
  • Rate limit note (200 requests/minute) matches the Zendesk Live Chat API introduction and the connector's HTTPAPIBudget policies.

Per the connector-docs playbook's "meaningful contribution" rule, I am not opening a separate documentation PR because I could not identify a factual inaccuracy, user-failing gap, or actively misleading content that warrants a change. No action is required on your part.


Devin session

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

Copy link
Copy Markdown
Contributor Author

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

@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

Project:airbyte-docs
Status: ✅  Deploy successful!
Preview URL:https://airbyte-docs-e9yh4la3b-airbyte-growth.vercel.app
Latest Commit:1173ec9

Deployed with vercel-action

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