)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"cf1fcf215345b09ba400c87fd06ac590b32051dd","unresolved":false,"context_lines":[{"line_number":11,"context_line":"However, sometimes the check failed because the update could not make"},{"line_number":12,"context_line":"it before the check. This reason is not concrete yet, but this problem"},{"line_number":13,"context_line":"happens three times in 60 minutes now."},{"line_number":14,"context_line":"So we need to solve this problem soon for smooth development, and this"},{"line_number":15,"context_line":"patch adds timeout method for alternative dns part."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I0cb98ff401b526c24388ea440e5629a8ebc925eb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a930d6b_7e355045","line":14,"updated":"2015-01-21 07:46:40.000000000","message":"This problem was continued to happen these days and made the development not-smooth.\nThen, I don\u0027t think now is a good time to unskip it again.\n\nMaybe we need a week at least before enabling the test again because of refreshing Logstash.","commit_id":"b308a063e79377b2aaf58b4a0c9a8d27ea193f42"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"2b1ecdbf676b951ed7e648d5baa46769a49fe07a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"it before the check. This reason is not concrete yet, but this problem"},{"line_number":13,"context_line":"happens three times in 60 minutes now."},{"line_number":14,"context_line":"So we need to solve this problem soon for smooth development, and this"},{"line_number":15,"context_line":"patch adds timeout method for alternative dns part."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I0cb98ff401b526c24388ea440e5629a8ebc925eb"},{"line_number":18,"context_line":"Closes-Bug: #1412325"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a930d6b_9e9e7c55","line":15,"updated":"2015-01-21 07:37:55.000000000","message":"Not knowing the root cause and adding timeouts doesn\u0027t sound like a reassuring solution to me.","commit_id":"b308a063e79377b2aaf58b4a0c9a8d27ea193f42"}],"tempest/scenario/test_network_basic_ops.py":[{"author":{"_account_id":1420,"name":"Adam Gandelman","email":"adamg@ubuntu.com","username":"gandelman-a"},"change_message_id":"85a26283000ae9f06ef543ea04efef7f08ccd1bb","unresolved":false,"context_lines":[{"line_number":470,"context_line":"        # Timeouts are defined as suggested by salvatore-orlando in"},{"line_number":471,"context_line":"        # https://bugs.launchpad.net/neutron/+bug/1412325/comments/3"},{"line_number":472,"context_line":"        renew_delay \u003d 5"},{"line_number":473,"context_line":"        renew_timeout \u003d 15"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self._setup_network_and_servers(dns_nameservers\u003d[initial_dns_server])"},{"line_number":476,"context_line":"        self.check_public_network_connectivity(should_connect\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_0113f0b8","line":473,"updated":"2015-01-22 00:04:55.000000000","message":"Id prefer these were configurable.  Can we just use existing CONF.network.build_timeout (default: 300) + CONF.network.build_interval (default: 1) ?  These are used for waits on other neutron async actions (ie, firewall creation)","commit_id":"b308a063e79377b2aaf58b4a0c9a8d27ea193f42"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2ffb6ae8eb9b173d9e49e182daf2060f14a194fe","unresolved":false,"context_lines":[{"line_number":491,"context_line":"            \"\"\"Server needs to renew its dhcp lease in order to get the new dns"},{"line_number":492,"context_line":"            definitions from subnet"},{"line_number":493,"context_line":"            NOTE(amuller): we are renewing the lease as part of the retry"},{"line_number":494,"context_line":"            becuase Neutron updats dnsmasq asynchronously after the"},{"line_number":495,"context_line":"            subnet-update API call returns."},{"line_number":496,"context_line":"            \"\"\""},{"line_number":497,"context_line":"            ssh_client.renew_lease(fixed_ip\u003dfloating_ip[\u0027fixed_ip_address\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_efe843b6","line":494,"updated":"2015-02-03 17:34:27.000000000","message":"Two typos in this single line...","commit_id":"d255bb98e6b67d077f329520c000c651d6165aa6"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"df802c3322d61501dc58405f9271bbff40c44b94","unresolved":false,"context_lines":[{"line_number":498,"context_line":"            try:"},{"line_number":499,"context_line":"                self._check_dns_server(ssh_client, [alt_dns_server])"},{"line_number":500,"context_line":"            except Exception as exc:"},{"line_number":501,"context_line":"                LOG.exception(exc)"},{"line_number":502,"context_line":"                return False"},{"line_number":503,"context_line":"            return True"},{"line_number":504,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_d06240ac","line":501,"updated":"2015-02-03 17:44:55.000000000","message":"The log level is too high. \n\nDuring my ran I got a couple of stack trace :\nMismatchError: set([\u00279.8.7.6\u0027]) !\u003d set([\u00271.2.3.4\u0027]): Looking for servers: [\u00279.8.7.6\u0027]. Retrieved DNS nameservers: [\u00271.2.3.4\u0027] From host: 172.24.4.4.\n\nAfter a dozen of seconds or so (and several similar stack traces), dns were indeed updated and the test could continue.\n\nMy point is, don\u0027t log at exception if some failures are expected.\n\n(Plus, log.exception(exc) doesn\"t make sense because log.exception will always append the exception to the log message)","commit_id":"d255bb98e6b67d077f329520c000c651d6165aa6"}]}