)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"ec3416c2d3b18a1fd1217faea25a297a21e4b788","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Iornic port with client-id in the extra means InfiniBand Port."},{"line_number":10,"context_line":"For this port we don\u0027t need the local_link_connection, because"},{"line_number":11,"context_line":"the vlan (pkey) confiugration is controller by the OpenSM (Subnet"},{"line_number":12,"context_line":"Manager)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2bfac4ccaf825bd9aa8ea0d2b447fcd7767acbc5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f9dbf25_7fc62c6a","line":11,"range":{"start_line":11,"start_character":42,"end_line":11,"end_character":43},"updated":"2018-03-02 12:23:39.000000000","message":"controlled","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"f58fade5490cdf244e44285a42a6d234598c73cc","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Iornic port with client-id in the extra means InfiniBand Port."},{"line_number":10,"context_line":"For this port we don\u0027t need the local_link_connection, because"},{"line_number":11,"context_line":"the vlan (pkey) confiugration is controller by the OpenSM (Subnet"},{"line_number":12,"context_line":"Manager)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2bfac4ccaf825bd9aa8ea0d2b447fcd7767acbc5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f9dbf25_652bf0ea","line":11,"range":{"start_line":11,"start_character":42,"end_line":11,"end_character":43},"in_reply_to":"1f9dbf25_7fc62c6a","updated":"2018-03-04 10:14:49.000000000","message":"Done","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"c9ab058fd31a30aa90cda5d75aa4bb912973e893","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Iornic port with client-id in the extra means InfiniBand Port."},{"line_number":10,"context_line":"For this port we don\u0027t need the local_link_connection, because"},{"line_number":11,"context_line":"the vlan (pkey) confiugration is controller by the OpenSM (Subnet"},{"line_number":12,"context_line":"Manager)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2bfac4ccaf825bd9aa8ea0d2b447fcd7767acbc5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f9dbf25_d07965fc","line":11,"range":{"start_line":11,"start_character":42,"end_line":11,"end_character":43},"in_reply_to":"1f9dbf25_7fc62c6a","updated":"2018-03-03 11:45:57.000000000","message":"nit s/confiugration/configuration/","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"f58fade5490cdf244e44285a42a6d234598c73cc","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Iornic port with client-id in the extra means InfiniBand Port."},{"line_number":10,"context_line":"For this port we don\u0027t need the local_link_connection, because"},{"line_number":11,"context_line":"the vlan (pkey) confiugration is controller by the OpenSM (Subnet"},{"line_number":12,"context_line":"Manager)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2bfac4ccaf825bd9aa8ea0d2b447fcd7767acbc5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f9dbf25_c5ee84b3","line":11,"range":{"start_line":11,"start_character":42,"end_line":11,"end_character":43},"in_reply_to":"1f9dbf25_d07965fc","updated":"2018-03-04 10:14:49.000000000","message":"Done","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"}],"ironic/common/neutron.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"ec3416c2d3b18a1fd1217faea25a297a21e4b788","unresolved":false,"context_lines":[{"line_number":460,"context_line":"        return True"},{"line_number":461,"context_line":"    if (node.network_interface \u003d\u003d \u0027neutron\u0027 and"},{"line_number":462,"context_line":"            not port.local_link_connection):"},{"line_number":463,"context_line":"        LOG.warning(\"The local_link_connection is required for \""},{"line_number":464,"context_line":"                    \"\u0027neutron\u0027 network interface and is not present \""},{"line_number":465,"context_line":"                    \"in the nodes %(node)s port %(port)s\","},{"line_number":466,"context_line":"                    {\u0027node\u0027: node.uuid, \u0027port\u0027: port.uuid})"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f9dbf25_3fbd04b7","line":463,"range":{"start_line":463,"start_character":50,"end_line":463,"end_character":58},"updated":"2018-03-02 12:23:39.000000000","message":"do you know if it is required by all other port types, or just some?","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"f58fade5490cdf244e44285a42a6d234598c73cc","unresolved":false,"context_lines":[{"line_number":460,"context_line":"        return True"},{"line_number":461,"context_line":"    if (node.network_interface \u003d\u003d \u0027neutron\u0027 and"},{"line_number":462,"context_line":"            not port.local_link_connection):"},{"line_number":463,"context_line":"        LOG.warning(\"The local_link_connection is required for \""},{"line_number":464,"context_line":"                    \"\u0027neutron\u0027 network interface and is not present \""},{"line_number":465,"context_line":"                    \"in the nodes %(node)s port %(port)s\","},{"line_number":466,"context_line":"                    {\u0027node\u0027: node.uuid, \u0027port\u0027: port.uuid})"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f9dbf25_28705da5","line":463,"range":{"start_line":463,"start_character":50,"end_line":463,"end_character":58},"in_reply_to":"1f9dbf25_3fbd04b7","updated":"2018-03-04 10:14:49.000000000","message":"As far as I know the other port type is Ethernet so this should be fine.","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"f58fade5490cdf244e44285a42a6d234598c73cc","unresolved":false,"context_lines":[{"line_number":460,"context_line":"        return True"},{"line_number":461,"context_line":"    if (node.network_interface \u003d\u003d \u0027neutron\u0027 and"},{"line_number":462,"context_line":"            not port.local_link_connection):"},{"line_number":463,"context_line":"        LOG.warning(\"The local_link_connection is required for \""},{"line_number":464,"context_line":"                    \"\u0027neutron\u0027 network interface and is not present \""},{"line_number":465,"context_line":"                    \"in the nodes %(node)s port %(port)s\","},{"line_number":466,"context_line":"                    {\u0027node\u0027: node.uuid, \u0027port\u0027: port.uuid})"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f9dbf25_88ec112a","line":463,"range":{"start_line":463,"start_character":50,"end_line":463,"end_character":58},"in_reply_to":"1f9dbf25_3fbd04b7","updated":"2018-03-04 10:14:49.000000000","message":"not sure what you mean by other port type. \nBut as far as I know we have ethernet port","commit_id":"0d5ce6b3458daaa5db89159b69bb4703d16ea6e5"}],"releasenotes/notes/dont-validate-local_link_connection-when-port-has-client-id-8e584586dc4fca50.yaml":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"2a0d0056189791abec0e24a3051ddf1988fc7051","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Iornic port with client-id in the extra means InfiniBand Port."},{"line_number":5,"context_line":"    For this port we don\u0027t need to validate the local_link_connection."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ff6b8bd7_3d4a3b80","line":4,"range":{"start_line":4,"start_character":4,"end_line":4,"end_character":10},"updated":"2018-03-05 07:46:18.000000000","message":"s/Iornic/Ironic/","commit_id":"e388b127897483e7e0e74541576f7b92397fc464"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"2a0d0056189791abec0e24a3051ddf1988fc7051","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ff6b8bd7_7d1e7371","line":6,"updated":"2018-03-05 07:46:18.000000000","message":"I think the release note should be updated to include more information like what is being fixed or link to the launchpad bug report.","commit_id":"e388b127897483e7e0e74541576f7b92397fc464"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"739eb51c5cd74c7d72fad454a2ed4610d211a331","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ff6b8bd7_f47fdfa8","line":6,"in_reply_to":"ff6b8bd7_7d1e7371","updated":"2018-03-06 15:39:26.000000000","message":"+1 for a link to the bug.","commit_id":"e388b127897483e7e0e74541576f7b92397fc464"}]}