Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-11
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?
20:12:36 artom Is default just for "actual" values?
20:15:21 dansmith artom: it's the default for if you ask the object to set its defaults
20:15:38 dansmith remember, an object can represent something that is a fraction of the total,
20:15:40 artom dansmith, ah, so not a fallback kind of thing
20:15:48 dansmith like instance if you don't ask for all the stuff to be filled out
20:15:54 artom Fair enough.
20:16:10 dansmith if we took the defaults everywhere, we'd overwrite things in the DB with the defaults because someone didn't query for them and then called .save() to update one little field
20:16:19 artom I still have the omnipresent 'if field in obj and obj.field is not None' construct ;)
20:16:43 artom It makes sense when you explain it that way :)
20:17:02 artom s/still have/still hate/
20:17:54 artom Or obj_attr_is_set
20:18:29 artom But yeah, I see why we need both "unset" and "None", and having automatic None defaults would clobber the former with the latter

Earlier   Later