)]}'
{"neutron/agent/linux/keepalived.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"b929b0137b3037542248e5cb546158b1fae3eeea","unresolved":false,"context_lines":[{"line_number":374,"context_line":"        self.config \u003d config"},{"line_number":375,"context_line":"        self.namespace \u003d namespace"},{"line_number":376,"context_line":"        self.process_monitor \u003d process_monitor"},{"line_number":377,"context_line":"        self.conf_path \u003d conf_path or tempfile.mkdtemp()"},{"line_number":378,"context_line":"        # configure throttler for spawn to introduce delay between SIGHUPs,"},{"line_number":379,"context_line":"        # otherwise keepalived master may unnecessarily flip to slave"},{"line_number":380,"context_line":"        if throttle_restart_value is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_dc15bd9d","line":377,"updated":"2018-03-28 21:12:04.000000000","message":"this should be probably be hard-coded to a different value if we want to fix the violation as keepalived expect this to persists when it\u0027s my understanding that it isn\u0027t if we used mkdtemp(). That said, this is overridden in [1] so it\u0027s probably not a big deal. When I introduced the bandit check I didn\u0027t think this was a risky violation check to omit, but if we do want to fix it we might as well fix the tox entry as pointed out by Miguel.\n\nSo I see two actions:\n\na) use a different hard-coded value, not mkdtemp() and fix tox.ini as well.\nb) Leave this be, as it seems to be harmless.\n\nYour call, but thanks for doublechecking!\n\n[1] http://git.openstack.org/cgit/openstack/neutron/tree/neutron/agent/l3/ha_router.py#n132","commit_id":"5d08aa5961350a08df31e32cf2e6c3c4a9dca4c5"}]}