fix(postgres): randomize test_reader username to prevent CI collisions - #5977
fix(postgres): randomize test_reader username to prevent CI collisions#5977sakshipatil-hue wants to merge 2 commits into
Conversation
|
@sakshipatil-hue - you'll need to sign your commits off with DCO |
cd83751 to
ae1cb82
Compare
|
@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. |
|
@sakshipatil-hue -- way too many changes here, seems like you triggered formatting with different config. |
1de34a0 to
26f70eb
Compare
Signed-off-by: sakshipatil-hue <sakshipatil8674788@gmail.com>
26f70eb to
b2d62d4
Compare
|
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! |
|
@sakshipatil-hue -- the ruff format/lint is failing. Make sure you've run The postgres test is failing. Some of the tests are still granting on the old username since its hardcoded. Tests that need adjusted:
Look at |
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
make styleand fixed any issuesmake fast-test)git commit -s) per the DCO