| opendevreview | Pierre Riteau proposed openstack/blazar master: Fix lease update when resource_type parameter is missing https://review.opendev.org/c/openstack/blazar/+/824579 | 14:26 |
|---|---|---|
| opendevreview | Mark Powers proposed openstack/blazar master: Restore lease status during a non fatal lease update exception https://review.opendev.org/c/openstack/blazar/+/810126 | 15:37 |
| priteau | #startmeeting blazar | 16:00 |
| opendevmeet | Meeting started Thu Jan 13 16:00:38 2022 UTC and is due to finish in 60 minutes. The chair is priteau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'blazar' | 16:00 |
| priteau | #topic Roll call | 16:01 |
| priteau | Hello mppowers! | 16:01 |
| mppowers | Hello priteau | 16:02 |
| priteau | Is Jason joining today? | 16:02 |
| mppowers | I think so, I'll ping | 16:02 |
| priteau | Hello diurnalist! | 16:03 |
| diurnalist | o7 | 16:04 |
| priteau | Happy New Year to you both | 16:04 |
| mppowers | Happy New Year to you too | 16:05 |
| priteau | Agenda for today: | 16:06 |
| priteau | * Blazar whiteboard | 16:06 |
| priteau | * Yoga release planning | 16:06 |
| priteau | #topic Blazar whiteboard | 16:06 |
| priteau | A long long time ago I created an Etherpad to act as a shared whiteboard to keep track of important notes | 16:07 |
| priteau | It hasn't been used much recently, but I have now refreshed it | 16:07 |
| priteau | #link https://etherpad.opendev.org/p/blazar-whiteboard | 16:07 |
| priteau | I will aim to keep it up to date from now on | 16:07 |
| diurnalist | nice | 16:08 |
| priteau | CI Status: will track whether CI is OK or broken. Currently only has master, but I will add branches once we have some backports to trigger jobs | 16:08 |
| priteau | Small patches ready to approve: if you have just a minute or two to spend on reviews, this is a list to check | 16:09 |
| priteau | All three are ready for you diurnalist | 16:09 |
| priteau | Review priorities will be listing patches that are deemed "ready to review" and of high importance for the next release | 16:10 |
| priteau | Needs work: patches that have been reviewed and need to be updated | 16:10 |
| priteau | You can also query similar things dynamically with Gerrit, but this will be a more curated list | 16:11 |
| diurnalist | sounds good, will check them out today | 16:11 |
| priteau | This brings us to the next topic | 16:12 |
| mppowers | Should I add my own patches to "ready to review" once they are? Or are you managing this priteau? | 16:12 |
| priteau | Feel free to add your patches, at this point any of your patches are a priority :) | 16:12 |
| mppowers | OK, will do | 16:12 |
| priteau | #topic Yoga release planning | 16:13 |
| priteau | We're past the Yoga-2 milestone now | 16:13 |
| priteau | The feature freeze is 6 weeks away | 16:14 |
| priteau | But maybe more importantly, Feb 14 - Feb 18 is final release for non-client libraries | 16:14 |
| priteau | This includes blazar-nova | 16:15 |
| priteau | So I would like to ask if you can review this revisited patch to add preemptibles support: https://review.opendev.org/c/openstack/blazar-nova/+/749151 | 16:16 |
| priteau | There is another patch which lives inside blazar, which I am still updating, I'll try to submit it tomorrow | 16:16 |
| priteau | But having early feedback on the blazar-nova one would be good | 16:16 |
| diurnalist | ok | 16:17 |
| priteau | Other review priorities: | 16:17 |
| priteau | 810126: Restore lease status during a non fatal lease update exception | https://review.opendev.org/c/openstack/blazar/+/810126 | 16:17 |
| priteau | Mark has just updated it, I'll review again after this meeting | 16:17 |
| priteau | While testing it I discovered a bug in blazar that was introduced as part of the usage enforcement framework | 16:18 |
| priteau | 824579: Fix lease update when resource_type parameter is missing | https://review.opendev.org/c/openstack/blazar/+/824579 | 16:18 |
| priteau | So this is a candidate for backport to xena and wallaby | 16:18 |
| priteau | As is Mark's patch | 16:18 |
| priteau | Reservation calendar in blazar-dashboard is another important one. It's one of the next ones on my list. | 16:20 |
| priteau | Thanks for pointing out 805461: Update list_allocations in hosts plugin to expose start and end dates | https://review.opendev.org/c/openstack/blazar/+/805461 | 16:21 |
| priteau | I've not reached this one yet | 16:22 |
| priteau | Then we have the patches that need work | 16:22 |
| priteau | I've left comments on resource properties patches, for both service and client | 16:22 |
| priteau | Third-party plugins and RPC layer, I've not yet reached, and they aren't passing CI | 16:23 |
| mppowers | My apologies on the resource properties patch, I fixed some of the issues Jason mentioned, but accidentally proposed a new patch that reverted it. I'll still need to update to support --all though. | 16:24 |
| priteau | I didn't notice that. That will be quicker to fix then. | 16:25 |
| priteau | My comment about splitting patches is only for submissions of new changes, this one can stay in a single change | 16:25 |
| mppowers | OK, sounds good. | 16:26 |
| priteau | Anything else which I may have overlooked? | 16:28 |
| mppowers | That is all I can think of | 16:30 |
| priteau | It's quite a list but I think we can merge a good chunk of it | 16:31 |
| diurnalist | awesome, i will get looking at the patches today too | 16:31 |
| priteau | I remember you deployed, or were thinking about deploying, some early version of my preemptibles patches. Is this still active? | 16:32 |
| diurnalist | yes, though i'm not sure if it's deployed to prod. we did deploy to dev for testing | 16:33 |
| diurnalist | it was pretty straightforward if i recall | 16:33 |
| priteau | It would be great to get some testing of the new patches once they're up | 16:35 |
| priteau | Is there anything that needs to be discussed live for any of these changes? | 16:36 |
| priteau | mppowers: what do you think of my suggestion to replace capability by property? | 16:37 |
| mppowers | I think it makes sense. I agree it is confusing to have both terminology | 16:38 |
| priteau | End users were not exposed at all to capabilities, so it's a good time to choose just one term and stick to it | 16:38 |
| priteau | Also property is used by many other OpenStack resources, on images, on flavors, etc. | 16:38 |
| diurnalist | agree | 16:39 |
| priteau | Anything else to discuss today? | 16:41 |
| mppowers | That's all for me | 16:42 |
| priteau | OK. Let's keep interacting via Gerrit. | 16:43 |
| priteau | Thanks both for joining, and let's talk in two weeks time. | 16:43 |
| mppowers | Talk to you then | 16:44 |
| priteau | #endmeeting | 16:44 |
| opendevmeet | Meeting ended Thu Jan 13 16:44:26 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:44 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/blazar/2022/blazar.2022-01-13-16.00.html | 16:44 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/blazar/2022/blazar.2022-01-13-16.00.txt | 16:44 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/blazar/2022/blazar.2022-01-13-16.00.log.html | 16:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!