There is no reason to terminate it nicely.
Depends on D8402.
Differential D8403
Kill GraphServerProcess on test teardown Authored by vlorentz on Sep 6 2022, 5:06 PM.
Details
There is no reason to terminate it nicely. Depends on D8402.
Diff Detail
Event TimelineComment Actions Build has FAILED Patch application report for D8403 (id=30323)Could not rebase; Attempt merge onto dd88f177c3... Updating dd88f17..f2ea86e Fast-forward swh/graph/tests/conftest.py | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-) Changes applied before testcommit f2ea86ec523c7ee58301718302dd275034dcf54a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Sep 6 17:06:39 2022 +0200
Kill GraphServerProcess on test teardown
There is no reason to terminate it nicely.
commit 2104133e41a924cf7d2678ac829b5bee44ba2138
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Sep 6 17:02:22 2022 +0200
conftest: Refactor GraphServerProcess to be more flexible
1. queue is now encapsulated
2. object in the queue is now a dict instead of a tuple (to allow
adding more keys easily in the future + for readability)
This will be used in a future commit, to get the queue's object
twice from different functions.Link to build: https://jenkins.softwareheritage.org/job/DGRPH/job/tests-on-diff/219/ Comment Actions Build has FAILED Patch application report for D8403 (id=30326)Could not rebase; Attempt merge onto dd88f177c3... Updating dd88f17..8489c14 Fast-forward swh/graph/tests/conftest.py | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) Changes applied before testcommit 8489c14e20ae3e4a57f3216c282a894cfff236bb
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Sep 6 17:06:39 2022 +0200
Kill GraphServerProcess on test teardown
There is no reason to terminate it nicely.
commit 46255335308446d7b077099fd9b1c68726faf140
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Sep 6 17:02:22 2022 +0200
conftest: Refactor GraphServerProcess to be more flexible
1. queue is now encapsulated
2. object in the queue is now a dict instead of a tuple (to allow
adding more keys easily in the future + for readability)
This will be used in a future commit, to get the queue's object
twice from different functions.Link to build: https://jenkins.softwareheritage.org/job/DGRPH/job/tests-on-diff/222/ Comment Actions Build is green Patch application report for D8403 (id=30330)Could not rebase; Attempt merge onto dd88f177c3... Updating dd88f17..4a7f3f7 Fast-forward swh/graph/tests/conftest.py | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) Changes applied before testcommit 4a7f3f7650a4dd9e29b26b072b019e20115b72b1
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Sep 6 17:06:39 2022 +0200
Kill GraphServerProcess on test teardown
There is no reason to terminate it nicely.
commit 81d3deb2f1c1b584a076484250cb6f349009f032
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Tue Sep 6 17:02:22 2022 +0200
conftest: Refactor GraphServerProcess to be more flexible
1. queue is now encapsulated
2. object in the queue is now a dict instead of a tuple (to allow
adding more keys easily in the future + for readability)
This will be used in a future commit, to get the queue's object
twice from different functions.See https://jenkins.softwareheritage.org/job/DGRPH/job/tests-on-diff/225/ for more details. |