| vishalmanchanda | #startmeeting horizon | 15:00 |
|---|---|---|
| opendevmeet | Meeting started Wed Dec 6 15:00:24 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:01 |
| vishalmanchanda | let's start the meeting. | 15:03 |
| vishalmanchanda | agenda of meeting can be found here https://etherpad.opendev.org/p/horizon-release-priorities#L45 | 15:03 |
| vishalmanchanda | #topic Notices | 15:04 |
| vishalmanchanda | first of all, a gentle reminder about the schedule. | 15:04 |
| vishalmanchanda | This week is R-17 week. | 15:04 |
| vishalmanchanda | For more info. about the Caracal cycle schedule please refer https://releases.openstack.org/caracal/schedule.html | 15:05 |
| vishalmanchanda | One more update from my side, I am on vacation next complete week. | 15:06 |
| vishalmanchanda | So I won't be able to chair the next weekly meeting. | 15:06 |
| vishalmanchanda | Please let me know if anyone interested in chairing next weekly meeting. | 15:07 |
| tmazur | I am on vacation starting next Wednesday, will be back in January | 15:08 |
| vishalmanchanda | ok | 15:08 |
| vishalmanchanda | I think many of the folks on vacation for last 2 weeks of December. | 15:09 |
| tmazur | Yeah | 15:10 |
| vishalmanchanda | anyway I will chair 20th December weekly meeting, if no-one around I will skip next weekly meeting then. | 15:11 |
| vishalmanchanda | moving to the next topic | 15:12 |
| vishalmanchanda | #topic Release priorities | 15:12 |
| vishalmanchanda | pytest-based selenium tests | 15:13 |
| vishalmanchanda | I can see few patches got merged for new pytest patches. | 15:13 |
| vishalmanchanda | Sorry i was not around for the reviews. | 15:14 |
| rdopiera | the coverage is improving, and the job seems to be stable so far | 15:14 |
| vishalmanchanda | thanks for reviewing and approving the patches. | 15:14 |
| rdopiera | that is, it doesn't fail randomly | 15:14 |
| vishalmanchanda | nice. | 15:14 |
| vishalmanchanda | yeah, let's check how many more tests are required for the new pytests job? | 15:15 |
| vishalmanchanda | moving to next topic | 15:16 |
| vishalmanchanda | no updates on Migrate to Django version =>4.2 from my side for this week. | 15:16 |
| vishalmanchanda | Use placement API for the hypervisor stats | 15:17 |
| vishalmanchanda | https://review.opendev.org/c/openstack/horizon/+/890525 | 15:17 |
| vishalmanchanda | I reviewed your patch, code looks fine to me. | 15:17 |
| vishalmanchanda | but I have few queries. | 15:18 |
| vishalmanchanda | tmazur: your patch adds a new tab "Resource Provider" for the Hypervisors Panel. | 15:19 |
| vishalmanchanda | but it doesn't solve "VCPU usuage" chart issue | 15:21 |
| vishalmanchanda | I mean it stills shows me Used 8 of 3. | 15:21 |
| tmazur | That's what Placement shows to us | 15:21 |
| rdopiera | would using a later microversion fix it? | 15:23 |
| tmazur | There was a bug filed long ago. https://bugs.launchpad.net/horizon/+bug/1397917 | 15:23 |
| tmazur | And it was explained there that what they call "total VCPUs" is actually "cores" | 15:23 |
| tmazur | So yes, it's confusing. I brought it up on PTG, maybe we should use quotas for total instead? Or just don't show the diagrams? | 15:24 |
| vishalmanchanda | As of now it looks like this to me https://ibb.co/Czqzgqn | 15:25 |
| tmazur | Yeah, same as before, when we were taking this stats from hypervisors | 15:26 |
| tmazur | Still, I'd prefer to have this patch merged, so we could have the Placement calls set up for Horizon, and decide about diagrams later. It's a huge chunk of code | 15:28 |
| vishalmanchanda | ok | 15:30 |
| vishalmanchanda | but I think you should update your patch commit-msg | 15:31 |
| tmazur | What do you think we should add? | 15:32 |
| vishalmanchanda | It doesn't closes the bug 1974470 and 1397917 | 15:32 |
| vishalmanchanda | you can use Related-bug or Partial-bug here. | 15:32 |
| tmazur | That's right, I'll mark as related bugs | 15:33 |
| tmazur | Just to keep tracking | 15:33 |
| vishalmanchanda | you can also mentioned in commit-msg that you are adding a new tab "Resource Provider" under Hypervisor panel. | 15:34 |
| tmazur | It is mentioned already | 15:34 |
| vishalmanchanda | ohh yes. | 15:35 |
| vishalmanchanda | moving to the next topic | 15:39 |
| opendevreview | Tatiana Ovchinnikova proposed openstack/horizon master: Use Placement API along with the hypervisor stats https://review.opendev.org/c/openstack/horizon/+/890525 | 15:39 |
| vishalmanchanda | #topic open-discussion | 15:39 |
| vishalmanchanda | Does anyone have any other topic to discuss? | 15:39 |
| tmazur | vishalmanchanda: I changed it to related bugs, please take a look when you have time | 15:40 |
| rdopiera | We did a bit of research about performance at a customer who has a very large deployment of openstack | 15:40 |
| rdopiera | and we found out that the admin/network and related views make hundreds of unnecessary requests | 15:40 |
| vishalmanchanda | tmazur: sure. | 15:40 |
| rdopiera | I have made two patches to help with that | 15:41 |
| rdopiera | https://review.opendev.org/c/openstack/horizon/+/901908 | 15:41 |
| rdopiera | https://review.opendev.org/c/openstack/horizon/+/901984 | 15:41 |
| vishalmanchanda | rdopiera: ack. | 15:42 |
| rdopiera | one removes checks for quotas, which makes sense for project/networks, but not for admin/networks, as there would be as many checks as there are tenants, and that can be hundreds | 15:42 |
| rdopiera | the other one adds a setting to remove the dhcp agents column, which is expensive to populate and doesn't make sense on many configurations | 15:43 |
| rdopiera | we would like to backport those patches for those customers who have performance problems as soon as possible | 15:43 |
| vishalmanchanda | ok. | 15:44 |
| vishalmanchanda | I will review it asap and will add amotoki as he have more knowledge on network side. | 15:46 |
| vishalmanchanda | Does anyone have any other topic to discuss? | 15:52 |
| opendevreview | Jan Jasek proposed openstack/horizon master: pytest-based selenium tests add host_aggregates create, delete tests https://review.opendev.org/c/openstack/horizon/+/902785 | 15:52 |
| vishalmanchanda | If no other topic to discuss, let's end this meeting. | 15:55 |
| vishalmanchanda | thanks everyone for joining. | 15:55 |
| vishalmanchanda | #endmeeting | 15:55 |
| opendevmeet | Meeting ended Wed Dec 6 15:55:32 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/horizon/2023/horizon.2023-12-06-15.00.html | 15:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/horizon/2023/horizon.2023-12-06-15.00.txt | 15:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/horizon/2023/horizon.2023-12-06-15.00.log.html | 15:55 |
| tmazur | o/ | 15:56 |
| opendevreview | Merged openstack/horizon master: pytest-based selenium tests add routers add, delete interface tests https://review.opendev.org/c/openstack/horizon/+/902299 | 16:42 |
| opendevreview | Merged openstack/horizon master: pytest-based selenium tests add project members, roles tests https://review.opendev.org/c/openstack/horizon/+/901982 | 16:42 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to restart it onto an updated replication key | 21:36 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!