| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-10 | |||
| 16:06:33 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when reverting resize https://review.opendev.org/712118 | |
| 16:41:49 | stephenfin | lyarwood: dead easy stable patch for you here https://review.opendev.org/711895 | |
| 16:48:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 16:58:51 | lyarwood | stephenfin: are there many more of these? | |
| 16:59:00 | stephenfin | I think that's the only other one | |
| 16:59:12 | lyarwood | kk | |
| 16:59:20 | stephenfin | it's the only one I could spot anyway | |
| 17:33:51 | openstackgerrit | Monty Taylor proposed openstack/nova master: Run sdk functional tests on nova changes https://review.opendev.org/712092 | |
| 17:45:25 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Enforce unified limits using oslo.limit https://review.opendev.org/615180 | |
| 17:45:25 | openstackgerrit | John Garbutt proposed openstack/nova master: Add unified limits configuration https://review.opendev.org/712137 | |
| 17:45:26 | openstackgerrit | John Garbutt proposed openstack/nova master: Add tests for the UnifiedLimits quota driver https://review.opendev.org/712138 | |
| 17:45:26 | openstackgerrit | John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139 | |
| 17:45:27 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Add tests for API using unified limits driver https://review.opendev.org/712140 | |
| 17:45:27 | openstackgerrit | John Garbutt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/712141 | |
| 17:45:28 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Enforce api and db limits https://review.opendev.org/712142 | |
| 17:45:28 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Update quota_class APIs for db and api limits https://review.opendev.org/712143 | |
| 19:54:16 | donnyd | sean-k-mooney: can you kick off a multi-numa job if you have anything that can be - I was hoping to test to make sure it works | |
| 19:59:39 | sean-k-mooney | donnyd: just rechecked https://review.opendev.org/#/c/708977/ | |
| 20:07:21 | sean-k-mooney | donnyd: seams to be working fine http://zuul.openstack.org/stream/a34b85c571ed47db8c4f1a4b2f1a0d82?logfile=console.log | |
| 20:50:24 | donnyd | Woot woot | |
| 21:11:06 | openstackgerrit | Monty Taylor proposed openstack/nova master: Run sdk functional tests on nova changes https://review.opendev.org/712092 | |
| 21:54:42 | openstackgerrit | Merged openstack/nova stable/stein: Fix hypervisors paginted collection_name. https://review.opendev.org/711669 | |
| #openstack-nova - 2020-03-11 | |||
| 00:28:23 | openstackgerrit | Merged openstack/nova master: Don't overwrite greenthread-local context in host manager https://review.opendev.org/582332 | |
| 00:29:12 | melwitt | rm_work: if you're around, do you recall what version you tried your test with re: your comment on this bug? https://bugs.launchpad.net/nova/+bug/1863190 | |
| 00:29:12 | openstack | Launchpad bug 1863190 in OpenStack Compute (nova) "Server group anti-affinity no longer works" [Undecided,New] | |
| 01:22:03 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Add retry_on_deadlock to migration_update DB API https://review.opendev.org/712212 | |
| 02:43:25 | openstackgerrit | melanie witt proposed openstack/nova master: Add config option for neutron client retries https://review.opendev.org/712226 | |
| 05:07:26 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: Ignore DiskNotFound during update_available_resource https://review.opendev.org/711279 | |
| 05:36:21 | rm_work | melwitt: so that was on rocky -- but we figured out that we had build retries set to 0 so it would never retry, thus breaking that logic | |
| 05:37:34 | rm_work | And with soft antiaffinity, obviously it would just "work" and so given certain packing schemes, we're almost guaranteed to have them land on the same host, which seems kinda dumb | |
| 05:38:27 | rm_work | The conclusion we came to is that relying on the retry behavior to fix schedule races did not seem like a good approach | |
| 05:38:59 | rm_work | And that there should be better more up front ways of handling that kind of locking | |
| 05:39:33 | rm_work | Octavia does similar locking using DB logic | |
| 06:19:35 | rm_work | melwitt: johnsom could detail some of that more probably | |
| 10:49:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Remove dead code https://review.opendev.org/703146 | |
| 10:50:48 | stephenfin | bauzas: Any chance you could blast these through this morning? https://review.opendev.org/#/c/703973/ https://review.opendev.org/#/c/703974/ | |
| 10:51:11 | stephenfin | Final two patches for nova-net removal | |
| 10:52:36 | stephenfin | bauzas: and if you're doing that, there's another dead easy one here https://review.opendev.org/#/c/686997/ | |
| 10:52:51 | openstackgerrit | Lee Yarwood proposed openstack/nova master: images: Move qemu-img info calls into privsep https://review.opendev.org/706897 | |
| 10:52:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: images: Allow the output format of qemu-img info to be controlled https://review.opendev.org/706898 | |
| 10:52:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: virt: Pass request context to extend_volume https://review.opendev.org/706899 | |
| 10:52:53 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Correctly resize encrypted LUKSv1 volumes https://review.opendev.org/706900 | |
| 10:55:40 | lyarwood | sean-k-mooney: ^ removed run_as_root and made it clear that the new func is privileged | |
| 10:56:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM - Test TEMPEST_EXTEND_ATTACHED_ENCRYPTED_VOLUME https://review.opendev.org/707593 | |
| 10:58:22 | bauzas | stephenfin: will try | |
| 11:11:06 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add new default roles in os-instance-actions policies https://review.opendev.org/706470 | |
| 11:56:42 | sean-k-mooney | lyarwood: yep reviewing your series now | |
| 11:56:59 | sean-k-mooney | and ya its clearer what you are doing | |
| 12:02:14 | lyarwood | sean-k-mooney: ack thanks | |
| 12:11:26 | lyarwood | stephenfin: ^ when you have time later could you hit https://review.opendev.org/#/q/topic:bug/1861071 again, I'll rebase the followups once this lands now to avoid extra churn in the gate. | |
| 12:12:02 | stephenfin | sure. ping me again if I don't have it done by EOD :) | |
| 12:12:23 | lyarwood | sure thing | |
| 12:12:33 | lyarwood | and thanks obviously :) | |
| 12:19:27 | sean-k-mooney | lyarwood: i think i reviewed them all | |
| 12:19:46 | sean-k-mooney | it looks like the chain is broken into 3 seriese so they shoudl all be rebased back into one | |
| 12:20:30 | sean-k-mooney | or at least be rebased so they are update with the bottom 4 patches | |
| 12:30:23 | lyarwood | sean-k-mooney: yup, I'll rebase the rest later today, just switching to something else for a whil.e | |
| 12:46:39 | brinzhang | johnthetubaguy: https://review.opendev.org/#/c/706470/ I used your suggestion, that looks like ok for me, I was update that you can review again if you have time ^^ | |
| 12:48:20 | brinzhang | gmann: you can review this patch, and move the problem from your list I asked before | |
| 13:06:38 | luyao | gibi: are you around? | |
| 13:07:10 | openstackgerrit | Brin Zhang proposed openstack/nova master: Granular GET os-instance-actions API policies https://review.opendev.org/711791 | |
| 13:07:14 | gibi | luyao: hi! | |
| 13:08:07 | luyao | gibi: Would you like review the vpmem live migration support? I need someone help me confirm if the implementations is OK. :D | |
| 13:08:30 | luyao | https://review.opendev.org/#/q/topic:support-live-migration-with-virtual-persistent-memory+(status:open+OR+status:merged) | |
| 13:08:59 | gibi | luyao: sure. I can try. I have to finish the review of the cyborg integration first but then I can look at your series | |
| 13:11:42 | luyao | gibi: Thank you. looking forward for your comments. FYI. I don't add enough testcases now, if the implementation is acceptable, I'll do that. | |
| 13:12:12 | gibi | luyao: OK, I will keep that in mind | |
| 13:12:42 | brinzhang | luyao: I will tyr to check your patch tomorrow ^^ | |
| 13:13:12 | luyao | brinzhang: Thanks, welcome. :D | |
| 13:13:37 | brinzhang | luyao: I already add that to me review list, and will do later | |
| 13:14:40 | brinzhang | We would like this feature | |
| 13:16:13 | luyao | brinzhang: Great. | |
| 13:26:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: [Community goal] Update contributor documentation https://review.opendev.org/712420 | |
| 13:27:20 | gibi | stephenfin: ussuri community goal doc change ^^ | |
| 13:27:30 | gibi | stephenfin: I feel you will have comments about it | |
| 13:53:13 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when reverting resize https://review.opendev.org/712118 | |
| 14:16:55 | dansmith | gibi: thoughts on the vmware deprecation patch? I'm glad it got some attention raised, but we've been a long time without vmware ci and despite some response, I feel like we probably need a deadline or something to make sure something happens | |
| 14:17:50 | dansmith | I'm not even sure when the last time the vmware ci commented on a nova change, but I feel like it's been a *long* time | |
| 14:23:58 | sean-k-mooney | dansmith: we could merge the patch propose a revert and hold it until m3. if the ci is running by m3 we merge the revert if not then leave it | |
| 14:24:31 | dansmith | yep, we could also convert this to just a quality warning, merge that now, and hold the deprecation warning on the ci | |
| 14:24:35 | dansmith | which i think is what we did for xen | |
| 14:25:47 | gmann | brinzhang: i will check after coffee. | |
| 14:26:10 | brinzhang | gmann: cool, thanks | |
| 14:30:22 | mriedem | we merged the deprecation warning for xen in train, it wasn't contingent | |
| 14:33:36 | dansmith | okay when I was looking for examples, I found a "merge quality warning" for one of the drivers before a deprecation | |
| 14:33:46 | gibi | dansmith: we can merge a deprecation warning without causing extra problem to hemna and undeprecate if vmware ci magicly shows up | |
| 14:33:47 | mriedem | looks like dan's patch has at least gotten the necessary attention, | |
| 14:33:53 | mriedem | whether that holds or not i guess we'll see | |
| 14:34:15 | dansmith | gibi: ack, well, that's what I've got up, so maybe go vote on that :) | |
| 14:34:16 | gibi | the real break happens if we delete something | |
| 14:34:25 | gibi | dansmith: will do | |
| 14:34:32 | mriedem | yeah the people -1ing the change are saying "please don't delete it" | |
| 14:34:35 | mriedem | which isn't what the patch does | |
| 14:34:52 | dansmith | mriedem: I quote you: " After three months since the quality warning change merged [1] | |
| 14:34:52 | dansmith | there has still been no progress in finding a maintainer for" | |
| 14:34:55 | mriedem | they can -1 the patch to remove the driver in a later release if it gets to that point | |
| 14:35:19 | dansmith | https://github.com/openstack/nova/commit/af280ffe3098b84123eae218989ea056e9935bf1 | |
| 14:35:47 | dansmith | but I'd rather go straight to deprecation, | |
| 14:35:47 | mriedem | good thing i was so verbose in my commit messages :) | |
| 14:36:06 | dansmith | and then we can avoid removing it if CI shows up, and un-deprecate if it looks like CI will stick around | |