Skip to content

chore(warehouse-sources): promote the pylon source to GA - #108904

Merged
trunk-io[bot] merged 1 commit into
masterfrom
posthog/pylon-source-ga
Sep 30, 2026
Merged

trunk-io[bot] merged 1 commit into
masterfrom
posthog/pylon-source-ga

Conversation

@Gilbert09

Copy link
Copy Markdown
Member

Problem

Anyone connecting a source in the data warehouse wizard sees Pylon labelled "alpha", which reads as "this might not work" and discourages people from picking it. The connector has earned its way out of that label: it has been live for over three months, and its imports failed 0.0% of the time over the last 7 days (the bar is under 2.5%).

Changes

  • The Pylon tile in the source catalog now shows no alpha badge, so people see a connector we stand behind.
  • releaseStatus on the Pylon source config moves from ReleaseStatus.ALPHA to ReleaseStatus.GA. That is the value /api/public_source_configs/ surfaces and the frontend renders.
  • Nothing gates Pylon today: it carries no unreleasedSource flag and no featureFlag, so there is no gating to remove. The test now asserts both stay absent.
  • No behavior change. Schemas, endpoints, pagination, and sync logic are untouched.

How did you test this code?

  • Ran products/warehouse_sources/backend/temporal/data_imports/sources/pylon/tests/test_pylon_source.py locally: 12 passed.
  • The updated assertion catches a regression that would re-hide Pylon behind a gate, by pinning unreleasedSource, featureFlag, and releaseStatus together.
  • Not run: the wizard UI was not rendered. The label is read straight from this field, and no frontend code changed.

Release status

  • No feature flag controls this change
  • This change is behind a feature flag and is not available to users
  • This change makes a previously flagged feature available to everyone

Automatic notifications

  • Publish to changelog?

Docs update

None.

🤖 Agent context

Autonomy: Fully autonomous

Agent: Claude Code, Opus 5

Skills invoked: /implementing-warehouse-sources, /writing-pr-descriptions.

No duplicate: searched open PRs for "Pylon", "releaseStatus", and "promote", and listed every open PR by the maintainer. The only other Pylon PR is #106228, which swaps the source icon and does not touch release status. #108805 is the same promotion for Neon and was used as the pattern for this diff.

Public artifact: the diff and this description carry no material from the agent session beyond the two aggregate metrics quoted above.


Created with PostHog Desktop

🤖 Generated with Claude Code

The Pylon source meets the GA bar: live over three months with a 0.0% import error rate in the last 7 days.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Generated-By: PostHog Desktop
Task-Id: d09b5108-23f4-4bf1-964b-25647479c2b0
Copilot AI balanced review requested due to automatic review settings September 30, 2026 05:03
@trunk-io

trunk-io Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

😎 Merged successfully - details.

@Gilbert09 Gilbert09 added the stamphog Request AI approval (no full review) label Sep 30, 2026 — with Talyn App

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@stamphog stamphog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved.

Single-field label change from alpha to GA with a matching test update; contained, reversible, no behavior change.

  • Author wrote 100% of the modified lines and has 0 merged PRs in these paths (familiarity STRONG).
  • copilot-pull-request-reviewer[bot] reviewed the current head.
Gate mechanics and policy version
Gate Result
prerequisites ✓ all clear
deny-list ✓ no deny categories matched
size ✓ 2L, 1F substantive, 7L/2F incl. docs/generated/snapshots — within ceiling
tier ✓ T1-agent / T1a-trivial (7L, 2F, single-area, chore)
stamphog 2.3.1 .stamphog/policy.yml @ 6e5f17a · reviewed head 6e5f17a

@pr-assigner-resolver-posthog
pr-assigner-resolver-posthog Bot requested a review from a team September 30, 2026 05:04
@github-actions

github-actions Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

🤖 CI report

⚠️ Trunk lane — backend Python lane (py:product:warehouse_sources)

This PR is assigned to the backend Python lane (py:product:warehouse_sources). It runs backend Python tests and may merge in parallel with PRs in other lanes.

✅ Duplication (Python) — clean

New Python code duplication introduced by this branch. Fails at 70+ tokens in app code, or 150+ tokens when both copies live in test files. Advisory while the gate proves itself: extract a shared helper instead of copying.

✅ Duplication (TypeScript) — clean

New TypeScript code duplication introduced by this branch. Fails at 70+ tokens in app code, or 150+ tokens when both copies live in test files. Advisory while the gate proves itself: extract a shared helper instead of copying.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: PostHog/posthog/.coderabbit.yaml

Review profile: QUIET

Plan: Enterprise

Run ID: 496cc6d9-e739-4991-bb82-a22c67385778

📥 Commits

Reviewing files that changed from the base of the PR and between 1c66a26 and 6e5f17a.

📒 Files selected for processing (2)
  • products/warehouse_sources/backend/temporal/data_imports/sources/pylon/source.py
  • products/warehouse_sources/backend/temporal/data_imports/sources/pylon/tests/test_pylon_source.py

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

The Pylon source configuration now sets its release status to GA instead of alpha. The test expects the GA status and verifies that featureFlag and unreleasedSource are unset.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 6e5f1

Pylon will be classified as generally available. The selected value is supported by the shared status contract, and no concrete merge-blocking risk is evidenced.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The description is complete and matches the required template. It explains the user impact, lists the code and test changes, identifies test coverage and limitations, selects exactly one release-statu…
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@trunk-io

trunk-io Bot commented Sep 30, 2026

Copy link
Copy Markdown

Static Badge   Static Badge   Static Badge

View Full Report ↗︎ ⋅ Docs

Copy link
Copy Markdown
Member Author

/trunk merge

@trunk-io
trunk-io Bot merged commit 764a77c into master Sep 30, 2026
260 checks passed
@trunk-io
trunk-io Bot deleted the posthog/pylon-source-ga branch September 30, 2026 09:19
@deployment-status-posthog

deployment-status-posthog Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-09-30 09:38 UTC Run
prod-us ✅ Deployed 2026-09-30 09:52 UTC Run
prod-eu ✅ Deployed 2026-09-30 09:54 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stamphog Request AI approval (no full review)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants