Posted	Nick	Remark
#openstack-nova - 2020-08-07
15:24:31	gibi	I'm wrapping up my week now. I'll be on PTO during the next two weeks. See you around 24th!
15:53:07	openstackgerrit	Merged openstack/nova master: Add new default roles in security_groups policies https://review.opendev.org/742763
15:53:14	openstackgerrit	Merged openstack/nova master: Pass the actual target in security_groups policy https://review.opendev.org/742764
15:53:21	openstackgerrit	Merged openstack/nova master: Add test coverage of networks policies https://review.opendev.org/742558
15:53:32	openstackgerrit	Merged openstack/nova master: Introduce scope_types in networks policy https://review.opendev.org/742561
15:53:51	openstackgerrit	Merged openstack/nova master: Add test coverage of volumes policies https://review.opendev.org/742773
15:53:58	openstackgerrit	Merged openstack/nova master: Introduce scope_types in volumes policy https://review.opendev.org/742774
15:58:06	openstackgerrit	Merged openstack/nova master: Add new default roles in volumes policies https://review.opendev.org/742777
16:10:23	elod	lyarwood: all good and +2'd :]
16:29:54	openstackgerrit	Merged openstack/nova master: Add test coverage of hosts policies https://review.opendev.org/743013
17:14:15	openstackgerrit	Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421
17:18:28	openstackgerrit	Merged openstack/nova master: Introduce scope_types in hosts policy https://review.opendev.org/743014
17:19:56	lyarwood	elod: thanks :)
17:26:29	smcginnis	Anyone know if the AOE and DISCO libvirt volume drivers are still used/supported?
17:26:39	smcginnis	They were both dropped from Cinder a long, long time ago.
17:27:00	smcginnis	I was looking at cleaning up some remnants in os-brick, but it looks like there is still some code present in nova.
17:31:29	sean-k-mooney	smcginnis: if they were droped form cinder then its dead code
17:31:42	lyarwood	I think they are only in nova because they were in cinder and os-brick so rm -rf away
17:31:51	lyarwood	yeah
17:32:47	sean-k-mooney	smcginnis: nova only uses os-brick for cinder volumes. if the cinder backend is remvoe then the nova code is nolonger useful in general unlesss the code is shared
17:33:17	sean-k-mooney	also wehn i see AOE i think fo Age of Empires
17:34:02	lyarwood	same haha
17:35:13	melwitt	Area Of Effect
17:41:39	lyarwood	^_^
17:43:55	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: migrate testing to ubuntu focal https://review.opendev.org/738126
17:54:27	openstackgerrit	Merged openstack/nova master: Add new default roles in hosts policies https://review.opendev.org/743015
17:54:33	openstackgerrit	Merged openstack/nova master: Add test coverage of extensions policies https://review.opendev.org/743045
18:01:24	lyarwood	melwitt: https://review.opendev.org/#/q/topic:bug/1889108+status:open - would you mind taking a look at that today if you have time
18:01:43	melwitt	lyarwood: sure thing
18:02:55	lyarwood	thanks
18:06:12	smcginnis	sean-k-mooney, lyarwood: Thanks, sounds good.
18:06:38	smcginnis	Do you think it needs to go through a deprecation cycle in nova? Or since it's just dead code anyway, all clear to just clean it out?
18:08:11	lyarwood	smcginnis: I think we can just remove it tbh
18:09:26	smcginnis	Sounds good. I'll put up something to do that. Then we can see if anyone has any objects on the patch.
18:09:33	lyarwood	smcginnis: the only way of loading these volume drivers is through cinder providing connection_info with driver_volume_type set to aoe or disco https://github.com/openstack/nova/blob/09ed4c21db5005b1c699a87bfb0e7cb051553342/nova/virt/libvirt/driver.py#L1593-L1597
18:09:56	lyarwood	smcginnis: if the c-vol drivers are dead then that can't happen
18:09:57	smcginnis	Ah, that's how that works. Thanks, I've never dug into the nova side of these things.
18:10:07	smcginnis	So this really has been dead code for quite awhile then.
18:10:12	lyarwood	yup
18:11:18	lyarwood	smcginnis: actually thinking about it they were dropped more than a release ago from cinder right?
18:11:42	smcginnis	Yeah. Like Ocata for some.
18:11:45	lyarwood	ha
18:11:49	lyarwood	okay rm -rf away sorry
18:12:31	smcginnis	There might be more than those two, but if there's anything I see that hasn't been gone since at least Stein, I'll leave it for later cleanup.
18:15:26	lyarwood	smcginnis: are the removals from cinder tracked anywhere outside of the releasenotes btw? Like an easy to read support table or something
18:16:24	smcginnis	We've started just marking them as unsupported, so there haven't been any new removals for at least a cycle now.
18:16:39	smcginnis	I think we did document the older ones though. Let me see if I can find that.
18:17:01	smcginnis	Probably incomplete though. I had to dig a bit to even figure out what the AOE connector was.
18:17:09	smcginnis	(assumed we didn't connect to the game ;) )
18:20:37	lyarwood	haha
18:23:28	openstackgerrit	Merged openstack/nova master: Add test coverage of FIP pools policies https://review.opendev.org/741795
18:23:41	openstackgerrit	Merged openstack/nova master: Add test coverage of baremetal nodes policies https://review.opendev.org/741707
18:33:22	smcginnis	lyarwood: Some incomplete driver removal history: https://docs.openstack.org/cinder/latest/reference/support-matrix.html#driver-removal-history
18:36:20	openstackgerrit	Merged openstack/os-vif master: [goal] migrate testing to ubuntu focal https://review.opendev.org/738130
18:42:39	lyarwood	smcginnis: thanks
19:01:30	openstackgerrit	Sean McGinnis proposed openstack/nova master: Remove dead volume driver code https://review.opendev.org/745393
19:30:28	openstackgerrit	Merged openstack/nova master: Fix multinic policy for admin_or_owner https://review.opendev.org/742317
19:30:35	openstackgerrit	Merged openstack/nova master: Add test coverage of multinic policies https://review.opendev.org/742315
19:30:48	openstackgerrit	Merged openstack/nova master: Pass the actual target in tenant networks policy https://review.opendev.org/742772
19:30:56	openstackgerrit	Merged openstack/nova master: Introduce scope_types in FIP pools https://review.opendev.org/741796
19:38:38	openstackgerrit	Merged openstack/nova master: Introduce scope_types in baremetal node https://review.opendev.org/741709
19:40:54	mloza	hello, I have running instance which is launched with a invalid AZ and now, I can't do live migration. Where can I fix the issue in the db?
20:30:16	mnaser	mloza: i didn't tell you this but nova.instances.availability_zone
20:30:32	mnaser	that is an ID though
20:32:08	mnaser	mloza: but you can use an id from one that you know is right, but its friday and you should play with db :)
21:04:03	openstackgerrit	Merged openstack/nova master: Introduce scope_types in multinic policy https://review.opendev.org/742328
21:26:16	mloza	mnaser: it shows the correct AZ in the nova db
21:26:26	mnaser	maybe check nova_api
21:27:49	mloza	look like its in nova_api db, it's the request_specs table which shows the invalid AZ
21:27:56	mloza	manage to get it corrected
21:27:58	mloza	update request_specs set spec = replace (spec,'"availability_zone": "old-name"','"availability_zone": "new-name"') where instance_uuid = 'xxxxxxxxxxxxxxx';
23:14:06	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP libvirt: Remove host CPU checks during check_can_live_migrate_destination https://review.opendev.org/745431
#openstack-nova - 2020-08-08
00:37:57	openstackgerrit	Merged openstack/nova stable/ussuri: tests: Define constants in '_IntegratedTestBase' https://review.opendev.org/744158
00:38:02	openstackgerrit	Merged openstack/nova stable/ussuri: func: Add _live_migrate helper to InstanceHelperMixin https://review.opendev.org/744159
00:39:17	openstackgerrit	Merged openstack/nova stable/ussuri: func: Add live migration rollback volume attachment tests https://review.opendev.org/744160
00:39:52	openstackgerrit	Merged openstack/nova stable/ussuri: Add regression tests for bug #1889108 https://review.opendev.org/744161
00:39:52	openstack	bug 1889108 in OpenStack Compute (nova) ussuri "failures during driver.pre_live_migration remove source attachments during rollback" [Undecided,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)
12:10:08	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512
12:10:20	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565
12:10:34	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898
12:24:03	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512
12:24:51	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565
12:25:17	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898
12:36:32	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224
12:37:16	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757
12:37:30	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777
12:37:51	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094
12:38:22	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109
12:51:17	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117
18:07:54	openstackgerrit	Merged openstack/nova master: Pass the actual target in volumes policy https://review.opendev.org/742779
22:37:51	openstackgerrit	Merged openstack/nova stable/ussuri: compute: Don't delete the original attachment during pre LM rollback https://review.opendev.org/744162
#openstack-nova - 2020-08-09
23:18:00	openstackgerrit	Hans Hazen proposed openstack/nova master: Fix FIP policy for admin_or_owner https://review.opendev.org/741811
#openstack-nova - 2020-08-10
04:22:55	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Raising a failure in case of 403 https://review.opendev.org/735068
06:12:24	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Raising a failure in case of 403 https://review.opendev.org/735068
06:52:49	openstackgerrit	Xinran WANG proposed openstack/nova-specs master: SRIOV Enabled Nic Support Specification https://review.opendev.org/742785
09:36:36	lyarwood	nova.tests.unit.virt.libvirt.storage.test_rbd.RbdTestCase.test_cleanup_volumes_fail_not_found 55.090 - hmm that isn't good
10:04:04	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Mock LoopingCallBase._sleep in rbd tests https://review.opendev.org/745512
10:04:15	lyarwood	stephenfin: ^ something nice and trivial for you on a Monday
10:16:29	jsuchome	lyarwood: good morning, I am wondering ... anything new about setup for https://review.opendev.org/#/c/743220/ ?
11:15:09	openstackgerrit	Merged openstack/nova stable/train: catch libvirt exception when nodedev not found. https://review.opendev.org/745116

Earlier   Later