| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-12-08 | |||
| 09:55:13 | lyarwood | the upstream issue yesterday was https://review.opendev.org/q/I67255fa1b919a27e92028da95d71ddd4bf53edc1 | |
| 09:55:25 | lyarwood | that should hopefully land in stable/victoria shortly | |
| 09:56:13 | openstack | Launchpad bug 1907157 in grenade "Target volume type is still in use" [Undecided,New] | |
| 09:56:13 | lyarwood | https://bugs.launchpad.net/grenade/+bug/1907157 also came up yesterday for the first time | |
| 09:59:38 | lyarwood | brb | |
| 10:21:58 | openstackgerrit | Mamduh proposed openstack/os-vif stable/rocky: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765962 | |
| 11:07:06 | stephenfin | sean-k-mooney: What do you think of deprecating the old-style isolate emulator thread policy? The one where the instance gets an extra host core all to itself just for emulator threads? | |
| 11:09:58 | stephenfin | sean-k-mooney: Basically we could start warning in hardware.py if a host NUMACell object doesn't have anything in 'cpuset' (how we report VCPUs now) when scheduling an instance with the 'isolate' policy. You'd see that warning on the scheduler and compute nodes. We could then remove in Y or whatever | |
| 11:10:13 | gibi | hi! I got a plenty of "openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/85b94d33f4774bebb61b0a4f7f5ae5e0 : ERROR Failed to update project None in 7m 25s" today from zuul, is it a known issue? | |
| 11:12:00 | stephenfin | Looks like something to ask infra about? | |
| 11:12:06 | stephenfin | I haven't seen that before | |
| 11:12:16 | gibi | pinged infra | |
| 11:12:39 | gibi | there was a gerrit outage and restart this morning so it could be related to that | |
| 11:13:21 | stephenfin | Also might be related to the removal of the python-searchlight and python-quinling project. I know some of the openstackclient jobs rely on that, and we use some of those in gate iirc? | |
| 11:14:07 | gibi | this was a simple nova patch e.g. https://review.opendev.org/c/openstack/nova/+/757109 | |
| 11:17:56 | openstackgerrit | Mamduh proposed openstack/os-vif stable/queens: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765941 | |
| 11:22:16 | openstackgerrit | Mamduh proposed openstack/os-vif stable/ussuri: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765967 | |
| 11:24:59 | openstackgerrit | Mamduh proposed openstack/os-vif stable/train: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765970 | |
| 11:28:15 | openstackgerrit | Lucian Petrut proposed openstack/nova master: Fix unshelved image ref https://review.opendev.org/c/openstack/nova/+/765924 | |
| 11:37:58 | openstackgerrit | Mamduh proposed openstack/os-vif stable/stein: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765974 | |
| 11:46:39 | openstackgerrit | Mamduh proposed openstack/os-vif stable/rocky: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765977 | |
| 11:55:44 | openstackgerrit | Mamduh proposed openstack/os-vif stable/queens: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765983 | |
| 12:41:24 | lyarwood | stephenfin: https://review.opendev.org/c/openstack/nova-specs/+/752284 - would you mind hitting this again if you have time today? | |
| 13:08:18 | gibi | stephenfin, melwitt: sean-k-mooney and I are positive on the doc patch https://review.opendev.org/c/openstack/nova/+/765871 so one of you can send it through | |
| 13:28:24 | openstackgerrit | Matteo Sposato proposed openstack/nova master: Refactoring of functional.regression.test_bug_1702454 https://review.opendev.org/c/openstack/nova/+/765997 | |
| 14:10:49 | stephenfin | gibi: I'll let melwitt hit that again since she's reviewed it previously | |
| 14:28:31 | openstackgerrit | Merged openstack/nova-specs master: Image and flavor defined ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/752284 | |
| 14:34:31 | gmann | stephenfin: detail=false tests are missing in this or I am missing something https://review.opendev.org/c/openstack/nova/+/764039/1/nova/tests/unit/api/openstack/compute/test_hypervisors.py#285 | |
| 15:05:02 | gibi | stephenfin: fair point | |
| 15:10:20 | openstackgerrit | Merged openstack/nova master: tools: Allow check-cherry-picks.sh to be disabled by an env var https://review.opendev.org/c/openstack/nova/+/765802 | |
| 15:38:52 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: tools: Allow check-cherry-picks.sh to be disabled by an env var https://review.opendev.org/c/openstack/nova/+/766023 | |
| 15:39:17 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: tools: Allow check-cherry-picks.sh to be disabled by an env var https://review.opendev.org/c/openstack/nova/+/766024 | |
| 15:39:53 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: tools: Allow check-cherry-picks.sh to be disabled by an env var https://review.opendev.org/c/openstack/nova/+/766025 | |
| 15:54:03 | sean-k-mooney | gibi: was just speaking to slaweq in the neutron irc meeting about http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019240.html | |
| 15:54:36 | sean-k-mooney | im not sure if you have seen the thread. basically the cirros image we use in teh ci has a periodic failure due to an unpatched kernel bug | |
| 15:54:56 | sean-k-mooney | one workaround without replaceing cirros si to disable the ioapic | |
| 15:55:18 | sean-k-mooney | to do that we would have to add a image metadata flag and modify devstack to set it | |
| 15:55:33 | sean-k-mooney | any objection to tracking this as a bugfix or specless blueprint | |
| 15:56:38 | sean-k-mooney | although we might action want to do this via a config option come to think of it | |
| 15:57:17 | sean-k-mooney | since we likely woudl want ot backprot it so [workaround]/disable_ioapic=True|false(default) | |
| 16:35:14 | stephenfin | I assume this isn't solveable by simply using a different CI image, e.g. alpine | |
| 16:36:42 | sean-k-mooney | stephenfin: it is but alpine does not have cloudinit installed and i need to rework my disk image builder patches | |
| 16:37:10 | sean-k-mooney | we cant use fedora ro ubuntu as they are boot too large and too slow to use without nested virt | |
| 16:37:45 | sean-k-mooney | so long term im planing to try rebuilding my alpine images usign the iso image instead of the mini-fs image im currently using | |
| 16:37:56 | sean-k-mooney | to work around the initramfs issue im hitting | |
| 16:38:12 | sean-k-mooney | if i can get that to work then ill be propsosing using alpine instead of cirros | |
| 16:46:32 | sean-k-mooney | stephenfin: gibi slaweq i have a patch written just running test locally now | |
| 16:47:05 | gibi | sorry I'm bussy upstream | |
| 16:47:07 | gibi | will read back | |
| 16:49:34 | openstackgerrit | sean mooney proposed openstack/nova master: add libvirt_disable_apic workaround https://review.opendev.org/c/openstack/nova/+/766043 | |
| 16:49:35 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing libvirt_disable_apic workaround option https://review.opendev.org/c/openstack/nova/+/766044 | |
| 17:22:01 | gibi | sean-k-mooney, slaweq: thanks for looking into the cirros crash. | |
| 17:22:11 | gibi | sean-k-mooney: could os_command_line glance image propert helps? | |
| 17:27:14 | gibi | sean-k-mooney: hm, that seems to be only work with amazon type images :? | |
| 17:27:16 | gibi | :/ | |
| 17:40:39 | gibi | could we make that image property to work with other images | |
| 17:40:40 | gibi | ? | |
| 18:36:39 | openstackgerrit | Matteo Sposato proposed openstack/nova master: Functional tests remmoved direct post call https://review.opendev.org/c/openstack/nova/+/766068 | |
| 18:39:09 | sean-k-mooney | gibi: os_command_line only works if you use teh kernel seperatly | |
| 18:39:29 | sean-k-mooney | it also work for lxc | |
| 18:40:05 | sean-k-mooney | so to use it to set noapic we would have to use the split kernel and root disk image | |
| 18:41:01 | sean-k-mooney | i belive we could make it work with other image formats but we would still have to use seperate kernel image so that qemu can act as the bootloader and pass the arges instead of grub | |
| 18:45:33 | sean-k-mooney | gibi: that is using https://libvirt.org/formatdomain.html#direct-kernel-boot | |
| 19:00:33 | sean-k-mooney | this is where that is used https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L5285-L5297 | |
| 19:04:21 | sean-k-mooney | gibi: slaweq we could use http://download.cirros-cloud.net/0.5.1/cirros-0.5.1-x86_64-uec.tar.gz | |
| 19:04:48 | sean-k-mooney | that woudl give use a sperate inird and vmlinuz which would allow us to use os_command_line | |
| 19:25:09 | sean-k-mooney | gibi: slaweq https://review.opendev.org/c/openstack/devstack/+/766079 should work although i have not tested it | |
| 19:25:32 | sean-k-mooney | gibi: slaweq that uses os_command_line and the uae cirros images | |
| 19:25:39 | sean-k-mooney | to pass noapic | |
| 21:20:31 | openstackgerrit | Merged openstack/nova master: Enhance simple usage API documentation https://review.opendev.org/c/openstack/nova/+/765871 | |
| 22:22:21 | openstackgerrit | melanie witt proposed openstack/nova master: Disallow CONF.compute.max_disk_devices_to_attach = 0 https://review.opendev.org/c/openstack/nova/+/755360 | |
| #openstack-nova - 2020-12-09 | |||
| 01:38:11 | prometheanfire | any work on the updated mock support? https://review.opendev.org/765680 | |
| 02:30:34 | melwitt | prometheanfire: I resolved the remaining issues in the proposed patch but need someone to rebase it and handle the merge conflicts. I hoped stephenfin could help with that | |
| 02:30:48 | melwitt | I don't have as much of the context on the rest of the patch | |
| 02:35:35 | prometheanfire | melwitt: thanks for the update :D | |
| 02:37:22 | melwitt | np | |
| 03:42:15 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Update Servers APIs https://review.opendev.org/c/openstack/nova/+/764292 | |
| 03:42:16 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | |
| 03:42:16 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects in List Server APIs https://review.opendev.org/c/openstack/nova/+/765311 | |
| 07:25:06 | LarsErikP | melwitt: I noticed whis was merged a few hours ago \o/ can it be backported to ussuri? | |
| 08:06:00 | openstackgerrit | Mamduh proposed openstack/os-vif stable/ussuri: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765419 | |
| 08:12:42 | gibi | LarsErikP: sure, it can be backported. But please propose a backport to stable/victoria first then stable/ussuri | |
| 08:15:46 | LarsErikP | melwitt: Uh.. Don't know what happended there. Forgot the link: https://review.opendev.org/c/openstack/nova/+/759348/ | |
| 08:16:51 | LarsErikP | gibi: Uhm.. not sure howto do that. Maybe melwitt could do it, as she fixed this originally? =) | |
| 08:18:11 | openstackgerrit | Mamduh proposed openstack/os-vif stable/ussuri: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765967 | |
| 08:20:30 | openstackgerrit | Lars Erik Pedersen proposed openstack/nova stable/victoria: Omit resource inventories from placement update if zero https://review.opendev.org/c/openstack/nova/+/766177 | |
| 08:20:53 | LarsErikP | gibi: I found the button ^ :P | |
| 08:23:13 | LarsErikP | should I add reviewers? and.. there is merge conflicts in ussuri. not feeling comfortable to deal with that :S | |
| 08:25:53 | openstackgerrit | Mamduh proposed openstack/os-vif stable/train: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765912 | |
| 08:30:29 | gibi | LarsErikP:added some stable cores to the review. Thanks for proposing the backport | |
| 08:31:37 | gibi | LarsErikP: as for the ussuri backport, lets merge the victoria one first then I think melwitt can resolv the merge conflict in ussuri or I can take it if needed | |
| 08:32:13 | LarsErikP | gibi: good plan :-) thanks so much! | |
| 08:32:37 | gibi | LarsErikP: :) | |
| 08:44:37 | bauzas | good morning Nova | |
| 08:55:18 | gibi | bauzas: O/ | |
| 09:30:54 | openstackgerrit | Adrian Chiris proposed openstack/os-vif stable/ussuri: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765967 | |
| 09:46:15 | lyarwood | elod: morning, have you had anytime to look at the pip failures in stable before I start digging in? | |
| 09:47:59 | lyarwood | elod: https://review.opendev.org/q/Ia2007bc63ef09931ea0197cef29d6a5614ed821a - I was checking in on this series and noticed that everything prior to victoria is failing with pip 20.2.4 | |
| 09:48:02 | elod | lyarwood: good morning :) I'm looking at several issues now, which pip failures do you mean? :) | |
| 09:48:13 | lyarwood | elod: https://zuul.opendev.org/t/openstack/build/cb6247d4b3644045ab6d83a064e812c6 for example | |