Earlier  
Posted Nick Remark
#openstack-nova - 2021-03-19
16:02:04 gibi stephenfin: I have to get back to this on Monday
16:04:08 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/778885
16:04:08 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Add nova-live-migration-ceph job https://review.opendev.org/c/openstack/nova/+/768466
16:04:09 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Remove nova-dsvm-multinode-base https://review.opendev.org/c/openstack/nova/+/778908
16:04:10 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Create and use nova-libvirt-irrelevant-files https://review.opendev.org/c/openstack/nova/+/778909
16:46:15 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Create and use nova-libvirt-irrelevant-files https://review.opendev.org/c/openstack/nova/+/778909
16:46:15 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Remove nova-dsvm-multinode-base https://review.opendev.org/c/openstack/nova/+/778908
17:48:43 openstackgerrit Merged openstack/nova stable/victoria: libvirt: Use specific user when probing encrypted rbd disks during extend https://review.opendev.org/c/openstack/nova/+/773443
17:49:40 openstackgerrit Merged openstack/nova master: doc: mark the max microversion for wallaby https://review.opendev.org/c/openstack/nova/+/780401
19:02:44 openstackgerrit Merged openstack/nova master: releasenotes: Fix typo https://review.opendev.org/c/openstack/nova/+/781725
19:48:07 openstackgerrit OpenStack Release Bot proposed openstack/os-vif stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/os-vif/+/781888
19:48:12 openstackgerrit OpenStack Release Bot proposed openstack/os-vif stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/os-vif/+/781889
19:48:27 openstackgerrit OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/wallaby https://review.opendev.org/c/openstack/os-vif/+/781890
19:48:32 openstackgerrit OpenStack Release Bot proposed openstack/os-vif master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/os-vif/+/781891
19:48:48 openstackgerrit OpenStack Release Bot proposed openstack/osc-placement stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/osc-placement/+/781892
19:48:52 openstackgerrit OpenStack Release Bot proposed openstack/osc-placement stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/osc-placement/+/781893
19:48:56 openstackgerrit OpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/wallaby https://review.opendev.org/c/openstack/osc-placement/+/781894
19:49:21 openstackgerrit OpenStack Release Bot proposed openstack/python-novaclient stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/781895
19:49:29 openstackgerrit OpenStack Release Bot proposed openstack/python-novaclient stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/781896
19:49:39 openstackgerrit OpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/781897
19:49:43 openstackgerrit OpenStack Release Bot proposed openstack/python-novaclient master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/python-novaclient/+/781898
#openstack-nova - 2021-03-20
01:57:58 openstackgerrit Merged openstack/python-novaclient master: Use well named anchor into the microversion history https://review.opendev.org/c/openstack/python-novaclient/+/781484
01:58:01 openstackgerrit Merged openstack/python-novaclient master: Update master for stable/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/781897
06:44:01 openstackgerrit Merged openstack/python-novaclient master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/python-novaclient/+/781898
10:27:41 openstackgerrit Merged openstack/nova master: api: Block unsupported actions with vDPA https://review.opendev.org/c/openstack/nova/+/780333
10:39:42 openstackgerrit Merged openstack/nova master: tests: Add functional test for vDPA device https://review.opendev.org/c/openstack/nova/+/780112
10:40:14 openstackgerrit Merged openstack/nova master: Add release note for vDPA https://review.opendev.org/c/openstack/nova/+/780866
15:36:31 openstackgerrit Merged openstack/nova stable/victoria: Use absolute path during qemu img rebase https://review.opendev.org/c/openstack/nova/+/757098
18:19:14 openstackgerrit Merged openstack/os-vif stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/os-vif/+/781888
18:26:14 openstackgerrit Merged openstack/os-vif stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/os-vif/+/781889
18:28:09 openstackgerrit Merged openstack/python-novaclient stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/781895
18:28:13 openstackgerrit Merged openstack/python-novaclient stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/781896
18:31:22 openstackgerrit Merged openstack/osc-placement stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/osc-placement/+/781892
18:31:25 openstackgerrit Merged openstack/osc-placement stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/osc-placement/+/781893
#openstack-nova - 2021-03-21
06:16:42 openstackgerrit Merged openstack/nova master: Add missed accel_uuids for _poll_shelved_instances https://review.opendev.org/c/openstack/nova/+/778440
07:11:43 gibi \o/ VDPA landed
13:20:31 openstackgerrit Takashi Natsume proposed openstack/python-novaclient stable/wallaby: Use well named anchor into the microversion history https://review.opendev.org/c/openstack/python-novaclient/+/781947
17:12:15 openstackgerrit Merged openstack/nova stable/victoria: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765769
#openstack-nova - 2021-03-22
07:04:12 Luzi gibi: i made a backport to victoria as you told me: https://review.opendev.org/c/openstack/nova/+/781211
07:42:27 pawan-gupta_ Hi, I am trying to create an instance using `adminPass` option and `nova.conf` is updated with `inject_password = True` but the provided password in `adminPass` does not work. I am using KVM hypervisor. Am I missing something?
07:49:11 frickler pawan-gupta_: are you running under py3? iirc this feature is broken there, check for errors in your logs
07:52:12 gibi Luzi: awesome, thank you. I've added two stable cores to the review to get it merged :)
07:52:23 Luzi gibi, thank you :)
07:56:10 gibi frickler: I guess you are referring to https://review.opendev.org/c/openstack/nova/+/781211
07:56:16 gibi frickler: sorry
07:56:19 gibi not that
07:56:27 gibi frickler: this https://bugs.launchpad.net/nova/+bug/1882421
07:56:28 openstack Launchpad bug 1882421 in OpenStack Compute (nova) "inject_password fails with python3" [Medium,Confirmed]
08:25:57 openstackgerrit Balazs Gibizer proposed openstack/nova master: trivial: fix word duplication in api ref https://review.opendev.org/c/openstack/nova/+/782028
08:28:50 gibi bauzas: hi! the rpc bump is in merge conflict
08:31:07 pawan-gupta_ frickler: thanks for the response but I am running it under py2.7.5
09:49:17 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add compute rpc version alias for wallaby as 5.13 https://review.opendev.org/c/openstack/nova/+/782115
09:58:07 bauzas gibi: yup, I need to rebase it
09:58:38 bauzas gibi: I'm working on the prelude atm
10:03:01 gibi bauzas: ack, thanks
10:13:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Update min supported service version for Xena. https://review.opendev.org/c/openstack/nova/+/782171
10:17:03 openstackgerrit Sylvain Bauza proposed openstack/nova master: Wallaby 22.0.0 prelude section https://review.opendev.org/c/openstack/nova/+/782172
10:17:16 bauzas gibi: first round of prelude ^
10:17:21 gibi bauzas: on it :) thank you
10:17:23 bauzas I need to dad taxi
10:17:40 bauzas gibi: do we have deprecations or removals this cycle ? AFAIK, nope
10:22:44 gibi bauzas: I will check that but I don't remember any deprecation
10:28:53 lyarwood gibi / bauzas ; do workarounds count?
10:42:01 gibi lyarwood: it is in context of the reno prelude. If this is an important to highlight in the prelude then lets do that
10:42:54 lyarwood gibi: it already has it's on releasenote so I wouldn't bother tbh, I just wasn't sure what the criteria was for the prelude
10:43:32 gibi I think there are no hard criterias
10:51:08 MrClayPole Hi, We are looking to start regular "apt" patching in our test environment we a view of pushing to production. Currently we are running on Ubuntu 18.04.1 with OSA Rocky in test. Is there anything we need to be aware or look at first before we start testing from a Nova prospective? I believe there may have been some issue is the past with live migration? Is that still an issue?
10:53:13 gibi MrClayPole: I suggest to read https://docs.openstack.org/nova/latest/user/upgrade.html
10:53:30 MrClayPole Thanks I'll take a look
10:53:52 gibi MrClayPole: and also suggest to read the relese notes related to the version you are upgrading to https://docs.openstack.org/releasenotes/nova/
12:32:43 gibi bauzas: left feedback in the reno prelude
13:53:16 lyarwood elod / melwitt / bauzas ; some stable/victoria changes ready for review if anyone has time btw - https://review.opendev.org/c/openstack/nova/+/773320 https://review.opendev.org/c/openstack/nova/+/772480 https://review.opendev.org/c/openstack/nova/+/773320 https://review.opendev.org/c/openstack/nova/+/773321/
13:53:37 lyarwood if there's anything I can review in return let me know!
14:00:57 elod lyarwood: ack, will try to review some
14:31:19 gibi cores, as far as I see we have a sort of rule that only admin can list deleted instances. What about soft-delete instances? As far as I see today only admins can list soft-delete instances. So a user knows the uuid of its accidentally deleted server then they can restore it but if they only know the name of the server then there is no way for the user to list the soft-delete instances to find out
14:31:25 gibi which one they want to restore
14:34:05 gibi so if I know the uuid then I can show it and restore it, but if I only know the name then there is no way I can find the uuid
14:43:09 gibi we do not list soft-delete vms by default https://github.com/openstack/nova/blob/3de7fb7c327db348d04d15d4cd3c4f811a336126/nova/db/sqlalchemy/api.py#L1557 and a filter like vm_state=soft-delete does not change this
14:43:28 sean-k-mooney gibi: im not sure that only admin can list deleted instance or at least all info about them
14:43:44 sean-k-mooney gibi: what im thinking about is the simple tenant usage api
14:44:05 sean-k-mooney where the interval will include the usage for deleted instancen that were active in that interval
14:44:30 sean-k-mooney we may or may not require admin for listing them normally, havent check the api ref
14:44:44 gibi sean-k-mooney: https://github.com/openstack/nova/blob/3de7fb7c327db348d04d15d4cd3c4f811a336126/nova/api/openstack/compute/servers.py#L268
14:44:47 sean-k-mooney but normal teant shave at least limited awarenes
14:45:42 gibi so GET /servers does not return deleted instances to non admins
14:45:44 sean-k-mooney gibi are you sure that takes effect if you use v2
14:46:07 sean-k-mooney i guess it should
14:47:42 sean-k-mooney "Show deleted items only. In some circumstances deleted items will still be accessible via the backend database, however there is no contract on how long, so this parameter should be used with caution. 1, t, true, on, y and yes are treated as True (case-insensitive). Other than them are treated as False.
14:47:44 sean-k-mooney This parameter is only valid when specified by administrators. If non-admin users specify this parameter, it is ignored.
14:47:54 sean-k-mooney so ya admin only in the api ref too
14:48:32 gmann yes, for v2 or v2.1, deleted instances are admin only
14:48:47 gibi and I agree that we cannot garantee data about deleted instances
14:48:55 gibi so there admin onlyness is OK to me
14:49:05 sean-k-mooney yep same
14:49:19 sean-k-mooney soft delete is not even guarentted by the api
14:49:19 gibi but soft-delete instances are always kept in the db so there we could return them to the owner
14:49:48 sean-k-mooney well isnt soft delete resoration an admin only api too
14:49:56 gibi no
14:50:00 gibi soft delete is allowed to owner
14:50:15 gmann yeah admin-or-owner

Earlier   Later