)]}'
{"nova/scheduler/client/report.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"04ac71cc0247e709e14d78ae033f7be000d4ca24","unresolved":false,"context_lines":[{"line_number":722,"context_line":"            # microversion 1.7 not available so try the earlier way"},{"line_number":723,"context_line":"            # TODO(cdent): When we\u0027re happy that all placement"},{"line_number":724,"context_line":"            # servers support microversion 1.7 we can remove this"},{"line_number":725,"context_line":"            # call and the associated code."},{"line_number":726,"context_line":"            return self._get_or_create_resource_class(name)"},{"line_number":727,"context_line":"        else:"},{"line_number":728,"context_line":"            msg \u003d _LE(\"Failed to ensure resource class record with \""}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_6a2bd981","line":725,"updated":"2017-04-18 10:30:37.000000000","message":"Hmm, actually, I wonder if we should log a warning here, so note we are falling back, and placement should be upgraded before the next upgrade is performed?","commit_id":"024e3982ede6d53a3b0b1a61ac509e23f69f73b3"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"88aff18a2697d6d759d070978d812d479a10bd16","unresolved":false,"context_lines":[{"line_number":720,"context_line":"            return name"},{"line_number":721,"context_line":"        elif response.status_code \u003d\u003d 406:"},{"line_number":722,"context_line":"            # microversion 1.7 not available so try the earlier way"},{"line_number":723,"context_line":"            # TODO(cdent): When we\u0027re happy that all placement"},{"line_number":724,"context_line":"            # servers support microversion 1.7 we can remove this"},{"line_number":725,"context_line":"            # call and the associated code."},{"line_number":726,"context_line":"            LOG.debug(\u0027Falling back to placement API microverison 1.2 \u0027"},{"line_number":727,"context_line":"                      \u0027for resource class management.\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7ffa3b31_9c285e97","line":724,"range":{"start_line":723,"start_character":27,"end_line":724,"end_character":47},"updated":"2017-04-20 15:32:04.000000000","message":"so you mean bumping the minimum supported version to 1.7 for placement ? Wow, not sure it\u0027d be something soon-ish but okay.","commit_id":"0744427cf0af35bb5151087df5939454ed0037d1"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"2b67e3058eef0a7c173c628b01a7924a124e1b23","unresolved":false,"context_lines":[{"line_number":721,"context_line":"        elif response.status_code \u003d\u003d 406:"},{"line_number":722,"context_line":"            # microversion 1.7 not available so try the earlier way"},{"line_number":723,"context_line":"            # TODO(cdent): When we\u0027re happy that all placement"},{"line_number":724,"context_line":"            # servers support microversion 1.7 we can remove this"},{"line_number":725,"context_line":"            # call and the associated code."},{"line_number":726,"context_line":"            LOG.debug(\u0027Falling back to placement API microverison 1.2 \u0027"},{"line_number":727,"context_line":"                      \u0027for resource class management.\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7ffa3b31_6fea720f","line":724,"in_reply_to":"7ffa3b31_9c285e97","updated":"2017-04-20 15:36:08.000000000","message":"No, when all deployed placement servers _can_ support 1.7, then we don\u0027t need this failover. Which translates to: when all deployed placement servers are pike, then the resource tracker no longer needs to support ocata in this code block.","commit_id":"0744427cf0af35bb5151087df5939454ed0037d1"}]}