Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-04
20:28:42 mriedem s/weird/where/?
20:28:51 mriedem https://review.opendev.org/#/c/688387/8/nova/compute/manager.py only does the port mapping stuff if evacuate
20:29:05 mriedem same in https://review.opendev.org/#/c/688387/8/nova/conductor/manager.py
20:29:20 mriedem or are you asking, why do we only do that for evacuate?
20:29:26 efried No
20:29:56 efried in one file the bool is called 'evacuate', so I can buy that it's only happening for evacuate. In the other it's called 'recreate' and the comments imply we're hitting the path for both evacuate and rebuild.
20:30:11 mriedem in compute manager that evacuate variable used to be called recreate as well,
20:30:13 efried so either it's the same and the var name and comments are confusing, or...
20:30:16 mriedem i changed that awhile back,
20:30:25 mriedem it's still recreate in conductor b/c no one has renamed the variable for clarity yet
20:30:53 efried okay. So recreate is evacuate.
20:31:06 efried and... `not recreate` is rebuild, which is a kind of recreate, but whatever.
20:31:16 mriedem right
20:31:19 mriedem hence the confusion
20:31:26 efried thanks for clarifying.
20:31:36 efried someone should rename that variable.
20:31:49 mriedem https://review.opendev.org/#/c/508190/
20:31:56 mriedem that's me doing it in compute
20:33:39 efried ah, so you didn't actually try to rename it all the way through the stack. That would indeed be hairy.
20:33:58 mriedem right, i wasn't going to touch rpc stuff for that
20:34:10 mriedem and the driver interface is likely nbd but it's at least an email
20:34:33 mriedem there is no in-tree driver that implements that anymore anyway, it was around for the old baremetal driver that eventually became ironic
20:39:33 openstackgerrit Eric Fried proposed openstack/nova master: cond: rename 'recreate' var to 'evacuate' https://review.opendev.org/692900
20:39:35 efried mriedem: ^
20:51:16 mriedem comments inline
21:03:52 openstackgerrit Eric Fried proposed openstack/nova master: cond: rename 'recreate' var to 'evacuate' https://review.opendev.org/692900
21:04:47 efried done and done
21:06:21 efried and +A on gibi_ptg's patches.
21:07:29 mriedem sweet
21:09:17 efried Gonna go do some more damage on osc, then come back to vtpm
21:24:49 mriedem heave ho
21:25:23 openstackgerrit Matt Riedemann proposed openstack/nova master: Follow up to I3e28c0163dc14dacf847c5a69730ba2e29650370 https://review.opendev.org/692856
21:25:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.opendev.org/635668
21:25:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Pass exception through TaskBase.rollback https://review.opendev.org/692689
21:25:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Plumb allow_cross_cell_resize into compute API resize() https://review.opendev.org/635684
21:25:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Refresh instance in MigrationTask.execute Exception handler https://review.opendev.org/669012
21:25:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.opendev.org/636253
21:25:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.opendev.org/636224
21:25:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592
21:25:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle target host cross-cell cold migration in conductor https://review.opendev.org/642591
21:25:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
21:25:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
21:25:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058
21:25:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing https://review.opendev.org/669013
21:25:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075
21:25:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070
21:25:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630
21:25:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
21:25:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453
21:30:54 openstackgerrit melanie witt proposed openstack/nova stable/stein: Remove redundant call to get/create default security group https://review.opendev.org/692907
21:30:54 openstack bug 1824435 in OpenStack Compute (nova) stein "fill_virtual_interface_list migration fails on second attempt" [Medium,Triaged] https://launchpad.net/bugs/1824435
21:30:54 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add regression test for bug 1824435 https://review.opendev.org/692906
21:48:43 mriedem melwitt: i saw this today https://i.imgur.com/beq9YYf.jpg
21:50:06 melwitt lol yesss
22:06:28 mriedem dansmith: another spec which we've basically said no to in the past but the pushback is "we have some customers that want nova to do this" https://review.opendev.org/#/c/672400/
22:06:41 mriedem because pre-creating a port is hard
22:09:05 mriedem if enough diverse users/vendors came forward asking for this like people did for passing through volume type on boot from volume then maybe it's a different discussion
22:09:41 dansmith well, I haven't read that other than the commit message, but..
22:09:45 dansmith I'm pretty conflicted on these
22:10:19 dansmith since we're split, it really seems like an impossible-to-win scenario chasing every attribute of the other service, especially when they version their api so differently from us
22:10:42 mriedem "we're split" meaning we (nova) from neutron?
22:10:45 dansmith but I also really sympathize with simple things being hard because of how we decided in 2013 to segregate the project
22:10:46 dansmith yes
22:11:49 mriedem i'm pretty sure vnic_type is going to be in any neutron deployment because we rely on that extension pretty heavily https://docs.openstack.org/api-ref/network/v2/index.html#port-binding-extended-attributes
22:12:38 mriedem alright, i won't block it
22:12:51 mriedem i'm fairly certain starlingx had that in their nova fork already
22:13:00 mriedem maybe sean-k-mooney would know if red hat wants that as well
22:13:06 dansmith I'm not saying we shouldn't block it
22:13:22 dansmith I'm saying we're screwed either way
23:00:32 openstackgerrit Merged openstack/nova master: Allow evacuating server with port resource request https://review.opendev.org/688387
23:00:38 openstackgerrit Merged openstack/nova master: Enable evacuation with qos ports https://review.opendev.org/688688
23:11:50 efried dansmith: help me here, I swear somewhere we had a functional test to validate the redirects in our .htaccess file??
23:14:24 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add integration testing for heal_allocations https://review.opendev.org/692923
23:14:24 openstackgerrit melanie witt proposed openstack/nova stable/stein: Remove redundant call to get/create default security group https://review.opendev.org/692907
23:14:24 openstack bug 1824435 in OpenStack Compute (nova) stein "fill_virtual_interface_list migration fails on second attempt" [Medium,In progress] https://launchpad.net/bugs/1824435 - Assigned to melanie witt (melwitt)
23:14:24 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add regression test for bug 1824435 https://review.opendev.org/692906
23:14:55 efried found it
23:43:01 openstackgerrit Merged openstack/nova stable/queens: Avoid error 500 on shelve task_state race https://review.opendev.org/692632
23:43:06 openstackgerrit Merged openstack/nova stable/stein: Imported Translations from Zanata https://review.opendev.org/692701
#openstack-nova - 2019-11-05
00:54:52 openstackgerrit Merged openstack/nova stable/train: Nova compute: add in log exception to help debug failures https://review.opendev.org/692662
00:56:18 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Nova compute: add in log exception to help debug failures https://review.opendev.org/692936
01:20:15 gibi_ptg efried: thanks for pushing the evacute patches for qos. I think https://blueprints.launchpad.net/nova/+spec/support-move-ops-with-qos-ports-ussuri is incorrectly put into completed state. I still need to implement support for live migrate and unshelve with qos as the bp describes
02:00:46 mriedem gibi_ptg: fixed
02:02:02 gibi_ptg mriedem: thanks
02:15:18 mriedem it's alive http://www.danplanet.com/blog/2019/11/04/start-and-monitor-image-pre-cache-operations-in-nova/
02:18:04 gibi_ptg mriedem: I will mention it in the nova project update as something we already done from the Ussuri plans
02:19:06 mriedem the image cache thing? yeah good idea.
02:19:26 gibi_ptg yepp
02:19:54 gibi_ptg the image download already come up in the war stories session yesterday
02:29:13 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Create BDM in API during attach volume https://review.opendev.org/692940
03:16:06 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add integration testing for heal_allocations https://review.opendev.org/692923
03:16:08 openstackgerrit melanie witt proposed openstack/nova stable/stein: Remove redundant call to get/create default security group https://review.opendev.org/692907
03:16:08 openstack bug 1824435 in OpenStack Compute (nova) stein "fill_virtual_interface_list migration fails on second attempt" [Medium,In progress] https://launchpad.net/bugs/1824435 - Assigned to melanie witt (melwitt)
03:16:08 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add regression test for bug 1824435 https://review.opendev.org/692906
05:20:56 gibi_ptg nova project update went OK in Shanghai. We got questions about SRIOV live migration with bonded virtio interface for failover. adrianc wil; lead a discussion around it on the PTG
08:26:20 openstackgerrit Tushar Patil proposed openstack/nova-specs master: Allow compute nodes to use DISK_GB from shared storage RP https://review.opendev.org/650188
11:06:19 openstackgerrit Adit Sarfaty proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247
11:13:20 openstackgerrit Adit Sarfaty proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247
12:50:39 openstackgerrit ya.wang proposed openstack/nova-specs master: Re-propose expose-auto-converge-post-copy spec for Ussuri https://review.opendev.org/687199
13:07:09 efried gibi_ptg: thanks for the update, and thanks to y'all for covering the session.

Earlier   Later