)]}'
{"sysinv/sysinv/sysinv/sysinv/api/controllers/v1/address_pool.py":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"4ab30a9c8fc4abf4545f52b4f52b64dfd991e581","unresolved":true,"context_lines":[{"line_number":338,"context_line":"            ipset.update(netaddr.IPRange(start, end))"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    def _check_pool_readonly(self, addrpool):"},{"line_number":341,"context_line":"        if addrpool.name not in SYSTEM_CONTROLLER_ADDRPOOLS:"},{"line_number":342,"context_line":"            networks \u003d pecan.request.dbapi.networks_get_by_pool(addrpool.id)"},{"line_number":343,"context_line":"            # Pool is considered readonly after the initial configuration is"},{"line_number":344,"context_line":"            # complete. During bootstrap it should be modifiable even though"}],"source_content_type":"text/x-python","patch_set":1,"id":"435fadf5_7065654d","line":341,"range":{"start_line":341,"start_character":8,"end_line":341,"end_character":60},"updated":"2021-04-26 20:33:22.000000000","message":"I\u0027d like to see a comment explaining the reason for this check to help people working on it in the future.","commit_id":"4cb18fb378146df438f0499f7b98b2dea2b1f3f9"},{"author":{"_account_id":31954,"name":"Yuxing Jiang","email":"Yuxing.Jiang@windriver.com","username":"yuxing"},"change_message_id":"16f5c527d0fce701eb56a07e163f2040b4db0beb","unresolved":false,"context_lines":[{"line_number":338,"context_line":"            ipset.update(netaddr.IPRange(start, end))"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    def _check_pool_readonly(self, addrpool):"},{"line_number":341,"context_line":"        if addrpool.name not in SYSTEM_CONTROLLER_ADDRPOOLS:"},{"line_number":342,"context_line":"            networks \u003d pecan.request.dbapi.networks_get_by_pool(addrpool.id)"},{"line_number":343,"context_line":"            # Pool is considered readonly after the initial configuration is"},{"line_number":344,"context_line":"            # complete. During bootstrap it should be modifiable even though"}],"source_content_type":"text/x-python","patch_set":1,"id":"6ce4160e_c07d459b","line":341,"range":{"start_line":341,"start_character":8,"end_line":341,"end_character":60},"in_reply_to":"435fadf5_7065654d","updated":"2021-04-26 21:21:51.000000000","message":"Done","commit_id":"4cb18fb378146df438f0499f7b98b2dea2b1f3f9"},{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"4ab30a9c8fc4abf4545f52b4f52b64dfd991e581","unresolved":true,"context_lines":[{"line_number":570,"context_line":"        addresses \u003d pecan.request.dbapi.addresses_get_by_pool("},{"line_number":571,"context_line":"            addrpool.id)"},{"line_number":572,"context_line":"        if addresses:"},{"line_number":573,"context_line":"            if cutils.is_initial_config_complete() and \\"},{"line_number":574,"context_line":"               (addrpool.name not in SYSTEM_CONTROLLER_ADDRPOOLS):"},{"line_number":575,"context_line":"                raise exception.AddressPoolInUseByAddresses()"},{"line_number":576,"context_line":"            else:"},{"line_number":577,"context_line":"                # Must be a request as a result of network reconfiguration"}],"source_content_type":"text/x-python","patch_set":1,"id":"c4e73180_632fe52f","line":574,"range":{"start_line":573,"start_character":13,"end_line":574,"end_character":66},"updated":"2021-04-26 20:33:22.000000000","message":"I\u0027d like to see a comment explaining the reason for this check to help people working on it in the future.","commit_id":"4cb18fb378146df438f0499f7b98b2dea2b1f3f9"},{"author":{"_account_id":31954,"name":"Yuxing Jiang","email":"Yuxing.Jiang@windriver.com","username":"yuxing"},"change_message_id":"16f5c527d0fce701eb56a07e163f2040b4db0beb","unresolved":false,"context_lines":[{"line_number":570,"context_line":"        addresses \u003d pecan.request.dbapi.addresses_get_by_pool("},{"line_number":571,"context_line":"            addrpool.id)"},{"line_number":572,"context_line":"        if addresses:"},{"line_number":573,"context_line":"            if cutils.is_initial_config_complete() and \\"},{"line_number":574,"context_line":"               (addrpool.name not in SYSTEM_CONTROLLER_ADDRPOOLS):"},{"line_number":575,"context_line":"                raise exception.AddressPoolInUseByAddresses()"},{"line_number":576,"context_line":"            else:"},{"line_number":577,"context_line":"                # Must be a request as a result of network reconfiguration"}],"source_content_type":"text/x-python","patch_set":1,"id":"e55cdd43_9e9a2596","line":574,"range":{"start_line":573,"start_character":13,"end_line":574,"end_character":66},"in_reply_to":"c4e73180_632fe52f","updated":"2021-04-26 21:21:51.000000000","message":"Done","commit_id":"4cb18fb378146df438f0499f7b98b2dea2b1f3f9"}]}