Skip to content

fix(postgres): randomize test_reader username to prevent CI collisions - #5977

Open
sakshipatil-hue wants to merge 1 commit into
SQLMesh:mainfrom
sakshipatil-hue:patch-1
Open

fix(postgres): randomize test_reader username to prevent CI collisions#5977
sakshipatil-hue wants to merge 1 commit into
SQLMesh:mainfrom
sakshipatil-hue:patch-1

Conversation

@sakshipatil-hue

@sakshipatil-hue sakshipatil-hue commented Aug 23, 2026

Copy link
Copy Markdown

Description

Resolves #5972. Appended a random UUID suffix to the Postgres test user creation logic to ensure parallel integration tests are completely isolated and do not collide.

Test Plan

Ran the test suite locally to ensure the user creation logic executes without syntax errors.

Checklist

  • I have run make style and fixed any issues
  • I have added tests for my changes (if applicable)
  • All existing tests pass (make fast-test)
  • My commits are signed off (git commit -s) per the DCO

@cmgoffena13 cmgoffena13 self-assigned this Aug 23, 2026
@cmgoffena13

Copy link
Copy Markdown
Collaborator

@sakshipatil-hue - you'll need to sign your commits off with DCO

@sakshipatil-hue

Copy link
Copy Markdown
Author

@cmgoffena13 I have amended the commit to include the DCO sign-off and ran the formatting tools to fix the style issues. Could you please approve the workflow run for the latest commit when you get a chance.

@cmgoffena13

Copy link
Copy Markdown
Collaborator

@sakshipatil-hue -- way too many changes here, seems like you triggered formatting with different config.

@sakshipatil-hue
sakshipatil-hue force-pushed the patch-1 branch 2 times, most recently from 1de34a0 to 26f70eb Compare August 26, 2026 06:49
Signed-off-by: sakshipatil-hue <sakshipatil8674788@gmail.com>
@sakshipatil-hue

Copy link
Copy Markdown
Author

Hi @cmgoffena13 , I’ve cleaned up the branch to remove the accidental formatting changes, leaving only the Postgres integration test fix with a proper DCO sign-off. Ready for review whenever you're free!

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.

CI Fix - Postgres CI Failing Sometimes

2 participants