)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"c4b3cc75b05272b0ffed6260ccf4f015be9db4b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3a72fbea_3b211bed","updated":"2022-11-03 20:14:11.000000000","message":"Hi Jie,\n\nThank you for putting together this change to create the new CFN repositories. I just have a few questions to clarify regarding the new repos.\n\nI saw that you are creating separate groups for every repo to hold access control information. Do you expect a different group of people to have approval rights within each repo or would this mainly be the same group everywhere?\n\nThanks,\nIldikó","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"bef0b390fe12bbb15be96124b7713c1e038389d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f2ce199e_16769b20","updated":"2022-11-03 17:07:42.000000000","message":"This is now a duplicate of https://review.opendev.org/c/openstack/project-config/+/863025.\n\nPlease learn the Gerrit workflow and do not submit duplicates, see https://docs.opendev.org/opendev/infra-manual/latest/developers.html .\n\nIn case you do by accident submit duplicates, please cleanup - so ABANDON those changes that are not needed.","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"5c84fc313c10908afaaa0dc72fe18a55873cc3e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0ef9a74b_42d80b6a","updated":"2022-11-03 17:13:11.000000000","message":"You need to follow the creator\u0027s guide and add the projects to Zuul as well, see https://docs.opendev.org/opendev/infra-manual/latest/creators.html#add-project-to-zuul .\n\nBefore we can merge this, you need to cleanup the top-level name as well, I just commented on one place but this needs changes everywhere.","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f4dd7925ea5539a82a8477f3395769bae8ee24b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f77cc0ff_4df4d639","in_reply_to":"f2ce199e_16769b20","updated":"2022-11-04 11:01:39.000000000","message":"Hi, \nsorry for the duplicates,\nMy account had some issue, so I deactivated it and re-registered.\nthe current change request is from newly registered account.\nbelow 2 commits was under the old account(inactive),  is it possible to delete is from backend?\n\nhttps://review.opendev.org/c/openstack/project-config/+/863112\nhttps://review.opendev.org/c/openstack/project-config/+/863025","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"28e3c341061c03acb2ace4c1e46630c9a9319b43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bcd36a7e_b957b043","in_reply_to":"f77cc0ff_4df4d639","updated":"2022-11-04 13:56:04.000000000","message":"I abandoned those two for you.","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f43221dd8f0635d9b352dcfa7ebff9fa7cb19952","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8de5fae1_20d7b398","updated":"2022-11-04 13:59:29.000000000","message":"You need to add the repos also to zuul/main.yaml.\n\nDo you have already existing content that you want to import or do you start with empty repositories and will add changes using the Gerrit workflow one by one?","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"b4a65e8b53455aef69df754a0be478671d2457a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f8d6c759_fd95595a","in_reply_to":"8de5fae1_20d7b398","updated":"2022-11-08 08:40:24.000000000","message":"Will start with empty repositories, and add changes using gerrit workflow.","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"d8f39d683d7e266c9bae170fbcc6a7871117b93b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e7eaa704_3aae4247","updated":"2022-11-15 02:15:50.000000000","message":"\u003e Patch Set 6: Code-Review+2 Workflow+1\n\u003e \n\u003e This probably would have been easier if all the projects shared the same Gerrit ACL and review groups to start with, but for the sake of expediency I\u0027ll go ahead and approve it as-is since it\u0027s been waiting a while.\n\nThanks for helping 😊\nnew tips learned~","commit_id":"1f4213ac919fb24587f411fc623dcd2623b1a031"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"efc8bc055078fc34ea530da3c35c45a71ad3120f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e65147ae_e0cc84a8","updated":"2022-11-10 21:14:19.000000000","message":"Hi Jie,\n\nThank you for the fixes, the patch looks good to me now.\n\nBest Regards,\nIldikó","commit_id":"1f4213ac919fb24587f411fc623dcd2623b1a031"}],"gerrit/projects.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"5c84fc313c10908afaaa0dc72fe18a55873cc3e7","unresolved":true,"context_lines":[{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: computing-force-network/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  acl-config: /home/gerrit2/acls/openinfra/cfn-overview.config"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"11fc1737_d17558fe","line":147,"range":{"start_line":147,"start_character":11,"end_line":147,"end_character":47},"updated":"2022-11-03 17:13:11.000000000","message":"If the above is the right top-level name, then fix the ACL config, it should go into the computing-force-network as well.\n\nSo, create the directory, move the ACL files around and remove the acl-config lines.","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f0e908339cdce6e9a87f573bde9a5fe10613c3e7","unresolved":false,"context_lines":[{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: computing-force-network/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  acl-config: /home/gerrit2/acls/openinfra/cfn-overview.config"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"a3fa57b9_f5de881f","line":147,"range":{"start_line":147,"start_character":11,"end_line":147,"end_character":47},"in_reply_to":"01c19739_ed8e3102","updated":"2022-11-08 13:36:02.000000000","message":"Done","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"c4b3cc75b05272b0ffed6260ccf4f015be9db4b1","unresolved":true,"context_lines":[{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: computing-force-network/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  acl-config: /home/gerrit2/acls/openinfra/cfn-overview.config"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3c4b1adc_982a9e2f","line":147,"range":{"start_line":147,"start_character":11,"end_line":147,"end_character":47},"in_reply_to":"11fc1737_d17558fe","updated":"2022-11-03 20:14:11.000000000","message":"To the point of namespace identifier, I wonder if \u0027computing-force-network\u0027 might be a bit too long? Could maybe the acronym \u0027cfn\u0027 work here as well?","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f4dd7925ea5539a82a8477f3395769bae8ee24b6","unresolved":true,"context_lines":[{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: computing-force-network/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  acl-config: /home/gerrit2/acls/openinfra/cfn-overview.config"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"01c19739_ed8e3102","line":147,"range":{"start_line":147,"start_character":11,"end_line":147,"end_character":47},"in_reply_to":"3c4b1adc_982a9e2f","updated":"2022-11-04 11:01:39.000000000","message":"Thanks Andreas and Ildiko for helping to review.\n\nI got misunderstanding about non-default acl-config file location, I thought it could be anywhere regardless top-level project name.\n\nAnyway, I referred other configs, now I understand the rules. will fix:)\n\n\nIldiko,\nyes \"cfn\" was also my initial thought, I will change it back :)","commit_id":"6f02e180806b9a66edde20312cc1bb6a094aa645"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f43221dd8f0635d9b352dcfa7ebff9fa7cb19952","unresolved":true,"context_lines":[{"line_number":141,"context_line":"    - airship"},{"line_number":142,"context_line":"- project: airship/zuul-airship-roles"},{"line_number":143,"context_line":"  description: Reusable Ansible roles for use in Airship Zuul jobs"},{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: cfn/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  groups:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"00effd0a_41506add","line":147,"range":{"start_line":144,"start_character":2,"end_line":147,"end_character":27},"updated":"2022-11-04 13:59:29.000000000","message":"the list is alphabetial, so cfn/ comes before cirros/","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"b4a65e8b53455aef69df754a0be478671d2457a5","unresolved":true,"context_lines":[{"line_number":141,"context_line":"    - airship"},{"line_number":142,"context_line":"- project: airship/zuul-airship-roles"},{"line_number":143,"context_line":"  description: Reusable Ansible roles for use in Airship Zuul jobs"},{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: cfn/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  groups:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e2046850_ea3e9c21","line":147,"range":{"start_line":144,"start_character":2,"end_line":147,"end_character":27},"in_reply_to":"00effd0a_41506add","updated":"2022-11-08 08:40:24.000000000","message":"comments received, many thanks, will fix","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f0e908339cdce6e9a87f573bde9a5fe10613c3e7","unresolved":false,"context_lines":[{"line_number":141,"context_line":"    - airship"},{"line_number":142,"context_line":"- project: airship/zuul-airship-roles"},{"line_number":143,"context_line":"  description: Reusable Ansible roles for use in Airship Zuul jobs"},{"line_number":144,"context_line":"- project: cirros/cirros"},{"line_number":145,"context_line":"  description: Provides linux disk and kernel/initramfs images."},{"line_number":146,"context_line":"  default-branch: main"},{"line_number":147,"context_line":"- project: cfn/cfn-overview"},{"line_number":148,"context_line":"  description: Overview contents for computing force network working group"},{"line_number":149,"context_line":"  use-storyboard: true"},{"line_number":150,"context_line":"  groups:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1534407c_ba58a795","line":147,"range":{"start_line":144,"start_character":2,"end_line":147,"end_character":27},"in_reply_to":"e2046850_ea3e9c21","updated":"2022-11-08 13:36:02.000000000","message":"Done","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f43221dd8f0635d9b352dcfa7ebff9fa7cb19952","unresolved":true,"context_lines":[{"line_number":150,"context_line":"  groups:"},{"line_number":151,"context_line":"    - computing-force-network"},{"line_number":152,"context_line":"- project: cfn/computing-native"},{"line_number":153,"context_line":"  description: computing native is to build a open and unified compiling platform"},{"line_number":154,"context_line":"    that eliminate the difference between heterogeneous hardwares like GPU,FPGA etc."},{"line_number":155,"context_line":"  use-storyboard: true"},{"line_number":156,"context_line":"  groups:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"cafa52d8_f8203d77","line":153,"range":{"start_line":153,"start_character":15,"end_line":153,"end_character":24},"updated":"2022-11-04 13:59:29.000000000","message":"Suggestion: STart with a capital letter in the description everywhere for consistency.","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f0e908339cdce6e9a87f573bde9a5fe10613c3e7","unresolved":false,"context_lines":[{"line_number":150,"context_line":"  groups:"},{"line_number":151,"context_line":"    - computing-force-network"},{"line_number":152,"context_line":"- project: cfn/computing-native"},{"line_number":153,"context_line":"  description: computing native is to build a open and unified compiling platform"},{"line_number":154,"context_line":"    that eliminate the difference between heterogeneous hardwares like GPU,FPGA etc."},{"line_number":155,"context_line":"  use-storyboard: true"},{"line_number":156,"context_line":"  groups:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"29255663_ed3b94c1","line":153,"range":{"start_line":153,"start_character":15,"end_line":153,"end_character":24},"in_reply_to":"cafa52d8_f8203d77","updated":"2022-11-08 13:36:02.000000000","message":"Done","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f43221dd8f0635d9b352dcfa7ebff9fa7cb19952","unresolved":true,"context_lines":[{"line_number":151,"context_line":"    - computing-force-network"},{"line_number":152,"context_line":"- project: cfn/computing-native"},{"line_number":153,"context_line":"  description: computing native is to build a open and unified compiling platform"},{"line_number":154,"context_line":"    that eliminate the difference between heterogeneous hardwares like GPU,FPGA etc."},{"line_number":155,"context_line":"  use-storyboard: true"},{"line_number":156,"context_line":"  groups:"},{"line_number":157,"context_line":"    - computing-force-network"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f3c2d1a_a85ee9f7","line":154,"range":{"start_line":154,"start_character":71,"end_line":154,"end_character":79},"updated":"2022-11-04 13:59:29.000000000","message":"space after \",\"","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f0e908339cdce6e9a87f573bde9a5fe10613c3e7","unresolved":false,"context_lines":[{"line_number":151,"context_line":"    - computing-force-network"},{"line_number":152,"context_line":"- project: cfn/computing-native"},{"line_number":153,"context_line":"  description: computing native is to build a open and unified compiling platform"},{"line_number":154,"context_line":"    that eliminate the difference between heterogeneous hardwares like GPU,FPGA etc."},{"line_number":155,"context_line":"  use-storyboard: true"},{"line_number":156,"context_line":"  groups:"},{"line_number":157,"context_line":"    - computing-force-network"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"b1a05df0_a847d9ab","line":154,"range":{"start_line":154,"start_character":71,"end_line":154,"end_character":79},"in_reply_to":"5f3c2d1a_a85ee9f7","updated":"2022-11-08 13:36:02.000000000","message":"Done","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f43221dd8f0635d9b352dcfa7ebff9fa7cb19952","unresolved":true,"context_lines":[{"line_number":167,"context_line":"  groups:"},{"line_number":168,"context_line":"    - computing-force-network"},{"line_number":169,"context_line":"- project: cfn/use-case-and-architecture"},{"line_number":170,"context_line":"  description: Contents for CFN definition, architecuture, use cases and e2e solution"},{"line_number":171,"context_line":"  use-storyboard: true"},{"line_number":172,"context_line":"  groups:"},{"line_number":173,"context_line":"    - computing-force-network"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"afdfc353_770e9095","line":170,"range":{"start_line":170,"start_character":44,"end_line":170,"end_character":57},"updated":"2022-11-04 13:59:29.000000000","message":"typo - architecture","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"},{"author":{"_account_id":35475,"name":"Jie Niu","email":"niujie@chinamobile.com","username":"jieniucm"},"change_message_id":"f0e908339cdce6e9a87f573bde9a5fe10613c3e7","unresolved":false,"context_lines":[{"line_number":167,"context_line":"  groups:"},{"line_number":168,"context_line":"    - computing-force-network"},{"line_number":169,"context_line":"- project: cfn/use-case-and-architecture"},{"line_number":170,"context_line":"  description: Contents for CFN definition, architecuture, use cases and e2e solution"},{"line_number":171,"context_line":"  use-storyboard: true"},{"line_number":172,"context_line":"  groups:"},{"line_number":173,"context_line":"    - computing-force-network"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"476674fa_b1bc899c","line":170,"range":{"start_line":170,"start_character":44,"end_line":170,"end_character":57},"in_reply_to":"afdfc353_770e9095","updated":"2022-11-08 13:36:02.000000000","message":"Done","commit_id":"f236dce8e510dad6f1cfe2e964dd69cc07ed3be4"}]}