Skip to content

docs: link SDM release guide to the internal base image runbook - #1140

Merged
Patrick Nilan (pnilan) merged 1 commit into
mainfrom
devin/1787960147-base-image-runbook-link
Aug 29, 2026
Merged

docs: link SDM release guide to the internal base image runbook#1140
Patrick Nilan (pnilan) merged 1 commit into
mainfrom
devin/1787960147-base-image-runbook-link

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Requested by AJ Steers (@aaronsteers) as a back-link to the new internal runbook.

docs/RELEASES.md explains how CDK and SDM releases work, but nothing in this repo mentions the base image that SDM is built on: the python-connector-base pin in the root Dockerfile is bumped by hand, and merging that bump publishes nothing — the new SDM image only ships with the next CDK release. This adds a paragraph under the title stating that and linking the internal runbook that documents the whole lifecycle:

https://internal.airbyte.ai/docs/internal-docs/teams/apis/data-replication/runbooks/connector-base-images

The link is Airbyte-employee-only, and 404s until the runbook lands (https://github.com/airbytehq/internal.airbyte.ai/pull/426).

Link to Devin session: https://app.devin.ai/sessions/19639cc07c8d4b32bcf6442d50243a07
Open in Devin Desktop: https://app.devin.ai/desktop/session/19639cc07c8d4b32bcf6442d50243a07?variant=devin

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, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review August 28, 2026 23:37
@devin-ai-integration
devin-ai-integration Bot requested a review from a team as a code owner August 28, 2026 23:37
@github-actions

Copy link
Copy Markdown

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

Testing This CDK Version

You can test this version of the CDK using the following:

# Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@devin/1787960147-base-image-runbook-link#egg=airbyte-python-cdk[dev]' --help

# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch devin/1787960147-base-image-runbook-link

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

  • /autofix - Fixes most formatting and linting issues
  • /poetry-lock - Updates poetry.lock file
  • /test - Runs connector tests with the updated CDK
  • /prerelease - Triggers a prerelease publish with default arguments
  • /poe build - Regenerate git-committed build artifacts, such as the pydantic models which are generated from the manifest JSON schema in YAML.
  • /poe <command> - Runs any poe command in the CDK environment
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

Copilot AI 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.

Pull request overview

Adds context to the release management guide to clarify how the SDM image relates to the python-connector-base pin and where to find the internal lifecycle/runbook documentation.

Changes:

  • Documents that bumping the python-connector-base pin in the root Dockerfile does not itself publish a new SDM image.
  • Adds an internal (employee-only) runbook link for the connector base image lifecycle.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/RELEASES.md
@github-actions

Copy link
Copy Markdown

PyTest Results (Fast)

4 363 tests  ±0   4 351 ✅ ±0   8m 22s ⏱️ - 1m 12s
    1 suites ±0      12 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit a004f59. ± Comparison against base commit 7c96f9d.

@pnilan
Patrick Nilan (pnilan) merged commit 64bdfc9 into main Aug 29, 2026
27 checks passed
@pnilan
Patrick Nilan (pnilan) deleted the devin/1787960147-base-image-runbook-link branch August 29, 2026 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants