)]}'
{"trove/extensions/mgmt/instances/models.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"3b5320711e5fa9bb41cd19c0093c423528558881","unresolved":false,"context_lines":[{"line_number":188,"context_line":"    @staticmethod"},{"line_number":189,"context_line":"    def _get_audit_period():"},{"line_number":190,"context_line":"        now \u003d datetime.datetime.now()"},{"line_number":191,"context_line":"        audit_start \u003d utils.isotime("},{"line_number":192,"context_line":"            now + datetime.timedelta("},{"line_number":193,"context_line":"                seconds\u003dCONF.exists_notification_interval),"},{"line_number":194,"context_line":"            subsecond\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_8472777b","line":191,"updated":"2016-01-20 15:11:00.000000000","message":"as best as I can tell, this will ensure that audit_end \u003c audit_start since CONF.exists_notification_interval is generally assumed to be a positive number.\n\nI.e. I\u0027m asserting that\n\nnow + positive_number \u003e now\n\nWhich seems like the wrong thing to do.\n\nIf this code is correct, a clarification as a comment would be highly advisable.","commit_id":"667d6460e9c0380b4666a118080b0c75a285aae3"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"05536a13acad6e0d8bc471a7cc0259e65a636f6d","unresolved":false,"context_lines":[{"line_number":188,"context_line":"    @staticmethod"},{"line_number":189,"context_line":"    def _get_audit_period():"},{"line_number":190,"context_line":"        now \u003d datetime.datetime.now()"},{"line_number":191,"context_line":"        audit_start \u003d utils.isotime("},{"line_number":192,"context_line":"            now + datetime.timedelta("},{"line_number":193,"context_line":"                seconds\u003dCONF.exists_notification_interval),"},{"line_number":194,"context_line":"            subsecond\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_951dbf11","line":191,"in_reply_to":"9a68dd71_8472777b","updated":"2016-01-20 15:52:54.000000000","message":"You are right, I need to change + for - in L192. That way we take the recently completed audit period.","commit_id":"667d6460e9c0380b4666a118080b0c75a285aae3"}]}
