Skip to content

resource cleanup after tests #687

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 30 commits into from
Closed

resource cleanup after tests #687

wants to merge 30 commits into from

Conversation

dothebart
Copy link
Collaborator

@dothebart dothebart commented Jun 3, 2025 •
edited
Loading

  • add deffer cleanup methods to drop databases created during the respective tests
  • add deffer cleanup methods to delete users created during the respective tests
  • increase the timeout to hot backup restore waiting
  • limit the license test to the versions pre-disk-usage.

@cla-bot cla-bot bot added the cla-signed label Jun 3, 2025
dothebart added 27 commits June 3, 2025 12:29
@dothebart
better use defer
5d207b4
@dothebart
better use defer
7e4ceaa
@dothebart
clean up users after test
6956994
@dothebart
lint
13d7c8f
@dothebart
more cleanup of leftover databases
c440970
@dothebart
lint
a82ff3a
@dothebart
fix name
a8676e7
@dothebart
lint
6cf71b0
@dothebart
r2 currently out of scope
978cf8f
@dothebart
fix query test, fix double delete
a6a9b92
@dothebart
fix duplicate database create & cleanup
7f58d49
@dothebart
add missing cleanup
3f5c6d3
@dothebart
add missing cleanup
d579efa
@dothebart
cleanup
b9c6d86
@dothebart
disable replication2 test
898a8b8
@dothebart
only in clusters
c8df2c9
@dothebart
later versions are incompatible with this test
ee61242
@dothebart
better error message
afbc2a1
@dothebart
longer timeouts
0a81472
@dothebart
no news is good news, proper error messages
3f7d4cf
@dothebart
add missing transaction cleanup
2fdfe02
@dothebart
add the error message to the failure print
d442dcb
@dothebart
choose uniq db name
88508ba
@dothebart
give dropping of database more time
666371d
@dothebart
longer deadline
d0ce3db
@dothebart
lint
602561d
@dothebart
only abort the transaction if its not yet committed
b6e9487
@dothebart dothebart changed the title Bug fix/v1 database cleanup resource cleanup after tests Jun 23, 2025
@dothebart
Copy link
Collaborator Author

replaced by #688

@dothebart dothebart closed this Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jwierzbo jwierzbo jwierzbo approved these changes

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dothebart @jwierzbo