Skip to content

ci: re-queue connector tags over PostgREST instead of direct psql#357

Merged
skord merged 1 commit into
masterfrom
mdanko/scoped-ci-tokens
Jun 23, 2026
Merged

ci: re-queue connector tags over PostgREST instead of direct psql#357
skord merged 1 commit into
masterfrom
mdanko/scoped-ci-tokens

Conversation

@skord

@skord skord commented Jun 9, 2026

Copy link
Copy Markdown
Member

Description:

Replaces the direct psql re-queue of connector_tags in .github/workflows/connectors.yml (both the connector-name and alias blocks) with authenticated calls to the control-plane API (PostgREST over HTTPS), so connector publishes no longer require a direct database connection from CI. Each block exchanges a revocable refresh token for a short-lived access token whose role claim is github_action_connector_refresh, then calls the requeue_connector_tag RPC.

Depends on (draft until both are done):

Replaces the POSTGRES_CONNECTOR_REFRESH_* secrets, which can be removed after cutover.

Notes for reviewers:

  • No new database permissions: the same connector_tags.job_status UPDATE the role already holds.
  • Kept as a draft until flow#3013 is deployed and the secrets are set.

@skord skord self-assigned this Jun 9, 2026
…ct psql

Replace the direct psql connector_tags re-queue (name and alias blocks) with
authenticated control-plane API calls over HTTPS via requeue_connector_tag.
Requires CONNECTOR_REFRESH_RT_ID/SECRET and estuary/flow#3013 deployed first.
@skord skord force-pushed the mdanko/scoped-ci-tokens branch from 0bfc24d to b2bf49d Compare June 17, 2026 14:35
@skord skord requested a review from a team June 23, 2026 17:47
@skord skord marked this pull request as ready for review June 23, 2026 17:47
@skord skord merged commit 8d2f6ca into master Jun 23, 2026
44 of 45 checks passed
@skord skord deleted the mdanko/scoped-ci-tokens branch June 23, 2026 18:58
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