)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"change_message_id":"66da05b028000fc6477838070ce08db72939615a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"improve \u0027Restrict existing strategies to their default scope\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It adds \u0027disabled_reason\u0027 filed into \u0027ComputeNode\u0027 resource, to distinguish"},{"line_number":10,"context_line":"which nodes are disabled by Watcher and which are not by Watcher."},{"line_number":11,"context_line":"So to server_consolidation, it should react on the nodes disabled by Watcher,"},{"line_number":12,"context_line":"besides enabled nodes."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1f485f77_8695a835","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":60},"updated":"2017-11-16 12:58:22.000000000","message":"This patch does not add \u0027disabled_reason\u0027 field into compute node element.\nPlease update commit message.","commit_id":"5bb5adf7508ed67ad4b8ab0a2d9c6b55fe4740b7"},{"author":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"change_message_id":"66da05b028000fc6477838070ce08db72939615a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"besides enabled nodes."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5909b5619d46a4e3e49a3f9c0aa854052d32c645"},{"line_number":15,"context_line":"Implements:blueprint extend-node-status"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1f485f77_26a4b4a5","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"updated":"2017-11-16 12:58:22.000000000","message":"\"Partially-Implements\"","commit_id":"5bb5adf7508ed67ad4b8ab0a2d9c6b55fe4740b7"},{"author":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"change_message_id":"19a401853863bf85f0087e55ced17e2711d68810","unresolved":false,"context_lines":[{"line_number":12,"context_line":"besides enabled nodes."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5909b5619d46a4e3e49a3f9c0aa854052d32c645"},{"line_number":15,"context_line":"Implements:blueprint extend-node-status"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1f485f77_9e7b9b79","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"1f485f77_26a4b4a5","updated":"2017-11-16 23:36:21.000000000","message":"This patch seems not to be related to disabled_reason field.","commit_id":"5bb5adf7508ed67ad4b8ab0a2d9c6b55fe4740b7"}],"watcher/decision_engine/strategy/strategies/basic_consolidation.py":[{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"215b3f1eb2048003477f37738b9829bbaab5c47f","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        disabled_nodes \u003d self.get_disabled_compute_nodes_with_reason("},{"line_number":203,"context_line":"            reason\u003dself.REASON_FOR_DISABLE)"},{"line_number":204,"context_line":""},{"line_number":205,"context_line":"        return dict(enabled_nodes, **disabled_nodes)"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    def check_migration(self, source_node, destination_node,"},{"line_number":208,"context_line":"                        instance_to_migrate):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_955beb85","line":205,"range":{"start_line":205,"start_character":20,"end_line":205,"end_character":33},"updated":"2017-10-05 14:46:20.000000000","message":"enabled_nodes is dict and **disabled_nodes is dict too,  \nI think fuction dict() could not combinate two dict to one but undate() can","commit_id":"bcf8527be1a699ec67295cab7bfaed75875e0530"},{"author":{"_account_id":24501,"name":"sue","display_name":"suzhengwei","email":"sugar-2008@163.com","username":"suzhengwei"},"change_message_id":"270a08eecabc4e5ef4dec0d6444b7a9e4cf6bc4a","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        disabled_nodes \u003d self.get_disabled_compute_nodes_with_reason("},{"line_number":203,"context_line":"            reason\u003dself.REASON_FOR_DISABLE)"},{"line_number":204,"context_line":""},{"line_number":205,"context_line":"        return dict(enabled_nodes, **disabled_nodes)"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    def check_migration(self, source_node, destination_node,"},{"line_number":208,"context_line":"                        instance_to_migrate):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_cd0e2f3a","line":205,"range":{"start_line":205,"start_character":20,"end_line":205,"end_character":33},"in_reply_to":"7f515b1d_955beb85","updated":"2017-10-09 06:55:51.000000000","message":"The function dict() can combinate two dict to one.\nYou can try a little test.","commit_id":"bcf8527be1a699ec67295cab7bfaed75875e0530"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"c7ab821ab6b86ae138cbddfe9ce050e4ee98ee66","unresolved":false,"context_lines":[{"line_number":202,"context_line":"        disabled_nodes \u003d self.get_disabled_compute_nodes_with_reason("},{"line_number":203,"context_line":"            reason\u003dself.REASON_FOR_DISABLE)"},{"line_number":204,"context_line":""},{"line_number":205,"context_line":"        return dict(enabled_nodes, **disabled_nodes)"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    def check_migration(self, source_node, destination_node,"},{"line_number":208,"context_line":"                        instance_to_migrate):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_f8564bce","line":205,"range":{"start_line":205,"start_character":20,"end_line":205,"end_character":33},"in_reply_to":"7f515b1d_cd0e2f3a","updated":"2017-10-09 07:09:47.000000000","message":"sorry, my fault, this ok.","commit_id":"bcf8527be1a699ec67295cab7bfaed75875e0530"}]}
