)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c0b55df4aaab04b875963a1dd8b91afe52bf1ce8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Skip check config changed while restart_check_config not enable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The ipsec.conf.old and ipsec.secrets.old files can only generate"},{"line_number":10,"context_line":"in pluto start, so in restart we should check restart_check_config"},{"line_number":11,"context_line":"configuration firstly, then check ipsec.conf and ipsec.secrets"},{"line_number":12,"context_line":"changed."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_271628aa","line":9,"range":{"start_line":9,"start_character":56,"end_line":9,"end_character":64},"updated":"2019-05-12 01:32:17.000000000","message":"-\u003e be generated (?)","commit_id":"00b83f53902cec8d1bed58b4b372c9d525c12b17"},{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"ea6b05e2597ebb47a1e209986a80fd25e91fc751","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Skip check config changed while restart_check_config not enable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The ipsec.conf.old and ipsec.secrets.old files can only generate"},{"line_number":10,"context_line":"in pluto start, so in restart we should check restart_check_config"},{"line_number":11,"context_line":"configuration firstly, then check ipsec.conf and ipsec.secrets"},{"line_number":12,"context_line":"changed."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_70bdb800","line":9,"range":{"start_line":9,"start_character":56,"end_line":9,"end_character":64},"in_reply_to":"dfbec78f_271628aa","updated":"2019-05-12 10:19:37.000000000","message":"Done","commit_id":"00b83f53902cec8d1bed58b4b372c9d525c12b17"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c0b55df4aaab04b875963a1dd8b91afe52bf1ce8","unresolved":false,"context_lines":[{"line_number":10,"context_line":"in pluto start, so in restart we should check restart_check_config"},{"line_number":11,"context_line":"configuration firstly, then check ipsec.conf and ipsec.secrets"},{"line_number":12,"context_line":"changed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Iccaf1aed26f54eeb49ce1cf2cac12a820310968c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_674660c3","line":13,"updated":"2019-05-12 01:32:17.000000000","message":"I think this looks like a bug. Could you file a bug?","commit_id":"00b83f53902cec8d1bed58b4b372c9d525c12b17"},{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"ea6b05e2597ebb47a1e209986a80fd25e91fc751","unresolved":false,"context_lines":[{"line_number":10,"context_line":"in pluto start, so in restart we should check restart_check_config"},{"line_number":11,"context_line":"configuration firstly, then check ipsec.conf and ipsec.secrets"},{"line_number":12,"context_line":"changed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Iccaf1aed26f54eeb49ce1cf2cac12a820310968c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfbec78f_d0c4a493","line":13,"in_reply_to":"dfbec78f_674660c3","updated":"2019-05-12 10:19:37.000000000","message":"Done","commit_id":"00b83f53902cec8d1bed58b4b372c9d525c12b17"}],"neutron_vpnaas/services/vpn/device_drivers/ipsec.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9dccbf53984dba92b942f63fbac6f66896eab80b","unresolved":false,"context_lines":[{"line_number":560,"context_line":"    def restart(self):"},{"line_number":561,"context_line":"        \"\"\"Restart the process.\"\"\""},{"line_number":562,"context_line":"        if cfg.CONF.pluto.restart_check_config:"},{"line_number":563,"context_line":"            if not self._config_changed():"},{"line_number":564,"context_line":"                return"},{"line_number":565,"context_line":"        # stop() followed immediately by a start() runs the risk that the"},{"line_number":566,"context_line":"        # current pluto daemon has not had a chance to shutdown. We check"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_a064e34a","line":563,"updated":"2019-09-30 08:09:03.000000000","message":"nit: it can be in one \"if\":\n\n    if cfg.CONF.pluto.restart_check_config and not self._config_changed():\n        return","commit_id":"22b897227861e447c184d76942759c277b708901"},{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"25f5c01b0570a650f4a588814e0fe46fb8c95669","unresolved":false,"context_lines":[{"line_number":560,"context_line":"    def restart(self):"},{"line_number":561,"context_line":"        \"\"\"Restart the process.\"\"\""},{"line_number":562,"context_line":"        if cfg.CONF.pluto.restart_check_config:"},{"line_number":563,"context_line":"            if not self._config_changed():"},{"line_number":564,"context_line":"                return"},{"line_number":565,"context_line":"        # stop() followed immediately by a start() runs the risk that the"},{"line_number":566,"context_line":"        # current pluto daemon has not had a chance to shutdown. We check"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_d6e205d5","line":563,"in_reply_to":"3fa7e38b_a064e34a","updated":"2019-09-30 10:22:30.000000000","message":"Thanks, will update.","commit_id":"22b897227861e447c184d76942759c277b708901"}]}
