Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-23
21:55:31 artom *so much
21:55:58 artom I'll email the ML
21:56:28 sean-k-mooney ya. well i think that reinforces the idea we proably should run that job on the mailing list but on the plus side they should be warned if they would breake whitebox atleast to some degree
22:01:19 artom sean-k-mooney, not sure how running that job would work, it depends on scripts in the tempest rpo
22:01:21 artom *repo
22:01:50 artom Anyways, we can continue this later, I need to make supper
22:02:19 sean-k-mooney ya im also done for the day. but runing that job would use the script form the tempest repo
22:02:41 sean-k-mooney it would tell use if we were making a change that woudl cause it to fail before we merge it
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

Earlier   Later