| *** elodilles_pto is now known as elodilles | 11:39 | |
| bauzas | dansmith: looks like we need to skip test_get_vnc_console from https://review.opendev.org/c/openstack/nova/+/899375/ | 12:11 |
|---|---|---|
| bauzas | we just start to test it by my change | 12:12 |
| gibi | bauzas: or you need [compute-feature-enabled]vnc_console=true in the tempest config to enable the vnc console | 12:42 |
| bauzas | gibi: I haven't checked the tempest config for this job, I can take a look | 12:57 |
| bauzas | humpf, actually call me stupid, the job worked fine with the vnc test in a previous run | 13:52 |
| dvo-plv | bauzas, Hello, Maybe you will have a time to review https://review.opendev.org/c/openstack/nova-specs/+/895924 ? | 14:09 |
| bauzas | dvo-plv: sorry I'm currently doing a lot of things, but we plan to do a spec review day on tomorrow | 14:15 |
| dvo-plv | Sure, thanks | 14:16 |
| bauzas | dansmith: I know sean-k-mooney is on PTO, but the more I think, the less I like to create a specific config option for SR-IOV GPUs | 14:33 |
| dansmith | bauzas: I didn't follow the rest of your conversation after our gmeet | 14:34 |
| dansmith | are you talking about the max_devices one or what? | 14:35 |
| bauzas | so I think I'll just create a new change for telling that we would support a new wilcard "*" for device_addresses which would be the default | 14:35 |
| bauzas | dansmith: yeah | 14:35 |
| dansmith | okay I'm not sure how that solves the prioblem | 14:35 |
| bauzas | dansmith: I tried to see how we could use max_instances (or max_devices) for both SR-IOV and non-SRIOV GPUs, but it would be dififcult | 14:35 |
| bauzas | so if max_instances would be only for SRIOV GPUs, it would be sadf | 14:36 |
| dansmith | I thought the non-SRIOV ones didn't need it, and that's why the default would be None or -1 to not apply that to those | 14:36 |
| dansmith | why? | 14:36 |
| dansmith | it's very specifically only needed for those, IIRC | 14:36 |
| bauzas | we have two different behaviours | 14:38 |
| bauzas | either we ask for SRIOV operators to provide all the VFs they want, | 14:38 |
| bauzas | or we try them to provide a specific option | 14:38 |
| *** efried1 is now known as efried | 14:39 | |
| bauzas | anyway, let me try to provide a specific change and we'll see | 14:41 |
| dansmith | mkay | 14:42 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [libvirt]Add migration_inbound_addr https://review.opendev.org/c/openstack/nova/+/900203 | 15:10 |
| gibi | bauzas, sean-k-mooney: ^^ it belongs to https://blueprints.launchpad.net/nova/+spec/libvirt-migrate-with-hostname-instead-of-ip which I will bring up for approval on the tomorrows meeting | 15:11 |
| bauzas | okay, but too many things I work on | 15:13 |
| bauzas | f nivia | 15:13 |
| gibi | this is just a fyi, we will discuss it further tomorrow on the meeting | 15:17 |
| *** gouthamr_ is now known as gouthamr | 16:09 | |
| opendevreview | Merged openstack/nova master: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/899375 | 16:10 |
| bauzas | \o/ | 16:13 |
| auniyal6 | bauzas can this be merged https://review.opendev.org/c/openstack/nova/+/865336, it was created by gibi then sean backported it till wallaby | 17:32 |
| auniyal6 | this whole topic only wallaby this pending https://review.opendev.org/q/topic:bug%252F1970467 | 17:33 |
| auniyal6 | this will be helpful for other backports | 17:33 |
| bauzas | sorry, can't do it, other priorities for the moment due to f... nvidia issues | 17:36 |
| auniyal6 | ack | 17:37 |
| dansmith | that seems like a pretty impactful backport to me, just skimming | 17:39 |
| dansmith | I'd like to chat with sean about it tomorrow when he's back before I review/approve | 17:40 |
| auniyal6 | thanks dansmith | 17:40 |
| bauzas | my head is just about to explode | 17:40 |
| bauzas | too many things on the pile, that's it | 17:41 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Fix the PCI device capability dict creation https://review.opendev.org/c/openstack/nova/+/899252 | 18:41 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Store pf_mac_address and vf_num in extra_info https://review.opendev.org/c/openstack/nova/+/899253 | 18:41 |
| opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/899254 | 18:41 |
| *** thelounge5514 is now known as thelounge551 | 19:17 | |
| *** JayF is now known as Guest6061 | 19:55 | |
| *** JasonF is now known as JayF | 19:55 | |
| *** zigo_ is now known as zigo | 19:57 | |
| opendevreview | Pierre Riteau proposed openstack/nova master: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/900242 | 20:15 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: add max_instances and a wildcard https://review.opendev.org/c/openstack/nova/+/900250 | 22:16 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: Support to pass PFs in device_addresses https://review.opendev.org/c/openstack/nova/+/900251 | 22:16 |
| opendevreview | Steven Relf proposed openstack/nova master: Adding basic auth to dynamic vendordata api calls https://review.opendev.org/c/openstack/nova/+/900252 | 22:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!