)]}'
{"id":"openstack%2Fdevstack~202331","triplet_id":"openstack%2Fdevstack~master~I343248f7d5e8bda19c93cb19092b5459b28244b0","project":"openstack/devstack","branch":"master","topic":"bug/1403492","hashtags":[],"change_id":"I343248f7d5e8bda19c93cb19092b5459b28244b0","subject":"Ensure images downloaded from a previous stack are fully downloaded","status":"ABANDONED","created":"2015-07-15 23:22:15.000000000","updated":"2015-09-09 18:24:28.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9d344dd904731f4de78c8ee0eb0a45d9716be5c4","_number":202331,"virtual_id_number":202331,"owner":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2015-07-21 03:59:35.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":1,"date":"2015-07-21 04:03:04.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"date":"2015-07-23 09:00:48.000000000","_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"date":"2015-07-28 05:57:08.000000000","_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"date":"2015-07-21 03:04:41.000000000","_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":1,"date":"2015-07-28 05:28:05.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"value":-1,"date":"2015-08-04 18:26:39.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"}],"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":{"all":[{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2015-07-22 23:26:27.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"date":"2015-07-22 23:27:14.000000000","_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"}],"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":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-07-21 03:59:35.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-07-21 04:03:04.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-07-22 23:26:27.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2015-07-22 23:27:14.000000000","updated_by":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"reviewer":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"state":"REVIEWER"},{"updated":"2015-07-23 09:00:48.000000000","updated_by":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"reviewer":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"state":"REVIEWER"},{"updated":"2015-07-28 05:57:08.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"},{"updated":"2015-08-04 18:26:39.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"}],"messages":[{"id":"2cdc0ec967665f6ce76e6c9e91ca631ec07f2023","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-07-15 23:22:15.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c41f96808c492e98cd29b8e5f2f1a4264b78a5a4","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-07-16 00:18:21.000000000","message":"Patch Set 1:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/31/202331/1/4404/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":1},{"id":"52ac67b894dd92436b35f92d304992ea76b73758","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-16 00:50:59.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-full/fb77341/ : SUCCESS in 51m 26s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-postgres-full/6b13e17/ : SUCCESS in 48m 43s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-neutron/f3866ca/ : SUCCESS in 42m 24s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-neutron-full/aecd491/ : SUCCESS in 57m 49s\n- gate-grenade-dsvm http://logs.openstack.org/31/202331/1/check/gate-grenade-dsvm/846080d/ : SUCCESS in 58m 37s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-large-ops/b5f717f/ : SUCCESS in 39m 36s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-neutron-large-ops/1881aa2/ : SUCCESS in 33m 30s\n- gate-devstack-bashate http://logs.openstack.org/31/202331/1/check/gate-devstack-bashate/180fdc2/ : SUCCESS in 17s\n- gate-devstack-unit-tests http://logs.openstack.org/31/202331/1/check/gate-devstack-unit-tests/dd230fd/ : SUCCESS in 16s\n- gate-devstack-dsvm-cells http://logs.openstack.org/31/202331/1/check/gate-devstack-dsvm-cells/657724f/ : SUCCESS in 27m 49s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/31/202331/1/check/gate-grenade-dsvm-partial-ncpu/000d5c5/ : SUCCESS in 48m 47s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-ironic-pxe_ssh/53b3a2c/ : SUCCESS in 41m 29s\n- gate-devstack-dsvm-updown http://logs.openstack.org/31/202331/1/check/gate-devstack-dsvm-updown/854b598/ : SUCCESS in 26m 50s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-f21/e1ad6d9/ : FAILURE in 53m 30s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-centos7/4f4bdd2/ : SUCCESS in 34m 35s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/31/202331/1/check/gate-devstack-publish-docs/d70e843//doc/build/html/ : SUCCESS in 1m 27s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/31/202331/1/check/gate-swift-dsvm-functional-nv/e5ef01a/ : SUCCESS in 33m 06s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/31/202331/1/check/gate-grenade-dsvm-neutron/9d07136/ : SUCCESS in 51m 41s\n- gate-tempest-dsvm-multinode-smoke http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-multinode-smoke/712758c/ : SUCCESS in 42m 33s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-smoke http://logs.openstack.org/31/202331/1/check/gate-tempest-dsvm-neutron-multinode-smoke/24fc364/ : FAILURE in 7m 08s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"c398259692091b863fa321923578107887c1f9dc","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-07-16 06:05:59.000000000","message":"Patch Set 1: Code-Review-1\n\nI think wget can handle this for us, and even better by  with --timestamping\n\n A file is considered new if one of these two conditions are met:\n\n  1. A file of that name does not already exist locally.\n\n  2. A file of that name does exist, but the remote file was modified\n     more recently than the local file.\n\n  If the local file does not exist, or the sizes of the files do not match, Wget will download the remote file no matter what the time-stamps say.\n\n?","accounts_in_message":[],"_revision_number":1},{"id":"8620f0099af6d9d640350490a1694add2d9ca986","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-07-16 16:52:21.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b047aed72794b1fcf5c3a748c1c41b76729d5ac3","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-07-16 16:54:27.000000000","message":"Patch Set 2:\n\nWow, I did not know about that flag! Very useful and has simplified this patch a bit.","accounts_in_message":[],"_revision_number":2},{"id":"60e0926e2150398da057936d58552ea5dfbebf7e","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-07-16 18:52:52.000000000","message":"Patch Set 2:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/31/202331/2/4519/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":2},{"id":"9197f7d1d979be5ed81f76a0ca4dbffab434d932","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-16 21:40:58.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-full/cfcfbc8/ : SUCCESS in 54m 05s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-postgres-full/26148ad/ : SUCCESS in 50m 12s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-neutron/223bcc2/ : SUCCESS in 40m 09s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-neutron-full/d0e1afe/ : SUCCESS in 59m 55s\n- gate-grenade-dsvm http://logs.openstack.org/31/202331/2/check/gate-grenade-dsvm/bdbf3d0/ : SUCCESS in 47m 03s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-large-ops/209fd8b/ : SUCCESS in 34m 38s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-neutron-large-ops/5909412/ : SUCCESS in 31m 31s\n- gate-devstack-bashate http://logs.openstack.org/31/202331/2/check/gate-devstack-bashate/7ed8d42/ : SUCCESS in 26s\n- gate-devstack-unit-tests http://logs.openstack.org/31/202331/2/check/gate-devstack-unit-tests/634063d/ : SUCCESS in 19s\n- gate-devstack-dsvm-cells http://logs.openstack.org/31/202331/2/check/gate-devstack-dsvm-cells/49987c3/ : SUCCESS in 29m 04s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/31/202331/2/check/gate-grenade-dsvm-partial-ncpu/011a7a9/ : SUCCESS in 48m 11s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-ironic-pxe_ssh/60d5fa9/ : SUCCESS in 44m 58s\n- gate-devstack-dsvm-updown http://logs.openstack.org/31/202331/2/check/gate-devstack-dsvm-updown/1b45343/ : SUCCESS in 27m 17s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-f21/2490577/ : FAILURE in 55m 20s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-centos7/667b2e1/ : SUCCESS in 47m 51s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/31/202331/2/check/gate-devstack-publish-docs/c563b24//doc/build/html/ : SUCCESS in 2m 33s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/31/202331/2/check/gate-swift-dsvm-functional-nv/afdf74d/ : SUCCESS in 32m 42s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/31/202331/2/check/gate-grenade-dsvm-neutron/67e6f54/ : SUCCESS in 59m 18s\n- gate-tempest-dsvm-multinode-smoke http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-multinode-smoke/ef089cc/ : FAILURE in 1m 23s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-smoke http://logs.openstack.org/31/202331/2/check/gate-tempest-dsvm-neutron-multinode-smoke/804a783/ : SUCCESS in 37m 05s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"220d2151d416fa61d3823272cc64f057eb44f8d9","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-07-16 23:25:20.000000000","message":"Patch Set 2: Code-Review+2\n\nLGTM, my understanding of the wget option is that this is very sane, only downloading if newer or size changed","accounts_in_message":[],"_revision_number":2},{"id":"32579c3aaff9396d9e5d68e1f73f55cc36eceed8","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2015-07-17 09:16:18.000000000","message":"Patch Set 2: Code-Review-1\n\nOn my local machine I get:\n\n  root@node-1:/tmp# wget --version\n  GNU Wget 1.13.4 built on linux-gnu.\n\n  root@node-1:/tmp# wget --timestamping http://cloud.centos.org/centos/7/images/CentOS-7-x86_64-GenericCloud.qcow2.xz -O CentOS-7-x86_64-GenericCloud.qcow2.xz\n  WARNING: timestamping does nothing in combination with -O. See the manual\n  for details.","accounts_in_message":[],"_revision_number":2},{"id":"81ff1e1670f58fd2b008537bed013206cb1b051b","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-07-21 01:46:27.000000000","message":"Patch Set 2: Code-Review-1\n\nThanks Jordan, the things you find when you read the man page huh.\n\nAlso, the [ ! -f ] check probably shouldn\u0027t be there, as we want to try downloading always and let wget decide if it\u0027s newer or not.\n\ni think this can be solved by just directly downloading with wget without the -O ... we don\u0027t transform the filename in any way, we just do \"basename\" on it","accounts_in_message":[],"_revision_number":2},{"id":"3e309f35679140368b21ab097351690822d88fc5","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-07-21 03:01:01.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"75ce28dd09df1b92d0de4d5f1d5feb86d8c60456","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-07-21 03:04:41.000000000","message":"Patch Set 3:\n\nGah, I feel stupid now. Fixed now. Here is the output I get when the image files stay the same:\n\n+ wget --progress\u003ddot:giga --timestamping -c http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-uec.tar.gz -P /home/slave/devstack/files\n...\nConnecting to download.cirros-cloud.net (download.cirros-cloud.net)|69.163.241.114|:80... connected.\nHTTP request sent, awaiting response... 416 Requested Range Not Satisfiable\n\n    The file is already fully retrieved; nothing to do.","accounts_in_message":[],"_revision_number":3},{"id":"624203cc3ccaaa29c41a9162bfa5822ce738d40d","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-07-21 03:59:35.000000000","message":"Patch Set 3:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/31/202331/3/5024/index.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":3},{"id":"c289fbce262eb33e770c6c0b44d533f7c4f316f4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-21 04:03:04.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-full http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-full/8281e14/ : SUCCESS in 54m 23s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-postgres-full/8182d73/ : SUCCESS in 50m 57s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-neutron-full/13008f4/ : SUCCESS in 1h 00m 23s\n- gate-grenade-dsvm http://logs.openstack.org/31/202331/3/check/gate-grenade-dsvm/55ce6ba/ : SUCCESS in 45m 10s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-large-ops/8209329/ : SUCCESS in 33m 25s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-neutron-large-ops/f8d9832/ : SUCCESS in 30m 46s\n- gate-devstack-bashate http://logs.openstack.org/31/202331/3/check/gate-devstack-bashate/282eb4d/ : SUCCESS in 18s\n- gate-devstack-unit-tests http://logs.openstack.org/31/202331/3/check/gate-devstack-unit-tests/b3753e4/ : SUCCESS in 18s\n- gate-devstack-dsvm-cells http://logs.openstack.org/31/202331/3/check/gate-devstack-dsvm-cells/efcb36b/ : SUCCESS in 27m 44s\n- gate-grenade-dsvm-partial-ncpu http://logs.openstack.org/31/202331/3/check/gate-grenade-dsvm-partial-ncpu/62da624/ : SUCCESS in 46m 50s\n- gate-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-ironic-pxe_ssh/312bfdb/ : SUCCESS in 37m 27s\n- gate-devstack-dsvm-updown http://logs.openstack.org/31/202331/3/check/gate-devstack-dsvm-updown/d6000f4/ : SUCCESS in 21m 53s\n- gate-tempest-dsvm-f21 http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-f21/dab6de9/ : FAILURE in 49m 09s (non-voting)\n- gate-tempest-dsvm-centos7 http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-centos7/0f22f5c/ : SUCCESS in 30m 54s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/31/202331/3/check/gate-devstack-publish-docs/521c033//doc/build/html/ : SUCCESS in 2m 14s\n- gate-swift-dsvm-functional-nv http://logs.openstack.org/31/202331/3/check/gate-swift-dsvm-functional-nv/e775e04/ : SUCCESS in 27m 51s (non-voting)\n- gate-grenade-dsvm-neutron http://logs.openstack.org/31/202331/3/check/gate-grenade-dsvm-neutron/a99e809/ : SUCCESS in 47m 46s\n- gate-tempest-dsvm-multinode-smoke http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-multinode-smoke/6360a4c/ : SUCCESS in 33m 38s (non-voting)\n- gate-tempest-dsvm-neutron-multinode-smoke http://logs.openstack.org/31/202331/3/check/gate-tempest-dsvm-neutron-multinode-smoke/0e8cfc8/ : SUCCESS in 40m 19s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"e36e42d3372089057a58ad95b3be084da9326330","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2015-07-23 09:00:48.000000000","message":"Patch Set 3:\n\nI don\u0027t think this is really safe.\n\nNot every server reply with a Last-Modified header. \n\nI also made this test: I downloaded a test image, truncated (using truncate -s) the file to a bigger size then used wget --timestamping and the image was not downloaded again although the image size was not the same...","accounts_in_message":[],"_revision_number":3},{"id":"89baa2245731a7174c979d0a53a966a6f5e3cb4d","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-07-24 18:56:45.000000000","message":"Patch Set 3:\n\nHi Jordan, I have tested on a server with the Last-Modified disabled and wget does take that into account and will re-download the file in those cases. This is what the log message shows:\n\n  \"Last-modified header missing -- time-stamps turned off.\"\n\nI also did a test with a few images using the same test that you tried with truncate -s (both trimming and adding bits) and got successful results:\n\n  Length: 2713 (2.6K) [application/json]\n  The sizes do not match (local 12) -- retrieving.\n\n  Length: 2713 (2.6K) [application/json]\n  The sizes do not match (local 5000) -- retrieving.","accounts_in_message":[],"_revision_number":3},{"id":"6210278e3aea72e4e5bf86d33ee61816ca29ca3d","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2015-07-27 14:25:05.000000000","message":"Patch Set 3:\n\nOk Ryan I must have made a mistake somewhere (which is very possible). Still I am a bit afraid that this could lead to a bug or a weird situation that would be hard to debug. But I am a bit \"young\" in that area so I\u0027ll let other reviewers review :)","accounts_in_message":[],"_revision_number":3},{"id":"c17f731da39b532976c615764e12c2ee1f698c8c","author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"date":"2015-07-28 05:28:05.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"568de6ec489835666a53c77b810cbc67a050d780","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-07-28 05:57:08.000000000","message":"Patch Set 3:\n\n@Jordan - i think you\u0027ve found a wget bug.  there\u0027s actually two I found\n\n1) later versions of wget use if-modified-since headers.  if this is in use, wget doesn\u0027t seem to check the size of the file-on-disk at all.  It only checks the timestamp.\n\n2) the way you truncated by *increasing* the size also triggers another bug when \"if-modified-since\" headers are not in use and you have continuation \"-c\" turned on.  When \"-c\" is enabled, wget sends \"range\" headers from the file-on-disk.  Because you have the entire file, you request from the end of the file (e.g., if the file is 100 bytes, when you use \"-c\" you send a range of 100).  The server responds with 416 \"Invalid range\" and wget assumes that means you have the whole file.  However, if you actually have more than the file (e.g. 200 bytes from the prior example) this is now an invalid assumption; actually we want wget to re-download.\n\nI have sent an email about this to the wget lists (it has not passed moderation/hit the archive yet)\n\nIn short, I think it\u0027s right that this isn\u0027t safe, unfortunately, especially with modern wget that supports if-modified-since headers","accounts_in_message":[],"_revision_number":3},{"id":"99d174ea9d2d3dcc4b27c228f22ec286429ac52c","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-07-28 05:57:40.000000000","message":"Patch Set 3:\n\np.s. maybe the answer is just use curl :)","accounts_in_message":[],"_revision_number":3},{"id":"5b85f1dd9f2b3973de90c38aed155c7d2faa3f41","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2015-07-28 08:34:09.000000000","message":"Patch Set 3:\n\nhttp://lists.gnu.org/archive/html/bug-wget/2015-07/msg00083.html","accounts_in_message":[],"_revision_number":3},{"id":"6634f7d5f228b487ba2780dbdd08ad2272a6dd2b","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2015-07-28 08:56:17.000000000","message":"Patch Set 3:\n\nThanks a lot Ian for the follow-up. I am watching this closely.","accounts_in_message":[],"_revision_number":3},{"id":"ee86e3c3b8c892f9452cc8c84d8d91f169540c58","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2015-08-04 18:26:39.000000000","message":"Patch Set 3: Code-Review-1\n\nthis is going to cause network traffic even if the files are cached, which is part of what we are trying to avoid.","accounts_in_message":[],"_revision_number":3},{"id":"852971675ef12d288edeada0ab168a09f17c47ba","author":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"date":"2015-09-09 18:24:28.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"b2b85af2ec4e5b8552a570aedfd1f0d358db7174","revisions":{"13f06e851b260c09664fe25bc2ea0cbd1929a3f1":{"kind":"REWORK","_number":1,"created":"2015-07-15 23:22:15.000000000","uploader":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"ref":"refs/changes/31/202331/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/202331/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/202331/1"}}},"commit":{"parents":[{"commit":"32a3e322b428d83fa5922c1b9e6adce047064bd6","subject":"Merge \"Use openstack cli for cinder type creation\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/32a3e322b428d83fa5922c1b9e6adce047064bd6"}]}],"author":{"name":"Ryan Hsu","email":"hryan@vmware.com","date":"2015-07-15 22:30:09.000000000","tz":-420},"committer":{"name":"Ryan Hsu","email":"hryan@vmware.com","date":"2015-07-15 23:21:08.000000000","tz":-420},"subject":"Ensure images downloaded from a previous stack are fully downloaded","message":"Ensure images downloaded from a previous stack are fully downloaded\n\nCurrently, stack.sh does not check if an image file downloaded from\na previous stack.sh execution is fully downloaded before using it.\nThis patch looks to see if the existing image at least matches the\nsize of the file on the server.\n\nChange-Id: I343248f7d5e8bda19c93cb19092b5459b28244b0\nCloses-Bug: #1403492\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/13f06e851b260c09664fe25bc2ea0cbd1929a3f1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/13f06e851b260c09664fe25bc2ea0cbd1929a3f1"}]},"branch":"refs/heads/master"},"5823dfb7e300b23cae4a95e45f38b06779bcbe58":{"kind":"REWORK","_number":2,"created":"2015-07-16 16:52:21.000000000","uploader":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"ref":"refs/changes/31/202331/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/202331/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/202331/2"}}},"commit":{"parents":[{"commit":"19533886ff5843624945a9d85b8bd02a71d75401","subject":"Merge \"Wait for command in worlddump\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/19533886ff5843624945a9d85b8bd02a71d75401"}]}],"author":{"name":"Ryan Hsu","email":"hryan@vmware.com","date":"2015-07-16 16:50:03.000000000","tz":-420},"committer":{"name":"Ryan Hsu","email":"hryan@vmware.com","date":"2015-07-16 16:52:21.000000000","tz":-420},"subject":"Ensure images downloaded from a previous stack are fully downloaded","message":"Ensure images downloaded from a previous stack are fully downloaded\n\nCurrently, stack.sh does not check if an image file downloaded from\na previous stack.sh execution is fully downloaded before using it.\nThis patch updates the wget command to use the --timestamping\noption which will re-download a file if the server modified date is\nmore recent than the local file as well as when the server file\nsize differs from the local file.\n\nChange-Id: I343248f7d5e8bda19c93cb19092b5459b28244b0\nCloses-Bug: #1403492\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5823dfb7e300b23cae4a95e45f38b06779bcbe58"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5823dfb7e300b23cae4a95e45f38b06779bcbe58"}]},"branch":"refs/heads/master"},"b2b85af2ec4e5b8552a570aedfd1f0d358db7174":{"kind":"REWORK","_number":3,"created":"2015-07-21 03:01:01.000000000","uploader":{"_account_id":8302,"name":"Ryan Hsu","email":"hryan@vmware.com","username":"rhsu"},"ref":"refs/changes/31/202331/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/31/202331/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/31/202331/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/31/202331/3"}}},"commit":{"parents":[{"commit":"19533886ff5843624945a9d85b8bd02a71d75401","subject":"Merge \"Wait for command in worlddump\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/19533886ff5843624945a9d85b8bd02a71d75401"}]}],"author":{"name":"Ryan Hsu","email":"hryan@vmware.com","date":"2015-07-16 16:50:03.000000000","tz":-420},"committer":{"name":"Ryan Hsu","email":"hryan@vmware.com","date":"2015-07-21 03:01:19.000000000","tz":-420},"subject":"Ensure images downloaded from a previous stack are fully downloaded","message":"Ensure images downloaded from a previous stack are fully downloaded\n\nCurrently, stack.sh does not check if an image file downloaded from\na previous stack.sh execution is fully downloaded before using it.\nThis patch updates the wget command to use the --timestamping\noption which will re-download a file if the server modified date is\nmore recent than the local file as well as when the server file\nsize differs from the local file.\n\nChange-Id: I343248f7d5e8bda19c93cb19092b5459b28244b0\nCloses-Bug: #1403492\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b2b85af2ec4e5b8552a570aedfd1f0d358db7174"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b2b85af2ec4e5b8552a570aedfd1f0d358db7174"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}