)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22042,"name":"Chris Smart","email":"distroguy@gmail.com","username":"csmart"},"change_message_id":"5361516345346d04936e49c68366721c2e4ee471","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor \"get_instance_info\" gathering"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change re-factors the \"get_instance_info\" function so that its"},{"line_number":10,"context_line":"logging everything to a file instead of stdout. This will help"},{"line_number":11,"context_line":"folks with debugging the gate. The function was logging most of"},{"line_number":12,"context_line":"important data to stdout which is only trapped by the main"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_ea796330","line":9,"updated":"2016-07-27 05:20:10.000000000","message":"Nit: Apologies if this is too picky (feel free to let me know if it is), should this be \"it\u0027s\" ?","commit_id":"416c9ea38a7c0dd371d4ea038b11d620b14d9061"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"818ff9cad26c2b2547ba6455e5d83d7a996750e4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor \"get_instance_info\" gathering"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change re-factors the \"get_instance_info\" function so that its"},{"line_number":10,"context_line":"logging everything to a file instead of stdout. This will help"},{"line_number":11,"context_line":"folks with debugging the gate. The function was logging most of"},{"line_number":12,"context_line":"important data to stdout which is only trapped by the main"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bacf61ea_3eac16ed","line":9,"in_reply_to":"dada55a8_ea796330","updated":"2016-07-27 17:47:24.000000000","message":"Not too nitpicky at all. /me will fix.","commit_id":"416c9ea38a7c0dd371d4ea038b11d620b14d9061"}],"scripts/scripts-library.sh":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"2a5c0efde32dde5f6733418f934a6e8c2083a5e9","unresolved":false,"context_lines":[{"line_number":183,"context_line":"    \"/openstack/log/instance-info/host_lxc_config_info_${TS}.log\" || true"},{"line_number":184,"context_line":"  (iptables -vnL \u0026\u0026 iptables -t nat -vnL \u0026\u0026 iptables -t mangle -vnL) \u003e \\"},{"line_number":185,"context_line":"    \"/openstack/log/instance-info/host_firewall_info_${TS}.log\" || true"},{"line_number":186,"context_line":"  ANSIBLE_HOST_KEY_CHECKING\u003dFalse \\"},{"line_number":187,"context_line":"    ansible -i \"localhost,\" localhost -m setup \u003e \\"},{"line_number":188,"context_line":"      \"/openstack/log/instance-info/host_system_info_${TS}.log\" || true"},{"line_number":189,"context_line":"  get_repos_info \u003e \\"},{"line_number":190,"context_line":"    \"/openstack/log/instance-info/host_repo_info_${TS}.log\" || true"},{"line_number":191,"context_line":"  dpkg-query --list \u003e \\"}],"source_content_type":"text/x-sh","patch_set":1,"id":"bacf61ea_cbf709f4","line":188,"range":{"start_line":186,"start_character":0,"end_line":188,"end_character":71},"updated":"2016-07-27 14:15:37.000000000","message":"The down side to this is that we get no data if ansible is not yet installed. Having this information prior to ansible being installed has been useful in the past. That said, it\u0027s a rare thing so I can live with this.","commit_id":"416c9ea38a7c0dd371d4ea038b11d620b14d9061"},{"author":{"_account_id":22042,"name":"Chris Smart","email":"distroguy@gmail.com","username":"csmart"},"change_message_id":"5361516345346d04936e49c68366721c2e4ee471","unresolved":false,"context_lines":[{"line_number":188,"context_line":"      \"/openstack/log/instance-info/host_system_info_${TS}.log\" || true"},{"line_number":189,"context_line":"  get_repos_info \u003e \\"},{"line_number":190,"context_line":"    \"/openstack/log/instance-info/host_repo_info_${TS}.log\" || true"},{"line_number":191,"context_line":"  dpkg-query --list \u003e \\"},{"line_number":192,"context_line":"    \"/openstack/log/instance-info/host_packages_info_${TS}.log\" || true"},{"line_number":193,"context_line":"}"},{"line_number":194,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"dada55a8_551c9469","line":191,"updated":"2016-07-27 05:20:10.000000000","message":"Nit: Will this exist on non-deb machines? Should we make this check for distro or the command itself? (We could run rpm or yum/dnf commands on an rpm box, I guess.)","commit_id":"416c9ea38a7c0dd371d4ea038b11d620b14d9061"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"818ff9cad26c2b2547ba6455e5d83d7a996750e4","unresolved":false,"context_lines":[{"line_number":188,"context_line":"      \"/openstack/log/instance-info/host_system_info_${TS}.log\" || true"},{"line_number":189,"context_line":"  get_repos_info \u003e \\"},{"line_number":190,"context_line":"    \"/openstack/log/instance-info/host_repo_info_${TS}.log\" || true"},{"line_number":191,"context_line":"  dpkg-query --list \u003e \\"},{"line_number":192,"context_line":"    \"/openstack/log/instance-info/host_packages_info_${TS}.log\" || true"},{"line_number":193,"context_line":"}"},{"line_number":194,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"bacf61ea_beeda627","line":191,"in_reply_to":"dada55a8_551c9469","updated":"2016-07-27 17:47:24.000000000","message":"Done","commit_id":"416c9ea38a7c0dd371d4ea038b11d620b14d9061"}]}
