| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-21 | |||
| 16:58:53 | gibi | sure still proper review on those patches but not blanket -2 | |
| 16:59:02 | artom | gibi, nope, I'm coming away from this with the understanding that func test refactors are fair game if they help make the fixes cleaner and easier to backport (and increase confidence in the reproducer test) | |
| 16:59:02 | bauzas | my other point can be deferred to next week | |
| 16:59:10 | gibi | ack | |
| 16:59:21 | gibi | moving on then | |
| 16:59:22 | gibi | (bauzas) Opportunities for low-hanging-fruits, anyone ? (only if we have time left) | |
| 16:59:31 | gibi | 40 secs :) | |
| 16:59:32 | bauzas | no time, let's punt to next week | |
| 16:59:36 | gibi | ack | |
| 16:59:46 | gibi | any last minute issue to raise today? | |
| 16:59:50 | bauzas | tl;dr: think about how to help on-off contributors like interns | |
| 17:00:10 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-06-21-15.59.log.html | |
| 17:00:10 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-06-21-15.59.txt | |
| 17:00:10 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-06-21-15.59.html | |
| 17:00:10 | opendevmeet | Meeting ended Tue Jun 21 17:00:10 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:00:10 | gibi | #endmeeting | |
| 17:00:21 | gibi | thanks folks for joining | |
| 17:00:23 | elodilles | thanks gibi o/ | |
| 17:00:26 | bauzas | thanks a lot gibi for running this one, I owe you some coin | |
| 17:00:37 | gibi | keerthi: sorry for the delay | |
| 17:00:50 | gibi | keerthi: so lets try that now | |
| 17:03:16 | keerthi | thanks sean | |
| 17:03:29 | artom | bauzas, hopefully not bitcoin | |
| 17:03:41 | gibi | nah it is cheap now | |
| 17:03:53 | keerthi | sean, this is the blueprint https://blueprints.launchpad.net/nova/+spec/define-max-volume-limit-at-flavor | |
| 17:03:54 | artom | Cheap, or worthless? :) | |
| 17:03:55 | bauzas | artom: just a nickel | |
| 17:05:12 | opendevreview | Merged openstack/nova master: add repoducer test for bug 1890244 https://review.opendev.org/c/openstack/nova/+/847000 | |
| 17:09:02 | keerthi | sean did you get chance to review the blueprint ? | |
| 17:12:09 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove unnecessary if condition https://review.opendev.org/c/openstack/nova/+/844418 | |
| 18:14:57 | opendevreview | sean mooney proposed openstack/nova master: ignore deleted server groups in validation https://review.opendev.org/c/openstack/nova/+/847001 | |
| #openstack-nova - 2022-06-22 | |||
| 03:52:50 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/846876 | |
| 06:09:13 | opendevreview | Amit Uniyal proposed openstack/nova master: Added validation for hw machine type in host capabilities https://review.opendev.org/c/openstack/nova/+/847126 | |
| 06:20:26 | opendevreview | Amit Uniyal proposed openstack/nova master: Added validation for hw machine type in host capabilities https://review.opendev.org/c/openstack/nova/+/847126 | |
| 07:03:28 | bauzas | good morning Nova | |
| 07:06:41 | gibi | bauzas: o/ | |
| 07:06:58 | bauzas | gibi: thanks for having written the email about the PCI series | |
| 07:07:05 | bauzas | gibi: today is review day for me | |
| 07:07:26 | gibi | bauzas: I felt I needed to give some guide to encourage reviewers | |
| 07:10:00 | bauzas | :) | |
| 07:10:16 | bauzas | I guess we will be only a few of them | |
| 07:10:32 | gibi | bauzas: I have to respin the series due to a doc issue | |
| 07:10:49 | gibi | just I want to finish the downstream backport review first | |
| 07:10:51 | bauzas | gibi: ok, no worries, I'll start with Uggla series | |
| 07:11:06 | bauzas | after a coffee... | |
| 07:11:35 | gibi | sure | |
| 08:37:56 | bauzas | Uggla: voted on https://review.opendev.org/c/openstack/nova/+/831507/14 | |
| 08:38:25 | bauzas | Uggla: tl;dr: yeah IMHO you could squash your patch with gibi's one but I'd also want you to modify how you use the sentinel | |
| 08:40:16 | Uggla | bauzas, ok I'm going to have a look and propose an updated version ASAP. | |
| 08:40:23 | bauzas | Uggla: thanks | |
| 08:40:41 | bauzas | with gibi's patch and just the thing I ask you, I don't think this is massive change | |
| 08:40:56 | bauzas | Uggla: one thing I also wrote, please bear with logs | |
| 08:41:27 | bauzas | Uggla: the point is, logs are very well appreciated, but large public clouds tend to disagree with us adding more logs :) | |
| 08:42:01 | bauzas | in particular when you can get information from other services, like scheduler | |
| 08:42:18 | Uggla | bauzas, oh ok. | |
| 08:42:34 | bauzas | no worries, this is something not largely known | |
| 08:42:53 | bauzas | but yeah, logs rotation are costly for them | |
| 08:43:13 | bauzas | DEBUG level is a bit different tho | |
| 08:43:23 | bauzas | but with INFO, we need to be cautious | |
| 08:43:56 | bauzas | I'll then switch to your Manila spec | |
| 08:44:16 | bauzas | the sooner we have it, the earlier we could merge the os-traits change | |
| 08:46:59 | Uggla | bauzas, anyway thx for the review | |
| 08:47:12 | bauzas | np pb, apologies this was that late | |
| 08:47:42 | bauzas | Uggla: I briefly reviewed your REST change, nothing controversial appeared | |
| 08:47:48 | bauzas | but I didn't vote | |
| 08:48:51 | Uggla | bauzas, regarding the manila one, Goutham sent comments yesterday. | |
| 08:49:18 | bauzas | nice | |
| 08:51:07 | Uggla | bauzas, remaining blocking point is the "extra spec / trait" if we want to have this explicit or not. I would rather explicit one. But we need a consensus here. | |
| 08:51:59 | Uggla | I would like to discuss this point with bauzas, gibi, sean-k-mooney. | |
| 08:52:30 | gibi | I'm around | |
| 08:53:16 | Uggla | gibi, o/ | |
| 08:54:28 | gibi | o/ | |
| 08:58:29 | bauzas | Uggla: gibi: give me 30 mins so I can recharge the context by reviewing the spec :) | |
| 08:58:52 | gibi | bauzas, Uggla: do we want to wait for sean-k-mooney too? | |
| 08:59:18 | bauzas | gibi: probably, again I need to review the spec | |
| 08:59:31 | Uggla | gibi, if possible yes. | |
| 08:59:31 | gibi | sure go ahead an review | |
| 09:00:21 | bauzas | gibi: Uggla: we could do some session by 14:30 our time if needed | |
| 09:10:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 09:10:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 09:10:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 09:10:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 09:10:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 09:10:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Stop if tracking is disable after it was enabled before https://review.opendev.org/c/openstack/nova/+/847009 | |
| 09:10:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 09:21:08 | bauzas | gibi: sean-k-mooney: Uggla yeah, so we have two concerns for the Manila spec | |
| 09:21:17 | bauzas | 1/ about flavor vs. image | |
| 09:21:29 | bauzas | 2/ about how to lock a share | |
| 09:21:53 | bauzas | if you want, we can discuss this by 1230UTC, ie. 14:30 for me and 13:30 for sean | |
| 09:22:31 | Uggla | bauzas, I think 1 is more explicit / implicit trait requirement. | |
| 09:22:54 | Uggla | bauzas, anyway ok for 14:30. | |
| 09:23:04 | gibi | yepp it is about asking the user to know about all technical prereqs for attaching a manila share | |
| 09:33:17 | sean-k-mooney[m] | for manila shars you should not explictly need to specify any traits | |
| 09:33:28 | sean-k-mooney[m] | you can optionally do so but it should not be a requiement | |
| 09:33:57 | sean-k-mooney[m] | we want this feature to work with existing flavors and images | |
| 09:34:22 | sean-k-mooney[m] | so documenting that you enither need file backed memory or hugepages | |
| 09:34:34 | sean-k-mooney[m] | and having the api check for the same should be sufficent | |
| 09:35:09 | sean-k-mooney[m] | operators can use traits for the former if they want to enforce that or they can use host aggreates or az to model which host has file backed memory | |
| 09:35:28 | sean-k-mooney[m] | and just tell there users to select that az | |
| 09:35:58 | sean-k-mooney[m] | so i dont think we need to modify the flavor/image for this feature | |