)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b5e6284c124cafd10ccdf71d14b0cc3d2d32838d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c00353ca_710ff1d2","updated":"2022-01-06 10:09:40.000000000","message":"Hi,\n\nIt\u0027s suggested to make an intermediate release of python-tackerclient in [1]. I think it\u0027s better to include this change to the release although it\u0027s still WIP. I\u0027d appreciate if you fix my comments and propose non-WIP patch soon if possible.\n\nIf there is something to be fixed remained and it takes some times, I will make the intermediate release without this update. What do you think?\n\n[1] https://review.opendev.org/c/openstack/releases/+/823344","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"05f1f6b9_b0c190e0","in_reply_to":"c00353ca_710ff1d2","updated":"2022-01-13 04:24:12.000000000","message":"Fixed all comments.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b889dfca1fc4ee3ba9737b904638cb36ef174f11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"880a8e67_3001206f","updated":"2022-01-26 00:19:05.000000000","message":"Thanks for your patch, please kindly confirm my comment.","commit_id":"5d6724f58892d49e66433bd72bf3dc46a9e9cd19"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d92d725ed4583be84f3f0d439af0765d862b5279","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ce0d2687_0ecd7f08","updated":"2022-01-27 23:56:22.000000000","message":"Thanks for your rework, LGTM.","commit_id":"1c73f54664533715ebf08f2cf7d8c93f521b0fb4"}],"tackerclient/osc/v1/vnflcm/vnflcm_op_occs.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b5e6284c124cafd10ccdf71d14b0cc3d2d32838d","unresolved":true,"context_lines":[{"line_number":201,"context_line":"        )"},{"line_number":202,"context_line":"        return parser"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude_fields\u003dNone):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Returns:"}],"source_content_type":"text/x-python","patch_set":4,"id":"bcb08efa_fe05b201","line":204,"range":{"start_line":204,"start_character":29,"end_line":204,"end_character":48},"updated":"2022-01-06 10:09:40.000000000","message":"Why don\u0027t you give default value `[]` instead to remove useless checking for `None` later? And one more misc comment, you don\u0027t need to use such a postfix of the name `_fields` by using empty list, `excluded` is enough in my opinion.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":201,"context_line":"        )"},{"line_number":202,"context_line":"        return parser"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude_fields\u003dNone):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Returns:"}],"source_content_type":"text/x-python","patch_set":4,"id":"b1df4113_8d7564ed","line":204,"range":{"start_line":204,"start_character":29,"end_line":204,"end_character":48},"in_reply_to":"81fee73e_8d2ed1b9","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":201,"context_line":"        )"},{"line_number":202,"context_line":"        return parser"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude_fields\u003dNone):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Returns:"}],"source_content_type":"text/x-python","patch_set":4,"id":"81fee73e_8d2ed1b9","line":204,"range":{"start_line":204,"start_character":29,"end_line":204,"end_character":48},"in_reply_to":"bcb08efa_fe05b201","updated":"2022-01-13 04:24:12.000000000","message":"Fixed as per suggestion.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b5e6284c124cafd10ccdf71d14b0cc3d2d32838d","unresolved":true,"context_lines":[{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude_fields\u003dNone):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Returns:"},{"line_number":208,"context_line":"            attributes([attributes]): a list of table entry definitions."},{"line_number":209,"context_line":"            Each entry should be a tuple consisting of"}],"source_content_type":"text/x-python","patch_set":4,"id":"d6ddfcc7_f24b0ac6","line":206,"updated":"2022-01-06 10:09:40.000000000","message":"Why don\u0027t you add an a description for the argument `excluded_fields` here?","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude_fields\u003dNone):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Returns:"},{"line_number":208,"context_line":"            attributes([attributes]): a list of table entry definitions."},{"line_number":209,"context_line":"            Each entry should be a tuple consisting of"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f01594a_d3b0005e","line":206,"in_reply_to":"5422859e_46643dfd","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude_fields\u003dNone):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Returns:"},{"line_number":208,"context_line":"            attributes([attributes]): a list of table entry definitions."},{"line_number":209,"context_line":"            Each entry should be a tuple consisting of"}],"source_content_type":"text/x-python","patch_set":4,"id":"5422859e_46643dfd","line":206,"in_reply_to":"d6ddfcc7_f24b0ac6","updated":"2022-01-13 04:24:12.000000000","message":"Added description as per suggestion.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b5e6284c124cafd10ccdf71d14b0cc3d2d32838d","unresolved":true,"context_lines":[{"line_number":229,"context_line":"        ]"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        attributes \u003d []"},{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"}],"source_content_type":"text/x-python","patch_set":4,"id":"868d971a_ac8d2a9f","line":232,"updated":"2022-01-06 10:09:40.000000000","message":"It\u0027s no need if the default value is an empty list.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":229,"context_line":"        ]"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        attributes \u003d []"},{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"}],"source_content_type":"text/x-python","patch_set":4,"id":"da5928db_9ac3ded3","line":232,"in_reply_to":"868d971a_ac8d2a9f","updated":"2022-01-13 04:24:12.000000000","message":"Fixed as per suggestion.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        ]"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        attributes \u003d []"},{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"}],"source_content_type":"text/x-python","patch_set":4,"id":"156e860d_7eaca870","line":232,"in_reply_to":"da5928db_9ac3ded3","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b5e6284c124cafd10ccdf71d14b0cc3d2d32838d","unresolved":true,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        attributes \u003d []"},{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"},{"line_number":236,"context_line":"                if field[\u0027value\u0027] not in exclude_fields:"}],"source_content_type":"text/x-python","patch_set":4,"id":"fbe2f490_02e2e8ea","line":233,"updated":"2022-01-06 10:09:40.000000000","message":"Could I confirm why this `strip()` is required for the elements in `excluded_fields`?","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        attributes \u003d []"},{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"},{"line_number":236,"context_line":"                if field[\u0027value\u0027] not in exclude_fields:"}],"source_content_type":"text/x-python","patch_set":4,"id":"a8b4ed09_83adf0c3","line":233,"in_reply_to":"e0c97fd7_c8acefff","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        attributes \u003d []"},{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"},{"line_number":236,"context_line":"                if field[\u0027value\u0027] not in exclude_fields:"}],"source_content_type":"text/x-python","patch_set":4,"id":"e0c97fd7_c8acefff","line":233,"in_reply_to":"fbe2f490_02e2e8ea","updated":"2022-01-13 04:24:12.000000000","message":"The fix has done considering the case of spaces between parameter.\nLike \u0027VNF Instance ID\u0027, \u0027Operation State\u0027, But seems fix not needed.\nRemoved that fix.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b5e6284c124cafd10ccdf71d14b0cc3d2d32838d","unresolved":true,"context_lines":[{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"},{"line_number":236,"context_line":"                if field[\u0027value\u0027] not in exclude_fields:"},{"line_number":237,"context_line":"                    attributes.extend([(field[\u0027key\u0027], field[\u0027value\u0027],"},{"line_number":238,"context_line":"                                      tacker_osc_utils.LIST_BOTH)])"}],"source_content_type":"text/x-python","patch_set":4,"id":"3adff528_06327ec0","line":235,"updated":"2022-01-06 10:09:40.000000000","message":"ditto","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"},{"line_number":236,"context_line":"                if field[\u0027value\u0027] not in exclude_fields:"},{"line_number":237,"context_line":"                    attributes.extend([(field[\u0027key\u0027], field[\u0027value\u0027],"},{"line_number":238,"context_line":"                                      tacker_osc_utils.LIST_BOTH)])"}],"source_content_type":"text/x-python","patch_set":4,"id":"c34dfa64_d5cac687","line":235,"in_reply_to":"3adff528_06327ec0","updated":"2022-01-13 04:24:12.000000000","message":"Fix done.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":232,"context_line":"        if exclude_fields:"},{"line_number":233,"context_line":"            exclude_fields \u003d [x.strip() for x in exclude_fields]"},{"line_number":234,"context_line":"        for field in fields:"},{"line_number":235,"context_line":"            if exclude_fields:"},{"line_number":236,"context_line":"                if field[\u0027value\u0027] not in exclude_fields:"},{"line_number":237,"context_line":"                    attributes.extend([(field[\u0027key\u0027], field[\u0027value\u0027],"},{"line_number":238,"context_line":"                                      tacker_osc_utils.LIST_BOTH)])"}],"source_content_type":"text/x-python","patch_set":4,"id":"180e6652_91c0509f","line":235,"in_reply_to":"c34dfa64_d5cac687","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"b889dfca1fc4ee3ba9737b904638cb36ef174f11","unresolved":true,"context_lines":[{"line_number":201,"context_line":"        )"},{"line_number":202,"context_line":"        return parser"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude\u003d[]):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Args:"}],"source_content_type":"text/x-python","patch_set":8,"id":"de0788ec_fd8d355a","line":204,"range":{"start_line":204,"start_character":37,"end_line":204,"end_character":39},"updated":"2022-01-26 00:19:05.000000000","message":"mutable object should not be given as default argument.\nhttps://google.github.io/styleguide/pyguide.html#2124-decision\n\nHow about changing as follows?\n```\ndef get_attributes(self, exclude\u003dNone):\n    if exclude is None:\n        exclude \u003d []\n```","commit_id":"5d6724f58892d49e66433bd72bf3dc46a9e9cd19"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"1aaaf7c6758067f8f62af832d7864875d666b038","unresolved":true,"context_lines":[{"line_number":201,"context_line":"        )"},{"line_number":202,"context_line":"        return parser"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude\u003d[]):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Args:"}],"source_content_type":"text/x-python","patch_set":8,"id":"fb0b1425_4a185566","line":204,"range":{"start_line":204,"start_character":37,"end_line":204,"end_character":39},"in_reply_to":"de0788ec_fd8d355a","updated":"2022-01-27 04:59:37.000000000","message":"Thanks for sharing this information. I have updated code to take care this.","commit_id":"5d6724f58892d49e66433bd72bf3dc46a9e9cd19"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"d92d725ed4583be84f3f0d439af0765d862b5279","unresolved":false,"context_lines":[{"line_number":201,"context_line":"        )"},{"line_number":202,"context_line":"        return parser"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"    def get_attributes(self, exclude\u003d[]):"},{"line_number":205,"context_line":"        \"\"\"Get attributes."},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        Args:"}],"source_content_type":"text/x-python","patch_set":8,"id":"5022d78a_f38705ca","line":204,"range":{"start_line":204,"start_character":37,"end_line":204,"end_character":39},"in_reply_to":"fb0b1425_4a185566","updated":"2022-01-27 23:56:22.000000000","message":"I\u0027ve confirmed in PS9.","commit_id":"5d6724f58892d49e66433bd72bf3dc46a9e9cd19"}],"tackerclient/tests/unit/osc/v1/test_vnflcm_op_occs.py":[{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"7a1702c335e827d413d800f5dd98d7ca1b5c0625","unresolved":true,"context_lines":[{"line_number":33,"context_line":"                \u0027Is Automatic Invocation\u0027, \u0027Is Cancel Pending\u0027,"},{"line_number":34,"context_line":"                \u0027Error\u0027, \u0027Links\u0027]"},{"line_number":35,"context_line":"    elif action \u003d\u003d \u0027list\u0027:"},{"line_number":36,"context_line":"        exclude_fields \u003d [\u0027ID\u0027, \u0027Operation State\u0027, \u0027VNF Instance ID\u0027,"},{"line_number":37,"context_line":"                          \u0027Operation\u0027]"},{"line_number":38,"context_line":"        if parameter is not None:"},{"line_number":39,"context_line":"            return [\u0027VNF Instance ID\u0027, \u0027Operation State\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"3c4ae579_9814c60e","line":36,"updated":"2022-01-06 10:39:38.000000000","message":"Is this exclude fields?\nI think expected fields for example, or return it directly in else?","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":33,"context_line":"                \u0027Is Automatic Invocation\u0027, \u0027Is Cancel Pending\u0027,"},{"line_number":34,"context_line":"                \u0027Error\u0027, \u0027Links\u0027]"},{"line_number":35,"context_line":"    elif action \u003d\u003d \u0027list\u0027:"},{"line_number":36,"context_line":"        exclude_fields \u003d [\u0027ID\u0027, \u0027Operation State\u0027, \u0027VNF Instance ID\u0027,"},{"line_number":37,"context_line":"                          \u0027Operation\u0027]"},{"line_number":38,"context_line":"        if parameter is not None:"},{"line_number":39,"context_line":"            return [\u0027VNF Instance ID\u0027, \u0027Operation State\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"e1c3d823_275851d6","line":36,"in_reply_to":"3c4ae579_9814c60e","updated":"2022-01-13 04:24:12.000000000","message":"Fix done as per suggestion","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":33,"context_line":"                \u0027Is Automatic Invocation\u0027, \u0027Is Cancel Pending\u0027,"},{"line_number":34,"context_line":"                \u0027Error\u0027, \u0027Links\u0027]"},{"line_number":35,"context_line":"    elif action \u003d\u003d \u0027list\u0027:"},{"line_number":36,"context_line":"        exclude_fields \u003d [\u0027ID\u0027, \u0027Operation State\u0027, \u0027VNF Instance ID\u0027,"},{"line_number":37,"context_line":"                          \u0027Operation\u0027]"},{"line_number":38,"context_line":"        if parameter is not None:"},{"line_number":39,"context_line":"            return [\u0027VNF Instance ID\u0027, \u0027Operation State\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"8a76798b_8cc34f79","line":36,"in_reply_to":"e1c3d823_275851d6","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"7a1702c335e827d413d800f5dd98d7ca1b5c0625","unresolved":true,"context_lines":[{"line_number":452,"context_line":"        actual_columns, data \u003d self.list_vnflcm_op_occ.take_action(parsed_args)"},{"line_number":453,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":454,"context_line":"            self.list_vnflcm_op_occ.get_attributes("},{"line_number":455,"context_line":"                exclude_fields\u003d[\u0027ID\u0027, \u0027Operation\u0027]),"},{"line_number":456,"context_line":"            long_listing\u003dTrue)"},{"line_number":457,"context_line":"        expected_data \u003d []"},{"line_number":458,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"}],"source_content_type":"text/x-python","patch_set":4,"id":"e54b62e2_1b863fe1","line":455,"updated":"2022-01-06 10:39:38.000000000","message":"How about change one to \u0027VNF Instance ID\u0027, \u0027Operation State\u0027 for the same type of string?","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"fc13ce3ae3366cf00250b0684f067bb9bce4bcb1","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        actual_columns, data \u003d self.list_vnflcm_op_occ.take_action(parsed_args)"},{"line_number":453,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":454,"context_line":"            self.list_vnflcm_op_occ.get_attributes("},{"line_number":455,"context_line":"                exclude_fields\u003d[\u0027ID\u0027, \u0027Operation\u0027]),"},{"line_number":456,"context_line":"            long_listing\u003dTrue)"},{"line_number":457,"context_line":"        expected_data \u003d []"},{"line_number":458,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fde807a_d70d37ba","line":455,"in_reply_to":"d0f281c6_8833fec4","updated":"2022-01-13 04:25:00.000000000","message":"Done","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"},{"author":{"_account_id":33558,"name":"renu rani","display_name":"Renur","email":"rani.renu@gmail.com","username":"Renu"},"change_message_id":"816717f43ad02be86e232c64a78b999d7fdba58f","unresolved":true,"context_lines":[{"line_number":452,"context_line":"        actual_columns, data \u003d self.list_vnflcm_op_occ.take_action(parsed_args)"},{"line_number":453,"context_line":"        headers, columns \u003d tacker_osc_utils.get_column_definitions("},{"line_number":454,"context_line":"            self.list_vnflcm_op_occ.get_attributes("},{"line_number":455,"context_line":"                exclude_fields\u003d[\u0027ID\u0027, \u0027Operation\u0027]),"},{"line_number":456,"context_line":"            long_listing\u003dTrue)"},{"line_number":457,"context_line":"        expected_data \u003d []"},{"line_number":458,"context_line":"        for vnflcm_op_occ_obj_idx in vnflcm_op_occs_obj:"}],"source_content_type":"text/x-python","patch_set":4,"id":"d0f281c6_8833fec4","line":455,"in_reply_to":"e54b62e2_1b863fe1","updated":"2022-01-13 04:24:12.000000000","message":"Updated code to consider  \u0027VNF Instance ID\u0027, \u0027Operation State\u0027.","commit_id":"e349c5ac363ab3472f4e8652d3cbd1cdb357232e"}]}