)]}'
{"id":"openstack%2Fdevstack~70754","triplet_id":"openstack%2Fdevstack~master~I6376d7afd59cc5ebba9ed69e5ee784a3d5934a10","project":"openstack/devstack","branch":"master","topic":"bash8_3","hashtags":[],"change_id":"I6376d7afd59cc5ebba9ed69e5ee784a3d5934a10","subject":"introduce if/then \u0026 for/do rules","status":"MERGED","created":"2014-02-03 15:49:02.000000000","updated":"2014-02-05 17:59:34.000000000","submitted":"2014-02-05 17:59:33.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"93a57355dcd1455653cd634493843b08e7ca3b25","_number":70754,"virtual_id_number":70754,"owner":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":2,"date":"2014-02-05 17:59:33.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"all":[{"value":2,"date":"2014-02-05 06:53:16.000000000","_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2014-02-05 06:53:16.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"date":"2014-02-05 08:30:36.000000000","_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2014-02-05 14:51:50.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-02-05 08:30:36.000000000","updated_by":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"reviewer":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"state":"REVIEWER"},{"updated":"2014-02-05 14:51:50.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":"2014-02-05 17:59:33.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"28c03ade2cbd2586c530b31cfb0f5d0dc8748594","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-03 16:28:15.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"6ad8d3580086b81d53f65f54a957b8bcf4208369","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-03 17:16:09.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/54/70754/1/check/gate-devstack-bash8/8fd2d3f : FAILURE in 6s\n- check-devstack-dsvm-cells http://logs.openstack.org/54/70754/1/check/check-devstack-dsvm-cells/2cbb2f7 : SUCCESS in 11m 53s\n- check-tempest-dsvm-full http://logs.openstack.org/54/70754/1/check/check-tempest-dsvm-full/e2177a7 : SUCCESS in 1h 24m 57s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/54/70754/1/check/check-tempest-dsvm-postgres-full/8092a22 : SUCCESS in 44m 55s\n- check-tempest-dsvm-neutron http://logs.openstack.org/54/70754/1/check/check-tempest-dsvm-neutron/f190da0 : SUCCESS in 52m 14s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/54/70754/1/check/gate-tempest-dsvm-large-ops/30635e9 : SUCCESS in 15m 59s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/54/70754/1/check/gate-tempest-dsvm-neutron-large-ops/bd7322f : SUCCESS in 11m 59s\n- check-grenade-dsvm http://logs.openstack.org/54/70754/1/check/check-grenade-dsvm/fa69553 : SUCCESS in 27m 00s\n- check-swift-dsvm-functional http://logs.openstack.org/54/70754/1/check/check-swift-dsvm-functional/5e5a36b : SUCCESS in 11m 34s\n- check-devstack-dsvm-neutron http://logs.openstack.org/54/70754/1/check/check-devstack-dsvm-neutron/82a1616 : FAILURE in 15m 32s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/54/70754/1/check/check-tempest-dsvm-ironic-nv/79dd258 : FAILURE in 21m 48s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/54/70754/1/check/check-tempest-dsvm-ironic-postgres-nv/fabbb18 : FAILURE in 24m 13s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"69e979992a1f4ca5add82f9e06e7e372674e9034","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-02-03 17:17:55.000000000","message":"Patch Set 1:\n\nI noticed tempest failed, I think you hit bug(s):\n\n- https://bugs.launchpad.net/bugs/1275267\n\nWe don\u0027t automatically recheck or reverify, so please consider\ndoing that manually if someone hasn\u0027t already. For a code review\nwhich is not yet approved, you can recheck by leaving a code\nreview comment with just the text:\n\n    recheck bug 1275267\n\nFor a code review which has been approved but failed to merge,\nyou can reverify by leaving a comment like this:\n\n    reverify bug 1275267","accounts_in_message":[],"_revision_number":1},{"id":"52a17c755312abff67919297cb8345c06b8bf4c8","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-02-03 20:35:04.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"fc132295128e230468ba0d97934d055d34da092d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-03 21:27:37.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/54/70754/2/check/gate-devstack-bash8/7036485 : SUCCESS in 7s\n- check-devstack-dsvm-cells http://logs.openstack.org/54/70754/2/check/check-devstack-dsvm-cells/b047142 : SUCCESS in 10m 54s\n- check-tempest-dsvm-full http://logs.openstack.org/54/70754/2/check/check-tempest-dsvm-full/6b4dad1 : SUCCESS in 45m 30s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/54/70754/2/check/check-tempest-dsvm-postgres-full/4df33a4 : SUCCESS in 52m 17s\n- check-tempest-dsvm-neutron http://logs.openstack.org/54/70754/2/check/check-tempest-dsvm-neutron/24c8127 : SUCCESS in 28m 40s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/54/70754/2/check/gate-tempest-dsvm-large-ops/6c45e64 : SUCCESS in 11m 29s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/54/70754/2/check/gate-tempest-dsvm-neutron-large-ops/4dc314f : SUCCESS in 14m 54s\n- check-grenade-dsvm http://logs.openstack.org/54/70754/2/check/check-grenade-dsvm/b9ba63a : SUCCESS in 29m 22s\n- check-swift-dsvm-functional http://logs.openstack.org/54/70754/2/check/check-swift-dsvm-functional/928e55e : SUCCESS in 11m 48s\n- check-devstack-dsvm-neutron http://logs.openstack.org/54/70754/2/check/check-devstack-dsvm-neutron/d8240d6 : FAILURE in 14m 04s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/54/70754/2/check/check-tempest-dsvm-ironic-nv/5f27f15 : FAILURE in 38m 18s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/54/70754/2/check/check-tempest-dsvm-ironic-postgres-nv/c4a9a2d : FAILURE in 33m 29s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"01c3999b5b0e16c99628fd42b1743ca810ad3117","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-04 14:06:23.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nI like that, we may want to add while and until on that regexp in subsequent patch (from a quick look the only offender is 3 places in the xen/* scripts)","accounts_in_message":[],"_revision_number":2},{"id":"6259c15b6b200939e884d2d43dadd52328802253","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-04 14:24:34.000000000","message":"Patch Set 2:\n\nSince that was quick and I wanted to see how that bash8.py look like I have uploaded https://review.openstack.org/#/c/70991/  we should add a rule to pep8 bash8.py or wait when it would be able to graduate to its own repo (and have tests) \n.","accounts_in_message":[],"_revision_number":2},{"id":"cef34410745e90117291cea2eba04d875d908d96","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-04 19:43:49.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"fd244e77fdfa51b1cb350287e61fa531367d35cc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-04 19:43:57.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"590dde22b282eb61f618da2562cc23313926c59a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-04 21:21:17.000000000","message":"Patch Set 2: Fails\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/54/70754/2/gate/gate-devstack-bash8/ddb6c73 : SUCCESS in 6s\n- gate-devstack-dsvm-cells http://logs.openstack.org/54/70754/2/gate/gate-devstack-dsvm-cells/8ac1157 : SUCCESS in 9m 45s\n- gate-tempest-dsvm-full http://logs.openstack.org/54/70754/2/gate/gate-tempest-dsvm-full/3d4dd8b : SUCCESS in 1h 37m 12s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/54/70754/2/gate/gate-tempest-dsvm-postgres-full/6d0747d : FAILURE in 55m 51s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/54/70754/2/gate/gate-tempest-dsvm-neutron/2dbe01b : SUCCESS in 35m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/54/70754/2/gate/gate-tempest-dsvm-large-ops/5adcfc6 : SUCCESS in 11m 02s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/54/70754/2/gate/gate-tempest-dsvm-neutron-large-ops/21947cb : SUCCESS in 13m 14s\n- gate-grenade-dsvm http://logs.openstack.org/54/70754/2/gate/gate-grenade-dsvm/849a56b : SUCCESS in 28m 56s\n- gate-swift-dsvm-functional http://logs.openstack.org/54/70754/2/gate/gate-swift-dsvm-functional/d2a3cfc : SUCCESS in 20m 45s\n","accounts_in_message":[],"_revision_number":2},{"id":"c74dd7f057b88cfca6ceac9bb0d4643c7320d92e","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-05 06:53:10.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"c251db7ba8f44b6a40aeb8fe89c88845fc3ffc8f","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-05 06:53:16.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":3},{"id":"4024bdfc791ca38d8fb33ce4ed887ab5c377b7e3","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-05 06:53:16.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nAutomatically re-added by Gerrit trivial rebase detection script.","accounts_in_message":[],"_revision_number":3},{"id":"aab85e09b1a2c148d07bce6705c84bc40e78d6d0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-05 07:54:23.000000000","message":"Patch Set 3: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-devstack-bash8 http://logs.openstack.org/54/70754/3/check/gate-devstack-bash8/639332a : SUCCESS in 9s\n- check-devstack-dsvm-cells http://logs.openstack.org/54/70754/3/check/check-devstack-dsvm-cells/ad39c2e : SUCCESS in 9m 29s\n- check-tempest-dsvm-full http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-full/04426df : SUCCESS in 48m 00s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-postgres-full/49ebc29 : FAILURE in 1h 00m 26s\n- check-tempest-dsvm-neutron http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-neutron/f8bfe1e : SUCCESS in 46m 42s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/54/70754/3/check/gate-tempest-dsvm-large-ops/378eeae : SUCCESS in 13m 20s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/54/70754/3/check/gate-tempest-dsvm-neutron-large-ops/165a8ad : SUCCESS in 11m 51s\n- check-grenade-dsvm http://logs.openstack.org/54/70754/3/check/check-grenade-dsvm/b98c313 : SUCCESS in 29m 18s\n- check-swift-dsvm-functional http://logs.openstack.org/54/70754/3/check/check-swift-dsvm-functional/e7be4a2 : SUCCESS in 18m 40s\n- check-devstack-dsvm-neutron http://logs.openstack.org/54/70754/3/check/check-devstack-dsvm-neutron/b0f64e8 : FAILURE in 16m 37s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-ironic-nv/16e3b9e : SUCCESS in 26m 07s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-ironic-postgres-nv/619927f : SUCCESS in 28m 37s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"bc08edc1ab6c2dd336d7c06add027dc8851e61dc","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-05 08:30:36.000000000","message":"Patch Set 3:\n\nrecheck bug 1275256","accounts_in_message":[],"_revision_number":3},{"id":"9ba03037b41f8348d7415934989eab0b799e15ac","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-05 09:26:45.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/54/70754/3/check/gate-devstack-bash8/3f2d40c : SUCCESS in 11s\n- check-devstack-dsvm-cells http://logs.openstack.org/54/70754/3/check/check-devstack-dsvm-cells/b4bdd82 : SUCCESS in 18m 02s\n- check-tempest-dsvm-full http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-full/324bf8a : SUCCESS in 50m 26s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-postgres-full/b4c9407 : SUCCESS in 55m 57s\n- check-tempest-dsvm-neutron http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-neutron/ce7b9e2 : SUCCESS in 35m 08s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/54/70754/3/check/gate-tempest-dsvm-large-ops/d272c77 : SUCCESS in 11m 43s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/54/70754/3/check/gate-tempest-dsvm-neutron-large-ops/a7b837e : SUCCESS in 11m 38s\n- check-grenade-dsvm http://logs.openstack.org/54/70754/3/check/check-grenade-dsvm/3836ef7 : SUCCESS in 43m 21s\n- check-swift-dsvm-functional http://logs.openstack.org/54/70754/3/check/check-swift-dsvm-functional/4c3f5f3 : SUCCESS in 12m 22s\n- check-devstack-dsvm-neutron http://logs.openstack.org/54/70754/3/check/check-devstack-dsvm-neutron/275a93f : FAILURE in 13m 09s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-ironic-nv/1d14db5 : SUCCESS in 25m 48s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/54/70754/3/check/check-tempest-dsvm-ironic-postgres-nv/585f710 : SUCCESS in 40m 32s (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"7fe658ad4e15d37c992bd79867966104c7172805","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-05 14:51:50.000000000","message":"Patch Set 3: Approved\n\n","accounts_in_message":[],"_revision_number":3},{"id":"80000cd83f88dd732fadcae6beec99f196af85ad","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-05 14:51:51.000000000","message":"Patch Set 3: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"e716785be85b1b45f370b6a774025a62f89c602b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-05 17:59:33.000000000","message":"Patch Set 3: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/54/70754/3/gate/gate-devstack-bash8/1df09c7 : SUCCESS in 12s\n- gate-devstack-dsvm-cells http://logs.openstack.org/54/70754/3/gate/gate-devstack-dsvm-cells/29a1030 : SUCCESS in 10m 29s\n- gate-tempest-dsvm-full http://logs.openstack.org/54/70754/3/gate/gate-tempest-dsvm-full/753565c : SUCCESS in 53m 44s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/54/70754/3/gate/gate-tempest-dsvm-postgres-full/3f73146 : SUCCESS in 49m 04s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/54/70754/3/gate/gate-tempest-dsvm-neutron/3394596 : SUCCESS in 32m 44s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/54/70754/3/gate/gate-tempest-dsvm-large-ops/890a1a9 : SUCCESS in 10m 59s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/54/70754/3/gate/gate-tempest-dsvm-neutron-large-ops/fb5aa9b : SUCCESS in 11m 48s\n- gate-grenade-dsvm http://logs.openstack.org/54/70754/3/gate/gate-grenade-dsvm/36322a0 : SUCCESS in 36m 26s\n- gate-swift-dsvm-functional http://logs.openstack.org/54/70754/3/gate/gate-swift-dsvm-functional/8b312c9 : SUCCESS in 11m 42s\n","accounts_in_message":[],"_revision_number":3},{"id":"28200f7dec6ff9a55a2e0b71968d0b1d698b21f0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-05 17:59:34.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"16dd8b3ed94d5cd217d22a26c18dca52bfca115e","revisions":{"0ba0988972c4073b9d3361c0db0e1763afd4e21a":{"kind":"REWORK","_number":1,"created":"2014-02-03 15:49:02.000000000","uploader":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"ref":"refs/changes/54/70754/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/54/70754/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/54/70754/1"}}},"commit":{"parents":[{"commit":"3e417ddf425ab7028ee1ed030a2c11a3c09cb37a","subject":"add ability to ignore rules in bash8","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3e417ddf425ab7028ee1ed030a2c11a3c09cb37a"}]}],"author":{"name":"Sean Dague","email":"sean.dague@samsung.com","date":"2014-02-03 00:10:54.000000000","tz":540},"committer":{"name":"Sean Dague","email":"sean.dague@samsung.com","date":"2014-02-03 15:45:30.000000000","tz":540},"subject":"introduce if/then \u0026 for/do rules","message":"introduce if/then \u0026 for/do rules\n\nwe mostly have a consistent style on if/then \u0026 for/do in devstack,\nexcept when we don\u0027t. This attempts to build a set of rules to\nenforce this.\n\nBecause there are times when lines are legitimately long, and there\nis a continuation, this starts off ignoring if and for loops with\ncontinuations. But for short versions, we should enforce this.\n\nChanges to make devstack pass are included. The fact that the\ncleanup patch was so small is pretty solid reason that this is\nactually the style we\u0027ve all agreed to.\n\nPart of a git stash from hong kong that I finally cleaned up.\n\nChange-Id: I6376d7afd59cc5ebba9ed69e5ee784a3d5934a10\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0ba0988972c4073b9d3361c0db0e1763afd4e21a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0ba0988972c4073b9d3361c0db0e1763afd4e21a"}]},"branch":"refs/heads/master"},"1bca203a91b3e06c1f5e39214b082a2bf1c065b8":{"kind":"REWORK","_number":2,"created":"2014-02-03 20:35:04.000000000","uploader":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"ref":"refs/changes/54/70754/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/54/70754/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/54/70754/2"}}},"commit":{"parents":[{"commit":"0656e12d6819f6dee671dd6200b2d0895e716c2c","subject":"add ability to ignore rules in bash8","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0656e12d6819f6dee671dd6200b2d0895e716c2c"}]}],"author":{"name":"Sean Dague","email":"sean.dague@samsung.com","date":"2014-02-03 00:10:54.000000000","tz":540},"committer":{"name":"Sean Dague","email":"sean.dague@samsung.com","date":"2014-02-03 20:32:38.000000000","tz":540},"subject":"introduce if/then \u0026 for/do rules","message":"introduce if/then \u0026 for/do rules\n\nwe mostly have a consistent style on if/then \u0026 for/do in devstack,\nexcept when we don\u0027t. This attempts to build a set of rules to\nenforce this.\n\nBecause there are times when lines are legitimately long, and there\nis a continuation, this starts off ignoring if and for loops with\ncontinuations. But for short versions, we should enforce this.\n\nChanges to make devstack pass are included. The fact that the\ncleanup patch was so small is pretty solid reason that this is\nactually the style we\u0027ve all agreed to.\n\nPart of a git stash from hong kong that I finally cleaned up.\n\nChange-Id: I6376d7afd59cc5ebba9ed69e5ee784a3d5934a10\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1bca203a91b3e06c1f5e39214b082a2bf1c065b8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1bca203a91b3e06c1f5e39214b082a2bf1c065b8"}]},"branch":"refs/heads/master"},"16dd8b3ed94d5cd217d22a26c18dca52bfca115e":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2014-02-05 06:53:10.000000000","uploader":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"ref":"refs/changes/54/70754/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/54/70754/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/54/70754/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/54/70754/3"}}},"commit":{"parents":[{"commit":"e38cad092ff96dc0178a19fe6f33f13db99eebb0","subject":"Merge \"Silence commands to echo copyright notices\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e38cad092ff96dc0178a19fe6f33f13db99eebb0"}]}],"author":{"name":"Sean Dague","email":"sean.dague@samsung.com","date":"2014-02-03 00:10:54.000000000","tz":540},"committer":{"name":"Chmouel Boudjnah","email":"chmouel@enovance.com","date":"2014-02-05 06:52:54.000000000","tz":60},"subject":"introduce if/then \u0026 for/do rules","message":"introduce if/then \u0026 for/do rules\n\nwe mostly have a consistent style on if/then \u0026 for/do in devstack,\nexcept when we don\u0027t. This attempts to build a set of rules to\nenforce this.\n\nBecause there are times when lines are legitimately long, and there\nis a continuation, this starts off ignoring if and for loops with\ncontinuations. But for short versions, we should enforce this.\n\nChanges to make devstack pass are included. The fact that the\ncleanup patch was so small is pretty solid reason that this is\nactually the style we\u0027ve all agreed to.\n\nPart of a git stash from hong kong that I finally cleaned up.\n\nChange-Id: I6376d7afd59cc5ebba9ed69e5ee784a3d5934a10\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/16dd8b3ed94d5cd217d22a26c18dca52bfca115e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/16dd8b3ed94d5cd217d22a26c18dca52bfca115e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}