GitHub PushEvent at 2014-07-24 14:38:09
Repository: nova
Actor: citrix-openstack-build
Payload: {"shas":[["98e6891dfd4408c56644f55fe3cff88703beb4bf","3720f2c8fcdc803d022d2066ae9ca8a8c3487406@us.ibm.com","Check for resize path on libvirt instance delete\n\nIf an instance is deleted after the instance's disk image path has\nbeen renamed by adding the \"_resize\" suffix to it but before the\nresize operation completes, the libvirt driver will not delete the\norphaned files and manual intervention is needed to get them deleted.\n\nThis fix addresses the issue by attempting to rename the instance path\nby adding a \"_del\" suffix and if that fails, renaming the instance path\nwith the \"_resize\" suffix by replacing the \"_resize\" suffix with the\n\"_del\" suffix.  If both renaming operations fail, the sequence is\nrepeated, in case the the disk image path initially had the \"_resize\"\nsuffix and another thread removed it before the second rename operation\nwas attempted. These rename operations are used in favor of checking\nfor the existence of paths and deleting if found, because rename\noperations are atomic whereas another thread could rename the path\nbetween the exist check and the deleting.\n\nRegardless of the outcome of the renaming operations, the existence of\nthe instance path with the \"_del\" suffix is verified and if it exists,\nit is deleted. This is done in case a prior delete operation that\nmanaged to create the \"_del\" path was subsequently interrupted before\nall instance files could be deleted.\n\nNote that the LibvirtConnTestCase.test_delete_instance_files test case\nwas removed in order to eliminate redundancy.\n\nCloses-Bug: #1308565\n\nChange-Id: Ifcb2e18211347ccf3e5472779c5917a729a6eced","John Warren",false],["8159e1daa21c315dcd10cd77164d3c364ee93047","aa9ace35aa8d43ab8690cdf7ea0089ea1df2866d@stillhq.com","Convert to importutils\n\nWe had some remaining users of __import__ that really deserved to\nbe converted to oslo's importutils.\n\nChange-Id: I8c0c3e885151018b974025087a4abc96dd6ae77c","Michael Still",true],["def40cfd88dd1f64559b0328a886c507cd310027","1c287fdb24a4ea18166b6bfee18147b57c1619f7@gmail.com","Raise NotImplemented default-security-group-rule api with neutron\n\nNeutron currently does not implement the nova extension\ndefault-security-group-rule which allows an admin to set which rules are in\nthe default security group when a tenant is onboarded. Implementing this\ntype of functionality should be done in neutron directly as proxying the\napi calls to do this from nova is not possible as neutron creates the\ndefault security group itself. Previously, if one tried to use this api\nwith neutron a 500 error was raised, now a 501 error is raised.\n\nChange-Id: I6d1bb1c9c2d79278a85b96aea800b4b1f3489225\nCloses-bug: 1326958","Aaron Rosen",false],["89e1dd73b8de4628908c10a6cfbbd79adf6e61be","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"Raise NotImplemented default-security-group-rule api with neutron\"","Jenkins",false],["c564d75719cf01dbff36956a8855ba2b4751b86e","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"Check for resize path on libvirt instance delete\"","Jenkins",false],["d810cbc2094e64c143ca10c0e804bbed7cad8ef0","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"Convert to importutils\"","Jenkins",true]],"size":6,"ref":"refs/heads/trunk-nova-network-full-latest","head":"d810cbc2094e64c143ca10c0e804bbed7cad8ef0"}