*** tosky has quit IRC | 00:01 | |
*** JamesBenson has quit IRC | 00:08 | |
*** LinPeiWen has joined #openstack-nova | 00:37 | |
*** JamesBenson has joined #openstack-nova | 00:43 | |
brinzhang0 | gmann: ack, thanks, you have added in the cleanup apis etherpad, works for me, keep the consistency with other apis is better ^^ | 00:48 |
---|---|---|
*** JamesBenson has quit IRC | 00:49 | |
*** derekh has quit IRC | 00:50 | |
*** tbachman has quit IRC | 00:52 | |
*** tbachman has joined #openstack-nova | 00:53 | |
*** hemna has quit IRC | 00:54 | |
*** hemna has joined #openstack-nova | 00:59 | |
*** jamesdenton has joined #openstack-nova | 01:02 | |
*** brinzhang has joined #openstack-nova | 01:03 | |
*** brinzhang0 has quit IRC | 01:06 | |
*** JamesBenson has joined #openstack-nova | 01:21 | |
*** hamalq_ has quit IRC | 01:24 | |
*** gyee has quit IRC | 01:24 | |
*** evrardjp has quit IRC | 01:24 | |
*** knikolla has quit IRC | 01:24 | |
*** amodi has quit IRC | 01:24 | |
*** CeeMac has quit IRC | 01:24 | |
*** ftarasenko has quit IRC | 01:24 | |
*** fnordahl has quit IRC | 01:24 | |
*** tinwood has quit IRC | 01:24 | |
*** nicolasbock has quit IRC | 01:24 | |
*** mnasiadka has quit IRC | 01:24 | |
*** nautik has quit IRC | 01:24 | |
*** flaviof has quit IRC | 01:24 | |
*** pas-ha has quit IRC | 01:24 | |
*** cap has quit IRC | 01:24 | |
*** Hazelesque_ has quit IRC | 01:24 | |
*** jroll has quit IRC | 01:24 | |
*** JamesBenson has quit IRC | 01:25 | |
*** mlavalle has quit IRC | 01:26 | |
*** hamalq_ has joined #openstack-nova | 01:30 | |
*** gyee has joined #openstack-nova | 01:30 | |
*** evrardjp has joined #openstack-nova | 01:30 | |
*** amodi has joined #openstack-nova | 01:30 | |
*** ftarasenko has joined #openstack-nova | 01:30 | |
*** knikolla has joined #openstack-nova | 01:30 | |
*** CeeMac has joined #openstack-nova | 01:30 | |
*** fnordahl has joined #openstack-nova | 01:30 | |
*** tinwood has joined #openstack-nova | 01:30 | |
*** mnasiadka has joined #openstack-nova | 01:30 | |
*** nicolasbock has joined #openstack-nova | 01:30 | |
*** nautik has joined #openstack-nova | 01:30 | |
*** flaviof has joined #openstack-nova | 01:30 | |
*** pas-ha has joined #openstack-nova | 01:30 | |
*** cap has joined #openstack-nova | 01:30 | |
*** Hazelesque_ has joined #openstack-nova | 01:30 | |
*** jroll has joined #openstack-nova | 01:30 | |
*** LinPeiWen has quit IRC | 01:38 | |
*** dave-mccowan has joined #openstack-nova | 01:49 | |
*** abhishekk has quit IRC | 02:03 | |
*** macz_ has quit IRC | 02:05 | |
*** macz_ has joined #openstack-nova | 02:05 | |
*** k_mouza has joined #openstack-nova | 02:07 | |
*** tinwood has quit IRC | 02:08 | |
*** tinwood has joined #openstack-nova | 02:11 | |
*** k_mouza has quit IRC | 02:12 | |
*** hemanth_n has joined #openstack-nova | 02:28 | |
openstackgerrit | Wenping Song proposed openstack/nova master: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/767449 | 02:45 |
*** LinPeiWen has joined #openstack-nova | 02:50 | |
*** Yumeng has joined #openstack-nova | 02:53 | |
*** hamalq_ has quit IRC | 03:00 | |
*** xinranwang has joined #openstack-nova | 03:02 | |
*** gyee has quit IRC | 03:11 | |
*** mkrai has joined #openstack-nova | 03:37 | |
*** mgoddard has quit IRC | 04:03 | |
*** mkrai has quit IRC | 04:27 | |
*** macz_ has quit IRC | 04:28 | |
*** mkrai has joined #openstack-nova | 04:29 | |
*** songwenping_ has quit IRC | 04:44 | |
*** songwenping_ has joined #openstack-nova | 04:44 | |
*** mgoddard has joined #openstack-nova | 05:05 | |
*** abhishekk has joined #openstack-nova | 05:05 | |
*** k_mouza has joined #openstack-nova | 05:09 | |
*** k_mouza has quit IRC | 05:13 | |
*** mkrai has quit IRC | 05:21 | |
*** mkrai_ has joined #openstack-nova | 05:21 | |
*** tbachman has quit IRC | 05:22 | |
*** tbachman has joined #openstack-nova | 05:22 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-nova | 05:33 | |
*** mgoddard has quit IRC | 06:00 | |
*** nautik has quit IRC | 06:06 | |
*** diconico07 has quit IRC | 06:06 | |
*** xinranwang has quit IRC | 06:11 | |
*** vishalmanchanda has joined #openstack-nova | 06:17 | |
*** LinPeiWen has quit IRC | 06:27 | |
*** dasp has quit IRC | 06:30 | |
*** LinPeiWen has joined #openstack-nova | 06:41 | |
*** dasp has joined #openstack-nova | 06:47 | |
*** zzzeek has quit IRC | 06:57 | |
*** zzzeek has joined #openstack-nova | 06:58 | |
*** mgoddard has joined #openstack-nova | 07:04 | |
*** ralonsoh has joined #openstack-nova | 07:14 | |
*** rcernin has quit IRC | 07:14 | |
*** rcernin has joined #openstack-nova | 07:30 | |
*** mkrai_ has quit IRC | 07:31 | |
*** slaweq has joined #openstack-nova | 07:34 | |
*** slaweq has quit IRC | 07:38 | |
*** belmoreira has joined #openstack-nova | 07:45 | |
*** slaweq has joined #openstack-nova | 07:50 | |
*** mkrai has joined #openstack-nova | 07:56 | |
*** slaweq has quit IRC | 08:01 | |
*** mgoddard has quit IRC | 08:01 | |
*** slaweq has joined #openstack-nova | 08:08 | |
*** andrewbonney has joined #openstack-nova | 08:11 | |
openstackgerrit | zhulipeng proposed openstack/nova master: Fixed multipath residue when evacuating the VM https://review.opendev.org/c/openstack/nova/+/767468 | 08:14 |
*** songwenping_ has quit IRC | 08:17 | |
*** songwenping_ has joined #openstack-nova | 08:18 | |
*** sapd1 has joined #openstack-nova | 08:18 | |
*** tesseract has joined #openstack-nova | 08:19 | |
bauzas | good morning Nova | 08:40 |
*** songwenping__ has joined #openstack-nova | 08:41 | |
*** jawad_axd has joined #openstack-nova | 08:42 | |
*** rcernin has quit IRC | 08:44 | |
*** songwenping_ has quit IRC | 08:44 | |
gibi | bauzas: good morning to you too | 08:53 |
bauzas | gibi: last 2 days for you too ? | 08:55 |
gibi | bauzas: yepp, I'm already a bit in vacation mood | 08:55 |
bauzas | LOL | 08:55 |
bauzas | I'm still trying to remove the -W flag from the routed networks change before I'm going south :) | 08:56 |
bauzas | and wow, Zuul gave me +1 on https://review.opendev.org/c/openstack/nova/+/749068/5 \o/ | 08:57 |
gibi | I know I will looking back at gerrit during the vacation periods as my wife has to work on some of the days | 08:57 |
bauzas | yeepeekay :) | 08:57 |
gibi | that is your present from Zuul :D | 08:57 |
bauzas | well, given how I battled with Zuul for this change, I'm more like this https://www.youtube.com/watch?v=YfpDSNNgYhI | 08:59 |
gibi | hehe, that is very vacation specific movie at least in my country :) | 09:01 |
gibi | almost like home alone | 09:01 |
*** mgoddard has joined #openstack-nova | 09:02 | |
*** ociuhandu has joined #openstack-nova | 09:06 | |
bauzas | agreed, same here | 09:06 |
*** derekh has joined #openstack-nova | 09:07 | |
bauzas | I don't expect Christmas periods without Die Hard 2 or 1 movies being broadcasted on at least one of them | 09:07 |
bauzas | the 2 gets more broadcasted tho | 09:07 |
bauzas | wow, TIL the reasoning behind https://www.youtube.com/watch?v=KAYIgEpds_Y | 09:09 |
bauzas | the French dubbing totally trampled this phrase | 09:09 |
*** tosky has joined #openstack-nova | 09:12 | |
*** ociuhandu has quit IRC | 09:16 | |
*** teoobo_ has joined #openstack-nova | 09:18 | |
gibi | bauzas: I will get you the hungarian dub after my morning call | 09:20 |
gibi | :) | 09:20 |
*** sapd1 has quit IRC | 09:27 | |
*** ociuhandu has joined #openstack-nova | 09:46 | |
*** ociuhandu has quit IRC | 09:53 | |
*** mgoddard has quit IRC | 10:01 | |
gibi | bauzas: https://www.youtube.com/watch?v=BeH-ODa-41o | 10:03 |
gibi | interestingly there is two different hun dub for die hard 1 | 10:04 |
*** mgoddard has joined #openstack-nova | 10:05 | |
gibi | gmann, sean-k-mooney, melwitt, bauzas, brinzhang: I'm OK with the proposal in the API etherpad. I'm glad we found an agreeable way forward | 10:08 |
bauzas | me too | 10:08 |
bauzas | fwiw, I just modified my change | 10:08 |
*** ociuhandu has joined #openstack-nova | 10:08 | |
bauzas | https://review.opendev.org/c/openstack/nova/+/749068/5/nova/tests/functional/integrated_helpers.py | 10:09 |
bauzas | but I'll split this hairy change into multiple pieces now that everything works | 10:09 |
bauzas | gibi: sharpen your pen, hopefully will upload the series later the day | 10:09 |
gibi | bauzas: sure, I will try to review it before I go | 10:10 |
bauzas | no rushes, we're half the cycle | 10:10 |
bauzas | but I'm glad this works, as proven by the functests | 10:10 |
*** ociuhandu has quit IRC | 10:12 | |
gibi | I'm happy too, it was something I promised making happen in the past so I'm happy to sink time on reviewing it | 10:12 |
*** ociuhandu has joined #openstack-nova | 10:12 | |
*** k_mouza has joined #openstack-nova | 10:15 | |
stephenfin | gibi: sorry, was out yesterday. Looking now | 10:15 |
gibi | stephenfin: ack, no worries | 10:16 |
stephenfin | in the meantime, bauzas/gibi: any chance you'd look at https://review.opendev.org/c/openstack/nova-specs/+/765796 to sync spec and code? | 10:16 |
brinzhang | gibi: yeah, me too, happy to see the ethpad for clean multi apis ^^ | 10:16 |
gibi | stephenfin: on it | 10:16 |
bauzas | stephenfin: kids taxi, but I'll look | 10:16 |
gibi | bauzas: you don't have to, gmann and I took care of it now | 10:17 |
bauzas | kk, ttyl | 10:17 |
gibi | lyarwood: I'm look at another possible race in tempest about volume handling https://zuul.opendev.org/t/openstack/build/fce0115deae94e38aaef4a349e271e20/log/job-output.txt#69070 | 10:18 |
stephenfin | Thanks! There's also a part 3, but that's got an open question on it from sean-k-mooney. I wonder if we should discuss in the team meeting today? https://review.opendev.org/c/openstack/nova-specs/+/765797/1/specs/wallaby/approved/modernize-os-hypervisors-api.rst | 10:19 |
lyarwood | gibi: oh fun | 10:21 |
lyarwood | gibi: let me take a look | 10:21 |
*** mkrai has quit IRC | 10:23 | |
*** k_mouza has quit IRC | 10:24 | |
*** k_mouza has joined #openstack-nova | 10:25 | |
elod | lyarwood, similarly like yesterday: can you hit now the bandit patch in queens: https://review.opendev.org/c/openstack/nova/+/767364 o:) | 10:25 |
*** mkrai has joined #openstack-nova | 10:27 | |
lyarwood | elod: ack will do | 10:28 |
openstackgerrit | Merged openstack/nova-specs master: Update modernize-os-hypervisors-api spec, pt. 2 https://review.opendev.org/c/openstack/nova-specs/+/765796 | 10:28 |
openstackgerrit | Alexandre arents proposed openstack/nova master: Revert "nova-multi-cell: Skip test_cold_migrate_unshelved_instance" https://review.opendev.org/c/openstack/nova/+/765933 | 10:30 |
zigo | bauzas: Hi man! | 10:31 |
zigo | One quick question about GPU in passthrough mode: what's the reason for moving from "type-PCI" to "type-PF" ? | 10:31 |
zigo | We had the issue when we upgraded from Rocky to Train ... | 10:32 |
bauzas | zigo: that depends on which GPU card you have | 10:37 |
bauzas | if it supports SR-IOV, you need to modify the config because Nova will see the PF | 10:37 |
zigo | bauzas: If it has SR-IOV support, then type-PF ? | 10:37 |
bauzas | yes | 10:37 |
zigo | Ah ok... | 10:37 |
bauzas | nvidia T4s | 10:37 |
zigo | bauzas: That's the model we have, indeed. | 10:37 |
bauzas | yeah, we had a problem with nvidia | 10:38 |
zigo | bauzas: Is this for both the alias in the scheduler and in the compute ? | 10:38 |
bauzas | they only say that SR-IOV is experimental for T4, but given this, you actuallly need to change nova for it | 10:38 |
bauzas | zigo: hem, yes AFAIK | 10:38 |
bauzas | stephenfin: right ? ^ | 10:38 |
lyarwood | gibi: odd, so one of the tests makes two attempts to detach the same volume | 10:40 |
stephenfin | zigo, bauzas: correct - if the device reports SR-IOV capabilities, you need to set the device type | 10:40 |
zigo | bauzas: Thanks a lot. I'll add a per-compute option to select PCI / PF then. | 10:42 |
stephenfin | On that note, I still have two patches open to improve SR-IOV testing, if anyone has a few minutes. Happy to review other stuff in exchange https://review.opendev.org/q/topic:%2522sriov-functional-tests%2522+status:open | 10:42 |
lyarwood | gibi: https://review.opendev.org/c/openstack/tempest/+/760357 - this added a cleanup to detach the volume that these tests already detaches | 10:43 |
lyarwood | I'll write up a bug shortly | 10:44 |
*** tosky has quit IRC | 10:47 | |
*** tosky has joined #openstack-nova | 10:47 | |
*** zzzeek has quit IRC | 10:47 | |
*** zzzeek has joined #openstack-nova | 10:49 | |
zigo | bauzas: IMO, it'd be really nice to automate this, with placement reporting the correct type and the scheduler being auto-magic. It took us 2 days to figure out what was going on... | 10:49 |
elod | thx lyarwood ! | 10:50 |
gibi | stephenfin: ack, I can add that to today's agenda | 10:51 |
*** lbragstad has quit IRC | 10:52 | |
gibi | lyarwood: thanks! (sorry I was pulled after I pinged you) | 10:53 |
sean-k-mooney | zigo: we do not report pci devices in placement yet | 10:53 |
zigo | sean-k-mooney: But that would be a nice improvement ! :) | 10:54 |
sean-k-mooney | zigo: also this is because of a upgrade in your t4 device firmware | 10:54 |
sean-k-mooney | zigo: yes its on our todolist for a few years | 10:54 |
sean-k-mooney | there is a series that does most of it that was abandoned 2 years ago | 10:54 |
sean-k-mooney | we just need to revive it | 10:54 |
sean-k-mooney | it was blocked because placmenet did not support nested resouce providers proerly at the time | 10:54 |
sean-k-mooney | that has now meen fixed on the placment side | 10:55 |
gibi | lyarwood: i don't see the double detach here: tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario | 10:55 |
*** lbragstad has joined #openstack-nova | 10:55 | |
zigo | sean-k-mooney: Is type-PCI and type-PF the only possible values? | 10:55 |
gibi | lyarwood: nvm, I see it now | 10:55 |
sean-k-mooney | no also type-VF | 10:55 |
zigo | (I'm implementing this in my deployment software and need to validate input) | 10:55 |
zigo | Ok, thanks. | 10:56 |
gibi | nova_volume_attach adds a cleanup and the test case itself adds the same cleanup | 10:56 |
sean-k-mooney | we are adding type-VDPA but i plan to not allow that in the alias | 10:56 |
sean-k-mooney | VDPA devices will be VF that cant be use with alias based passthough as they require an additonal contolplain to function | 10:56 |
sean-k-mooney | zigo: by the way are you currently supporting devname in the pci whitelist? | 10:57 |
sean-k-mooney | zigo: we woudl really like to remove that in the future | 10:57 |
zigo | sean-k-mooney: Yeah, I do ! | 10:57 |
sean-k-mooney | ok what we would prefer is for installer tools to convert that to the pci adress | 10:58 |
zigo | For each compute node, I can set: use_gpu, gpu_name, gpu_vendor_id, gpu_product_id and gpu_device_type. | 10:58 |
sean-k-mooney | gpu_name? | 10:58 |
zigo | I just need to expose gpu_device_type in my API which I didn't do... | 10:58 |
zigo | Yeah, a free name which goes in the alias. | 10:59 |
sean-k-mooney | oh that is differnt | 10:59 |
sean-k-mooney | im not talking about the alias | 10:59 |
sean-k-mooney | im talking about the whitelist where we support devname for nics e.g. eht1 | 10:59 |
*** rpittau|afk is now known as rpittau | 10:59 | |
sean-k-mooney | eth1 | 10:59 |
zigo | We don't do SR-IOV yet. | 10:59 |
zigo | Just GPUs... | 11:00 |
sean-k-mooney | vGPU or pci passthough | 11:00 |
sean-k-mooney | if your using pci passthough via the alias and pci whitelist that is most of the config for neutron sriov | 11:01 |
sean-k-mooney | well sriov is jsut a tech its not related to networkign some gpus supprot it too | 11:01 |
sean-k-mooney | e.g. you can create VF on some gpus and passthough those to the guest | 11:02 |
sean-k-mooney | instead of the PF | 11:02 |
zigo | sean-k-mooney: Maybe for later, I'll implement it then... Thanks for the details! | 11:04 |
sean-k-mooney | no worries i just want to make sure you dont implement someitng we want to remove and then have to deal with migratin users off the use of devname | 11:05 |
sean-k-mooney | in the pci whitelist | 11:05 |
openstackgerrit | Merged openstack/nova stable/queens: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/767364 | 11:07 |
*** teoobo_ has quit IRC | 11:11 | |
*** sapd1 has joined #openstack-nova | 11:14 | |
*** sapd1 has quit IRC | 11:18 | |
*** k_mouza has quit IRC | 11:19 | |
*** k_mouza has joined #openstack-nova | 11:24 | |
*** k_mouza has quit IRC | 11:25 | |
*** k_mouza has joined #openstack-nova | 11:26 | |
*** lbragstad has quit IRC | 11:38 | |
*** lbragstad has joined #openstack-nova | 11:42 | |
stephenfin | sean-k-mooney: fyi I closed this as WONTFIX https://bugs.launchpad.net/nova/+bug/1908507 | 11:47 |
openstack | Launchpad bug 1908507 in OpenStack Compute (nova) "vif quotas not set for tap interface" [Undecided,Won't fix] | 11:47 |
stephenfin | lemme know if you disagree | 11:48 |
sean-k-mooney | hum for calico that should technically work | 11:48 |
sean-k-mooney | i agree we are unlikely to fix it and they shoudl just use neutron qos | 11:49 |
openstackgerrit | Elod Illes proposed openstack/nova stable/rocky: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766492 | 11:54 |
*** ociuhandu has quit IRC | 11:55 | |
*** brinzhang_ has joined #openstack-nova | 11:57 | |
*** brinzhang has quit IRC | 12:00 | |
*** JamesBenson has joined #openstack-nova | 12:01 | |
*** JamesBenson has quit IRC | 12:07 | |
*** JamesBenson has joined #openstack-nova | 12:07 | |
elod | lyarwood, sorry, one more thing o:) the ussuri gate fixing patch looks good now... if you have a minute: https://review.opendev.org/c/openstack/nova/+/766738 | 12:07 |
lyarwood | elod: LGTM, could you +2/+W as I own it | 12:13 |
elod | lyarwood: ACK | 12:14 |
*** raildo has joined #openstack-nova | 12:23 | |
gibi | stephenfin: thanks for the lot of bugtriage the numbers now looks really good | 12:23 |
stephenfin | np | 12:23 |
gibi | I think we have an all time low untriaged bug count | 12:27 |
gibi | (all time low since I'm looking) | 12:27 |
*** ociuhandu has joined #openstack-nova | 12:31 | |
*** ociuhandu has quit IRC | 12:39 | |
*** ociuhandu has joined #openstack-nova | 12:47 | |
*** hemanth_n has quit IRC | 12:51 | |
*** ociuhandu has quit IRC | 12:52 | |
Yumeng | hi gibi, good afternoon. | 12:56 |
gibi | Yumeng: hi | 12:56 |
Yumeng | are you almost in holiday? :) | 12:56 |
Yumeng | I saw nova is going to cancel the next two weekly meeting. | 12:57 |
gibi | I'm off from next week | 12:57 |
gibi | Yumeng: yes, today is the last meeting this yera | 12:57 |
gibi | year | 12:57 |
Yumeng | wow, sounds excited and happy. | 12:58 |
Yumeng | I'm gonna catch you to discuss vGPU before you off. lol | 12:58 |
gibi | OK | 12:59 |
*** macz_ has joined #openstack-nova | 12:59 | |
Yumeng | current nova code has this to support get accel_info to resume guest state when a host is booted. https://review.opendev.org/c/openstack/nova/+/767273/1/nova/virt/libvirt/driver.py#3490 | 13:01 |
Yumeng | and your concern is if cyborg-agent service starts after nova-compute, nova can not get acc_info as expected,how should we solve this issue? | 13:01 |
sean-k-mooney | we shoudl not in nova, at least not entirely | 13:02 |
*** k_mouza_ has joined #openstack-nova | 13:02 | |
sean-k-mooney | the deployment tools shoudl use systemd before/after to order the service starts | 13:03 |
gibi | I guess we need to make clear in the doc that we have an service restart ordering dependency on cyborg | 13:03 |
gibi | other than that nova could simply fail to reboot those VMs during service startup that needs cyborg | 13:04 |
sean-k-mooney | well ideally the cyborg agent shoudl have a "before: nova-compute.service" requirement | 13:04 |
sean-k-mooney | not the other way around | 13:04 |
*** macz_ has quit IRC | 13:04 | |
sean-k-mooney | gibi: well we can and should call cyborgs api and do the async wait for the arq bidnigns | 13:04 |
sean-k-mooney | if the agent comes online in that tiem and responds it would be fine but yes we coudl skip them if not | 13:05 |
sean-k-mooney | the resume guest on host reboot feature does not work unless you use system to force the ordering anyway | 13:05 |
sean-k-mooney | libvirtd and openvswitch for example both need to be started before nova-compute | 13:05 |
Yumeng | "the resume guest on host reboot feature does not work unless you use system to force the ordering anyway" +1 | 13:06 |
gibi | Today we don't do async in this code path | 13:06 |
gibi | at least nova.compute.manager.ComputeManager._get_accel_info does not do that | 13:06 |
*** k_mouza has quit IRC | 13:06 | |
sean-k-mooney | are we just getting the info or rebining on reboot | 13:06 |
gibi | just getting infor | 13:07 |
gibi | no rebind as far as I see | 13:07 |
sean-k-mooney | if its just getting the info there is no depenciy on the agent being running | 13:07 |
sean-k-mooney | that info shoudl come form the db | 13:07 |
gibi | yeah, good point | 13:07 |
sean-k-mooney | if we however need the agent to create mdevs | 13:07 |
gibi | but I'm not familiar with cyborg internal arch | 13:07 |
sean-k-mooney | for exampel then we cant use GET | 13:07 |
sean-k-mooney | or at least if we did we would need to have cyborg reflect the status fo the binding as not active or complete | 13:08 |
Yumeng | I think we need use GET and also the agent to create mdevs | 13:08 |
sean-k-mooney | then we coudl poll or better wait for the externa event form the agent to signal cojmplete of its startup | 13:08 |
sean-k-mooney | like we do for spawn | 13:09 |
sean-k-mooney | Yumeng: did we reject or at lest strong advise agaisnt makeing GET magic in the cyborg api | 13:09 |
*** spatel has joined #openstack-nova | 13:09 | |
sean-k-mooney | where it would cause the agent to reporvison the attemnet/device if the host restarted | 13:10 |
sean-k-mooney | Yumeng: i though we were goign to reflect the status in the db and have the agent automaticaly create the devices on start up and signle that its not finised provioning in the api via a status field | 13:11 |
sean-k-mooney | gibi: that is why i tought we were doing the async event waiting by the way ^ | 13:11 |
gibi | ack | 13:12 |
sean-k-mooney | Yumeng: if we allow GET on the bindings to change state it basically means that cyborg cannot support the READONLY api personces as part of the keyston RBAC/policys effort | 13:13 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP db: Add machine_type to instance extras https://review.opendev.org/c/openstack/nova/+/767531 | 13:14 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP objects: Add machine_type to instance https://review.opendev.org/c/openstack/nova/+/767532 | 13:14 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Record the machine_type of instances during init_host https://review.opendev.org/c/openstack/nova/+/767533 | 13:14 |
lyarwood | gibi / sean-k-mooney / stephenfin ; http://paste.openstack.org/show/801124/ ^ I'm getting this on instance.save() and can't for the life of me work out why, any ideas? I think I'm missing something basic in the db layer. | 13:14 |
* sean-k-mooney clicks | 13:14 | |
*** spatel has quit IRC | 13:15 | |
Yumeng | sean-k-mooney: nope. it is not allowed to change sate in GET | 13:15 |
lyarwood | I *think* instance_update_and_get_original is trying to update machine_type in the instance db for some reason | 13:15 |
brinzhang_ | gibi, sean-k-mooney: we found this issue from you comments in vGPU support spec, depends on the "resume_guests_state_on_host_boot=True" config | 13:15 |
brinzhang_ | https://review.opendev.org/c/openstack/nova-specs/+/750116/9/specs/wallaby/approved/support-vGPU-nova-cyborg-interaction.rst#183 | 13:15 |
sean-k-mooney | lyarwood: ill take a look but we shoudl not have a machine_type field. it is ment to be img_machine_type in the instance_system_metadata table | 13:16 |
Yumeng | sean-k-mooney: "the agent automaticaly create the devices on start up" Does this mean cyborg will need another periodic task to sync arq in db with mdevs in the sys path? | 13:16 |
sean-k-mooney | so "mapper.column_attrs[key], value" looks wrong to me | 13:16 |
lyarwood | sean-k-mooney: the above series adds machine_type as a field to the instance_extras table etc | 13:16 |
lyarwood | sean-k-mooney: I've just missed something somewhere leading to this error when I set it in the instance object and try to save | 13:17 |
sean-k-mooney | lyarwood right but you shoudl not be doing that | 13:17 |
sean-k-mooney | lyarwood: wasnt the plan to have no db migraiton requireed for this | 13:18 |
sean-k-mooney | that is what we dicussed at the ptg | 13:18 |
*** k_mouza_ has quit IRC | 13:18 | |
lyarwood | I think you suggested adding this to system metadata but I then said it may as well go into instance extras | 13:19 |
lyarwood | it's in the spec as an instance extra | 13:19 |
openstackgerrit | Elod Illes proposed openstack/nova stable/stein: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766487 | 13:19 |
sean-k-mooney | ok i think that is not the right way to do this | 13:19 |
sean-k-mooney | at least not as a general pattern if we are adding more fileds | 13:19 |
sean-k-mooney | which is why it wanted it in system metadata to begin with | 13:20 |
sean-k-mooney | since we will likely want to do this again going forward when we change defaults | 13:20 |
*** k_mouza has joined #openstack-nova | 13:20 | |
*** k_mouza has quit IRC | 13:20 | |
lyarwood | what's the issue with this being an instance extra? | 13:20 |
*** k_mouza has joined #openstack-nova | 13:20 | |
sean-k-mooney | it will require a new column and db migration for every filed for not real value | 13:21 |
sean-k-mooney | if we use system metadata we dont require either | 13:21 |
sean-k-mooney | its also where the image metadata is currently sotred | 13:22 |
sean-k-mooney | so if we recored itn in img_machine_type we dont need to update any code that currently uses it | 13:22 |
sean-k-mooney | that way we dont miss anythign and it minimsies the code changes | 13:22 |
lyarwood | oh hell no, I'm not overloading img_machine_type | 13:22 |
sean-k-mooney | that was the whole point | 13:23 |
*** zzzeek has quit IRC | 13:23 | |
*** k_mouza_ has joined #openstack-nova | 13:23 | |
*** k_mouza has quit IRC | 13:23 | |
lyarwood | I didn't get that memo tbh, I've always wanted to track the machine type of the instance seperate to anything else | 13:23 |
lyarwood | so we can move it forward in the future without overwriting the image meta etc | 13:24 |
sean-k-mooney | we could use a different prefix in the system metadata | 13:24 |
sean-k-mooney | but im pretty stongly -1 on a new db column | 13:25 |
*** zzzeek has joined #openstack-nova | 13:25 | |
lyarwood | kk let me look into using it instead | 13:25 |
lyarwood | I honestly didn't think a new extra col for this would be a big deal | 13:26 |
lyarwood | given what's already in there | 13:26 |
*** mkrai has quit IRC | 13:26 | |
sean-k-mooney | well given we want to evenually recored the value of all image properties and the machine_type propsoal was just a scoped down verion of that i think its a problematic design | 13:26 |
lyarwood | it wasn't a scoped down version of that at all | 13:27 |
lyarwood | I think you're crossing wires here tbh | 13:27 |
sean-k-mooney | it was ment to be | 13:27 |
lyarwood | it was always just about recording the machine type of existing instances | 13:28 |
sean-k-mooney | thats what i discussed with stephen when we were deciding if i or you would write the spec | 13:28 |
*** k_mouza_ has quit IRC | 13:28 | |
lyarwood | the image properties are part of it but not all instances will have the machine type set that way | 13:28 |
sean-k-mooney | yes i know | 13:29 |
lyarwood | this was always about capturing the currently used machine type and stashing it somewhere | 13:29 |
sean-k-mooney | yep | 13:29 |
lyarwood | allowing the config to change | 13:29 |
lyarwood | I didn't take from that the need to use the image properties | 13:29 |
lyarwood | I wasn't even planning to look at the image properties tbh | 13:30 |
lyarwood | Just the defined domains | 13:30 |
sean-k-mooney | well i had a very differnet plan for implementing it then | 13:30 |
sean-k-mooney | i will also point out form a downstream perspecitve we may need to backport this to 16.2 to support 17 upgrades | 13:31 |
sean-k-mooney | that would depend on how we handel the defualt machine type in 17/wallaby | 13:32 |
sean-k-mooney | its intended to be q35 for all new installs | 13:32 |
lyarwood | I wasn't planning on backporting this | 13:32 |
sean-k-mooney | for upgrade we need to ensure that the machine type does not change | 13:32 |
lyarwood | new installs would be q35 | 13:32 |
lyarwood | upgrades would be the old default | 13:33 |
*** tesseract has quit IRC | 13:33 | |
lyarwood | until the env was fully upgraded | 13:33 |
lyarwood | and the original machine types stashed | 13:33 |
sean-k-mooney | ok but that means the ooo work is more involed but that is fine | 13:33 |
lyarwood | yeah it's a wrinkle we would need to document | 13:33 |
lyarwood | if we go with system metadata then I could backport it | 13:33 |
*** tesseract has joined #openstack-nova | 13:33 | |
sean-k-mooney | yep | 13:34 |
lyarwood | so I'll take another look at that | 13:34 |
*** hemanth_n has joined #openstack-nova | 13:35 | |
*** hemanth_n has quit IRC | 13:35 | |
gibi | lyarwood, sean-k-mooney: I barely followed the machine type dicussion above, if your decide on a change in direction then please propose a spec update accordingly | 13:41 |
sean-k-mooney | ack im not sure it sa change in direction at elast form the ptg discussion | 13:42 |
sean-k-mooney | https://etherpad.opendev.org/p/nova-wallaby-ptg line 196 is the aggreed statement | 13:42 |
sean-k-mooney | *796 | 13:42 |
gibi | sean-k-mooney: anyhow, please keep the spec in sync with the impl : | 13:43 |
gibi | :) | 13:43 |
sean-k-mooney | but the main delta is lyarwood is going to look at using system metatdata table instead of a new column which would be a change in spec | 13:43 |
gibi | ack | 13:43 |
sean-k-mooney | i guess its tbd how easy that is but i think it shoudl be a smaller code change in general | 13:44 |
*** k_mouza has joined #openstack-nova | 13:47 | |
*** k_mouza_ has joined #openstack-nova | 13:50 | |
*** k_mouza has quit IRC | 13:51 | |
*** k_mouza has joined #openstack-nova | 13:51 | |
*** k_mouza has quit IRC | 13:52 | |
*** k_mouza has joined #openstack-nova | 13:54 | |
*** k_mouza has quit IRC | 13:55 | |
*** k_mouza_ has quit IRC | 13:55 | |
*** k_mouza has joined #openstack-nova | 13:56 | |
*** nweinber has joined #openstack-nova | 13:58 | |
*** ociuhandu has joined #openstack-nova | 14:07 | |
lyarwood | sean-k-mooney: sorry I can't recall why I dropped the generic record all the metadata props part of this | 14:08 |
sean-k-mooney | it was likely not in the downstream bz/dod | 14:09 |
lyarwood | sean-k-mooney: I think it might be due to that approach not capturing the config side | 14:09 |
lyarwood | sean-k-mooney: yeah or that | 14:09 |
sean-k-mooney | it was going to capture the config too | 14:09 |
sean-k-mooney | the idea was to sotre the config value in the imgage prerty filed if it was not set | 14:09 |
sean-k-mooney | but its fine to strat with just machien_type | 14:09 |
sean-k-mooney | if its easy to repate the same pattern for the other value when we need them | 14:10 |
lyarwood | yup, let me rewrite this and update the spec | 14:11 |
gibi | ^^ +1 | 14:11 |
gibi | :) | 14:11 |
gibi | on the spec side | 14:11 |
lyarwood | yup sorry about this | 14:11 |
gibi | no problem on my side | 14:12 |
*** ociuhandu has quit IRC | 14:13 | |
*** ociuhandu has joined #openstack-nova | 14:17 | |
*** dklyle has quit IRC | 14:19 | |
*** ociuhandu_ has joined #openstack-nova | 14:20 | |
*** jkulik has quit IRC | 14:20 | |
*** lbragstad has quit IRC | 14:21 | |
*** ociuhandu has quit IRC | 14:23 | |
*** lbragstad has joined #openstack-nova | 14:23 | |
*** k_mouza has quit IRC | 14:29 | |
openstackgerrit | YumengBao proposed openstack/nova-specs master: libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116 | 14:39 |
*** mkrai has joined #openstack-nova | 14:40 | |
stephenfin | bauzas: any chance you could look at https://review.opendev.org/q/topic:%2522sriov-functional-tests%2522+status:open too | 14:40 |
stephenfin | ?? | 14:40 |
* bauzas clicks | 14:40 | |
Yumeng | gibi: just updated vGPU spec according to today's discuss. Please help to mention in nova's meeting today so cores can know about the update! need their review and comment,Thanks! https://review.opendev.org/c/openstack/nova-specs/+/750116 | 14:45 |
gibi | Yumeng: thanks for the update. I will mention the fact that the spec needs core review on today's meeting | 14:45 |
*** brinzhang0 has joined #openstack-nova | 14:46 | |
*** brinzhang_ has quit IRC | 14:49 | |
*** k_mouza has joined #openstack-nova | 14:51 | |
openstackgerrit | Lance Bragstad proposed openstack/placement master: Bump oslo.log version to 4.3.0 https://review.opendev.org/c/openstack/placement/+/760229 | 14:52 |
openstackgerrit | Lance Bragstad proposed openstack/placement master: Pass context objects to oslo.policy directly https://review.opendev.org/c/openstack/placement/+/766785 | 14:53 |
openstackgerrit | Aleksey Ulyanov proposed openstack/nova master: Fix graceful shutdown timeout https://review.opendev.org/c/openstack/nova/+/764927 | 14:54 |
*** sean-k-mooney has quit IRC | 14:56 | |
*** ociuhandu_ has quit IRC | 15:00 | |
*** ociuhandu has joined #openstack-nova | 15:00 | |
openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers https://review.opendev.org/c/openstack/placement/+/760240 | 15:02 |
openstackgerrit | Aleksey Ulyanov proposed openstack/nova master: Fix graceful shutdown timeout https://review.opendev.org/c/openstack/nova/+/764927 | 15:02 |
*** ociuhandu has quit IRC | 15:12 | |
*** ociuhandu has joined #openstack-nova | 15:13 | |
bauzas | Yumeng: gibi: saw the update but honestly, I won't have time for reviewing this large spec before end of this year | 15:13 |
bauzas | which is for me in 24 hours (with Friday afternoon full of meetings) | 15:13 |
gibi | bauzas: no worries | 15:17 |
gibi | Yumeng knows that we are just about to go for PTO for two weeks | 15:18 |
*** ociuhandu has quit IRC | 15:18 | |
*** sean-k-mooney has joined #openstack-nova | 15:21 | |
bauzas | gibi: actually, I lied, I just commented out loud | 15:23 |
bauzas | because I was horrified by what I saw | 15:24 |
*** nautik has joined #openstack-nova | 15:25 | |
*** ociuhandu has joined #openstack-nova | 15:33 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: WIP run-evacuate-hook: Use osc to evacuate instances https://review.opendev.org/c/openstack/nova/+/767570 | 15:35 |
gibi | sean-k-mooney: you were right and I was wrong. https://bugs.launchpad.net/nova/+bug/1823251 is a timeout issue really. Now I can reporduce it locally. | 15:36 |
openstack | Launchpad bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed] | 15:36 |
sean-k-mooney | gibi: oh ok so can we do what manial did or is it different? | 15:36 |
*** sapd1 has joined #openstack-nova | 15:36 | |
gibi | sean-k-mooney: I think we can try what manial did, but I'm affraid at some point we will run out the time budget for the whole test run | 15:37 |
gibi | test job run | 15:37 |
sean-k-mooney | you mentioned something about scale factor in the bug | 15:37 |
gibi | yeah, we have a 160sec per test case timeout | 15:37 |
sean-k-mooney | ah self.useFixture(timeout.Timeout( | 15:37 |
sean-k-mooney | scaling_factor=self.TIMEOUT_SCALING_FACTOR)) | 15:37 |
gibi | but the mysql tests uses 4 * 160 sec | 15:38 |
gibi | and still not enough | 15:38 |
*** macz_ has joined #openstack-nova | 15:38 | |
gibi | somehow the test VM is pretty IO limited | 15:38 |
sean-k-mooney | hum well they defalt to 300 second for manila | 15:38 |
sean-k-mooney | gibi: this only happens on limestone right | 15:39 |
gibi | the original report said so | 15:39 |
gibi | but I havent validated that recently | 15:39 |
sean-k-mooney | im not sure how new the hardware is, we have had io issue in the past where qos was limiting the disk to 100 IOPS on ovh? | 15:39 |
*** macz_ has quit IRC | 15:42 | |
gibi | sean-k-mooney: now I see more this happening on vexxhost-ca-ymq-1 provider | 15:45 |
*** mlavalle has joined #openstack-nova | 15:45 | |
gibi | but maybe it is just because we run test more there | 15:45 |
gibi | i dont know | 15:45 |
gibi | hm, here is a neutron migration test with postgres failing for the same reason https://b2b5edd1153190007318-36597f05313fd3fdc073d75819a76269.ssl.cf1.rackcdn.com/766204/7/check/neutron-functional-with-uwsgi/8533ed9/job-output.txt | 15:47 |
gibi | so it is not at all nova or mysql specific | 15:47 |
gibi | they have ~ 200sec timeout | 15:47 |
sean-k-mooney | ya it proably affect all project with a lot of migrations | 15:47 |
gibi | (nova has 160 for the api db test, and 640 for the nova db test) | 15:47 |
sean-k-mooney | stephenfin's patch shoudl help with that | 15:47 |
gibi | the squashing migration one? | 15:48 |
sean-k-mooney | yep | 15:48 |
sean-k-mooney | stephenfin: have you pushed that yet ^ | 15:48 |
gibi | nova meeting starts in 10 minutes... | 15:50 |
stephenfin | sean-k-mooney: yup, there's a whole series here https://review.opendev.org/q/topic:%22bp%252Fcompact-db-migrations-wallaby%22+(status:open%20OR%20status:merged) Just waiting for reviews :) | 15:53 |
gibi | stephenfin: nice | 15:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Rework some CPU pinning tests to use new config https://review.opendev.org/c/openstack/nova/+/767575 | 15:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for '[DEFAULT] vcpu_pin_set' https://review.opendev.org/c/openstack/nova/+/767576 | 15:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Drop support for pinning of host siblings https://review.opendev.org/c/openstack/nova/+/767577 | 15:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove '[workarounds] disable_fallback_pcpu_query' https://review.opendev.org/c/openstack/nova/+/767578 | 15:54 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Drop VCPU -> PCPU reshape https://review.opendev.org/c/openstack/nova/+/767579 | 15:54 |
gibi | I can check how much faster the test on my intentionally loaded machine with those compacted migrations | 15:55 |
stephenfin | sean-k-mooney: Series to drop 'vcpu_pin_set' there ^ | 15:56 |
*** dklyle has joined #openstack-nova | 15:56 | |
stephenfin | I gave up trying to keep the reshaper around and just removed it | 15:56 |
sean-k-mooney | ok ill try and look at both today i might start with the vcpu one | 15:57 |
openstackgerrit | Merged openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/c/openstack/nova/+/746950 | 16:05 |
*** markguz_ has quit IRC | 16:08 | |
*** macz_ has joined #openstack-nova | 16:09 | |
gmann | stephenfin: sean-k-mooney gibi any final direction on this ?, I am holding the code review for this which is very close to merge. https://review.opendev.org/c/openstack/nova-specs/+/765797/1/specs/wallaby/approved/modernize-os-hypervisors-api.rst#205 | 16:11 |
gibi | gmann: we will touch it today on the meeting | 16:12 |
gmann | gibi: sorry, forgot about meeting :) thanks for reminder | 16:13 |
openstackgerrit | Merged openstack/nova master: functional: Expand SR-IOV live migration tests with NUMA https://review.opendev.org/c/openstack/nova/+/749360 | 16:13 |
*** martinkennelly has joined #openstack-nova | 16:33 | |
martinkennelly | are you in charge of SDN room? | 16:33 |
*** mkrai has quit IRC | 16:35 | |
gibi | stephenfin: your db migration compating patches make the runtime of those test down by more than 75% | 16:48 |
stephenfin | hurrah | 16:48 |
stephenfin | that makes sense tbf | 16:48 |
stephenfin | there is also an API DB compaction, but I guess those are separate tests | 16:49 |
gibi | yes, there are unit tests for the nova db and functional test for the api db (dont ask why) | 16:49 |
gibi | I measured the nova db ones now | 16:50 |
gibi | anyhow your series will eventually solve https://bugs.launchpad.net/nova/+bug/1823251 | 16:50 |
openstack | Launchpad bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed] | 16:50 |
gibi | I think | 16:50 |
gibi | but until that I will push timeout increases | 16:50 |
openstackgerrit | Ghanshyam proposed openstack/nova master: DNM: testing minbw new tests https://review.opendev.org/c/openstack/nova/+/767587 | 16:57 |
gibi | gmann: good idea ^^ | 17:01 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Increase timeout for db migration tests https://review.opendev.org/c/openstack/nova/+/767590 | 17:06 |
gibi | sean-k-mooney, stephenfin ^^ | 17:07 |
stephenfin | LGTM. Do you want to run that a couple of times through CI or merge and let it test itself? | 17:08 |
gibi | stephenfin: merge it then we can revert if it does not help | 17:09 |
gibi | :) | 17:09 |
stephenfin | ack, +2 from moi | 17:09 |
sean-k-mooney | ah so we increase the scaling factor for the migration and decresss it for other test | 17:09 |
sean-k-mooney | oh no | 17:10 |
gibi | sean-k-mooney: nope, it is increase it for all our migration test just different amount | 17:10 |
sean-k-mooney | we set it to 5 for the main db and 2 for the api | 17:10 |
gibi | the api db mifration tests were defaulted to 1 | 17:10 |
sean-k-mooney | right so increased for both | 17:10 |
sean-k-mooney | ok cool | 17:10 |
*** belmoreira has quit IRC | 17:10 | |
gibi | yepp | 17:10 |
sean-k-mooney | and you have reference stephens series too | 17:11 |
sean-k-mooney | ya this looks good to me | 17:11 |
gibi | if it doesn't really help then I can try to look at serializing them | 17:11 |
gibi | but it would be better to merge stephen's series instead | 17:11 |
gibi | anyhow this is the end of my day see you all tomorrow | 17:12 |
stephenfin | o/ | 17:12 |
* stephenfin is on PTO again tomorrow, fwiw, so have a good weekend o/ | 17:12 | |
gibi | stephenfin: o/ | 17:12 |
*** k_mouza has quit IRC | 17:17 | |
*** k_mouza has joined #openstack-nova | 17:18 | |
*** ralonsoh has quit IRC | 17:23 | |
*** jawad_axd has quit IRC | 17:33 | |
*** jawad_axd has joined #openstack-nova | 17:33 | |
*** jawad_axd has quit IRC | 17:35 | |
*** jawad_axd has joined #openstack-nova | 17:36 | |
*** jawad_axd has quit IRC | 17:41 | |
*** ociuhandu_ has joined #openstack-nova | 17:41 | |
*** ociuhandu has quit IRC | 17:44 | |
*** ociuhandu_ has quit IRC | 17:45 | |
*** gyee has joined #openstack-nova | 17:49 | |
*** hamalq has joined #openstack-nova | 17:50 | |
*** hamalq_ has joined #openstack-nova | 17:52 | |
*** derekh has quit IRC | 17:53 | |
*** hamalq has quit IRC | 17:56 | |
gmann | gibi: yeah, we can check in advance specially if those new test needs to skip in multi-cell job | 17:57 |
*** vishalmanchanda has quit IRC | 18:06 | |
*** rpittau is now known as rpittau|afk | 18:20 | |
sean-k-mooney | artom: i see you added your self to https://review.opendev.org/c/openstack/nova/+/767368 | 18:21 |
sean-k-mooney | that is required before https://review.opendev.org/c/openstack/neutron/+/766277 can merge on the neutron side | 18:22 |
sean-k-mooney | and we need to backport it to at least train | 18:22 |
*** martinkennelly has quit IRC | 18:22 | |
sean-k-mooney | its related to https://review.opendev.org/c/openstack/nova/+/602432 also | 18:23 |
sean-k-mooney | we are currenly up to 6 pathces we need to backport to fix live migration | 18:23 |
sean-k-mooney | 3 on nova and 3 on neutron | 18:23 |
sean-k-mooney | 2 are merged on the neutron side already and 1 on the nova side | 18:24 |
sean-k-mooney | if we need to go before train we will need to backport thing in os-vif also | 18:24 |
sean-k-mooney | anyway im going to be calling it a day soon then going on PTO till january so ill respond it comments when im back if there are any | 18:25 |
sean-k-mooney | gibi: just saw this in the conflicts list https://review.opendev.org/c/openstack/nova/+/709658 we proably should arcive that just to not distract peopel with it | 18:27 |
*** tesseract has quit IRC | 18:28 | |
*** k_mouza has quit IRC | 18:29 | |
sean-k-mooney | oh enjoy the break all o/ i may or may not be around over the next 2 weeks but im on pto until the 4th of january | 18:29 |
lyarwood | \o enjoy either way, catch you in the new year | 18:55 |
* lyarwood -> drops | 18:55 | |
artom | sean-k-mooney, yeah, seemed like a thing I need to look at | 19:10 |
artom | (Sorry for the lag, was doing some errands and getting a haircut, before Shutdown 2 Electric Boogalloo after Christmas) | 19:10 |
sean-k-mooney | no worries | 19:11 |
*** andrewbonney has quit IRC | 19:11 | |
sean-k-mooney | tl;dr it only every worked because of buggy behavior in neutron that when fix means we need to actully wait properly in nova | 19:11 |
sean-k-mooney | basically we were migration before because fo a network-vif-plugged event form the dhcp agent | 19:12 |
sean-k-mooney | not form the l2 agent that we were waiting for | 19:12 |
artom | sean-k-mooney, I suspect this'll be another "Artom and Sean spent 2 hours on GMeet" kind of situation :) | 19:12 |
sean-k-mooney | about an hour and with radolfo but ya | 19:12 |
sean-k-mooney | i had a suption this was true for a while but we found the atcual code that was at fault friday last week | 19:13 |
sean-k-mooney | we have had a much of unxepcect event in the logs for a few releases and this is why many of them happened | 19:14 |
sean-k-mooney | its all related to the multipe port binding work in rocky | 19:14 |
*** dviroel has quit IRC | 19:43 | |
*** spatel has joined #openstack-nova | 20:06 | |
*** slaweq has quit IRC | 20:59 | |
*** ociuhandu has joined #openstack-nova | 21:35 | |
*** k_mouza has joined #openstack-nova | 21:39 | |
*** nweinber has quit IRC | 21:40 | |
*** k_mouza has quit IRC | 21:47 | |
*** ociuhandu has quit IRC | 21:59 | |
*** tbachman has quit IRC | 22:01 | |
*** tbachman has joined #openstack-nova | 22:03 | |
openstackgerrit | Ghanshyam proposed openstack/placement master: Remove deprecated [placement]/policy_file config option https://review.opendev.org/c/openstack/placement/+/764409 | 22:12 |
openstackgerrit | Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394 | 22:12 |
*** rcernin has joined #openstack-nova | 22:15 | |
*** ociuhandu has joined #openstack-nova | 22:17 | |
*** slaweq has joined #openstack-nova | 22:21 | |
*** ociuhandu has quit IRC | 22:28 | |
*** ociuhandu has joined #openstack-nova | 22:29 | |
*** slaweq has quit IRC | 22:33 | |
*** ociuhandu has quit IRC | 22:33 | |
openstackgerrit | Ghanshyam proposed openstack/nova stable/rocky: DNM: testing gate https://review.opendev.org/c/openstack/nova/+/767027 | 22:38 |
openstackgerrit | melanie witt proposed openstack/nova master: Add functional regression test for bug 1853009 https://review.opendev.org/c/openstack/nova/+/695012 | 22:38 |
openstack | bug 1853009 in OpenStack Compute (nova) ussuri "Ironic node rebalance race can lead to missing compute nodes in DB" [High,In progress] https://launchpad.net/bugs/1853009 - Assigned to Mark Goddard (mgoddard) | 22:38 |
openstackgerrit | melanie witt proposed openstack/nova master: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/c/openstack/nova/+/695187 | 22:38 |
openstackgerrit | melanie witt proposed openstack/nova master: Invalidate provider tree when compute node disappears https://review.opendev.org/c/openstack/nova/+/695188 | 22:38 |
openstackgerrit | melanie witt proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/c/openstack/nova/+/694802 | 22:38 |
openstackgerrit | melanie witt proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/c/openstack/nova/+/695189 | 22:38 |
*** martinkennelly has joined #openstack-nova | 22:43 | |
*** dviroel has joined #openstack-nova | 22:58 | |
*** ociuhandu has joined #openstack-nova | 23:00 | |
openstackgerrit | Lance Bragstad proposed openstack/placement master: Bump oslo.log version to 4.3.0 https://review.opendev.org/c/openstack/placement/+/760229 | 23:07 |
openstackgerrit | Lance Bragstad proposed openstack/placement master: Pass context objects to oslo.policy directly https://review.opendev.org/c/openstack/placement/+/766785 | 23:07 |
openstackgerrit | Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers https://review.opendev.org/c/openstack/placement/+/760240 | 23:07 |
*** ociuhandu has quit IRC | 23:12 | |
*** lbragstad has quit IRC | 23:18 | |
*** raildo has quit IRC | 23:31 | |
*** tbachman has quit IRC | 23:31 | |
*** tbachman has joined #openstack-nova | 23:34 | |
*** ociuhandu has joined #openstack-nova | 23:41 | |
*** tosky has quit IRC | 23:43 | |
*** ociuhandu has quit IRC | 23:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!