Friday, 2020-08-21

« Thursday, 2020-08-20 Index Saturday, 2020-08-22 »
*** evrardjp has quit IRC00:05
*** evrardjp has joined #openstack-glance00:06
*** Liang__ has joined #openstack-glance01:26
*** gyee has quit IRC01:55
*** rcernin has quit IRC02:37
*** rcernin has joined #openstack-glance02:58
*** rcernin has quit IRC03:04
*** rcernin has joined #openstack-glance03:04
*** evrardjp_ has joined #openstack-glance03:12
*** evrardjp has quit IRC03:15
*** Liang__ has quit IRC04:24
*** Liang__ has joined #openstack-glance04:24
*** evrardjp_ has quit IRC04:33
*** evrardjp has joined #openstack-glance04:35
*** udesale has joined #openstack-glance04:58
*** ratailor has joined #openstack-glance05:13
*** Liang__ has quit IRC05:56
*** Liang__ has joined #openstack-glance05:58
*** jdillaman has quit IRC06:38
*** jdillaman has joined #openstack-glance06:43
openstackgerritMerged openstack/glance_store stable/ussuri: zuul: glance_store-src-ceph-tempest replaces a legacy job  https://review.opendev.org/74414806:45
*** belmoreira has joined #openstack-glance06:57
openstackgerritMerged openstack/python-glanceclient master: Fix hacking min version to 3.0.1  https://review.opendev.org/72753307:04
*** happyhemant has joined #openstack-glance07:21
*** rcernin has quit IRC07:22
*** priteau has joined #openstack-glance07:40
*** udesale has quit IRC07:50
*** rcernin has joined #openstack-glance08:08
*** rcernin has quit IRC08:13
*** udesale has joined #openstack-glance08:22
*** lpetrut has joined #openstack-glance08:22
*** k_mouza has joined #openstack-glance08:37
*** openstackgerrit has quit IRC08:54
*** rcernin has joined #openstack-glance09:07
*** openstackgerrit has joined #openstack-glance09:08
openstackgerritMerged openstack/glance_store master: Ramp up rbd resize to avoid excessive calls  https://review.opendev.org/74657909:08
*** k_mouza has quit IRC09:27
*** ratailor has quit IRC09:38
*** k_mouza has joined #openstack-glance09:42
*** baojg has joined #openstack-glance09:43
*** baojg has quit IRC09:59
*** belmoreira has quit IRC10:07
*** rcernin has quit IRC10:13
*** k_mouza has quit IRC10:31
*** k_mouza has joined #openstack-glance10:42
*** Liang__ has quit IRC10:48
*** rcernin has joined #openstack-glance10:58
*** rcernin has quit IRC11:12
*** k_mouza has quit IRC11:34
*** k_mouza has joined #openstack-glance12:06
*** k_mouza has quit IRC12:21
*** k_mouza has joined #openstack-glance12:21
*** k_mouza has quit IRC12:26
*** baojg has joined #openstack-glance12:49
*** udesale_ has joined #openstack-glance12:59
*** udesale has quit IRC13:02
*** happyhemant has quit IRC13:10
*** k_mouza has joined #openstack-glance13:19
openstackgerritDan Smith proposed openstack/glance master: Make our ceph job enable thin provisioning  https://review.opendev.org/74717813:34
*** baojg has quit IRC13:48
*** baojg has joined #openstack-glance13:49
openstackgerritDan Smith proposed openstack/glance_store master: Handle sparse images in glance_store  https://review.opendev.org/74428214:03
dansmithabhishekk: ^ rebased and fixed conflicts after the rbd resize patch merged, hopefully we can get a run of the thin job on it now14:04
*** lpetrut has quit IRC14:15
abhishekkdansmith, ack14:18
abhishekkI guess something is wrong with race-condition patch chain as well14:18
dansmithoh?14:21
dansmithah yeah14:21
dansmithguh14:21
abhishekkyeah14:22
dansmithit's been sitting a while, so things have drifted underneath14:22
abhishekkhmm14:23
abhishekkalso https://review.opendev.org/#/c/747178 is in merged conflict14:26
dansmithI think it's not, it's running in the check queue, zuul will update that after it's done14:27
*** baojg has quit IRC14:28
dansmithbtw, I saw another TIMED_OUT failure yesterday, but it's something different than the one I fixed, because test_reload had already run14:28
*** baojg has joined #openstack-glance14:29
openstackgerritDan Smith proposed openstack/glance master: Implement time-limited import locking  https://review.opendev.org/74359714:30
openstackgerritDan Smith proposed openstack/glance master: Add FakeData generator test utility  https://review.opendev.org/74674614:30
openstackgerritDan Smith proposed openstack/glance master: Add functional test for task status updating  https://review.opendev.org/74539214:30
openstackgerritDan Smith proposed openstack/glance master: Move SynchronousAPIBase to a generalized location  https://review.opendev.org/74556614:30
openstackgerritDan Smith proposed openstack/glance master: Handle atomic image properties separately  https://review.opendev.org/74651814:30
openstackgerritDan Smith proposed openstack/glance master: Functional test enhancement for lock busting  https://review.opendev.org/74653114:30
openstackgerritDan Smith proposed openstack/glance master: Add ImageLock to base flow checks  https://review.opendev.org/74730514:30
openstackgerritDan Smith proposed openstack/glance master: Cleanup import status information after busting a lock  https://review.opendev.org/74655414:30
dansmithlost all the +2s on that base patch because it needed an actual fix due to the merge conflict :/14:31
abhishekkrosmaita, ^^14:31
abhishekksean is not around14:31
*** baojg has quit IRC14:31
*** baojg has joined #openstack-glance14:32
*** baojg has quit IRC14:32
*** baojg has joined #openstack-glance14:34
rosmaitaabhishekk: that's https://review.opendev.org/#/c/743597/19 ?14:41
abhishekkyes14:41
dansmithI guess I could have collapsed that test follow up patch since I had to rebase14:42
dansmithif you'd prefer, let me know14:42
rosmaitano, that's fine to do it separately14:42
rosmaitathanks for doing it14:42
dansmithnp14:42
rosmaita(sorry, i am in a meeting)14:55
abhishekkno14:56
abhishekknp14:56
rosmaitaok, i +2'd the base patch -- do we have a consensus to +A?14:56
abhishekkwaiting for tests(zuul)  to finish14:57
*** baojg has quit IRC14:58
*** baojg has joined #openstack-glance14:58
* abhishekk going for dinner14:59
openstackgerritDan Smith proposed openstack/glance master: Add FakeData generator test utility  https://review.opendev.org/74674615:18
openstackgerritDan Smith proposed openstack/glance master: Add functional test for task status updating  https://review.opendev.org/74539215:18
openstackgerritDan Smith proposed openstack/glance master: Move SynchronousAPIBase to a generalized location  https://review.opendev.org/74556615:18
openstackgerritDan Smith proposed openstack/glance master: Handle atomic image properties separately  https://review.opendev.org/74651815:18
openstackgerritDan Smith proposed openstack/glance master: Functional test enhancement for lock busting  https://review.opendev.org/74653115:18
openstackgerritDan Smith proposed openstack/glance master: Add ImageLock to base flow checks  https://review.opendev.org/74730515:18
openstackgerritDan Smith proposed openstack/glance master: Cleanup import status information after busting a lock  https://review.opendev.org/74655415:18
*** tkajinam has quit IRC15:26
*** gyee has joined #openstack-glance15:34
*** udesale_ has quit IRC15:51
*** baojg has quit IRC16:01
*** baojg has joined #openstack-glance16:02
*** baojg has quit IRC16:10
*** baojg has joined #openstack-glance16:12
*** k_mouza has quit IRC16:30
*** priteau has quit IRC17:07
abhishekkI lost network for past hour and still struggling to setup hotspot due to less speed17:10
*** k_mouza has joined #openstack-glance17:17
abhishekkdansmith, around?19:03
* abhishekk signing out for the day19:17
*** yoctozepto1 has joined #openstack-glance19:30
*** yoctozepto has quit IRC19:32
*** yoctozepto1 is now known as yoctozepto19:32
*** gyee has quit IRC19:32
*** jv_ has quit IRC19:32
*** jv_ has joined #openstack-glance19:33
*** gyee has joined #openstack-glance19:36
dansmithsorry, was eating19:39
rosmaitadansmith: problem on https://review.opendev.org/#/c/743597/1919:44
dansmithrosmaita: I'm replying.. that test is super racy, I've fixed two races on it in the last month :)19:44
rosmaitafun times19:45
dansmithbefore this patch, there wasn't a good way for it to know when it's safe to start the next import, but since there was no locking, it could do so before the first one finished19:46
dansmitha better way would be to use the os_glance_import_task field to get the task-id and poll on that, but I can't do that until this patch, and I hate to refactor that test *in* this patch along with everything else19:47
rosmaitadansmith: i would be ok with a recheck or maybe even a skip-test until you can poll the task-id19:50
rosmaitai agree that you don't want to add more changes to this patch19:51
dansmithI'mm a put a retry-on-409 below it to prove it doesn't break anything, and hopefully also prevent the lock patch and descendants from hitting this afterwards19:51
rosmaitaok19:52
dansmithsince we just landed one of the fixes to that test like yesterday, the timing characteristics of this test have changed19:53
dansmithwe hadn't previously seen it fail here19:53
openstackgerritDan Smith proposed openstack/glance master: Implement time-limited import locking  https://review.opendev.org/74359719:53
openstackgerritDan Smith proposed openstack/glance master: Add FakeData generator test utility  https://review.opendev.org/74674619:53
openstackgerritDan Smith proposed openstack/glance master: Add functional test for task status updating  https://review.opendev.org/74539219:53
openstackgerritDan Smith proposed openstack/glance master: Move SynchronousAPIBase to a generalized location  https://review.opendev.org/74556619:53
openstackgerritDan Smith proposed openstack/glance master: Handle atomic image properties separately  https://review.opendev.org/74651819:53
openstackgerritDan Smith proposed openstack/glance master: Functional test enhancement for lock busting  https://review.opendev.org/74653119:53
openstackgerritDan Smith proposed openstack/glance master: Add ImageLock to base flow checks  https://review.opendev.org/74730519:53
openstackgerritDan Smith proposed openstack/glance master: Cleanup import status information after busting a lock  https://review.opendev.org/74655419:53
openstackgerritDan Smith proposed openstack/glance master: Make test_copy_image_revert_lifecycle handle 409 on import retry  https://review.opendev.org/74747019:53
*** k_mouza has quit IRC22:11
« Thursday, 2020-08-20 Index Saturday, 2020-08-22 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!