)]}'
{"ironic/common/neutron.py":[{"author":{"_account_id":7711,"name":"Yuriy Zveryanskyy","email":"yzveryanskyy@mirantis.com","username":"yuriyz"},"change_message_id":"3512cebe7264ea80a02937326ef9aa90e9d4064b","unresolved":false,"context_lines":[{"line_number":417,"context_line":"    _cleaning_network_uuid \u003d None"},{"line_number":418,"context_line":"    _provisioning_network_uuid \u003d None"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def get_cleaning_network_uuid(self):"},{"line_number":421,"context_line":"        self._cleaning_network_uuid \u003d validate_network("},{"line_number":422,"context_line":"            CONF.neutron.cleaning_network, _(\u0027cleaning network\u0027))"},{"line_number":423,"context_line":"        return self._cleaning_network_uuid"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff0f0b1f_cba9f1df","line":420,"updated":"2017-05-18 10:18:34.000000000","message":"This change causes additional neutron api request l373 on every interface validate() call.","commit_id":"3f6f9908c808f5e876a5bbaa01d966f860619ec2"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"1c12948abfe666c59a9e404baf790db67fb97704","unresolved":false,"context_lines":[{"line_number":417,"context_line":"    _cleaning_network_uuid \u003d None"},{"line_number":418,"context_line":"    _provisioning_network_uuid \u003d None"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def get_cleaning_network_uuid(self):"},{"line_number":421,"context_line":"        self._cleaning_network_uuid \u003d validate_network("},{"line_number":422,"context_line":"            CONF.neutron.cleaning_network, _(\u0027cleaning network\u0027))"},{"line_number":423,"context_line":"        return self._cleaning_network_uuid"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff0f0b1f_713bf337","line":420,"in_reply_to":"ff0f0b1f_abe6bd15","updated":"2017-05-18 13:38:02.000000000","message":"Well, there was a reason why we cached it :-)\n\nhere\u0027s the question. This change fixes the problem at the expense of an additional neutron API request. the bug is a high -- so is it severe enough that we should land this patch as-is?\n\nAnd then someone should file a bug about how we should be smarter about caching this so as to reduce the number of neutron API requests. I\u0027m a bit worried that the solution to this part will take awhile to land.","commit_id":"3f6f9908c808f5e876a5bbaa01d966f860619ec2"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"91f527949a104f6cb0002e5fe631683dea1b1460","unresolved":false,"context_lines":[{"line_number":417,"context_line":"    _cleaning_network_uuid \u003d None"},{"line_number":418,"context_line":"    _provisioning_network_uuid \u003d None"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"    def get_cleaning_network_uuid(self):"},{"line_number":421,"context_line":"        self._cleaning_network_uuid \u003d validate_network("},{"line_number":422,"context_line":"            CONF.neutron.cleaning_network, _(\u0027cleaning network\u0027))"},{"line_number":423,"context_line":"        return self._cleaning_network_uuid"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff0f0b1f_abe6bd15","line":420,"in_reply_to":"ff0f0b1f_cba9f1df","updated":"2017-05-18 10:26:15.000000000","message":"++ that should not be the case I suppose. Not what is the better way to deal with this though. Putting it in cache with expiration ?","commit_id":"3f6f9908c808f5e876a5bbaa01d966f860619ec2"}]}