Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-13
16:06:56 mriedem with dansmith and my preferred option
16:07:04 mriedem which also allows us to start draining the bottom half of the series
16:07:18 mriedem (11:01:38 AM) mriedem: i think the 3 options i pointed out yesterday were:
16:07:19 mriedem (11:02:13 AM) mriedem: 1. make RequestSpec.network_metadata nullable so that create_resource_requests can return None and we just blindly set it (which is what he'd be doing as coded today),
16:07:19 mriedem (11:02:49 AM) mriedem: 2. keep reqspec.network_metadata non-nullable and then: (a) he has to return an 'empty' NetworkMetadata if no networking or (b) leave RequestSpec.network_metadata unset
16:07:23 mriedem we both want 2-b
16:08:06 stephenfin I'm fine with 1 or 2-b. 2-a does feel weird
16:08:21 mriedem ok 2-b it is and we can start approving patches
16:09:10 stephenfin cool. I'll throw an eye over the last few patches again and address all those comments of yours I missed yesterday
16:10:40 mriedem dansmith: there +W up to https://review.openstack.org/#/c/564442/
16:10:45 mriedem bbiab
16:11:19 dansmith woot
16:34:10 stephenfin dansmith: Thoughts on https://review.openstack.org/#/c/564444/14/nova/objects/fields.py@76 ?
16:34:50 stephenfin I'd read that comment as "these are the things we don't use directly in our models" but that's not what it means. Just remove that comment?
16:36:31 dansmith stephenfin: those are aliases, because much of that file was initially all in nova,
16:36:48 dansmith and so we did that so we could remove the top half, move it to ovo without changing all the references to things like "Set" below
16:36:55 dansmith well, not set, that clearly got added later
16:37:11 dansmith as the comment says, that whole list should go away, and should have years ago
16:37:14 dansmith you shouldn't add to it,
16:37:20 dansmith just use fields.String below where you use String now
16:37:32 stephenfin Oh, so I should have used 'fields.String' directly?
16:37:36 dansmith yes
16:37:46 stephenfin Cool. I'll fix that in this follow-up
16:43:08 dansmith commented for the credit^W^Wposterity
16:47:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
16:47:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
16:47:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562
17:41:22 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
17:41:23 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441
17:41:24 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452
17:41:25 openstackgerrit Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651
18:33:52 openstackgerrit Matt Riedemann proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.64 https://review.openstack.org/573136
18:46:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
18:46:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562
18:46:18 openstackgerrit Matt Riedemann proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.64 https://review.openstack.org/573136
18:48:51 mriedem dansmith: i assume you're doneso
18:49:03 dansmith soon
18:49:05 dansmith wassup?
18:49:08 dansmith o
18:49:32 mriedem likely not a series you want to hit while you're in vacation brain
18:50:04 mriedem this is the compute service version check i added in there fwiw https://review.openstack.org/#/c/573136/8/nova/compute/api.py
18:51:52 mriedem didn't realize that was still in a runway slot, so i guess some other intrepid core will hit those
18:52:00 dansmith and we pass that flag from the api if the microversion was new enough?
18:52:03 mriedem if only one were around...
18:52:05 mriedem yeah
18:52:13 dansmith ack yeah
18:52:13 mriedem https://review.openstack.org/#/c/573136/8/nova/api/openstack/compute/server_migrations.py
18:53:13 dansmith yeah looks okay to me at skim-level
19:06:09 mriedem looks like we have a new functional test race bug https://bugs.launchpad.net/nova/+bug/1781648
19:06:10 openstack Launchpad bug 1781648 in OpenStack Compute (nova) "heal_allocations test randomly failing with "ValueError: Field `compute_node_uuid' cannot be None"" [Medium,Confirmed]
19:07:05 openstackgerrit Eric Fried proposed openstack/nova master: PoC: reshaper guts with inventory two-step https://review.openstack.org/582663
19:27:00 openstackgerrit Merged openstack/nova master: Heal allocations with incomplete consumer information https://review.openstack.org/574488
19:36:12 mriedem https://blueprints.launchpad.net/nova/+spec/ironic-rescue-mode is now in a runway slot
19:36:19 mriedem needs final +2s
19:36:22 mriedem pretty simple
19:43:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/582674
20:00:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Stop using HostAPI.service_delete https://review.openstack.org/582680
21:19:02 fried_rice mriedem, needssleep: Sent ironic-rescue-mode patches to the gate
21:19:51 needssleep fried_rice: That is like... the best news possible for the end of a Friday! Thanks!
21:19:57 fried_rice :)
21:23:57 needssleep uhg, postfails in ironic's gate
21:24:27 needssleep and nova, and swift
21:24:29 needssleep woot
21:25:28 needssleep static and logs.o.o is apparently non-responsive
21:25:44 needssleep Some chat in the infra room
21:26:03 needssleep Anyway, I'm going to go not sit in front of a computer for a while, thanks everyone!
22:28:41 openstackgerrit Eric Fried proposed openstack/nova master: perform reshaper operations in single transaction https://review.openstack.org/582383
22:28:42 openstackgerrit Eric Fried proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
22:28:43 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236
#openstack-nova - 2018-07-14
01:18:23 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: DNM: Test a common "irrelevant-files" https://review.openstack.org/582700
02:30:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix ServerMigrationSampleJsonTests to use sample files from version dir https://review.openstack.org/582702
02:31:21 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix ServerMigrationSampleJsonTests to use sample files from version dir https://review.openstack.org/582702
08:04:12 openstackgerrit Chen proposed openstack/nova master: Clean up group members in db when delete https://review.openstack.org/581403
08:49:40 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix ServerMigrationSampleJsonTests to use sample files from version dir https://review.openstack.org/582702
12:37:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Add another up-call to the cells v2 caveats list https://review.openstack.org/581910
12:38:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Add note about reschedules and num_attempts in filter_properties https://review.openstack.org/582412
12:49:07 mriedem yikun: we might have a problem https://bugs.launchpad.net/nova/+bug/1781710
12:49:08 openstack Launchpad bug 1781710 in OpenStack Compute (nova) "ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity failing with "Servers are on the same host"" [High,Confirmed]
12:49:15 mriedem regression in anti-affinity testing starting on july 9
17:20:17 openstackgerrit Merged openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929
#openstack-nova - 2018-07-15
01:19:14 openstackgerrit Merged openstack/nova master: ironic: provide facilities to gracefully navigate versions https://review.openstack.org/582416
07:49:37 openstackgerrit Merged openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
07:49:45 openstackgerrit Merged openstack/nova master: ironic: Log an error when API version is not available https://review.openstack.org/582447
12:30:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-manage placement sync_aggregates https://review.openstack.org/575912
13:14:55 openstackgerrit Merged openstack/nova master: Transform instance.live_migration_post notification https://review.openstack.org/480119
13:25:46 openstackgerrit Merged openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214
14:17:39 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675
14:37:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
15:14:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561
15:17:21 openstackgerrit Merged openstack/nova master: Skip more rebuild tests for cells v1 job https://review.openstack.org/581717
20:57:11 openstackgerrit Gaudenz Steinlin proposed openstack/nova master: Ignore some updates from virt driver https://review.openstack.org/523006
23:46:44 rallies TESTING TESTING
23:46:49 rallies MAINTENANCE IN PROGRESS
23:47:04 larsks rallies: IT DIDN'T WORK TRY AGAIN
23:47:23 rallies larsks this channel is closed
23:47:32 rallies sorry dir
23:47:35 larsks ...or is it?
23:47:36 rallies sorry sir
23:47:47 rallies it is closed for maintenance
23:47:56 rallies we will be done in like an hour

Earlier   Later