)]}'
{"trove/guestagent/common/operating_system.py":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"cf0c359135b271cb7e51586c164fccc6624ef310","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    Retrieves IP address which assigned to given network interface"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    @parameter ifname network interface (ethX, wlanX, etc.)"},{"line_number":92,"context_line":"    \"\"\""},{"line_number":93,"context_line":"    addresses_by_family \u003d netifaces.ifaddresses(ifname)[address_family]"},{"line_number":94,"context_line":"    return addresses_by_family.pop()[\u0027addr\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_ec8d4c36","line":91,"updated":"2014-08-12 09:47:14.000000000","message":"please consider adding a @parameter to document the \u0027address_family\u0027.","commit_id":"5f879886cb0f1ae5a89522566ea8782e808d3d63"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"686eae679fe391014543fd51fdadc2c6010610cf","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    Retrieves IP address which assigned to given network interface"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    @parameter ifname network interface (ethX, wlanX, etc.)"},{"line_number":92,"context_line":"    \"\"\""},{"line_number":93,"context_line":"    addresses_by_family \u003d netifaces.ifaddresses(ifname)[address_family]"},{"line_number":94,"context_line":"    return addresses_by_family.pop()[\u0027addr\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_825d64ec","line":91,"in_reply_to":"baada198_216106a2","updated":"2014-08-17 18:04:10.000000000","message":"Agreed, will put up a new patchset.","commit_id":"5f879886cb0f1ae5a89522566ea8782e808d3d63"},{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"e52c8398a5c9042c011c35a92ebc7ab13758da93","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    Retrieves IP address which assigned to given network interface"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    @parameter ifname network interface (ethX, wlanX, etc.)"},{"line_number":92,"context_line":"    \"\"\""},{"line_number":93,"context_line":"    addresses_by_family \u003d netifaces.ifaddresses(ifname)[address_family]"},{"line_number":94,"context_line":"    return addresses_by_family.pop()[\u0027addr\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_216106a2","line":91,"in_reply_to":"baada198_ec8d4c36","updated":"2014-08-12 12:26:07.000000000","message":"Agree with boden. New parameter is added so it should be documented.","commit_id":"5f879886cb0f1ae5a89522566ea8782e808d3d63"}]}