Skip to content

chore: nerf the load pgbouncer test#4264

Open
grutt wants to merge 1 commit into
mainfrom
chore--disable-load-pgbouncer
Open

chore: nerf the load pgbouncer test#4264
grutt wants to merge 1 commit into
mainfrom
chore--disable-load-pgbouncer

Conversation

@grutt

@grutt grutt commented Jun 23, 2026

Copy link
Copy Markdown
Member

Description

Removes constantly failing test suite.

Type of change

  • CI (any automation pipeline changes)

What's Changed

  • Removes pg bouncer variant of the load test

Checklist

Changes have been:

  • Tested (unit, integration, or manually with steps specified)
  • Linted and formatted
  • Documented (where applicable)
  • Added to CHANGELOG (where applicable) -- see Keep a Changelog

🤖 AI Disclosure
  • I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.
  • Details: n/a

Copilot AI review requested due to automatic review settings June 23, 2026 14:49
@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hatchet-docs Ready Ready Preview, Comment Jun 23, 2026 2:51pm

Request Review

@github-actions github-actions Bot added the github_actions Pull requests that update GitHub Actions code label Jun 23, 2026
@grutt grutt requested a review from mrkaye97 June 23, 2026 14:50

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

This PR updates the CI workflow to remove the PgBouncer variant of the load-test job, with the goal of eliminating a consistently failing/flaky test run from the matrix.

Changes:

  • Removes the load-pgbouncer job from .github/workflows/test.yml.
  • Keeps the existing load and load-online-migrate jobs intact.

@mrkaye97 mrkaye97 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.

woohoo! I'm curious to remove the optimistic scheduling stuff from the testing matrix while we're at it, since that's also not running anywhere... maybe slightly riskier though idk

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

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants