15:00:24 <vishalmanchanda> #startmeeting horizon 15:00:24 <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:24 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:24 <opendevmeet> The meeting name has been set to 'horizon' 15:00:44 <rdopiera> o/ 15:01:00 <vishalmanchanda> Hello everyone 15:03:14 <vishalmanchanda> let's start the meeting. 15:03:33 <vishalmanchanda> agenda of meeting can be found here https://etherpad.opendev.org/p/horizon-release-priorities#L45 15:04:02 <vishalmanchanda> #topic Notices 15:04:40 <vishalmanchanda> first of all, a gentle reminder about the schedule. 15:04:54 <vishalmanchanda> This week is R-17 week. 15:05:31 <vishalmanchanda> For more info. about the Caracal cycle schedule please refer https://releases.openstack.org/caracal/schedule.html 15:06:13 <vishalmanchanda> One more update from my side, I am on vacation next complete week. 15:06:30 <vishalmanchanda> So I won't be able to chair the next weekly meeting. 15:07:01 <vishalmanchanda> Please let me know if anyone interested in chairing next weekly meeting. 15:08:00 <tmazur> I am on vacation starting next Wednesday, will be back in January 15:08:09 <vishalmanchanda> ok 15:09:17 <vishalmanchanda> I think many of the folks on vacation for last 2 weeks of December. 15:10:11 <tmazur> Yeah 15:11:53 <vishalmanchanda> anyway I will chair 20th December weekly meeting, if no-one around I will skip next weekly meeting then. 15:12:05 <vishalmanchanda> moving to the next topic 15:12:29 <vishalmanchanda> #topic Release priorities 15:13:31 <vishalmanchanda> pytest-based selenium tests 15:13:57 <vishalmanchanda> I can see few patches got merged for new pytest patches. 15:14:10 <vishalmanchanda> Sorry i was not around for the reviews. 15:14:32 <rdopiera> the coverage is improving, and the job seems to be stable so far 15:14:34 <vishalmanchanda> thanks for reviewing and approving the patches. 15:14:38 <rdopiera> that is, it doesn't fail randomly 15:14:57 <vishalmanchanda> nice. 15:15:46 <vishalmanchanda> yeah, let's check how many more tests are required for the new pytests job? 15:16:22 <vishalmanchanda> moving to next topic 15:16:51 <vishalmanchanda> no updates on Migrate to Django version =>4.2 from my side for this week. 15:17:25 <vishalmanchanda> Use placement API for the hypervisor stats 15:17:33 <vishalmanchanda> https://review.opendev.org/c/openstack/horizon/+/890525 15:17:52 <vishalmanchanda> I reviewed your patch, code looks fine to me. 15:18:01 <vishalmanchanda> but I have few queries. 15:19:20 <vishalmanchanda> tmazur: your patch adds a new tab "Resource Provider" for the Hypervisors Panel. 15:21:04 <vishalmanchanda> but it doesn't solve "VCPU usuage" chart issue 15:21:27 <vishalmanchanda> I mean it stills shows me Used 8 of 3. 15:21:54 <tmazur> That's what Placement shows to us 15:23:08 <rdopiera> would using a later microversion fix it? 15:23:09 <tmazur> There was a bug filed long ago. https://bugs.launchpad.net/horizon/+bug/1397917 15:23:36 <tmazur> And it was explained there that what they call "total VCPUs" is actually "cores" 15:24:40 <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:25:44 <vishalmanchanda> As of now it looks like this to me https://ibb.co/Czqzgqn 15:26:33 <tmazur> Yeah, same as before, when we were taking this stats from hypervisors 15:28:29 <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:30:49 <vishalmanchanda> ok 15:31:27 <vishalmanchanda> but I think you should update your patch commit-msg 15:32:04 <tmazur> What do you think we should add? 15:32:15 <vishalmanchanda> It doesn't closes the bug 1974470 and 1397917 15:32:56 <vishalmanchanda> you can use Related-bug or Partial-bug here. 15:33:22 <tmazur> That's right, I'll mark as related bugs 15:33:33 <tmazur> Just to keep tracking 15:34:18 <vishalmanchanda> you can also mentioned in commit-msg that you are adding a new tab "Resource Provider" under Hypervisor panel. 15:34:52 <tmazur> It is mentioned already 15:35:10 <vishalmanchanda> ohh yes. 15:39:33 <vishalmanchanda> moving to the next topic 15:39:34 <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:44 <vishalmanchanda> #topic open-discussion 15:39:59 <vishalmanchanda> Does anyone have any other topic to discuss? 15:40:09 <tmazur> vishalmanchanda: I changed it to related bugs, please take a look when you have time 15:40:10 <rdopiera> We did a bit of research about performance at a customer who has a very large deployment of openstack 15:40:36 <rdopiera> and we found out that the admin/network and related views make hundreds of unnecessary requests 15:40:41 <vishalmanchanda> tmazur: sure. 15:41:06 <rdopiera> I have made two patches to help with that 15:41:17 <rdopiera> https://review.opendev.org/c/openstack/horizon/+/901908 15:41:25 <rdopiera> https://review.opendev.org/c/openstack/horizon/+/901984 15:42:15 <vishalmanchanda> rdopiera: ack. 15:42:19 <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:43:03 <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:38 <rdopiera> we would like to backport those patches for those customers who have performance problems as soon as possible 15:44:38 <vishalmanchanda> ok. 15:46:06 <vishalmanchanda> I will review it asap and will add amotoki as he have more knowledge on network side. 15:52:18 <vishalmanchanda> Does anyone have any other topic to discuss? 15:52:35 <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:55:14 <vishalmanchanda> If no other topic to discuss, let's end this meeting. 15:55:21 <vishalmanchanda> thanks everyone for joining. 15:55:32 <vishalmanchanda> #endmeeting