)]}'
{"id":"openstack%2Fproject-config~680717","triplet_id":"openstack%2Fproject-config~master~I0c054714ca9a709bd58b85570a5c415a76cacd1c","project":"openstack/project-config","branch":"master","topic":"add-airship-groups","hashtags":[],"change_id":"I0c054714ca9a709bd58b85570a5c415a76cacd1c","subject":"Add per-project Airship groups","status":"MERGED","created":"2019-09-06 14:54:06.000000000","updated":"2019-09-09 20:35:06.000000000","submitted":"2019-09-09 20:35:06.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"680717-1568061306500-70508045","meta_rev_id":"80a957d33779a257312b7bf2e6e0f873fa91f3f5","_number":680717,"virtual_id_number":680717,"owner":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},{"value":2,"date":"2019-09-09 20:35:06.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2019-09-09 14:52:32.000000000","_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"all":[{"value":2,"date":"2019-09-09 20:20:50.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":2,"date":"2019-09-09 15:08:34.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":1,"date":"2019-09-09 17:34:10.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"all":[{"value":1,"date":"2019-09-09 20:20:50.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-09-09 15:08:34.000000000","updated_by":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"reviewer":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"state":"REVIEWER"},{"updated":"2019-09-09 17:34:10.000000000","updated_by":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"reviewer":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"state":"REVIEWER"},{"updated":"2019-09-09 20:20:50.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"REVIEWER"},{"updated":"2019-09-09 20:35:06.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"75a4514eadf0eac2bab1c08ffdfaf608d924d759","author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"date":"2019-09-06 14:54:06.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e449f70f9578380dde955412ae25bbc9dcad93d2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-06 15:07:21.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-zuul-jobs-linters https://zuul.opendev.org/t/openstack/build/9e261e41c80448b892a0ed660474d8b9 : SUCCESS in 6m 14s\n- project-config-gerrit https://zuul.opendev.org/t/openstack/build/1cfc940fb1a0400eb9488118dc07a0cb : SUCCESS in 4m 40s","accounts_in_message":[],"_revision_number":1},{"id":"3573f0734308cd90a2b66448dae278f50ca13e62","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2019-09-08 13:06:18.000000000","message":"Patch Set 1: Code-Review-1\n\nIs gerrit/acls/airship/airship.config still used after this change? If not, please remove.\n\n-1 since I didn\u0027t find any usage","accounts_in_message":[],"_revision_number":1},{"id":"49bbab0d1b045819733e10376ff8013fbd863258","author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"date":"2019-09-09 14:41:00.000000000","message":"Patch Set 1:\n\nWe do want to keep the airship-core group, so that it can be added as a member of the new groups.  If we can keep the airship-core group without the airship.config ACL, then we can certainly remove the ACL.","accounts_in_message":[],"_revision_number":1},{"id":"9b27f32ad3bf2cedca9020cf2d613d313abf7c48","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2019-09-09 14:44:21.000000000","message":"Patch Set 1:\n\nyou can keep the *group* but you don\u0027t need the ACL. Removing the file will not remove the group.\n\nIn this case: Update the commit message to explain that you want to keep the group.","accounts_in_message":[],"_revision_number":1},{"id":"3e699d509df4ac4fa96cd9b13ceff2272d9e0054","author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"date":"2019-09-09 14:52:03.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"05910aea4711efb5825031270a3ebf689a3ad766","author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"date":"2019-09-09 14:52:32.000000000","message":"Patch Set 2:\n\nThanks Andreas - done.","accounts_in_message":[],"_revision_number":2},{"id":"f030a2ae69d0c669917dfa1c7601eddeba4fd292","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-09 15:05:56.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-zuul-jobs-linters https://zuul.opendev.org/t/openstack/build/5d4dd5bd928640f2a004aedd031d40ce : SUCCESS in 8m 25s\n- project-config-gerrit https://zuul.opendev.org/t/openstack/build/4f80d588fa8d4172bfad87d6dd7fa4ca : SUCCESS in 5m 16s","accounts_in_message":[],"_revision_number":2},{"id":"898c33afcef70eb06c7886cf5072268bba11a9f9","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2019-09-09 15:08:16.000000000","message":"Patch Set 2: Code-Review-1\n\nwhere did you remove the airship.config file?","accounts_in_message":[],"_revision_number":2},{"id":"f0e0970bdab5e6db72cf4dee2b4a7337c78daaf0","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2019-09-09 15:08:34.000000000","message":"Patch Set 2: Code-Review+2\n\nfound it - a rename...","accounts_in_message":[],"_revision_number":2},{"id":"780a1e60614826079f97304d639ec59e4909ae31","author":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"date":"2019-09-09 17:34:10.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"b9c6b8a8675e651aa9479e0dd08591ab6c7d026a","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2019-09-09 20:20:50.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"98d80738a252abe8f5be0e11429a3a5518d1918c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-09 20:21:04.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"ce4b695e78e4f4ab186f2c37c1b81f66e49c1aba","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-09 20:35:06.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-zuul-jobs-linters https://zuul.opendev.org/t/openstack/build/a77de06606ab44f6ad37140cc60f0e4b : SUCCESS in 6m 33s\n- project-config-gerrit https://zuul.opendev.org/t/openstack/build/69428399a8324d5ab325b0d7cdbe64b0 : SUCCESS in 3m 17s","accounts_in_message":[],"_revision_number":2},{"id":"c2caa3da268ffd65bf8fbbeaa7fdb3fd09ab976e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-09 20:35:06.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"ee86e24c5bc9e810110a51a7110d07d58121d4d5","revisions":{"b3445f789b04729e6c10bab7680eaeaa0de51dc8":{"kind":"REWORK","_number":1,"created":"2019-09-06 14:54:06.000000000","uploader":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"ref":"refs/changes/17/680717/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/project-config","ref":"refs/changes/17/680717/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/project-config refs/changes/17/680717/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/project-config refs/changes/17/680717/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/project-config refs/changes/17/680717/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/project-config refs/changes/17/680717/1"}}},"commit":{"parents":[{"commit":"e1c2c349260566ae8db604a55c3f819d0e236ea0","subject":"Normalize projects.yaml","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/e1c2c349260566ae8db604a55c3f819d0e236ea0"}]}],"author":{"name":"Matt McEuen","email":"madgin@madgin.net","date":"2019-09-06 14:14:03.000000000","tz":-300},"committer":{"name":"Matt McEuen","email":"madgin@madgin.net","date":"2019-09-06 14:54:05.000000000","tz":-300},"subject":"Add per-project Airship groups","message":"Add per-project Airship groups\n\nPer discussion at the last PTG, new Airship projects have been\ngetting created with per-project core groups, instead of the older\nairship-core group.  This change catches up existing Airship projects\nto this setup, as they either have new core reviewers being added\nor will in the future.  A few projects which are closely and\nlogically related to other projects are configured to share their\nACLs for simplicity\u0027s sake.\n\nWith (or ahead of) this change, please seed the existing airship-core\ngroup as a member of all the newly created groups, to ensure\nuninterrupted review/merge ability.\n\nChange-Id: I0c054714ca9a709bd58b85570a5c415a76cacd1c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/b3445f789b04729e6c10bab7680eaeaa0de51dc8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/b3445f789b04729e6c10bab7680eaeaa0de51dc8"}]},"branch":"refs/heads/master"},"ee86e24c5bc9e810110a51a7110d07d58121d4d5":{"kind":"REWORK","_number":2,"created":"2019-09-09 14:52:03.000000000","uploader":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"ref":"refs/changes/17/680717/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/project-config","ref":"refs/changes/17/680717/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/project-config refs/changes/17/680717/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/project-config refs/changes/17/680717/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/project-config refs/changes/17/680717/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/project-config refs/changes/17/680717/2"}}},"commit":{"parents":[{"commit":"e1c2c349260566ae8db604a55c3f819d0e236ea0","subject":"Normalize projects.yaml","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/e1c2c349260566ae8db604a55c3f819d0e236ea0"}]}],"author":{"name":"Matt McEuen","email":"madgin@madgin.net","date":"2019-09-06 14:14:03.000000000","tz":-300},"committer":{"name":"Matt McEuen","email":"matt.mceuen@att.com","date":"2019-09-09 14:51:05.000000000","tz":-300},"subject":"Add per-project Airship groups","message":"Add per-project Airship groups\n\nPer discussion at the last PTG, new Airship projects have been\ngetting created with per-project core groups, instead of the older\nairship-core group.  This change catches up existing Airship projects\nto this setup, as they either have new core reviewers being added\nor will in the future.  A few projects which are closely and\nlogically related to other projects are configured to share their\nACLs for simplicity\u0027s sake.\n\nNote that this removes the airship.config ACL, but we do need to retain\nthe airship-core group (for now).\n\nWith (or ahead of) this change, please seed the existing airship-core\ngroup as a member of all the newly created groups, to ensure\nuninterrupted review/merge ability.\n\nChange-Id: I0c054714ca9a709bd58b85570a5c415a76cacd1c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/ee86e24c5bc9e810110a51a7110d07d58121d4d5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/project-config/commit/ee86e24c5bc9e810110a51a7110d07d58121d4d5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}