Posted	Nick	Remark
#openstack-nova - 2021-01-29
16:58:51	stephenfin	rechecks are all or nothing
16:59:29	sean-k-mooney	kashyap: if its third party you can normlaly recheck those cis seperatly
16:59:41	sean-k-mooney	but no we dont support that because of the green check policy
17:00:00	sean-k-mooney	its too easy to allow broken code to merge if you can do it for indivigual jobs
17:00:49	sean-k-mooney	we used to have reverify to just rerun the gate pipeline seperate to check and we remove it for that reason
17:02:09	kashyap	stephenfin: sean-k-mooney: Yeah, fair enough
17:04:32	sean-k-mooney	kashyap: is a speicifc job failing consitently?
17:04:50	sean-k-mooney	kashyap: i think lee/gibi fixed something related to nova-next or nova-lvm recently
17:04:54	kashyap	Nah; just a Ceph timed-out for the first time
17:05:01	kashyap	s/Ceph/Ceph job/
17:05:01	sean-k-mooney	ah ok
17:08:24	kashyap	Have a good weekend!
17:08:35	sean-k-mooney	same o/
17:10:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove unused 'shutdown_terminate' flag https://review.opendev.org/c/openstack/nova/+/773156
22:33:14	openstackgerrit	Dan Smith proposed openstack/nova master: Restart glance after updating policy in job https://review.opendev.org/c/openstack/nova/+/773192
23:18:20	openstackgerrit	Dan Smith proposed openstack/nova master: Restart glance after updating policy in job https://review.opendev.org/c/openstack/nova/+/773192
#openstack-nova - 2021-01-30
00:29:03	brtknr	sean-k-mooney: is there a way to increase retries with cinder api? also how can i make use of service tokens?
00:29:54	brtknr	thanks for your detailed reply btw
14:36:33	openstackgerrit	Khomesh Thakre proposed openstack/nova master: Adding a validation on os-resetState so that when vm is shelved offload, reseting the VM state is blocked. In shelved state VM doesnt existed physically on compute nodes. https://review.opendev.org/c/openstack/nova/+/773238
14:42:42	openstackgerrit	Khomesh Thakre proposed openstack/nova master: Adding a validation on os-resetState so that when vm is shelved offload, reseting the VM state is blocked. In shelved state VM does not existed physically on compute nodes. https://review.opendev.org/c/openstack/nova/+/773238
15:22:59	openstackgerrit	Jessie Lass proposed openstack/nova master: Add emulation support if host arch != guest arch. https://review.opendev.org/c/openstack/nova/+/772156
16:57:08	openstackgerrit	Khomesh Thakre proposed openstack/nova master: Adding a validation on os-resetState so that when vm is shelved offload, reseting the VM state is blocked. In shelved state VM doesnt existed physically on compute nodes. https://review.opendev.org/c/openstack/nova/+/773238
17:06:28	openstackgerrit	Merged openstack/nova stable/ussuri: Omit resource inventories from placement update if zero https://review.opendev.org/c/openstack/nova/+/766825
17:29:41	openstackgerrit	Merged openstack/nova master: api: Reject requests to detach a volume when the compute is down https://review.opendev.org/c/openstack/nova/+/768352
17:38:48	openstackgerrit	Merged openstack/nova stable/victoria: Fallback to same-cell resize with qos ports https://review.opendev.org/c/openstack/nova/+/769727
17:51:48	openstackgerrit	Merged openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/c/openstack/nova/+/754700
17:54:21	openstackgerrit	Merged openstack/nova master: libvirt: Remove 'hypervisor_version' from 'libvirt_info' https://review.opendev.org/c/openstack/nova/+/744199
20:25:08	openstackgerrit	Merged openstack/nova master: db: Compact Kilo database migrations https://review.opendev.org/c/openstack/nova/+/758396
#openstack-nova - 2021-02-01
08:40:03	hemanth_n	Hi Nova, tempest-slow job fails for the following test cases tempest.scenario.test_network_v6.* on stable/rocky
09:02:41	lyarwood	hemanth_n: can you raise a neutron bug for that?
09:04:53	lyarwood	hemanth_n: https://review.opendev.org/q/Id5d8ac09a38c656619f88a6f87b8f384fe4c55a8 - I assume it has something to do with this landing
09:06:21	lyarwood	gibi: \o morning, ack re your changes in the detach series, I'll go over them later today, FYI I'm offline on Friday's until March
09:08:40	hemanth_n	@lyarwood: wil check and raise bug accordingly, thanks for taking a quick look
09:09:13	lyarwood	hemanth_n: np and thanks
09:22:15	openstackgerrit	Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1909120 https://review.opendev.org/c/openstack/nova/+/773320
09:22:18	openstack	bug 1909120 in OpenStack Compute (nova) "n-api should reject requests to detach a volume when the compute is down" [Medium,Confirmed] https://launchpad.net/bugs/1909120 - Assigned to Lee Yarwood (lyarwood)
09:22:19	openstackgerrit	Lee Yarwood proposed openstack/nova stable/victoria: api: Reject requests to detach a volume when the compute is down https://review.opendev.org/c/openstack/nova/+/773321
09:28:06	gibi	lyarwood: o/, thanks
09:45:25	openstackgerrit	Lee Yarwood proposed openstack/nova stable/ussuri: Add regression test for bug #1909120 https://review.opendev.org/c/openstack/nova/+/773333
09:45:27	openstack	bug 1909120 in OpenStack Compute (nova) "n-api should reject requests to detach a volume when the compute is down" [Medium,Confirmed] https://launchpad.net/bugs/1909120 - Assigned to Lee Yarwood (lyarwood)
09:45:28	openstackgerrit	Lee Yarwood proposed openstack/nova stable/ussuri: api: Reject requests to detach a volume when the compute is down https://review.opendev.org/c/openstack/nova/+/773334
10:24:18	elod	I've pushed victoria + ussuri + train release patches: https://review.opendev.org/q/project:openstack/releases+intopic:nova-+status:open
10:24:27	elod	gibi lyarwood bauzas : ^^^^
10:24:57	elod	I did not find any patch that would worth minor version bump
10:25:34	elod	also, let me know if you think there's any patch that should be part of a release
10:26:02	lyarwood	elod: ack I'll review later this morning, thanks!
10:27:06	elod	(the Train release is a ~'last but final' release before entering Extended Maintenance in May, and I thought it's a good idea to release Victoria and Ussuri together, too)
10:27:33	elod	lyarwood: ++
10:48:22	bauzas	elod: ack, will look this afternoon
10:53:34	elod	bauzas: cool, thx
10:58:54	openstackgerrit	Merged openstack/os-traits master: Remove translation sections from setup.cfg https://review.opendev.org/c/openstack/os-traits/+/728160
11:04:04	openstackgerrit	Stephen Finucane proposed openstack/os-vif master: Drop use of deprecated collections classes https://review.opendev.org/c/openstack/os-vif/+/773360
11:07:41	hemanth_n	lyarwood: bug raised, for your reference, https://bugs.launchpad.net/neutron/+bug/1914037
11:07:42	openstack	Launchpad bug 1914037 in neutron "scenario tests tempest.scenario.test_network_v6.TestGettingAddress fails" [Undecided,New]
12:35:00	openstackgerrit	Lee Yarwood proposed openstack/nova master: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/766976
12:35:02	openstackgerrit	Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380
12:35:02	openstackgerrit	Lee Yarwood proposed openstack/nova master: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/768472
12:35:02	openstack	bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Undecided,New] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood)
12:40:13	openstackgerrit	Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380
12:40:41	lyarwood	stephenfin: https://review.opendev.org/c/openstack/nova/+/772869 - would you mind hitting this today, trivial fix needed downstream
12:40:50	stephenfin	sure
12:41:20	lyarwood	stephenfin: thanks
12:46:07	lyarwood	gibi: https://review.opendev.org/c/openstack/nova/+/772869 - would you mind taking a look at this also, hopefully trivial but let me know if you have any issues with it.
12:48:47	lyarwood	https://review.opendev.org/q/topic:nova-lvm - not urgent but if any cores have time, I'm trying to get nova-lvm voting again here
12:56:29	sean-k-mooney	stephenfin: https://review.opendev.org/c/openstack/os-vif/+/773360 passed everthing but lower constraits
12:56:43	sean-k-mooney	stephenfin: there is a patch to drop lc but i would prefer to fix it if we can
12:56:54	sean-k-mooney	stephenfin: in this case it seams like its just hacking
12:57:15	sean-k-mooney	stephenfin: do you have time to address that in a seperate patch if not ill take a crack at it later this week
12:57:23	stephenfin	I can take a look
12:58:10	sean-k-mooney	i think the decision was to only list direct deps in lc right
12:58:21	sean-k-mooney	i was only partly following the ml thread
13:08:27	gibi	elod: ack, on it
13:59:38	gibi	lyarwood: I'm not familiar with rdb just wondering if user name is enough, is there some kind of password / key that also needs to be provided to access the encrypted disk? https://review.opendev.org/c/openstack/nova/+/772869
14:07:39	lyarwood	gibi: https://docs.ceph.com/en/latest/rbd/qemu-rbd/#usage - I can repsin with this link that explains this a little but tl;dr the username refers to the a local file (the keyring) used to authenticate the request
14:07:51	lyarwood	to a*
14:08:14	gibi	lyarwood: no need to respin, thanks
14:34:33	openstackgerrit	Lee Yarwood proposed openstack/nova master: docs: Move the LibvirtDistroSupportMatrix wiki page into our docs https://review.opendev.org/c/openstack/nova/+/771981
14:42:31	dansmith	gibi: if you could +W this, it's blocking glance's ability to land the policy goal patches: https://review.opendev.org/c/openstack/nova/+/773192
14:42:41	dansmith	since we define the job, we're holding them hostage :)
14:42:56	dansmith	(or anyone else ^)
14:43:27	sean-k-mooney	ya that makes sense
14:44:21	gibi	dansmith: looking
14:44:27	sean-k-mooney	dansmith: i assume we are not doing that policy file upstae in [post-config| /etc/glance/policy.json]
14:44:29	sean-k-mooney	for some reason
14:44:58	sean-k-mooney	the default policy file shoudl be empty
14:45:29	sean-k-mooney	also i guess at somepoint that need to be a policy.yaml file
14:46:04	lyarwood	would post-config bounce the service?
14:46:17	sean-k-mooney	no but it runs before its started
14:46:22	sean-k-mooney	so it does not need too
14:46:43	sean-k-mooney	post-config is after all the files are generated but before the services are started for the first time
14:47:03	sean-k-mooney	excluddign rabbit and mariadb which start before post-config for reasons
14:47:04	lyarwood	ah understood
14:47:32	sean-k-mooney	if that was doing something more complicated post-cofnig would not work
14:47:52	sean-k-mooney	but for this i think it would and they could then override it by inheriting form the job if they needed too
14:48:13	sean-k-mooney	as a quick fix however that will work hence +1
14:50:13	openstackgerrit	Lee Yarwood proposed openstack/nova master: Add regression test for bug #1908075 https://review.opendev.org/c/openstack/nova/+/766976
14:50:15	openstackgerrit	Lee Yarwood proposed openstack/nova master: fup: Merge duplicate volume attachment checks https://review.opendev.org/c/openstack/nova/+/773380
14:50:15	openstackgerrit	Lee Yarwood proposed openstack/nova master: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/768472
14:50:15	openstack	bug 1908075 in OpenStack Compute (nova) "Nova allows a non-multiattach volume to be attached to multiple instances *if* its volume state is reset by an admin" [Undecided,New] https://launchpad.net/bugs/1908075 - Assigned to Lee Yarwood (lyarwood)
14:52:16	gibi	dansmith: done

Earlier   Later