Page MenuHomePhabricator
Create Task
Maniphest T315536

Create program to interact with Atlas RIPE API
Closed, ResolvedPublic
Actions

Description

The goal is to create a program that will be able to do the following:

  • Create ping measurement based on country or continent to one or all of our 6 DCs.
  • List and print all measurements by the user
  • Retrieve results printing them on screen or have the results stored in a CSV file
  • Parse retrieved measurements and provide an ordered list of DCs (from low latency to high latency) mapped to each country the measurement was created.

Below are some references to RIPE's API documentation and tools.

Details

SubjectRepoBranchLines +/-
Add latency measurement programoperations/software/latency-measurementmaster+635 -0
utils: Add latency measurement programoperations/dnsmaster+635 -0
Customize query in gerrit

Event Timeline

• MMandere created this task.Aug 18 2022, 9:29 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 18 2022, 9:29 AM
Maintenance_bot added a project: SRE.Aug 18 2022, 9:45 AM
cmooney subscribed.Aug 18 2022, 9:51 AM
gerritbot added a comment.Aug 18 2022, 10:34 AM

Change 824452 had a related patch set uploaded (by MMandere; author: MMandere):

[operations/dns@master] utils: Add latency measurement program

https://gerrit.wikimedia.org/r/824452

gerritbot added a project: Patch-For-Review.Aug 18 2022, 10:34 AM
Aklapper removed • MMandere as the assignee of this task.Sep 7 2022, 2:38 PM

Removing inactive assignee (please do so as part of offboarding - thanks!)

BCornwall triaged this task as Medium priority.Sep 8 2022, 4:04 PM
gerritbot added a comment.Sep 21 2022, 7:27 PM

Change 833848 had a related patch set uploaded (by BCornwall; author: MMandere):

[operations/software/latency-measurement@master] Add latency measurement program

https://gerrit.wikimedia.org/r/833848

gerritbot added a comment.Sep 21 2022, 7:29 PM

Change 824452 abandoned by BCornwall:

[operations/dns@master] utils: Add latency measurement program

Reason:

Moving to operations/software/latency-measurement (https://gerrit.wikimedia.org/r/c/operations/software/latency-measurement/+/833848)

https://gerrit.wikimedia.org/r/824452

BCornwall claimed this task.Sep 21 2022, 7:35 PM
BCornwall added a subscriber: ssingh.Sep 27 2022, 2:15 PM

While we have https://gerrit.wikimedia.org/r/c/operations/software/latency-measurement/+/833848 available for review, I hear that there'd be pushback for not having any tests.

I see two paths forward:

  1. We commit the current codebase without tests and create a follow-up ticket to create some; or,
  1. We write some tests before committing the code, leaving it in Gerrit

After speaking with @ssingh it sounds like the second option is the preference.

gerritbot added a comment.Oct 18 2022, 2:59 PM

Change 833848 merged by BCornwall:

[operations/software/latency-measurement@master] Add latency measurement program

https://gerrit.wikimedia.org/r/833848

Maintenance_bot removed a project: Patch-For-Review.Oct 18 2022, 3:31 PM
BCornwall closed this task as Resolved.Oct 18 2022, 8:29 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL