Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-03
13:57:20 gibi_cape efried: the summaries are really good. Thanks!
13:57:39 efried thanks gibi_cape
14:16:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
14:22:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
15:37:22 johnthetubaguy gmann: cool, thanks, will take a peak
15:37:38 johnthetubaguy jaypipes: I think this is what I had in my head: https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/compute/manager.py#L1838
15:38:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.opendev.org/637716
15:38:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Enable cross-cell resize in the nova-multi-cell job https://review.opendev.org/656656
15:46:03 mriedem johnthetubaguy: this is where the global lock is during the resource claim https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/compute/manager.py#L2196
15:53:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-status upgrade check for minimum required cinder API version https://review.opendev.org/649759
15:53:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061
16:05:04 mriedem mdbooth: are you cool with dansmith's image disk_format capability patch now? https://review.opendev.org/#/c/653454/
16:09:35 mdbooth mriedem: Looking, btw
16:27:55 imacdonn mdbooth: still around ?
16:28:11 dansmith mriedem: totally fine with waiting to approve that stack until I write the docs, btw
16:28:42 mdbooth imacdonn: I am. In #nova
16:29:31 imacdonn mdbooth: #nova ?
16:29:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove unused param from _fill_provider_mapping https://review.opendev.org/655107
16:29:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: prepare func test env for moving servers with bandwidth https://review.opendev.org/655109
16:29:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: Move _fill_provider_mapping to the scheduler_utils https://review.opendev.org/655108
16:29:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass network API to the conducor's MigrationTask https://review.opendev.org/655111
16:29:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: allow getting resource request of every bound ports of an instance https://review.opendev.org/655110
16:29:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: re-calculate provider mapping during migration https://review.opendev.org/655112
16:29:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec to server move RPC calls https://review.opendev.org/655721
16:29:42 openstackgerrit Balazs Gibizer proposed openstack/nova master: update allocation in binding profile during migrate https://review.opendev.org/656422
16:29:42 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extend NeutronFixture to handle migrations https://review.opendev.org/655114
16:29:43 openstackgerrit Balazs Gibizer proposed openstack/nova master: func test for migrate server with ports having resource request https://review.opendev.org/655113
16:29:54 mdbooth imacdonn: Nova room, ptg
16:30:53 imacdonn mdbooth: oh, ok .. I'm not invited :P If you guys get a chance / if you haven't already, could you discuss the eventlet patching vs. WSGI issue?
16:31:47 mdbooth imacdonn: Yep. melwitt has patches in progress to remove the eventlet dep.
16:33:11 imacdonn mdbooth: OK ... so long as that can be backported to Stein, that's good news ... (I'm stalled w.r.t. upgrading until I get a solution). Will look for details next week, I suppose. Thanks!
16:35:32 mdbooth imacdonn: I added it to the tech debt section here: https://etherpad.openstack.org/p/nova-ptg-train
16:39:35 imacdonn mdbooth: hmm, not seeing it?
16:40:15 mdbooth Line 187
16:40:39 imacdonn line 187 is "Fixing OSC's live migrate interface (mriedem)" for me
16:41:32 mdbooth Oh, weird.
16:41:36 mdbooth I’ll put it back
16:42:07 imacdonn I see it appearing now ;)
16:48:48 openstackgerrit garyk proposed openstack/nova master: Improve metadata performance https://review.opendev.org/615435
16:51:03 wwriverrat Likely asked and answered... (sorry)... Is there a new channel out there for placement?
16:51:33 wwriverrat Or do the questions still fall here?
16:51:48 gibi_cape wwriverrat: #openstack-placement
16:52:43 mriedem dansmith: mdbooth: question on that libvirt image type patch - https://review.opendev.org/#/c/653454/ - i need a decoder ring for the ternary config option of doom
17:08:08 mriedem tssurya: i see this is incomplete for ironic https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/api/openstack/compute/services.py#L270
17:09:45 mriedem tssurya: because it only deletes the provider for the first compute node
17:09:50 mriedem but there could be 1000
17:10:27 mriedem https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/objects/service.py#L313
17:12:28 mriedem jaypipes: i'm assuming the answer is yes here, but will this model_query soft_delete delete all records if there is more than one compute node per service (ironic): https://github.com/openstack/nova/blob/b9bcbab86b8314fbaaeb2d2af6282d4a612aeb8d/nova/db/sqlalchemy/api.py#L393
17:46:52 mriedem efried: there is an open runway slot but the next in the queue is gibi's bug for the port requested resources heal allocation stuff, so i'm not sure if you want to allow that since it's not technically a blueprint, or what
17:47:29 mriedem efried: also, i nixed the "Fixing OSC's live migrate interface" entry from the saturday agenda since the recap on that is in the ML - just looking for owners for the identified work items
17:47:33 mriedem artom already signed up for one
17:48:26 efried mriedem: I've never been one to be strict about must-be-blueprint-to-be-in-runway, so gibi_cape's thing would be fine by me.
17:55:17 openstackgerrit Dan Smith proposed openstack/nova master: Make libvirt expose supported image types https://review.opendev.org/653454
17:55:18 openstackgerrit Dan Smith proposed openstack/nova master: Add ironic driver image type capabilities https://review.opendev.org/655729
17:55:18 openstackgerrit Dan Smith proposed openstack/nova master: Add vmware driver image type capabilities https://review.opendev.org/655730
17:55:19 openstackgerrit Dan Smith proposed openstack/nova master: Add zvm driver image type capabilities https://review.opendev.org/655732
17:55:19 openstackgerrit Dan Smith proposed openstack/nova master: Add xenapi driver image type capabilities https://review.opendev.org/655731
17:55:20 openstackgerrit Dan Smith proposed openstack/nova master: Add image type request filter https://review.opendev.org/656413
17:55:20 openstackgerrit Dan Smith proposed openstack/nova master: Enable image type query support in nova-next https://review.opendev.org/656903
18:00:33 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Make libvirt able to trigger a backend image copy when needed https://review.opendev.org/656998
18:31:23 openstackgerrit Merged openstack/os-traits master: Dropping the py35 testing https://review.opendev.org/654651
19:28:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/657016
19:31:47 mriedem tssurya: ^
19:33:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/657016
19:35:34 tssurya mriedem: yea forgot we didn't handle the 1:N case for ironic
19:42:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in delete_resource_provider https://review.opendev.org/657021
19:48:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Change the default of notification_format to unversioned https://review.opendev.org/603079
19:51:33 cdent mriedem: do you know which provider is limestone-regionone?
19:53:25 mriedem cdent: limestone? https://www.limestonenetworks.com/wordpress/tag/openstack-cloud/
19:53:37 mriedem https://www.limestonenetworks.com/blog/2018/12/10/openstack-public-cloud-at-limestone-networks/
19:53:54 cdent ah, how incredibly obvious. I blame that giant sugar soaked cookie at lunch
19:54:13 cdent they've timed out some jobs on the placement deletion
19:55:47 cdent i seem to recall that region coming up fairly often
19:56:23 mriedem gibi_cape: what's the change of heart in https://review.opendev.org/#/c/603079/ ?
19:59:34 gibi_cape mriedem: I guess I accepted that we cannot make our notification consumers to change to versioned.
20:00:07 gibi_cape mriedem: and emiting both notification by default is not logical as most of the consumers consume either legacy or versioned but noth both
20:06:07 openstackgerrit Dan Smith proposed openstack/nova master: Add docs for image type support request filter https://review.opendev.org/657025
20:08:29 mriedem gibi_cape: ah, did that come up during your talk?
20:08:39 mriedem i haven't seen the recording yet (probably not posted)
20:13:09 tssurya gibi_cape: for the record I don't think you should do this upstream as a default, if deployments are annoyed with it they can just set it right ?
20:13:51 mriedem tssurya: most projects consuming nova notifications aren't consuming versioned notifications, right?
20:14:05 mriedem so if the default is versioned, most deployments have to change,
20:14:13 tssurya mriedem: yea they are not
20:14:21 mriedem if the default is both (current), most deployments have extra traffic/performance hit of generating an unused versioned notification
20:14:29 mriedem hence the bug
20:14:44 tssurya but if we have put in a lot of work to get it done we have it available, if they want they can set it in the config
20:14:52 tssurya yea i know the bug :) was there was for his presentation
20:15:20 mriedem i'd argue that we (nova) should work on getting versioned notification support in other projects in openstack
20:15:24 mriedem if we want people using it
20:15:35 mriedem once that is done, then change the default
20:16:10 bauzas uh, my cellsv2 skills are rusty
20:16:12 tssurya sure :) that effort has been going on I guess. I agree, so I think we are saying the same thing right ? keep the default as both for now ?
20:16:33 bauzas I wonder whether the migration.execute() code is run under the super-cond or the cond
20:16:37 bauzas I guess the latter
20:16:45 bauzas since we used the cell MQ
20:16:54 mriedem bauzas: what migration.execute() code?
20:17:03 mriedem tssurya: no i'm not saying that
20:17:05 bauzas mriedem: the conductor task
20:17:16 mriedem "if the default is both (current), most deployments have extra traffic/performance hit of generating an unused versioned notification"
20:17:26 tssurya okay so you want the default to be unversioned and then move when ll projects have moved to both ?

Earlier   Later