)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25571,"name":"Rajat Sharma","email":"rajat.sharma@india.nec.com","username":"rajat29"},"change_message_id":"7d63383f9e8613c9db60c76f86cbc838b2b7373b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, watcher start actionplan command uses update request"},{"line_number":10,"context_line":"to start the action plan."},{"line_number":11,"context_line":"   PATCH /v1/action_plans¶"},{"line_number":12,"context_line":"This PS. will send the request in the POST /v1/actionplan/action"},{"line_number":13,"context_line":"REST API."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f6a8fd7_43342e93","line":11,"range":{"start_line":11,"start_character":25,"end_line":11,"end_character":26},"updated":"2018-04-24 04:18:40.000000000","message":"Nit","commit_id":"cdc26cd801cb92778417a3f17d973e72410928a9"},{"author":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"change_message_id":"d8dea5f827de0a9403d686de326f2521b8ab7008","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, watcher start actionplan command uses update request"},{"line_number":10,"context_line":"to start the action plan."},{"line_number":11,"context_line":"   PATCH /v1/action_plans"},{"line_number":12,"context_line":"This PS. will send the request in the POST /v1/actionplan/action"},{"line_number":13,"context_line":"REST API."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Depends-on: I5353e4aa58d1675d8afb94bea35d9b953514129a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9f6a8fd7_1f20909a","line":12,"updated":"2018-04-26 04:25:41.000000000","message":"Please mention the correct request\nPOST /v1/actionplan/actionplan_uuid/start","commit_id":"94bce00a459db6e0c34a3e62a6f5845f02296a11"}],"watcherclient/common/base.py":[{"author":{"_account_id":19055,"name":"Alexander Chadin","email":"joker946@gmail.com","username":"joker946"},"change_message_id":"b1d13675ea0e96b6a261c7e8c4c816f5c15d500a","unresolved":false,"context_lines":[{"line_number":126,"context_line":"        return [obj_class(self, res, loaded\u003dTrue) for res in data if res]"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    def _update(self, url, body, method\u003d\u0027PATCH\u0027, response_key\u003dNone):"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"        resp, body \u003d self.api.json_request(method, url, body\u003dbody)"},{"line_number":131,"context_line":"        # PATCH/PUT requests may not return a body"},{"line_number":132,"context_line":"        if body:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5f7c97a3_50db0dec","line":129,"updated":"2018-05-08 09:56:42.000000000","message":"remove it","commit_id":"e81e594f51773686019d9a6e343e9174d3a34a6c"}],"watcherclient/tests/unit/v1/test_action_plan.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5ff03089ea1a62b34df7ce7ee4df58f3e5a0c41b","unresolved":false,"context_lines":[{"line_number":38,"context_line":"UPDATED_ACTION_PLAN \u003d copy.deepcopy(ACTION_PLAN1)"},{"line_number":39,"context_line":"NEW_STATE \u003d \u0027PENDING\u0027"},{"line_number":40,"context_line":"UPDATED_ACTION_PLAN[\u0027state\u0027] \u003d NEW_STATE"},{"line_number":41,"context_line":"START_ACTION_PLAN \u003d copy.deepcopy(ACTION_PLAN1)"},{"line_number":42,"context_line":"START_ACTION_PLAN[\u0027state\u0027] \u003d NEW_STATE"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"fake_responses \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_5fe4df0c","line":41,"updated":"2018-04-25 09:21:46.000000000","message":"why a new variable is declared with same value as of UPDATED_ACTION_PLAN","commit_id":"94bce00a459db6e0c34a3e62a6f5845f02296a11"},{"author":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"change_message_id":"b9a09c5c93e25169e65bfe4d62dd6d29e8e94790","unresolved":false,"context_lines":[{"line_number":38,"context_line":"UPDATED_ACTION_PLAN \u003d copy.deepcopy(ACTION_PLAN1)"},{"line_number":39,"context_line":"NEW_STATE \u003d \u0027PENDING\u0027"},{"line_number":40,"context_line":"UPDATED_ACTION_PLAN[\u0027state\u0027] \u003d NEW_STATE"},{"line_number":41,"context_line":"START_ACTION_PLAN \u003d copy.deepcopy(ACTION_PLAN1)"},{"line_number":42,"context_line":"START_ACTION_PLAN[\u0027state\u0027] \u003d NEW_STATE"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"fake_responses \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_9a2365a7","line":41,"in_reply_to":"9f6a8fd7_5fe4df0c","updated":"2018-04-25 09:47:26.000000000","message":"START_ACTION_PLAN and UPDATED_ACTION_PLAN currently holds same value because NEW_STATE is same, currently watcher allows to start an action plan via PATCH API(state Transition RECOMMENDED-\u003ePENDING) also, In future when the Proposed START API gets merged, we may deprecate action plan start via PATCH API, then we need to update PATCH test cases also and in that case NEW_STATE will be different and START_ACTION_PLAN and UPDATE_ACTION_PLAN will hold different values.","commit_id":"94bce00a459db6e0c34a3e62a6f5845f02296a11"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"5ff03089ea1a62b34df7ce7ee4df58f3e5a0c41b","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    {"},{"line_number":76,"context_line":"        \u0027POST\u0027: ("},{"line_number":77,"context_line":"            {},"},{"line_number":78,"context_line":"            START_ACTION_PLAN,"},{"line_number":79,"context_line":"        ),"},{"line_number":80,"context_line":"    },"},{"line_number":81,"context_line":"    \u0027/v1/action_plans/detail?uuid\u003d%s\u0027 % ACTION_PLAN1[\u0027uuid\u0027]:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f6a8fd7_dfc1cf55","line":78,"updated":"2018-04-25 09:21:46.000000000","message":"why UPDATED_ACTION_PLAN is not used here?","commit_id":"94bce00a459db6e0c34a3e62a6f5845f02296a11"}],"watcherclient/v1/action_plan.py":[{"author":{"_account_id":19055,"name":"Alexander Chadin","email":"joker946@gmail.com","username":"joker946"},"change_message_id":"b1d13675ea0e96b6a261c7e8c4c816f5c15d500a","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    def _path(id\u003dNone, q_param\u003dNone):"},{"line_number":31,"context_line":"        if id and q_param:"},{"line_number":32,"context_line":"            return \u0027/v1/action_plans/%s/%s\u0027 % (id, q_param)\\"},{"line_number":33,"context_line":"                if id else \u0027/v1/action_plans\u0027"},{"line_number":34,"context_line":"        elif id:"},{"line_number":35,"context_line":"            return \u0027/v1/action_plans/%s\u0027 % id"},{"line_number":36,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5f7c97a3_50778dd5","line":33,"range":{"start_line":33,"start_character":16,"end_line":33,"end_character":45},"updated":"2018-05-08 09:56:42.000000000","message":"it is useless verification since you have already checked that id and q_param are True. Shouldn\u0027t it be removed?","commit_id":"e81e594f51773686019d9a6e343e9174d3a34a6c"}]}