Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-06-12
18:31:47 aspiers I missed that that patch does other stuff
18:31:49 aspiers OK
18:32:08 sean-k-mooney e.g. without looking too close i think we dont handel stop followed by start proerly
18:32:11 efried yeah, the patch I pointed to has nothing to do with SEV. It has to do with "hide hypervisor ID to make NVIDIA drivers work
18:32:13 efried "
18:32:20 aspiers yeah, got it now
18:32:37 aspiers yikes, feels like I need another 40k spec just to remember all the plans we made in the last hour
18:32:52 efried You don't need to update the spec with any of this.
18:32:53 aspiers thank god for IRC logs
18:33:13 aspiers I know, was just being silly :)
18:33:32 aspiers it's a lot of detail for a newbie to remember, but I think I'll be ok
18:33:49 sean-k-mooney anyway this time im really going to go and rest. o/
18:33:52 aspiers I guess my right to play the newbie card is gradually fading though
18:33:58 aspiers cya sean-k-mooney, thanks for your help o/
18:33:59 efried - existing usages of ^ logic cut over to using ^, converting exceptions to contextually-appropriate ones.
18:33:59 efried - ^ methods raise exceptions
18:33:59 efried - create two consolidated methods in hardware.py: a) parse/validate flavor/image for SEV yes-or-no; b) introspect host for SEV capability
18:34:26 aspiers I think b) and callers of b) do not need to change
18:34:29 aspiers it's only a)
18:34:36 efried ight
18:34:41 aspiers b) is all in the driver
18:35:00 aspiers and available via the supports_amd_sev instance variable
18:35:21 efried "instance" == compute driver instance, not "instance" == "nova server"
18:35:38 aspiers sure :)
18:36:07 aspiers and then there's Instance instances too :)
18:36:35 aspiers OK, gonna take a break or probably stop for today
18:36:44 aspiers thanks a lot all
18:37:46 efried dustinc: Not rechecking the ironicclient part of the sdk series for yesterday's doubled_words snafu - pep8 needs resolving on the lowermost one.
18:39:44 dustinc efried: saw that, once I am done with the spec update I am working on I will take care of it - thanks
18:40:10 efried dustinc: no hurry, just letting you know why it's not going to be included in my "recheck everything" sweep :P
18:40:39 efried dustinc: also lmk if at some point you want to talk through providers.yaml
18:42:16 dustinc efried: providers.yaml: will do..I def. want to at some point
19:10:13 openstackgerrit Eric Fried proposed openstack/nova master: Make RequestContext(instance_lock_checked) fail https://review.opendev.org/665003
19:10:25 efried mriedem: resolves a TODO of yours ^
19:11:38 openstackgerrit Merged openstack/nova master: Remove unnecessary setUp methods https://review.opendev.org/663179
19:12:15 mriedem efried: does it or does it not have to do with SEV
19:13:37 efried does not
19:14:02 efried I was looking into removing a hack for glance using get_endpoint_data through _ContextAuthPlugin (fail) and just happened to notice this.
19:14:19 efried mriedem: what was that bug you opened for the doubled_words hacking test yesterday? Did you mark it as a dup of https://bugs.launchpad.net/nova/+bug/1804062 already?
19:14:21 openstack Launchpad bug 1804062 in OpenStack Compute (nova) "test_hacking fails for python 3.6.7 and newer" [High,In progress] - Assigned to Stephen Finucane (stephenfinucane)
19:15:24 mriedem yes
19:15:33 mriedem https://bugs.launchpad.net/nova/+bug/1832392
19:15:35 openstack Launchpad bug 1804062 in OpenStack Compute (nova) "duplicate for #1832392 test_hacking fails for python 3.6.7 and newer" [High,In progress] - Assigned to Stephen Finucane (stephenfinucane)
19:15:36 mriedem efried: question in your patch
19:16:28 melwitt efried: after the discussion about nested magic 1 in office hours today, how are you feeling about the it vs the nova spec for numa affinity for vgpus?
19:17:43 openstackgerrit Eric Fried proposed openstack/nova master: Make RequestContext(instance_lock_checked) fail https://review.opendev.org/665003
19:17:53 efried mriedem: done
19:18:15 melwitt I wasn't sure if I understood whether nested magic 1 will take much more effort than previously believed
19:19:40 efried melwitt: Components of it - specifically the ones that allow us to do NUMA modeling/affinity for simple cases like VGPU - are not under conflict. I still think the VGPU numa affinity thing doesn't move us forward in a useful way, and just introduces tech debt we'll have to remove very soon.
19:19:58 efried but I don't feel strongly enough to downvote.
19:20:09 openstackgerrit Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881
19:20:10 openstackgerrit Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
19:20:10 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Include direct-physical in compute manager events check https://review.opendev.org/664431
19:21:09 melwitt efried: ack. and do you know if there are people available to work on nested magic 1 or does it need an owner?
19:21:24 melwitt for the implementation after the spec merges
19:23:17 dansmith artom: ooh, did somebody find a bug while writing their tests? :)
19:23:35 artom dansmith, I don't think so?
19:23:52 dansmith artom: okay, just looking at your events=None change in the latest patch
19:23:52 artom But parts of the code were untested, yeah
19:24:04 artom Oh, err, yeah
19:24:15 artom That may have been a problem :/
19:24:19 dansmith sweet
19:24:26 dansmith now I feel less bad
19:27:00 artom dansmith, so not only am I making Nova better, I'm also contributing to your mental health
19:27:11 artom Come to think of it, maybe the latter isn't an advantage >;)
20:39:19 mriedem dansmith: for when you're on the can http://lists.openstack.org/pipermail/openstack-discuss/2019-June/007097.html
20:49:57 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live_migration_claim() https://review.opendev.org/635669
20:49:58 openstackgerrit Artom Lifshitz proposed openstack/nova master: LM: add support for sending NUMAMigrateData to the source https://review.opendev.org/634828
20:49:58 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827
20:49:59 openstackgerrit Artom Lifshitz proposed openstack/nova master: RPC changes to prepare for NUMA live migration https://review.opendev.org/634605
20:49:59 openstackgerrit Artom Lifshitz proposed openstack/nova master: LM: add support for updating NUMA-related XML on the source https://review.opendev.org/635229
20:50:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: CONF.enable_numa_live_migration is not needed >= Stein https://review.opendev.org/640021
20:50:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606
20:50:55 efried melwitt: cdent has been picking up most of it one way or another. More bodies would be nice, but probably not critical.
20:51:18 efried melwitt: and we've not been sticking to "spec must merge first" rules. We've already cut a microversion with part of the feature set.
20:51:45 melwitt oh, interesting. ok
20:52:52 efried yeah, it's the wild wild west over here in placement-land
20:55:07 melwitt heh
20:55:57 melwitt well, if there's anything I can do to help, give me a holler
21:02:00 efried melwitt: oo, now you're in for it :P
21:03:40 melwitt :)
21:23:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.opendev.org/614036
21:23:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.opendev.org/614012
21:23:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.opendev.org/631123
21:23:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.opendev.org/633853
21:23:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.opendev.org/631581
21:23:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.opendev.org/627892
21:23:06 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_dest compute method https://review.opendev.org/633293
21:23:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.opendev.org/634832
21:23:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtDestTask https://review.opendev.org/627890
21:23:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtSourceTask https://review.opendev.org/627891
21:23:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova.compute.utils.delete_image https://review.opendev.org/637605
21:38:30 efried dansmith: Thanks for the review on the providers.yaml spec. I agree with you (mark your calendar).
22:54:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.opendev.org/576020
22:54:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.opendev.org/576027
22:54:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.opendev.org/576031
23:10:42 openstackgerrit Merged openstack/nova stable/rocky: Fix live-migration when glance image deleted https://review.opendev.org/662153
23:21:09 openstackgerrit Merged openstack/nova master: Refresh instance network info on deletion https://review.opendev.org/660761
23:22:27 openstackgerrit sean mooney proposed openstack/os-vif master: set ignore_basepython_conflict = True in tox.ini https://review.opendev.org/665034
23:36:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.opendev.org/576299

Earlier   Later