GitHub PushEvent at 2014-06-26 08:39:52
Repository: ironic
Actor: hanlind
Payload: {"shas":[["230ff8d82154bd1f3290ed136032f541090e1792","ef89b923b1145c9ec2fd96eee513279016a7a963@gmail.com","Return the HTTP Location for newly created resources\n\nTo provide information about the location of a newly created resource\n(201) the Ironic API now returns the URI for the new object in the HTTP\nLocation header.\n\nA new function build_url() was created in the link.py by factoring out\npart of the the Link.make_link() method so that we can reuse that part\nof the code to build a URL without having to create a new Link object.\n\nPartial-Bug: #1317044\nChange-Id: I88b033d446aaed93fc90ab6497578d5cd729dea8","Lucas Alvares Gomes",true],["b5eeb1485eb930c6e270021989e81218f7a8fccf","ef89b923b1145c9ec2fd96eee513279016a7a963@gmail.com","Return the HTTP Location for accepted requestes\n\nAfter the server accepts a request (202) it now returns an URI in the\nHTTP Location header so that clients will know how to track the status\nof the request.\n\nCloses-Bug: #1317044\nChange-Id: I5286ed762426dea02a6513c0e31cbc9a8023f803","Lucas Alvares Gomes",true],["27dfedbc9d3653df8ab27acfa010d4c356ab2b2b","ef89b923b1145c9ec2fd96eee513279016a7a963@gmail.com","PXE driver to validate the requested image in Glance\n\nThis patch adds a check in the PXE driver validate() method to check if\nthe requested image exist in Glance and if it contains the 'kernel_id'\nand 'ramdisk_id' properties set which are needed for the PXE boot.\n\nCloses-Bug: #1324840\nChange-Id: I42bf7fcb27286afb3db0fda2a3a09ea49c77496b","Lucas Alvares Gomes",true],["f662f5fa4ceb69336011d29d291d5c47253d74f5","42dc46c030a2ed8fcb1cb70943e146952141c5c6@gmail.com","Log exceptions from deploy and tear_down\n\nIf deploy or tear_down raise an exception, an error is set on the Node\nand visible in the API, but nothing is logged. This change adds\nlogging in the ConductorManager of exceptions raised from either\nmethod.\n\nChange-Id: I17a9328770b6fc9fce167a189d9692b5c4c2687f","Russell Haering",true],["ffa8c91a951f37b3613a5abdb29c58fa917906cc","1cd02e31b43620d7c664e038ca42a060d61727b9@jimrollenhagen.com","Fix concurrent deletes in virt driver\n\nThe _unprovision function used a class member to track the number\nof calls to ironic to check the provision state. When more than one\ndelete is happening concurrently, the deletes will both add to the\nsame counter. This commit fixes this issue to use a local counter.\n\nPartial-Bug: #1326364\n\nChange-Id: I2552332445fe0e614970ec28eddf7c1ae94e34be","Jim Rollenhagen",true],["be70a1f961663f39a8b20841c5f0244dac9762d5","e380c54048b41940363be7c59d4b48a5776480a4@mxn.nes.nec.co.jp","Fix the logging message to identify node by uuid\n\nThe logging message of _build_pxe_config method in\nironic/drivers/modules/pxe.py is like below:\nBuilding PXE config for deployment 4.\n\n\"4\" is a ID of the node, but users don't know ID of the node\neven if using CLI or API.\nUsers can know only uuid of the node.\nSo that, uuid of the node should be written in the logging message.\n\nThis patch makes identify node by uuid not by id.\n\nChange-Id: Ida220a97d5fc8f6c5adc5b5acf239bbee14ff847\nCloses-bug: #1327085","Yuiko Takada",true],["295ff70983e71a16365ab2e2b2a0459237eb9089","ef89b923b1145c9ec2fd96eee513279016a7a963@gmail.com","Reuse validate_instance_and_node() Nova ironic Driver\n\nThe driver already counts with a function called\nvalidate_instance_and_node() that checks which node is associated to\na given instance and raises InstanceNotFound if the instance UUID is\nnot found in Ironic. The same functionality was being duplicate in many\nother parts of the driver, this patch simply update those parts to use\nthe validate_instance_and_node() function.\n\nChange-Id: I9a2f90c7cdd0fdfcc00d49a49581c61e4f073888","Lucas Alvares Gomes",true],["e32d95e4285bd6eb76cf3ec8cb705133dc98f829","ef89b923b1145c9ec2fd96eee513279016a7a963@gmail.com","Fix AttributeError when calling create_engine()\n\nThe module which contains the create_engine() method is the sqlalchemy,\nnot slqalchemy.api.\n\nChange-Id: I0b9e2f5b25f0c8ea9f7e55d863fa170cc9c09564\nPartial-Bug: #1327225","Lucas Alvares Gomes",true],["ef83e3a6fb6901efbc3156ea6e33320ec9f6d801","17b9e1c64588c7fa6419b4d29dc1f4426279ba01@the-davies.net","Improving unit tests for _do_sync_power_state\n\nImprove the unit tets coverage, and tightening up existing\ntests surrounding the _do_sync_power_state method.\n\nChange-Id: I9fd4c4c42936fcf8fc4f36bf35b6e691c495b8b0\nCloses-bug: #1297404","Michael Davies",true],["f9c7e4344a49a37edbb575e0b37254a8714d1d38","73c7819b146e140ef1b504c64cda275535901b16@gmail.com","Skip migrations test_walk_versions instead of pass\n\nWhen there is no db engine configured for the migration tests,\nthe test_walk_versions test was passing, rather than skipping,\nbecause it contained a for-loop and no check for the length of the\nsequence being iterated over.\n\nThis patch checks the length of the configured engines and\nproperly skips the test when there are no engines.\n\nChange-Id: I8a5c0a66f65a99c5169b251be02a8fbe6c727641\nCloses-bug: #1327397","Devananda van der Veen",true],["87aac8823b468a7bb80c994cd207ec5df66163c5","edf97d05d0a6276bac8d2ddc7075d5d9796f742d@codestud.com","Add __init__.py for nova scheduler filters\n\nTo use these filters out of tree right now, you have to specify them in\nnova's available_scheduler filters by full path. But in order for the\nmodules to be found and loaded, you must have an __init__.py.\n\nChange-Id: I4816d9dbe0e93056792fad48e6228ffdd531e346","Chris Behrens",true],["9d97cce8c11d03da93008448a6573305e3ed1b75","c9384d091c0fc43ad6431d9c6d8eb94cc7b7e285@mirantis.com","Cleanup mock patch without `with` part 2\n\nA good idea because it makes the code easier to read to get rid of\nthe `with` using mock.patch and mock.patch.object in favor decorators,\nwhere possible in the tests.\n\nThis patch affects Nova driver tests.\n\nChange-Id: Ifd2e4d36c3cbcf82cb9b83c375ba0a3e98019adf","Mikhail Durnosvistov",true],["990d6545fb7b3488769f7e2d15382a6f895a67b9","73c7819b146e140ef1b504c64cda275535901b16@gmail.com","Mock seamicroclient lib in unit tests if not present\n\nMock the python-seamicroclient library from within ironic/tests/drivers\nso that test_seamicro.py can import ironic/drivers/modules/seamicro.py\nand run the unit tests on it, even when the external library is not\npresent.\n\nChange-Id: I4f2cd75b2c64a5a280a5d45d5cea058fd432815b\nCloses-bug: #1312321","Devananda van der Veen",true],["72ef9158ec195ee0e0ea5284591c359987255181","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"Log exceptions from deploy and tear_down\"","Jenkins",true],["af398ee3c47cfd5a0db5f9e760d6b54872c89501","6c80f942f4383a81b501736230c68bf78e1e9f59@tenshu.net","Work around iscsiadm delete failures\n\nAttempting to delete an iSCSI target immediately after the initiator has\nlogged out of the session can sometimes cause iscsiadm's database update\nto fail.  This allows the delete command to be retried up to 5 times on\nfailure.  We also accept exit code 21 (no records found) as success.\n\nCloses-bug: #1324670\nChange-Id: I064fcd01e3bd713b0ec19cc1fdb562ca7fd8b666","Chris Jones",true],["bc1751067551842f47d77fb17ec654191033f88f","73c7819b146e140ef1b504c64cda275535901b16@gmail.com","Remove 'fake' and 'ssh' drivers from default enabled list\n\nRemove the 'fake' and 'ssh' drivers, since they should not show up in\nproduction, and it is better to have production-oriented sane defaults.\n\nUpdates the option doc string with a lengthy description.\n\nUpdates the developer documentation with a link to the wiki page,\nwhere a list of CI-tested drivers will be maintained.\n\nChange-Id: I1151ace22752efdf6a8a43e279a57d728ea6cce2","Devananda van der Veen",true],["4d4aeab4e57d623ca06bdfdceddbee2791851d03","aec3dfbec6da97399a077a99827a550fc2958d51@ubuntu.com","Update Nova driver's list_instance_uuids()\n\nlist_instance_uuid() was dependent on list_instances().  The latter\nwas fixed to return the correct results but the former was not,\ncausing nova-compute start up failures.  This updates it accordingly\nto return the expected output via a client API call.\n\nChange-Id: I27651ca01dbdcc95ca729da7dbbeafdc616eec4e\nCloses-bug: #1326988","Adam Gandelman",true],["b77063e566cba96283d6032a1063bf49b1a590f8","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"Fix concurrent deletes in virt driver\"","Jenkins",true],["90acb6e09896f9c0e8ff96a57b1eed1457d89c6e","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"PXE driver to validate the requested image in Glance\"","Jenkins",true],["656884d60b90c28981659747fd2795d1a509e4ac","f1b8b03584d630ff99ee01e1e0fe478e38733445@kth.se","Update IronicHostManager tests to better match how code works\n\nThis is the first in a series of patches to prepare Ironic for the\nremoval of capabilities in Nova HostManager.\n\nIronicHostManager used to rely on capabilities info to decide which\nhost state class to use, VM or baremetal. This has been broken since\nhost capabilities reporting were removed in Nova, resulting in Nova\nHostState class always being used.\n\nIronicBareMetal tests make use of an internal Nova HostManager variable\nin a way that hides the fact that IronicNodeState class is no longer\nused. This patch rewrites tests to better reflect how Ironic/Nova code\nnow works and prepares for further changes that needs to land in Ironic\nbefore Nova HostManager capabilities can be removed.\n\nChange-Id: I795eaed58f6587585d89366d8f1e521588866a58\nPartial-Bug: #1260265","Hans Lindgren",false]],"size":116,"ref":"refs/heads/master","head":"67adf3c8d8e59c371c9c731aedb0a05b30417b7f"}