)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"837c07fe670c1229b98afa279e06f097e2da8f03","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now, this patch will support standard_attrs for"},{"line_number":10,"context_line":"firewall_group/firewall_rule/firewall_policy."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1986906"},{"line_number":13,"context_line":"Change-Id: Ib7b06d604a0950a104215bcf4386e14b77d20d12"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"7616f285_f2e97b79","line":11,"updated":"2023-04-27 09:30:54.000000000","message":"you can add here Depends-On to make the review more cleaner, but you need anyway a new n-lib release to make the CI happy with the import of the extension file from neutron-lib","commit_id":"965b60fba7d948f0b265a1a6a0284690da58cc22"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"81253836cdc7a0c16a9723326ccfe91c3fdf94c3","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now, this patch will support standard_attrs for"},{"line_number":10,"context_line":"firewall_group/firewall_rule/firewall_policy."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1986906"},{"line_number":13,"context_line":"Change-Id: Ib7b06d604a0950a104215bcf4386e14b77d20d12"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3c66cb8b_43c7eee1","line":11,"in_reply_to":"7616f285_f2e97b79","updated":"2023-04-27 09:42:36.000000000","message":"Done","commit_id":"965b60fba7d948f0b265a1a6a0284690da58cc22"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"086204447aa2c8bd207726bf4a5611a227b6b45b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"39d8775e_68bb0b8c","updated":"2023-02-28 11:08:47.000000000","message":"This patch is creating a db migration in Zed while we are in Antelope (and close to Bobcat)","commit_id":"5096ef4a6974b90773d519e29fe8188dd2c736a0"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"002fadc88c362373e9437ff9de51ff3255a75f18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e128e863_31eb08a6","in_reply_to":"39d8775e_68bb0b8c","updated":"2023-03-01 02:51:50.000000000","message":"Alright, I will put it under the directory of bocat.","commit_id":"5096ef4a6974b90773d519e29fe8188dd2c736a0"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"bc8962a9d7f4940abd89e9dbd8e9b4b09dad2d7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6e60e529_79894214","updated":"2023-03-01 06:42:59.000000000","message":"recheck","commit_id":"17530d426cb2f5a5fc87c6c3cd5fe513943e67f5"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4240cf3957cea5d7606ced722ef2260bafbd4caa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d981c8a0_18360d93","updated":"2023-03-02 08:36:50.000000000","message":"You should wait until we open the bobcat branch","commit_id":"13cc08cf08f49d781fe38bb304fb6f681f2bc6d9"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"3ea7f3fdf835e42b755938f45f0515bae108d85f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"59d06383_c4102d40","updated":"2023-03-02 07:48:36.000000000","message":"recheck","commit_id":"13cc08cf08f49d781fe38bb304fb6f681f2bc6d9"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c89a604fdadad567d2f4471a8e98cb1d997f969e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"eab620de_e073fa46","updated":"2023-04-26 18:12:15.000000000","message":"You need this doc, if not working please ask for advice on #neutron channel:\nhttps://opendev.org/openstack/neutron/src/branch/master/doc/source/contributor/internals/db_layer.rst#the-standard-attribute-table","commit_id":"965b60fba7d948f0b265a1a6a0284690da58cc22"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"47a9f4b2ab2dec4f9e07f018ab23bd63e06dd6be","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1c7c605b_dc038e1b","updated":"2023-04-26 17:55:07.000000000","message":"as this is an API change you need a new extension like this one: https://review.opendev.org/c/openstack/neutron-lib/+/562331","commit_id":"965b60fba7d948f0b265a1a6a0284690da58cc22"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"c29bc1b9a605cd6af2bbaaca61d03b650a0da155","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3e089be6_e3b9edb3","updated":"2023-04-27 07:23:06.000000000","message":"neutron-lib patch:\nhttps://review.opendev.org/c/openstack/neutron-lib/+/881694","commit_id":"965b60fba7d948f0b265a1a6a0284690da58cc22"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8cd3bbc8840e48cd4a022936359df2be759f3107","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"911e6c74_f0d84886","in_reply_to":"eab620de_e073fa46","updated":"2023-04-26 20:06:46.000000000","message":"I checked again and from this perspective the patch looks ok, forget my last comment, but an API extension is necessary.\nI tested your patch and otherwise it looks ok, thanks for working on this","commit_id":"965b60fba7d948f0b265a1a6a0284690da58cc22"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"710437fb1ba5ae03ef288d7a4dd088a1c6379551","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"899d5449_fa26da1d","updated":"2023-04-28 07:23:18.000000000","message":"Please add the extension reference also to this folder:\nhttps://opendev.org/openstack/neutron-fwaas/src/branch/master/neutron_fwaas/extensions\n\nexample from Neutron: https://opendev.org/openstack/neutron/src/branch/master/neutron/extensions/standard_attr_segment.py","commit_id":"989bcdc3b1f92ec7f1ad9d0ebbeabc60c66ba9e0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a4931822696991400b2061b3ce2c79160ab4309e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"d245ae6e_02a943d3","updated":"2023-05-24 12:01:50.000000000","message":"Code looks ok, please address the comments.","commit_id":"00f78e6f20a0cab2a180f4ad5036c8bdf7199ba1"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"7919d869f6031c724d15ee1a1c19f5729a290123","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2133b1ec_980ee8e7","updated":"2023-05-05 09:06:59.000000000","message":"Looks ok, thanks","commit_id":"00f78e6f20a0cab2a180f4ad5036c8bdf7199ba1"}],"neutron_fwaas/db/migration/alembic_migrations/versions/2023.1/expand/6941ce70131e_add_standard_attr_id.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a4931822696991400b2061b3ce2c79160ab4309e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"9c834075_8fb896d8","updated":"2023-05-24 12:01:50.000000000","message":"This should be in 2023.2, not 2023.1, as commented in PS8","commit_id":"00f78e6f20a0cab2a180f4ad5036c8bdf7199ba1"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"f523c3b8f31dc7f70d1cc918d3b2977d3123412d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"3bee27d0_6d6825e7","in_reply_to":"9c834075_8fb896d8","updated":"2023-05-24 15:00:52.000000000","message":"Done","commit_id":"00f78e6f20a0cab2a180f4ad5036c8bdf7199ba1"}],"neutron_fwaas/extensions/_firewall_v2_stdattrs.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a4931822696991400b2061b3ce2c79160ab4309e","unresolved":true,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":"TODO(liushy): This module should be deleted once neutron-lib containing"},{"line_number":17,"context_line":"https://review.openstack.org/#/c/881694/ change is released."},{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"f087215c_5d91783c","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":60},"updated":"2023-05-24 12:01:50.000000000","message":"This is already in n-lib 3.6.1: https://github.com/openstack/neutron-lib/blob/3.6.1/neutron_lib/api/definitions/firewall_v2_stdattrs.py","commit_id":"00f78e6f20a0cab2a180f4ad5036c8bdf7199ba1"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"f523c3b8f31dc7f70d1cc918d3b2977d3123412d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":"TODO(liushy): This module should be deleted once neutron-lib containing"},{"line_number":17,"context_line":"https://review.openstack.org/#/c/881694/ change is released."},{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"cc787d29_06affd7a","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":60},"in_reply_to":"f087215c_5d91783c","updated":"2023-05-24 15:00:52.000000000","message":"Done","commit_id":"00f78e6f20a0cab2a180f4ad5036c8bdf7199ba1"}]}
