)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"d547f88e5f77773e9a9481a9cf7d155f1635e745","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove check if enabled for OneViewInspect"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since the Inspector already checks if inspect is enabled,"},{"line_number":10,"context_line":"the OneView drivers dont need to do this when creating"},{"line_number":11,"context_line":"OneViewInspect."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff346bd7_4d188a80","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":56},"updated":"2017-07-22 08:19:21.000000000","message":"Can you please elaborate where is Inspector checking for \u0027inspect\u0027 being enabled. It may be useful if you add the class/method name here.","commit_id":"a1d91c71e7796eb398cdc4ba6c7e7d3eea2875d9"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"f0c58535f98bbdf8898695ca0424ec46d85f4ee0","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove check if enabled for OneViewInspect"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since the Inspector already checks if inspect is enabled,"},{"line_number":10,"context_line":"the OneView drivers dont need to do this when creating"},{"line_number":11,"context_line":"OneViewInspect."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff346bd7_3a59da59","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":56},"in_reply_to":"ff346bd7_4d188a80","updated":"2017-07-24 15:10:45.000000000","message":"Done.","commit_id":"a1d91c71e7796eb398cdc4ba6c7e7d3eea2875d9"}],"ironic/drivers/oneview.py":[{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"44e69b511723a056bdfe7d237535537a726e4f50","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        self.management \u003d management.OneViewManagement()"},{"line_number":83,"context_line":"        self.boot \u003d pxe.PXEBoot()"},{"line_number":84,"context_line":"        self.deploy \u003d deploy.OneViewAgentDeploy()"},{"line_number":85,"context_line":"        self.inspect \u003d inspect.OneViewInspect()"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"class ISCSIPXEOneViewDriver(base.BaseDriver):"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff346bd7_bc002736","line":85,"range":{"start_line":85,"start_character":8,"end_line":85,"end_character":47},"updated":"2017-07-24 15:38:06.000000000","message":"This change will break the \u0027agent_pxe_oneview\u0027 driver. You do not need this change as this portion of the code will not get used when one would create a driver using \u0027oneview\u0027 hardware type. They would directly set the \u0027OneViewInspect\u0027 as the inspection interface. This code is part of classical driver \u0027agent_pxe_oneview\u0027.","commit_id":"4fbed6c507e250bbb204d643b90baa2a4e309775"}]}