| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-25 | |||
| 16:50:01 | bauzas | ok, I'm done for this (busy) week | |
| 16:50:05 | bauzas | see ya folks | |
| 16:56:52 | whoami-rajat | dansmith, hey, the OSC client rebuild call is calling novaclient (AFAICS) so we do need the change in novaclient as well right? https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server.py#L3541 | |
| 16:57:27 | whoami-rajat | or there will be a structural change proposal to remove dependency of novaclient somehow... | |
| 16:57:30 | dansmith | whoami-rajat: I'm not on top of the state of that stuff.. I expect there are a bunch of calls not converted to native that still use novaclient? not sure | |
| 16:57:51 | dansmith | but if rebuild is currently going through novaclient today, I doubt anyone is going to make you change that before you merge :) | |
| 16:59:07 | whoami-rajat | ack, I'm totally confused with the deprecating novaclient and proposing changes directly to OSC first then... | |
| 16:59:19 | whoami-rajat | sean-k-mooney, can you help with my query above? ^ | |
| 17:00:42 | dansmith | whoami-rajat: yeah just not sure, but you're right to be confused/frustrated.. I'll look for sean-k-mooney to answer aw well | |
| 17:00:43 | dansmith | *as | |
| 17:03:07 | whoami-rajat | good to know I'm not the only one :) | |
| 17:16:14 | whoami-rajat | so this is my patch for OSC but it won't work without the novaclient patch https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |
| 17:17:28 | whoami-rajat | similar to the --hostname parameter introduced in 2.90 which depends on the novaclient patch https://github.com/openstack/python-openstackclient/commit/8e833a3ed26467a1190ba69d8ba716a7cd1cccb3 | |
| 17:19:37 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 17:39:14 | dmitriis | sean-k-mooney: I ended up getting the same result while trying to do a revert for an instance with a non-remote-managed PCI device https://paste.opendev.org/show/blQLSgsFtQbryq3q6ZJM/. I'll see if I can modify the fixture in a follow-up to test this. | |
| 17:55:21 | sean-k-mooney | whoami-rajat: we deprecated the nova cli not the python-novaclient package | |
| 17:55:57 | sean-k-mooney | it can still be used as a lib but we declared we woudl not continue to extend the nova command line clint and only add new microversion to osc | |
| 17:59:25 | sean-k-mooney | whoami-rajat: you can helps to novaclient, openstack sdk or just osc direclty without updating novaclient | |
| 18:01:16 | dansmith | the latter would require him to refactor all of the rebuild code in osc, so not likely right? | |
| 18:01:26 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 18:02:16 | sean-k-mooney | yes. we idealy want people to update sdk instead of nova python client | |
| 18:03:22 | sean-k-mooney | the only thing we decided to deprecate this cycle was the nova cli so im fine with removing the -2 form the novaclient patch after rc1 if we remove the addtion of the new partmar to the cli | |
| 18:04:01 | whoami-rajat | if i update my novaclient patch without the shell changes (not extending nova CLI) and modify the rebuild method called by OSC, would it be acceptable? | |
| 18:04:20 | sean-k-mooney | yep | |
| 18:04:24 | sean-k-mooney | that is perfectly fine | |
| 18:04:40 | whoami-rajat | ack | |
| 18:04:53 | sean-k-mooney | we will be keeping it as a lib for other projefct to use to programticaly call nova for a few releases | |
| 18:05:03 | sean-k-mooney | eventually we would like heat ectra to use the sdk | |
| 18:05:21 | sean-k-mooney | but we have not made plans to remove it as a lib in the near term | |
| 18:06:15 | sean-k-mooney | but now that we have feature partiy for api microversion in osc we dont wnat ot continute adding support for microversion in nova client | |
| 18:10:27 | whoami-rajat | got it | |
| 18:10:32 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 18:15:27 | opendevreview | Rajat Dhasmana proposed openstack/python-novaclient master: Add support to rebuild boot volume https://review.opendev.org/c/openstack/python-novaclient/+/827163 | |
| 18:32:59 | sean-k-mooney[m] | whoami-rajat: I dropped -2 on ^ but the release note still reference the nova cli change you dropped | |
| 18:34:34 | whoami-rajat | sean-k-mooney[m], ack, will update it | |
| 18:35:18 | opendevreview | Rajat Dhasmana proposed openstack/python-novaclient master: Add support to rebuild boot volume https://review.opendev.org/c/openstack/python-novaclient/+/827163 | |
| 19:15:39 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 19:19:23 | chateaulav | sean-k-mooney: last individual run, then i should be able to tie a bow on the ci job | |
| 20:18:40 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 21:37:49 | opendevreview | Merged openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 21:58:32 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 21:59:49 | chateaulav | sean-k-mooney: ci is good to go, I uncommented the other jobs and moved it under the other nova-XX jobs. let me know if you need anything further for it. | |
| 21:59:56 | chateaulav | o/ | |
| 22:20:08 | opendevreview | Merged openstack/nova master: Log which instance event was timed out https://review.opendev.org/c/openstack/nova/+/819817 | |
| 22:29:55 | opendevreview | Merged openstack/nova master: Add debug log for scheduler weight calculation https://review.opendev.org/c/openstack/nova/+/817632 | |
| #openstack-nova - 2022-02-26 | |||
| 00:22:10 | sean-k-mooney[m] | chateaulav: nice ill take a look at it on Monday but cool to see it green | |
| 01:52:55 | opendevreview | Merged openstack/nova master: Server actions APIs scoped to project scope https://review.opendev.org/c/openstack/nova/+/824358 | |
| 01:53:02 | opendevreview | Merged openstack/nova master: Modify remaining APIs as per RBAC new guidelines https://review.opendev.org/c/openstack/nova/+/828994 | |
| 06:38:59 | opendevreview | Takashi Natsume proposed openstack/nova master: doc: mark the max microversion for yoga https://review.opendev.org/c/openstack/nova/+/831075 | |
| 06:46:02 | opendevreview | Takashi Natsume proposed openstack/nova master: Update compute rpc version alias for yoga https://review.opendev.org/c/openstack/nova/+/831077 | |
| 08:46:30 | opendevreview | Merged openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 09:08:45 | opendevreview | Merged openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 09:08:53 | opendevreview | Merged openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 11:44:37 | opendevreview | Merged openstack/nova master: Separate flavor extra specs policy for server APIs https://review.opendev.org/c/openstack/nova/+/829626 | |
| 12:39:50 | opendevreview | Merged openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 20:08:10 | opendevreview | melanie witt proposed openstack/nova master: Retry in CellDatabases fixture when global DB state changes https://review.opendev.org/c/openstack/nova/+/831084 | |
| 20:41:51 | melwitt | bauzas, gibi: attempt to fix gate bug ^ | |
| 23:44:54 | opendevreview | Merged openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 23:45:02 | opendevreview | Merged openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| #openstack-nova - 2022-02-27 | |||
| 03:17:29 | opendevreview | Merged openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 03:24:35 | opendevreview | Merged openstack/nova master: neutron: Remove references to dead 'qos-queue' extension https://review.opendev.org/c/openstack/nova/+/754448 | |
| 04:01:51 | opendevreview | Merged openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 04:01:58 | opendevreview | Merged openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 04:50:14 | opendevreview | Merged openstack/nova master: Complete phase-1 of RBAC community-wide goal https://review.opendev.org/c/openstack/nova/+/829866 | |
| 10:33:15 | opendevreview | norman shen proposed openstack/nova master: Fix wrong string format string https://review.opendev.org/c/openstack/nova/+/831089 | |
| 12:41:29 | opendevreview | Merged openstack/nova stable/xena: Retry image download if it's corrupted https://review.opendev.org/c/openstack/nova/+/819179 | |
| 16:15:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add oslo.limit options to generated nova.conf https://review.opendev.org/c/openstack/nova/+/831101 | |
| #openstack-nova - 2022-02-28 | |||
| 03:59:06 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 08:18:12 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 08:36:45 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 09:28:03 | elodilles | bauzas: hi, if you have time, could you please review this WA patch for stable/wallaby? https://review.opendev.org/c/openstack/nova/+/830656 | |
| 10:09:07 | bauzas | elodilles: sure, looking | |
| 10:29:54 | elodilles | thanks o/ | |
| 10:44:08 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs(db+object) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 10:44:08 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs(manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 11:14:10 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Add service version check workaround for FFU https://review.opendev.org/c/openstack/nova/+/831174 | |
| 11:23:40 | gibi | melwitt: thanks for proposing the fix for the DB fixture. I left a comment. I haven't had time to dig deep into the problem so maybe my comment is not valid | |
| 12:50:01 | gibi | bauzas: there are two housekeeping patches for nova https://review.opendev.org/c/openstack/nova/+/831075 https://review.opendev.org/c/openstack/nova/+/831077 we need these before RC1 | |
| 12:52:04 | gibi | and there will be one more related housekeeping but after RC1 to do this for Yoga->Zed https://review.opendev.org/c/openstack/nova/+/809932 | |
| 12:59:17 | sean-k-mooney | ah the rpc alias | |
| 13:00:00 | sean-k-mooney | gibi: unified limist has mostly merged did that not ahve a microverion in it? | |
| 13:00:18 | gibi | it did not | |
| 13:00:28 | sean-k-mooney | huh ok | |
| 13:00:43 | gibi | unified limits is a configurable limits driver | |
| 13:00:56 | gibi | * quota driver | |
| 13:01:25 | gibi | so there is no meaningful way to ask the old behavior with an old microversion | |
| 13:01:34 | gibi | as the behavior is config driven | |
| 13:02:03 | sean-k-mooney | ack that makes sense | |
| 13:02:44 | sean-k-mooney | i tought we were going to have one at one point but i understand now why. did we disucss having one if we revometed the proxy apis? | |
| 13:03:23 | gibi | I'm not aware of such discussion | |
| 13:03:56 | sean-k-mooney | ok i just tought we discussed one at one point but maybe the out come was it did not make sense for the reasons you gave above | |
| 13:04:34 | sean-k-mooney | anyway in that case both of those patches look fine to me. did you want bauzas specificly to approve them or will i just review them and send them on there way | |
| 13:07:44 | gibi | hm, the PTL guide says that the RPC alias is more appropriate just before RC1 | |
| 13:07:48 | gibi | so lets wait a bit | |
| 13:08:03 | sean-k-mooney | ack | |
| 13:08:03 | gibi | I think the microversion doc is OK to go in as that is only documentation | |
| 13:08:42 | sean-k-mooney | i have +2'd them but ill leave teh +w to bauzas | |
| 13:09:14 | gibi | ack, thanks | |
| 13:12:15 | opendevreview | Merged openstack/nova stable/wallaby: workarounds: Add libvirt_disable_apic https://review.opendev.org/c/openstack/nova/+/805628 | |
| 14:22:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add oslo.limit options to generated nova.conf https://review.opendev.org/c/openstack/nova/+/831101 | |