)]}'
{"ironic/tests/unit/api/controllers/v1/test_utils.py":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"4a5acd1f54b488d0fd831c7466c51d8274abeebb","unresolved":true,"context_lines":[{"line_number":1010,"context_line":"        self._test_vendor_passthru_attach(b\u0027\\x00\\x01\u0027, b\u0027\\x00\\x01\u0027)"},{"line_number":1011,"context_line":""},{"line_number":1012,"context_line":"    def test_get_controller_reserved_names(self):"},{"line_number":1013,"context_line":"        expected \u003d [\u0027maintenance\u0027, \u0027management\u0027, \u0027states\u0027,"},{"line_number":1014,"context_line":"                    \u0027vendor_passthru\u0027, \u0027validate\u0027, \u0027detail\u0027]"},{"line_number":1015,"context_line":"        self.assertEqual(sorted(expected),"},{"line_number":1016,"context_line":"                         sorted(utils.get_controller_reserved_names("}],"source_content_type":"text/x-python","patch_set":2,"id":"b4f61c19_c0ffa2f2","side":"PARENT","line":1013,"range":{"start_line":1013,"start_character":36,"end_line":1013,"end_character":46},"updated":"2021-02-18 21:56:40.000000000","message":"Looking at the API[1] the only name that really needs to be reserved is \u0027detail\u0027, so I think its find that \u0027management\u0027 is no longer on this list.\n\nSince all subcontrollers are a subpath of /v1/nodes/{node_ident}, it looks like our reserved names list is unnecessarily long. We could choose to fix this in the future.\n\n[1] https://docs.openstack.org/api-ref/baremetal/#list-nodes-detailed","commit_id":"17952a975610e55b1abe3b60bf506a95d9cd7c9f"}]}