| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-30 | |||
| 14:16:38 | lyarwood | noonedeadpunk: it was discussed earlier in the channel, you need libvirt >= 6.1.0 | |
| 14:16:39 | lyarwood | noonedeadpunk: it was discussed earlier in the channel, you need libvirt >= 6.1.0 | |
| 14:16:45 | dansmith | because from the perspective of someone trying to figure out what it's looking for, the numbers don't match up | |
| 14:16:45 | dansmith | because from the perspective of someone trying to figure out what it's looking for, the numbers don't match up | |
| 14:16:46 | lyarwood | noonedeadpunk: we don't workaround this in nova | |
| 14:16:46 | lyarwood | noonedeadpunk: we don't workaround this in nova | |
| 14:16:56 | dansmith | gibi: but if that's the goal, then I guess it's okay | |
| 14:16:56 | dansmith | gibi: but if that's the goal, then I guess it's okay | |
| 14:17:26 | noonedeadpunk | lyarwood: yeah, I see that, but libvirt 6.1.0 is smth that seems needs to be built manually I guess | |
| 14:17:26 | noonedeadpunk | lyarwood: yeah, I see that, but libvirt 6.1.0 is smth that seems needs to be built manually I guess | |
| 14:17:32 | noonedeadpunk | as UCA doesn't provide that :( | |
| 14:17:32 | noonedeadpunk | as UCA doesn't provide that :( | |
| 14:17:40 | lyarwood | noonedeadpunk: then raise a bug with the UCA folks | |
| 14:17:40 | lyarwood | noonedeadpunk: then raise a bug with the UCA folks | |
| 14:18:01 | gibi | dansmith: at least it is documented in the code comment above the alias https://github.com/openstack/nova/blob/dab4ec1a534d4774e61a24cdace0f3491788e4e7/nova/objects/service.py#L206 so the intention is clear for the reader | |
| 14:18:01 | gibi | dansmith: at least it is documented in the code comment above the alias https://github.com/openstack/nova/blob/dab4ec1a534d4774e61a24cdace0f3491788e4e7/nova/objects/service.py#L206 so the intention is clear for the reader | |
| 14:18:10 | noonedeadpunk | I believe there were solid reasons not to workaround in nova? | |
| 14:18:10 | noonedeadpunk | I believe there were solid reasons not to workaround in nova? | |
| 14:18:27 | dansmith | gibi: okay :) | |
| 14:18:27 | dansmith | gibi: okay :) | |
| 14:18:29 | lyarwood | noonedeadpunk: yeah it would require us to rebase the disks at launch | |
| 14:18:29 | lyarwood | noonedeadpunk: yeah it would require us to rebase the disks at launch | |
| 14:18:38 | noonedeadpunk | Just thinking if it make sense to look into this direction or not | |
| 14:18:38 | noonedeadpunk | Just thinking if it make sense to look into this direction or not | |
| 14:18:41 | gibi | dansmith: :) | |
| 14:18:41 | gibi | dansmith: :) | |
| 14:18:56 | noonedeadpunk | yeah, that's exactly how I'm fixing at atm | |
| 14:18:56 | noonedeadpunk | yeah, that's exactly how I'm fixing at atm | |
| 14:19:13 | gibi | dansmith: I was positively suprised that deployers look at the warning we emit from the compute version check | |
| 14:19:13 | gibi | dansmith: I was positively suprised that deployers look at the warning we emit from the compute version check | |
| 14:19:44 | dansmith | some will ignore it I'm sure | |
| 14:19:44 | dansmith | some will ignore it I'm sure | |
| 14:21:16 | noonedeadpunk | eventually it's pretty straightforward fix for nova I guess? or some consideration why not to run rebase when we're failing as fallback? | |
| 14:21:16 | noonedeadpunk | eventually it's pretty straightforward fix for nova I guess? or some consideration why not to run rebase when we're failing as fallback? | |
| 14:21:36 | lyarwood | noonedeadpunk: it's just extra overhead we didn't want to carry when the fix in libvirt is already available | |
| 14:21:36 | lyarwood | noonedeadpunk: it's just extra overhead we didn't want to carry when the fix in libvirt is already available | |
| 14:21:46 | lyarwood | noonedeadpunk: el distros were quick to pick it up | |
| 14:21:46 | lyarwood | noonedeadpunk: el distros were quick to pick it up | |
| 14:22:03 | noonedeadpunk | yeah, as libvirt is el thing.... | |
| 14:22:03 | noonedeadpunk | yeah, as libvirt is el thing.... | |
| 14:22:22 | lyarwood | it's not like it's closed source :) | |
| 14:22:22 | lyarwood | it's not like it's closed source :) | |
| 14:22:40 | noonedeadpunk | * noonedeadpunk wishes el was picking up other stuff as fast (like py38 libselinux bindings) | |
| 14:22:57 | noonedeadpunk | ok, gotcha, thanks | |
| 14:22:57 | noonedeadpunk | ok, gotcha, thanks | |
| 16:51:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Adapt to SQLAlchemy 1.4 https://review.opendev.org/c/openstack/nova/+/788471 | |
| 17:56:23 | openstackgerrit | Merged openstack/nova stable/ussuri: Rebase qcow2 images when unshelving an instance https://review.opendev.org/c/openstack/nova/+/785625 | |
| 17:56:23 | openstackgerrit | Merged openstack/nova stable/ussuri: Rebase qcow2 images when unshelving an instance https://review.opendev.org/c/openstack/nova/+/785625 | |
| 17:59:48 | openstackgerrit | Merged openstack/nova stable/ussuri: Update image_base_image_ref during rebuild. https://review.opendev.org/c/openstack/nova/+/787943 | |
| 17:59:48 | openstackgerrit | Merged openstack/nova stable/ussuri: Update image_base_image_ref during rebuild. https://review.opendev.org/c/openstack/nova/+/787943 | |
| 18:12:52 | openstackgerrit | Merged openstack/nova master: Consolidate device detach error handling https://review.opendev.org/c/openstack/nova/+/778978 | |
| 18:12:52 | openstackgerrit | Merged openstack/nova master: Consolidate device detach error handling https://review.opendev.org/c/openstack/nova/+/778978 | |
| 18:45:16 | outbrito | Good day folks! I'm working on the StarlingX Openstack project and I just found a problem with one of nova's dependencies and I'd like to get youe feedback about how I'm thinking about fixing it | |
| 18:45:16 | outbrito | Good day folks! I'm working on the StarlingX Openstack project and I just found a problem with one of nova's dependencies and I'd like to get youe feedback about how I'm thinking about fixing it | |
| 18:46:25 | outbrito | What I see here on stable/ussuri is an AttributeError: module 'oslo_service' has no attribute 'service_hub' | |
| 18:46:25 | outbrito | What I see here on stable/ussuri is an AttributeError: module 'oslo_service' has no attribute 'service_hub' | |
| 18:46:47 | outbrito | I see that oslo.service dropped this at this commit: https://github.com/openstack/oslo.service/commit/383701e2317e96d81f05a9749e4c106929bb8a34 | |
| 18:46:47 | outbrito | I see that oslo.service dropped this at this commit: https://github.com/openstack/oslo.service/commit/383701e2317e96d81f05a9749e4c106929bb8a34 | |
| 18:47:11 | outbrito | It seems like it was released on Feb 8th on oslo.service 2.5.0 | |
| 18:47:11 | outbrito | It seems like it was released on Feb 8th on oslo.service 2.5.0 | |
| 18:48:05 | outbrito | Nova is still loading this at https://github.com/openstack/nova/blob/stable/ussuri/nova/monkey_patch.py#L78 | |
| 18:48:06 | outbrito | Nova is still loading this at https://github.com/openstack/nova/blob/stable/ussuri/nova/monkey_patch.py#L78 | |
| 18:49:33 | outbrito | So, I'm thinking about changing the requirement on requirements.txt to oslo.service<2.5.0. Is this the correct approach? I wasn't around in openstack when openstack/requirements came to be, so I'm a little confused if I need to change something there first... | |
| 18:49:33 | outbrito | So, I'm thinking about changing the requirement on requirements.txt to oslo.service<2.5.0. Is this the correct approach? I wasn't around in openstack when openstack/requirements came to be, so I'm a little confused if I need to change something there first... | |
| 18:50:48 | clarkb | outbrito: I think you're allowed to be more restrictive than global-requirements now but not allow more than global-requirements. Also if you push the update ci will tell you | |
| 18:50:49 | clarkb | outbrito: I think you're allowed to be more restrictive than global-requirements now but not allow more than global-requirements. Also if you push the update ci will tell you | |
| 18:52:30 | clarkb | outbrito: and https://opendev.org/openstack/requirements/src/branch/stable/ussuri/upper-constraints.txt#L68-L71 is what would've been current with nova at the time of the ussuri release. | |
| 18:52:30 | clarkb | outbrito: and https://opendev.org/openstack/requirements/src/branch/stable/ussuri/upper-constraints.txt#L68-L71 is what would've been current with nova at the time of the ussuri release. | |
| 19:00:45 | openstackgerrit | Merged openstack/nova master: vmware: Use oslo.vmware's get_moref_value() https://review.opendev.org/c/openstack/nova/+/786636 | |
| 19:00:45 | openstackgerrit | Merged openstack/nova master: vmware: Use oslo.vmware's get_moref_value() https://review.opendev.org/c/openstack/nova/+/786636 | |
| 19:00:57 | outbrito | clarkb, thanks for the reference. So should I pin the requirements.txt to 2.1.1? I don't think we need python_version==2.7 on ussuri since we dropped the support officially, right?! | |
| 19:00:57 | outbrito | clarkb, thanks for the reference. So should I pin the requirements.txt to 2.1.1? I don't think we need python_version==2.7 on ussuri since we dropped the support officially, right?! | |
| 19:01:34 | clarkb | no I wouldn't pin to 2.1.1, the constraints already do that (and you might consider using the constraints in starglingx to avoid these problems). Was more just a data point that 2.1.1 is less than 2.5.0 so there isn't a conflict there | |
| 19:01:34 | clarkb | no I wouldn't pin to 2.1.1, the constraints already do that (and you might consider using the constraints in starglingx to avoid these problems). Was more just a data point that 2.1.1 is less than 2.5.0 so there isn't a conflict there | |
| 19:03:57 | outbrito | clarkb, Oh, ok. So I think LOCI will do that for me... | |
| 19:03:57 | outbrito | clarkb, Oh, ok. So I think LOCI will do that for me... | |
| #openstack-nova - 2021-05-01 | |||
| 00:32:49 | openstackgerrit | Merged openstack/nova master: Fix bond_mode enum 802.1ad -> 802.3ad https://review.opendev.org/c/openstack/nova/+/788790 | |
| 00:32:49 | openstackgerrit | Merged openstack/nova master: Fix bond_mode enum 802.1ad -> 802.3ad https://review.opendev.org/c/openstack/nova/+/788790 | |
| 04:06:58 | openstackgerrit | Merged openstack/nova master: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/602432 | |
| 04:06:58 | openstackgerrit | Merged openstack/nova master: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/602432 | |
| 04:08:10 | openstackgerrit | Merged openstack/nova master: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/783084 | |
| 04:08:10 | openstackgerrit | Merged openstack/nova master: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/783084 | |
| 11:02:20 | noonedeadpunk | lyarwood: returning back to the topic of the libvirt issue and backing image. Reading code through, I managed to get a patch, which limits really to http://paste.openstack.org/show/804896/ plus https://review.opendev.org/c/openstack/oslo.utils/+/789051 | |
| 11:02:20 | noonedeadpunk | lyarwood: returning back to the topic of the libvirt issue and backing image. Reading code through, I managed to get a patch, which limits really to http://paste.openstack.org/show/804896/ plus https://review.opendev.org/c/openstack/oslo.utils/+/789051 | |
| 11:02:36 | noonedeadpunk | do you still think it doesn't make sense having that upstream? | |
| 11:02:36 | noonedeadpunk | do you still think it doesn't make sense having that upstream? | |
| 11:03:28 | noonedeadpunk | eventually I believe it's possible to workaround without https://review.opendev.org/c/openstack/oslo.utils/+/789051 as well | |
| 11:03:28 | noonedeadpunk | eventually I believe it's possible to workaround without https://review.opendev.org/c/openstack/oslo.utils/+/789051 as well | |
| 11:03:39 | noonedeadpunk | just slightly more code needed | |
| 11:03:39 | noonedeadpunk | just slightly more code needed | |
| 11:04:06 | noonedeadpunk | (ie to get backing image path) | |
| 11:04:06 | noonedeadpunk | (ie to get backing image path) | |
| 11:06:35 | noonedeadpunk | basically this one https://opendev.org/openstack/nova/src/commit/24595b6ca981ad8342cde691218cbc39b323144b/nova/virt/libvirt/driver.py#L4291-L4294 | |
| 11:06:35 | noonedeadpunk | basically this one https://opendev.org/openstack/nova/src/commit/24595b6ca981ad8342cde691218cbc39b323144b/nova/virt/libvirt/driver.py#L4291-L4294 | |
| 12:46:06 | openstackgerrit | Merged openstack/nova master: Remove 'create_db_entry_for_new_instance' https://review.opendev.org/c/openstack/nova/+/778545 | |
| 12:46:06 | openstackgerrit | Merged openstack/nova master: Remove 'create_db_entry_for_new_instance' https://review.opendev.org/c/openstack/nova/+/778545 | |
| 18:31:27 | openstackgerrit | Merged openstack/nova master: vmware: Handle folder renames in rescue cmd https://review.opendev.org/c/openstack/nova/+/775852 | |
| 18:31:27 | openstackgerrit | Merged openstack/nova master: vmware: Handle folder renames in rescue cmd https://review.opendev.org/c/openstack/nova/+/775852 | |
| #openstack-nova - 2021-05-02 | |||
| 02:02:12 | openstackgerrit | Tetsuro Nakamura proposed openstack/osc-placement master: Support microversion 1.30 - /reshaper API https://review.opendev.org/c/openstack/osc-placement/+/696663 | |
| 17:38:14 | openstackgerrit | Merged openstack/nova stable/wallaby: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787251 | |