)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8690fbc9b055dedd20f97949d661ac776e9590bb","unresolved":false,"context_lines":[{"line_number":12,"context_line":"\"inherit\" allows to copy the inner ToS into the outer packet header."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1987382"},{"line_number":15,"context_line":"Depends-on: https://review.opendev.org/854221"},{"line_number":16,"context_line":"Change-Id: Ia7c09fb987cbdca50a4ba693bfd79b8b3c503808"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5c948f01_61baf1dc","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":45},"updated":"2022-08-31 13:39:41.000000000","message":"this is not required but would not harm, and helps us make sure the dependent change is merged (especially when backporting this to wallaby)","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"411f8ed145a6b0503c31c35a1e8e1dd65f2599a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8afc955e_72137ea5","updated":"2022-08-31 13:31:50.000000000","message":"Hi Takashi, can you review this patch again? Thanks in advance.","commit_id":"04fea2797711b1cf1ef1a774c12f9d77083ead63"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"120ea6a760116400a3233a78506396eebf21c175","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"febd8d49_df5dfec6","updated":"2022-09-01 05:07:24.000000000","message":"I\u0027m not sure if Zuul captures outputs from `ovs-vsctl get open . external_ids` to check this is applied. But we can indeed see the variable being set which implies this particular patch works as described.\n\nhttps://zuul.opendev.org/t/openstack/build/02b8825a218e444699c94d856112df44/log/logs/undercloud/home/zuul/overcloud-deploy/overcloud/config-download/overcloud/group_vars/Controller#705","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9dbe1d7cd78e857cbb96c66cf1c694a56d1959d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ecff6a4c_310d4e3c","updated":"2022-08-31 13:41:17.000000000","message":"The change looks good now. I\u0027ll double-check CI result to make sure the parameter is set as expected.","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"7128ad2628d6a2d03bdc22422d084b00f842e0f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6799f2c3_873bd45b","updated":"2022-09-12 08:52:10.000000000","message":"https://7c8408c9823eb345d94f-318a5dfd9731b2b67982b45629cc2601.ssl.cf5.rackcdn.com/854127/3/check/tripleo-ci-centos-9-standalone/245dd4f/logs/undercloud/var/log/extra/ovn.txt\n\nOutput of ovs-vsctl get open . external_ids\n{hostname\u003dstandalone.localdomain, ovn-bridge\u003dbr-int, ovn-bridge-mappings\u003d\"datacentre:br-ctlplane,tenant:br-tenant\", ovn-chassis-mac-mappings\u003d\"datacentre:fa:16:3a:00:53:00,tenant:fa:16:3a:00:53:01\", ovn-encap-ip\u003d\"192.168.24.1\", ovn-encap-tos\u003d\"0\", ovn-encap-type\u003dgeneve, ovn-match-northd-version\u003d\"true\", ovn-monitor-all\u003d\"true\", ovn-openflow-probe-interval\u003d\"60\", ovn-remote\u003d\"tcp:192.168.24.1:6642\", ovn-remote-probe-interval\u003d\"60000\", rundir\u003d\"/var/run/openvswitch\", system-id\u003d\"ae63bdbe-1589-472c-8090-4d01b0ddb49f\"}\n\n","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"61ec51b8b0b0cbcd06bbeac4c6f2edb05a7d7f3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"295da1b9_1b61f256","updated":"2022-09-06 05:54:35.000000000","message":"recheck\n\nThe previous run did not pull puppet-ovn with the dependent change.","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ac9fb9baa86c15f6f73a48e8e42590929f8e4dd2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a18f3fa1_649f9f62","updated":"2022-09-06 15:33:29.000000000","message":"recheck RETRY_LIMIT","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ff1f4251973fca787aea1acb249212ea215fac8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9cf74b93_bcb79656","updated":"2022-09-12 01:35:55.000000000","message":"recheck content provider wallaby","commit_id":"70509f5c3f48fec2554e905d37077ecd521b177b"}],"deployment/ovn/ovn-controller-container-puppet.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c1935fcce6f9d465572aad8bc2e37426b905c2e6","unresolved":true,"context_lines":[{"line_number":214,"context_line":"                    - - OVNCMSOptions"},{"line_number":215,"context_line":"                      - \",availability-zones\u003d\""},{"line_number":216,"context_line":"                      - {get_param: OVNAvailabilityZone}"},{"line_number":217,"context_line":"              ovn::controller::ovn_encap_tos: OVNEncapTos"},{"line_number":218,"context_line":"              vswitch::ovs::enable_hw_offload: OvsHwOffload"},{"line_number":219,"context_line":"              container_cpuset_cpus: OVNContainerCpusetCpus"},{"line_number":220,"context_line":"              ContainerOvnControllerImage: ContainerOvnControllerImage"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"03c564b8_552df98a","line":217,"range":{"start_line":217,"start_character":14,"end_line":217,"end_character":45},"updated":"2022-08-23 14:39:29.000000000","message":"This parameter is not yet implemented. I\u0027ve pushed the patch to add this parameter.\n\nhttps://review.opendev.org/c/openstack/puppet-ovn/+/854221","commit_id":"db4f6ec1ae39e00bb11f301bea0728c3f8b094fe"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6b4e8545a94b64cdb44605e0dc0e8393d4b159b2","unresolved":true,"context_lines":[{"line_number":214,"context_line":"                    - - OVNCMSOptions"},{"line_number":215,"context_line":"                      - \",availability-zones\u003d\""},{"line_number":216,"context_line":"                      - {get_param: OVNAvailabilityZone}"},{"line_number":217,"context_line":"              ovn::controller::ovn_encap_tos: OVNEncapTos"},{"line_number":218,"context_line":"              vswitch::ovs::enable_hw_offload: OvsHwOffload"},{"line_number":219,"context_line":"              container_cpuset_cpus: OVNContainerCpusetCpus"},{"line_number":220,"context_line":"              ContainerOvnControllerImage: ContainerOvnControllerImage"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a3166cc5_3b5aba43","line":217,"range":{"start_line":217,"start_character":14,"end_line":217,"end_character":45},"in_reply_to":"03c564b8_552df98a","updated":"2022-08-23 14:56:34.000000000","message":"Thanks!","commit_id":"db4f6ec1ae39e00bb11f301bea0728c3f8b094fe"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8690fbc9b055dedd20f97949d661ac776e9590bb","unresolved":false,"context_lines":[{"line_number":214,"context_line":"                    - - OVNCMSOptions"},{"line_number":215,"context_line":"                      - \",availability-zones\u003d\""},{"line_number":216,"context_line":"                      - {get_param: OVNAvailabilityZone}"},{"line_number":217,"context_line":"              ovn::controller::ovn_encap_tos: OVNEncapTos"},{"line_number":218,"context_line":"              vswitch::ovs::enable_hw_offload: OvsHwOffload"},{"line_number":219,"context_line":"              container_cpuset_cpus: OVNContainerCpusetCpus"},{"line_number":220,"context_line":"              ContainerOvnControllerImage: ContainerOvnControllerImage"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f8a28146_21e7ffb4","line":217,"range":{"start_line":217,"start_character":14,"end_line":217,"end_character":45},"in_reply_to":"a3166cc5_3b5aba43","updated":"2022-08-31 13:39:41.000000000","message":"Done","commit_id":"db4f6ec1ae39e00bb11f301bea0728c3f8b094fe"}]}