{"push_id":1145306418,"size":3,"distinct_size":1,"ref":"refs/heads/master","head":"3e32c9e48f97fbf76f641330bda75d966c0adeeb","before":"fcc77d99a6af6c9dbca3c7f5dc2ac29be16deebf","commits":[{"sha":"949b7c47b1c8083a75fc0c162d3e981b6f074a07","author":{"name":"Lauri at Zalando","email":"9641c2d2e8991bfe843675a36a1b4bfcbe3ffa3d@gmail.com"},"message":"Update monitoringonaws.rst\n\nLine 7: \"This section assumes that you're running zmon-aws-agent\" ... what if you're not?\r\nLine 12: \"access from ZMON.\" Access *what*?\r\nLine 12: Reference to Prometheus node exporter seems pretty random. A quick search of the RtD turns up no other references.\r\n\r\nLine 14: This is unclear: \"Not having the proper security groups configured is mainly visible by not getting the expected results at all, as packages are dropped by the EC2 instance rather then e.g. getting a connection refused.\" I *think* you're trying to say this: \"Not configuring the proper security groups will not produce expected (or \"accurate\"?) results, because the EC2 instance will drop packages instead of refusing the connection outright.\"\r\nLine 22: \"you can only get space used from CloudWatch\"--used FROM or used BY? In any case, this sentence still seems not quite right.\r\nLine 22: \"pull this data\" --\"this data\" isn't super-clear, the first part of the sentence refers to diskspace\r\nLine 28: \"you can either query via CloudWatch\". You can query CloudWatch, or \"run a query via CloudWatch\"?\r\nLine 28: \"In general, we don't encourage this.\" The last of the three options, or all three options?\r\nLine 38: \"hosts spawn into a productive state.\" Does spawn = produce in this case? If so, the wording doesn't quite work yet.\r\nLine 43: I added a prompt here.\r\nLine 48: same.\r\nLine 50: Application API Monitoring section seems more appropriate as the top section.\r\nLine 53: \"a very rough impression\" -- I removed\r\nLine 55: \"ZMON gathers the data by querying a JSON endpoint adhering to the DropWizard metrics standard with some convention on the naming of timers. But basically on timer per API PATH and status code.\" Unclear; do you mean that the JSON endpoint adheres to the DropWizard metrics standard, or that *ZMON* adheres ...? The second sentence isn't a sentence. :)","distinct":false,"url":"https://api.github.com/repos/zalando-zmon/zmon-docs/commits/949b7c47b1c8083a75fc0c162d3e981b6f074a07"},{"sha":"cf58499fcf44af76107f3cf2eb6fcfcf0226413d","author":{"name":"Lauri at Zalando","email":"9641c2d2e8991bfe843675a36a1b4bfcbe3ffa3d@gmail.com"},"message":"Update monitoringonaws.rst","distinct":false,"url":"https://api.github.com/repos/zalando-zmon/zmon-docs/commits/cf58499fcf44af76107f3cf2eb6fcfcf0226413d"},{"sha":"3e32c9e48f97fbf76f641330bda75d966c0adeeb","author":{"name":"Henning Jacobs","email":"0449eeec675029bab4c2485a8adb97ea0a8d3e41@jacobs1.de"},"message":"Merge pull request #15 from zalando-zmon/LappleApple-patch-2\n\nUpdate monitoringonaws.rst","distinct":true,"url":"https://api.github.com/repos/zalando-zmon/zmon-docs/commits/3e32c9e48f97fbf76f641330bda75d966c0adeeb"}]}
{"actor":{"display_login":"hjacobs"}}