)]}'
{"environments/services/neutron-ovn-dvr-ha.yaml":[{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"45af13af49d06a368d68ae7dceba5f28f14400ca","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  NeutronVniRanges: [\u00271:65536\u0027, ]"},{"line_number":27,"context_line":"  NeutronEnableDVR: true"},{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_452f6d3e","line":29,"updated":"2018-04-23 08:07:41.000000000","message":"I\u0027m not sure that we\u0027d like to clear the Bridge Mappings on computes for DVR? , shouldn\u0027t the bridge mappings be set in that case?","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"cf6e0563990d440f8a7ef37d11ce48a2b198ff6d","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  NeutronVniRanges: [\u00271:65536\u0027, ]"},{"line_number":27,"context_line":"  NeutronEnableDVR: true"},{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_a5c3e956","line":29,"in_reply_to":"9f6a8fd7_452f6d3e","updated":"2018-04-23 08:11:33.000000000","message":"true! i missed it sorry","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"298c850133727e7b235fb3e88d0fee2fdb5d61b1","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  NeutronVniRanges: [\u00271:65536\u0027, ]"},{"line_number":27,"context_line":"  NeutronEnableDVR: true"},{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_a0c67767","line":29,"in_reply_to":"9f6a8fd7_a5c3e956","updated":"2018-04-23 09:00:58.000000000","message":"Agree, thanks","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"7bb89970ec153c83bf34673afdb706ff7df5b8d1","unresolved":false,"context_lines":[{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"},{"line_number":33,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_8574a537","line":31,"updated":"2018-04-23 08:14:01.000000000","message":"Why do we set them in \"controllers\" ovn-northd? wouldn\u0027t it be just \"Networkers\" ?","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"8fed04acb8bbad81874d3e2ac0bb1db6149e59e2","unresolved":false,"context_lines":[{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"},{"line_number":33,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_c0428bab","line":31,"in_reply_to":"9f6a8fd7_00cac34c","updated":"2018-04-23 09:02:25.000000000","message":"Yeah, i think we\u0027re just using Controller role right now. It should be a for a different patch to start using Networker role. Thanks Anil!","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"298c850133727e7b235fb3e88d0fee2fdb5d61b1","unresolved":false,"context_lines":[{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"},{"line_number":33,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_00cac34c","line":31,"in_reply_to":"9f6a8fd7_8574a537","updated":"2018-04-23 09:00:58.000000000","message":"roles_data.yaml which is used in ovn tripleo deployment http://paste.openstack.org/show/719719/ is not defining Networkder role and only defining Controller role and this role has - OS::TripleO::Services::OVNController\n\nLooks like we are using Controller role for both running database services and hosting gateway routers.\n\nIf it is only hosting database services, it shouldn\u0027t run\nOS::TripleO::Services::OVNController service.\n\nAlso in commit [1] Numan is adding this option in ControllerParameters as\n  ControllerParameters:\n    OVNCMSOptions: \"enable-chassis-as-gw\"\n\n[1] https://github.com/openstack/tripleo-heat-templates/commit/71d59bb0a34349f3ed2b95d70452b771cc8039d2#diff-bdb9af0031906100cdbc39af8dfa1e6e","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"d718820cc51c16bf833a585594cb1c93e9e3b6d3","unresolved":false,"context_lines":[{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"},{"line_number":33,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_6957e3cb","line":31,"in_reply_to":"9f6a8fd7_c0428bab","updated":"2018-04-23 14:13:30.000000000","message":"Ok, we need to fix that if we can, or otherwise they can\u0027t create a separate networker if they need to.","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"}],"environments/services/neutron-ovn-dvr.yaml":[{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"45af13af49d06a368d68ae7dceba5f28f14400ca","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  NeutronVniRanges: [\u00271:65536\u0027, ]"},{"line_number":27,"context_line":"  NeutronEnableDVR: true"},{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_85ffc5a9","line":29,"updated":"2018-04-23 08:07:41.000000000","message":"I\u0027m not sure that we\u0027d like to clear the Bridge Mappings on computes for DVR? , shouldn\u0027t the bridge mappings be set in that case?","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"3cb20e0efd45e3ceac1807b104e80aff23cb57a1","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  NeutronEnableDVR: true"},{"line_number":28,"context_line":"  ComputeParameters:"},{"line_number":29,"context_line":"    NeutronBridgeMappings: \"\""},{"line_number":30,"context_line":"  ControllerParameters:"},{"line_number":31,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""},{"line_number":32,"context_line":"  NetworkerParameters:"},{"line_number":33,"context_line":"    OVNCMSOptions: \"enable-chassis-as-gw\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_458bed1d","line":30,"updated":"2018-04-23 08:13:38.000000000","message":"Why do we set them in \"controllers\" ovn-northd? wouldn\u0027t it be just \"Networkers\" ?","commit_id":"d14de3958c5f70c9096061051df99641d80b8a7e"}]}