)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a42172ec59f327ceb6f5b7c1373000deb3df0077","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"xenapi: separate disk for cinder volumes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Some kernels has lockup issues while serving cinder volumes from"},{"line_number":10,"context_line":"file-loopback-lvm kind of cinder setup. This patch provides a way to"},{"line_number":11,"context_line":"create an additional drive to the OpenStack VM, and use that to store"},{"line_number":12,"context_line":"cinder volumes, thus eliminating the lockup issue. It will help to test"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"AAAAPX%2F%2FzPQ%3D","line":9,"updated":"2013-06-10 12:22:15.000000000","message":"have, not has","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a42172ec59f327ceb6f5b7c1373000deb3df0077","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Some kernels has lockup issues while serving cinder volumes from"},{"line_number":10,"context_line":"file-loopback-lvm kind of cinder setup. This patch provides a way to"},{"line_number":11,"context_line":"create an additional drive to the OpenStack VM, and use that to store"},{"line_number":12,"context_line":"cinder volumes, thus eliminating the lockup issue. It will help to test"},{"line_number":13,"context_line":"XenServer. Citrix is using this approach on its internal CI system to"},{"line_number":14,"context_line":"run tests against OpenStack. This is a workaround for this bug:"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"AAAAPX%2F%2FzPE%3D","line":12,"updated":"2013-06-10 12:22:15.000000000","message":"maybe, \"help when testing XenServer\"?","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"53aa0331cf128df15ed16610fe45b2d8b5f454bc","unresolved":false,"context_lines":[{"line_number":14,"context_line":"system to run tests against OpenStack. This is a workaround for this"},{"line_number":15,"context_line":"bug:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  https://bugs.launchpad.net/cinder/+bug/1023755"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Related to blueprint xenapi-devstack-cleanup"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"AAAAPX%2F%2Fp10%3D","line":17,"updated":"2013-06-13 12:27:20.000000000","message":"Technically the commit message should be readable even if that link stops working.","commit_id":"2b6c8df7cb572a5a12972aa489d814016ed27dbb"}],"tools/xen/build_xva.sh":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a42172ec59f327ceb6f5b7c1373000deb3df0077","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    disk_for_volumes\u003d\\$(sed -e \u0027s/.* disk_for_volumes\u003d\\([[:alpha:]]*\\).*$/\\1/g\u0027 /proc/cmdline)"},{"line_number":146,"context_line":"    sudo vgremove stack-volumes || true"},{"line_number":147,"context_line":"    sudo pvcreate /dev/\\$disk_for_volumes"},{"line_number":148,"context_line":"    sudo vgcreate stack-volumes /dev/\\$disk_for_volumes"},{"line_number":149,"context_line":"fi"},{"line_number":150,"context_line":"VIRT_DRIVER\u003dxenserver FORCE\u003dyes MULTI_HOST\u003d$MULTI_HOST HOST_IP_IFACE\u003d$HOST_IP_IFACE $STACKSH_PARAMS ./stack.sh"},{"line_number":151,"context_line":"EOF"}],"source_content_type":"text/x-sh","patch_set":11,"id":"AAAAPX%2F%2FzNo%3D","line":148,"updated":"2013-06-10 12:22:15.000000000","message":"While this works, it seems a little messy.\n\nIs it possible to integrate better with the current Cinder lvm configuration scripts?\n\nNo is a perfectly acceptable answer :)","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"277f5be52c98fd8c5536a17daef74a4e6b9b293f","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    disk_for_volumes\u003d\\$(sed -e \u0027s/.* disk_for_volumes\u003d\\([[:alpha:]]*\\).*$/\\1/g\u0027 /proc/cmdline)"},{"line_number":146,"context_line":"    sudo vgremove stack-volumes || true"},{"line_number":147,"context_line":"    sudo pvcreate /dev/\\$disk_for_volumes"},{"line_number":148,"context_line":"    sudo vgcreate stack-volumes /dev/\\$disk_for_volumes"},{"line_number":149,"context_line":"fi"},{"line_number":150,"context_line":"VIRT_DRIVER\u003dxenserver FORCE\u003dyes MULTI_HOST\u003d$MULTI_HOST HOST_IP_IFACE\u003d$HOST_IP_IFACE $STACKSH_PARAMS ./stack.sh"},{"line_number":151,"context_line":"EOF"}],"source_content_type":"text/x-sh","patch_set":11,"id":"AAAAPX%2F%2FvYo%3D","line":148,"in_reply_to":"AAAAPX%2F%2FzNo%3D","updated":"2013-06-11 16:13:26.000000000","message":"The other route would be to make it possible for cinder to have a device specified, and use that instead of file + loopback. With that, we could get rid of this magic. I agree, it\u0027s a bit dodgy. Let me see what would the cinder version look like...","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"}],"tools/xen/xenrc":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a42172ec59f327ceb6f5b7c1373000deb3df0077","unresolved":false,"context_lines":[{"line_number":80,"context_line":"# OpenStack VM, and create a volume group stack-volumes. By using this,"},{"line_number":81,"context_line":"# you can avoid kernel lockups:"},{"line_number":82,"context_line":"#   https://bugs.launchpad.net/cinder/+bug/1023755"},{"line_number":83,"context_line":"XEN_CREATE_DISK_FOR_VOLUMES\u003d${XEN_CREATE_DISK_FOR_VOLUMES:-False}"},{"line_number":84,"context_line":"XEN_DISK_SIZE_FOR_VOLUMES\u003d\"10GiB\""},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"source ../../stackrc"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"AAAAPX%2F%2FzO0%3D","line":83,"updated":"2013-06-10 12:22:15.000000000","message":"Given cinder is used by default, maybe go for True? Or maybe default to True if cinder is in the services list?","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"277f5be52c98fd8c5536a17daef74a4e6b9b293f","unresolved":false,"context_lines":[{"line_number":80,"context_line":"# OpenStack VM, and create a volume group stack-volumes. By using this,"},{"line_number":81,"context_line":"# you can avoid kernel lockups:"},{"line_number":82,"context_line":"#   https://bugs.launchpad.net/cinder/+bug/1023755"},{"line_number":83,"context_line":"XEN_CREATE_DISK_FOR_VOLUMES\u003d${XEN_CREATE_DISK_FOR_VOLUMES:-False}"},{"line_number":84,"context_line":"XEN_DISK_SIZE_FOR_VOLUMES\u003d\"10GiB\""},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"source ../../stackrc"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"AAAAPX%2F%2FvYM%3D","line":83,"in_reply_to":"AAAAPX%2F%2FzO0%3D","updated":"2013-06-11 16:13:26.000000000","message":"Yeah, let\u0027s go with True as a default, and create the disk only, if cinder is enabled.","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a42172ec59f327ceb6f5b7c1373000deb3df0077","unresolved":false,"context_lines":[{"line_number":81,"context_line":"# you can avoid kernel lockups:"},{"line_number":82,"context_line":"#   https://bugs.launchpad.net/cinder/+bug/1023755"},{"line_number":83,"context_line":"XEN_CREATE_DISK_FOR_VOLUMES\u003d${XEN_CREATE_DISK_FOR_VOLUMES:-False}"},{"line_number":84,"context_line":"XEN_DISK_SIZE_FOR_VOLUMES\u003d\"10GiB\""},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"source ../../stackrc"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"AAAAPX%2F%2FzOY%3D","line":84,"updated":"2013-06-10 12:22:15.000000000","message":"Not sure this is consistent with settings like: OSDOMU_VDI_GB\n\nIf this is on by default, might be worth reducing OSDOMU_VDI_GB?","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"277f5be52c98fd8c5536a17daef74a4e6b9b293f","unresolved":false,"context_lines":[{"line_number":81,"context_line":"# you can avoid kernel lockups:"},{"line_number":82,"context_line":"#   https://bugs.launchpad.net/cinder/+bug/1023755"},{"line_number":83,"context_line":"XEN_CREATE_DISK_FOR_VOLUMES\u003d${XEN_CREATE_DISK_FOR_VOLUMES:-False}"},{"line_number":84,"context_line":"XEN_DISK_SIZE_FOR_VOLUMES\u003d\"10GiB\""},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"source ../../stackrc"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"AAAAPX%2F%2FvYg%3D","line":84,"in_reply_to":"AAAAPX%2F%2FzOY%3D","updated":"2013-06-11 16:13:26.000000000","message":"Okay, let\u0027s try another way...","commit_id":"775697d796ac52829f4000fbd058638d7b476fe7"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"96948f824cfbfea3ea24d6f44bd98b520df7029f","unresolved":false,"context_lines":[{"line_number":76,"context_line":"UBUNTU_INST_NETMASK\u003d\"\""},{"line_number":77,"context_line":"UBUNTU_INST_GATEWAY\u003d\"\""},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"# Create a separate virtual disk for Cinder volumes By using this, you can"},{"line_number":80,"context_line":"# avoid kernel lockups:"},{"line_number":81,"context_line":"#   https://bugs.launchpad.net/cinder/+bug/1023755"},{"line_number":82,"context_line":"XEN_CREATE_DISK_FOR_VOLUMES\u003d${XEN_CREATE_DISK_FOR_VOLUMES:-True}"}],"source_content_type":"application/x-shellscript","patch_set":14,"id":"AAAAPX%2F%2Fp2Y%3D","line":79,"updated":"2013-06-13 12:26:18.000000000","message":"I guess you could put \"Note(matelakat)\" here","commit_id":"2b6c8df7cb572a5a12972aa489d814016ed27dbb"}]}