)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"64318ff2fa3f8321f856581d96cee8f94cd3be15","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"gate_hook: Compile ovs only with supported kernels"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The intention of this patch is not to brake current compilations on"},{"line_number":10,"context_line":"Xenial boxes that run with 4.4 kernel which is not supported by OVS"},{"line_number":11,"context_line":"openvswitch kernel module. After we are confident to switch master"},{"line_number":12,"context_line":"branch to be running on Xenial only, we will get rid of the compilation"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a89bdaa_eb2e2af9","line":9,"updated":"2016-09-06 10:27:20.000000000","message":"break","commit_id":"3479803c13189456bf672fc541bc20d62e8ff679"}],"devstack/lib/ovs":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"356d4d4749a4160f640ead9e0aec81f80c55acab","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    fi"},{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# is_kernel_supported() - OVS supports only kernel up to 4.3.x"},{"line_number":35,"context_line":"function is_kernel_supported {"},{"line_number":36,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":37,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"9a89bdaa_58d55841","line":34,"updated":"2016-09-05 10:01:02.000000000","message":"you mean \u0027*from-source* OVS? The system package should work with newer kernel still.","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"1b1b84e69511485c1557a4c497d4dac7614d4935","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    fi"},{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# is_kernel_supported() - OVS supports only kernel up to 4.3.x"},{"line_number":35,"context_line":"function is_kernel_supported {"},{"line_number":36,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":37,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"9a89bdaa_0ebc6a04","line":34,"in_reply_to":"9a89bdaa_58d55841","updated":"2016-09-05 10:35:21.000000000","message":"Yep, I\u0027ll add from source","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"356d4d4749a4160f640ead9e0aec81f80c55acab","unresolved":false,"context_lines":[{"line_number":35,"context_line":"function is_kernel_supported {"},{"line_number":36,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":37,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"},{"line_number":38,"context_line":"    if [ $major -le 4 -a $minor -le 3 ]; then"},{"line_number":39,"context_line":"        return 0"},{"line_number":40,"context_line":"    fi"},{"line_number":41,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"9a89bdaa_78585cfb","line":38,"updated":"2016-09-05 10:01:02.000000000","message":"So when the kernel 5.0 will ship, we will attempt compilation?","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"1b1b84e69511485c1557a4c497d4dac7614d4935","unresolved":false,"context_lines":[{"line_number":35,"context_line":"function is_kernel_supported {"},{"line_number":36,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":37,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"},{"line_number":38,"context_line":"    if [ $major -le 4 -a $minor -le 3 ]; then"},{"line_number":39,"context_line":"        return 0"},{"line_number":40,"context_line":"    fi"},{"line_number":41,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"9a89bdaa_aedd3e72","line":38,"in_reply_to":"9a89bdaa_78585cfb","updated":"2016-09-05 10:35:21.000000000","message":"No, if major \u003d\u003d 5 then this statement is False so we return 1 and won\u0027t attempt to compile.","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"2f8d9e502ebe3c01a86d25eb1c47ac4a4f6dbe5a","unresolved":false,"context_lines":[{"line_number":35,"context_line":"function is_kernel_supported {"},{"line_number":36,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":37,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"},{"line_number":38,"context_line":"    if [ $major -le 4 -a $minor -le 3 ]; then"},{"line_number":39,"context_line":"        return 0"},{"line_number":40,"context_line":"    fi"},{"line_number":41,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"9a89bdaa_4eccb235","line":38,"in_reply_to":"9a89bdaa_aedd3e72","updated":"2016-09-05 10:42:36.000000000","message":"Oh right you are.","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"e7f77749c0c9a6fa956cab82460b8b9cd34c59df","unresolved":false,"context_lines":[{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# is_kernel_supported() - compilation of openvswitch kernel module supports"},{"line_number":35,"context_line":"                          only kernels up to 4.3.x, function determines whether"},{"line_number":36,"context_line":"                          compilation from source will succeed on this machine"},{"line_number":37,"context_line":"function is_kernel_supported {"},{"line_number":38,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":39,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"9a89bdaa_f9262415","line":36,"range":{"start_line":35,"start_character":0,"end_line":36,"end_character":78},"updated":"2016-09-05 14:10:59.000000000","message":"you need to comment those lines","commit_id":"672682713587b7eec4e3d799c36948209fad290c"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"476cabc1ba218e9bfbdacfb344e58ef31eebeb06","unresolved":false,"context_lines":[{"line_number":32,"context_line":"}"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# is_kernel_supported() - compilation of openvswitch kernel module supports"},{"line_number":35,"context_line":"                          only kernels up to 4.3.x, function determines whether"},{"line_number":36,"context_line":"                          compilation from source will succeed on this machine"},{"line_number":37,"context_line":"function is_kernel_supported {"},{"line_number":38,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":39,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"9a89bdaa_4cfaf043","line":36,"range":{"start_line":35,"start_character":0,"end_line":36,"end_character":78},"in_reply_to":"9a89bdaa_f9262415","updated":"2016-09-05 14:34:10.000000000","message":"yeah :D Thanks","commit_id":"672682713587b7eec4e3d799c36948209fad290c"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"738e73fb21531446baafc7c3418250cacd5d89d1","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# is_kernel_supported() - compilation of openvswitch kernel module supports"},{"line_number":35,"context_line":"#                         only kernels up to 4.3.x, function determines whether"},{"line_number":36,"context_line":"#                         compilation from source will succeed on this machine"},{"line_number":37,"context_line":"function is_kernel_supported {"},{"line_number":38,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":39,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"9a89bdaa_1cce2d5a","line":36,"updated":"2016-09-06 12:38:40.000000000","message":"This is only true of the branch used in your gate_hook, not the module in general.  I\u0027d at least update the comment, or maybe take an argument that specifies which kernel version to check against.\n\nAnother thing you can do is just run configure for ovs, which will fail if the kernel is not compatible.  You could handle that more gracefully.","commit_id":"5f564594d8d82ffbd9fee7a7a2fa6b5bd2971598"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"b744fefa5ae45bc0a2d0229c0dd704028105756b","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# is_kernel_supported() - compilation of openvswitch kernel module supports"},{"line_number":35,"context_line":"#                         only kernels up to 4.3.x, function determines whether"},{"line_number":36,"context_line":"#                         compilation from source will succeed on this machine"},{"line_number":37,"context_line":"function is_kernel_supported {"},{"line_number":38,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":39,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"9a89bdaa_6d0065ab","line":36,"in_reply_to":"9a89bdaa_1cce2d5a","updated":"2016-09-06 13:23:00.000000000","message":"Done","commit_id":"5f564594d8d82ffbd9fee7a7a2fa6b5bd2971598"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"e3217c2ca6a5bff7ea96b915f879a2d85620834d","unresolved":false,"context_lines":[{"line_number":38,"context_line":"function is_kernel_supported_for_ovs25 {"},{"line_number":39,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":40,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"},{"line_number":41,"context_line":"    if [ $major -le 4 -a $minor -le 3 ]; then"},{"line_number":42,"context_line":"        return 0"},{"line_number":43,"context_line":"    fi"},{"line_number":44,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"9a89bdaa_2eba20c1","line":41,"updated":"2016-09-09 09:24:30.000000000","message":"fyi this check is wrong, should be -o not -a: https://review.openstack.org/#/c/367851/","commit_id":"fa5e23cbe62424896ce200882e2d255b66ec7abe"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"f836092525c0b0deaae1115fdde92d1279feba20","unresolved":false,"context_lines":[{"line_number":38,"context_line":"function is_kernel_supported_for_ovs25 {"},{"line_number":39,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":40,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"},{"line_number":41,"context_line":"    if [ $major -le 4 -a $minor -le 3 ]; then"},{"line_number":42,"context_line":"        return 0"},{"line_number":43,"context_line":"    fi"},{"line_number":44,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"9a89bdaa_496d9ecb","line":41,"in_reply_to":"9a89bdaa_2eba20c1","updated":"2016-09-09 09:34:47.000000000","message":"let me respin this patch","commit_id":"fa5e23cbe62424896ce200882e2d255b66ec7abe"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"d8afdeeff1adc33a7f0e44eca4b9c14c24b866db","unresolved":false,"context_lines":[{"line_number":38,"context_line":"function is_kernel_supported_for_ovs25 {"},{"line_number":39,"context_line":"    major\u003d$(uname -r | cut -d\\. -f 1)"},{"line_number":40,"context_line":"    minor\u003d$(uname -r | cut -d\\. -f 2)"},{"line_number":41,"context_line":"    if [ $major -le 4 -a $minor -le 3 ]; then"},{"line_number":42,"context_line":"        return 0"},{"line_number":43,"context_line":"    fi"},{"line_number":44,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"9a89bdaa_9bce07b3","line":41,"in_reply_to":"9a89bdaa_496d9ecb","updated":"2016-09-09 10:25:53.000000000","message":"There was some humour virus outbreak among neutron devs today.","commit_id":"fa5e23cbe62424896ce200882e2d255b66ec7abe"}],"neutron/tests/contrib/gate_hook.sh":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"356d4d4749a4160f640ead9e0aec81f80c55acab","unresolved":false,"context_lines":[{"line_number":45,"context_line":"    configure_host_for_func_testing"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    if is_kernel_supported; then"},{"line_number":48,"context_line":"        # The OVS_BRANCH variable is used by git checkout. In the case below,"},{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"},{"line_number":51,"context_line":"        # https://github.com/openvswitch/ovs/commit/741f47cf35df2bfc7811b2cff75c9bb8d05fd26f"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a89bdaa_d8e4080f","line":48,"updated":"2016-09-05 10:01:02.000000000","message":"I would hope at least in gate jobs we would be more explicit about which version of ovs we run there. Like a envvar to set to disable compilation? Otherwise external events like change in distro shipped kernel version may affect our gate job without us knowing.","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"2f8d9e502ebe3c01a86d25eb1c47ac4a4f6dbe5a","unresolved":false,"context_lines":[{"line_number":45,"context_line":"    configure_host_for_func_testing"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    if is_kernel_supported; then"},{"line_number":48,"context_line":"        # The OVS_BRANCH variable is used by git checkout. In the case below,"},{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"},{"line_number":51,"context_line":"        # https://github.com/openvswitch/ovs/commit/741f47cf35df2bfc7811b2cff75c9bb8d05fd26f"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a89bdaa_8eedda8d","line":48,"in_reply_to":"9a89bdaa_2ea12eaa","updated":"2016-09-05 10:42:36.000000000","message":"Ack. Assuming it will go away once we switch to Xenial, I am good.","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"1b1b84e69511485c1557a4c497d4dac7614d4935","unresolved":false,"context_lines":[{"line_number":45,"context_line":"    configure_host_for_func_testing"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    if is_kernel_supported; then"},{"line_number":48,"context_line":"        # The OVS_BRANCH variable is used by git checkout. In the case below,"},{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"},{"line_number":51,"context_line":"        # https://github.com/openvswitch/ovs/commit/741f47cf35df2bfc7811b2cff75c9bb8d05fd26f"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a89bdaa_2ea12eaa","line":48,"in_reply_to":"9a89bdaa_d8e4080f","updated":"2016-09-05 10:35:21.000000000","message":"Ideally we won\u0027t compile OVS unless there is a feature requiring bleeding edge. I take this change only for trusty-\u003exenial switch as we currently we have a chicken-egg problem.\n\nAlternatively we can remove the compilation first, make Xenial jobs stable and then switch. But as the time window of the switch is unknown, it introduces a potential long time for running functional jobs without 2.5.","commit_id":"828778e6e71d9f4a9fb3e00c926e47fa7e801783"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"9cc69041a40b73cc5c47b6dabc2244d9a1c6bb19","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    configure_host_for_func_testing"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    if is_kernel_supported; then"},{"line_number":48,"context_line":"        # The OVS_BRANCH variable is used by git checkout. In the case below,"},{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"}],"source_content_type":"text/x-sh","patch_set":5,"id":"9a89bdaa_bc9aa161","line":47,"updated":"2016-09-06 12:37:09.000000000","message":"Your is_kernel_supported code makes an assumption about which branch is being used.  This will break OVN, which uses master.","commit_id":"5f564594d8d82ffbd9fee7a7a2fa6b5bd2971598"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"738e73fb21531446baafc7c3418250cacd5d89d1","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    configure_host_for_func_testing"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    if is_kernel_supported; then"},{"line_number":48,"context_line":"        # The OVS_BRANCH variable is used by git checkout. In the case below,"},{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"}],"source_content_type":"text/x-sh","patch_set":5,"id":"9a89bdaa_dcb4c5d2","line":47,"in_reply_to":"9a89bdaa_bc9aa161","updated":"2016-09-06 12:38:40.000000000","message":"Well, this is your gate_hook.sh, not something OVN uses, nevermind ....","commit_id":"5f564594d8d82ffbd9fee7a7a2fa6b5bd2971598"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"b5968708dcefec5bd32fc90a7d3246a93eb8789d","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"},{"line_number":51,"context_line":"        # https://github.com/openvswitch/ovs/commit/741f47cf35df2bfc7811b2cff75c9bb8d05fd26f"},{"line_number":52,"context_line":"        OVS_BRANCH\u003d042326c3fcf61e8638fa15926f984ce5ae142f4b"},{"line_number":53,"context_line":"        remove_ovs_packages"},{"line_number":54,"context_line":"        compile_ovs True /usr /var"},{"line_number":55,"context_line":"        start_new_ovs"}],"source_content_type":"text/x-sh","patch_set":5,"id":"9a89bdaa_01ffaf67","line":52,"updated":"2016-09-06 11:02:35.000000000","message":"should we not write this so that it can be overridden (but maybe that is not related ot this patch)","commit_id":"5f564594d8d82ffbd9fee7a7a2fa6b5bd2971598"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"c51e5a49d8a46169c30e051530b3935e257d7530","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        # we use a current (2016-08-19) HEAD commit from branch-2.5 that contains"},{"line_number":50,"context_line":"        # a fix for usage of VXLAN tunnels on a single node:"},{"line_number":51,"context_line":"        # https://github.com/openvswitch/ovs/commit/741f47cf35df2bfc7811b2cff75c9bb8d05fd26f"},{"line_number":52,"context_line":"        OVS_BRANCH\u003d042326c3fcf61e8638fa15926f984ce5ae142f4b"},{"line_number":53,"context_line":"        remove_ovs_packages"},{"line_number":54,"context_line":"        compile_ovs True /usr /var"},{"line_number":55,"context_line":"        start_new_ovs"}],"source_content_type":"text/x-sh","patch_set":5,"id":"9a89bdaa_81301f03","line":52,"in_reply_to":"9a89bdaa_01ffaf67","updated":"2016-09-06 11:08:19.000000000","message":"We still want to compile on Trusty from this commit hash","commit_id":"5f564594d8d82ffbd9fee7a7a2fa6b5bd2971598"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"f4d64530471d698502cc1eb7f763c1b4f02d96d8","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        # https://github.com/openvswitch/ovs/commit/741f47cf35df2bfc7811b2cff75c9bb8d05fd26f"},{"line_number":52,"context_line":"        OVS_BRANCH\u003d042326c3fcf61e8638fa15926f984ce5ae142f4b"},{"line_number":53,"context_line":"        remove_ovs_packages"},{"line_number":54,"context_line":"        compile_ovs True /usr /var"},{"line_number":55,"context_line":"        start_new_ovs"},{"line_number":56,"context_line":"    fi"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"9a89bdaa_222fd988","line":54,"range":{"start_line":54,"start_character":20,"end_line":54,"end_character":24},"updated":"2016-09-08 09:15:39.000000000","message":"I wonder whether we should just not compile the kernel part. On the other hand having user-space without kernel won\u0027t make much sense.","commit_id":"fa5e23cbe62424896ce200882e2d255b66ec7abe"}]}