Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-08-07
13:56:21 sean-k-mooney its been quite a while since i looked at them
13:57:43 sean-k-mooney but ya that would be quite nice if it coudl be made to work
13:58:50 sean-k-mooney mnaser: https://docs.openstack.org/keystoneauth/latest/authentication-plugins.html
13:59:05 mnaser sean-k-mooney: it actually should 100% work :)
13:59:11 mnaser keystoneauth does support it yeah
14:07:09 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Add reproducer for bug #1879878 https://review.opendev.org/744950
14:07:09 openstack bug 1879878 in OpenStack Compute (nova) "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Medium,In progress] https://launchpad.net/bugs/1879878 - Assigned to Stephen Finucane (stephenfinucane)
14:07:09 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/744958
14:07:09 openstackgerrit Stephen Finucane proposed openstack/nova master: compute: Add type hints for resize functions https://review.opendev.org/745341
14:31:22 lyarwood elod: https://review.opendev.org/#/q/topic:bug/1889108+status:open - would you have anytime to look at this today
14:31:52 lyarwood elod: I'm including a number of func test refactors here as I'd like to keep things sane going back to stable/train
14:37:54 openstackgerrit Merged openstack/nova master: Add test coverage of tenant networks policies https://review.opendev.org/742765
14:38:01 openstackgerrit Merged openstack/nova master: Introduce scope_types in tenant networks policy https://review.opendev.org/742766
14:38:10 openstackgerrit Merged openstack/nova master: Add new default roles in tenant networks policies https://review.opendev.org/742771
14:38:27 openstackgerrit Merged openstack/nova master: Add test coverage of security_groups policies https://review.opendev.org/742726
14:38:34 openstackgerrit Merged openstack/nova master: Introduce scope_types in security_groups policy https://review.opendev.org/742757
14:40:55 elod lyarwood: sure, looking!
14:41:14 lyarwood elod: thanks! :)
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

Earlier   Later