*** k_mouza has quit IRC | 00:01 | |
*** hoonetorg has quit IRC | 00:04 | |
*** tetsuro has joined #openstack-nova | 00:10 | |
*** k_mouza has joined #openstack-nova | 00:12 | |
*** k_mouza has quit IRC | 00:17 | |
*** hoonetorg has joined #openstack-nova | 00:17 | |
*** hoonetorg has quit IRC | 00:36 | |
*** gyee has quit IRC | 00:52 | |
*** brinzhang has joined #openstack-nova | 00:57 | |
*** brinzhang_ has joined #openstack-nova | 01:33 | |
*** brinzhang has quit IRC | 01:36 | |
*** bbowen has joined #openstack-nova | 01:40 | |
*** mkrai has joined #openstack-nova | 01:48 | |
*** spatel has joined #openstack-nova | 01:55 | |
*** brinzhang0 has joined #openstack-nova | 01:56 | |
*** brinzhang has joined #openstack-nova | 01:57 | |
*** brinzhang_ has quit IRC | 01:59 | |
*** brinzhang0 has quit IRC | 02:00 | |
*** brinzhang_ has joined #openstack-nova | 02:17 | |
*** brinzhang has quit IRC | 02:20 | |
*** efried has quit IRC | 02:20 | |
*** spatel has quit IRC | 02:34 | |
*** efried has joined #openstack-nova | 02:36 | |
*** efried has quit IRC | 02:42 | |
*** spatel has joined #openstack-nova | 02:51 | |
*** psachin has joined #openstack-nova | 02:59 | |
*** tetsuro has quit IRC | 03:04 | |
*** mkrai has quit IRC | 03:26 | |
*** mkrai has joined #openstack-nova | 03:33 | |
openstackgerrit | zhangboye proposed openstack/nova-specs master: migrate testing to ubuntu focal https://review.opendev.org/740863 | 03:43 |
---|---|---|
*** sapd1 has quit IRC | 03:54 | |
*** iurygregory has quit IRC | 03:54 | |
*** TheJulia has quit IRC | 03:54 | |
*** sapd1_x has joined #openstack-nova | 03:54 | |
*** TheJulia has joined #openstack-nova | 03:55 | |
*** sapd__x has joined #openstack-nova | 03:57 | |
*** psachin has quit IRC | 04:07 | |
*** ratailor has joined #openstack-nova | 04:20 | |
*** tetsuro has joined #openstack-nova | 04:22 | |
*** tetsuro_ has joined #openstack-nova | 04:26 | |
*** tetsuro has quit IRC | 04:27 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-nova | 04:33 | |
*** mkrai has quit IRC | 04:40 | |
*** tetsuro_ has quit IRC | 04:41 | |
*** tetsuro has joined #openstack-nova | 04:46 | |
*** psachin has joined #openstack-nova | 04:46 | |
*** tetsuro has quit IRC | 04:58 | |
*** ociuhandu has joined #openstack-nova | 05:01 | |
*** ociuhandu has quit IRC | 05:05 | |
*** spatel has quit IRC | 05:09 | |
*** udesale has joined #openstack-nova | 05:29 | |
*** psachin has quit IRC | 05:29 | |
openstackgerrit | Takashi Kajinami proposed openstack/nova master: Change default num_retries for glance to 3 https://review.opendev.org/740389 | 05:33 |
*** psachin has joined #openstack-nova | 05:39 | |
*** vishalmanchanda has joined #openstack-nova | 05:41 | |
*** links has joined #openstack-nova | 05:50 | |
*** sapd__x has quit IRC | 05:59 | |
*** elod is now known as elod_off | 06:15 | |
*** ccamacho has joined #openstack-nova | 06:18 | |
*** hoonetorg has joined #openstack-nova | 06:19 | |
*** dklyle has quit IRC | 06:39 | |
*** jsuchome has joined #openstack-nova | 06:42 | |
*** tony_su has joined #openstack-nova | 06:48 | |
*** rcernin has quit IRC | 07:05 | |
*** maciejjozefczyk has joined #openstack-nova | 07:10 | |
*** iurygregory_ has joined #openstack-nova | 07:10 | |
*** kevinz has joined #openstack-nova | 07:26 | |
*** iurygregory_ is now known as iurygregory | 07:31 | |
*** ralonsoh has joined #openstack-nova | 07:31 | |
*** nightmare_unreal has joined #openstack-nova | 07:31 | |
*** ccamacho has quit IRC | 07:32 | |
*** tosky has joined #openstack-nova | 07:37 | |
*** xek has joined #openstack-nova | 07:40 | |
*** bhagyashris|afk is now known as bhagyashris | 07:44 | |
*** sapd__x has joined #openstack-nova | 07:55 | |
*** avolkov has joined #openstack-nova | 07:55 | |
*** dtantsur|afk is now known as dtantsur | 07:56 | |
*** Habib-A has joined #openstack-nova | 08:01 | |
Habib-A | Hello all please help :D https://ask.openstack.org/en/question/128468/windows-server-2019-with-ovs-and-nova-compute/ | 08:02 |
*** rcernin has joined #openstack-nova | 08:10 | |
*** rcernin has quit IRC | 08:14 | |
*** rcernin has joined #openstack-nova | 08:15 | |
*** mkrai has joined #openstack-nova | 08:21 | |
gibi | kashyap: hi! you might have knowledge that I could use. I see a strange behavior from libvirt / qemu . with libvirt 4.0.0 and qemu 2.11 I can only hotplug an SRIOV VF to a guest if I detach the VF first from the host even if I specify managed=yes in the xml. The domain create works, libvirt automatically detach the device from the host. | 08:26 |
kashyap | gibi: Mornin | 08:27 |
gibi | kashyap: then I upgraded to libvirt 6.0.0 and qemu 4.2 and now hotplug also works without a manual detach | 08:27 |
gibi | I looked at the changelog of libvirt but nothing pops up | 08:27 |
kashyap | gibi: So a "feature was introduced silently"? :-) | 08:28 |
gibi | I think it is more like bugfix than a feature | 08:28 |
*** martinkennelly has joined #openstack-nova | 08:29 | |
kashyap | gibi: A question: is the libvirt-6.0.0 & qemu-4.2 behaviour desirable to you, or not? (Let's keep the surprise part aside :)) | 08:29 |
kashyap | Right, I first wrote 'bug-fix' indeed | 08:29 |
gibi | desirable | 08:29 |
gibi | very mutch | 08:29 |
gibi | I'm working on nova to support sriov attach | 08:29 |
gibi | and for that this behavior is needed | 08:30 |
kashyap | gibi: So you want to know whether this is intended or not? | 08:30 |
gibi | I want to know what is the minimum libvirt / qemu version that this behavior exists | 08:30 |
*** brinzhang_ has quit IRC | 08:31 | |
*** brinzhang0 has joined #openstack-nova | 08:31 | |
kashyap | gibi: Ah, good one; I can check w/ one of the upstream libvirt folks | 08:31 |
gibi | kashyap: thanks a lot! | 08:32 |
kashyap | gibi: I'll get back to you in a bit. Today also I need to finish some administrative paperwork (deadline is 16 Jul; dammit). But it's on my to-do | 08:32 |
gibi | kashyap: thanks. it is not urgent | 08:32 |
kashyap | gibi: But yeah; silently introducing such a user-visible bug fix without even a changelog is not nice | 08:33 |
gibi | I'm glad they fixed it :) | 08:38 |
*** martinkennelly has quit IRC | 08:41 | |
*** rcernin has quit IRC | 08:42 | |
*** martinkennelly has joined #openstack-nova | 08:42 | |
*** derekh has joined #openstack-nova | 08:48 | |
*** ociuhandu has joined #openstack-nova | 08:50 | |
*** donnyd has quit IRC | 08:53 | |
*** donnyd has joined #openstack-nova | 08:53 | |
*** ccamacho has joined #openstack-nova | 09:00 | |
*** brinzhang_ has joined #openstack-nova | 09:05 | |
*** brinzhang0 has quit IRC | 09:08 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Use constants for libvirt version checks https://review.opendev.org/740921 | 09:09 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for Intel CMT events https://review.opendev.org/740922 | 09:09 |
*** xek has quit IRC | 09:12 | |
*** jangutter_ has joined #openstack-nova | 09:28 | |
*** jangutter has quit IRC | 09:31 | |
stephenfin | gibi: You're working on SR-IOV attach, you say? | 09:33 |
stephenfin | That's interesting, because that's on the backlogs of sean-k-mooney and I for quite some time now, and we simply hadn't gotten to it yet | 09:33 |
*** k_mouza has joined #openstack-nova | 09:35 | |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267 | 09:37 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: refactor the code for CPU policy sanity check https://review.opendev.org/740925 | 09:37 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: refactor the code for CPU policy sanity check https://review.opendev.org/740925 | 09:40 |
stephenfin | gibi: I parroted your question back to danpb, given kashyap was occupied - as far as he's concerned, the behaviour you saw on the older version is bogus and reflects a bug | 09:40 |
stephenfin | "as managed=yes stuff has been around forever and never required a manual detach" | 09:41 |
kashyap | sfinucan: Thanks; you checked off a TODO on my list :) | 09:41 |
* kashyap crawls back into his cave | 09:41 | |
*** frickler is now known as frickler_pto | 09:44 | |
*** frickler_pto is now known as frickler | 09:47 | |
*** dtantsur is now known as dtantsur|bbl | 09:49 | |
*** tkajinam has quit IRC | 10:12 | |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: [DNM] bump nova-compute version for mixed instance in API https://review.opendev.org/740933 | 10:12 |
huaqiang | stephenfin: I have sent out another ongoing change as a possible alternative of https://review.opendev.org/716271 | 10:14 |
huaqiang | https://review.opendev.org/716271 checks the nova-compute service version in conductor layer | 10:14 |
huaqiang | while https://review.opendev.org/740933 is doing this in API layer | 10:15 |
huaqiang | just for comparison | 10:15 |
*** Habib-A has quit IRC | 10:15 | |
gibi | stephenfin: yeah, I did not assigned the bp or file a spec for it as I did not know how much time I will have during this cycle. So basically I spent yesterday putting a PoC together that started working today morning. I will do the paperwork at some point before M2 | 10:19 |
gibi | regarding the libvirt bug, it would be nice to know what is the minimum libvirt version that works for documentation purposes | 10:20 |
*** gouthamr has quit IRC | 10:22 | |
*** gouthamr has joined #openstack-nova | 10:23 | |
*** Jeffrey4l has quit IRC | 10:23 | |
*** Jeffrey4l has joined #openstack-nova | 10:24 | |
stephenfin | huaqiang: ack, will look | 10:28 |
stephenfin | gibi: Okay, cool. I know Sean and I would be keen to take a look at that, so if/when you do lemme know | 10:29 |
stephenfin | I can't help with testing at the moment since my SR-IOV machines are at home (I'm staying with my folks in the countryside since March), but hopefully it was just a single bad version of libvirt. Can probably test by building various versions from source | 10:30 |
stephenfin | Once I get access to those machines, I could do that testing | 10:30 |
*** jangutter has joined #openstack-nova | 10:34 | |
*** jangutter_ has quit IRC | 10:37 | |
gibi | stephenfin: OK, thanks. I have access to an SRIOV capable machine so I can try different libvirt version too | 10:39 |
*** mkrai has quit IRC | 10:49 | |
gibi | I think I will be able to post some WIP code this week | 10:50 |
*** martinkennelly has quit IRC | 10:53 | |
*** mkrai has joined #openstack-nova | 11:13 | |
*** rcernin has joined #openstack-nova | 11:13 | |
stephenfin | huaqiang: Okay, I prefer https://review.opendev.org/#/c/740933/ | 11:20 |
stephenfin | May be wise to get someone that knows RPC better than I do (dansmith or gibi, perhaps?) to weigh in | 11:21 |
stephenfin | lyarwood: I had my message asking you to look at the vTPM work already typed out before I checked Gerrit 😅 Thanks for the reviews | 11:25 |
*** mkrai has quit IRC | 11:31 | |
*** mkrai has joined #openstack-nova | 11:32 | |
gibi | stephenfin, huaqiang: looking at the alternatives I think I would go with the check in the api service | 11:32 |
stephenfin | gibi: so you mean prefer https://review.opendev.org/#/c/740933/ over https://review.opendev.org/716271 ? | 11:33 |
gibi | yes I prefer https://review.opendev.org/#/c/740933/ | 11:34 |
gibi | it is a bit more explicit what operations we block if the check is in the API | 11:37 |
gibi | and also understanding the code is easier for me if the check is not burried down in a long call path | 11:38 |
openstackgerrit | Merged openstack/nova stable/stein: Check cherry-pick hashes in pep8 tox target https://review.opendev.org/737922 | 11:39 |
openstackgerrit | Jinsheng Zhang proposed openstack/nova-specs master: Add nova-support-multiple-boot-volume-with-boot-order-selection spec https://review.opendev.org/737897 | 11:41 |
huaqiang | stephenfin and gibi, thanks for comments! | 11:45 |
*** iurygregory has quit IRC | 11:46 | |
*** sapd__x has quit IRC | 11:50 | |
*** spatel has joined #openstack-nova | 11:58 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Attempt to restore some sanity to snapshot https://review.opendev.org/740336 | 11:59 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Use better variable names for '_create_guest' https://review.opendev.org/740945 | 11:59 |
*** iurygregory has joined #openstack-nova | 12:00 | |
*** raildo has joined #openstack-nova | 12:02 | |
*** spatel has quit IRC | 12:03 | |
stephenfin | lyarwood: replied at https://review.opendev.org/#/c/739209/4 | 12:09 |
* stephenfin goes for lunch | 12:09 | |
lyarwood | stephenfin: just back sorry | 12:21 |
lyarwood | stephenfin: ack thanks | 12:21 |
*** udesale_ has joined #openstack-nova | 12:22 | |
*** xek has joined #openstack-nova | 12:23 | |
*** derekh has quit IRC | 12:23 | |
*** udesale has quit IRC | 12:25 | |
*** mkrai has quit IRC | 12:25 | |
*** artom has quit IRC | 12:43 | |
*** artom has joined #openstack-nova | 12:44 | |
*** artom has joined #openstack-nova | 12:45 | |
*** rcernin has quit IRC | 12:48 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: Silence amqp heartbeat warning https://review.opendev.org/728057 | 12:55 |
*** Luzi has joined #openstack-nova | 12:55 | |
lyarwood | melwitt: https://review.opendev.org/#/c/732708/ - would you mind hitting this once you're online today, the stable/stein change has merged. | 12:57 |
lyarwood | and the func test below it please :) | 12:57 |
*** spatel has joined #openstack-nova | 12:58 | |
*** efried has joined #openstack-nova | 13:00 | |
*** derekh has joined #openstack-nova | 13:03 | |
*** eharney has joined #openstack-nova | 13:11 | |
*** xek has quit IRC | 13:13 | |
*** Luzi has quit IRC | 13:29 | |
*** benj_ has quit IRC | 13:39 | |
*** benj_ has joined #openstack-nova | 13:41 | |
*** psachin has quit IRC | 13:44 | |
*** Luzi has joined #openstack-nova | 13:45 | |
*** dtantsur|bbl is now known as dtantsur | 13:45 | |
*** sean-k-mooney has joined #openstack-nova | 13:46 | |
*** ralonsoh has quit IRC | 13:49 | |
*** frickler is now known as frickler_pto | 13:50 | |
*** mlavalle has joined #openstack-nova | 13:58 | |
*** ralonsoh has joined #openstack-nova | 14:00 | |
*** sapd__x has joined #openstack-nova | 14:00 | |
*** dave-mccowan has joined #openstack-nova | 14:01 | |
*** maciejjozefczyk has quit IRC | 14:01 | |
mnaser | so neutron makes 0 use of wrap_db_retry and has ssome convluted 4 year old retry mechanism which tries to avoid nested use of wrap_db_retry (well, if it detects it, it removes it, sounds like a workaround to me) | 14:06 |
mnaser | does nova have some policy on what is retried and what isn't? | 14:06 |
dansmith | mnaser: we decorate the methods we want to retry, IIRC | 14:07 |
mnaser | dansmith: ok so there isn't a "if X then decorate" policy that i can try and adopt for neutron's db stuff | 14:08 |
dansmith | mnaser: I think we also don't have a lot of nesting of things that might be called from the top, or called from within, so no nesting | 14:08 |
dansmith | mnaser: yes, the policy is "if we get a bug for a thing, decorate it" :) | 14:08 |
mnaser | ok, so the current what-seems-to-be-neutron policy is... decorate everything with a decorator that removes an extra one if it finds one to avoid nested retries | 14:08 |
mnaser | so this is going to be very fun to unwind | 14:09 |
*** mriedem has joined #openstack-nova | 14:10 | |
*** ratailor has quit IRC | 14:17 | |
*** Luzi has quit IRC | 14:18 | |
*** knikolla has joined #openstack-nova | 14:21 | |
*** jobewan has joined #openstack-nova | 14:29 | |
*** dave-mccowan has quit IRC | 14:33 | |
*** dklyle has joined #openstack-nova | 14:34 | |
*** dave-mccowan has joined #openstack-nova | 14:37 | |
*** johnthetubaguy has quit IRC | 14:50 | |
*** johnthetubaguy has joined #openstack-nova | 14:52 | |
*** artom has quit IRC | 14:56 | |
dansmith | gmann: tres woot: https://review.opendev.org/#/c/734184/ | 14:56 |
*** artom has joined #openstack-nova | 14:56 | |
*** artom has quit IRC | 14:57 | |
*** artom has joined #openstack-nova | 14:57 | |
gmann | dansmith: nice, checking | 14:57 |
dansmith | (moved to -qa) | 14:58 |
*** artom has quit IRC | 14:59 | |
*** xek has joined #openstack-nova | 15:00 | |
*** artom has joined #openstack-nova | 15:01 | |
*** artom has quit IRC | 15:01 | |
*** artom has joined #openstack-nova | 15:01 | |
*** artom has quit IRC | 15:04 | |
*** xek has quit IRC | 15:19 | |
*** hamalq has joined #openstack-nova | 15:23 | |
*** mkrai has joined #openstack-nova | 15:24 | |
*** artom has joined #openstack-nova | 15:27 | |
*** hamalq has quit IRC | 15:29 | |
*** hamalq has joined #openstack-nova | 15:35 | |
*** udesale_ has quit IRC | 15:35 | |
*** mkrai has quit IRC | 15:39 | |
*** mkrai has joined #openstack-nova | 15:40 | |
*** gyee has joined #openstack-nova | 15:50 | |
*** spatel has quit IRC | 15:54 | |
*** mkrai has quit IRC | 15:57 | |
*** jsuchome has quit IRC | 15:59 | |
*** artom has quit IRC | 15:59 | |
*** links has quit IRC | 15:59 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: [WIP] Support SRIOV interface attach and detach https://review.opendev.org/740995 | 16:09 |
gibi | stephenfin, sean-k-mooney: this might be intresting for you ^^ | 16:10 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make our ceph job test with glance in multistore mode https://review.opendev.org/734184 | 16:12 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: notifications: add the pcpuset info to instance numa cell payload https://review.opendev.org/728480 | 16:42 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267 | 16:42 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: refactor the code for CPU policy sanity check https://review.opendev.org/740925 | 16:42 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: libvirt: apply mixed instance CPU policy https://review.opendev.org/728481 | 16:42 |
*** k_mouza has quit IRC | 16:46 | |
*** nightmare_unreal has quit IRC | 16:52 | |
*** ociuhandu_ has joined #openstack-nova | 16:58 | |
*** derekh has quit IRC | 16:58 | |
*** ociuhandu has quit IRC | 17:01 | |
*** huaqiang has quit IRC | 17:01 | |
*** ociuhandu_ has quit IRC | 17:02 | |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Calculate the CPU usage for mixed instance https://review.opendev.org/716269 | 17:08 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: scheduler: mixed instance CPU resource translating https://review.opendev.org/728482 | 17:08 |
*** huaqiang has joined #openstack-nova | 17:15 | |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Validate CPU pinning configuration for mixed instance https://review.opendev.org/716268 | 17:25 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Calculate the CPU usage for mixed instance https://review.opendev.org/716269 | 17:25 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: scheduler: mixed instance CPU resource translating https://review.opendev.org/728482 | 17:25 |
*** k_mouza has joined #openstack-nova | 17:25 | |
*** k_mouza has quit IRC | 17:30 | |
*** dtantsur is now known as dtantsur|afk | 17:31 | |
*** mgariepy has quit IRC | 17:36 | |
*** ralonsoh has quit IRC | 17:57 | |
*** xek has joined #openstack-nova | 18:04 | |
*** sapd__x has quit IRC | 18:18 | |
*** mgariepy has joined #openstack-nova | 18:22 | |
*** k_mouza has joined #openstack-nova | 18:32 | |
*** k_mouza has quit IRC | 18:37 | |
openstackgerrit | Merged openstack/nova master: utils: Move 'get_bdm_image_metadata' to nova.block_device https://review.opendev.org/739996 | 18:53 |
*** slaweq has joined #openstack-nova | 18:54 | |
*** slaweq has quit IRC | 18:56 | |
*** mgariepy has quit IRC | 19:03 | |
*** ociuhandu has joined #openstack-nova | 19:14 | |
*** ociuhandu has quit IRC | 19:23 | |
dansmith | sean-k-mooney: replied. thanks for reviewing and for your initial help in that of course | 20:30 |
*** k_mouza has joined #openstack-nova | 20:32 | |
sean-k-mooney | no worries | 20:33 |
sean-k-mooney | dansmith: ya makes sense to me | 20:36 |
sean-k-mooney | im glad you finally got it working by the way. | 20:36 |
sean-k-mooney | its always nice when the ci verifys something like this for us | 20:37 |
sean-k-mooney | the glance folks might even want to take a look for there own testing | 20:37 |
*** k_mouza has quit IRC | 21:06 | |
*** markvoelker has joined #openstack-nova | 21:23 | |
*** markvoelker has quit IRC | 21:27 | |
openstackgerrit | Merged openstack/nova stable/rocky: Add functional test for bug 1550919 https://review.opendev.org/732707 | 21:36 |
openstack | bug 1550919 in OpenStack Compute (nova) rocky "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood) | 21:36 |
*** spatel has joined #openstack-nova | 21:45 | |
dansmith | sean-k-mooney: yeah, they'll have a lot more surface they can test once my two devstack patches land.. I hope they will make use of that, but.. who knows | 21:54 |
*** spatel has quit IRC | 22:04 | |
*** spatel has joined #openstack-nova | 22:23 | |
*** spatel has quit IRC | 22:23 | |
sean-k-mooney | dansmith: well if we are testing it its better then nothing | 22:24 |
*** vishalmanchanda has quit IRC | 22:39 | |
*** rcernin has joined #openstack-nova | 22:44 | |
*** tosky has quit IRC | 22:50 | |
*** rcernin has quit IRC | 22:51 | |
*** rcernin has joined #openstack-nova | 22:51 | |
*** tkajinam has joined #openstack-nova | 22:58 | |
*** hamalq has quit IRC | 22:58 | |
*** mlavalle has quit IRC | 23:01 | |
openstackgerrit | Merged openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708 | 23:20 |
*** xek has quit IRC | 23:36 | |
mnaser | melwitt: i know you've worked some with the quota code. does nova have any locking on quota usages? neutron currently has a row-level lock on a resource/tenant_id which is causing me worlds of pain on highly parallelized port creation | 23:37 |
mnaser | it pretty much holds a lock the whole time (with a select for update) while it calculates the usage and updates... | 23:38 |
gmann | dansmith: seems oslo policy not able to format the target data for copy_image - https://zuul.opendev.org/t/openstack/build/aa1a5cd62aaa4210b19a5877e77b87db/log/controller/logs/screen-g-api.txt#600 | 23:46 |
gmann | trying to reproduce this locally to see if oslo trying to change 'hw_rng_model' for some reason. | 23:47 |
gmann | mnaser: not sure about lock (melwitt can confirm if we do have that) but nova does recheck quota again just before the resource creation to avoid(or to minimize as much as possible) any wrong calculation or over-quota in parallel request cases. | 23:52 |
mnaser | gmann: ok so nova doesnt do any row level locking for quota usage -- also yeah, i remember that, it's even an option to recheck quota in config if i remember right | 23:52 |
gmann | * after resource creation. this config - https://github.com/openstack/nova/blob/7385adb2f526166bb174dab055d060a428d4d4ff/nova/conf/quota.py#L189 | 23:53 |
gmann | mnaser: afaik, no locks but melwitt or dansmith can confirm it. | 23:56 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!