{"push_id":1938108126,"size":18,"distinct_size":18,"ref":"refs/heads/master","head":"b7e0f6dc75d7ec43df9b945fc25c78d84e36d909","before":"41ff707efd2aab3b2dfa4bf92f859ea7b952fa93","commits":[{"sha":"9d6acfaf7cd09710accbbae4486114d1707515d0","author":{"name":"jichenjc","email":"c38141d9c6fa1d3e42353bc085e7e8bf080f5277@cn.ibm.com"},"message":"Add description on maximum placement API version\n\n1.10 is maximum version\n\nChange-Id: Icbe281d295a9d05f5c1c62848046a2c394fe31ee","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/9d6acfaf7cd09710accbbae4486114d1707515d0"},{"sha":"6787d62bf6f77dbf0fd5cbfeb3fad98298d596af","author":{"name":"Matt Riedemann","email":"f8cef0df93789985701fdabb7801907b6c4983e8@gmail.com"},"message":"Add track_instance_changes note in disable_group_policy_check_upcall\n\nIf you set disable_group_policy_check_upcall=True then you probably\nalso don't need to track instance changes, so this change provides\na link from disable_group_policy_check_upcall to track_instance_changes\nin the config option help text.\n\nChange-Id: Ide939d762f738d5d860f9a09d5b6fdd983e6b15a","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/6787d62bf6f77dbf0fd5cbfeb3fad98298d596af"},{"sha":"8ab41c83a8fdd258d8bc98bd0aaf1f5cea864677","author":{"name":"Matt Riedemann","email":"f8cef0df93789985701fdabb7801907b6c4983e8@gmail.com"},"message":"doc: code review considerations for online data migrations\n\nOnline data migrations that move something from the cell database\nto the API database typically create new things in the API DB and\non a query, look in the API DB first and then fallback to the cell\ndatabase. We started supporting multiple cells in Pike, so any online\ndata migrations that move things to the API DB after that will need\nto be multi-cell aware for the fallback lookup code.\n\nThis just adds a reminder in the code review guide.\n\nChange-Id: If0d7d9b80e336b696aaf87ec13ac18daa1068357","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/8ab41c83a8fdd258d8bc98bd0aaf1f5cea864677"},{"sha":"9e2a0163d36fd0c2152b39a714e028752d70677b","author":{"name":"Matt Riedemann","email":"f8cef0df93789985701fdabb7801907b6c4983e8@gmail.com"},"message":"Skip test_rebuild_server_in_error_state for cells v1\n\nThis test randomly fails due to a timeout in cells v1\njobs and is a latent issue. Since cells v1 is deprecated\nand we aren't fixing latent bugs, let's just skip this.\n\nChange-Id: I386df03f406dd0f1847a0d091e070df7786f616e\nRelated-Bug: #1709985","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/9e2a0163d36fd0c2152b39a714e028752d70677b"},{"sha":"c6b0d8ff5a80bf87f49124d8d9e4621d157c51e1","author":{"name":"Chris Dent","email":"b09c2ed88207e1e58f60cc200f45a56d69290315@anticdent.org"},"message":"Reset client session when placement endpoint not found\n\nIf the report client is able to access keystone to get a service\ncatalog, but that catalog does not include a placement service (because\nit hasn't been added to the catalog yet), creating resource providers\nand other placement entities will fail. This is expected.\n\nWhat's not expected is that creating entities will continue to fail for\nquite some time, even if placement is added to the catalog. This is\nbecause the keystone session caches the service catalog for some amount\nof time.\n\nTherefore we need to create a new client session when EndpointNotFound\nhappens. This has been added in this change by extracting creation of\nthe report client's _client to its own method that we can call from the\nexception handler. The resource provider and aggregate maps are wiped at\nthis time, to make sure we are starting from a clean slate. While this\nisn't likely to cause a problem in real life scenarios, in the manual\ntesting I was doing it created issues.\n\nI've made the _client method synchronized so that in the unlikely event\nthat the resource tracker is trying to do its update job while some\nother thing is happening, we won't waste the client. This may not be\nnecessary, but probably doesn't harm anything.\n\nChange-Id: I02ac615dc26a4a0d1fd28a638f622777e41d14e4\nCo-Authored-By: zhangzhenzhong <zzzhang0118@gmail.com>\nCloses-Bug: #1697825","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/c6b0d8ff5a80bf87f49124d8d9e4621d157c51e1"},{"sha":"b7675bd4b295f5df8548814d7933b74979cbba9a","author":{"name":"gaozx","email":"d08ae591923fd62fb675201157c3a841b358633c@fiberhome.com"},"message":"Fix a wrong link\n\nChange-Id: Ie238513abc1ca5f9fc89d5b32b8df2116b6c59c7","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/b7675bd4b295f5df8548814d7933b74979cbba9a"},{"sha":"ca61be287cda73ed5c43c22742d5dad8a8f5caa5","author":{"name":"Stephen Finucane","email":"5afd11d8faa38c8411f82ab9f3d25476784ef017@redhat.com"},"message":"trivial: Remove dead function, variable\n\nThese are no longer used anywhere and can be removed.\n\nChange-Id: Iad73cdf92e4aec94b70d05f2f9a823432b1dea1a","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/ca61be287cda73ed5c43c22742d5dad8a8f5caa5"},{"sha":"52d732d6bdb78ca5c940112e8dff6eef7f6828e8","author":{"name":"Lajos Katona","email":"bd03ed427f9009642037161d858a2f1ea59d8edf@ericsson.com"},"message":"Add functional force live migrate test\n\nForced live migration now seems to be failing, as after the movement of\nthe VM, the allocations are only present on the source host, and not on\nthe destination host.\nThe related assertions are commented out and replaced with wrong ones to\nmake the test pass.\n\nRelated-Bug: #1712008\nChange-Id: I6856f57426db6f2f49daea86679b50d5d019fe19","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/52d732d6bdb78ca5c940112e8dff6eef7f6828e8"},{"sha":"1ce7a136b362fbfb2d3d231ba8e1cd77081139ce","author":{"name":"Lajos Katona","email":"bd03ed427f9009642037161d858a2f1ea59d8edf@ericsson.com"},"message":"Add functional live migrate test\n\nLive migration now seems to be failing, as after the movement of the VM,\nthe allocations are both present on the source host, and on the\ndestination host.\nThe related assertions are commented out and replaced with wrong ones to\nmake the test pass.\n\nChange-Id: Ib15d34c8da14dd006a0782e325d3c5e7b83fd0f1\nRelated-Bug: #1712045","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/1ce7a136b362fbfb2d3d231ba8e1cd77081139ce"},{"sha":"145487a177e78fc087ee26d318e9ef6bfc455385","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Reset client session when placement endpoint not found\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/145487a177e78fc087ee26d318e9ef6bfc455385"},{"sha":"afa54ca49ee8ae05f906db9b1b3fede58cf9bdb4","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"trivial: Remove dead function, variable\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/afa54ca49ee8ae05f906db9b1b3fede58cf9bdb4"},{"sha":"daf4fa7456bd43d84b97bed44891b7ef2cc8279a","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Add functional force live migrate test\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/daf4fa7456bd43d84b97bed44891b7ef2cc8279a"},{"sha":"7dd94e8c0d25b32188740e36d8236895deb6d0c6","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Add functional live migrate test\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/7dd94e8c0d25b32188740e36d8236895deb6d0c6"},{"sha":"a930fe989c30f2fade5b5180cdffd804a80ed118","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Add description on maximum placement API version\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/a930fe989c30f2fade5b5180cdffd804a80ed118"},{"sha":"b6b2c353de55f90b1d33e5e8ee1898ea82108128","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Add track_instance_changes note in disable_group_policy_check_upcall\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/b6b2c353de55f90b1d33e5e8ee1898ea82108128"},{"sha":"6a70fd97492a4da00b37e6b1fc8b13ec3e665683","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"doc: code review considerations for online data migrations\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/6a70fd97492a4da00b37e6b1fc8b13ec3e665683"},{"sha":"d9db60d6039e66a9532d82c3e53ede3e2e3c2a2e","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Fix a wrong link\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/d9db60d6039e66a9532d82c3e53ede3e2e3c2a2e"},{"sha":"b7e0f6dc75d7ec43df9b945fc25c78d84e36d909","author":{"name":"Jenkins","email":"61eb39d4445155efb6068840385577c11776cd20@review.openstack.org"},"message":"Merge \"Skip test_rebuild_server_in_error_state for cells v1\"","distinct":true,"url":"https://api.github.com/repos/YuPengZTE/nova/commits/b7e0f6dc75d7ec43df9b945fc25c78d84e36d909"}]}
{"actor":{"display_login":"YuPengZTE"}}