| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-23 | |||
| 21:52:20 | sean-k-mooney | artom: hehe you bet me too it | |
| 21:53:45 | artom | sean-k-mooney, ah, I think that tempest job dynamically gets a list of projects that are tempest plugins from gerrit: https://opendev.org/openstack/tempest/src/branch/master/tools/generate-tempest-plugins-list.py | |
| 21:54:20 | sean-k-mooney | ya so i guess its trying to make sure that changes to tempest dont break plugins | |
| 21:55:06 | artom | Risky as hell though | |
| 21:55:24 | artom | Though they can blacklist projects too | |
| 21:55:29 | artom | So maybe not to much | |
| 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: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593 | |
| 09:53:02 | openstackgerrit | Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856 | |
| 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: Assert quota related API behavior when noop https://review.opendev.org/712140 | |
| 10:32:37 | openstackgerrit | John Garbutt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/712137 | |
| 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 | |