)]}'
{"id":"openstack%2Fdevstack~25456","triplet_id":"openstack%2Fdevstack~master~Ida19130ce9499d4d1fd2360fadc6b870b24bbc95","project":"openstack/devstack","branch":"master","topic":"rhel64","hashtags":[],"change_id":"Ida19130ce9499d4d1fd2360fadc6b870b24bbc95","subject":"Add rhel to DISTRO match","status":"MERGED","created":"2013-03-26 23:15:35.000000000","updated":"2013-04-04 20:58:28.000000000","submitted":"2013-04-04 20:58:27.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":15,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"332390f8920ff9ac8179a90d96854225ed9c2072","_number":25456,"virtual_id_number":25456,"owner":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-04-04 20:58:27.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-04-03 22:00:56.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":2,"date":"2013-04-03 22:14:28.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":0,"date":"2013-04-03 22:01:16.000000000","permitted_voting_range":{"min":0,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2013-04-04 20:17:26.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":0,"date":"2013-04-03 22:14:28.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":1,"date":"2013-04-04 20:17:26.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"value":0,"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-04-02 22:40:33.000000000","updated_by":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"reviewer":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"state":"REVIEWER"},{"updated":"2013-04-03 22:00:56.000000000","updated_by":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"reviewer":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"state":"REVIEWER"},{"updated":"2013-04-03 22:14:28.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-04-04 20:17:26.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-04-04 20:58:27.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"b2610f92b93b544baad1fdf74bea42f03afa000f","author":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"date":"2013-03-27 01:16:55.000000000","message":"Patch Set 1: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"e3d2dc86a2d0a33ef8a197043adae76dfa65151b","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-03-27 03:11:01.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"9587ea8a5df3f622346278d88a6c3ce2d416489c","author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"date":"2013-03-27 03:42:20.000000000","message":"Patch Set 1: (4 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"f99186e92e7870ea5b2fa465c6350bbf2103a8b2","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-03-27 04:08:46.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"96b6f96dfdf5820569d3b517cf3edecdbb909d9f","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-03-27 11:42:08.000000000","message":"Patch Set 1: (3 inline comments)\n\ntabs need cleaning up\n\nalso, can we just get an is_rh which is the is_fedora || is_rhel6, it seems like most of this is just adding more || is_rhel6, and I\u0027d rather have that in one place.","accounts_in_message":[],"_revision_number":1},{"id":"0d500e87e92e142e88e6fbe6f8e18494cb646804","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-03-27 22:16:22.000000000","message":"Patch Set 2: Work In Progress\n\nReworking to make is_RHEL6 a sub-check of is_fedora","accounts_in_message":[],"_revision_number":2},{"id":"fa0b51f05577a3acc30ad9d56935e587251eaa8c","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-03-27 23:07:53.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"8548aaf46c17c40db6e994d59dcfec5c7fb44bc8","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-03-28 15:55:47.000000000","message":"Patch Set 1: (2 inline comments)\n\nIMHO the primary change needed here is to add the Red Hat/CentOS test to GetDistro().  If the name of is_fedora() needs to change to better reflect that it is a family test, then let\u0027s consider that.  Otherwise just use $os_VENDOR and $os_RELEASE to test for specific releases.","accounts_in_message":[],"_revision_number":1},{"id":"43eec6952c19b1a54d930b354155a8023bded8ad","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-01 22:52:34.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"3b3aec5d64ae8d52c092ff88537d9e994e71c3a1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-01 23:38:34.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/25456/3/check/gate-tempest-devstack-vm-full/11179 : SUCCESS As best as we can tell this change works.\n- http://logs.openstack.org/25456/3/check/gate-tempest-devstack-vm-postgres-full/10895 : SUCCESS Postgres users everywhere thank you.\n- http://logs.openstack.org/25456/3/check/gate-tempest-devstack-vm-quantum/16839 : SUCCESS The bits must flow.\n- http://logs.openstack.org/25456/3/check/gate-grenade-devstack-vm/155 : FAILURE ... maybe horse shoes is more up your alley (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"e7e9060fc94f41c927e63fc529bf5a508e175b23","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-04-02 00:01:50.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(2 inline comments)\n\nhard tabs","accounts_in_message":[],"_revision_number":3},{"id":"d1fe4a23b1638fe0599cb054d2b8d7ce2d74e1d7","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-04-02 00:06:36.000000000","message":"Patch Set 3:\n\nbasically the whole patch series needs a hard tabs scrub, once that is done and rebased, I\u0027ll take a look on content.","accounts_in_message":[],"_revision_number":3},{"id":"a5cf682e94ab34424e7c75eb064bf428b62337f6","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-02 03:14:30.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"7747abcb7e445ed3ceefa3f078f554eb8112bdc9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-02 03:54:41.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/25456/4/check/gate-tempest-devstack-vm-full/11199 : SUCCESS As best as we can tell this change works.\n- http://logs.openstack.org/25456/4/check/gate-tempest-devstack-vm-postgres-full/10915 : SUCCESS Postgres users everywhere thank you.\n- http://logs.openstack.org/25456/4/check/gate-tempest-devstack-vm-quantum/16866 : SUCCESS The bits must flow.\n- http://logs.openstack.org/25456/4/check/gate-grenade-devstack-vm/161 : FAILURE ... maybe horse shoes is more up your alley (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"15fe439b3646838ab67723b33606cf91a2e519d0","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-04-02 05:56:56.000000000","message":"Patch Set 4: Do not merge\n\nWe do not need an is_rhel6() function.  Test $os_DISTRO \u003d\u003d rhel6 instead.","accounts_in_message":[],"_revision_number":4},{"id":"0e76b81b9cd91143fc078a19ca4095629dcd4e58","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-02 21:42:50.000000000","message":"Patch Set 4:\n\n\u003e Test $os_DISTRO \u003d\u003d rhel6 instead.\n\nOK; one issue I see here is dealing with CentOS.  This would imply CentOS\u0027s os_DISTRO is set to rhel6, which seems a bit wrong, or we start having lots of checks like\n\nif is_fedora {\n  if [ $os_DISTRO \u003d \"rhel6\" || $os_DISTRO \u003d \"centos6\" ] {\n\nSeeing as this will be done multiple times and almost always the same way, I feel like the intrinsic documentation of \"is_rhel6\" would be clearer and shorter.","accounts_in_message":[],"_revision_number":4},{"id":"cebb005d22ec247098c3c99c2c56955a4b305693","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-04-02 22:12:40.000000000","message":"Patch Set 4:\n\n[err, I meant $DISTRO above rather than $os_DISTRO]\n\nI\u0027m unclear how \"is_rhel6()\" returning true for CentOS is different than $DISTRO containing \"rhel6\" for CentOS.  I said earlier that $DISTRO should report CentOS 6 as \u0027rhel6\u0027.  $os_VENDOR will always give you the actual vendor name if you need it.","accounts_in_message":[],"_revision_number":4},{"id":"ff9e7b43fa36d4b43f4baff84bb38f80e07e8ea9","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-02 22:34:36.000000000","message":"Patch Set 4:\n\nOk, let me make the change and see how it all looks","accounts_in_message":[],"_revision_number":4},{"id":"bdc44dbd6f4b4c559d422788857096a897314fb1","author":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"date":"2013-04-02 22:40:33.000000000","message":"Patch Set 4:\n\nIs it not worth moving to a \"driver\" model?\n\n so have a script per distro (distros/rhel6.sh in this case)\n and instead of having lots of \"if is_what_ever()\" we just\n call out to \"distro_do_this()\" and if a new distro comes\n along they just need to implement a list of functions.","accounts_in_message":[],"_revision_number":4},{"id":"1b5858f9723f301a85eeadbee2a2ca9a39a4eaf4","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-03 02:07:42.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"f1bdd413d33091c602be4b6ae0d778cba8550a97","author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"date":"2013-04-03 02:30:06.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"d3bcc3f1a41477c3a6ed5b525843de16d711690a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-03 02:50:43.000000000","message":"Patch Set 5: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/25456/5/check/gate-tempest-devstack-vm-full/11374 : SUCCESS\n- http://logs.openstack.org/25456/5/check/gate-tempest-devstack-vm-postgres-full/11089 : SUCCESS\n- http://logs.openstack.org/25456/5/check/gate-tempest-devstack-vm-quantum/17065 : SUCCESS\n- http://logs.openstack.org/25456/5/check/gate-grenade-devstack-vm/184 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":5},{"id":"271696986b254ffe2c524f012aa82a2d4ecb0656","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-04-03 14:25:12.000000000","message":"Patch Set 5:\n\nThe code looks good but the patch isn\u0027s atomic.  There is no reason for it to depend on any of the other patches in your set.  Please rebase it against master.","accounts_in_message":[],"_revision_number":5},{"id":"b9c03e706bc9f82c476049f07fcf485573b1091c","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-03 22:00:38.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"86492ff4a10b057a16a879ff888b4c1d11a33a80","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-04-03 22:00:56.000000000","message":"Patch Set 6:\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":6},{"id":"cdb032ebc95a2459659e998317717596d97a053e","author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"date":"2013-04-03 22:00:56.000000000","message":"Patch Set 6: Looks good to me, but someone else must approve\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":6},{"id":"9ff3b319aeb5639897678040428e15bfc35025b5","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2013-04-03 22:01:16.000000000","message":"Patch Set 6:\n\nRebased to master","accounts_in_message":[],"_revision_number":6},{"id":"a19005575c76062fc11d3fb0847280c5865f9afb","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-04-03 22:14:28.000000000","message":"Patch Set 6: Looks good to me (core reviewer)\n\nThanks Ian","accounts_in_message":[],"_revision_number":6},{"id":"bbc0b14e98028539b4a8361ae08aef58b33669ff","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-03 22:51:18.000000000","message":"Patch Set 6: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/25456/6/check/gate-tempest-devstack-vm-full/11512 : SUCCESS\n- http://logs.openstack.org/25456/6/check/gate-tempest-devstack-vm-postgres-full/11227 : SUCCESS\n- http://logs.openstack.org/25456/6/check/gate-tempest-devstack-vm-quantum/17218 : SUCCESS\n- http://logs.openstack.org/25456/6/check/gate-grenade-devstack-vm/195 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":6},{"id":"c6ba67e0f503e327bfb0983f0796ff122910e78c","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-04-04 20:17:26.000000000","message":"Patch Set 6: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":6},{"id":"a51a887f045f3cc97d6aae6af9c0aa0e829ef78b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-04 20:17:30.000000000","message":"Patch Set 6: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":6},{"id":"cefdb249c7c43b47446fb328df29efd64285be6c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-04 20:58:27.000000000","message":"Patch Set 6: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/25456/6/gate/gate-tempest-devstack-vm-full/11642 : SUCCESS\n- http://logs.openstack.org/25456/6/gate/gate-tempest-devstack-vm-postgres-full/11357 : SUCCESS\n- http://logs.openstack.org/25456/6/gate/gate-tempest-devstack-vm-quantum/17375 : SUCCESS\n","accounts_in_message":[],"_revision_number":6},{"id":"a0284f5d83eee0f67727a53ed82ca651a1fbd401","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-04 20:58:28.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"d857f4b4a4af2ed468c219d24da9d98416aff5d1","revisions":{"774b4e641d7605bff7ab2c7ad02012866591bab4":{"kind":"REWORK","_number":1,"created":"2013-03-26 23:15:35.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/56/25456/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/56/25456/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/56/25456/1"}}},"commit":{"parents":[{"commit":"9a3ba4b3e1a6286cd1ccaafdb2c1d9960c91da9c","subject":"Fix typo in baremetal header comment.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9a3ba4b3e1a6286cd1ccaafdb2c1d9960c91da9c"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-20 03:51:06.000000000","tz":660},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-26 22:10:41.000000000","tz":660},"subject":"Add is_RHEL6","message":"Add is_RHEL6\n\nAdd is_RHEL6 function to test for RHEL6 as the first step for getting\nsupport.\n\nChange some existing checks for Fedora such that they activate on\nRHEL6 as well.\n\nChange-Id: Ida19130ce9499d4d1fd2360fadc6b870b24bbc95\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/774b4e641d7605bff7ab2c7ad02012866591bab4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/774b4e641d7605bff7ab2c7ad02012866591bab4"}]},"branch":"refs/heads/master"},"9c251178f5e8920b9f82dab6246ae582ca3e7cd6":{"kind":"REWORK","_number":2,"created":"2013-03-27 04:08:46.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/56/25456/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/56/25456/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/56/25456/2"}}},"commit":{"parents":[{"commit":"9a3ba4b3e1a6286cd1ccaafdb2c1d9960c91da9c","subject":"Fix typo in baremetal header comment.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9a3ba4b3e1a6286cd1ccaafdb2c1d9960c91da9c"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-20 03:51:06.000000000","tz":660},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-27 03:43:40.000000000","tz":660},"subject":"Add is_RHEL6","message":"Add is_RHEL6\n\nAdd is_RHEL6 function to test for RHEL6 as the first step for getting\nsupport.\n\nChange some existing checks for Fedora such that they activate on\nRHEL6 as well.\n\nChange-Id: Ida19130ce9499d4d1fd2360fadc6b870b24bbc95\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9c251178f5e8920b9f82dab6246ae582ca3e7cd6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9c251178f5e8920b9f82dab6246ae582ca3e7cd6"}]},"branch":"refs/heads/master"},"c2c805db734c16da398d3869d9bb07a78176a510":{"kind":"REWORK","_number":3,"created":"2013-04-01 22:52:34.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/56/25456/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/56/25456/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/56/25456/3"}}},"commit":{"parents":[{"commit":"e4f0cd7eed3981086b4a0db967501fffa6a07c1e","subject":"refactor the install/configure split","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e4f0cd7eed3981086b4a0db967501fffa6a07c1e"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-20 03:51:06.000000000","tz":660},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-04-01 22:51:20.000000000","tz":660},"subject":"Add is_rhel6","message":"Add is_rhel6\n\nAdd is_rhel6 function to test for RHEL6 as the first step for getting\nsupport.  RHEL6 is considered to be a special version of Fedora, so\nis_fedora still matches on RHEL6.  This avoids a lot of code churn,\nbut does mean you have to be slightly careful with ordering of checks\nto get RHEL6 specific behaviour.\n\nChange-Id: Ida19130ce9499d4d1fd2360fadc6b870b24bbc95\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c2c805db734c16da398d3869d9bb07a78176a510"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/c2c805db734c16da398d3869d9bb07a78176a510"}]},"branch":"refs/heads/master"},"0476965bfc8487cdecfda1fcfb505d01aa50b6b9":{"kind":"REWORK","_number":4,"created":"2013-04-02 03:14:30.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/56/25456/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/56/25456/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/56/25456/4"}}},"commit":{"parents":[{"commit":"ebf8f594224825438fbbdb4f8679549903e5b668","subject":"Add post-prereq phase","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ebf8f594224825438fbbdb4f8679549903e5b668"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-20 03:51:06.000000000","tz":660},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-04-02 02:47:06.000000000","tz":660},"subject":"Add is_rhel6","message":"Add is_rhel6\n\nAdd is_rhel6 function to test for RHEL6 as the first step for getting\nsupport.  RHEL6 is considered to be a special version of Fedora, so\nis_fedora still matches on RHEL6.  This avoids a lot of code churn,\nbut does mean you have to be slightly careful with ordering of checks\nto get RHEL6 specific behaviour.\n\nChange-Id: Ida19130ce9499d4d1fd2360fadc6b870b24bbc95\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0476965bfc8487cdecfda1fcfb505d01aa50b6b9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0476965bfc8487cdecfda1fcfb505d01aa50b6b9"}]},"branch":"refs/heads/master"},"a3f358a57738d0c57a92a51621f28f250b09d644":{"kind":"REWORK","_number":5,"created":"2013-04-03 02:07:42.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/56/25456/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/56/25456/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/56/25456/5"}}},"commit":{"parents":[{"commit":"5b43d7209af7d39c5fbe27f417a4501a22310a93","subject":"Add post-prereq phase","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b43d7209af7d39c5fbe27f417a4501a22310a93"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-20 03:51:06.000000000","tz":660},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-04-03 00:31:28.000000000","tz":660},"subject":"Add rhel to DISTRO match","message":"Add rhel to DISTRO match\n\nMatch RHEL and CentOS and provide them as a DISTRO prefix of \"rhel\"\n\nChange-Id: Ida19130ce9499d4d1fd2360fadc6b870b24bbc95\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3f358a57738d0c57a92a51621f28f250b09d644"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a3f358a57738d0c57a92a51621f28f250b09d644"}]},"branch":"refs/heads/master"},"d857f4b4a4af2ed468c219d24da9d98416aff5d1":{"kind":"TRIVIAL_REBASE","_number":6,"created":"2013-04-03 22:00:38.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/56/25456/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/56/25456/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/56/25456/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/56/25456/6"}}},"commit":{"parents":[{"commit":"4b1dbb5dabb6a30097030c0a625830a17c5c065f","subject":"Keystone doesn\u0027t depend on bcrypt any more","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b1dbb5dabb6a30097030c0a625830a17c5c065f"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-03-20 03:51:06.000000000","tz":660},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2013-04-03 21:59:51.000000000","tz":660},"subject":"Add rhel to DISTRO match","message":"Add rhel to DISTRO match\n\nMatch RHEL and CentOS and provide them as a DISTRO prefix of \"rhel\"\n\nChange-Id: Ida19130ce9499d4d1fd2360fadc6b870b24bbc95\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d857f4b4a4af2ed468c219d24da9d98416aff5d1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d857f4b4a4af2ed468c219d24da9d98416aff5d1"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}