Skip to content
This repository has been archived by the owner on Aug 18, 2021. It is now read-only.

Commit

Permalink
Use standard 36-character UUID representation for telemetry (#305)
Browse files Browse the repository at this point in the history
The new telemetry pipeline in GCP is more strict about requiring that
submissions include a valid UUID as the document_id in the URI.
Specifically, the pipeline requires the standard 36-character representation
that includes dashes.

Currently, wpt pings are identified as invalid and sent to error output in the
GCP telemetry pipeline due to using 32-character UUIDs.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1597217
  • Loading branch information
jklukas authored and stephendonner committed Nov 18, 2019
1 parent 201a6f5 commit 3b08dc4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion send_to_telemetry.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ def main(path):
validate(asdict(result), schema)

# send to telemetry
wpt_run_uuid = uuid.uuid4().hex
wpt_run_uuid = str(uuid.uuid4())
telemetry_url = f"https://incoming.telemetry.mozilla.org/submit/webpagetest/webpagetest-run/1/{wpt_run_uuid}"
results_json = json.dumps(asdict(result))
r = requests.post(
Expand Down

0 comments on commit 3b08dc4

Please sign in to comment.