Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-04
18:50:09 mriedem i do the fake compute thing in a 8vcpu/8gb ram vm sometimes and can't really go over 30 computes
18:50:10 dansmith about 19" wide
18:50:30 mriedem cpu/ram
18:50:44 dansmith oh, 32G memory, 8 cores if that's what you mean
18:50:54 mriedem yeah
18:51:10 mriedem did you adjust API_WORKERS or let it do ncpu/2?
18:51:31 dansmith just defaults
18:52:25 mriedem ah. fake computes huh. so all that time trying to figure out which network to delete and you found out you can't create a fake driver vm with networking anyway right?
18:52:48 mriedem i always forget that when i do a devstack with the fake driver
18:53:36 mriedem i had written up some of that here at one point https://docs.openstack.org/devstack/latest/guides/nova.html#fake-virt-driver
18:53:37 dansmith one real compute and then a bunch of extra fakes on the same machine
18:53:41 mriedem ah ok
18:53:49 mriedem smells like someone is playing with aggregates
19:17:58 mriedem wonder if we can nuke this yet, nothing in or out of tree returns True for it that i can find https://opendev.org/openstack/nova/src/branch/master/nova/virt/driver.py#L1740
19:18:53 mriedem oh nvm i guess it defaults to True in the ComputeDriver parent class and zvm doesn't override it...
19:18:58 mriedem or xenapi
20:07:57 mriedem artom: https://review.opendev.org/#/c/594139/ - am i misunderstanding your comment?
20:08:28 artom mriedem, lemme reload context
20:09:49 artom o_O
20:10:00 artom I am apparently retarded.
20:10:08 mriedem did your brain fart on how if/elif works?
20:10:14 artom Clearly
20:10:18 artom I remember double-checking myself as well
20:10:23 artom Thinking "no way Matt did that"
20:10:46 mriedem s'ok
20:11:10 mriedem that rollback code is all gorpy
20:12:45 artom mriedem, fixed
20:12:53 artom mriedem, lemme look it over once again, then +1
20:13:04 mriedem sure, thanks
20:13:39 artom I was busy shitting all over https://review.opendev.org/#/c/512815/8
20:13:42 artom Politely
20:18:10 openstackgerrit Merged openstack/nova stable/rocky: Avoid error 500 on shelve task_state race https://review.opendev.org/692631
20:21:12 eandersson btw anyone got any experience with the keep alive issue and openstack services? I noticed a bug or two reported for nova and failures
20:21:58 eandersson > 'Connection aborted.', BadStatusLine("''",)
20:22:13 eandersson A few of the bug reports indicates that disabling keepalive would resolve some of these.
20:26:00 efried mriedem: One question on the qos patches (I'm on the bottom one rn): It looks like some of the code is being hit for both evacuate and rebuild. It wouldn't do any harm for the latter, since it's just populating the rg/rp mappings. But just wanted to confirm that's what I'm seeing?
20:26:05 mriedem artom: oh wow that's fun, smells like starlingx
20:26:29 artom mriedem, yeah eh?
20:26:32 mriedem starlingx did a lot of hacks with libvirt channels
20:26:37 efried mriedem: specifically these changes? https://review.opendev.org/#/c/688387/8/nova/conductor/manager.py
20:27:45 mriedem artom: good comments on that patch btw
20:27:46 mriedem all valid poitns
20:27:48 mriedem *points
20:27:53 artom I'm useful!
20:28:04 mriedem makes up for your lack of if/elif knowledge :)
20:28:08 artom Hahaha
20:28:38 mriedem efried: hmm, weird
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: Pass exception through TaskBase.rollback https://review.opendev.org/692689
21:25:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.opendev.org/635668
21:25:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Refresh instance in MigrationTask.execute Exception handler https://review.opendev.org/669012
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:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.opendev.org/636224
21:25:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.opendev.org/636253
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:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592
21:25:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
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: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:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058
21:25:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070
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:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
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: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: Add regression test for bug 1824435 https://review.opendev.org/692906
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: Remove redundant call to get/create default security group https://review.opendev.org/692907
21:48:43 mriedem melwitt: i saw this today https://i.imgur.com/beq9YYf.jpg

Earlier   Later