{"push_id":1065153932,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"2a305c563073a3066aac3f07aab3c895ec2cd2fb","before":"b98f2134b246e1337761be9a7846d61117f8ad8b","commits":[{"sha":"6ed8f45fdf529bacae32b074844aa1320b005b51","author":{"email":"4a33ccdd7c661717e0b66f4b5b20deef50b546b3@infoblox.com","name":"Pavel Bondar"},"message":"Always call ipam driver on subnet update\n\nPreviously ipam driver was not called on subnet update\nif allocation_pools are not in request.\nChanging it to call ipam driver each time subnet update is requested.\nIf subnet_update is called without allocation_pools, then old allocation\npools are passed to ipam driver.\n\nContains a bit of refactoring to make that happen:\n\n- validate_allocation_pools is already called during update\nsubnet workflow in update_subnet method, so just removing it;\n\n- reworked update_db_subnet workflow;\n\nprevious workflow was:\ncall driver allocation -> make local allocation -> rollback driver\nallocation in except block if local allocation failed.\n\nnew workflow:\nmake local allocation -> call driver allocation.\nBy changing order of execution we eliminating need of rollback in this\nmethod, since failure in local allocation is rolled back by database\ntransaction rollback.\n\n- make a copy of incoming subnet dict;\n_update_subnet_allocation_pools from ipam_backend_mixin removes\n'allocation_pools' from subnet_dict, so create an unchanged copy to\npass it to ipam driver\n\nChange-Id: Ie4bd85d2ff2ab39bf803c5ef6c6ead151dbb74d7\nCloses-Bug: #1534625","distinct":true,"url":"https://api.github.com/repos/openstack/neutron/commits/6ed8f45fdf529bacae32b074844aa1320b005b51"},{"sha":"2a305c563073a3066aac3f07aab3c895ec2cd2fb","author":{"email":"d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","name":"Jenkins"},"message":"Merge \"Always call ipam driver on subnet update\"","distinct":true,"url":"https://api.github.com/repos/openstack/neutron/commits/2a305c563073a3066aac3f07aab3c895ec2cd2fb"}]}