Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-04-23
21:31:15 aspiers efried: not just with danpb but already covered in the spec
21:31:21 aspiers sean-k-mooney: yeah most likely
21:33:13 sean-k-mooney jaypipes: thanks for reviewing the sriov mighration pathces :)
21:33:45 sean-k-mooney now if only stephenfin or maybe efried could leave the final +2+w it could merge before the ptg
21:34:03 sean-k-mooney or more importantly before one of the 41 conflicting patches does
21:34:19 efried sean-k-mooney: link please
21:34:29 sean-k-mooney https://review.opendev.org/#/c/629589/27
21:34:46 sean-k-mooney that and https://review.opendev.org/#/c/620115/35
21:34:53 efried When you say things like sriov and live migration I immediately think, "I hope someone else has got this."
21:35:27 aspiers sean-k-mooney, efried: BTW funnily enough Brijesh's reply when I asked why there is a limit "We have limited number of slots in memory controller, each slot gets a different encryption key."
21:35:43 efried hah!
21:35:45 sean-k-mooney ya
21:35:52 efried MEM_ENCRYPTION_KEY_SLOTS
21:36:06 efried again, pretty impl specific. Let's stick with CONTEXT
21:36:17 aspiers OK X-D
21:36:52 sean-k-mooney it had to be somthign like that e.g. they what a fix number of keys they coudl handel
21:36:57 aspiers yeah
21:37:27 sean-k-mooney with this kid of thing you donw want those stored in ram which means sticking them in a register file somewhere
21:37:56 sean-k-mooney and you only have so much spcae on a die to add that memory so there will always be a limit like this
21:38:19 sean-k-mooney at least 15/16 is resonable
21:38:47 aspiers right
21:38:53 sean-k-mooney i remember being asked to enable QOS policies with intel QAT devices at one point where the limit was 4
21:39:00 aspiers haha
21:39:05 sean-k-mooney you could have 32 VF
21:39:20 sean-k-mooney but only 4 QOS policies
21:39:34 sean-k-mooney and it was a 1:1 maping between QOS policey and a singel VF
21:39:53 sean-k-mooney if you used any of the other ones they ignroed the qos policy
21:40:06 sean-k-mooney we never enabeld in in openstack.
21:40:11 openstackgerrit Matt Riedemann proposed openstack/nova master: AZ list performance optimization: avoid double service list DB fetch https://review.opendev.org/636947
21:40:16 mriedem efried: i found and cleaned this up with test wrinkles and addressed some of my own comments ^ let me know if you think it needs more
21:40:25 mriedem but yeah that's a really nice perf improvement from avolkov
21:41:01 efried mriedem: ack, looking.
21:41:20 efried are you recusing yourself from approving at this point?
21:41:29 sean-k-mooney mriedem: ya it look like its about half the time
21:41:48 mriedem efried: not necessarily
21:41:55 mriedem i just added test stuff and some code comments
21:43:24 openstackgerrit Merged openstack/python-novaclient master: Updates for OpenDev transition https://review.opendev.org/654429
21:43:45 sean-k-mooney ya the delta for what you added is mainly non functional changes
21:45:09 sean-k-mooney so is the main sped up form getting the avilablity zones for jsut the enabled serivces
21:46:27 efried mriedem: lgtm. Why did you switch from monkeypatch to mock.patch?
21:46:38 mriedem so i can count the calls
21:46:46 mriedem silly pants
21:47:10 mriedem sean-k-mooney: it's that we don't get enabled services across 10K hosts twice
21:47:41 aspiers did you folks spot the pysnooper tip?
21:48:04 sean-k-mooney ah
21:48:11 sean-k-mooney ya
21:48:25 sean-k-mooney im not sure if ill end up using it or not but we will see
21:48:46 aspiers it looks pretty nice
21:52:49 sean-k-mooney ya but im not sure it will actully be helpful fo more complicate things. but next time i ned to debug some thing complex and im not usgin a grapical debugger ill give it a try
21:53:46 efried mriedem: oh, stub_out doesn't return the fixture. And apparently MonkeyPatch doesn't have a .mock attribute, like MockPatch does. Remind me why we have stub_out?
21:53:51 efried (rhetorical)
21:55:34 gmann mriedem: RE: on job, yes it will merge the var from parent and derived jobs.
21:57:25 gmann mriedem: few zuul var are not append-able for example files, irrelevant-files etc which are regex not array. but devstack var are all merged from all hierarchy of job
21:59:44 mriedem jaypipes: fat ol -2 on this https://review.opendev.org/#/c/625755/ if you remember it
21:59:56 mriedem efried: to get off mox?
22:00:24 mriedem gmann: yup thanks. i'm now just waiting a few days for ci results. :)
22:07:08 sean-k-mooney mriedem: by the way did you want to review the sriov migrtion stuff or are you happy to leave it to jay and stephen
22:09:00 mriedem want to?
22:09:11 mriedem why don't you have it in a runway if you're looking for reviews?
22:09:23 sean-k-mooney oh good point
22:12:21 sean-k-mooney its going to take w while form e to get opendev.org into fingre memory. im really glad the got the redirects in place
22:12:52 sean-k-mooney anyway night all o/
22:15:58 mriedem speaking of it looks like we have busted jobs on stable branches b/c the openstack-infra/devstack-gate change wasn't made there in our .zuul.yaml
22:19:35 mriedem eandersson: are you going to work on backports for these? https://review.opendev.org/#/q/topic:bug/1825583+status:open+project:openstack/nova
22:20:36 eandersson If I get the time this week. Got a busy schedule, and well next week is Denver.
22:20:46 eandersson So can't promise that I will be able to do it before Denver.
22:20:48 mriedem minions...
22:20:50 eandersson =]
22:30:06 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Move legacy-grenade-dsvm-neutron-multinode-live-migration in-tree https://review.opendev.org/640207
22:42:41 efried o/
22:50:46 openstackgerrit Merged openstack/nova master: Only set oslo_messaging_notifications.driver if using RPCFixture https://review.opendev.org/653954
22:56:22 openstackgerrit Adam Spiers proposed openstack/nova-specs master: Re-approve AMD SEV support for Train https://review.opendev.org/641994
23:02:52 aspiers efried, sean-k-mooney, jaypipes: patch set 12 now up, hopefully the final one https://review.opendev.org/#/c/641994/11..12//COMMIT_MSG
23:02:58 openstackgerrit Ivens Zambrano proposed openstack/nova-specs master: RMD Plugin: Energy Efficiency using CPU Core P-State control The power state of a core can be setup between a minimum and the maximum frequency on the cores as defined in the factory. Each core on a CPU can be defined individually to perform at specific f https://review.opendev.org/651024
23:09:40 openstackgerrit Adam Spiers proposed openstack/nova master: Add infrastructure for invoking libvirt's getDomainCapabilities API https://review.opendev.org/655268
23:12:34 openstackgerrit Tony Breeds proposed openstack/nova stable/pike: Move legacy-grenade-dsvm-neutron-multinode-live-migration in-tree https://review.opendev.org/640207
23:27:54 openstackgerrit Adam Spiers proposed openstack/nova master: Add detection of SEV support from QEMU/AMD-SP/libvirt on AMD hosts https://review.opendev.org/633855
23:32:36 aspiers kashyap: ^^^ I finally gave you what you wanted ;-)
23:32:53 aspiers you'll note that the SEV patch is extremely readable now
#openstack-nova - 2019-04-24
00:34:27 openstackgerrit melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.opendev.org/650171
00:34:28 openstackgerrit melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.opendev.org/650172
00:34:28 openstackgerrit melanie witt proposed openstack/nova master: Revert "Fix target_cell usage for scatter_gather_cells" https://review.opendev.org/653894
01:14:01 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Don't persist _nova_check_type flag in database https://review.opendev.org/655278
01:45:04 openstackgerrit Michael Still proposed openstack/nova master: Privsepify ipv4 forwarding enablement. https://review.opendev.org/635431
01:45:05 openstackgerrit Michael Still proposed openstack/nova master: Privsep the ebtables modification code. https://review.opendev.org/635435
01:45:05 openstackgerrit Michael Still proposed openstack/nova master: Remove unused FP device creation and deletion methods. https://review.opendev.org/635433
01:45:06 openstackgerrit Michael Still proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.opendev.org/636508
01:45:06 openstackgerrit Michael Still proposed openstack/nova master: Move adding vlans to interfaces to privsep. https://review.opendev.org/635436
01:45:07 openstackgerrit Michael Still proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.opendev.org/639281
01:45:07 openstackgerrit Michael Still proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.opendev.org/639280
01:45:08 openstackgerrit Michael Still proposed openstack/nova master: Move setting of device trust to privsep. https://review.opendev.org/639283
01:45:08 openstackgerrit Michael Still proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.opendev.org/639282
01:45:09 openstackgerrit Michael Still proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.opendev.org/639581
01:45:09 openstackgerrit Michael Still proposed openstack/nova master: Move final bridge commands to privsep. https://review.opendev.org/639580
01:45:10 openstackgerrit Michael Still proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.opendev.org/639826
01:45:10 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.opendev.org/554438
01:45:11 openstackgerrit Michael Still proposed openstack/nova master: Remove no longer required "inner" methods. https://review.opendev.org/655282
01:45:11 openstackgerrit Michael Still proposed openstack/nova master: Improve test coverage of nova.privsep.utils. https://review.opendev.org/655281
02:01:05 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
02:08:28 openstackgerrit Merged openstack/nova master: Fix {min|max}_version in ironic Adapter setup https://review.opendev.org/654457

Earlier   Later