| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-13 | |||
| 19:14:38 | markguz_ | sean-k-mooney: i think it would be simpler for me to just upgrade to ussuri | |
| 19:14:58 | sean-k-mooney | if that is an option yes | |
| 19:15:37 | openstackgerrit | Merged openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/c/openstack/nova/+/761924 | |
| 19:15:43 | markguz_ | fortunately for me this is an internal deployment that is not used by paying customers so i have some degree of flexibility on it's availability | |
| 19:16:23 | sean-k-mooney | what do you use to deploy/manage it | |
| 19:21:11 | markguz_ | sean-k-mooney: originally i deployed kilo with rdo packstack. since then it's become a bit of a bit of hodgepodge of manual installs. I mostly use ansible to keep things up to date | |
| 19:21:48 | sean-k-mooney | ah i see | |
| 19:22:01 | sean-k-mooney | packstack has more or less been unsupported for a few years now | |
| 19:22:27 | sean-k-mooney | i think it still technicaly exists but redhat stop supporting in with our product in queens i think | |
| 19:22:47 | markguz_ | yeah. i generally just install from the package manager. and have some ansible plays that configure compute nodes etc etc. | |
| 19:22:49 | sean-k-mooney | e.g. we moved to require all customer deploy with triplo around queens | |
| 19:23:12 | markguz_ | i've got some bits an pieces that are installed from git master for things like magnum and heat. | |
| 19:23:13 | sean-k-mooney | markguz_: you should look into openstack ansible so | |
| 19:23:27 | markguz_ | it's on my todo list :-) | |
| 19:26:15 | markguz_ | The patch seems to have fixed my problem... spawning is happening at the expected rate now | |
| 19:26:47 | sean-k-mooney | did you just bump the oslo concurancy version and apply th nova patch | |
| 19:26:58 | markguz_ | sean-k-mooney: yup | |
| 19:27:12 | sean-k-mooney | there is still goign to be contention on the lock but it should now preserve order | |
| 19:27:35 | markguz_ | so spliting to multiple compute nodes is still probably the best path | |
| 19:30:15 | sean-k-mooney | long term proably but at leat your current issue is mitagated | |
| 19:30:29 | sean-k-mooney | i wont say solved but managemable | |
| 19:33:33 | openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: `socket` PCI NUMA-affinity Policy https://review.opendev.org/c/openstack/nova-specs/+/765551 | |
| 19:33:52 | artom | sean-k-mooney, stephenfin (though I suspect you're done for the day) ^^ | |
| 19:34:21 | sean-k-mooney | more or less ill leave it open however for tomorrow | |
| 19:34:35 | artom | ... with some snow shovelling and kid driving thrown in the mix | |
| 23:54:03 | openstackgerrit | Merged openstack/nova stable/stein: [stable-only] Cap bandit and make lower-constraints job non-voting https://review.opendev.org/c/openstack/nova/+/766487 | |
| #openstack-nova - 2021-01-14 | |||
| 00:41:48 | brinzhang_ | gmann: hi are you around? | |
| 00:42:08 | brinzhang_ | gmann: what's your mean of this https://review.opendev.org/c/openstack/nova/+/729563/26/nova/api/openstack/compute/shelve.py#59 | |
| 00:42:39 | brinzhang_ | gmann: do we need to merge your fix into the shelve/unshelve patch? | |
| 01:34:16 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs https://review.opendev.org/c/openstack/nova/+/768509 | |
| 02:14:48 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | |
| 02:47:25 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852 | |
| 04:32:32 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in os-quota-sets path https://review.opendev.org/c/openstack/nova/+/768851 | |
| 06:05:45 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116 | |
| 06:08:48 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenant_id with project_id in Limits API https://review.opendev.org/c/openstack/nova/+/768862 | |
| 06:08:49 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenant* with project* in codes https://review.opendev.org/c/openstack/nova/+/769329 | |
| 08:39:52 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Rebuild Server API https://review.opendev.org/c/openstack/nova/+/766380 | |
| 08:39:53 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726 | |
| 09:06:48 | gibi | good morning | |
| 09:25:30 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Warn when starting services with older than N-1 computes https://review.opendev.org/c/openstack/nova/+/770764 | |
| 09:25:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Add upgrade check about old computes https://review.opendev.org/c/openstack/nova/+/770765 | |
| 09:44:09 | lyarwood | sean-k-mooney: https://review.opendev.org/q/topic:spec/libvirt-default-machine-type - when you're around and have time can we have a chat about the direction of this series before I finish things up and write docs etc? | |
| 09:56:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1896463 in func env https://review.opendev.org/c/openstack/nova/+/770768 | |
| 09:56:37 | openstack | bug 1896463 in OpenStack Compute (nova) rocky "evacuation failed: Port update failed : Unable to correlate PCI slot " [Low,In progress] https://launchpad.net/bugs/1896463 | |
| 09:56:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Set instance host and drop migration under lock https://review.opendev.org/c/openstack/nova/+/770769 | |
| 11:31:58 | sean-k-mooney | lyarwood: https://review.opendev.org/c/openstack/nova/+/767533/4/nova/virt/libvirt/driver.py#5775 | |
| 11:33:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: utils: Remove periods from instance hostnames https://review.opendev.org/c/openstack/nova/+/764482 | |
| 11:34:16 | sean-k-mooney | lyarwood: over all that pathch looks good to me be the precidence relationship is wrong in my view | |
| 11:34:45 | sean-k-mooney | lyarwood: im going to go grab coffee and ill be back in a minute but after that im free to talk whenever suits | |
| 11:41:41 | lyarwood | sean-k-mooney: cool thanks, image, sysmeta and then config. I'll respin this afternoon. | |
| 11:47:27 | lyarwood | stephenfin: https://review.opendev.org/c/openstack/openstacksdk/+/767526 sorry about the spam on here btw, I'm using gibi's tampermonkey script to display zuul results and got confused when I saw failures from a previous run. | |
| 11:57:10 | stephenfin | lyarwood: heh, nw :) | |
| 12:08:16 | sean-k-mooney | stephenfin: https://review.opendev.org/c/openstack/python-openstackclient/+/765366 and the other osc changes faild the gate by the way | |
| 12:08:40 | sean-k-mooney | that one seams to have pagination issues for list servers which you did not change | |
| 12:08:51 | sean-k-mooney | it had a bunch of json decode errors | |
| 12:09:53 | sean-k-mooney | oh you have a recheck going | |
| 12:10:35 | sean-k-mooney | looks like the same error on all of them | |
| 12:35:08 | openstackgerrit | sean mooney proposed openstack/nova stable/victoria: only wait for plugtime events in pre-live-migration https://review.opendev.org/c/openstack/nova/+/770744 | |
| 12:37:31 | openstackgerrit | sean mooney proposed openstack/nova stable/ussuri: only wait for plugtime events in pre-live-migration https://review.opendev.org/c/openstack/nova/+/770745 | |
| 12:45:08 | openstackgerrit | sean mooney proposed openstack/nova stable/ussuri: only wait for plugtime events in pre-live-migration https://review.opendev.org/c/openstack/nova/+/770745 | |
| 14:03:39 | gibi | lyarwood: I'm wondering if I should do the cleanup MIN_LIBVIRT_VERSION >= 4.1.0 before I change the detach logic as that cleanup would simplyfy the code that I have to handle | |
| 14:03:43 | gibi | https://github.com/openstack/nova/blob/e6f5e814050a19d6f027037424556b2889514ec3/nova/virt/libvirt/guest.py#L418 | |
| 14:16:25 | lyarwood | gibi: hey sorry, so that's all part of https://review.opendev.org/q/topic:%22bump-libvirt-qemu-wallaby%22 if I ever get around to it | |
| 14:16:32 | lyarwood | gibi: which actually came up downstream earlier today | |
| 14:16:50 | lyarwood | gibi: I can refresh that and try to remove that logic for you later today if you'd like | |
| 14:16:55 | lyarwood | gibi: avoid adding another thing to your plate | |
| 14:16:57 | stephenfin | sean-k-mooney: Sorry, missed the ping. Yeah, the functional tests for OSC aren't great. They're not fully isolated which means running them in parallel can produce all kinds of weird results, depending on the speed on the CI node | |
| 14:17:19 | sean-k-mooney | ah i see | |
| 14:17:20 | stephenfin | sean-k-mooney: so rechecking is the only thing to do until I've time to rework them all | |
| 14:17:53 | sean-k-mooney | well that or run the serially but that proably will be too long | |
| 14:18:03 | stephenfin | yeah, and that just masks the problem | |
| 14:18:13 | sean-k-mooney | yep | |
| 14:18:25 | openstackgerrit | Merged openstack/nova master: tests: Clean up 'test_hypervisors' https://review.opendev.org/c/openstack/nova/+/769903 | |
| 14:18:44 | sean-k-mooney | it need to do what tempest does and create seperate tenant per senario when its doing things like testing list instnaces | |
| 14:18:53 | stephenfin | yes, exactly | |
| 14:19:05 | stephenfin | but I just haven't had time to write a fixture or whatever to do that | |
| 14:20:34 | lyarwood | just dumping this here while I work on something downstream | |
| 14:20:45 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/c/openstack/nova/+/754700 | |
| 14:20:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_{LIBVIRT,QEMU}_BLOCKDEV https://review.opendev.org/c/openstack/nova/+/754701 | |
| 14:20:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_LIBVIRT_VIR_ERR_DEVICE_MISSING https://review.opendev.org/c/openstack/nova/+/754702 | |
| 14:20:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_LIBVIRT_VTPM https://review.opendev.org/c/openstack/nova/+/754703 | |
| 14:20:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove MIN_LIBVIRT_S390X_CPU_COMPARE https://review.opendev.org/c/openstack/nova/+/754704 | |
| 14:20:51 | lyarwood | .. git-review really is slow since the update to gerrit .. | |
| 14:21:03 | stephenfin | I find it varies | |
| 14:21:11 | stephenfin | It's usually fast after a restart | |
| 14:21:28 | stephenfin | *feels fast | |
| 14:21:31 | stephenfin | I've no data to back up that assertion :-\ | |
| 14:23:03 | stephenfin | We're getting close to API microversion 2.100. It could be another cycle or two, but that's coming | |
| 14:30:10 | sean-k-mooney | at some point we might go to nova v4 | |
| 14:41:40 | dansmith | lyarwood: has seemed painfully slow to me as well | |
| 14:41:46 | dansmith | longer for multi-patch uploads | |
| 14:47:35 | gmann | brinzhang0: no, your patch is all ok as it is and fixed the 500 ->403, mine is to improve it as general when other API use this exception further with unit tests | |
| 14:48:34 | gmann | brinzhang0: to avoid conflict I made mine depends on top of your otherwise they both can go independent also | |
| 14:54:26 | sean-k-mooney | dansmith: i havent really noticed much of a change but i havent done many multi patch updates recently | |
| 14:59:58 | gibi | lyarwood: thanks for https://review.opendev.org/c/openstack/nova/+/754702 I will review that and rebase the libvirt event patch top of that | |
| 15:00:31 | lyarwood | gibi: I'll clean that series up later once I'm done with something downstream | |
| 15:00:38 | lyarwood | and np :) | |
| 15:01:33 | gibi | lyarwood: ack | |
| 15:29:15 | artom | stephenfin, when you have time, mind taking another look at https://review.opendev.org/c/openstack/nova-specs/+/765551 ? It feels awefully short to me, but I think I've covered everything...? | |
| 15:29:36 | gibi | lyarwood: as far as I see https://review.opendev.org/c/openstack/nova/+/754702 does not need to depend on the libvirt bump as our current minimum libvirt (5.0.0) already bigger than what is needed (4.1.0) for this patch | |