)]}'
{"id":"openstack%2Fkeystone~326152","triplet_id":"openstack%2Fkeystone~master~I192864675a8b34082e34f60caac770da76ca02aa","project":"openstack/keystone","branch":"master","hashtags":[],"change_id":"I192864675a8b34082e34f60caac770da76ca02aa","subject":"Revert \"Install necessary files in etc/\"","status":"MERGED","created":"2016-06-06 21:54:16.000000000","updated":"2016-06-29 08:42:33.000000000","submitted":"2016-06-29 02:19:49.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1fdaab2226d9a13fcef1137b3b8207c2d39cfedb","_number":326152,"virtual_id_number":326152,"owner":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":2,"date":"2016-06-29 02:19:49.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"disliked":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"all":[{"value":-1,"date":"2016-06-07 14:18:55.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},{"value":-1,"date":"2016-06-07 10:47:16.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"value":2,"date":"2016-06-10 23:22:50.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"date":"2016-06-06 21:54:45.000000000","_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"date":"2016-06-11 01:52:20.000000000","_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"value":1,"date":"2016-06-29 00:16:59.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-06-06 21:54:45.000000000","updated_by":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"reviewer":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"state":"REVIEWER"},{"updated":"2016-06-07 10:47:16.000000000","updated_by":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"reviewer":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"state":"REVIEWER"},{"updated":"2016-06-07 14:18:55.000000000","updated_by":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"reviewer":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"state":"REVIEWER"},{"updated":"2016-06-11 01:52:20.000000000","updated_by":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"reviewer":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"state":"REVIEWER"},{"updated":"2016-06-29 00:16:59.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2016-06-29 02:19:49.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"501a17402aab08890a26a3a3119aa9800e3593ad","author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"date":"2016-06-06 21:54:16.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"623381998faa290a52d5f1baa6c8b92ef3171a5f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-06 22:59:48.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 http://docs-draft.openstack.org/52/326152/1/check/gate-keystone-docs/ede9112//doc/build/html/ : SUCCESS in 4m 52s\n- gate-keystone-pep8 http://logs.openstack.org/52/326152/1/check/gate-keystone-pep8/43bfc40/ : SUCCESS in 2m 19s\n- gate-keystone-python27-db http://logs.openstack.org/52/326152/1/check/gate-keystone-python27-db/3401500/ : SUCCESS in 8m 46s\n- gate-keystone-python34-db http://logs.openstack.org/52/326152/1/check/gate-keystone-python34-db/46159d7/ : SUCCESS in 11m 42s\n- gate-tempest-dsvm-full http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-full/bcc0c81/ : SUCCESS in 42m 56s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-postgres-full/dbafb69/ : SUCCESS in 44m 44s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-neutron-full/14c0a07/ : FAILURE in 32m 35s\n- gate-grenade-dsvm http://logs.openstack.org/52/326152/1/check/gate-grenade-dsvm/3b09568/ : SUCCESS in 37m 20s\n- gate-keystone-tox-db-legacy_drivers http://logs.openstack.org/52/326152/1/check/gate-keystone-tox-db-legacy_drivers/a701c03/ : SUCCESS in 5m 56s\n- gate-keystone-dsvm-functional http://logs.openstack.org/52/326152/1/check/gate-keystone-dsvm-functional/8bd6508/ : SUCCESS in 23m 24s\n- gate-keystone-dsvm-functional-v3-only-nv http://logs.openstack.org/52/326152/1/check/gate-keystone-dsvm-functional-v3-only-nv/dc53ac2/ : SUCCESS in 22m 35s (non-voting)\n- gate-tempest-dsvm-keystone-uwsgi-full-nv http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-keystone-uwsgi-full-nv/c93f1db/ : SUCCESS in 1h 03m 23s (non-voting)\n- gate-tempest-dsvm-neutron-identity-v3-only-full-nv http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-nv/6c11b94/ : SUCCESS in 54m 50s (non-voting)\n- keystone-coverage-db http://logs.openstack.org/52/326152/1/check/keystone-coverage-db/265941f/ : SUCCESS in 14m 49s","accounts_in_message":[],"_revision_number":1},{"id":"5c1102a844262867382dc4cf68a4e1512c85a288","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2016-06-06 23:00:01.000000000","message":"Patch Set 1:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1298006 https://bugs.launchpad.net/bugs/1455123 https://bugs.launchpad.net/bugs/1282876\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":1},{"id":"eb44a2106ec1034deff182fa84b0362d39318c75","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-06-07 00:35:12.000000000","message":"Patch Set 1:\n\nis there a modification we can do so this doesn\u0027t install under /usr/etc when running with pbr?","accounts_in_message":[],"_revision_number":1},{"id":"faec79935d14c70e6dc14e69fc42099d8195faf6","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2016-06-07 10:47:16.000000000","message":"Patch Set 1: Code-Review-1\n\nSteve: no unfortunately, no tool support that for now. I did wrote a patch for pbr to handle that https://review.openstack.org/#/c/274077/ but it was denied because that should be fixed upper in the stack – which I agree though it\u0027s not very pragmatic.\n\nInstalling etc files is still important as it makes sure Keystone works seamlessly in virtualenv, and enables its usage by tools such as pifpaf (https://github.com/jd/pifpaf).\n\nPackagers can adapt :)","accounts_in_message":[],"_revision_number":1},{"id":"1188a0a243a9d592834dceae9d7ab8b4a0031f87","author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"date":"2016-06-07 14:18:55.000000000","message":"Patch Set 1: Code-Review-1\n\nI agree with Julien that installing the etc files is still important.\n\nFor example, for packages published in PyPi and installed within a virtualenv, the files are copied into the virtualenv\u0027s basedir, then needs to be moved from there. Similarly, I\u0027d expect packagers to adapt, as pointed out by Julien","accounts_in_message":[],"_revision_number":1},{"id":"1720091affa4f644a6170287c49979b74cdad13f","author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"date":"2016-06-07 17:10:26.000000000","message":"Patch Set 1:\n\nThe problem is that this works *only* for the venv, and breaks absolutely all downstream distros which have to undo what\u0027s been done in Julien\u0027s patch. This means 5 places to fix, instead of just devstack.\n\nUntil we find a solution for fixing PBR, as per https://review.openstack.org/#/c/274077/ then the kind of patch Julien proposed for Keystone shouldn\u0027t be attempted.","accounts_in_message":[],"_revision_number":1},{"id":"b72f2dae6017b674be9c5f08f0b106a4b2d65b45","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2016-06-07 17:47:00.000000000","message":"Patch Set 1:\n\nTelemetry projects are already doing that so…","accounts_in_message":[],"_revision_number":1},{"id":"0fb120f40bc4cf772498f6e01fc11c16ecaf8d25","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-06-07 18:22:03.000000000","message":"Patch Set 1:\n\nIt seems like networking [1] and telemetry projects (except ceilometer?) [2] and [3] are defining etc files with data_files ... \n\nI don\u0027t want to start a bike shed argument here between devs and packagers, but is there direction from the TC for what to do? This affects all projects and is being done differently between 2 (now 3 -- keystone) already. \n\n[1] http://git.openstack.org/cgit/openstack/neutron/tree/setup.cfg#n24\n[2] http://git.openstack.org/cgit/openstack/gnocchi/tree/setup.cfg#n87\n[3] http://git.openstack.org/cgit/openstack/aodh/tree/setup.cfg#n30","accounts_in_message":[],"_revision_number":1},{"id":"4a7e0a32a338bb7a72c2f2bfeab24f4c32314d91","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2016-06-08 07:34:41.000000000","message":"Patch Set 1:\n\nGood catch for Ceilo, patch is up https://review.openstack.org/326883\n\n\nAs already stated, the bug if there\u0027s any is in setuptools, so that\u0027s where and what should be fixed.","accounts_in_message":[],"_revision_number":1},{"id":"28801ba4291b8c1f5a43e00e741e8a699c292552","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-06-10 23:22:50.000000000","message":"Patch Set 1: Code-Review+2\n\ni think we need to apply the revert, it\u0027s significant of a change that packagers and maintainers are in a tight spot.\n\ndespite my +2 for the revert, i still want to see this approach work. i think https://review.openstack.org/#/c/274077/6 is worth looking at, but i appreciate direction from the TC on this topic.","accounts_in_message":[],"_revision_number":1},{"id":"fb751ec521b3864fed63d9902f63527e85563df0","author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"date":"2016-06-11 01:52:20.000000000","message":"Patch Set 1:\n\nI would like to see this stay in, but it is also fair that the packagers have a lot of overhead for this type of thing. I\u0027ve spun up http://lists.openstack.org/pipermail/openstack-dev/2016-June/097123.html this thread to help come to a resolution and a definition of data-files (at least in the short term) so that the expectation can be set for what packagers need to do.","accounts_in_message":[],"_revision_number":1},{"id":"45341bd2f9cbb8bf8b80a1b0218367e746c1dd19","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2016-06-11 09:00:02.000000000","message":"Patch Set 1:\n\nDistro can handle that, they already do for Gnocchi and Aodh. It\u0027s exactly the same strategy to apply.\n\nReverting will disallow tools to use directly Keystone from a venv (such as pifpaf), it\u0027s a shame :(\n\nIt\u0027s be better to push the patch into pbr, as Thomas suggested – that\u0027ll help the packagers.","accounts_in_message":[],"_revision_number":1},{"id":"6045b7b4030033d5c9da72e10c415de21cd19c2d","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-06-29 00:16:59.000000000","message":"Patch Set 1: Workflow+1\n\ni have to unfortunately revert this change before newton-2. sorry. there\u0027s clearly a need for a standard approach for all openstack project, but this isn\u0027t something we can do at the moment.","accounts_in_message":[],"_revision_number":1},{"id":"f5603521be6e9ec21b8bc6666386414f8c805404","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-06-29 00:17:06.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"51007c169788784d977c18bb51e1259e123e9a2a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-29 01:23:38.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-docs http://docs-draft.openstack.org/52/326152/1/check/gate-keystone-docs/e2edb71//doc/build/html/ : SUCCESS in 5m 01s\n- gate-keystone-pep8 http://logs.openstack.org/52/326152/1/check/gate-keystone-pep8/3c133f3/ : SUCCESS in 4m 14s\n- gate-keystone-python27-db http://logs.openstack.org/52/326152/1/check/gate-keystone-python27-db/23f0fc3/ : SUCCESS in 14m 37s\n- gate-keystone-python34-db http://logs.openstack.org/52/326152/1/check/gate-keystone-python34-db/976ea65/ : SUCCESS in 13m 04s\n- gate-tempest-dsvm-full http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-full/78fe13a/ : SUCCESS in 43m 07s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-postgres-full/a2ca4f4/ : SUCCESS in 50m 12s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-neutron-full/f8bcc87/ : SUCCESS in 1h 03m 39s\n- gate-grenade-dsvm http://logs.openstack.org/52/326152/1/check/gate-grenade-dsvm/34dc2b8/ : SUCCESS in 42m 49s\n- gate-keystone-tox-db-legacy_drivers http://logs.openstack.org/52/326152/1/check/gate-keystone-tox-db-legacy_drivers/ac3f4a1/ : SUCCESS in 7m 12s\n- gate-keystone-dsvm-functional http://logs.openstack.org/52/326152/1/check/gate-keystone-dsvm-functional/4f2df88/ : SUCCESS in 23m 34s\n- gate-keystone-dsvm-functional-v3-only-nv http://logs.openstack.org/52/326152/1/check/gate-keystone-dsvm-functional-v3-only-nv/e471896/ : SUCCESS in 23m 17s (non-voting)\n- gate-tempest-dsvm-keystone-uwsgi-full-nv http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-keystone-uwsgi-full-nv/dacab29/ : FAILURE in 1h 00m 42s (non-voting)\n- gate-tempest-dsvm-neutron-identity-v3-only-full-nv http://logs.openstack.org/52/326152/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-nv/c66b667/ : SUCCESS in 50m 40s (non-voting)\n- keystone-coverage-db http://logs.openstack.org/52/326152/1/check/keystone-coverage-db/139d1aa/ : SUCCESS in 13m 06s","accounts_in_message":[],"_revision_number":1},{"id":"16a464430d1755dfa4508ab2b291f1ae5fa446d6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-29 01:23:48.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"5a3a2f0d15ff5f1994f02eb81bd090f59014c957","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-29 02:19:49.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-keystone-docs http://docs-draft.openstack.org/52/326152/1/gate/gate-keystone-docs/44d3c67//doc/build/html/ : SUCCESS in 3m 08s\n- gate-keystone-pep8 http://logs.openstack.org/52/326152/1/gate/gate-keystone-pep8/c143c99/ : SUCCESS in 3m 01s\n- gate-keystone-python27-db http://logs.openstack.org/52/326152/1/gate/gate-keystone-python27-db/d1b507f/ : SUCCESS in 11m 33s\n- gate-keystone-python34-db http://logs.openstack.org/52/326152/1/gate/gate-keystone-python34-db/8ce08fc/ : SUCCESS in 12m 16s\n- gate-tempest-dsvm-full http://logs.openstack.org/52/326152/1/gate/gate-tempest-dsvm-full/9bd5001/ : SUCCESS in 41m 11s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/52/326152/1/gate/gate-tempest-dsvm-postgres-full/a01ea74/ : SUCCESS in 33m 01s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/52/326152/1/gate/gate-tempest-dsvm-neutron-full/ef2cf66/ : SUCCESS in 55m 26s\n- gate-grenade-dsvm http://logs.openstack.org/52/326152/1/gate/gate-grenade-dsvm/17a9d79/ : SUCCESS in 25m 56s\n- gate-keystone-dsvm-functional http://logs.openstack.org/52/326152/1/gate/gate-keystone-dsvm-functional/66a0138/ : SUCCESS in 13m 07s\n- gate-keystone-tox-db-legacy_drivers http://logs.openstack.org/52/326152/1/gate/gate-keystone-tox-db-legacy_drivers/655b476/ : SUCCESS in 4m 33s","accounts_in_message":[],"_revision_number":1},{"id":"c9a4bd0dbc6b66e1740d2239502bcf11c0e6eab7","date":"2016-06-29 02:19:50.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1},{"id":"1fdaab2226d9a13fcef1137b3b8207c2d39cfedb","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2016-06-29 08:42:33.000000000","message":"Patch Set 1:\n\nSo how are we supposed to start Keystone from a venv now? :(","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"4db20470d148283a29d6141176bb6657174d405b","revisions":{"4db20470d148283a29d6141176bb6657174d405b":{"kind":"REWORK","_number":1,"created":"2016-06-06 21:54:16.000000000","uploader":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"ref":"refs/changes/52/326152/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/52/326152/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/326152/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/326152/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/326152/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/52/326152/1"}}},"commit":{"parents":[{"commit":"b4e2ce74f8c2532ab903762598ab554d632f0679","subject":"Merge \"Adding role assignment lists unit tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/b4e2ce74f8c2532ab903762598ab554d632f0679"}]}],"author":{"name":"Thomas Goirand","email":"zigo@debian.org","date":"2016-06-06 21:22:27.000000000","tz":120},"committer":{"name":"Thomas Goirand","email":"zigo@debian.org","date":"2016-06-06 21:54:13.000000000","tz":120},"subject":"Revert \"Install necessary files in etc/\"","message":"Revert \"Install necessary files in etc/\"\n\nThis reverts commit 5486f0ae4a14a35138687fe7b20b66d33c380112.\nThe files installed this way are installed in /usr/etc when\nrunning PBR in the context of distros.\n\nChange-Id: I192864675a8b34082e34f60caac770da76ca02aa\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/4db20470d148283a29d6141176bb6657174d405b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/4db20470d148283a29d6141176bb6657174d405b"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b4e2ce74f8c2532ab903762598ab554d632f0679","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}