{"push_id":629883926,"size":339,"distinct_size":189,"ref":"refs/heads/metadata-use-objects","head":"735a33351d6e98a012f01ff370db553933447a7e","before":"3d99dc19752fc1e0f3ee678cf1b932e5caaf9f01","commits":[{"sha":"62fc5682ec869cf7857b0a29db378886a8de169f","author":{"email":"0fe0bd399dd39b6e2574442cf896a57f5611d8c7@intel.com","name":"He Jie Xu"},"message":"Populates retry info when unshelve offloaded instance\n\nThis patch populates retry info into filter_properties when unshelve\noffloaded instance. This is used to support rescheduling for unshelve\ninstance.\n\nChange-Id: I111e9acf956f84ede9596bda5e5077feb559cda2\nPartial-Bug: #1400015","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/62fc5682ec869cf7857b0a29db378886a8de169f"},{"sha":"3a915e37dc70992291635788979978f47a5e298f","author":{"email":"78c2c705d32f41ddd0dddd01cc56e39491384260@cn.ibm.com","name":"jichenjc"},"message":"consoliate set_availability_zones usage\n\nset_availability_zones in availability_zone.py is used\nat several place, actually, the param of object allows\nit to be set inside object function. So switch to\nobject with param input to clean unnessary code.\n\nChange-Id: I3fd8a661677d6bfee513512d2315e8daeb57d422","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/3a915e37dc70992291635788979978f47a5e298f"},{"sha":"d60453dd1ba06a38a625f46c17bb55b50b207faa","author":{"email":"7ad66e2a4a1be3b108eb756e888688c0d085c066@mirantis.com","name":"Sergey Nikitin"},"message":"Improved performance of db method network_in_use_on_host\n\nMethod network_in_use_on_host checks whether a number of\nfixed IPs in network is greater than 0. For that this method\ntakes a list of fixed IPs for network and checks the length\nof this list. Actually we don't need to get a list of\nfixed IPs with useless information. We can just get count of\nthese IPs using query.count() method.\n\nIt will improve performance because we won't collect\nuseless information about fixed IPs by db query.\n\nWe don't need to cover this change by tests because\nit's already covered by tests from test_db_api.\n\nChange-Id: I2b8222566471a3c22c4c6c919f4c80a9917e32f5","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/d60453dd1ba06a38a625f46c17bb55b50b207faa"},{"sha":"fde77d49ff550b73f5f1671edc7366c9b7646200","author":{"email":"fd0d713a38ab297d4f0f590081fce670756218c1@nttdata.com","name":"Rajesh Tailor"},"message":"Delete instance files from dest host in revert-resize\n\nWhen revert-resize call is finished, it doesn't clear instance\nfiles from destination node for non-shared instance storage.\n\nThe driver call, which is responsible for destroying destination\nhost instance is not deleting instance files, because it finds that\nboth source and destination host shares the same instance storage.\n\nThe check_instance_shared_storage rpc-call had no host information,\nhence the manager call for check_instance_shared_storage was checking\ninstance storage on destination host itself and returning True even\nif it is non-shared storage.\n\nAdded host parameter to check_instance_shared_storage rpc-call,\nso that in manager call check_instance_shared_storage method get\nexecuted on source compute node and returns True/False based on\nshared/non-shared storage.\n\nCloses-Bug: #1418855\nChange-Id: Ic529b1c2c1cfd914facb14941e4bb641db6a0e82","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/fde77d49ff550b73f5f1671edc7366c9b7646200"},{"sha":"4e7fc7ac56f861b9fe2a535dcafc5b84b249ae24","author":{"email":"1037709ca3f8170c934c8d5b4861ade50f083c70@cn.ibm.com","name":"Jay Lau"},"message":"Enable retry when there are multiple force hosts/nodes\n\nCurrently, when there are forced hosts/nodes in a scheduler request,\nretry will be disabled. This is not right as there might be multiple\nforced hosts/nodes, we should not disable retry for such case.\n\nCloses-Bug: #1343946\n\nDocImpact\n1) If there is only one force host/node, then retry filter will not\ntake effect for such case.\n2) If there are multiple force hosts/nodes, retry filter will help\nretry on the force hosts/nodes if there are VM boot failure.\n\nChange-Id: I6831cab70a70ebbb9609689d9132c731ff84c665","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/4e7fc7ac56f861b9fe2a535dcafc5b84b249ae24"},{"sha":"26424b883e3cbd4c1abc1738b417eb6823b0e3cd","author":{"email":"2c1c04960e5094d519e914b7f827193df42cff08@us.ibm.com","name":"Matt Riedemann"},"message":"Better power_state logging in _sync_instance_power_state\n\nCommit aa1792eb4c1d10e9a192142ce7e20d37871d916a added more verbose\nlogging of the various database and hypervisor states when\n_sync_instance_power_state is called (which can be called from\nhandle_lifecycle_event - triggered by the libvirt driver, or from the\n_sync_power_states periodic task).\n\nThis change adds logging of when the power_state in the database does\nnot match the power_state from the hypervisor, and then the power_state\nin the database is updated to match what's in the hypervisor.\n\nAlso, we save the original database power_state for logging when we have\ncases of stopping an active instance due to vm_state / power_state\nconflicts, or stopping a \"stopped\" instance but the hypervisor says is\nstill running.\n\nThis is all needed because debugging the various things that can hit\nthis code at different times and cause problems is, well, terrible.\n\nChange-Id: I297ca0037f43535c80afa8a4b5086ccb30558dc3","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/26424b883e3cbd4c1abc1738b417eb6823b0e3cd"},{"sha":"202a777ae097d1374e81a3c53b34de685746df2b","author":{"email":"764f66f25875b5a3a6385ec265e50d4631e9dec9@dektech.com.au","name":"Trung Trinh"},"message":"Avoid KeyError Exception in extract_flavor()\n\nIn the function extract_flavor(), if the returned sys_meta\nof the instance is empty dict or does not exist then the function\nis returned to avoid KeyError Exception.\n\nChange-Id: Ic996cc0ca0303772c4e6a8bbb9f4165ddcdfafff\nCloses-Bug: #1226083","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/202a777ae097d1374e81a3c53b34de685746df2b"},{"sha":"21389a315e444e9bbdc9beb7c6f28f237c98b282","author":{"email":"98ca971a0b6218a1f1ebb8064a831cbef3e791f7@huawei.com","name":"shihanzhang"},"message":"Fix 'nova show' return incorrect mac info\n\nthis patch fixes 'nova show' return incorrect mac info when a VM has\nmultiple VIFs.\n\nChange-Id: I08f7472044365a8af07d391b4762a97ed233c86e\nCloses-Bug: #1428429","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/21389a315e444e9bbdc9beb7c6f28f237c98b282"},{"sha":"7560620da7704671e6e5a304c25e506e12f213bd","author":{"email":"2c1c04960e5094d519e914b7f827193df42cff08@us.ibm.com","name":"Matt Riedemann"},"message":"neutronv2: only create client once when adding/removing fixed IPs\n\nWe don't need to create a neutron client object (session) every time we\nneed to make an API call in add_fixed_ip_to_instance or\nremove_fixed_ip_from_instance, especially when looping over ports and\nsubnets.\n\nJust create the client once at the beginning of the method and reuse it.\n\nCloses-Bug: #1430481\n\nChange-Id: If9181c2ef5fb94652d0205d6d69521bcc5d85cff","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/7560620da7704671e6e5a304c25e506e12f213bd"},{"sha":"56a6c6485d197eb15c3bdf421c5580c6dc29f816","author":{"email":"84df40ec8e2270eb6994a3e4818f2872dd203c26@cloudbasesolutions.com","name":"Adelina Tuvenie"},"message":"Fix copy configdrive during live-migration on HyperV\n\nThis bug was introduced due to an oversight when fixing\nhttps://launchpad.net/bugs/1322096 . In that fix, we use the\n\"config_drive_cdrom\" value from the conf to determine if a\nthere is a iso configdrive attached to the instance. This\napproach is wrong due to the fact that that config value can be\nmodified by the user. If that happens, instances that still have\niso configdrives will be ommitted.\n\nThis path fixes this problem by checking the instance itself if\nit has any iso attached to it.\n\nCloses-Bug: #1427467\nChange-Id: Ia64389b4405109d7db6fbf9c000a0136e9862298","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/56a6c6485d197eb15c3bdf421c5580c6dc29f816"},{"sha":"e91e7c1fd31c346e6272c5229d66cc1639b0fed4","author":{"email":"c740d364ac25267f4f40e96683125121edc170d1@nttdata.com","name":"PranaliDeore"},"message":"Fix typo in _live_migration_cleanup_flags method\n\nFixed typo intance -> instance in _live_migration_cleanup_flags()\nmethod of nova.compute.manager\n\nCloses-Bug: #1431193\nChange-Id: Ifebfeb6b6803f6566dc78670c800a18bf0ff0aef","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/e91e7c1fd31c346e6272c5229d66cc1639b0fed4"},{"sha":"bc68acca27c8c7512e02b5753bc3d44a9e4ca1df","author":{"email":"5d8d18748f16662d1b18967be65277582fc05902@redhat.com","name":"Dan Smith"},"message":"Fix backporting objects with sub-objects that can look falsey\n\nList objects can appear falsey if they have no elements. So, when we backport,\nwe need to explicitly check for Noneness instead of falsiness.\n\nChange-Id: I8457889f9725c77efaa7ecbe03d62b7c593eeac4","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/bc68acca27c8c7512e02b5753bc3d44a9e4ca1df"},{"sha":"deb7a2402fc29294391616168ae7a074ef9ccfb9","author":{"email":"8f24c48ff5013d8049d27d24d81ebe29387e2a7a@hp.com","name":"Alexis Lee"},"message":"Move InstanceClaimTestCase.test_claim_and_audit\n\nThe resource tracker is being converted to use objects. This patch\nremoves InstanceClaimTestCase.test_claim_and_audit from\ntest_resource_tracker.py and adds equivalent tests to test_tracker.py.\nBy doing it this way, it can be seen that the old and new tests both\npass in the gate before the resource tracker is converted to use\nmigration objects.\n\nPartially implements blueprint make-resource-tracker-use-objects\n\nChange-Id: I7682fe86ebdecca10abad0f816e8b0c93cb498c5","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/deb7a2402fc29294391616168ae7a074ef9ccfb9"},{"sha":"f69113d93ae6ede1eb089a3c792a51f8ab037487","author":{"email":"8f24c48ff5013d8049d27d24d81ebe29387e2a7a@hp.com","name":"Alexis Lee"},"message":"Update some ResizeClaimTestCase tests\n\nThe resource tracker is being converted to use objects. This patch\nremoves some tests from ResizeClaimTestCase in test_resource_tracker.py\nand adds equivalent tests to test_tracker.py. By doing it this way, it\ncan be seen that the old and new tests both pass in the gate before the\nresource tracker is converted to use migration objects.\n\nPartially implements blueprint make-resource-tracker-use-objects\n\nChange-Id: I464e1180b5c3e0d876d40318df334b4e07005a42","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/f69113d93ae6ede1eb089a3c792a51f8ab037487"},{"sha":"48b499d3a7eddd6af88580ff1573f942dfda05b4","author":{"email":"aa71f94694e581551b4f662abcc23a9271bdb41d@vmware.com","name":"Gary Kotton"},"message":"libvirt: remove unnecessary flavor parameter\n\nThe instance now has the flavor as part of the object. There is\nno reason to pass this to:\n - _get_guest_config_meta\n - _get_guest_xml\n\nThe unit test test_get_guest_config was also updated. The flavor in\nthe test has 3 disks and these need to be validated correctly.\n\nChange-Id: I1170aefcd4912e091f4b85e91d351b4372522fa9","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/48b499d3a7eddd6af88580ff1573f942dfda05b4"},{"sha":"6ca7d598e3fd56784c02af6be7fc6d69c9b63b26","author":{"email":"b7a5da1e6209f9510c07a03f0423c37727add225@redhat.com","name":"Matthew Booth"},"message":"Fix PciDeviceDBApiTestCase with referential constraint checking\n\nPciDeviceDBApiTestCase relies on being able to create database rows\nwithout referential constraint checking, which is the default on\nsqlite but not other backends. This change adds the required compute\nnode row when creating pci device rows.\n\nChange-Id: I5c31f75d612946237a939acd86c143d895580fda","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/6ca7d598e3fd56784c02af6be7fc6d69c9b63b26"},{"sha":"8aa429897c640434d974d90ea002d7fe8217519d","author":{"email":"ac84f775b4e581f8fe1f02661f66a69d7356479c@easystack.cn","name":"ChangBo Guo(gcb)"},"message":"libvirt: Remove minidom usage in driver.py\n\nWe usually use etree to parse/handle xml in driver, and etree\nsupports XPath to find specific node, so we don't need find\nthe node with comparing its tag and attribute. This commit\nremoved the minidom usage and uses Xpath in method get_vnc_console\nand get_spice_console.\n\nNote: Unit tests had covered this change in\nunit/virt/libvirt/test_driver.py\n\nPartial-Bug: #1407915\nChange-Id: Idd15d2b6c5436cf80520b76ee4025acfe536ec8e","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/8aa429897c640434d974d90ea002d7fe8217519d"},{"sha":"b749bca5d13f07ffb8af210022ca153cfcd623a5","author":{"email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@dague.net","name":"Sean Dague"},"message":"remove unneeded teardown related code\n\nThe DatabaseFixture ensures we have a blank database at the beginning\nof a test, previously there was a teardown method here, which was\ndeleted. However there remains a bunch of state tracking which was\nrelated to that teardown. Cleanup up all the extra variable references\nto avoid future confusion with this test.\n\nChange-Id: I5527413dfcc6b9e0691bfc229247e7707b75cb45","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/b749bca5d13f07ffb8af210022ca153cfcd623a5"},{"sha":"32c10850ae733c524e403592771b02491a96e67d","author":{"email":"5c7e7a1b9dd68895553ea361dd09b55647837ea3@gmail.com","name":"Eduardo Costa"},"message":"Workaround for race condition in libvirt\n\nMethod \"info\" is affected by a race condition in libvirt.\nMethods \"vcpus\" and \"memoryStats\" are also affected, but their\nerrors are handled gracefully by present code.\n\nThis patch detects the race and then performs a retry on error.\n\nNew code should avoid calling libvirt.virDomain.info directly.\nInstead, method Host.get_domain_info() should be called.\n\nA full fix for this bug would involve upgrading libvirt to\nversion >=1.2.11.\n\nChange-Id: I4e3baceabdbc843635d07652abf80d39d58d9e06\nPartial-Bug: #1372670","distinct":false,"url":"https://api.github.com/repos/hanlind/nova/commits/32c10850ae733c524e403592771b02491a96e67d"},{"sha":"39c0ce46b6d9bd93c2efabdb722dafac62b1f0f6","author":{"email":"80f3d9c8f10b600169ff46547616e6b5b34bf9fd@intel.com","name":"ShaoHe Feng"},"message":"add neutron api NotImplemented test cases for Network V2.1\n\nNow these test cases are tested twice, and just for V2.\nThis patch fixes it.\n\nAnd it blocks the bp v3-api-policy implementation.\n\nCloses-Bug: 1429729\n\nChange-Id: Ia2eb281a9ba2e02c0720f99d01971623c7664063","distinct":true,"url":"https://api.github.com/repos/hanlind/nova/commits/39c0ce46b6d9bd93c2efabdb722dafac62b1f0f6"}]}