| opendevreview | Rajesh Tailor proposed openstack/nova master: Add upgrade status check for duplicate cell names https://review.opendev.org/c/openstack/nova/+/901810 | 04:17 |
|---|---|---|
| opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceExists exception for duplicate instance https://review.opendev.org/c/openstack/nova/+/860938 | 04:50 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Add retry on attachment_update https://review.opendev.org/c/openstack/nova/+/901815 | 08:00 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Add retry on attachment_update https://review.opendev.org/c/openstack/nova/+/901815 | 08:04 |
| opendevreview | Haidong Pang proposed openstack/nova master: Catch MessagingTimeout during evacuation https://review.opendev.org/c/openstack/nova/+/901817 | 08:10 |
| opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote sonsole shutdown https://review.opendev.org/c/openstack/nova/+/901824 | 10:01 |
| bauzas | gibi: thanks for having commented https://review.opendev.org/c/openstack/nova/+/899625 | 10:22 |
| bauzas | good point, I haven't verified about providing *,* | 10:22 |
| * bauzas goes offline for a couple of hours (lunch) | 10:42 | |
| bauzas | gibi: sean-k-mooney: got a sec for discussing about https://review.opendev.org/c/openstack/nova/+/899625/ ? | 13:44 |
| bauzas | if not, no worries | 13:46 |
| bauzas | lemme explain what I'll do | 13:46 |
| bauzas | first, I'll work on pas-ha[m] patch https://review.opendev.org/c/openstack/nova/+/899406 for having it as a dependency for https://review.opendev.org/c/openstack/nova/+/899625/ | 13:48 |
| bauzas | secondly, I'll split my own patch in two, one for let's make device_addresses to be None | 13:49 |
| bauzas | (instead of a wildcard) | 13:49 |
| bauzas | and then, the third change will be https://review.opendev.org/c/openstack/nova/+/899625/ without the wildcard | 13:49 |
| bauzas | pas-ha[m]: are you around? | 13:49 |
| bauzas | pas-ha[m]: fwiw, I'm gonna add a few UTs | 14:12 |
| bauzas | to your patch | 14:12 |
| pas-ha[m] | bauzas: hi, sure | 14:18 |
| bauzas | on it already, shall be quick hopefully | 14:20 |
| bauzas | I feel sad, I just saw https://review.opendev.org/c/openstack/nova/+/858012 | 14:20 |
| bauzas | people discovered the bug with SRIOV VFs like one year ago, but due to a difficult bug report and no way to reproduce it, I skipped it | 14:21 |
| gibi | bauzas: sean-k-mooney is out today | 14:44 |
| bauzas | ack | 14:46 |
| bauzas | my point is that I want to split the fact that we modify the option from the other bugfix, which is how to not use the other inventories | 14:46 |
| gibi | make sense | 14:47 |
| bauzas | and then we could discuss whether we prefer to have a wildcard or just not using device_addresses | 14:48 |
| bauzas | IMHO, after thinking about it, I prefer the latter | 14:48 |
| gibi | not using device_addresses simplifies the config parsing | 14:49 |
| gibi | it make the meaning of the config a bit less obvious | 14:49 |
| gibi | but we can document that | 14:49 |
| gibi | is the reason of wildcarding to make the number of inventories dynamic? | 14:50 |
| bauzas | no, just because it would be large to provide all of the VFs for example with a L2 | 14:51 |
| bauzas | which has 32 of them :) | 14:51 |
| gibi | ahh | 14:51 |
| bauzas | this is simplier to say 'let's just let nova to find the needed VFs' | 14:51 |
| gibi | but it has a side effect that if the number of VF changes nova will dynamically pick up the change | 14:52 |
| bauzas | well | 14:54 |
| bauzas | it's the same if they change the GPU | 14:54 |
| bauzas | the VFs shouldn't change unless the GPU is changed | 14:54 |
| bauzas | and fwiw, for that, the operator needs to first to either delete or shelve the instances | 14:54 |
| bauzas | because of the allocations | 14:55 |
| gibi | OK so the number of VFs is constant for a given pGPU | 14:55 |
| bauzas | yeah | 14:55 |
| gibi | cool | 14:55 |
| bauzas | this isn't changing | 14:55 |
| auniyal6 | I saw VMware NSX CI job running first time, but logs link is not accessible | 15:03 |
| auniyal6 | or job link | 15:04 |
| auniyal6 | http://p3-ci-int.eng.vmware.com:8080/job/ext-nova-zuul/42056/ | 15:04 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Use configuration for single enabled mdev type https://review.opendev.org/c/openstack/nova/+/899406 | 15:15 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: WIP : libvirt: Cap with max_instances GPU types https://review.opendev.org/c/openstack/nova/+/899625 | 15:15 |
| bauzas | pas-ha[m]: added UTs, your patch sounds good to me | 15:16 |
| * bauzas disappears for family taxi reasons | 15:17 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!