)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"b3941016f54b51b6305d320ed9d785def1f0dc16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6757396d_c9d7f67f","updated":"2021-12-08 11:19:21.000000000","message":"@Emilien, WDYT? IIRC you\u0027ve had some experience with changing defaults related to CPUset mappings in the past. Should we by default change nothing, to be on the safe side, or go with opinionated \u0027all\u0027 defaults?","commit_id":"5b9d3820a51b20edfebc6268eb229aa4806bfa76"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"fc496b8006d8697fe31e0ca0f8e17a6c9b073310","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"09641d4d_e73a4e03","in_reply_to":"6757396d_c9d7f67f","updated":"2021-12-08 13:48:04.000000000","message":"What\u0027s the default behavior here \"none\" or \"all\" ?  This value should contain the default behavior...  I guess we can tweak it so if it\u0027s empty, we don\u0027t even add it.","commit_id":"5b9d3820a51b20edfebc6268eb229aa4806bfa76"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ef3365e36444f4f22419fde4ebbeb64c66699f5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4ccb0130_b7c0f7a5","updated":"2021-12-08 14:16:08.000000000","message":"There has been a requirement to support it for other services i.e we had it for libvirt which was dropped in train. Probably better to have \u003cservice_name\u003e: \u003ccpus\u003e map rather than having parameter per service.","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"fe84d92a6442ac81281183023defb0e413760543","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8174a031_a54689a6","updated":"2021-12-08 14:05:38.000000000","message":"Would this be better ?","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"d4ecadf439347af7a7b907c343b98949f0bca3b2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fea03bb0_b4b951a3","in_reply_to":"4ccb0130_b7c0f7a5","updated":"2021-12-22 16:04:48.000000000","message":"Ok I\u0027m not sure how to do this one.","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"b322c097e5656f62976accd172bdafcf135060a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e3f4fa86_ca11271e","updated":"2021-12-22 16:01:06.000000000","message":"recheck\n","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"509a252bb19b3f83e8c9ffccc69308080f06335c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"399e10a6_5923a5f7","updated":"2022-01-05 06:22:34.000000000","message":"Its a nit, it is required to understand what the parameter accepts.","commit_id":"c5b0b1ffc4ab185eed4c7fe0509ad2624f830b29"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ee02d8229ba18ae614765290c543dfa701e37e9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"62389a5b_fe813741","updated":"2022-01-11 06:01:32.000000000","message":"recheck","commit_id":"a170d70e2a81cdc9db1681e4bf41b4e33d628653"}],"deployment/ovn/ovn-controller-container-puppet.yaml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"e3fec903595e447547dd34c78fc98e62022fe933","unresolved":true,"context_lines":[{"line_number":128,"context_line":"    default: \u0027\u0027"},{"line_number":129,"context_line":"    description: Override the private key size used when creating the"},{"line_number":130,"context_line":"                 certificate for this service"},{"line_number":131,"context_line":"  OVNContainerCpusetCpus:"},{"line_number":132,"context_line":"    description: Limit the specific CPUs or cores a container can use."},{"line_number":133,"context_line":"    type: string"},{"line_number":134,"context_line":"    default: \u0027all\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6e8a07fb_5b59f906","line":131,"updated":"2021-12-08 11:17:01.000000000","message":"I think this default \u0027all\u0027 change should be an opt-in, and controled by an extra flag (set to False) by default.","commit_id":"5b9d3820a51b20edfebc6268eb229aa4806bfa76"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ef3365e36444f4f22419fde4ebbeb64c66699f5e","unresolved":true,"context_lines":[{"line_number":321,"context_line":"                   - /run/openvswitch:/run/openvswitch:shared,z"},{"line_number":322,"context_line":"            environment:"},{"line_number":323,"context_line":"              TRIPLEO_DEPLOY_IDENTIFIER: {get_param: DeployIdentifier}"},{"line_number":324,"context_line":"          ozn_controller:"},{"line_number":325,"context_line":"            map_merge:"},{"line_number":326,"context_line":"              - start_order: 1"},{"line_number":327,"context_line":"                image: *ovn_controller_image"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5bf9db9c_da8b52ed","line":324,"range":{"start_line":324,"start_character":10,"end_line":324,"end_character":25},"updated":"2021-12-08 14:16:08.000000000","message":"typo","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"d4ecadf439347af7a7b907c343b98949f0bca3b2","unresolved":false,"context_lines":[{"line_number":321,"context_line":"                   - /run/openvswitch:/run/openvswitch:shared,z"},{"line_number":322,"context_line":"            environment:"},{"line_number":323,"context_line":"              TRIPLEO_DEPLOY_IDENTIFIER: {get_param: DeployIdentifier}"},{"line_number":324,"context_line":"          ozn_controller:"},{"line_number":325,"context_line":"            map_merge:"},{"line_number":326,"context_line":"              - start_order: 1"},{"line_number":327,"context_line":"                image: *ovn_controller_image"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"222c9d16_7c30d8f1","line":324,"range":{"start_line":324,"start_character":10,"end_line":324,"end_character":25},"in_reply_to":"5bf9db9c_da8b52ed","updated":"2021-12-22 16:04:48.000000000","message":"Ack","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ef3365e36444f4f22419fde4ebbeb64c66699f5e","unresolved":true,"context_lines":[{"line_number":322,"context_line":"            environment:"},{"line_number":323,"context_line":"              TRIPLEO_DEPLOY_IDENTIFIER: {get_param: DeployIdentifier}"},{"line_number":324,"context_line":"          ozn_controller:"},{"line_number":325,"context_line":"            map_merge:"},{"line_number":326,"context_line":"              - start_order: 1"},{"line_number":327,"context_line":"                image: *ovn_controller_image"},{"line_number":328,"context_line":"                net: host"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0ee76c85_d20c3c2a","line":325,"range":{"start_line":325,"start_character":1,"end_line":325,"end_character":22},"updated":"2021-12-08 14:16:08.000000000","message":"You don\u0027t need map_merge here..\ncpuset_cpus:\n  if:\n    - ovn_cpu_set\n    - {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"d4ecadf439347af7a7b907c343b98949f0bca3b2","unresolved":false,"context_lines":[{"line_number":322,"context_line":"            environment:"},{"line_number":323,"context_line":"              TRIPLEO_DEPLOY_IDENTIFIER: {get_param: DeployIdentifier}"},{"line_number":324,"context_line":"          ozn_controller:"},{"line_number":325,"context_line":"            map_merge:"},{"line_number":326,"context_line":"              - start_order: 1"},{"line_number":327,"context_line":"                image: *ovn_controller_image"},{"line_number":328,"context_line":"                net: host"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"35b46b07_23adcd84","line":325,"range":{"start_line":325,"start_character":1,"end_line":325,"end_character":22},"in_reply_to":"0ee76c85_d20c3c2a","updated":"2021-12-22 16:04:48.000000000","message":"Ack","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ef3365e36444f4f22419fde4ebbeb64c66699f5e","unresolved":true,"context_lines":[{"line_number":331,"context_line":"                restart: always"},{"line_number":332,"context_line":"                depends_on:"},{"line_number":333,"context_line":"                  - openvswitch.service"},{"line_number":334,"context_line":"              - if ovn_cpu_set:"},{"line_number":335,"context_line":"                - cpuset_cpus: {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}"},{"line_number":336,"context_line":"              - healthcheck:"},{"line_number":337,"context_line":"                  test:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0123be34_bf01c49f","line":334,"range":{"start_line":334,"start_character":14,"end_line":334,"end_character":31},"updated":"2021-12-08 14:16:08.000000000","message":"Wrong syntax for if:","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"d4ecadf439347af7a7b907c343b98949f0bca3b2","unresolved":false,"context_lines":[{"line_number":331,"context_line":"                restart: always"},{"line_number":332,"context_line":"                depends_on:"},{"line_number":333,"context_line":"                  - openvswitch.service"},{"line_number":334,"context_line":"              - if ovn_cpu_set:"},{"line_number":335,"context_line":"                - cpuset_cpus: {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}"},{"line_number":336,"context_line":"              - healthcheck:"},{"line_number":337,"context_line":"                  test:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"4559aa46_2f34d821","line":334,"range":{"start_line":334,"start_character":14,"end_line":334,"end_character":31},"in_reply_to":"0123be34_bf01c49f","updated":"2021-12-22 16:04:48.000000000","message":"Ack","commit_id":"53299cdf9ee2c9535301f85f76dba0a94fcae6af"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ee9670a5b86079441d4abed3e0dcdb3dc3056f10","unresolved":true,"context_lines":[{"line_number":157,"context_line":"    or:"},{"line_number":158,"context_line":"      - and:"},{"line_number":159,"context_line":"        - not: {equals: [ {get_param: OVNContainerCpusetCpus}, \u0027\u0027]}"},{"line_number":160,"context_line":"        - equals: [{get_param: [RoleParameters, OVNContainerCpusetCpus]}, \u0027\u0027]"},{"line_number":161,"context_line":"      - not: {equals: [{get_param: [RoleParameters, OVNContainerCpusetCpus]}, \u0027\u0027 ]}"},{"line_number":162,"context_line":"  key_size_override_set:"},{"line_number":163,"context_line":"    not: {equals: [{get_param: ContainerOvnCertificateKeySize}, \u0027\u0027]}"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"816e7fda_f049b0d5","line":160,"range":{"start_line":160,"start_character":7,"end_line":160,"end_character":77},"updated":"2021-12-23 05:14:55.000000000","message":"This is redundant. If both global and role parameters are set, container_cpuset_cpus in OVNBridgeMappingsValue would not be empty.","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"b5ce219ec53a4630a4095fdd3c7fa62bf8eed015","unresolved":false,"context_lines":[{"line_number":157,"context_line":"    or:"},{"line_number":158,"context_line":"      - and:"},{"line_number":159,"context_line":"        - not: {equals: [ {get_param: OVNContainerCpusetCpus}, \u0027\u0027]}"},{"line_number":160,"context_line":"        - equals: [{get_param: [RoleParameters, OVNContainerCpusetCpus]}, \u0027\u0027]"},{"line_number":161,"context_line":"      - not: {equals: [{get_param: [RoleParameters, OVNContainerCpusetCpus]}, \u0027\u0027 ]}"},{"line_number":162,"context_line":"  key_size_override_set:"},{"line_number":163,"context_line":"    not: {equals: [{get_param: ContainerOvnCertificateKeySize}, \u0027\u0027]}"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fee01c16_1910df23","line":160,"range":{"start_line":160,"start_character":7,"end_line":160,"end_character":77},"in_reply_to":"816e7fda_f049b0d5","updated":"2021-12-23 12:38:59.000000000","message":"Ack","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ee9670a5b86079441d4abed3e0dcdb3dc3056f10","unresolved":true,"context_lines":[{"line_number":334,"context_line":"              restart: always"},{"line_number":335,"context_line":"              depends_on:"},{"line_number":336,"context_line":"                - openvswitch.service"},{"line_number":337,"context_line":"            - if:"},{"line_number":338,"context_line":"              - ovn_cpu_set"},{"line_number":339,"context_line":"              - cpuset_cpus: {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}"},{"line_number":340,"context_line":"            - healthcheck:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"66223691_14826283","line":337,"range":{"start_line":337,"start_character":12,"end_line":337,"end_character":17},"updated":"2021-12-23 05:14:55.000000000","message":"This won\u0027t work as mentioned earlier. The job failures are related, startup config is not correct[1]. If you use \u0027if\u0027 like this then you\u0027ve to use map_merge, else as I mentioned earlier use the condition only for the value.\n\n[1] https://11dd0df37f114ab1a199-cf4df5e5ffe136c56cf2c9d8dbb6eb03.ssl.cf5.rackcdn.com/820535/6/check/tripleo-ci-centos-8-containers-multinode/fc01680/logs/subnode-1/var/lib/tripleo-config/container-startup-config/step_4/ovn_controller.json","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"b5ce219ec53a4630a4095fdd3c7fa62bf8eed015","unresolved":false,"context_lines":[{"line_number":334,"context_line":"              restart: always"},{"line_number":335,"context_line":"              depends_on:"},{"line_number":336,"context_line":"                - openvswitch.service"},{"line_number":337,"context_line":"            - if:"},{"line_number":338,"context_line":"              - ovn_cpu_set"},{"line_number":339,"context_line":"              - cpuset_cpus: {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}"},{"line_number":340,"context_line":"            - healthcheck:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"95a48594_211929e7","line":337,"range":{"start_line":337,"start_character":12,"end_line":337,"end_character":17},"in_reply_to":"66223691_14826283","updated":"2021-12-23 12:38:59.000000000","message":"Ack\n\nI think we need the if because if we don\u0027t have one, it\u0027ll have an empty value and kolla doesn\u0027t like that .","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"ee9670a5b86079441d4abed3e0dcdb3dc3056f10","unresolved":true,"context_lines":[{"line_number":336,"context_line":"                - openvswitch.service"},{"line_number":337,"context_line":"            - if:"},{"line_number":338,"context_line":"              - ovn_cpu_set"},{"line_number":339,"context_line":"              - cpuset_cpus: {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}"},{"line_number":340,"context_line":"            - healthcheck:"},{"line_number":341,"context_line":"                test:"},{"line_number":342,"context_line":"                  list_join:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"5f0da286_593bc82a","line":339,"range":{"start_line":339,"start_character":41,"end_line":339,"end_character":60},"updated":"2021-12-23 05:14:55.000000000","message":"Wrong resource, should be OVNBridgeMappingsValue.","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"b5ce219ec53a4630a4095fdd3c7fa62bf8eed015","unresolved":false,"context_lines":[{"line_number":336,"context_line":"                - openvswitch.service"},{"line_number":337,"context_line":"            - if:"},{"line_number":338,"context_line":"              - ovn_cpu_set"},{"line_number":339,"context_line":"              - cpuset_cpus: {get_attr: [RoleParametersValue, value, container_cpuset_cpus]}"},{"line_number":340,"context_line":"            - healthcheck:"},{"line_number":341,"context_line":"                test:"},{"line_number":342,"context_line":"                  list_join:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"235487b1_8f65c491","line":339,"range":{"start_line":339,"start_character":41,"end_line":339,"end_character":60},"in_reply_to":"5f0da286_593bc82a","updated":"2021-12-23 12:38:59.000000000","message":"I put it in the wrong MappingsValue... I changed it to the right one.","commit_id":"29546b0b1376533d2dc4d345316a3138d0f6e24b"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"509a252bb19b3f83e8c9ffccc69308080f06335c","unresolved":true,"context_lines":[{"line_number":129,"context_line":"    description: Override the private key size used when creating the"},{"line_number":130,"context_line":"                 certificate for this service"},{"line_number":131,"context_line":"  OVNContainerCpusetCpus:"},{"line_number":132,"context_line":"    description: Limit the specific CPUs or cores a container can use."},{"line_number":133,"context_line":"    type: string"},{"line_number":134,"context_line":"    default: \u0027\u0027"},{"line_number":135,"context_line":"    tags:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"fb318e48_c4a71100","line":132,"updated":"2022-01-05 06:22:34.000000000","message":"Explain further in description on what should be values? Should these cores be isolated? whether these are host cpus? is there any minimal number of cpus? does it accept range? does it accept comma separated cpus?","commit_id":"c5b0b1ffc4ab185eed4c7fe0509ad2624f830b29"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"039fc3609ea40cd7f84417b229190ffd225c9e13","unresolved":false,"context_lines":[{"line_number":129,"context_line":"    description: Override the private key size used when creating the"},{"line_number":130,"context_line":"                 certificate for this service"},{"line_number":131,"context_line":"  OVNContainerCpusetCpus:"},{"line_number":132,"context_line":"    description: Limit the specific CPUs or cores a container can use."},{"line_number":133,"context_line":"    type: string"},{"line_number":134,"context_line":"    default: \u0027\u0027"},{"line_number":135,"context_line":"    tags:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"4631025e_2d91a47a","line":132,"in_reply_to":"fb318e48_c4a71100","updated":"2022-01-05 13:48:59.000000000","message":"Ack","commit_id":"c5b0b1ffc4ab185eed4c7fe0509ad2624f830b29"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"04f56fb204e2887b356d26f93c449edc79617d2e","unresolved":true,"context_lines":[{"line_number":133,"context_line":"      Limit the specific CPUs or cores a container can use."},{"line_number":134,"context_line":"      It can be specified as a single core (ex. 0), as a comma-separated"},{"line_number":135,"context_line":"      list (ex. 0,1), as a range (ex. 0-3) or a combination if previous"},{"line_number":136,"context_line":"      methods (ex 0-3,7,11-15).  The selected cores should be isolated "},{"line_number":137,"context_line":"      from guests and hypervisor in order to obtain best possible"},{"line_number":138,"context_line":"      performance."},{"line_number":139,"context_line":"    type: string"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"15d7a7f1_16b90499","line":136,"range":{"start_line":136,"start_character":70,"end_line":136,"end_character":71},"updated":"2022-01-07 02:14:07.000000000","message":"trailing space","commit_id":"7961683b6a074df06713093dc54728af656cd2ae"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"653061f5d0691d8d8061ccfca8b39a43d2a98410","unresolved":false,"context_lines":[{"line_number":133,"context_line":"      Limit the specific CPUs or cores a container can use."},{"line_number":134,"context_line":"      It can be specified as a single core (ex. 0), as a comma-separated"},{"line_number":135,"context_line":"      list (ex. 0,1), as a range (ex. 0-3) or a combination if previous"},{"line_number":136,"context_line":"      methods (ex 0-3,7,11-15).  The selected cores should be isolated "},{"line_number":137,"context_line":"      from guests and hypervisor in order to obtain best possible"},{"line_number":138,"context_line":"      performance."},{"line_number":139,"context_line":"    type: string"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"9b51d28b_1a5812af","line":136,"range":{"start_line":136,"start_character":70,"end_line":136,"end_character":71},"in_reply_to":"15d7a7f1_16b90499","updated":"2022-01-07 23:09:52.000000000","message":"Ack","commit_id":"7961683b6a074df06713093dc54728af656cd2ae"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"04f56fb204e2887b356d26f93c449edc79617d2e","unresolved":true,"context_lines":[{"line_number":214,"context_line":"              OVNStaticBridgeMacMappings: {get_param: OVNStaticBridgeMacMappings}"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":""},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"outputs:"},{"line_number":219,"context_line":"  role_data:"},{"line_number":220,"context_line":"    description: Role data for the Ovn Controller agent."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"9d7ed107_eb5fa19f","line":217,"range":{"start_line":217,"start_character":0,"end_line":217,"end_character":0},"updated":"2022-01-07 02:14:07.000000000","message":"too many blank lines.","commit_id":"7961683b6a074df06713093dc54728af656cd2ae"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"653061f5d0691d8d8061ccfca8b39a43d2a98410","unresolved":false,"context_lines":[{"line_number":214,"context_line":"              OVNStaticBridgeMacMappings: {get_param: OVNStaticBridgeMacMappings}"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":""},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"outputs:"},{"line_number":219,"context_line":"  role_data:"},{"line_number":220,"context_line":"    description: Role data for the Ovn Controller agent."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"c4d95709_ab28ce4e","line":217,"range":{"start_line":217,"start_character":0,"end_line":217,"end_character":0},"in_reply_to":"9d7ed107_eb5fa19f","updated":"2022-01-07 23:09:52.000000000","message":"Ack","commit_id":"7961683b6a074df06713093dc54728af656cd2ae"}]}