Page MenuHomePhabricator

Initial CloudWatch metric reporting support
ClosedPublic

Authored by amckinley on Apr 12 2018, 11:25 PM.
Tags
None
Referenced Files
F15543848: D19363.id46329.diff
Sat, Apr 26, 1:34 AM
F15509519: D19363.id46318.diff
Wed, Apr 16, 1:09 PM
F15492005: D19363.id46319.diff
Sat, Apr 12, 9:44 AM
F15472433: D19363.id46328.diff
Sat, Apr 5, 1:36 PM
F15456150: D19363.diff
Sun, Mar 30, 7:21 AM
F15438681: D19363.id46329.diff
Mar 26 2025, 2:39 AM
F15436846: D19363.id46318.diff
Mar 25 2025, 4:39 PM
F15434005: D19363.id46328.diff
Mar 25 2025, 1:31 AM
Subscribers

Details

Summary

Ref T12857. Ref T13076. Initial infrastrucutre for creating and submitting CloudWatch metrics to AWS.

Test Plan

Submitted some metrics, saw them show up.

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20143
Build 27328: Run Core Tests
Build 27327: arc lint + arc unit

Event Timeline

wrong concatenation operator

epriestley added inline comments.
src/future/aws/PhutilCloudWatchMetric.php
47

I think this construction means that adding the same dimension with different values will silently do nothing the second time, which is maybe a little weird:

$metric->addDimension('a', 1);
$metric->addDimension('a', 2); // Does nothing.

Consider:

$this->dimensions[$name] = $value;

...to make a second call override, or an array_key_exists() test plus throw ... to make a second call an error.

This revision is now accepted and ready to land.Apr 13 2018, 11:53 AM
This revision was automatically updated to reflect the committed changes.