opendevreview | Vishal Manchanda proposed openstack/horizon master: Rename x/xstatic-angular-fileupload->openstack/xstatic-angular-fileupload https://review.opendev.org/c/openstack/horizon/+/883995 | 03:54 |
---|---|---|
opendevreview | Vishal Manchanda proposed openstack/horizon master: Rename x/xstatic-angular-fileupload->openstack/xstatic-angular-fileupload https://review.opendev.org/c/openstack/horizon/+/883995 | 03:55 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/zed: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883918 | 04:02 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/yoga: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883919 | 04:03 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/xena: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883920 | 04:03 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/wallaby: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883921 | 04:03 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/victoria: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883922 | 04:03 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/ussuri: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883923 | 04:04 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/ussuri: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883923 | 04:11 |
opendevreview | Vishal Manchanda proposed openstack/horizon stable/ussuri: Remove usage of openstack/xstatic-font-awesome https://review.opendev.org/c/openstack/horizon/+/883923 | 04:36 |
opendevreview | Thomas Goirand proposed openstack/horizon master: WIP: Add TOTP support https://review.opendev.org/c/openstack/horizon/+/884184 | 12:22 |
vishalmanchanda | #startmeeting horizon | 15:00 |
opendevmeet | Meeting started Wed May 24 15:00:10 2023 UTC and is due to finish in 60 minutes. The chair is vishalmanchanda. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'horizon' | 15:00 |
rdopiera | o/ | 15:00 |
vishalmanchanda | Hello everyone | 15:00 |
tmazur | o/ | 15:01 |
amotoki | hi | 15:02 |
vishalmanchanda | let's start the meeting | 15:02 |
vishalmanchanda | agenda of meeting can be found here https://etherpad.opendev.org/p/horizon-release-priorities#L39 | 15:02 |
vishalmanchanda | I have no announcements for this week, so moving to release priorities topic. | 15:03 |
vishalmanchanda | #topic Release priorities | 15:03 |
vishalmanchanda | Update-jQuery | 15:04 |
vishalmanchanda | I have pushed a patch to bump XStatic-JQuery-Migrate version from 1.2.1.1->3.3.2.1 and horizon-integration job also pass. | 15:04 |
vishalmanchanda | Please take a look https://review.opendev.org/c/openstack/horizon/+/883660 | 15:05 |
vishalmanchanda | The next step would be updating the XStatic-jQuery version from 1.12.4.1->3.5.1.1 in horizon CI jobs. | 15:05 |
vishalmanchanda | I hope it doesn't broke the CI jobs. | 15:07 |
vishalmanchanda | specifically horizon-integration job. | 15:07 |
vishalmanchanda | moving to next thing | 15:08 |
amotoki | the failure rate of the integration job is already super high, so I don't think we can evaluate whether the version bump affects it :-( | 15:08 |
vishalmanchanda | Setuptools issue https://bugs.launchpad.net/horizon/+bug/2007574 | 15:08 |
vishalmanchanda | amotoki: yeah, but I am testing it manually for all the panels but few things I may miss in my testing. | 15:09 |
amotoki | vishalmanchanda: the failure rate of the integration job is another topic. perhaps we need to discuss it separately | 15:10 |
vishalmanchanda | amotoki: yes, I was saying it didn't breaks due to changes I am doing for jQuery update. | 15:11 |
vishalmanchanda | For setuptool, patch to use latest version of XStatic =1.0.3 is merged in both openstack/requirements and openstack/horizon. | 15:11 |
vishalmanchanda | Now it is using XStatic =1.0.3, see https://zuul.opendev.org/t/openstack/build/730cdebb1a864d6289d0afca99d6e39e/log/kolla/build/horizon.log#145 | 15:12 |
vishalmanchanda | but still getting the same error https://zuul.opendev.org/t/openstack/build/730cdebb1a864d6289d0afca99d6e39e/log/kolla/build/horizon.log#1883 | 15:12 |
vishalmanchanda | rdopiera: did you get the time to look into it? | 15:13 |
rdopiera | vishalmanchanda: no | 15:14 |
rdopiera | vishalmanchanda: I suppose that means we have to make that change to all xstatic packages... | 15:14 |
rdopiera | but before we do that, I need to test it | 15:14 |
rdopiera | on some fake packages | 15:14 |
vishalmanchanda | rdopiera: yes, we start with XStatic-Moment-Timezone pypi | 15:16 |
vishalmanchanda | that's all update from my side on this topic | 15:18 |
vishalmanchanda | moving to next topic | 15:19 |
vishalmanchanda | #topic open-discussion | 15:19 |
vishalmanchanda | I have pushed a patch to rename x/xstatic-angular-fileupload->openstack/xstatic-angular-fileupload but getting some errors. | 15:20 |
vishalmanchanda | https://review.opendev.org/c/openstack/horizon/+/883995 | 15:20 |
vishalmanchanda | Does anyone have any idea how to fix it? | 15:21 |
amotoki | we hit the same issue on xstatic-fond-awesome too | 15:21 |
vishalmanchanda | I think this series of patch needs to be merged till stable/ussuri branch to fix it https://review.opendev.org/q/I9c75b084705dfdc6df52aca1659f89444c8a1728 | 15:22 |
vishalmanchanda | amotoki: yes. | 15:22 |
amotoki | I think it is better to ask it to the opendev infra team | 15:22 |
vishalmanchanda | amotoki: ok, I will check with them | 15:22 |
amotoki | stable/ussuri syntax error is just what we see | 15:22 |
amotoki | it is just because ussuri branch zuul config is checked first | 15:22 |
vishalmanchanda | ohh. | 15:23 |
amotoki | I think we will hit the same error one branch by one | 15:23 |
amotoki | but it is just my guess, so let's discuss it with the TACT SIG member | 15:24 |
vishalmanchanda | yeah, we have to rename x/xstatic-angular-fileupload and remove xstatic-font-awesome in a single patch in stable branches, but anyway I will check with them | 15:24 |
vishalmanchanda | We have 1 new bug reported this week https://bugs.launchpad.net/horizon/+bug/2020630 | 15:26 |
vishalmanchanda | I haven't looked at the bug yet, If anyone get some time please take a look. | 15:26 |
tmazur | I will take a look. Might be connected to the angular upgrade. | 15:27 |
vishalmanchanda | tmazur: thanks. | 15:28 |
vishalmanchanda | Does anyone setup keystone Federation env. on devstack and provide me some references? | 15:29 |
amotoki | as I mentioned above, the failure rate of the integration job is super high https://grafana.opendev.org/d/260356fb6c/horizon-failure-rate?orgId=1&from=1684335968345&to=1684940768345&viewPanel=4 | 15:29 |
amotoki | does it happen for purely random test? | 15:29 |
amotoki | or does it happen in a specific set of tests? | 15:29 |
amotoki | if it fails in a specific set of tests, over 90% failure might be a good indicator to skip them | 15:30 |
vishalmanchanda | I have noticed pagination and filteration related integration tests fails too much. | 15:32 |
vishalmanchanda | we can also rewrite those random failing tests rather then skipping it. | 15:33 |
amotoki | theose tests often fail for long.... but the failure rate these weeks are too high. I wonder something has changed | 15:34 |
amotoki | at least "recheck random failure" mesage is confusing | 15:34 |
vishalmanchanda | hmm, we can add more detail info. while rechecking for a patch. | 15:35 |
amotoki | vishalmanchanda: do you suggest to continue recheck recheck recheck.... until we succeed to rewrite them? | 15:35 |
vishalmanchanda | amotoki: you want to skip those failed tests? | 15:36 |
tmazur | rdopiera and I were also thinking about rewriting integration tests as scenario-based. This will help to decrease the amount of tests running | 15:37 |
amotoki | vishalmanchanda: IMHO we can skip them once and explore how to make them stable | 15:37 |
amotoki | we are wasting CI resources and power. we are not energy friendly :p | 15:38 |
tmazur | Basically we test create something, and then we test update the same thing, and for this we run create again. So we can skip the create test and include it into the scenario | 15:38 |
vishalmanchanda | ok, I am fine with that but just wondering about side-effect of that. | 15:38 |
vishalmanchanda | tmazur: I like the idea. | 15:40 |
amotoki | we are doing almost the same thing as "bare recheck" and other tests verifies the basic operation, so I think we can skip them | 15:40 |
vishalmanchanda | amotoki: that's true. | 15:40 |
vishalmanchanda | what we can do now, make a list of these random failing integration tests and then decide to skip that test or not. | 15:45 |
vishalmanchanda | where we can save these type of data about the failed integration test which everyone of us can see and update? | 15:47 |
vishalmanchanda | etherpad? | 15:47 |
vishalmanchanda | or somewhere else in tabular format. | 15:47 |
amotoki | I think we can use etherpad and collect failing tests for example last week | 15:47 |
tmazur | +1 for etherpad | 15:48 |
vishalmanchanda | ok | 15:48 |
amotoki | launchpad bug can be used as an entry point too (as it allows us to link a bug from a gerrit change) | 15:48 |
amotoki | but we can collect stats on failing tests in an etherpad | 15:49 |
vishalmanchanda | I will create a new etherpad and add some info. their. | 15:49 |
vishalmanchanda | Feel free to update and add some more info. there. | 15:50 |
amotoki | thanks | 15:50 |
vishalmanchanda | https://etherpad.opendev.org/p/random-failed-horizon-integration-tests | 15:52 |
vishalmanchanda | Does anyone have any other topic to discuss? | 15:52 |
amotoki | none from me | 15:56 |
tmazur | nothing | 15:56 |
vishalmanchanda | Does anyone of you ever setup Keystone Federation env. on devstack and provide me some references? | 15:58 |
vishalmanchanda | ok let's end this meeting. | 16:00 |
amotoki | I have no experience on it. devstack itself does not support it and perhaps you need extra config after running devstack | 16:00 |
tmazur | Never did that | 16:01 |
vishalmanchanda | ok | 16:01 |
vishalmanchanda | thanks, everyone for joining. | 16:01 |
vishalmanchanda | #endmeeting | 16:01 |
opendevmeet | Meeting ended Wed May 24 16:01:26 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/horizon/2023/horizon.2023-05-24-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/horizon/2023/horizon.2023-05-24-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/horizon/2023/horizon.2023-05-24-15.00.log.html | 16:01 |
amotoki | o/ | 16:01 |
tmazur | o/ | 16:02 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!