Each test should clear the tmp directory (ds/dirsrvtests/tmp/), excluding the README, in the initialization phase. Future scripts will need to use this change in their script templates.
lib389 - add function to clear tmp dir 0001-Ticket-47855-Add-function-to-clear-tmp-directory.patch
attachment 0001-Ticket-47855-clear-tmp-directory-at-the-start-of-eac.patch
Maybe, it'd be nice if we could learn from the log message why it failed (printing errno or something similar, if any?) But I guess the failure is very rare?
Replying to [comment:2 nhosoi]:
Yes it should be very rare, if it fails something very seriously wrong is happening on the file system. Then os.remove() will throw an exception with the reason, and it will stop the run.
Replying to [comment:3 mreynolds]:
Replying to [comment:2 nhosoi]: Maybe, it'd be nice if we could learn from the log message why it failed (printing errno or something similar, if any?) But I guess the failure is very rare? Yes it should be very rare, if it fails something very seriously wrong is happening on the file system. Then os.remove() will throw an exception with the reason, and it will stop the run. Thanks for your explanation! You have my second "ack". :)
Yes it should be very rare, if it fails something very seriously wrong is happening on the file system. Then os.remove() will throw an exception with the reason, and it will stop the run. Thanks for your explanation! You have my second "ack". :)
To ssh://git.fedorahosted.org/git/389/lib389.git 387135d..b488201 master -> master commit b488201dd4033030f390c1da41eba4f5a9b4c74b
git merge ticket47855 Updating 0dfe006..84477da Fast-forward dirsrvtests/tickets/ticket47313_test.py | 3 +++ dirsrvtests/tickets/ticket47490_test.py | 5 ++++- dirsrvtests/tickets/ticket47553_single_aci_test.py | 3 +++ dirsrvtests/tickets/ticket47560_test.py | 5 ++++- dirsrvtests/tickets/ticket47573_test.py | 5 ++++- dirsrvtests/tickets/ticket47619_test.py | 5 ++++- dirsrvtests/tickets/ticket47653MMR_test.py | 5 ++++- dirsrvtests/tickets/ticket47653_test.py | 5 ++++- dirsrvtests/tickets/ticket47676_test.py | 5 ++++- dirsrvtests/tickets/ticket47721_test.py | 5 ++++- dirsrvtests/tickets/ticket47781_test.py | 3 +++ dirsrvtests/tickets/ticket47787_test.py | 5 ++++- dirsrvtests/tickets/ticket47808_test.py | 5 ++++- dirsrvtests/tickets/ticket47815_test.py | 3 +++ dirsrvtests/tickets/ticket47819_test.py | 3 +++ dirsrvtests/tickets/ticket47823_test.py | 3 +++ dirsrvtests/tickets/ticket47829_test.py | 5 ++++- 17 files changed, 62 insertions(+), 11 deletions(-)
To ssh://git.fedorahosted.org/git/389/ds.git 0dfe006..84477da master -> master
commit 84477da Author: Mark Reynolds mreynolds@redhat.com Date: Mon Jul 14 12:37:11 2014 -0400
Incorrectly used "standalone.clearTmpDir() in all the testcases, but not all the testcases use "standalone" for the instance name. Working on a new patch.
Fix errors with previous patch 0001-Ticket-47855-Fix-previous-commit.patch
git merge ticket47855 Updating 42fa2e6..b218ad7 Fast-forward dirsrvtests/tickets/ticket47490_test.py | 2 +- dirsrvtests/tickets/ticket47553_single_aci_test.py | 2 +- dirsrvtests/tickets/ticket47573_test.py | 2 +- dirsrvtests/tickets/ticket47619_test.py | 2 +- dirsrvtests/tickets/ticket47653MMR_test.py | 2 +- dirsrvtests/tickets/ticket47676_test.py | 2 +- dirsrvtests/tickets/ticket47721_test.py | 2 +- dirsrvtests/tickets/ticket47787_test.py | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-)
To ssh://git.fedorahosted.org/git/389/ds.git 42fa2e6..b218ad7 master -> master commit b218ad7
Metadata Update from @nhosoi: - Issue assigned to mreynolds - Issue set to the milestone: 1.3.3 - 8/14 (August)
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/1186
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.