)]}'
{"id":"openstack%2Fdevstack~28673","triplet_id":"openstack%2Fdevstack~master~Ic9fc564c4ad0f43e2e536854335ebe14791d0255","project":"openstack/devstack","branch":"master","topic":"bp/xenapi-devstack-cleanup","hashtags":[],"change_id":"Ic9fc564c4ad0f43e2e536854335ebe14791d0255","subject":"xenapi: Ubuntu installation repository/proxy fix","status":"MERGED","created":"2013-05-09 14:13:06.000000000","updated":"2013-05-15 11:12:21.000000000","submitted":"2013-05-15 11:12:20.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2a9a2580af478c3b6446b251de4a48c1ce3b6d26","_number":28673,"virtual_id_number":28673,"owner":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-05-15 11:12:20.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"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"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":2,"date":"2013-05-14 14:26:08.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-05-14 11:05:02.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":2,"date":"2013-05-15 09:55:21.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":"","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":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":0,"date":"2013-05-14 14:26:08.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":1,"date":"2013-05-15 09:55:21.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":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"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":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"_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":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-05-14 11:05:02.000000000","updated_by":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"reviewer":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"state":"REVIEWER"},{"updated":"2013-05-14 14:26:08.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-05-15 09:55:21.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-05-15 11:12:20.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"05dc3666a607456ddbc81b48cc9c8eba11278abc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-09 14:59:11.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/28673/1/check/gate-tempest-devstack-vm-full/15214 : SUCCESS in 00:42:31\n- http://logs.openstack.org/28673/1/check/gate-tempest-devstack-vm-postgres-full/14929 : FAILURE in 00:45:50 (non-voting)\n- http://logs.openstack.org/28673/1/check/gate-tempest-devstack-vm-quantum/21477 : SUCCESS in 00:21:00\n- http://logs.openstack.org/28673/1/check/gate-grenade-devstack-vm/2953 : SUCCESS in 00:14:15 (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"bc3e02b3883d2d2c252ec4b69b033eac8cd1e667","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-05-09 15:42:00.000000000","message":"Patch Set 1:\n\nWould it be reasonable to think that if the user needs to set UBUNTU_INST_HTTP_PROXY that the standard HTTP_PROXY or http_proxy env var might also be set and use that as a default?","accounts_in_message":[],"_revision_number":1},{"id":"b001776ada9b7c3e7e62310d8c58e45fd3e71451","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-05-09 15:58:30.000000000","message":"Patch Set 1:\n\nIt might not be the same. For example, we are using an squid-deb-proxy to speed up the Ubuntu install, but that proxy is limited to some websites. It really depends on the user. Having these as separated parameters gives more flexibility.","accounts_in_message":[],"_revision_number":1},{"id":"e0d840941e0e359558d2e49e64a40efd69dbbdba","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-05-13 11:06:39.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"4cb87c374f1906d6fde565f0e7b157b17d1f004c","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-05-13 14:33:13.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\nMate: ok, I\u0027ll buy that","accounts_in_message":[],"_revision_number":1},{"id":"7231e1a3f850c3e4fa8ce081070cbd826b2e2426","author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"date":"2013-05-14 10:41:29.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b4eb40de879e9274292393f61ba66e907f368875","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2013-05-14 11:05:02.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"cbf457dc07c021a1f6ee3be9b5bdf965fe04ffc3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-14 11:26:37.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/28673/2/check/gate-tempest-devstack-vm-full/15866 : SUCCESS in 00:44:51\n- http://logs.openstack.org/28673/2/check/gate-tempest-devstack-vm-postgres-full/15582 : FAILURE in 00:44:09 (non-voting)\n- http://logs.openstack.org/28673/2/check/gate-tempest-devstack-vm-quantum/22306 : SUCCESS in 00:21:45\n- http://logs.openstack.org/28673/2/check/gate-grenade-devstack-vm/3515 : FAILURE in 00:10:35 (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"428b5bfdeb3f482315c96c5880a886988d60fbe8","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-05-14 14:26:08.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"06d3ffdf3ebe0a3b45fc292cf7864beedef6df22","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-05-15 09:55:21.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"dbf5f50e7930dc16ea02f541301d3085f8fd4fcb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-15 09:55:23.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"0de6a79c9436b22a67fc9ab63e33c3466907494a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-15 11:12:20.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/28673/2/gate/gate-tempest-devstack-vm-full/16043 : SUCCESS in 48m 09s\n- http://logs.openstack.org/28673/2/gate/gate-tempest-devstack-vm-postgres-full/15759 : FAILURE in 46m 18s (non-voting)\n- http://logs.openstack.org/28673/2/gate/gate-tempest-devstack-vm-quantum/22524 : SUCCESS in 21m 35s\n- http://logs.openstack.org/28673/2/gate/gate-grenade-devstack-vm/3676 : FAILURE in 10m 22s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"802675943fbda9ec44d088cf84598b0dddd98319","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-15 11:12:21.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"d3740f70b52763d547d2a1d861829e6f853775bc","revisions":{"efc3b4141c80fdd541578369426d28de2bc75ee7":{"kind":"REWORK","_number":1,"created":"2013-05-09 14:13:06.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/73/28673/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/73/28673/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/28673/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/28673/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/28673/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/73/28673/1"}}},"commit":{"parents":[{"commit":"b240ad472be39786c587acc75ef40de6e61a4b2d","subject":"Merge \"Ensure correct polkit config written for RHEL6\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b240ad472be39786c587acc75ef40de6e61a4b2d"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-05-09 14:02:21.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-05-09 14:08:57.000000000","tz":60},"subject":"xenapi: Ubuntu installation repository/proxy fix","message":"xenapi: Ubuntu installation repository/proxy fix\n\nGet rid of UBUNTU_INST_REPOSITORY, use UBUNTU_INST_HTTP_HOSTNAME and\nUBUNTU_INST_HTTP_DIRECTORY instead. User can also specify\nUBUNTU_INST_HTTP_PROXY to utilize a proxy for the OpenStack VM\ninstallation. The answer file will be edited to contain the specified\nvalues. Also get rid of the magic, undocumented MIRROR variable.\n\nThis is related to blueprint xenapi-devstack-cleanup\n\nChange-Id: Ic9fc564c4ad0f43e2e536854335ebe14791d0255\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/efc3b4141c80fdd541578369426d28de2bc75ee7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/efc3b4141c80fdd541578369426d28de2bc75ee7"}]},"branch":"refs/heads/master"},"d3740f70b52763d547d2a1d861829e6f853775bc":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2013-05-14 10:41:29.000000000","uploader":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"ref":"refs/changes/73/28673/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/73/28673/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/28673/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/28673/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/73/28673/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/73/28673/2"}}},"commit":{"parents":[{"commit":"ad573f8543d5c0b98177ffc141423ff937962173","subject":"Merge \"Only search the lib dir for rpc_backend\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/ad573f8543d5c0b98177ffc141423ff937962173"}]}],"author":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-05-09 14:02:21.000000000","tz":60},"committer":{"name":"Mate Lakat","email":"mate.lakat@citrix.com","date":"2013-05-14 10:37:07.000000000","tz":60},"subject":"xenapi: Ubuntu installation repository/proxy fix","message":"xenapi: Ubuntu installation repository/proxy fix\n\nGet rid of UBUNTU_INST_REPOSITORY, use UBUNTU_INST_HTTP_HOSTNAME and\nUBUNTU_INST_HTTP_DIRECTORY instead. User can also specify\nUBUNTU_INST_HTTP_PROXY to utilize a proxy for the OpenStack VM\ninstallation. The answer file will be edited to contain the specified\nvalues. Also get rid of the magic, undocumented MIRROR variable.\n\nThis is related to blueprint xenapi-devstack-cleanup\n\nChange-Id: Ic9fc564c4ad0f43e2e536854335ebe14791d0255\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d3740f70b52763d547d2a1d861829e6f853775bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d3740f70b52763d547d2a1d861829e6f853775bc"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}