)]}'
{"id":"openstack%2Fdevstack~11017","triplet_id":"openstack%2Fdevstack~master~Ia5357114f8e4248a8de4bd0327e08323c487e897","project":"openstack/devstack","branch":"master","topic":"tempest-build-params","hashtags":[],"change_id":"Ia5357114f8e4248a8de4bd0327e08323c487e897","subject":"Splits out build config params in Tempest.","status":"MERGED","created":"2012-08-08 09:45:57.000000000","updated":"2012-08-30 17:00:46.000000000","submitted":"2012-08-30 17:00:46.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"8a106341f016870f8b7726c67418e8af08a5889a","_number":11017,"virtual_id_number":11017,"owner":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"date":"2012-08-08 15:20:50.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":2,"date":"2012-08-30 17:00:46.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":3085,"name":"Tushar Patil","inactive":true}],"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"all":[{"value":2,"date":"2012-08-08 15:20:50.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2012-08-15 01:10:24.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":1,"date":"2012-08-30 15:33:29.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"date":"2012-08-08 15:17:10.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},{"value":2,"date":"2012-08-30 16:27:18.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"date":"2012-08-14 23:31:22.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":3085,"name":"Tushar Patil","inactive":true}],"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":"2012-08-08 15:20:50.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-08-14 20:42:03.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"date":"2012-08-08 14:04:19.000000000","_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},{"value":1,"date":"2012-08-30 16:27:18.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":3085,"name":"Tushar Patil","inactive":true}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"value":0,"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":3085,"name":"Tushar Patil","inactive":true}],"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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":3085,"name":"Tushar Patil","inactive":true},{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-08-08 15:20:50.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"},{"updated":"2012-08-14 20:42:03.000000000","updated_by":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"reviewer":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"state":"REVIEWER"},{"updated":"2012-08-14 23:31:22.000000000","updated_by":{"_account_id":3085,"name":"Tushar Patil","inactive":true},"reviewer":{"_account_id":3085,"name":"Tushar Patil","inactive":true},"state":"REVIEWER"},{"updated":"2012-08-30 15:33:29.000000000","updated_by":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"reviewer":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"state":"REVIEWER"},{"updated":"2012-08-30 16:27:18.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":"2012-08-30 17:00:46.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"dab536186118e8895693c25a7246128e1c74029b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-08 09:55:35.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed\n\n- https://jenkins.openstack.org/job/gate-devstack-merge/912/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/7008/consoleFull : FAILURE\n","accounts_in_message":[],"_revision_number":1},{"id":"bcbb270b20ed68cefdb9d059a2445e024c280db4","author":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"date":"2012-08-08 14:04:19.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"0d81a35f8da09a0519d8663dfdac7ef4842020e9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-08 14:14:19.000000000","message":"Patch Set 2: Works for me\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-devstack-merge/914/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/7024/consoleFull : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"21e640fa611f31bc89094b5269c5ca788edefae3","author":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"date":"2012-08-08 15:17:10.000000000","message":"Patch Set 2:\n\nI\u0027ll push another patch to remove the old parameter names.","accounts_in_message":[],"_revision_number":2},{"id":"b8c0423d74fece72c0b3a518d38e91503f65ec27","author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"date":"2012-08-08 15:20:50.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"e83710a3b25f75a16ba6fb7036434a1c456df4ac","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-08-10 18:07:19.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nThis won\u0027t work until changes in Tempest that modify tempest.cnf.tpl for renaming BUILD_XXX to COMPUTE_BUILD_XXX is done. Rohit, which Tempest patch does that?","accounts_in_message":[],"_revision_number":2},{"id":"1a59b55daf77d9b8b8673867cfa2c7aa8997ce23","author":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"date":"2012-08-13 14:04:10.000000000","message":"Patch Set 2:\n\nHi Jay, \nhttps://review.openstack.org/#/c/10597 is the one.\n\nThanks\n-Rohit","accounts_in_message":[],"_revision_number":2},{"id":"b660b6ef842fe19b8909e7a2c7897fdac036db42","author":{"_account_id":3085,"name":"Tushar Patil","inactive":true},"date":"2012-08-14 20:17:38.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nhttps://review.openstack.org/#/c/10597 is dependent on this review. Unless this review is not approved and merged, https://review.openstack.org/#/c/10597 jenkins job will always fail. Request all core reviewers to please approve this review if you don\u0027t foresee any problems with these changes. \n\nThanks.","accounts_in_message":[],"_revision_number":2},{"id":"be828659121366b93276546e61f959c31e47c5b8","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-08-14 20:42:03.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\nlooks good.","accounts_in_message":[],"_revision_number":2},{"id":"6704c536ad7a4951ee77cd37daf867006dde8e67","author":{"_account_id":3085,"name":"Tushar Patil","inactive":true},"date":"2012-08-14 23:31:22.000000000","message":"Patch Set 2: No score\n\nLooks like this review has got 2 approvals from core reviewers but it\u0027s not merged as it is not approved.","accounts_in_message":[],"_revision_number":2},{"id":"6c1b0d97f270b5ba416eed09cb99dc298786ca5a","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-08-15 01:10:24.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nwe\u0027re waiting on jay to remove his -1. He says there is other stuff that needs to merge first. Downgrading to +1 to reflect that.","accounts_in_message":[],"_revision_number":2},{"id":"838b66ebba17dc6fb529991aac3e2bcf3194a598","author":{"_account_id":3085,"name":"Tushar Patil","inactive":true},"date":"2012-08-16 01:04:28.000000000","message":"Patch Set 2:\n\nHi Jay,\n\nIn this patch, Rohit has added new variables so these changes are independent. IMO, this patch should be merged before https://review.openstack.org/#/c/10597.\n\nRohit: Can you please confirm?","accounts_in_message":[],"_revision_number":2},{"id":"eab6ed5f8de360754d9f9e138eaf874df9bf7c42","author":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"date":"2012-08-16 15:32:28.000000000","message":"Patch Set 2:\n\nYes, this patch is not dependent on the tempest patch. It should be merged independently.\nThe tempest patch requires these build parameters.","accounts_in_message":[],"_revision_number":2},{"id":"6caacb0c69a62590765d128706c5fa18513ad22a","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-08-24 06:03:03.000000000","message":"Patch Set 2:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":2},{"id":"8b563b23ed713edddce3c07241fdfbe4caa600dc","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-08-24 06:03:03.000000000","message":"Patch Set 2: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":2},{"id":"fdaf83a245668b223ce9ca996063cb90978e620f","author":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"date":"2012-08-24 13:32:08.000000000","message":"Patch Set 2: Restored\n\nRestoring since this patch needs the -1 to be removed based on my last comment.\nKindly remove the -1 as there are multiple tempest patches dependent on it.","accounts_in_message":[],"_revision_number":2},{"id":"97a4b617d98c439330f20fa7436efe606d88b74f","author":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"date":"2012-08-24 13:32:33.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"27edfb2f2f701fdd84deada6584934f4ca3682a5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-24 13:47:07.000000000","message":"Patch Set 2:\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-devstack-merge/1013/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/9137/consoleFull : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"c4765e04fc9f1102e185ccfca9e7778b0c740c21","author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"date":"2012-08-30 15:33:29.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nlgtm. thx rohit.","accounts_in_message":[],"_revision_number":2},{"id":"ae3d9565238a2c00e60c7f29c09f445f4f0c44b6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-30 16:27:18.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"d98d97604b4fbdff652080a649661389c34521e4","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2012-08-30 16:27:18.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"2cb103b9a46a771219d4bf0a1a1f1fcd12621513","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-30 17:00:46.000000000","message":"Patch Set 2: Verified\n\nBuild successful\n\n- http://logs.openstack.org/11017/2/gate/gate-devstack-merge/1034 : SUCCESS\n- http://logs.openstack.org/11017/2/gate/gate-tempest-devstack-vm/9823 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"af49973bf8c485c029a8cb78b054028c69a759b5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-30 17:00:46.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"2eb9a13cce822251fd8e3115bf71ee62b777c265","revisions":{"b2a77992973b2e7adbe4999072ef6dd0084cef11":{"kind":"REWORK","_number":1,"created":"2012-08-08 09:45:57.000000000","uploader":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"ref":"refs/changes/17/11017/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/17/11017/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/11017/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/11017/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/11017/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/17/11017/1"}}},"commit":{"parents":[{"commit":"0416f332fdbb55a2dbeb68810fa165bdb1e0f4a4","subject":"Merge \"enable testing of openvz guests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0416f332fdbb55a2dbeb68810fa165bdb1e0f4a4"}]}],"author":{"name":"Rohit Karajgi","email":"rohit.karajgi@nttdata.com","date":"2012-08-08 09:34:51.000000000","tz":-420},"committer":{"name":"Rohit Karajgi","email":"rohit.karajgi@nttdata.com","date":"2012-08-08 09:44:46.000000000","tz":-420},"subject":"Rename build config params in Tempest.","message":"Rename build config params in Tempest.\n\nModifies the following:\nBUILD_INTERVAL --\u003e COMPUTE_BUILD_INTERVAL\nBUILD_TIMEOUT --\u003e COMPUTE_BUILD_TIMEOUT\n\nAdds the following:\nVOLUME_BUILD_INTERVAL\nVOLUME_BUILD_TIMEOUT\n\nChange-Id: Ia5357114f8e4248a8de4bd0327e08323c487e897\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b2a77992973b2e7adbe4999072ef6dd0084cef11"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b2a77992973b2e7adbe4999072ef6dd0084cef11"}]},"branch":"refs/heads/master"},"2eb9a13cce822251fd8e3115bf71ee62b777c265":{"kind":"REWORK","_number":2,"created":"2012-08-08 14:04:19.000000000","uploader":{"_account_id":4120,"name":"Rohit Karajgi","email":"rohit.karajgi@ril.com","username":"rohitkarajgi"},"ref":"refs/changes/17/11017/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/17/11017/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/11017/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/11017/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/17/11017/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/17/11017/2"}}},"commit":{"parents":[{"commit":"0416f332fdbb55a2dbeb68810fa165bdb1e0f4a4","subject":"Merge \"enable testing of openvz guests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0416f332fdbb55a2dbeb68810fa165bdb1e0f4a4"}]}],"author":{"name":"Rohit Karajgi","email":"rohit.karajgi@nttdata.com","date":"2012-08-08 09:34:51.000000000","tz":-420},"committer":{"name":"Rohit Karajgi","email":"rohit.karajgi@nttdata.com","date":"2012-08-08 14:01:00.000000000","tz":-420},"subject":"Splits out build config params in Tempest.","message":"Splits out build config params in Tempest.\n\nThe build parameters for instances and volumes should be configured separately.\n\nThis patch adds the following Tempest vars:\nCOMPUTE_BUILD_INTERVAL\nCOMPUTE_BUILD_TIMEOUT\nVOLUME_BUILD_INTERVAL\nVOLUME_BUILD_TIMEOUT\n\nChange-Id: Ia5357114f8e4248a8de4bd0327e08323c487e897\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2eb9a13cce822251fd8e3115bf71ee62b777c265"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2eb9a13cce822251fd8e3115bf71ee62b777c265"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}