| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 03:28 |
|---|---|---|
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 04:06 |
| opendevreview | Merged openstack/horizon stable/victoria: Get ports directly instead of via loop https://review.opendev.org/c/openstack/horizon/+/866894 | 05:08 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 05:35 |
| *** gthiemon1e is now known as gthiemonge | 06:07 | |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 07:11 |
| vishalmanchanda | rdopiera: hello, morning chromedriver and chrome is installed in the patch now https://review.opendev.org/c/openstack/horizon/+/886867 | 08:32 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 08:37 |
| gthiemonge | Hey Folks, the latest chrome release breaks the octavia-dashboard (see https://bugs.launchpad.net/octavia/+bug/2024188) I proposed a patch that fixes this issue: https://review.opendev.org/c/openstack/octavia-dashboard/+/886255 | 08:42 |
| gthiemonge | I'm not a dashboard/horizon expert, could you tell me if this patch makes sense? | 08:42 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 09:52 |
| rdopiera | vishalmanchanda: looks like the code that retrieves the browser.log wasn't used for so long, that it's broken now too | 10:02 |
| vishalmanchanda | rdopiera: ohh, so no benefit of running integration job with chrome? | 10:05 |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 10:10 |
| rdopiera | vishalmanchanda: it's a simple fix, I just added it | 10:11 |
| rdopiera | basically b'' instead of '' | 10:12 |
| vishalmanchanda | rdopiera: ok, let's see. | 10:15 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 11:55 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 12:01 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 12:04 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 13:14 |
| opendevreview | Thomas Goirand proposed openstack/horizon master: Django 4: fix CSRF reasons https://review.opendev.org/c/openstack/horizon/+/885578 | 13:58 |
| vishalmanchanda | rdopiera: ``browser.log``, you wants to check but I can't see any error msg here https://zuul.opendev.org/t/openstack/build/5f63e9ae7edb4e74a0de4d3e13258ee9/log/tox/test_reports/TestFloatingipAssociateDisassociate.test_floatingip_associate_disassociate/browser.log | 15:26 |
| rdopiera | vishalmanchanda: yeah, that failure is weird, the video looks like it timed out waiting for the instance to finish launching... | 15:41 |
| vishalmanchanda | rdopiera: the same test passed with firefox job | 15:42 |
| rdopiera | vishalmanchanda: ah, the video is from the firefox job | 15:43 |
| rdopiera | it looks like both jobs put the files in the same directory and overwrite them | 15:43 |
| vishalmanchanda | ohh then we should change the directory. | 15:44 |
| rdopiera | it's weird, because there should be different directories for different tests | 15:54 |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: [DNM] Test chromedriver with integartion job https://review.opendev.org/c/openstack/horizon/+/886867 | 15:57 |
| rdopiera | there was one more error with encoding the browser log | 15:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!