chore(zoom): bump connector to 1.2.1 to republish the Cloudflare-skip manifest - #1936
Conversation
… manifest constructorfabric#1933 fixed the participants stream (skip partitions 403-blocked by Cloudflare's WAF) in connector.yaml but left descriptor version at 1.2.0 — the version already active in Airbyte since Jul 14. Reconcile republishes a manifest only on descriptor-version drift, so it no-ops and the pre-fix manifest keeps running (and failing) every nightly sync. Patch bump per ADR-0015: republish + catalog re-discover, no full-refresh — the fix only skips poisoned partitions at extract time; bronze→silver semantics are unchanged. Refs constructorfabric#1932, constructorfabric#1933. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Signed-off-by: Roman Mitasov <Roman.Mitasov@constructor.tech>
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Zoom connector descriptor was bumped from version ChangesZoom descriptor republish
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related issues
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Why
#1933 fixed the
participantsstream (skip partitions 403-blocked by Cloudflare's WAF) inconnector.yamlbut left the descriptor version at1.2.0— the version already active in Airbyte (published Jul 14, pre-fix). Reconcile republishes a manifest only on descriptor-version drift (current 1.2.0 == target 1.2.0→ noop), so the fixed manifest never reaches Airbyte and every nightly zoom sync keeps dying on the blocked meeting UUID.What
One-line descriptor bump
1.2.0→1.2.1(+ version-history comment, same style as m365's).Per ADR-0015 a patch bump dispatches republish + catalog re-discover, no full-refresh — correct here: the fix only skips poisoned partitions at extract time, bronze→silver semantics are unchanged.
Verification plan (dev-vhc, after merge+deploy)
1.2.1(active manifest description in Airbyte).argo submit --from cronwf/zoom-default-sync -n insight→ sync green, participants stream skips the CF-blocked meeting,_meetingsparent cursor finally advances.Refs #1932, #1933.
🤖 Generated with Claude Code
Summary by CodeRabbit