)]}'
{"doc/source/drivers/vbox.rst":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"36081d206e25a95eff713cca1ce15932771333ea","unresolved":false,"context_lines":[{"line_number":25,"context_line":"* Install VirtualBox on your desktop or laptop."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Create a VM for the cloud controller. Do not power on the VM now."},{"line_number":28,"context_line":"  Let\u0027s name in ``cloud-controller``."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* In VirtualBox Manager, Select ``cloud-controller`` VM -\u003e Click Settings -\u003e"},{"line_number":31,"context_line":"  Network -\u003e Adapter 2 -\u003e Select \u0027Enable Network Adapter\u0027 -\u003e"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_d3e2929e","line":28,"updated":"2015-02-02 14:00:29.000000000","message":"Seems to be a nit here, \u0027name it\u0027? Maybe better \u0027for example, ``cloud-controller``\u0027?","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"3046e75fedf99950056ea01c5b3a73ed293df1a2","unresolved":false,"context_lines":[{"line_number":25,"context_line":"* Install VirtualBox on your desktop or laptop."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"* Create a VM for the cloud controller. Do not power on the VM now."},{"line_number":28,"context_line":"  Let\u0027s name in ``cloud-controller``."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* In VirtualBox Manager, Select ``cloud-controller`` VM -\u003e Click Settings -\u003e"},{"line_number":31,"context_line":"  Network -\u003e Adapter 2 -\u003e Select \u0027Enable Network Adapter\u0027 -\u003e"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_c26c449f","line":28,"in_reply_to":"da86d52c_d3e2929e","updated":"2015-02-10 11:18:47.000000000","message":"Done","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"4ed5cffbbc369d2382ae1ccd7ede40983a1464a0","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* Install ZSI library and pyremotevbox on ``cloud-controller``::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    sudo apt-get install python-ZSI"},{"line_number":57,"context_line":"    sudo pip install pyremotevbox"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Enable ``pxe_vbox`` or ``agent_vbox`` in ``enabled_drivers`` in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_b2e38604","line":56,"updated":"2015-01-30 21:14:50.000000000","message":"This is on pypi, any reason not to use pip here? Or better yet, make it a dependency of pyremotevbox so pip pulls it in?","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"3046e75fedf99950056ea01c5b3a73ed293df1a2","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* Install ZSI library and pyremotevbox on ``cloud-controller``::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    sudo apt-get install python-ZSI"},{"line_number":57,"context_line":"    sudo pip install pyremotevbox"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Enable ``pxe_vbox`` or ``agent_vbox`` in ``enabled_drivers`` in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_e26788b5","line":56,"in_reply_to":"da86d52c_3c3c69e2","updated":"2015-02-10 11:18:47.000000000","message":"Done","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"264d30f7ecea1518a3f86d3a41a12347a328e8c9","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* Install ZSI library and pyremotevbox on ``cloud-controller``::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    sudo apt-get install python-ZSI"},{"line_number":57,"context_line":"    sudo pip install pyremotevbox"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Enable ``pxe_vbox`` or ``agent_vbox`` in ``enabled_drivers`` in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_cb3d9cbd","line":56,"in_reply_to":"da86d52c_b2e38604","updated":"2015-01-31 03:01:34.000000000","message":"None of ZSI versions available in pypi seem to work. I have tried using many ZSIs from pypi, but all seem to fail for one reason or another.  Once I figure this out, I will remove it and make it part of pyremotevbox dependency.","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f774c9eaae399c758a6038c9ae4d17ee79a724f0","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* Install ZSI library and pyremotevbox on ``cloud-controller``::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    sudo apt-get install python-ZSI"},{"line_number":57,"context_line":"    sudo pip install pyremotevbox"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Enable ``pxe_vbox`` or ``agent_vbox`` in ``enabled_drivers`` in"}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_3c3c69e2","line":56,"in_reply_to":"da86d52c_cb3d9cbd","updated":"2015-02-10 10:20:09.000000000","message":"If you\u0027re putting a new patch-set for this patch, can you please add the fedora package for it too?\n\n# fedora/RHEL/CentOS\n\nsudo yum install python-ZSI","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f774c9eaae399c758a6038c9ae4d17ee79a724f0","unresolved":false,"context_lines":[{"line_number":54,"context_line":"* Install ZSI library and pyremotevbox on ``cloud-controller``::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    sudo apt-get install python-ZSI"},{"line_number":57,"context_line":"    sudo pip install pyremotevbox"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Enable ``pxe_vbox`` or ``agent_vbox`` in ``enabled_drivers`` in"},{"line_number":60,"context_line":"  ``/etc/ironic/ironic.conf`` and restart Ironic conductor."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_5c3f5dd6","line":57,"updated":"2015-02-10 10:20:09.000000000","message":"You might want to include this here too https://github.com/openstack/ironic/blob/master/driver-requirements.txt","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"3046e75fedf99950056ea01c5b3a73ed293df1a2","unresolved":false,"context_lines":[{"line_number":54,"context_line":"* Install ZSI library and pyremotevbox on ``cloud-controller``::"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    sudo apt-get install python-ZSI"},{"line_number":57,"context_line":"    sudo pip install pyremotevbox"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"* Enable ``pxe_vbox`` or ``agent_vbox`` in ``enabled_drivers`` in"},{"line_number":60,"context_line":"  ``/etc/ironic/ironic.conf`` and restart Ironic conductor."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_02065cce","line":57,"in_reply_to":"da86d52c_5c3f5dd6","updated":"2015-02-10 11:18:47.000000000","message":"Done","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"4b1a7a0d5eafe358d19bedfff288e76ef3249c21","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"* Enroll the VirtualBox node::"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    ironic node-create -d pxe_vbox -i vbox_host\u003d\u002710.0.2.2\u0027 vbox_vmname\u003d\u0027baremetal\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"* Add other Node properties and trigger provisioning on the bare metal node."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_254d73c5","line":67,"updated":"2015-02-08 22:25:25.000000000","message":"I think the properties you need to supply here are \u0027virtualbox_host\u0027 and \u0027virtualbox_vmname\u0027 - and do you need a second \u0027-i\u0027?","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f774c9eaae399c758a6038c9ae4d17ee79a724f0","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"* Enroll the VirtualBox node::"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    ironic node-create -d pxe_vbox -i vbox_host\u003d\u002710.0.2.2\u0027 vbox_vmname\u003d\u0027baremetal\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"* Add other Node properties and trigger provisioning on the bare metal node."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_c73456e0","line":67,"in_reply_to":"da86d52c_254d73c5","updated":"2015-02-10 10:20:09.000000000","message":"Good catch,\n\nYeah according to the code the options are under the \"virtualbox_*\" namespace. \n\n ironic/drivers/modules/virtualbox.py:    \u0027virtualbox_host\u0027: _(\"IP address or hostname of the VirtualBox host. \"\n\nThe -i option is also required as rightly pointed here by Stig","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"3046e75fedf99950056ea01c5b3a73ed293df1a2","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"* Enroll the VirtualBox node::"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    ironic node-create -d pxe_vbox -i vbox_host\u003d\u002710.0.2.2\u0027 vbox_vmname\u003d\u0027baremetal\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"* Add other Node properties and trigger provisioning on the bare metal node."}],"source_content_type":"text/x-rst","patch_set":2,"id":"da86d52c_2209a0dd","line":67,"in_reply_to":"da86d52c_c73456e0","updated":"2015-02-10 11:18:47.000000000","message":"Done","commit_id":"e9ccfe3936ccbdfa341296869e33546afd02c282"},{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"f0d838bc82a217ab569cea9e50fa3184ea600198","unresolved":false,"context_lines":[{"line_number":37,"context_line":"* In VirtualBox Manager, Select ``baremetal`` VM -\u003e Click Settings -\u003e"},{"line_number":38,"context_line":"  Network -\u003e Adapter 1 -\u003e Select \u0027Enable Network Adapter\u0027 -\u003e"},{"line_number":39,"context_line":"  Select Attached to: Internel Network -\u003e Select Name: intnet"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"* Configure the VirtualBox web service to disable authentication (This is"},{"line_number":42,"context_line":"  only a suggestion, enable authentication if you want with appropriate"},{"line_number":43,"context_line":"  web service authentication library)::"}],"source_content_type":"text/x-rst","patch_set":3,"id":"da86d52c_d9d0ba55","line":40,"updated":"2015-02-10 11:47:14.000000000","message":"I still think you need to add instructions on setting the boot order.  When I tried the driver without configuring boot order, my VM popped up but wanted a CD image.","commit_id":"953b595919044a1fb49875aa10794feb1b7fe2c0"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"a407617de45a0c04c4bb47c32d90a24751bba7ec","unresolved":false,"context_lines":[{"line_number":37,"context_line":"* In VirtualBox Manager, Select ``baremetal`` VM -\u003e Click Settings -\u003e"},{"line_number":38,"context_line":"  Network -\u003e Adapter 1 -\u003e Select \u0027Enable Network Adapter\u0027 -\u003e"},{"line_number":39,"context_line":"  Select Attached to: Internel Network -\u003e Select Name: intnet"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"* Configure the VirtualBox web service to disable authentication (This is"},{"line_number":42,"context_line":"  only a suggestion, enable authentication if you want with appropriate"},{"line_number":43,"context_line":"  web service authentication library)::"}],"source_content_type":"text/x-rst","patch_set":3,"id":"da86d52c_1b832205","line":40,"in_reply_to":"da86d52c_d9d0ba55","updated":"2015-02-10 14:15:15.000000000","message":"pxe_vbox has a management interface which sets the boot order to network.  It\u0027s a virtualbox thing for the first time for any newly created VM, it pops up that dialog asking for a cd image. I checked and it doesn\u0027t come second time.  I will add a note here.","commit_id":"953b595919044a1fb49875aa10794feb1b7fe2c0"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2ab009d42b7d54c0c786504022ba87fed4476c52","unresolved":false,"context_lines":[{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VirtualBox drivers helps to use VirtualBox VMs as bare metals in Ironic."},{"line_number":11,"context_line":"This is primararily intended for Ironic developers running Windows on their"},{"line_number":12,"context_line":"laptops/desktops.  This driver enables the developers to configure cloud"},{"line_number":13,"context_line":"controller on one VirtualBox VM and use other VMs in the same VirtualBox as"},{"line_number":14,"context_line":"bare metals for the cloud controller. These drivers uses VirtualBox web service"}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_df149a45","line":11,"updated":"2015-02-12 20:43:05.000000000","message":"s/primararily/primarily/","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"e383fdbec25551577afd55f5224d281ba4e1f73d","unresolved":false,"context_lines":[{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VirtualBox drivers helps to use VirtualBox VMs as bare metals in Ironic."},{"line_number":11,"context_line":"This is primararily intended for Ironic developers running Windows on their"},{"line_number":12,"context_line":"laptops/desktops.  This driver enables the developers to configure cloud"},{"line_number":13,"context_line":"controller on one VirtualBox VM and use other VMs in the same VirtualBox as"},{"line_number":14,"context_line":"bare metals for the cloud controller. These drivers uses VirtualBox web service"}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_aa6954bf","line":11,"in_reply_to":"da86d52c_df149a45","updated":"2015-02-13 04:40:45.000000000","message":"Done","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2ab009d42b7d54c0c786504022ba87fed4476c52","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VirtualBox drivers helps to use VirtualBox VMs as bare metals in Ironic."},{"line_number":11,"context_line":"This is primararily intended for Ironic developers running Windows on their"},{"line_number":12,"context_line":"laptops/desktops.  This driver enables the developers to configure cloud"},{"line_number":13,"context_line":"controller on one VirtualBox VM and use other VMs in the same VirtualBox as"},{"line_number":14,"context_line":"bare metals for the cloud controller. These drivers uses VirtualBox web service"},{"line_number":15,"context_line":"to control the VirtualBox VMs."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_9f6ce2d1","line":12,"updated":"2015-02-12 20:43:05.000000000","message":"s/desktops./desktops, although they can be used on other operating systems as well./\n\nThere is actually a nice description in the spec about using this vs SSH. It might be useful for people trying to decide which driver to use.","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"e383fdbec25551577afd55f5224d281ba4e1f73d","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"VirtualBox drivers helps to use VirtualBox VMs as bare metals in Ironic."},{"line_number":11,"context_line":"This is primararily intended for Ironic developers running Windows on their"},{"line_number":12,"context_line":"laptops/desktops.  This driver enables the developers to configure cloud"},{"line_number":13,"context_line":"controller on one VirtualBox VM and use other VMs in the same VirtualBox as"},{"line_number":14,"context_line":"bare metals for the cloud controller. These drivers uses VirtualBox web service"},{"line_number":15,"context_line":"to control the VirtualBox VMs."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_ca64c8e6","line":12,"in_reply_to":"da86d52c_9f6ce2d1","updated":"2015-02-13 04:40:45.000000000","message":"Done","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2ab009d42b7d54c0c786504022ba87fed4476c52","unresolved":false,"context_lines":[{"line_number":14,"context_line":"bare metals for the cloud controller. These drivers uses VirtualBox web service"},{"line_number":15,"context_line":"to control the VirtualBox VMs."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Currently there 2 drivers are available:"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"* ``pxe_vbox`` - Uses iSCSI based deployment mechanism."},{"line_number":20,"context_line":"* ``agent_vbox`` - Uses agent based deployment mechanism."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_7fa00ef4","line":17,"updated":"2015-02-12 20:43:05.000000000","message":"s/Currently there 2/Two /\n\nThere\u0027s also fake_vbox. Maybe not worth mentioning?","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"e383fdbec25551577afd55f5224d281ba4e1f73d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"bare metals for the cloud controller. These drivers uses VirtualBox web service"},{"line_number":15,"context_line":"to control the VirtualBox VMs."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Currently there 2 drivers are available:"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"* ``pxe_vbox`` - Uses iSCSI based deployment mechanism."},{"line_number":20,"context_line":"* ``agent_vbox`` - Uses agent based deployment mechanism."}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_6af11c00","line":17,"in_reply_to":"da86d52c_7fa00ef4","updated":"2015-02-13 04:40:45.000000000","message":"Done.  Don\u0027t know if it is worth mentioning about fake_vbox.  Nobody would want to confirm if ironic driver in vbox works fine. They would want to test changes of deploy and other things. But I will just mention it. :)","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2ab009d42b7d54c0c786504022ba87fed4476c52","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    ironic node-create -d pxe_vbox -i virtualbox_host\u003d\u002710.0.2.2\u0027 -i virtualbox_vmname\u003d\u0027baremetal\u0027"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"* Add other Node properties and trigger provisioning on the bare metal node."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_3f6f7661","line":86,"updated":"2015-02-12 20:43:05.000000000","message":"I think it is useful (although I don\u0027t know if the other drivers have documented this) to document \n\n1 the driver_info properties - required \u0026 optional (or mention how to get that info)\n\n2. the configs if any. Looks like there is \u0027port\u0027.","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"e383fdbec25551577afd55f5224d281ba4e1f73d","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    ironic node-create -d pxe_vbox -i virtualbox_host\u003d\u002710.0.2.2\u0027 -i virtualbox_vmname\u003d\u0027baremetal\u0027"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"* Add other Node properties and trigger provisioning on the bare metal node."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"da86d52c_aadeb46a","line":86,"in_reply_to":"da86d52c_3f6f7661","updated":"2015-02-13 04:40:45.000000000","message":"Done","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"}],"ironic/drivers/modules/virtualbox.py":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"2ab009d42b7d54c0c786504022ba87fed4476c52","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        vm_object \u003d host.find_vm(driver_info[\u0027vmname\u0027])"},{"line_number":159,"context_line":"    except virtualbox_exc.PyRemoteVBoxException as exc:"},{"line_number":160,"context_line":"        LOG.error(_LE(\"Failed while creating a VirtualMachine object for \""},{"line_number":161,"context_line":"                      \"node %(node_id)s. Error: %(error)s.\"),"},{"line_number":162,"context_line":"                  {\u0027node_id\u0027: node.uuid, \u0027error\u0027: exc})"},{"line_number":163,"context_line":"        raise exception.VirtualBoxOperationFailed(operation\u003dvm_object_method,"},{"line_number":164,"context_line":"                                                  error\u003dexc)"}],"source_content_type":"text/x-python","patch_set":5,"id":"da86d52c_3fea16d8","line":161,"updated":"2015-02-12 20:43:05.000000000","message":"I don\u0027t like that you fixed a bug here, in a patch that is dealing with documentation.","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"e383fdbec25551577afd55f5224d281ba4e1f73d","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        vm_object \u003d host.find_vm(driver_info[\u0027vmname\u0027])"},{"line_number":159,"context_line":"    except virtualbox_exc.PyRemoteVBoxException as exc:"},{"line_number":160,"context_line":"        LOG.error(_LE(\"Failed while creating a VirtualMachine object for \""},{"line_number":161,"context_line":"                      \"node %(node_id)s. Error: %(error)s.\"),"},{"line_number":162,"context_line":"                  {\u0027node_id\u0027: node.uuid, \u0027error\u0027: exc})"},{"line_number":163,"context_line":"        raise exception.VirtualBoxOperationFailed(operation\u003dvm_object_method,"},{"line_number":164,"context_line":"                                                  error\u003dexc)"}],"source_content_type":"text/x-python","patch_set":5,"id":"da86d52c_ea526cdc","line":161,"in_reply_to":"da86d52c_3fea16d8","updated":"2015-02-13 04:40:45.000000000","message":"Ruby, this was just a follow-up patch to the actual patch.  And this is just fixing a bug in the log message during error-case. Does it deserve to be fixed in a patch of alone ? If you still feel so, I will remove it and fix it in another patch.","commit_id":"dc2461fd53ec39bf2e60b59e328db8cda49fd436"}]}