)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"5a7675b4cc05192040f7fa39872971bd77515cdd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ad6cf281_3a419b6d","updated":"2022-12-26 12:22:40.000000000","message":"Thank you for your review. Please kindly check my replies and updated PS.","commit_id":"8da9e1f51538403f83ef64ffc68f042e4f30f39c"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"af5eb3947d74a22abbc6bc132b1b6c92ddf72170","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"582fc901_56fc4644","updated":"2022-12-27 23:47:06.000000000","message":"Thank you for your review.","commit_id":"5cac5728543053c0270352c99413002ffb15c53a"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5141f27e671c6f533115820a64dbda4129197c5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"326313e7_a4328816","updated":"2022-12-28 04:21:02.000000000","message":"Thanks for your patch, LGTM.","commit_id":"2f959940bcd93c59a7bc5f2ab7588303481b07db"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"04eb530b1cea835bfcd6b62bb8c9b38c8b2814ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"abe8e0ea_7fb5e539","updated":"2022-12-28 00:02:21.000000000","message":"Thanks for your rework！ LGTM","commit_id":"2f959940bcd93c59a7bc5f2ab7588303481b07db"}],"specs/2023.1/enhance-tacker-policy.rst":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c9ea958ab21bd8344e1bc10d568f8045ad435fe5","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"  seqdiag {"},{"line_number":90,"context_line":"    Client -\u003e Keystonemiddleware [label \u003d \"request\"];"},{"line_number":91,"context_line":"    Keystonemiddleware -\u003e \"API Process\" [label \u003d \"1. Request with user info\"];"},{"line_number":92,"context_line":"    \"API Process\" -\u003e TackerDB [label \u003d \"2. Get the accessed \\nresources from the TackerDB.\"]"},{"line_number":93,"context_line":"    \"API Process\" \u003c-- TackerDB [label \u003d \"return the accessed\\n resources\"];"},{"line_number":94,"context_line":"    \"API Process\" -\u003e \"API Process\" [label \u003d \"3. Get required \\nresource \\nattributes \\nfrom accessed \\nresources\"];"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aad6207_e2f98ffb","line":91,"updated":"2022-12-22 02:45:10.000000000","message":"Regarding sentences 1-7, please unify whether to put a period at the end of the sentence.","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c7049f07be3ca4cca6c9c3e62080cb82010d18c7","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"  seqdiag {"},{"line_number":90,"context_line":"    Client -\u003e Keystonemiddleware [label \u003d \"request\"];"},{"line_number":91,"context_line":"    Keystonemiddleware -\u003e \"API Process\" [label \u003d \"1. Request with user info\"];"},{"line_number":92,"context_line":"    \"API Process\" -\u003e TackerDB [label \u003d \"2. Get the accessed \\nresources from the TackerDB.\"]"},{"line_number":93,"context_line":"    \"API Process\" \u003c-- TackerDB [label \u003d \"return the accessed\\n resources\"];"},{"line_number":94,"context_line":"    \"API Process\" -\u003e \"API Process\" [label \u003d \"3. Get required \\nresource \\nattributes \\nfrom accessed \\nresources\"];"}],"source_content_type":"text/x-rst","patch_set":1,"id":"d66690e4_6f12173e","line":91,"in_reply_to":"081c8163_a696e6ca","updated":"2022-12-27 06:55:35.000000000","message":"Done","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"5a7675b4cc05192040f7fa39872971bd77515cdd","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"  seqdiag {"},{"line_number":90,"context_line":"    Client -\u003e Keystonemiddleware [label \u003d \"request\"];"},{"line_number":91,"context_line":"    Keystonemiddleware -\u003e \"API Process\" [label \u003d \"1. Request with user info\"];"},{"line_number":92,"context_line":"    \"API Process\" -\u003e TackerDB [label \u003d \"2. Get the accessed \\nresources from the TackerDB.\"]"},{"line_number":93,"context_line":"    \"API Process\" \u003c-- TackerDB [label \u003d \"return the accessed\\n resources\"];"},{"line_number":94,"context_line":"    \"API Process\" -\u003e \"API Process\" [label \u003d \"3. Get required \\nresource \\nattributes \\nfrom accessed \\nresources\"];"}],"source_content_type":"text/x-rst","patch_set":1,"id":"081c8163_a696e6ca","line":91,"in_reply_to":"9aad6207_e2f98ffb","updated":"2022-12-26 12:22:40.000000000","message":"I fixed it in PS2. (delete a period at the end of the sentence.)\nIn addition, I fixed it in the flow of a policy filter too.","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c9ea958ab21bd8344e1bc10d568f8045ad435fe5","unresolved":true,"context_lines":[{"line_number":830,"context_line":"* VENDOR_all"},{"line_number":831,"context_line":"* NAMESPACE_all"},{"line_number":832,"context_line":""},{"line_number":833,"context_line":"The tenant(area) manager needs to be assigned the following roles:"},{"line_number":834,"context_line":""},{"line_number":835,"context_line":"* manager"},{"line_number":836,"context_line":"* AREA_area_A@region_A (or AREA_area_B@region_A or"},{"line_number":837,"context_line":"  AREA_area_A@region_B or AREA_area_B@region_B)"},{"line_number":838,"context_line":"* VENDOR_all"},{"line_number":839,"context_line":"* NAMESPACE_all"},{"line_number":840,"context_line":""},{"line_number":841,"context_line":"The tenant user needs to be assigned the following roles:"},{"line_number":842,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"24cfd1ab_4aa02b1a","line":839,"range":{"start_line":833,"start_character":0,"end_line":839,"end_character":15},"updated":"2022-12-22 02:45:10.000000000","message":"It seems to be the same description as L.818-824.\nIs it necessary to write \"tenant(area) manager\" and \"area manager\" separately?","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"5a7675b4cc05192040f7fa39872971bd77515cdd","unresolved":true,"context_lines":[{"line_number":830,"context_line":"* VENDOR_all"},{"line_number":831,"context_line":"* NAMESPACE_all"},{"line_number":832,"context_line":""},{"line_number":833,"context_line":"The tenant(area) manager needs to be assigned the following roles:"},{"line_number":834,"context_line":""},{"line_number":835,"context_line":"* manager"},{"line_number":836,"context_line":"* AREA_area_A@region_A (or AREA_area_B@region_A or"},{"line_number":837,"context_line":"  AREA_area_A@region_B or AREA_area_B@region_B)"},{"line_number":838,"context_line":"* VENDOR_all"},{"line_number":839,"context_line":"* NAMESPACE_all"},{"line_number":840,"context_line":""},{"line_number":841,"context_line":"The tenant user needs to be assigned the following roles:"},{"line_number":842,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"cf8e36bd_ea3e3767","line":839,"range":{"start_line":833,"start_character":0,"end_line":839,"end_character":15},"in_reply_to":"24cfd1ab_4aa02b1a","updated":"2022-12-26 12:22:40.000000000","message":"Thank you for your comments.Their typical roles are indeed the same.\nThe reason why they are written separately is because they are different managers.\nThe difference between \"tenant(area) manager\" and \"area manager\" is the owned project:\n- \"tenant(area) manager\" generally has one project\n- \"area manager\" has multiple projects\n\nTherefore, I added Note about the difference between \"tenant(area) manager\" and \"area manager\" in PS2.","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"04eb530b1cea835bfcd6b62bb8c9b38c8b2814ef","unresolved":false,"context_lines":[{"line_number":830,"context_line":"* VENDOR_all"},{"line_number":831,"context_line":"* NAMESPACE_all"},{"line_number":832,"context_line":""},{"line_number":833,"context_line":"The tenant(area) manager needs to be assigned the following roles:"},{"line_number":834,"context_line":""},{"line_number":835,"context_line":"* manager"},{"line_number":836,"context_line":"* AREA_area_A@region_A (or AREA_area_B@region_A or"},{"line_number":837,"context_line":"  AREA_area_A@region_B or AREA_area_B@region_B)"},{"line_number":838,"context_line":"* VENDOR_all"},{"line_number":839,"context_line":"* NAMESPACE_all"},{"line_number":840,"context_line":""},{"line_number":841,"context_line":"The tenant user needs to be assigned the following roles:"},{"line_number":842,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"d3852349_14e11598","line":839,"range":{"start_line":833,"start_character":0,"end_line":839,"end_character":15},"in_reply_to":"6a72dd38_aea4b6ca","updated":"2022-12-28 00:02:21.000000000","message":"Done","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"af5eb3947d74a22abbc6bc132b1b6c92ddf72170","unresolved":true,"context_lines":[{"line_number":830,"context_line":"* VENDOR_all"},{"line_number":831,"context_line":"* NAMESPACE_all"},{"line_number":832,"context_line":""},{"line_number":833,"context_line":"The tenant(area) manager needs to be assigned the following roles:"},{"line_number":834,"context_line":""},{"line_number":835,"context_line":"* manager"},{"line_number":836,"context_line":"* AREA_area_A@region_A (or AREA_area_B@region_A or"},{"line_number":837,"context_line":"  AREA_area_A@region_B or AREA_area_B@region_B)"},{"line_number":838,"context_line":"* VENDOR_all"},{"line_number":839,"context_line":"* NAMESPACE_all"},{"line_number":840,"context_line":""},{"line_number":841,"context_line":"The tenant user needs to be assigned the following roles:"},{"line_number":842,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"6a72dd38_aea4b6ca","line":839,"range":{"start_line":833,"start_character":0,"end_line":839,"end_character":15},"in_reply_to":"6c1ad5fa_88e774ae","updated":"2022-12-27 23:47:06.000000000","message":"Thank you for your comments. I would like to add your ideas to the spec.\nI fixed it in PS3.","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c7049f07be3ca4cca6c9c3e62080cb82010d18c7","unresolved":true,"context_lines":[{"line_number":830,"context_line":"* VENDOR_all"},{"line_number":831,"context_line":"* NAMESPACE_all"},{"line_number":832,"context_line":""},{"line_number":833,"context_line":"The tenant(area) manager needs to be assigned the following roles:"},{"line_number":834,"context_line":""},{"line_number":835,"context_line":"* manager"},{"line_number":836,"context_line":"* AREA_area_A@region_A (or AREA_area_B@region_A or"},{"line_number":837,"context_line":"  AREA_area_A@region_B or AREA_area_B@region_B)"},{"line_number":838,"context_line":"* VENDOR_all"},{"line_number":839,"context_line":"* NAMESPACE_all"},{"line_number":840,"context_line":""},{"line_number":841,"context_line":"The tenant user needs to be assigned the following roles:"},{"line_number":842,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"6c1ad5fa_88e774ae","line":839,"range":{"start_line":833,"start_character":0,"end_line":839,"end_character":15},"in_reply_to":"cf8e36bd_ea3e3767","updated":"2022-12-27 06:55:35.000000000","message":"If the roles are the same, I felt that it would be easier to understand if it was described as follows. How about combining them into one?\n```\nThe area manager and the tenant(area) manager need to be assigned the following roles:\n```","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c9ea958ab21bd8344e1bc10d568f8045ad435fe5","unresolved":true,"context_lines":[{"line_number":881,"context_line":"Other end user impact"},{"line_number":882,"context_line":"---------------------"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"As the resources created in the previous version of Tacker may not have enhanced"},{"line_number":885,"context_line":"policy attributes, if the enhanced policy attributes are used as comparison"},{"line_number":886,"context_line":"attributes in the policy rule, this rule will prevent users from accessing those"},{"line_number":887,"context_line":"resources without these attributes as the comparison result is always false."},{"line_number":888,"context_line":""},{"line_number":889,"context_line":"Performance Impact"},{"line_number":890,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"49529bdd_050f7597","line":887,"range":{"start_line":884,"start_character":0,"end_line":887,"end_character":76},"updated":"2022-12-22 02:45:10.000000000","message":"Considering migration, I think one idea is to set a default value if the required value is not set.\nFrom a security point of view, are you in the direction of not taking such measures?","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"5a7675b4cc05192040f7fa39872971bd77515cdd","unresolved":true,"context_lines":[{"line_number":881,"context_line":"Other end user impact"},{"line_number":882,"context_line":"---------------------"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"As the resources created in the previous version of Tacker may not have enhanced"},{"line_number":885,"context_line":"policy attributes, if the enhanced policy attributes are used as comparison"},{"line_number":886,"context_line":"attributes in the policy rule, this rule will prevent users from accessing those"},{"line_number":887,"context_line":"resources without these attributes as the comparison result is always false."},{"line_number":888,"context_line":""},{"line_number":889,"context_line":"Performance Impact"},{"line_number":890,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"69bb9e7e_fea81b20","line":887,"range":{"start_line":884,"start_character":0,"end_line":887,"end_character":76},"in_reply_to":"49529bdd_050f7597","updated":"2022-12-26 12:22:40.000000000","message":"Thank you for your comments.\nYes, I described this implementation policy and impact due to the security perspective.","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"c7049f07be3ca4cca6c9c3e62080cb82010d18c7","unresolved":false,"context_lines":[{"line_number":881,"context_line":"Other end user impact"},{"line_number":882,"context_line":"---------------------"},{"line_number":883,"context_line":""},{"line_number":884,"context_line":"As the resources created in the previous version of Tacker may not have enhanced"},{"line_number":885,"context_line":"policy attributes, if the enhanced policy attributes are used as comparison"},{"line_number":886,"context_line":"attributes in the policy rule, this rule will prevent users from accessing those"},{"line_number":887,"context_line":"resources without these attributes as the comparison result is always false."},{"line_number":888,"context_line":""},{"line_number":889,"context_line":"Performance Impact"},{"line_number":890,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"771dc3ff_cc665d2b","line":887,"range":{"start_line":884,"start_character":0,"end_line":887,"end_character":76},"in_reply_to":"69bb9e7e_fea81b20","updated":"2022-12-27 06:55:35.000000000","message":"Ack","commit_id":"8c90fd9fb9eda8fe0775a96a074b491366177a4d"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"5a7675b4cc05192040f7fa39872971bd77515cdd","unresolved":true,"context_lines":[{"line_number":859,"context_line":"* VENDOR_all"},{"line_number":860,"context_line":"* NAMESPACE_all"},{"line_number":861,"context_line":""},{"line_number":862,"context_line":"The vendor manager needs to be assigned the following roles:"},{"line_number":863,"context_line":"* manager"},{"line_number":864,"context_line":"* AREA_all@all"},{"line_number":865,"context_line":"* VENDOR_vendor_A (or VENDOR_vendor_B)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"2513498a_76b12414","line":862,"updated":"2022-12-26 12:22:40.000000000","message":"[FYI]\nTo enhance this spec, I added the vendor manager role in PS2.","commit_id":"8da9e1f51538403f83ef64ffc68f042e4f30f39c"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"af5eb3947d74a22abbc6bc132b1b6c92ddf72170","unresolved":false,"context_lines":[{"line_number":859,"context_line":"* VENDOR_all"},{"line_number":860,"context_line":"* NAMESPACE_all"},{"line_number":861,"context_line":""},{"line_number":862,"context_line":"The vendor manager needs to be assigned the following roles:"},{"line_number":863,"context_line":"* manager"},{"line_number":864,"context_line":"* AREA_all@all"},{"line_number":865,"context_line":"* VENDOR_vendor_A (or VENDOR_vendor_B)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5653f540_af4bf13d","line":862,"in_reply_to":"2513498a_76b12414","updated":"2022-12-27 23:47:06.000000000","message":"Done","commit_id":"8da9e1f51538403f83ef64ffc68f042e4f30f39c"}]}