| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-28 | |||
| 08:17:56 | lyarwood | let me fix this in devstack actually | |
| 08:18:07 | lyarwood | these jobs really need to be updated anyway | |
| 08:18:07 | lyarwood | these jobs really need to be updated anyway | |
| 08:20:35 | gibi | OK | |
| 08:20:51 | gibi | I continue review the patch that adds the new grenade job to nova | |
| 08:20:51 | gibi | I continue review the patch that adds the new grenade job to nova | |
| 08:35:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM - testing nova-grenade-multinode fix https://review.opendev.org/c/openstack/nova/+/788430 | |
| 08:35:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM - testing nova-grenade-multinode fix https://review.opendev.org/c/openstack/nova/+/788430 | |
| 08:39:35 | lyarwood | gibi: https://review.opendev.org/c/openstack/devstack/+/788429 should fix all jobs | |
| 08:39:35 | lyarwood | gibi: https://review.opendev.org/c/openstack/devstack/+/788429 should fix all jobs | |
| 08:45:39 | gibi | does that ^^ superseeds https://review.opendev.org/c/openstack/nova/+/788425 ? | |
| 08:45:39 | gibi | does that ^^ superseeds https://review.opendev.org/c/openstack/nova/+/788425 ? | |
| 08:47:18 | lyarwood | gibi: well I've posted three changes in all, one that fixes just nova-grenade-multinode as it currently is, one that fixes nova-grenade-multinode by moving to the zuulv3 based jobs and a devstack change to fix all bionic based jobs | |
| 08:47:18 | lyarwood | gibi: well I've posted three changes in all, one that fixes just nova-grenade-multinode as it currently is, one that fixes nova-grenade-multinode by moving to the zuulv3 based jobs and a devstack change to fix all bionic based jobs | |
| 08:47:40 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/788425 is something we can land quickly IMHO | |
| 08:47:41 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/788425 is something we can land quickly IMHO | |
| 08:47:41 | gibi | I've just approved the zuul v3 move | |
| 08:47:42 | gibi | I've just approved the zuul v3 move | |
| 08:47:47 | lyarwood | ah there we go then | |
| 08:47:47 | lyarwood | ah there we go then | |
| 08:47:58 | lyarwood | I'll drop https://review.opendev.org/c/openstack/nova/+/788425 | |
| 08:47:59 | lyarwood | I'll drop https://review.opendev.org/c/openstack/nova/+/788425 | |
| 08:48:01 | gibi | Ok | |
| 08:48:01 | gibi | Ok | |
| 08:48:29 | gibi | thanks | |
| 08:48:48 | lyarwood | the zuulv3 change should also move our grenade testing to focal to focal finally | |
| 08:48:48 | lyarwood | the zuulv3 change should also move our grenade testing to focal to focal finally | |
| 08:48:55 | lyarwood | we dropped bionic back in ussuri | |
| 08:48:56 | lyarwood | we dropped bionic back in ussuri | |
| 09:11:09 | sean-k-mooney | does anyone know why we have an os_type column in the instances table | |
| 09:11:10 | sean-k-mooney | does anyone know why we have an os_type column in the instances table | |
| 09:12:03 | sean-k-mooney | as far as i can tell we always use the os_type image metadata property when generating xmls or schduling | |
| 09:12:04 | sean-k-mooney | as far as i can tell we always use the os_type image metadata property when generating xmls or schduling | |
| 09:12:11 | sean-k-mooney | not an os_type form the instance | |
| 09:12:12 | sean-k-mooney | not an os_type form the instance | |
| 09:23:04 | stephenfin | bauzas: Are you okay with me addressing your nits and then reviewing https://review.opendev.org/c/openstack/nova/+/695012 myself? I think the bulk of the change will stay the same | |
| 09:23:04 | stephenfin | bauzas: Are you okay with me addressing your nits and then reviewing https://review.opendev.org/c/openstack/nova/+/695012 myself? I think the bulk of the change will stay the same | |
| 09:23:47 | bauzas | stephenfin: woah, long story here, I can't remember the context but sure | |
| 09:23:47 | bauzas | stephenfin: woah, long story here, I can't remember the context but sure | |
| 09:24:42 | sean-k-mooney | we really should try and get that serise merged | |
| 09:24:42 | sean-k-mooney | we really should try and get that serise merged | |
| 09:26:45 | sean-k-mooney | hum i wonder why i never left any review comments on that unless its a different seriese form mark | |
| 09:26:45 | sean-k-mooney | hum i wonder why i never left any review comments on that unless its a different seriese form mark | |
| 09:29:56 | sean-k-mooney | oh i was thinking of https://review.opendev.org/c/openstack/nova/+/710848 https://review.opendev.org/c/openstack/nova/+/710847 and https://review.opendev.org/c/openstack/nova/+/760354 | |
| 09:29:56 | sean-k-mooney | oh i was thinking of https://review.opendev.org/c/openstack/nova/+/710848 https://review.opendev.org/c/openstack/nova/+/710847 and https://review.opendev.org/c/openstack/nova/+/760354 | |
| 09:50:38 | stephenfin | gibi: Could I get you to review the small diff on https://review.opendev.org/c/openstack/nova/+/676209 yet again, please? /o\ | |
| 09:50:38 | stephenfin | gibi: Could I get you to review the small diff on https://review.opendev.org/c/openstack/nova/+/676209 yet again, please? /o\ | |
| 09:51:01 | stephenfin | lyarwood: Any chance you'd be able to slog through that so I can finally close it out? I genuinely think it's a useful addition | |
| 09:51:01 | stephenfin | lyarwood: Any chance you'd be able to slog through that so I can finally close it out? I genuinely think it's a useful addition | |
| 09:58:12 | lyarwood | stephenfin: yup I'll queue it up for later today if that's okay | |
| 09:58:12 | lyarwood | stephenfin: yup I'll queue it up for later today if that's okay | |
| 09:58:22 | stephenfin | wfm. Thanks | |
| 09:58:22 | stephenfin | wfm. Thanks | |
| 09:58:32 | lyarwood | stephenfin: and I'll likely ask for the same later in the cycle for the block layer | |
| 09:58:33 | lyarwood | stephenfin: and I'll likely ask for the same later in the cycle for the block layer | |
| 09:58:39 | stephenfin | fair :) | |
| 09:58:39 | stephenfin | fair :) | |
| 10:12:14 | gibi | stephenfin: I will check it after my lunch | |
| 10:12:14 | gibi | stephenfin: I will check it after my lunch | |
| 10:24:12 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/788467 | |
| 10:24:47 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/788468 | |
| 10:24:47 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/788468 | |
| 10:26:33 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/788469 | |
| 10:26:33 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Ignore device already in the process of unplug errors https://review.opendev.org/c/openstack/nova/+/788469 | |
| 10:41:04 | sean-k-mooney | stephenfin: +1 on the first patch for nova.pci | |
| 10:41:04 | sean-k-mooney | stephenfin: +1 on the first patch for nova.pci | |
| 10:41:15 | stephenfin | thanks :) | |
| 10:41:15 | stephenfin | thanks :) | |
| 10:41:28 | sean-k-mooney | stephenfin: i have 2 other review i want to complte first then ill see if i can get back to the rest | |
| 10:41:28 | sean-k-mooney | stephenfin: i have 2 other review i want to complte first then ill see if i can get back to the rest | |
| 10:45:02 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704 | |
| 11:00:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: Testing with sqlalchemy 1.4 https://review.opendev.org/c/openstack/nova/+/788471 | |
| 11:00:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: Testing with sqlalchemy 1.4 https://review.opendev.org/c/openstack/nova/+/788471 | |
| 11:01:13 | gibi | fyi we are expected failures and therefore work here ^^ | |
| 11:01:13 | gibi | fyi we are expected failures and therefore work here ^^ | |
| 11:01:21 | gibi | s/expected/expecting/ | |
| 11:01:22 | gibi | s/expected/expecting/ | |
| 11:11:22 | 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 | |
| 11:11:33 | gibi | reported https://bugs.launchpad.net/nova/+bug/1926426 to track the work | |
| 11:11:35 | openstack | Launchpad bug 1926426 in OpenStack Compute (nova) "Nova is not compatible with sqlalchemy 1.4" [High,Triaged] | |
| 11:11:35 | gibi | reported https://bugs.launchpad.net/nova/+bug/1926426 to track the work | |
| 11:11:35 | openstack | Launchpad bug 1926426 in OpenStack Compute (nova) "Nova is not compatible with sqlalchemy 1.4" [High,Triaged] | |
| 11:13:17 | sean-k-mooney | is the issue the test or the production code? | |
| 11:13:17 | sean-k-mooney | is the issue the test or the production code? | |
| 11:13:56 | sean-k-mooney | looks like maybe both | |
| 11:13:56 | sean-k-mooney | looks like maybe both | |
| 11:14:02 | sean-k-mooney | have we capped it in UC | |
| 11:14:02 | sean-k-mooney | have we capped it in UC | |
| 11:15:24 | sean-k-mooney | ah we had to 1.3.23 previously | |
| 11:15:24 | sean-k-mooney | ah we had to 1.3.23 previously | |
| 11:15:50 | gibi | it is a test coming from a proposed UC bump | |
| 11:15:50 | gibi | it is a test coming from a proposed UC bump | |
| 11:16:07 | gibi | so far we see one oslo.db issue but we also seem to have some nova specific failure modes as well | |
| 11:16:07 | gibi | so far we see one oslo.db issue but we also seem to have some nova specific failure modes as well | |
| 11:56:26 | gibi | sean-k-mooney: Did I understood the comment from the PTG well about when we need direction aware and when we need direction less pps resource? https://review.opendev.org/c/openstack/neutron-specs/+/785236/3/specs/xena/qos-minimum-guaranteed-packet-rate.rst#266 | |
| 11:56:26 | gibi | sean-k-mooney: Did I understood the comment from the PTG well about when we need direction aware and when we need direction less pps resource? https://review.opendev.org/c/openstack/neutron-specs/+/785236/3/specs/xena/qos-minimum-guaranteed-packet-rate.rst#266 | |
| 11:58:37 | sean-k-mooney | sorry have nto got to your spec yet today but ill take a look at that comment now one sec | |
| 11:58:37 | sean-k-mooney | sorry have nto got to your spec yet today but ill take a look at that comment now one sec | |
| 11:59:41 | sean-k-mooney | gibi: so ya | |
| 11:59:41 | sean-k-mooney | gibi: so ya | |