)]}'
{"id":"openstack%2Fironic-python-agent~428410","triplet_id":"openstack%2Fironic-python-agent~master~I587f9ec75832847b8ad6f87c733e2e9ed7510571","project":"openstack/ironic-python-agent","branch":"master","topic":"bug/1661328","hashtags":[],"change_id":"I587f9ec75832847b8ad6f87c733e2e9ed7510571","subject":"Select right image url before download","status":"ABANDONED","created":"2017-02-02 20:33:57.000000000","updated":"2017-02-08 16:12:57.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ec6fc01f84e32bc23c2848bde20038fe09cc9d15","_number":428410,"virtual_id_number":428410,"owner":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2017-02-02 22:49:29.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"}],"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":{"all":[{"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":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"}],"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":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-02-02 20:46:57.000000000","updated_by":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"reviewer":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"state":"REVIEWER"},{"updated":"2017-02-02 22:49:29.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"a7b88c966e7f499d0d9af870bb67ade2b8c4f5e5","author":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"date":"2017-02-02 20:33:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f55518b92b7ed4c53ed3ed7da999aeb6bd4b0f5e","author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"date":"2017-02-02 20:46:57.000000000","message":"Patch Set 1: Code-Review-1\n\nThe patch itself seems fine, but I\u0027m not sure the purpose, and it isn\u0027t clearly resolving the bug indicated.","accounts_in_message":[],"_revision_number":1},{"id":"d66b8d25b6cc185c6610fa77621df0df11860089","author":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"date":"2017-02-02 20:51:18.000000000","message":"Patch Set 1: Workflow-1\n\nThere are further commits that will address the bug. This will make more sense withing the whole","accounts_in_message":[],"_revision_number":1},{"id":"0c1a715498c957d796c0ed65670bca01a35b217e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-02 21:38:10.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ironic-python-agent-docs-ubuntu-xenial http://docs-draft.openstack.org/10/428410/1/check/gate-ironic-python-agent-docs-ubuntu-xenial/9dac700//doc/build/html/ : SUCCESS in 2m 19s\n- gate-ironic-python-agent-pep8-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-ironic-python-agent-pep8-ubuntu-xenial/42b98d6/ : FAILURE in 2m 54s\n- gate-ironic-python-agent-python27-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-ironic-python-agent-python27-ubuntu-xenial/53f971f/ : SUCCESS in 2m 04s\n- gate-ironic-python-agent-python35 http://logs.openstack.org/10/428410/1/check/gate-ironic-python-agent-python35/cdabc34/ : SUCCESS in 3m 40s\n- gate-ironic-python-agent-tox-func-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-ironic-python-agent-tox-func-ubuntu-xenial/3c5e1ac/ : SUCCESS in 2m 05s (non-voting)\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial/6592dec/ : SUCCESS in 33m 37s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial/552ab8e/ : SUCCESS in 35m 40s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/2c52dff/ : SUCCESS in 32m 40s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/ab05816/ : SUCCESS in 34m 20s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/a978fb7/ : SUCCESS in 33m 13s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/4890dbd/ : SUCCESS in 33m 05s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/32372bb/ : SUCCESS in 33m 53s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/029fc3c/ : SUCCESS in 52m 12s\n- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv http://logs.openstack.org/10/428410/1/check/gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv/406ce1c/ : SUCCESS in 1h 01m 48s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"46df48a12004f916438fc0b8306ea5bf4aa07809","author":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"date":"2017-02-02 21:46:39.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"01354a7a417d2f049d8154b5d718bfa93a2cb060","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-02 22:49:29.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-ironic-python-agent-docs-ubuntu-xenial http://docs-draft.openstack.org/10/428410/2/check/gate-ironic-python-agent-docs-ubuntu-xenial/7972ee1//doc/build/html/ : SUCCESS in 4m 09s\n- gate-ironic-python-agent-pep8-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-ironic-python-agent-pep8-ubuntu-xenial/9b2e087/ : SUCCESS in 3m 41s\n- gate-ironic-python-agent-python27-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-ironic-python-agent-python27-ubuntu-xenial/be88f15/ : SUCCESS in 4m 08s\n- gate-ironic-python-agent-python35 http://logs.openstack.org/10/428410/2/check/gate-ironic-python-agent-python35/94dcee5/ : SUCCESS in 1m 53s\n- gate-ironic-python-agent-tox-func-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-ironic-python-agent-tox-func-ubuntu-xenial/830f072/ : SUCCESS in 4m 36s (non-voting)\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-coreos-src-ubuntu-xenial/05208f2/ : SUCCESS in 33m 06s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-coreos-src-ubuntu-xenial/ab308d7/ : SUCCESS in 34m 34s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/a0eac48/ : FAILURE in 20m 42s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-agent_ipmitool-tinyipa-src-ubuntu-xenial/d4ffa12/ : SUCCESS in 47m 39s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/bf4fd2a/ : SUCCESS in 46m 48s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-tinyipa-src-ubuntu-xenial/ff72856/ : SUCCESS in 48m 57s\n- gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-partition-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/8ff79b7/ : SUCCESS in 48m 12s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_ipmitool-coreos-src-ubuntu-xenial/0244350/ : SUCCESS in 33m 19s\n- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv http://logs.openstack.org/10/428410/2/check/gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv/526bfe2/ : SUCCESS in 1h 01m 38s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"6e42f9bf2f7148621997faed5d16cbd959452089","author":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"date":"2017-02-08 16:12:57.000000000","message":"Abandoned\n\nthis will be addressed in change Ibf742198e83ae13f90767b28cc1858f0a17c3a95","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"25a64ea76248e96356fe297bffdb7684e45dd27b","revisions":{"6796e4e37675cade2b6aa2f106db3eef52b7b817":{"kind":"REWORK","_number":1,"created":"2017-02-02 20:33:57.000000000","uploader":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"ref":"refs/changes/10/428410/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/10/428410/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/1"}}},"commit":{"parents":[{"commit":"73f316d804abff4cfbe45bf01aa4c81c2450ff2f","subject":"Remove switch_port_descr and switch_chassis_descr","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73f316d804abff4cfbe45bf01aa4c81c2450ff2f"}]}],"author":{"name":"Félix Bouliane","email":"fbouliane@internap.com","date":"2017-02-01 20:00:31.000000000","tz":-300},"committer":{"name":"Félix Bouliane","email":"fbouliane@internap.com","date":"2017-02-02 20:34:01.000000000","tz":-300},"subject":"Select right image url before download","message":"Select right image url before download\n\nThe selection of the image is currently done within the ImageDownload\nclass which responsability is to find the right url and then download\nthe right image while calculating a MD5 hash.\n\nThis commit get the image url selection algorithm outside the\nImageDownload class allowing the caller _stream_raw_image_onto_device\nto know the right url. This will allow to know the image size before\ndownloading it in order to allocate the right amount of space on\nthe target disk.\n\nChange-Id: I587f9ec75832847b8ad6f87c733e2e9ed7510571\nPartial-bug: 1661328\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/6796e4e37675cade2b6aa2f106db3eef52b7b817"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/6796e4e37675cade2b6aa2f106db3eef52b7b817"}]},"branch":"refs/heads/master"},"25a64ea76248e96356fe297bffdb7684e45dd27b":{"kind":"REWORK","_number":2,"created":"2017-02-02 21:46:39.000000000","uploader":{"_account_id":20902,"name":"Félix Bouliane","email":"fbouliane@inap.com","username":"fbouliane"},"ref":"refs/changes/10/428410/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/10/428410/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/10/428410/2"}}},"commit":{"parents":[{"commit":"73f316d804abff4cfbe45bf01aa4c81c2450ff2f","subject":"Remove switch_port_descr and switch_chassis_descr","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/73f316d804abff4cfbe45bf01aa4c81c2450ff2f"}]}],"author":{"name":"Félix Bouliane","email":"fbouliane@internap.com","date":"2017-02-01 20:00:31.000000000","tz":-300},"committer":{"name":"Félix Bouliane","email":"fbouliane@internap.com","date":"2017-02-02 21:46:37.000000000","tz":-300},"subject":"Select right image url before download","message":"Select right image url before download\n\nThe selection of the image is currently done within the ImageDownload\nclass which responsability is to find the right url and then download\nthe right image while calculating a MD5 hash.\n\nThis commit get the image url selection algorithm outside the\nImageDownload class allowing the caller _stream_raw_image_onto_device\nto know the right url. This will allow to know the image size before\ndownloading it in order to allocate the right amount of space on\nthe target disk.\n\nChange-Id: I587f9ec75832847b8ad6f87c733e2e9ed7510571\nPartial-bug: 1661328\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/25a64ea76248e96356fe297bffdb7684e45dd27b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/25a64ea76248e96356fe297bffdb7684e45dd27b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}