Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-11
14:57:34 mriedem thanks
15:13:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Ignore VolumeAttachmentNotFound exception in compute.manager https://review.openstack.org/632657
15:24:54 jackding mdbooth_: Could you please revisit https://review.openstack.org/#/c/616692/ since your last -1? It
15:32:30 gibi hey cores, can somebody push in https://review.openstack.org/#/c/633776/ it is a follow up and mriedem already +2 on it
15:42:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix race in test_volume_swap_server_with_error https://review.openstack.org/634334
15:43:46 sean-k-mooney gibi: ya that makes sense. it conflict with a tone of patches so it would be good to merge sooner rather then later
15:44:33 gibi sean-k-mooney: exactly
15:55:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Raise InstanceFaultRollback for UnableToMigrateToSelf from _prep_resize https://review.openstack.org/633227
16:04:57 sean-k-mooney stephenfin: before you hit the sriov migration seires want to quickly look at gibi's patch https://review.openstack.org/634334. its a faily small change
16:05:05 stephenfin ack
16:05:15 gibi stephenfin: thanks
16:08:57 stephenfin sean-k-mooney: Right link? That's mriedem's change you linked
16:09:10 stephenfin wait - I see what you were referring to
16:16:50 efried Is there an etherpad for the PTG yet?
16:17:26 efried mriedem, melwitt, cdent, other people who might know. ^
16:17:44 sean-k-mooney efried: not that im aware of
16:18:00 mriedem https://etherpad.openstack.org/p/nova-ptg-train
16:18:01 mriedem nope
16:18:15 efried ight, thx
16:18:34 sean-k-mooney efried: well i guess you might as well start one with ^
16:19:34 sean-k-mooney we normally use the location in the name but give this is the third denver ptg swapping to the release name is likely better
16:21:39 sean-k-mooney efried: by the way you will be waiting a while for intel to appove. its always late.
16:22:01 efried :)
16:35:56 openstackgerrit Jim Rollenhagen proposed openstack/nova master: ironic: partition compute services by conductor group https://review.openstack.org/635006
16:52:12 mriedem the location in etherpad name thing is a forum thing, not really the ptg, https://etherpad.openstack.org/p/nova-ptg-queens https://etherpad.openstack.org/p/nova-ptg-rocky https://etherpad.openstack.org/p/nova-ptg-stein
16:57:22 dansmith mriedem: thoughts on my comment here? https://review.openstack.org/#/c/635146/8/nova/api/openstack/compute/views/servers.py
17:03:43 mriedem dansmith: commented
17:06:15 mriedem i don't feel the need for the guard i guess
17:10:20 sean-k-mooney mriedem: ya you can get the original version from the timeline but it is annoying
17:11:09 mriedem ?
17:11:35 sean-k-mooney mriedem: the non translated queens etherpad
17:11:36 mriedem i personally don't need the location in the etherpad name since i don't care where it was, i care when it was
17:11:43 mriedem oh yeah - that was annoying
17:12:51 sean-k-mooney it confused the hell out of me in the nova room at the following ptg i spent 20 trying to figure out what was up with my brower just to realise i was looking at the wronge page
17:13:03 sean-k-mooney *20 mins
17:15:26 mriedem pebkac
17:17:22 openstackgerrit Merged openstack/nova stable/rocky: PCI: do not force remove allocated devices https://review.openstack.org/635071
17:53:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Plumb allow_cross_cell_resize into compute API resize() https://review.openstack.org/635684
17:53:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.openstack.org/635668
17:55:25 mriedem stephenfin: i have a couple of comments in your novaclient change, was on the fence about -1ing https://review.openstack.org/#/c/635131/
17:55:43 stephenfin ack
17:59:25 dansmith mriedem: okay, it just seems like we're linking two sentinels somewhat via tribal knowledge and that we're setting this up for breakage later, but it's not a huge deal to me
18:03:19 mriedem sean-k-mooney: i've got some questions in https://review.openstack.org/#/c/624842/ which sound like they mirror things leakypipes was asking about before
18:03:50 sean-k-mooney looking
18:04:07 mriedem leakypipes must be stuck in friday limbo
18:07:18 mriedem dansmith: yeah it's definitely tight coupling but i'm not terribly worried about it right now. i wonder if setting an attribute on the instance (not persisted) would help, but idk
18:07:25 dansmith ack
18:07:37 mriedem instance.from_down_cell = True or something
18:07:53 mriedem but then you have to check if that attribute is set on the object as well
18:08:01 mriedem or we default to False i guess
18:10:06 kukacz hi, I'm getting "MigrationPreCheckError: ... CPU doesn't have compatibility." while trying to live-migrate between hosts with same model of CPU. Turned out that by mistake I got newer package with /usr/share/libvirt/cpu_map.xml differing between those 2 hosts. downgraded libvirt-bin, restarted libvirt-bin and nova-compute services but still seeing same precheck error. where does the precheck take the cpu model from?
18:11:10 kukacz (had newer package on one of the two hosts)
18:13:58 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Microversion 2.68: Remove 'forced' live migrations, evacuations https://review.openstack.org/635131
18:14:07 sean-k-mooney mriedem_lunch: replied inline https://review.openstack.org/#/c/624842/8//COMMIT_MSG
18:14:15 stephenfin mriedem_lunch: When you're back ^
18:31:32 artom Who's our PowerVM guy? cfriesen?
18:32:19 artom I guess efried used to "it"
18:36:40 artom There's a unit test issue with to_legacy_dict() in the PowerVMLiveMigrateData... but as far as I can tell nothing actually calls to_legacy_dict anywhere
18:36:47 artom So I'm wondering if I can just get rid of it
18:40:24 artom In fact, the only place I can find *any* _legacy_dict methods is in the migrate_data.py objects and a couple of times in the libvirt migration code where we check if we got an ovo
18:40:30 cfriesen artom: I'm x86. :) used to be we called to_legacy_dict() from check_can_live_migrate_source()
18:40:34 artom I feel like that's left over from a long time ago and I can remove all of that
18:41:01 melwitt edmondsw ^
18:41:30 artom cfriesen, ah, sorry got confused about your arch allegiance ;)
18:41:47 artom You know what, I'll propose a thing that gets rid of it all, and we'll see what breaks :)
18:41:58 edmondsw artom in a mtg but will look in a few
18:43:39 artom edmondsw, cheers! I really need to relocate, but leave your reply here and I'll check eavesdrop
18:52:01 mriedem_lunch artom: i think that was dropped when we bumped the compute rpc api version to 5.0
18:52:38 mriedem https://github.com/openstack/nova/blob/stable/queens/nova/compute/rpcapi.py#L535
19:32:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
19:32:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Move finish_resize.(start|end) notifications to helper method https://review.openstack.org/635079
19:32:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
19:32:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.openstack.org/614035
19:32:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353
19:32:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
19:32:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.openstack.org/627892
19:32:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123
19:32:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.openstack.org/633853
19:32:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581
19:32:21 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add prep_snapshot_based_resize_at_dest compute method https://review.openstack.org/633293
19:32:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313
19:32:22 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.openstack.org/634832
19:32:22 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890
19:32:23 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add finish_snapshot_based_resize_at_dest compute method https://review.openstack.org/635080
19:32:23 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
19:32:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Add FinishResizeAtDestTask https://review.openstack.org/635646
19:32:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.openstack.org/635668
19:35:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Follow up (#2) for the bw resource provider series https://review.openstack.org/634767
20:01:57 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live_migration_claim() https://review.openstack.org/635669
20:01:57 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add migration param to check_can_live_migrate_destination https://review.openstack.org/634605
20:01:58 openstackgerrit Artom Lifshitz proposed openstack/nova master: New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
20:01:58 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Use live_migration_claim() to check dest resources https://review.openstack.org/634606
20:01:59 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: update NUMA-related XML on the source https://review.openstack.org/635229
20:01:59 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] LM: generate new NUMA XML on the dest and send it to the source https://review.openstack.org/634828
20:02:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: Remove _legacy_dict methods https://review.openstack.org/636210
20:03:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Avoid BadRequest error log on volume attachment https://review.openstack.org/581453
20:03:19 artom mriedem, ^^ The bottom-most patch there is remove the _to_dict stuff
20:03:45 artom (I assume the bot just spammed the channel with my patches, I wasn't actually connected, and eavesdrop isn't showing anything (yet?))
20:04:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Provide a useful error message when trying to update non-compute services https://review.openstack.org/620667
20:12:09 artom Wait, so what's the point of default=None in Nova objects if the field doesn't actually get set to None when you init the object without it?

Earlier   Later