Repository navigation
chore(source-amazon-seller-partner): DO NOT MERGE - pin prerelease SDM 7.39.0.post5.dev39879313492 (airbyte-python-cdk#1248) - #89694
Conversation
…M 7.39.0.post5.dev39879313492 (airbyte-python-cdk#1248) Co-Authored-By: patrick.nilan@airbyte.io <patrick.nilan@airbyte.io>
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksPR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful Resources
|
|
/ai-prove-fix specifically want to confirm this stack of PRs functions as expected.
|
🧪 Fix Validation Evidence🟢 Outcome: Fix ProvenOn the prerelease SDM/CDK Next steps: Before treating the whole stack as proven, get a human-approved live check for source-airtable and source-bing-ads. Both use OAuth refresh tokens, so testing them on a customer connection needs approval. The "bing-ads is faster" claim is still unmeasured.
Per-connector results
📝 Evidence Plan
✅ Pre-flight Checklist
🔍 Detailed Evidence Log
|
|
|
Closing: test-only DO NOT MERGE PR; the /ai-prove-fix run is complete (see the Fix Validation Evidence comment above). Requested by Patrick Nilan (@pnilan). |
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksPR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful Resources
|
|
|
Prerelease published from this PR's original head commit, as requested. Closing again (DO NOT MERGE test PR). |
DO NOT MERGE. Test-only PR, requested by Patrick Nilan (@pnilan). It pins
source-amazon-seller-partnerto the SDM prerelease built from airbyte-python-cdk#1248 so a connector prerelease can be published and tested.What
Points
connectorBuildOptions.baseImageatairbyte/source-declarative-manifest:7.39.0.post5.dev39879313492. That image was built by CDK publish run 39879313492 fromdevin/1792688475-check-skip-dynamic-streams.How
metadata.yaml:baseImagechanges from SDM7.37.1to7.39.0.post5.dev39879313492@sha256:a45420a851310e75d893c93c73e8a353d610bc214788c39384f7bd776b614d7f.dockerImageTaggoes from7.0.0to7.0.1.Review guide
airbyte-integrations/connectors/source-amazon-seller-partner/metadata.yamlUser Impact
None. This is never merged. Use
/publish-connectors-prereleaseto get a7.0.1-preview.<sha>image for pinning.The title contains "DO NOT MERGE", so the semantic-title check fails on purpose. Checks that expect a changelog entry may fail too.
Can this PR be safely reverted and rolled back?
Link to Devin session: https://app.devin.ai/sessions/a359f078a4634bbab6d79afaaf709c17
Open in Devin Desktop: https://app.devin.ai/desktop/session/a359f078a4634bbab6d79afaaf709c17?variant=devin
Important
Autopilot Progressive Rollout Enabled
Autopilot progressive rollouts are enabled for one or more connector(s) modified in this PR. Check the box below if you need to bypass normal rollout safety processes and release to all users immediately upon merge:
Note: