)]}'
{"id":"openstack%2Fkeystone~447607","triplet_id":"openstack%2Fkeystone~master~I045d1f865348fcb3f9c1ddc63badbc1b5230c0d6","project":"openstack/keystone","branch":"master","topic":"bp/policy-docs","hashtags":[],"change_id":"I045d1f865348fcb3f9c1ddc63badbc1b5230c0d6","subject":"Move policy from RuleDefault to DocumentedRuleDefault 2","status":"ABANDONED","created":"2017-03-20 17:12:50.000000000","updated":"2017-03-23 15:11:11.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"659768f8ec83f0e8f36cb3b8fd2b812dd560857d","_number":447607,"virtual_id_number":447607,"owner":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"actions":{},"labels":{"Verified":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"}],"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":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-03-20 21:14:29.000000000","updated_by":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"reviewer":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"state":"REVIEWER"},{"updated":"2017-03-23 03:13:57.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"4027298965dfb6e17f4b835e2fa4f31a2eb24f1e","author":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"date":"2017-03-20 17:12:50.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3b425b05a68cd3e719df7b3d39720365c33062aa","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-20 18:00:46.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-keystone-docs-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-keystone-docs-ubuntu-xenial/227ec7c/ : FAILURE in 3m 16s\n- gate-keystone-pep8-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-keystone-pep8-ubuntu-xenial/02ccd1f/ : SUCCESS in 6m 33s\n- gate-keystone-python27-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-keystone-python27-ubuntu-xenial/58a5b42/ : FAILURE in 3m 02s\n- gate-keystone-python35 http://logs.openstack.org/07/447607/1/check/gate-keystone-python35/abfd37c/ : FAILURE in 2m 47s\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe85a9a/ : FAILURE in 8m 14s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/fe8f406/ : FAILURE in 41m 49s\n- gate-tempest-dsvm-py35-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-tempest-dsvm-py35-ubuntu-xenial/eb50600/ : FAILURE in 14m 26s\n- gate-keystoneclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/07/447607/1/check/gate-keystoneclient-dsvm-functional-ubuntu-xenial-nv/20655c4/ : FAILURE in 7m 52s (non-voting)\n- gate-keystone-dsvm-functional-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/gate-keystone-dsvm-functional-ubuntu-xenial/54b4357/ : FAILURE in 15m 38s\n- gate-keystone-dsvm-functional-v3-only-ubuntu-xenial-nv http://logs.openstack.org/07/447607/1/check/gate-keystone-dsvm-functional-v3-only-ubuntu-xenial-nv/560b85a/ : FAILURE in 16m 01s (non-voting)\n- gate-keystone-dsvm-py35-functional-v3-only-ubuntu-xenial-nv http://logs.openstack.org/07/447607/1/check/gate-keystone-dsvm-py35-functional-v3-only-ubuntu-xenial-nv/ae75557/ : FAILURE in 15m 28s (non-voting)\n- gate-tempest-dsvm-keystone-uwsgi-full-ubuntu-xenial-nv http://logs.openstack.org/07/447607/1/check/gate-tempest-dsvm-keystone-uwsgi-full-ubuntu-xenial-nv/b65a69a/ : FAILURE in 15m 57s (non-voting)\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/07/447607/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/09ff308/ : FAILURE in 14m 18s (non-voting)\n- keystone-coverage-ubuntu-xenial http://logs.openstack.org/07/447607/1/check/keystone-coverage-ubuntu-xenial/d63904f/ : FAILURE in 2m 37s","accounts_in_message":[],"_revision_number":1},{"id":"09ff8d1c5912f19cbfee1c9650f0b1cf9792359c","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2017-03-20 21:14:29.000000000","message":"Patch Set 1:\n\nYeah - this is all going to fail until we have a new release of oslo.policy. Just waiting on a couple last minute nits on https://review.openstack.org/#/c/443330/ and we can do a new release.","accounts_in_message":[],"_revision_number":1},{"id":"dddc98139417055f46eed63fc7aafdf0cbaf4ded","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2017-03-22 20:12:13.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nThanks for getting an early start on this.","accounts_in_message":[],"_revision_number":1},{"id":"af4f29d79be041a53d83525318a0d3d944d000dc","author":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"date":"2017-03-22 21:07:51.000000000","message":"Topic changed from bp/policy-in-code to bp/policy-docs","accounts_in_message":[],"_revision_number":1},{"id":"03e040f44ce5419ea185b3a378f26312d5d98a22","author":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"date":"2017-03-22 21:33:30.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"45167912a5871c33453f637ac7da9d6e30d86b46","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-22 22:54:21.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-docs-ubuntu-xenial http://docs-draft.openstack.org/07/447607/2/check/gate-keystone-docs-ubuntu-xenial/234a4ac//doc/build/html/ : SUCCESS in 3m 03s\n- gate-keystone-pep8-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/gate-keystone-pep8-ubuntu-xenial/2eea56c/ : SUCCESS in 3m 15s\n- gate-keystone-python27-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/gate-keystone-python27-ubuntu-xenial/384ba8b/ : SUCCESS in 12m 37s\n- gate-keystone-python35 http://logs.openstack.org/07/447607/2/check/gate-keystone-python35/2c76247/ : SUCCESS in 12m 23s\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/3235f44/ : SUCCESS in 1h 09m 00s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/380d63a/ : SUCCESS in 36m 33s\n- gate-tempest-dsvm-py35-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/gate-tempest-dsvm-py35-ubuntu-xenial/f910a44/ : SUCCESS in 1h 00m 57s\n- gate-keystoneclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/07/447607/2/check/gate-keystoneclient-dsvm-functional-ubuntu-xenial-nv/2c2df52/ : SUCCESS in 28m 55s (non-voting)\n- gate-keystone-dsvm-functional-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/gate-keystone-dsvm-functional-ubuntu-xenial/fa0a7cc/ : SUCCESS in 28m 32s\n- gate-keystone-dsvm-functional-v3-only-ubuntu-xenial-nv http://logs.openstack.org/07/447607/2/check/gate-keystone-dsvm-functional-v3-only-ubuntu-xenial-nv/bf5eb34/ : SUCCESS in 33m 52s (non-voting)\n- gate-keystone-dsvm-py35-functional-v3-only-ubuntu-xenial-nv http://logs.openstack.org/07/447607/2/check/gate-keystone-dsvm-py35-functional-v3-only-ubuntu-xenial-nv/10f8604/ : SUCCESS in 29m 08s (non-voting)\n- gate-tempest-dsvm-keystone-uwsgi-full-ubuntu-xenial-nv http://logs.openstack.org/07/447607/2/check/gate-tempest-dsvm-keystone-uwsgi-full-ubuntu-xenial-nv/c3e31d5/ : FAILURE in 41m 47s (non-voting)\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/07/447607/2/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/433c6ed/ : SUCCESS in 1h 17m 06s (non-voting)\n- keystone-coverage-ubuntu-xenial http://logs.openstack.org/07/447607/2/check/keystone-coverage-ubuntu-xenial/27c6a77/ : SUCCESS in 13m 20s","accounts_in_message":[],"_revision_number":2},{"id":"4b72d3d84c1b3c48eef2e71573624f0cf21a2a4d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-23 03:13:57.000000000","message":"Patch Set 2: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":2},{"id":"29dacf7f7c027e01addcbc853a66e6866d98e55f","author":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"date":"2017-03-23 14:39:10.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"659768f8ec83f0e8f36cb3b8fd2b812dd560857d","author":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"date":"2017-03-23 15:11:11.000000000","message":"Abandoned\n\nAbandon, will separate each policy update into single commits so reviewers know directly what\u0027s being changed.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"314fcbdfc9d4937dd2541bdb138d508bba5d60ab","revisions":{"10af96500f333b19f6489d45f6f5dd59c0e20b62":{"kind":"REWORK","_number":1,"created":"2017-03-20 17:12:50.000000000","uploader":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"ref":"refs/changes/07/447607/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/07/447607/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/07/447607/1"}}},"commit":{"parents":[{"commit":"30afa481988c496ff9a713eb4dec1c6d204657a9","subject":"Policy in code (part 2)","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/30afa481988c496ff9a713eb4dec1c6d204657a9"}]}],"author":{"name":"Anthony Washington","email":"anthony.washington@intel.com","date":"2017-03-20 17:09:41.000000000","tz":0},"committer":{"name":"Anthony Washington","email":"anthony.washington@intel.com","date":"2017-03-20 17:12:30.000000000","tz":0},"subject":"Minor clean up for 435751","message":"Minor clean up for 435751\n\nWith the merge of [0] and the idea to add\nmore description to the generated policies. This patch changes\nthe class from policy.RuleDefault to policy.DocumentedRuleDefault\nwhich supports the more descriptive approach.\n\n[0] : https://review.openstack.org/#/c/439070/\nPartially-Implements: bp policy-in-code\n\nChange-Id: I045d1f865348fcb3f9c1ddc63badbc1b5230c0d6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/10af96500f333b19f6489d45f6f5dd59c0e20b62"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/10af96500f333b19f6489d45f6f5dd59c0e20b62"}]},"branch":"refs/heads/master"},"84a57c89849ce9d4503e8c17526c052068d563f2":{"kind":"REWORK","_number":2,"created":"2017-03-22 21:33:30.000000000","uploader":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"ref":"refs/changes/07/447607/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/07/447607/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/07/447607/2"}}},"commit":{"parents":[{"commit":"7242125481bab503cfaa14ce509a0730cee7624c","subject":"Policy in code (part 2)","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/7242125481bab503cfaa14ce509a0730cee7624c"}]}],"author":{"name":"Anthony Washington","email":"anthony.washington@intel.com","date":"2017-03-20 17:09:41.000000000","tz":0},"committer":{"name":"Anthony Washington","email":"anthony.washington@intel.com","date":"2017-03-22 21:28:44.000000000","tz":0},"subject":"Move policy from RuleDefault to DocumentedRuleDefault 2","message":"Move policy from RuleDefault to DocumentedRuleDefault 2\n\nThis commit defines richer policy objects using the new\ndefined policy.DocumentedRuleDefault class for:\n        - credential policies\n        - domain policies\n        - ec2 credential policies\n        - group policies\n        - project policies\n        - user policies\n\nPartially-Implements: bp policy-docs\nChange-Id: I045d1f865348fcb3f9c1ddc63badbc1b5230c0d6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/84a57c89849ce9d4503e8c17526c052068d563f2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/84a57c89849ce9d4503e8c17526c052068d563f2"}]},"branch":"refs/heads/master"},"314fcbdfc9d4937dd2541bdb138d508bba5d60ab":{"kind":"REWORK","_number":3,"created":"2017-03-23 14:39:10.000000000","uploader":{"_account_id":22219,"name":"Anthony Washington","email":"anthony.washington@intel.com","username":"antwash"},"ref":"refs/changes/07/447607/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/07/447607/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/07/447607/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/07/447607/3"}}},"commit":{"parents":[{"commit":"98a712e7bc4d94d35a9b31f5c16aa56293643a9b","subject":"Merge \"Don\u0027t persist revocation events when deleting a role\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/98a712e7bc4d94d35a9b31f5c16aa56293643a9b"}]}],"author":{"name":"Anthony Washington","email":"anthony.washington@intel.com","date":"2017-03-20 17:09:41.000000000","tz":0},"committer":{"name":"Anthony Washington","email":"anthony.washington@intel.com","date":"2017-03-23 14:39:01.000000000","tz":0},"subject":"Move policy from RuleDefault to DocumentedRuleDefault 2","message":"Move policy from RuleDefault to DocumentedRuleDefault 2\n\nThis commit defines richer policy objects using the new\ndefined policy.DocumentedRuleDefault class for:\n        - credential policies\n        - domain policies\n        - ec2 credential policies\n        - group policies\n        - project policies\n        - user policies\n\nPartially-Implements: bp policy-docs\nChange-Id: I045d1f865348fcb3f9c1ddc63badbc1b5230c0d6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/314fcbdfc9d4937dd2541bdb138d508bba5d60ab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/314fcbdfc9d4937dd2541bdb138d508bba5d60ab"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}