)]}'
{"watcher/decision_engine/strategy/strategies/outlet_temp_control.py":[{"author":{"_account_id":19055,"name":"Alexander Chadin","email":"joker946@gmail.com","username":"joker946"},"change_message_id":"54dacd41acfe0686d9efbbf7de26297aaed1114a","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Outlet (Exhaust Air) Temperature is one of the important thermal"},{"line_number":28,"context_line":"telemetries to measure thermal/workload status of server."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This strategy makes decisions to migrate workloads to the hosts with good"},{"line_number":31,"context_line":"thermal condition (lowest outlet temperature) when the outlet temperature"},{"line_number":32,"context_line":"of source hosts reach a configurable threshold."},{"line_number":33,"context_line":"\"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_411be17f","line":32,"range":{"start_line":30,"start_character":0,"end_line":32,"end_character":47},"updated":"2018-03-30 08:14:13.000000000","message":"for what reason have you left it outside of class?","commit_id":"92dad3be2d7054f81c92aba5257d94aeb36d3acf"},{"author":{"_account_id":27947,"name":"Egor Panfilov","email":"edpanfilov@sbcloud.ru","username":"erakli"},"change_message_id":"a061a3f4abb51dd8ad08ad8bcf3b45ffade0c56c","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Outlet (Exhaust Air) Temperature is one of the important thermal"},{"line_number":28,"context_line":"telemetries to measure thermal/workload status of server."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This strategy makes decisions to migrate workloads to the hosts with good"},{"line_number":31,"context_line":"thermal condition (lowest outlet temperature) when the outlet temperature"},{"line_number":32,"context_line":"of source hosts reach a configurable threshold."},{"line_number":33,"context_line":"\"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_45b0c28a","line":32,"range":{"start_line":30,"start_character":0,"end_line":32,"end_character":47},"in_reply_to":"bf659307_411be17f","updated":"2018-03-30 22:31:37.000000000","message":"It is a copy of the same paragraph from doc/source/strategies/outlet_temp_control.rst. It was placed as plain text instead of reference to docstring of class. \n\nI found that there is a reference to this docstring in adition to class\u0027 docstring in generated index of codebase. Also, this class has docstring that explains it in a PoC way. I\u0027m not sure, is it a good idea to place this paragraph to the description of PoC.","commit_id":"92dad3be2d7054f81c92aba5257d94aeb36d3acf"}]}