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