Earlier  
Posted Nick Remark
#openstack-nova - 2020-03-23
22:02:58 artom OK, so should work then
22:03:04 artom But yeah, tomorrow
22:03:05 artom o/
22:03:10 sean-k-mooney o/
23:42:52 openstackgerrit melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
23:42:54 openstack bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
#openstack-nova - 2020-03-24
00:28:48 openstackgerrit Merged openstack/nova master: Add service version check for live migrate with qos https://review.opendev.org/713679
00:29:40 openstackgerrit Merged openstack/nova master: Add service version check for evacuate with qos https://review.opendev.org/713680
00:29:46 openstackgerrit Merged openstack/nova stable/queens: Stop setting glance_api_version in cinder.conf in nova-live-migration https://review.opendev.org/713843
00:29:52 openstackgerrit Merged openstack/nova stable/queens: Replace ansible --sudo with --become in live_migration/hooks scripts https://review.opendev.org/713844
00:29:59 openstackgerrit Merged openstack/nova stable/queens: nova-live-migration: Wait for n-cpu services to come up after configuring Ceph https://review.opendev.org/713845
00:49:07 openstackgerrit melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
00:49:10 openstack bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
03:02:42 openstackgerrit Brin Zhang proposed openstack/nova master: [Trivial] fixing some nits in instance actions policy tests https://review.opendev.org/714560
03:16:03 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: WIP Microversion 2.83 - action event fault details https://review.opendev.org/714561
03:21:38 openstackgerrit melanie witt proposed openstack/nova stable/train: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/713116
03:21:39 openstack bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
03:40:45 openstackgerrit Kevin Zhao proposed openstack/nova master: [DNM] Run tempest jobs on Arm64 needed fix https://review.opendev.org/714564
09:08:10 brinzhang_ sean-k-mooney: https://review.opendev.org/#/c/674726/3/nova/compute/api.py@3389, do we need to block the rebuild?
09:10:40 brinzhang_ In https://review.opendev.org/#/c/631245/62/nova/virt/libvirt/driver.py@3484, when we rebuild a server with accerators, but I think the accel_info always None, so I think we also should block the rebuild compute API
09:36:27 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.83 - action event fault details https://review.opendev.org/714561
09:53:02 openstackgerrit Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
09:53:02 openstackgerrit Luyao Zhong proposed openstack/nova master: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593
10:30:04 openstackgerrit jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
10:30:42 stephenfin gibi: could you revisit https://review.opendev.org/#/c/666915/ today, please?
10:30:51 gibi stephenfin: sure
10:30:56 stephenfin ta
10:31:35 nightmare_unreal hey can someone look at my patch . I am stuck.
10:31:48 nightmare_unreal a comment in right direction can be helpful. Thanks
10:32:37 openstackgerrit John Garbutt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/712137
10:32:37 openstackgerrit John Garbutt proposed openstack/nova master: Assert quota related API behavior when noop https://review.opendev.org/712140
10:32:38 openstackgerrit John Garbutt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/712141
10:40:39 gibi stephenfin: responded.
10:41:00 gibi nightmare_unreal: link the patch please
10:42:11 nightmare_unreal https://review.opendev.org/714459
10:42:48 nightmare_unreal bug : https://bugs.launchpad.net/nova/+bug/1868531
10:42:49 openstack Launchpad bug 1868531 in OpenStack Compute (nova) "nova manage placement doesn't support registration per Cell." [Undecided,New] - Assigned to jayaditya gupta (jayssj11)
10:44:04 gibi looking
10:46:43 nightmare_unreal Thanks
10:52:21 openstackgerrit Merged openstack/nova master: [Trivial] fixing some nits in instance actions policy tests https://review.opendev.org/714560
10:52:23 brinzhang stephenfin: can you review this patch https://review.opendev.org/#/c/694430/?
10:54:41 gibi nightmare_unreal: I left some comment I hope they help
10:55:19 nightmare_unreal thanks
11:22:23 lyarwood kashyap: https://review.opendev.org/#/c/696834/ - would you mind hitting this again?
11:23:16 lyarwood elod: https://review.opendev.org/#/c/713036/ - should be ready for stable/pike now, hopefully unblocking the branch.
11:23:38 kashyap lyarwood: Yep, a sec
11:24:51 elod lyarwood: thanks, looking!
11:26:15 elod lyarwood: just a minor thing: it's not backkported from the latest commit
11:26:25 elod from queens
11:26:27 lyarwood hmm it should be
11:27:30 lyarwood elod: ah it's PS2, I pushed the stable/pike change to the stable/queens change in PS3 by accident
11:27:38 kashyap lyarwood: "+1", FWIW. I don't see anything else that is blocking there.
11:27:43 lyarwood elod: https://review.opendev.org/#/c/713845/2..4
11:28:31 lyarwood elod: FWIW I've never really enforced the commit SHA's being correct
11:29:00 lyarwood elod: we end up in an endless game up updates if the earlier changes are rebased etc
11:29:11 lyarwood elod: if the content is the same I typically ACK
11:33:16 elod lyarwood: well, yes, other teams don't even bother with conflict lines either. and it's not a big thing, i know. maybe this forces people to backport things when the patch on newer branch has merged o:)
11:34:22 elod fot me it seemed mriedem kept the rules quite strictly so I try to do the same :D
11:34:29 lyarwood yup that's cool
11:34:51 lyarwood I wanted to revisit the SHA issue at some point but until then feel free to enforce if you want to
11:37:47 elod lyarwood: ok, no problem, I'll edit the SHA as that is the only thing that's needed :)
11:44:18 openstackgerrit Elod Illes proposed openstack/nova stable/pike: nova-live-migration: Wait for n-cpu services to come up after configuring Ceph https://review.opendev.org/713036
11:51:38 lyarwood elod: many thanks
11:52:48 elod lyarwood: thanks, too :)
12:01:01 lyarwood gibi: https://review.opendev.org/#/c/696834/ - would you mind taking a look at this today please?
12:11:06 kevinz stephenfin: lyarwood: would you mind to re-visit this one: https://review.opendev.org/#/c/709494/? Thanks
12:17:33 alex_xu dansmith: query another thing, https://review.opendev.org/#/c/713354/1/nova/objects/fields.py@277, as I understand, we can't change a field. But we won't support mixed policy in the middle of upgrade, we only enable it after upgrade finished. so can we change a field like that? just do a version object bump.
12:19:12 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models https://review.opendev.org/666915
12:19:13 openstackgerrit Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775
12:20:15 stephenfin gibi: Tests added ^
12:28:30 luyao alex_xu: https://review.opendev.org/#/c/687856 comments addressed, thanks for review
12:29:29 luyao stephenfin: could you review vpmem live migration https://review.opendev.org/#/c/687856 if you get time?
12:30:46 stephenfin kevinz: replied. one more thing needed, I think
12:31:12 stephenfin luyao: I can, but I was waiting until alex_xu had most of his things addressed before :)
12:35:04 luyao stephenfin: OK, thanks
12:36:06 openstackgerrit Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494
12:53:07 alex_xu stephenfin: I think most of my things addressed...
13:33:52 gibi lyarwood, stephenfin: ack on both, I'm on a call will check it after
13:35:14 openstackgerrit John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139
14:04:17 stephenfin lyarwood: do my comments on https://review.opendev.org/#/c/712607/ make sense?
14:04:29 openstackgerrit John Garbutt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/712142
14:05:02 dansmith artom: sorry, just got to your base test patch.. fix that thing right quick and I can +2
14:05:34 artom (Dammit, missed the panix possibility)
14:07:47 dansmith alex_xu: we can but you have to do it in a specific way.. I commented
14:08:04 dansmith alex_xu: the hint is the test_objects hash change with no version bump
14:08:16 dansmith alex_xu: good job catching that
14:10:12 openstackgerrit John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139
14:10:13 openstackgerrit John Garbutt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/712142
14:11:14 openstackgerrit Artom Lifshitz proposed openstack/nova master: func tests: move _run_periodics() into base class https://review.opendev.org/714228
14:11:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: tests: work around malformed serial XML https://review.opendev.org/705546
14:11:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595
14:11:16 openstackgerrit Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404
14:11:17 openstack bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms)
14:11:23 artom dansmith, ^^
14:14:14 artom \o/
14:20:58 nightmare_unreal gibi: what if user pass both instance-id and cell-id
14:20:58 openstackgerrit Luyao Zhong proposed openstack/nova master: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593
14:20:59 openstackgerrit Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
14:20:59 openstackgerrit Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/714653
14:30:01 openstackgerrit Huaqiang Wang proposed openstack/nova master: Take the instance dedicated CPU list from 'cpu_pinning' if possible https://review.opendev.org/713352

Earlier   Later