)]}'
{"id":"openstack%2Fironic-python-agent~318538","triplet_id":"openstack%2Fironic-python-agent~master~I830fe7da1a59b46e348213b6f451c2ee55f6008c","project":"openstack/ironic-python-agent","branch":"master","topic":"bug/1561137","hashtags":[],"change_id":"I830fe7da1a59b46e348213b6f451c2ee55f6008c","subject":"Get root device hints from the node object","status":"MERGED","created":"2016-05-19 09:50:08.000000000","updated":"2016-05-30 21:31:57.000000000","submitted":"2016-05-30 13:03:34.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":10,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2f937d9e57fbcf117edc04bd779491e55842a322","_number":318538,"virtual_id_number":318538,"owner":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},{"value":2,"date":"2016-05-30 13:03:34.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},{"value":0,"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},{"value":0,"date":"2016-05-30 21:31:57.000000000","post_submit":true,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"recommended":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"all":[{"value":1,"date":"2016-05-27 08:20:17.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2016-05-26 14:07:59.000000000","_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},{"value":1,"date":"2016-05-27 03:42:47.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},{"value":2,"date":"2016-05-26 13:55:53.000000000","_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},{"value":0,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},{"value":2,"date":"2016-05-30 12:19:59.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":{"approved":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"all":[{"value":0,"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},{"value":0,"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},{"value":0,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},{"value":1,"date":"2016-05-30 12:19:59.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},{"value":0,"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},{"value":0,"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},{"value":0,"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"value":0,"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},{"value":0,"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},{"_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":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-05-19 09:51:03.000000000","updated_by":{"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},"reviewer":{"_account_id":7080,"name":"Mathieu Mitchell","email":"mat128@gmail.com","username":"mat128"},"state":"REVIEWER"},{"updated":"2016-05-25 15:19:46.000000000","updated_by":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"reviewer":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"state":"REVIEWER"},{"updated":"2016-05-25 19:03:24.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":"2016-05-26 13:55:53.000000000","updated_by":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"reviewer":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"state":"REVIEWER"},{"updated":"2016-05-26 14:07:59.000000000","updated_by":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"reviewer":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"state":"REVIEWER"},{"updated":"2016-05-27 03:42:47.000000000","updated_by":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"reviewer":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"state":"REVIEWER"},{"updated":"2016-05-27 08:20:17.000000000","updated_by":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"reviewer":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"state":"REVIEWER"},{"updated":"2016-05-30 12:19:59.000000000","updated_by":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"reviewer":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"state":"REVIEWER"},{"updated":"2016-05-30 13:03:34.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-05-30 21:31:57.000000000","updated_by":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"reviewer":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"state":"REVIEWER"}],"messages":[{"id":"899fccf5160beb021056d12b9c5d9735df9babe7","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-19 09:50:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5dfec7e6e7ce470e7031d2ec341d1458da3f0cca","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-19 10:38:13.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 http://docs-draft.openstack.org/38/318538/1/check/gate-ironic-python-agent-docs/f94de04//doc/build/html/ : SUCCESS in 3m 09s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/1/check/gate-ironic-python-agent-pep8/565c11b/ : SUCCESS in 3m 31s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/1/check/gate-ironic-python-agent-python27/1153836/ : FAILURE in 3m 18s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/1/check/gate-ironic-python-agent-python34/82c5131/ : FAILURE in 4m 10s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/1/check/gate-tempest-dsvm-ironic-agent_ssh-src/9cf61eb/ : SUCCESS in 38m 41s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/1/check/gate-tempest-dsvm-ironic-pxe_ipa-src/5c137bb/ : SUCCESS in 45m 19s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/1/check/gate-ironic-inspector-dsvm-ipa-src-nv/bc1eaff/ : SUCCESS in 34m 51s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/1/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/3b12f79/ : SUCCESS in 42m 44s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"78eb3892e06996521441d9e53b45480396a7ada5","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2016-05-20 12:11:36.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)\n\nglobal variables, sigh...\n\nok, 2 issues inline + please write tests and a release note","accounts_in_message":[],"_revision_number":1},{"id":"3b1effa4227c6ef51ada0b358a5849529cf83b9f","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-20 16:47:32.000000000","message":"Patch Set 1:\n\n(2 comments)\n\nThanks for the review, \n\nAbout the tests I explained in the commit message that I want to see if people are OK with a global variable before writing it (so I don\u0027t waste the time)","accounts_in_message":[],"_revision_number":1},{"id":"e48f8d89f25f03a1502bdcefd78fbd0bd9580577","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":"2016-05-20 17:02:20.000000000","message":"Patch Set 1: Code-Review-1\n\nOnce the comments inline are fixed and tests added, this is good to me! :)","accounts_in_message":[],"_revision_number":1},{"id":"dd9a935f1218fcb7253d4db12223e59dafe590c4","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-23 10:06:24.000000000","message":"Patch Set 1: Workflow-1\n\nCool thanks guys,\n\nI will update the patch and add tests. I also don\u0027t like a module variable, but the options are few and we need to to avoid breakages as much as possible.","accounts_in_message":[],"_revision_number":1},{"id":"7b4746b36279b069fd1e0112d1a4b0b27f60b1d1","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-23 11:11:10.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e6913d245f9d83b09735bf2475dbfde4c82b8e4b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-23 12:08:53.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/2/check/gate-ironic-python-agent-docs/f86caa8//doc/build/html/ : SUCCESS in 2m 33s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/2/check/gate-ironic-python-agent-pep8/6fc3722/ : SUCCESS in 2m 47s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/2/check/gate-ironic-python-agent-python27/509b327/ : SUCCESS in 2m 05s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/2/check/gate-ironic-python-agent-python34/5494a72/ : SUCCESS in 3m 20s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/2/check/gate-tempest-dsvm-ironic-agent_ssh-src/b8e70ed/ : SUCCESS in 36m 18s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/2/check/gate-tempest-dsvm-ironic-pxe_ipa-src/3d1c2d7/ : SUCCESS in 40m 32s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/2/check/gate-ironic-inspector-dsvm-ipa-src-nv/f661dfb/ : SUCCESS in 35m 00s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/2/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/e10b9d3/ : SUCCESS in 40m 01s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"6ae094b743a183557f8b2036cfb11bc2373cd49a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-23 12:10:38.000000000","message":"Patch Set 2: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":2},{"id":"a284f81b444eb3277b7394d12c0ff847cfd3b6c3","author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"date":"2016-05-23 12:30:11.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nI\u0027m a bit disturbed that you\u0027re having to call `cache_node` in multiple places - have you definitely covered all the code paths that might need access to the cached node? Is there some way of combining them into one call, so we can always be sure that the node has been cached?","accounts_in_message":[],"_revision_number":2},{"id":"eb84106d17812eef91917991bce4a946af1d2296","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-23 13:28:48.000000000","message":"Patch Set 2: Workflow-1\n\n(1 comment)\n\nThanks Miles for the suggestion,\n\nBelieve me, I don\u0027t like having to call cache_node() or having a module variable like this, but unless we agree that we should fix the interfaces even tho it will break out-of-tree managers our hands are tied due the flawed design of this code :-/","accounts_in_message":[],"_revision_number":2},{"id":"757dd75e32e8e78c866ea93271205d113462e16f","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-23 13:29:35.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"0b5191e78e50580f88b2bde5dd87a2d9cbc26898","author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"date":"2016-05-23 13:32:20.000000000","message":"Patch Set 3: Code-Review+1\n\nOh, I see - what a pain :-(","accounts_in_message":[],"_revision_number":3},{"id":"af9c77a0c8566e8d7a14445c824a15ac8245bb67","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":"2016-05-23 15:23:37.000000000","message":"Patch Set 3:\n\nMiles, you misunderstand.\n\nThe node isn\u0027t fetched by the agent; it\u0027s passed in for certain API calls. Lucas\u0027 code makes sure that anywhere the node is passed into an API call, IPA will cache it. \n\nDue to authentication trouble you can\u0027t really have the node fetched on demand from Ironic.","accounts_in_message":[],"_revision_number":3},{"id":"2bba97c3d4ff4ce056d8b65595eb4d60a91fc621","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":"2016-05-23 15:24:04.000000000","message":"Patch Set 3: Code-Review+1\n\ncore review will be given pending tests","accounts_in_message":[],"_revision_number":3},{"id":"ef72fc315911870852cddd6b1e0392e29e854ef3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-23 15:29:24.000000000","message":"Patch Set 3: 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 http://docs-draft.openstack.org/38/318538/3/check/gate-ironic-python-agent-docs/8fc8c15//doc/build/html/ : SUCCESS in 2m 57s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/3/check/gate-ironic-python-agent-pep8/8e7aab3/ : SUCCESS in 2m 00s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/3/check/gate-ironic-python-agent-python27/088edef/ : SUCCESS in 3m 55s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/3/check/gate-ironic-python-agent-python34/259e010/ : FAILURE in 3m 35s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/3/check/gate-tempest-dsvm-ironic-agent_ssh-src/48ddcae/ : SUCCESS in 40m 33s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/3/check/gate-tempest-dsvm-ironic-pxe_ipa-src/5a1f223/ : SUCCESS in 37m 39s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/3/check/gate-ironic-inspector-dsvm-ipa-src-nv/2773c13/ : FAILURE in 34m 19s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/3/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/0e4544a/ : SUCCESS in 35m 51s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"5e266617e4cc245cf02e68288fdedc4cb74cfab6","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-23 15:51:03.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"d1470e88a208f214836daa491698f86bd8186c4d","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-23 15:52:39.000000000","message":"Patch Set 4:\n\n- py34 tests should be fixed now (after rebasing with https://review.openstack.org/#/c/317969/)\n\n- The other job failure seems unrelated (from gate-ironic-inspector-dsvm-ipa-src-nv):\n\nNo matching distribution found for Django\u003d\u003d\u003d1.8.13 (from -c /opt/stack/new/requirements/upper-constraints.txt (line 5))","accounts_in_message":[],"_revision_number":4},{"id":"92ef91f5e8490978c234ceb4bb37bdc9578e4b09","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-23 17:46:16.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/4/check/gate-ironic-python-agent-docs/2e8407f//doc/build/html/ : SUCCESS in 2m 51s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/4/check/gate-ironic-python-agent-pep8/d27cbc3/ : SUCCESS in 2m 55s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/4/check/gate-ironic-python-agent-python27/ba7ca1d/ : SUCCESS in 2m 40s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/4/check/gate-ironic-python-agent-python34/ae49795/ : SUCCESS in 3m 13s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/4/check/gate-tempest-dsvm-ironic-agent_ssh-src/b575bbd/ : SUCCESS in 41m 40s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/4/check/gate-tempest-dsvm-ironic-pxe_ipa-src/fd7cbf9/ : SUCCESS in 44m 41s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/4/check/gate-ironic-inspector-dsvm-ipa-src-nv/054315b/ : SUCCESS in 48m 43s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/4/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/1ba2b54/ : SUCCESS in 38m 29s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"d387159cefa8324d40aaef55c45c6093a4b7baea","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-23 20:02:36.000000000","message":"Patch Set 4: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":4},{"id":"b5af387cf76aceb5f2fe41ff74f77d38a14fc102","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":"2016-05-23 22:56:32.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"75ac0fd4d2abedc2a1c6776de40e3fae80d845b1","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":"2016-05-23 22:58:28.000000000","message":"Patch Set 5: Code-Review+2\n\nI simply rebased this, and fixed a test, so still voting +2","accounts_in_message":[],"_revision_number":5},{"id":"b4aa9e13a3fd4fc3ac077fb1cd2bd4e2679c19f4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-23 23:41:21.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/5/check/gate-ironic-python-agent-docs/700648b//doc/build/html/ : SUCCESS in 3m 53s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/5/check/gate-ironic-python-agent-pep8/2a7a9ad/ : SUCCESS in 2m 17s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/5/check/gate-ironic-python-agent-python27/16087df/ : SUCCESS in 2m 23s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/5/check/gate-ironic-python-agent-python34/f40d094/ : SUCCESS in 2m 37s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/5/check/gate-tempest-dsvm-ironic-agent_ssh-src/4da1a9a/ : SUCCESS in 40m 17s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/5/check/gate-tempest-dsvm-ironic-pxe_ipa-src/a985142/ : SUCCESS in 42m 05s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/5/check/gate-ironic-inspector-dsvm-ipa-src-nv/71ad161/ : SUCCESS in 36m 21s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/5/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/8a43f8c/ : SUCCESS in 37m 22s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"b53fe7529a854629f15d20e0fe937a92964f2f9e","author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"date":"2016-05-24 04:33:08.000000000","message":"Patch Set 5: Code-Review-1\n\n(1 comment)\n\nMost looks good, have a comment inline.","accounts_in_message":[],"_revision_number":5},{"id":"1047c250ddde82542f07d01e0623491248dd929c","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-24 08:16:14.000000000","message":"Patch Set 5: Workflow-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"546c134c0db65354bf2c6c15ec0c1f769c073378","author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"date":"2016-05-24 09:58:44.000000000","message":"Patch Set 5:\n\nJay: ah, I see! So ideally we\u0027d pass the node object through a chain of method calls from the request-handler to where it\u0027s needed, but we can\u0027t do that without changing our internal APIs and breaking out-of-tree hardware managers.","accounts_in_message":[],"_revision_number":5},{"id":"a241f81f75b54e103ed7b7f1e25e1bd0695c31f1","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-24 16:05:12.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"5f50b84e243eb2dc6e6c0e42f6e4d1c74324496e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-24 22:03:14.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/6/check/gate-ironic-python-agent-docs/edce6fb//doc/build/html/ : SUCCESS in 1m 38s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/6/check/gate-ironic-python-agent-pep8/90f4470/ : SUCCESS in 1m 30s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/6/check/gate-ironic-python-agent-python27/d3b0c23/ : SUCCESS in 1m 30s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/6/check/gate-ironic-python-agent-python34/ca63e8a/ : SUCCESS in 3m 47s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/6/check/gate-tempest-dsvm-ironic-agent_ssh-src/6be1812/ : SUCCESS in 24m 55s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/6/check/gate-tempest-dsvm-ironic-pxe_ipa-src/75d5cb5/ : SUCCESS in 24m 50s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/6/check/gate-ironic-inspector-dsvm-ipa-src-nv/8136e5f/ : SUCCESS in 24m 12s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/6/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/a1795bf/ : SUCCESS in 36m 17s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"cc0862f769102ad9d9fd3e630282bc8e04f4ce22","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-25 09:43:00.000000000","message":"Patch Set 6: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":6},{"id":"901a10184c467be333ebcb5e8cc04cca15e48774","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-25 09:53:05.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"1883d32ac5133031aaff10f9b5f620526edf82da","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-25 10:44:30.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/7/check/gate-ironic-python-agent-docs/8a3e0d5//doc/build/html/ : SUCCESS in 2m 14s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/7/check/gate-ironic-python-agent-pep8/9655346/ : SUCCESS in 1m 30s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/7/check/gate-ironic-python-agent-python27/b32a0c9/ : SUCCESS in 1m 56s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/7/check/gate-ironic-python-agent-python34/8bfdca0/ : SUCCESS in 2m 14s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/7/check/gate-tempest-dsvm-ironic-agent_ssh-src/955b977/ : SUCCESS in 45m 36s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/7/check/gate-tempest-dsvm-ironic-pxe_ipa-src/c2aaa88/ : SUCCESS in 33m 57s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/7/check/gate-ironic-inspector-dsvm-ipa-src-nv/bf2de3d/ : SUCCESS in 29m 11s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/7/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/aa62309/ : SUCCESS in 42m 17s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"512ab30efd0a9a965039b1e576151e88891f8678","author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"date":"2016-05-25 15:19:46.000000000","message":"Patch Set 7: Code-Review+1","accounts_in_message":[],"_revision_number":7},{"id":"04e90f805505066d79622504d9554ccebd1fef1c","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":"2016-05-25 19:03:24.000000000","message":"Patch Set 7: Code-Review+2","accounts_in_message":[],"_revision_number":7},{"id":"ef553e11cf26e26f704ccad143a3d07fd995fc6d","author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"date":"2016-05-26 03:50:16.000000000","message":"Patch Set 7: Code-Review+1\n\nThank you. LGTM.","accounts_in_message":[],"_revision_number":7},{"id":"368db9625ea9a81b7a7214cf94f28e0cee14966f","author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"date":"2016-05-26 08:01:36.000000000","message":"Patch Set 7: Code-Review+1\n\nLGTM, thanks for the update","accounts_in_message":[],"_revision_number":7},{"id":"34adaeb64633b4a657e737b6bd2e801822e209af","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2016-05-26 13:39:58.000000000","message":"Patch Set 7: Code-Review+2\n\nLGTM. Giving more folks the last chance to take a look","accounts_in_message":[],"_revision_number":7},{"id":"aa1a1787b94fcf305050b011b95ac9a44bfd6331","author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"date":"2016-05-26 13:44:57.000000000","message":"Patch Set 7: Code-Review-1\n\nThis totally drops the kernel command line interface to root device hints, we at a minimum need a release note (but should we also deprecate that properly?)","accounts_in_message":[],"_revision_number":7},{"id":"467cf2494c2a5085f7e682e5f0088eab1475641e","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2016-05-26 13:52:25.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"67dae0e0d0d662357d97802adef66b2a46fb9c62","author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"date":"2016-05-26 13:55:53.000000000","message":"Patch Set 8: Code-Review+2","accounts_in_message":[],"_revision_number":8},{"id":"fac3eed3ad57313cd9b5995b4b1266c65228cd44","author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"date":"2016-05-26 14:07:59.000000000","message":"Patch Set 8: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"a8354fbe2a956dfac5138b6b44bab436b1b78c98","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-26 15:32:38.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/8/check/gate-ironic-python-agent-docs/4092769//doc/build/html/ : SUCCESS in 2m 39s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/8/check/gate-ironic-python-agent-pep8/daf180b/ : SUCCESS in 2m 55s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/8/check/gate-ironic-python-agent-python27/0d27065/ : SUCCESS in 5m 22s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/8/check/gate-ironic-python-agent-python34/eef82b5/ : SUCCESS in 3m 45s\n- gate-ironic-python-agent-releasenotes http://docs-draft.openstack.org/38/318538/8/check/gate-ironic-python-agent-releasenotes/cbccb03//releasenotes/build/html/ : SUCCESS in 3m 19s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/8/check/gate-tempest-dsvm-ironic-agent_ssh-src/a30ea81/ : SUCCESS in 41m 47s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/8/check/gate-tempest-dsvm-ironic-pxe_ipa-src/2743d14/ : SUCCESS in 42m 51s\n- gate-ironic-inspector-dsvm-ipa-src-nv http://logs.openstack.org/38/318538/8/check/gate-ironic-inspector-dsvm-ipa-src-nv/be90c6c/ : SUCCESS in 51m 02s (non-voting)\n- gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv http://logs.openstack.org/38/318538/8/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-src-nv/c745887/ : SUCCESS in 36m 05s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"4f40168f2334dd86a44b1bca7e641697d9dd567d","author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"date":"2016-05-27 03:42:47.000000000","message":"Patch Set 8: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"dc617c445236dbd94a54ba206133014202291926","author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"date":"2016-05-27 08:20:17.000000000","message":"Patch Set 8: Code-Review+1","accounts_in_message":[],"_revision_number":8},{"id":"03b41c8bbc7cd6ee8bf5584ea7c4827bfa722010","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2016-05-30 12:19:59.000000000","message":"Patch Set 8: Workflow+1 Code-Review+2","accounts_in_message":[],"_revision_number":8},{"id":"a0030d8fc297dd4b15bb994deb79c6583e943ea7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-30 12:20:38.000000000","message":"Patch Set 8: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":8},{"id":"1593461c17fef1c1099e52b75e3063aea76a2da4","date":"2016-05-30 13:03:34.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":8},{"id":"5737b701afd927fe983501c7c5b28306ef4596c3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-05-30 13:03:34.000000000","message":"Patch Set 8: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-ironic-python-agent-docs http://docs-draft.openstack.org/38/318538/8/gate/gate-ironic-python-agent-docs/b68a11c//doc/build/html/ : SUCCESS in 4m 12s\n- gate-ironic-python-agent-pep8 http://logs.openstack.org/38/318538/8/gate/gate-ironic-python-agent-pep8/d2276bb/ : SUCCESS in 2m 37s\n- gate-ironic-python-agent-python27 http://logs.openstack.org/38/318538/8/gate/gate-ironic-python-agent-python27/19d16a2/ : SUCCESS in 1m 40s\n- gate-ironic-python-agent-python34 http://logs.openstack.org/38/318538/8/gate/gate-ironic-python-agent-python34/5a130e9/ : SUCCESS in 1m 51s\n- gate-ironic-python-agent-releasenotes http://docs-draft.openstack.org/38/318538/8/gate/gate-ironic-python-agent-releasenotes/9d8527d//releasenotes/build/html/ : SUCCESS in 1m 50s\n- gate-tempest-dsvm-ironic-agent_ssh-src http://logs.openstack.org/38/318538/8/gate/gate-tempest-dsvm-ironic-agent_ssh-src/a965ad7/ : SUCCESS in 41m 50s\n- gate-tempest-dsvm-ironic-pxe_ipa-src http://logs.openstack.org/38/318538/8/gate/gate-tempest-dsvm-ironic-pxe_ipa-src/7d97339/ : SUCCESS in 41m 54s","accounts_in_message":[],"_revision_number":8},{"id":"50e871a4f89a1eba9289cb7ea2d78982ffd054ac","author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"date":"2016-05-30 21:31:57.000000000","message":"Patch Set 8:\n\n@Lucas. Was the RFE approved? It isn\u0027t tagged if it was. Looks like we still need to agree on the operators?","accounts_in_message":[],"_revision_number":8}],"current_revision_number":8,"current_revision":"33535cd572b90a817b9c96b42a0d2fe54751351d","revisions":{"c323088d200d2ef895f6695818e411bfc7e2042d":{"kind":"REWORK","_number":1,"created":"2016-05-19 09:50:08.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/1"}}},"commit":{"parents":[{"commit":"89f4f33128c7e86e44faf61aa75fe5e2080f8ab9","subject":"Merge \"TinyIPA: Ensure that kernel modules are loaded at boot\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/89f4f33128c7e86e44faf61aa75fe5e2080f8ab9"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:49:43.000000000","tz":60},"subject":"Store node in the hardware module","message":"Store node in the hardware module\n\nThis patch adds a new function to the hardware.py module called\n\"cache_node\". The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods.\n\nWIP: No tests, checking if people are OK with this idea before.\n\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/c323088d200d2ef895f6695818e411bfc7e2042d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/c323088d200d2ef895f6695818e411bfc7e2042d"}]},"branch":"refs/heads/master"},"6fc28bf831cc36584c59513b2f55ad2a642d683e":{"kind":"REWORK","_number":2,"created":"2016-05-23 11:11:10.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/2"}}},"commit":{"parents":[{"commit":"89f4f33128c7e86e44faf61aa75fe5e2080f8ab9","subject":"Merge \"TinyIPA: Ensure that kernel modules are loaded at boot\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/89f4f33128c7e86e44faf61aa75fe5e2080f8ab9"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-23 11:04:24.000000000","tz":60},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without breaking\nthe compatibility with out-of-tree hardware managers (for not changing\nthe methods signatures).\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/6fc28bf831cc36584c59513b2f55ad2a642d683e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/6fc28bf831cc36584c59513b2f55ad2a642d683e"}]},"branch":"refs/heads/master"},"0e17e401a1919b39b5ba6225d1acf6bcff2c2ad2":{"kind":"REWORK","_number":3,"created":"2016-05-23 13:29:35.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/3"}}},"commit":{"parents":[{"commit":"c688b98917622cc1d9c323006db76273722f18ed","subject":"Merge \"Add hardware manager interface for hardware initialization\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/c688b98917622cc1d9c323006db76273722f18ed"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-23 13:28:52.000000000","tz":60},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without changing\nmethod signatures, which would break compatibility with out-of-tree\nhardware managers.\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/0e17e401a1919b39b5ba6225d1acf6bcff2c2ad2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/0e17e401a1919b39b5ba6225d1acf6bcff2c2ad2"}]},"branch":"refs/heads/master"},"64ba77ac98627fb479816b7a8536e55754440bcd":{"kind":"REWORK","_number":4,"created":"2016-05-23 15:51:03.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/4"}}},"commit":{"parents":[{"commit":"c688b98917622cc1d9c323006db76273722f18ed","subject":"Merge \"Add hardware manager interface for hardware initialization\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/c688b98917622cc1d9c323006db76273722f18ed"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-23 15:48:48.000000000","tz":60},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without changing\nmethod signatures, which would break compatibility with out-of-tree\nhardware managers.\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/64ba77ac98627fb479816b7a8536e55754440bcd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/64ba77ac98627fb479816b7a8536e55754440bcd"}]},"branch":"refs/heads/master"},"f2ec4dd03762a99552e61f99ada769d20d9c929e":{"kind":"REWORK","_number":5,"created":"2016-05-23 22:56:32.000000000","uploader":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"ref":"refs/changes/38/318538/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/5"}}},"commit":{"parents":[{"commit":"015fad6054d0572daadd193b68ba17c9de0ae8e0","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/015fad6054d0572daadd193b68ba17c9de0ae8e0"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Jay Faulkner","email":"jay@jvf.cc","date":"2016-05-23 22:56:23.000000000","tz":0},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without changing\nmethod signatures, which would break compatibility with out-of-tree\nhardware managers.\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/f2ec4dd03762a99552e61f99ada769d20d9c929e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/f2ec4dd03762a99552e61f99ada769d20d9c929e"}]},"branch":"refs/heads/master"},"56e11e33d5554728c682130abe2a83e4ebf64558":{"kind":"REWORK","_number":6,"created":"2016-05-24 16:05:12.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/6"}}},"commit":{"parents":[{"commit":"015fad6054d0572daadd193b68ba17c9de0ae8e0","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/015fad6054d0572daadd193b68ba17c9de0ae8e0"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-24 16:04:18.000000000","tz":60},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without changing\nmethod signatures, which would break compatibility with out-of-tree\nhardware managers.\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nThe function parse_root_device_hints() and its tests were removed since\nit\u0027s not used/needed anymore.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/56e11e33d5554728c682130abe2a83e4ebf64558"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/56e11e33d5554728c682130abe2a83e4ebf64558"}]},"branch":"refs/heads/master"},"d41b37d06aecc9e1a3f15d59e6cfb513e0bb45fe":{"kind":"REWORK","_number":7,"created":"2016-05-25 09:53:05.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/7"}}},"commit":{"parents":[{"commit":"962ee1afb5390080b30444652bc201dd59382220","subject":"Merge \"Wait for at least one suitable disk to appear on start up\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/962ee1afb5390080b30444652bc201dd59382220"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-25 09:49:05.000000000","tz":60},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without changing\nmethod signatures, which would break compatibility with out-of-tree\nhardware managers.\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nThe function parse_root_device_hints() and its tests were removed since\nit\u0027s not used/needed anymore.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/d41b37d06aecc9e1a3f15d59e6cfb513e0bb45fe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/d41b37d06aecc9e1a3f15d59e6cfb513e0bb45fe"}]},"branch":"refs/heads/master"},"33535cd572b90a817b9c96b42a0d2fe54751351d":{"kind":"REWORK","_number":8,"created":"2016-05-26 13:52:25.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/38/318538/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/38/318538/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/38/318538/8"}}},"commit":{"parents":[{"commit":"962ee1afb5390080b30444652bc201dd59382220","subject":"Merge \"Wait for at least one suitable disk to appear on start up\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/962ee1afb5390080b30444652bc201dd59382220"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-19 09:48:02.000000000","tz":60},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2016-05-26 13:52:15.000000000","tz":60},"subject":"Get root device hints from the node object","message":"Get root device hints from the node object\n\nIn order to support a more complex syntax for root device hints (e.g\noperators: greater than, less than, in, etc...) we need to stop relying\non the kernel command line for passing the root device hints. This patch\nchanges this approach by getting the root device hints from a cached\nnode object that was set in the hardware module.\n\nTwo new functions: \"cache_node\" and \"get_cached_node\" were added to the\nhardware module. The idea is to facilitate the access to a node object\nrepresentation from the hardware extension methods without changing\nmethod signatures, which would break compatibility with out-of-tree\nhardware managers.\n\nNote that the new \"get_cached_node\" is just a guard function to\nfacilitate the tests for the code.\n\nThe function parse_root_device_hints() and its tests were removed since\nit\u0027s not used/needed anymore.\n\nPartial-Bug: #1561137\nChange-Id: I830fe7da1a59b46e348213b6f451c2ee55f6008c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/33535cd572b90a817b9c96b42a0d2fe54751351d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/33535cd572b90a817b9c96b42a0d2fe54751351d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}