)]}'
{"neutron_lib/api/definitions/security_groups_remote_address_group.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ad325e6dc1b86e5757c285f0ec514e22c1015a14","unresolved":false,"context_lines":[{"line_number":19,"context_line":"UPDATED_TIMESTAMP \u003d \u00272020-08-25T10:00:00-00:00\u0027"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"RESOURCE_ATTRIBUTE_MAP \u003d {"},{"line_number":22,"context_line":"    \u0027security_group_rules\u0027: {"},{"line_number":23,"context_line":"        \u0027remote_address_group\u0027: {"},{"line_number":24,"context_line":"            \u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":25,"context_line":"            \u0027default\u0027: None, \u0027is_visible\u0027: True,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_2630089f","line":22,"range":{"start_line":22,"start_character":5,"end_line":22,"end_character":25},"updated":"2020-08-28 07:50:40.000000000","message":"yeah, this is still defined in neutron ([1]), I started a patch for rehoming, but not sure if it worth the time:\nhttps://review.opendev.org/698741\n\n[1]: https://opendev.org/openstack/neutron/src/branch/master/neutron/extensions/securitygroup.py#L213","commit_id":"642812c4ec0b7d31f66fade51506d3781dff3c8b"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ad325e6dc1b86e5757c285f0ec514e22c1015a14","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"RESOURCE_ATTRIBUTE_MAP \u003d {"},{"line_number":22,"context_line":"    \u0027security_group_rules\u0027: {"},{"line_number":23,"context_line":"        \u0027remote_address_group\u0027: {"},{"line_number":24,"context_line":"            \u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":25,"context_line":"            \u0027default\u0027: None, \u0027is_visible\u0027: True,"},{"line_number":26,"context_line":"            \u0027is_sort_key\u0027: True, \u0027is_filter\u0027: True},"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_66dfa03a","line":23,"range":{"start_line":23,"start_character":9,"end_line":23,"end_character":29},"updated":"2020-08-28 07:50:40.000000000","message":"nit: this can be a \"constant\" like:\nREMOTE_ADDRESS_GROUP \u003d \u0027remote_address_group\u0027\n...\n   REMOTE_ADDRESS_GROUP: {.....}\n\nThat makes it simple to reference from neutron code for example without the possibility of syntax issue.","commit_id":"642812c4ec0b7d31f66fade51506d3781dff3c8b"},{"author":{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},"change_message_id":"b032f84d3b7d3a1d3531ef8cef16b81c0812c220","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"RESOURCE_ATTRIBUTE_MAP \u003d {"},{"line_number":22,"context_line":"    \u0027security_group_rules\u0027: {"},{"line_number":23,"context_line":"        \u0027remote_address_group\u0027: {"},{"line_number":24,"context_line":"            \u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":25,"context_line":"            \u0027default\u0027: None, \u0027is_visible\u0027: True,"},{"line_number":26,"context_line":"            \u0027is_sort_key\u0027: True, \u0027is_filter\u0027: True},"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_72d46c48","line":23,"range":{"start_line":23,"start_character":9,"end_line":23,"end_character":29},"in_reply_to":"9f560f44_66dfa03a","updated":"2020-08-28 19:26:30.000000000","message":"Good point. I also updated the field name to remote_address_group_id, which is a bit lengthy but more explicit.","commit_id":"642812c4ec0b7d31f66fade51506d3781dff3c8b"}]}
