)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"0eb261adb92c062a297f3e258db07718f92a8233","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Saket B Sharma \u003csaket.bihari@atsgen.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-07-09 09:54:53 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for Xenial images"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"loci images still support xenial alongwith bionic adding"},{"line_number":10,"context_line":"the same support here as well"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id6e586f3cc234383dedfcd8402245bed7fe85dba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7faddb67_0bd6daf2","line":10,"range":{"start_line":7,"start_character":0,"end_line":10,"end_character":29},"updated":"2019-07-09 11:47:19.000000000","message":"Here is what I propose as commit message instead:\n```\nBuild Xenial OVS images\n\nAnother patch introduced the Bionic image building, but forgot the Xenial image building.\n\nThis adds support of Xenial using the same Dockerfile as Bionic, for re-use.\n```\n\nThis is more clear about the fact it\u0027s only about OVS, and clarify the intent of having a single file.","commit_id":"a98f6c696a082f74d5fcad88d3cb57d98d56d820"}],"zuul.d/openvswitch.yaml":[{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"7e31af6ad22553d0a46def836e94bc350ee0862b","unresolved":false,"context_lines":[{"line_number":64,"context_line":"          dockerfile: Dockerfile.ubuntu"},{"line_number":65,"context_line":"          tags:"},{"line_number":66,"context_line":"            - latest-ubuntu_xenial"},{"line_number":67,"context_line":"        - context: openvswitch"},{"line_number":68,"context_line":"          repository: openstackhelm/openvswitch"},{"line_number":69,"context_line":"          build_args:"},{"line_number":70,"context_line":"            - \"FROM\u003d\u0027docker.io/ubuntu:xenial\u0027\""},{"line_number":71,"context_line":"          dockerfile: Dockerfile.ubuntu-dpdk"},{"line_number":72,"context_line":"          tags:"},{"line_number":73,"context_line":"            - latest-ubuntu_xenial-dpdk"},{"line_number":74,"context_line":"    files: \u0026openvswitch_files"},{"line_number":75,"context_line":"      - openvswitch/.*"},{"line_number":76,"context_line":"      - zuul.d/openvswitch.yaml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_3a899ec9","line":73,"range":{"start_line":67,"start_character":8,"end_line":73,"end_character":39},"updated":"2019-07-11 01:06:18.000000000","message":"We may don\u0027t need xenial based ovs-dpdk image, as it\u0027s mentioned in the release note[1] that dpdk vhost-user needs ovs version of at least 2.6. While the xenial ovs version is 2.5.5 \n\n[1] https://docs.openstack.org/releasenotes/neutron/ocata.html","commit_id":"e5d16c670b00f78ac7544965f2af267c10e07055"},{"author":{"_account_id":30232,"name":"Prabhjot Singh Sethi","email":"prabhjot@atsgen.com","username":"psethi-ats"},"change_message_id":"8aa180299717f0181ba72d5360ca89d6fe33fd3c","unresolved":false,"context_lines":[{"line_number":64,"context_line":"          dockerfile: Dockerfile.ubuntu"},{"line_number":65,"context_line":"          tags:"},{"line_number":66,"context_line":"            - latest-ubuntu_xenial"},{"line_number":67,"context_line":"        - context: openvswitch"},{"line_number":68,"context_line":"          repository: openstackhelm/openvswitch"},{"line_number":69,"context_line":"          build_args:"},{"line_number":70,"context_line":"            - \"FROM\u003d\u0027docker.io/ubuntu:xenial\u0027\""},{"line_number":71,"context_line":"          dockerfile: Dockerfile.ubuntu-dpdk"},{"line_number":72,"context_line":"          tags:"},{"line_number":73,"context_line":"            - latest-ubuntu_xenial-dpdk"},{"line_number":74,"context_line":"    files: \u0026openvswitch_files"},{"line_number":75,"context_line":"      - openvswitch/.*"},{"line_number":76,"context_line":"      - zuul.d/openvswitch.yaml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_2332b6a8","line":73,"range":{"start_line":67,"start_character":8,"end_line":73,"end_character":39},"in_reply_to":"7faddb67_3a899ec9","updated":"2019-07-11 05:12:44.000000000","message":"What release note says is that reconnect feature for vhost-user will be missing, however support for vhost itself is available in 2.5.5.\n\ncopying the exact reno for reference.\n\"vhost-user reconnect requires dpdk 16.07 and qemu 2.7 and openvswitch 2.6 to function. if an older qemu is used, reconnect will not be available but vhost-user will still function.\"","commit_id":"e5d16c670b00f78ac7544965f2af267c10e07055"}]}
