[CDRIVER-4172] Migrate uses of json.send to perf.send in Evergreen project YAML Created: 01/Oct/21 Updated: 28/Oct/23 Resolved: 02/Nov/21 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.20.0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Maria van Keulen | Assignee: | Roberto Sanchez |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Upstream Changes Summary: | DRIVERS-1892: |
||||||||||||||||
| Description |
SummaryAs of PM-2172, the TIPS team would like to remove its legacy performance gathering and analysis API in favor of a new, more robust option. In order to migrate to the new API, uses of json.send in Evergreen project YAML should be removed in favor of using perf.send (assuming it is desirable to continue to maintain / run the testing that uses json.send) . The following instructions describe how to use perf.send. We have also added docs for migrating away from json.send and onto perf.send. MotivationWho is the affected end user?This ticket tracks the mongo-python-driver-perf change specifically. How does this affect the end user?The trend charts UI will continue to display any legacy performance data that had previously been gathered with json.send even after migration to perf.send. Additionally, it appears that change points are not presently being detected for these projects even in cases where performance data is being collected; migration to perf.send should avoid these sorts of errors. How likely is it that this problem or use case will occur?N/A If the problem does occur, what are the consequences and how severe are they?N/A Is this issue urgent?TIPS would like to remove the json.send tooling as part of PM-2172, which we are working on this quarter. If all the usages could be removed or migrated to the new tooling by October 25th, that would be great. Is this ticket required by a downstream team?Yes. It is required by TIPS for PM-2172. Is this ticket only for tests?Yes. This is specific to gathering and analyzing performance data. |
| Comments |
| Comment by Githook User [ 02/Nov/21 ] |
|
Author: {'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}Message: |
| Comment by Githook User [ 02/Nov/21 ] |
|
Author: {'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}Message: |
| Comment by Roberto Sanchez [ 01/Nov/21 ] |
|
maria.vankeulen I was able to access it. Thanks for your assistance. I have implemented the necessary changes to the C driver project and they are pending review. |
| Comment by Maria van Keulen [ 01/Nov/21 ] |
|
roberto.sanchez were you able to receive access to the repo? IIRC you can request access via MANA. |
| Comment by Maria van Keulen [ 28/Oct/21 ] |
|
Hello alexander.golin, do we have a sense when there will be time to work on this? |