Tuesday, 2023-10-03

vishalmanchandazigo: hello.03:54
vishalmanchandazigo: yes. I am working for NEC, Technical Lead.03:55
opendevreviewTobias Urdin proposed openstack/horizon master: Fix allowed address pair row unique ID  https://review.opendev.org/c/openstack/horizon/+/89365005:58
zigovishalmanchanda: Thanks!06:13
*** kopecmartin|off is now known as kopecmartin07:04
opendevreviewThomas Goirand proposed openstack/horizon master: Use repr to compare with objects  https://review.opendev.org/c/openstack/horizon/+/89720910:20
opendevreviewThomas Goirand proposed openstack/horizon master: WIP: Use repr to compare with objects  https://review.opendev.org/c/openstack/horizon/+/89720910:21
zigovishalmanchanda: amotoki: Your thoughs on this patch, before I finalise it (ie: fix all the pep8 thingy...)?10:22
zigoI also have another class of unit test failures like this one: https://paste.opendev.org/show/bw2T7UqOprvtqutx2cCr/10:24
zigoMore than 100 failures of this type, and I have no idea what's going on. Any clue?10:24
vishalmanchandazigo: hello, Could you please if your issue fixed by patch https://review.opendev.org/c/openstack/horizon/+/894390 ?10:48
zigoAh, that's what I thought ... it's Django 4 related. Thanks for the hint!11:46
zigovishalmanchanda: Trying the patch right away.11:47
zigovishalmanchanda: It's much nicer, I have only 10 failures remaining now ! :)12:04
zigoSome being selenium related (and I don't do Selenium tests in my package, so I need to blacklist them correctly ...).12:05
zigoI have only one single unit test failure now, after adding the patch from vish above, plus my own for the repr() calls. Here's the failure:14:07
zigohttps://paste.opendev.org/show/bC5TQnWmnd05DV9QiXut/14:07
zigoNote: this is on Django 4...14:08
opendevreviewMerged openstack/horizon master: pytest-based selenium tests add flavor create, delete tests  https://review.opendev.org/c/openstack/horizon/+/89701023:30

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