{"push_id":1215080158,"size":3,"distinct_size":3,"ref":"refs/heads/5.8.x","head":"e2b78b621d81491e9e6aff968675323f062aa3dc","before":"7b8883aa16f78528c44eeb32f96d9201839c9cf9","commits":[{"sha":"bba38335ec4ff68a9c5ab97aa7ac0e9c53aac06d","author":{"name":"Massimiliano Pippi","email":"c5ab13aca8d97d3a7b9672a0e58b3c94e30050d0@datadoghq.com"},"message":"use a more reliable external resource","distinct":true,"url":"https://api.github.com/repos/DataDog/dd-agent/commits/bba38335ec4ff68a9c5ab97aa7ac0e9c53aac06d"},{"sha":"9640c1d7af603e25f6ee2907f8e396df839f0e5d","author":{"name":"Olivier Vielpeau","email":"e3b17ac2646dc415cb986eb71a3590835b2dac63@users.noreply.github.com"},"message":"[haproxy] Fix `KeyError` when an unknown status is found (#2681)\n\n* [haproxy] Fix `KeyError` when an unknown status is found\r\n\r\nAn unknown status (i.e. one that's not listed in\r\n`Services.ALL_STATUSES`) would make the check fail with a `KeyError`\r\nbecause that status wasn't in the initialized `dict` of statuses.\r\n\r\nThe check now sends unknown statuses as:\r\n* an `UNAVAILABLE` status if `collate_status_tags_per_host` is enabled,\r\nin order to keep the number of contexts low\r\n* the unknown status as-is if `collate_status_tags_per_host` is left\r\ndisabled, in order to give maximum visibility in the actual statuses\r\n\r\nThis commit also improves the parsing of status suffixes:\r\n* Introduce normalization of statuses to regard `UP`, `UP 1/2`,\r\nand `UP (agent)` statuses as the same `up` status. This fixes a\r\nregression.\r\n* Use the same logic for the `haproxy.backend_hosts` metric\r\n\r\nSee related documentation (thanks @alexism for the links):\r\nhttps://cbonte.github.io/haproxy-dconv/configuration-1.5.html#9.1 :\r\n\r\n`17. status [LFBS]: status (UP/DOWN/NOLB/MAINT/MAINT(via)...)`\r\n\r\nMore precisely, the haproxy code shows that 2 suffixes might be used: ` %d/%d` and ` (agent)`\r\nhttps://github.com/haproxy/haproxy/blob/a5de024d42c4113fc6e189ea1d0ba6335219e151/src/dumpstats.c#L4117-L4129\r\n\r\nMisc:\r\n* move the remaining mock test in `integration/` to `mock/`","distinct":true,"url":"https://api.github.com/repos/DataDog/dd-agent/commits/9640c1d7af603e25f6ee2907f8e396df839f0e5d"},{"sha":"e2b78b621d81491e9e6aff968675323f062aa3dc","author":{"name":"Quentin Madec","email":"7322734742b56be8896cd196de39bce96bd7765d@users.noreply.github.com"},"message":"[core] don't use docker hostname if it's a EC2 one (#2661)\n\nDocker info sometimes returns default EC2 hostnames and `get_hostname`\nwasn't checking whether it was possible to use the instance id.\nThis commits change this behavior by allowing `get_hostname` to check\nthe docker proposed hostname before using it.","distinct":true,"url":"https://api.github.com/repos/DataDog/dd-agent/commits/e2b78b621d81491e9e6aff968675323f062aa3dc"}]}
{"actor":{"display_login":"olivielpeau"}}