)]}'
{"neutron/conf/policies/port.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"78e61b074158637bd17f217b01e23069a5f08af6","unresolved":true,"context_lines":[{"line_number":84,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":85,"context_line":"        name\u003d\u0027create_port:device_id\u0027,"},{"line_number":86,"context_line":"        check_str\u003dneutron_policy.policy_or("},{"line_number":87,"context_line":"            base.ADMIN_OR_PROJECT_MEMBER,"},{"line_number":88,"context_line":"            base.SERVICE),"},{"line_number":89,"context_line":"        scope_types\u003d[\u0027project\u0027],"},{"line_number":90,"context_line":"        description\u003d\u0027Specify ``device_id`` attribute when creating a port\u0027,"},{"line_number":91,"context_line":"        operations\u003dACTION_POST,"}],"source_content_type":"text/x-python","patch_set":2,"id":"56477af5_3a0a14fd","line":88,"range":{"start_line":87,"start_character":12,"end_line":88,"end_character":24},"updated":"2025-04-15 10:17:34.000000000","message":"I don\u0027t think, by default, a project member should be able to change a port ``device_id``. If that is possible, a non-admin user can remove it. If I\u0027m not wrong, that could interfere the live-migration process because the Neutron API won\u0027t return any result to the query for a port attached to this device ID.\n\nI think this should be limited to ``base.ADMIN_OR_SERVICE``.","commit_id":"c981cfd658e5a75d87f1cd635bcd39b551945e7b"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6da33095ee3c1b6326e39be0de24028534cfd549","unresolved":false,"context_lines":[{"line_number":84,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":85,"context_line":"        name\u003d\u0027create_port:device_id\u0027,"},{"line_number":86,"context_line":"        check_str\u003dneutron_policy.policy_or("},{"line_number":87,"context_line":"            base.ADMIN_OR_PROJECT_MEMBER,"},{"line_number":88,"context_line":"            base.SERVICE),"},{"line_number":89,"context_line":"        scope_types\u003d[\u0027project\u0027],"},{"line_number":90,"context_line":"        description\u003d\u0027Specify ``device_id`` attribute when creating a port\u0027,"},{"line_number":91,"context_line":"        operations\u003dACTION_POST,"}],"source_content_type":"text/x-python","patch_set":2,"id":"c52a6e02_bed9f8d3","line":88,"range":{"start_line":87,"start_character":12,"end_line":88,"end_character":24},"in_reply_to":"0d0faaf1_d2e53945","updated":"2025-04-15 10:39:55.000000000","message":"Cool, that\u0027s perfect then.","commit_id":"c981cfd658e5a75d87f1cd635bcd39b551945e7b"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8185c0955542b5c687e4166307a5b33a41af7814","unresolved":true,"context_lines":[{"line_number":84,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":85,"context_line":"        name\u003d\u0027create_port:device_id\u0027,"},{"line_number":86,"context_line":"        check_str\u003dneutron_policy.policy_or("},{"line_number":87,"context_line":"            base.ADMIN_OR_PROJECT_MEMBER,"},{"line_number":88,"context_line":"            base.SERVICE),"},{"line_number":89,"context_line":"        scope_types\u003d[\u0027project\u0027],"},{"line_number":90,"context_line":"        description\u003d\u0027Specify ``device_id`` attribute when creating a port\u0027,"},{"line_number":91,"context_line":"        operations\u003dACTION_POST,"}],"source_content_type":"text/x-python","patch_set":2,"id":"0d0faaf1_d2e53945","line":88,"range":{"start_line":87,"start_character":12,"end_line":88,"end_character":24},"in_reply_to":"56477af5_3a0a14fd","updated":"2025-04-15 10:38:04.000000000","message":"I think we\u0027ve discussed it during the PTG and decided to keep it like that (see https://etherpad.opendev.org/p/apr2025-ptg-neutron#L295), as it was set that way also without this patch. If we wan\u0027t to change it to something more strict, IMO we should do it in separate patch with proper release note, etc.","commit_id":"c981cfd658e5a75d87f1cd635bcd39b551945e7b"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"78e61b074158637bd17f217b01e23069a5f08af6","unresolved":true,"context_lines":[{"line_number":477,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":478,"context_line":"        name\u003d\u0027update_port:device_id\u0027,"},{"line_number":479,"context_line":"        check_str\u003dneutron_policy.policy_or("},{"line_number":480,"context_line":"            base.ADMIN_OR_PROJECT_MEMBER,"},{"line_number":481,"context_line":"            base.SERVICE),"},{"line_number":482,"context_line":"        scope_types\u003d[\u0027project\u0027],"},{"line_number":483,"context_line":"        description\u003d\u0027Update ``device_id`` attribute of a port\u0027,"},{"line_number":484,"context_line":"        operations\u003dACTION_PUT,"}],"source_content_type":"text/x-python","patch_set":2,"id":"a4522507_f9a73283","line":481,"range":{"start_line":480,"start_character":12,"end_line":481,"end_character":24},"updated":"2025-04-15 10:17:34.000000000","message":"ditto","commit_id":"c981cfd658e5a75d87f1cd635bcd39b551945e7b"}]}