)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"a4c4133d038152cd52a68d9ef5738eb17a3b4a7b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tools/Xen: failed to install domU in new XenServer"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Current install_os_domU.sh depends on some keywords which get changed"},{"line_number":10,"context_line":"in the next version XenServer. This patch is to make it to be more"},{"line_number":11,"context_line":"common."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba15a1d1_c28fba21","line":9,"updated":"2015-09-22 13:34:12.000000000","message":"I should have commented earlier, sorry.  Instead of \"some keywords\" please expand - e.g. \"Next XS upgrades to CentOS 7 which has a different format output for ifconfig\"","commit_id":"adf2670dbb1e709f3a166c886843b20a5a81ed00"},{"author":{"_account_id":17920,"name":"Jianghua Wang","email":"jianghua.wang@citrix.com","username":"wjhfresh"},"change_message_id":"d2494394e4ee0f1269f7eeb0b01337499a45d35a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tools/Xen: failed to install domU in new XenServer"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Current install_os_domU.sh depends on some keywords which get changed"},{"line_number":10,"context_line":"in the next version XenServer. This patch is to make it to be more"},{"line_number":11,"context_line":"common."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba15a1d1_8fc0e651","line":9,"in_reply_to":"ba15a1d1_c28fba21","updated":"2015-09-23 08:41:06.000000000","message":"thanks. I will update the commit message soon.","commit_id":"adf2670dbb1e709f3a166c886843b20a5a81ed00"}],"tools/xen/functions":[{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"ba3abaa3452a48674ba3972ec2146c4af807ca57","unresolved":false,"context_lines":[{"line_number":179,"context_line":"    local bridge_or_net_name"},{"line_number":180,"context_line":"    bridge_or_net_name\u003d$1"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    ifconfig $(bridge_for \"$bridge_or_net_name\") | grep \"inet\" | \\"},{"line_number":183,"context_line":"    grep -o \"[[:digit:]]*\\.[[:digit:]]*\\.[[:digit:]]*\\.[[:digit:]]*\" | head -1"},{"line_number":184,"context_line":"}"},{"line_number":185,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba15a1d1_f3d24bd0","line":182,"updated":"2015-09-18 10:28:42.000000000","message":"Since the format of ifconfig changes in XenServer, I wonder if we should just go back to using ip?\n\nip addr show $(bridge_for ...) then parse that would be more stable.","commit_id":"5763042c2ec0e892f43c4e5b281999d6ac0c7005"}],"tools/xen/install_os_domU.sh":[{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"ba3abaa3452a48674ba3972ec2146c4af807ca57","unresolved":false,"context_lines":[{"line_number":193,"context_line":"            TMP_DIR\u003d/tmp/temp.$RANDOM"},{"line_number":194,"context_line":"            mkdir -p $TMP_DIR"},{"line_number":195,"context_line":"            mount -o loop $TOOLS_ISO $TMP_DIR"},{"line_number":196,"context_line":"            DEB_FILE\u003d$(ls $TMP_DIR/Linux/*amd64.deb)"},{"line_number":197,"context_line":"            if [ -n \"$DEB_FILE\" ]; then"},{"line_number":198,"context_line":"                DEB_FILE\u003d$(ls $TMP_DIR/Linux/*all.deb)"},{"line_number":199,"context_line":"            fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"ba15a1d1_73df1bd6","line":196,"updated":"2015-09-18 10:28:42.000000000","message":"A comment here would be good.\n# Default to 64-bit deb if there is an architecture-specific one.  Otherwise use the architecture-independant deb.","commit_id":"5763042c2ec0e892f43c4e5b281999d6ac0c7005"}]}