| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-08-06 | |||
| 10:48:58 | gibi | mlycka, lyarwood: does the sync - async problem was resolved in the spec? | |
| 10:50:25 | mlycka | gibi: I think the idea was to treat is as sync for now and see where that gets us. The spec allows for a change later on during implementation. | |
| 10:51:35 | gibi | mlycka: I will add it to the meeting agenda. but we would need johnthetubaguy and Groka to check your answers to their questions as now we have -1s on the patch | |
| 10:52:11 | mlycka | gibi: Right right, thank you. Should I contact them directly or are they likely to be present in the meeting? | |
| 10:52:38 | lyarwood | mlycka: well the os-server-external-events API that you're using is async so I don't see how this could ever be sync tbh | |
| 10:52:55 | sean-k-mooney | that api is not for enduser to invoke | |
| 10:53:01 | sean-k-mooney | its for other services | |
| 10:53:15 | lyarwood | and that's also a valid point that I missed | |
| 10:53:20 | gibi | mlycka: given that it is summer I don't think they will get back to the spec today or tomorrow | |
| 10:53:21 | mlycka | Yeah, I think a separate endpoint is probably the way to go. | |
| 10:53:43 | sean-k-mooney | mlycka: quiesing a vm shoudl be a server action | |
| 10:53:59 | sean-k-mooney | as would undoing it | |
| 10:54:08 | sean-k-mooney | like pause and unpause | |
| 10:54:24 | mlycka | sean-k-mooney: What do you mean? Yeah, it would have to be done by the VM. | |
| 10:54:42 | sean-k-mooney | https://docs.openstack.org/api-ref/compute/#servers-run-an-action-servers-action | |
| 10:55:13 | sean-k-mooney | i mean you should be extending /servers/{server_id}/action | |
| 10:57:05 | mlycka | Okay, that doesn't sound unreasonable. Any objections? | |
| 10:57:32 | sean-k-mooney | i object to you using the event api :) but other then that no :) | |
| 10:58:58 | mlycka | Sure, makes sense. I won't have the time change the spec today, but I should be able to get to it tomorrow | |
| 10:59:58 | lyarwood | mlycka: would you also be able to start posting code for this? | |
| 11:00:25 | mlycka | lyarwood: I mean, not tomorrow but yeah, by next week I could start producing code. | |
| 11:00:35 | lyarwood | kk | |
| 11:03:13 | sean-k-mooney | mlycka: by the way the fact you used the service only external events api is proably why peopel assume this is only for use with cinder | |
| 11:04:01 | mlycka | sean-k-mooney: Might be. I think I misunderstood the use of that endpoint when I first found out about it. | |
| 11:04:27 | sean-k-mooney | ya that is why we have the warning https://docs.openstack.org/api-ref/compute/?expanded=add-associate-floating-ip-addfloatingip-action-deprecated-detail#create-external-events-os-server-external-events | |
| 11:05:34 | mlycka | Yep yep, clearly a reading comprehension failure on my part. | |
| 11:21:43 | openstackgerrit | sean mooney proposed openstack/os-vif master: support pyroute2 0.5.13 https://review.opendev.org/744809 | |
| 11:24:42 | openstackgerrit | sean mooney proposed openstack/nova stable/train: catch libvirt exception when nodedev not found. https://review.opendev.org/745116 | |
| 12:35:43 | artom | Dammit, we never updated the emulator thread policy flavor extra spec docs following the CPU resources patches: https://docs.openstack.org/nova/ussuri/user/flavors.html | |
| 12:36:05 | artom | (Scroll down to Emulator threads policy) | |
| 12:36:26 | sean-k-mooney | artom: well its still technically correct | |
| 12:36:37 | sean-k-mooney | artom: it just does not mention that the second half is only true | |
| 12:36:43 | artom | Only if cpu_dedicated_set isn't also set | |
| 12:36:43 | sean-k-mooney | if you use vcpu_pin_set | |
| 12:36:53 | sean-k-mooney | correct | |
| 12:37:15 | artom | And what's the point of documenting the old legacy behavior | |
| 12:37:21 | sean-k-mooney | vcpu_pin_set is deprecated not removed and we default to the legacy behavior | |
| 12:37:31 | artom | I suppose | |
| 12:37:42 | artom | Still, stuff is definitely missing | |
| 12:37:55 | sean-k-mooney | stephen can update it when he removes vcpu_pin_set in a few weeks | |
| 12:38:49 | sean-k-mooney | i guess you could update them to make it clear for train and ussuri before that | |
| 12:39:12 | sean-k-mooney | but its not going to be very useful on master since it wont matter when victoria is released | |
| 12:43:06 | artom | True | |
| 12:43:47 | sean-k-mooney | i guess you could look at it as motivation | |
| 12:43:57 | sean-k-mooney | if you want to impove it better do it quick before its gone | |
| 12:46:02 | artom | I need waaaay stronger motivation than that :P | |
| 12:46:21 | artom | Improving something only for it to disappear is not motivating :) | |
| 12:46:45 | artom | I realize that taken to the extreme I've just endorsed nihilism, but... | |
| 12:50:39 | openstackgerrit | sean mooney proposed openstack/nova master: remove deprecated schduler filters https://review.opendev.org/744800 | |
| 13:10:48 | lyarwood | can I just edit https://wiki.openstack.org/wiki/Meetings/Nova with agenda items for todays meeting? | |
| 13:14:11 | stephenfin | lyarwood: yup | |
| 13:15:12 | lyarwood | stephenfin: cool done thanks | |
| 13:37:00 | openstackgerrit | Merged openstack/nova master: compute: As cellsv1 is no more fold _detach_volume back into detach_volume https://review.opendev.org/731506 | |
| 13:43:26 | artom | stephenfin, we're not proposing bauzas's routed networks scheduling spec for exception? | |
| 13:43:45 | sean-k-mooney | i would have expected bauzas to do that | |
| 13:44:05 | artom | PTO | |
| 13:44:18 | sean-k-mooney | oh then yes we should | |
| 13:44:35 | sean-k-mooney | he was hoping that gibi would readd his +2 on monday | |
| 13:44:43 | sean-k-mooney | i think | |
| 13:45:17 | artom | I don't recall anyone except stephenfin being +2 | |
| 13:45:19 | kashyap | lyarwood: Hey, do you remember: when you cold-migrate an instance, will it in the end do a 'rebuild' on the destination? (/me goes to check the code) | |
| 13:45:27 | gibi | sean-k-mooney: ohh, I missed that | |
| 13:45:33 | sean-k-mooney | artom: gibi was +2 on v4 | |
| 13:46:11 | sean-k-mooney | gibi: we were ment to ping you to remind you, bauzas asked us too since he is on PTO | |
| 13:46:45 | sean-k-mooney | i mean we can still request an exception it technically needs one | |
| 13:46:50 | gibi | I will check it shortly and if I'm OK with it then I will ask for an exception on the todays meeting | |
| 13:55:16 | lyarwood | kashyap: resize | |
| 13:55:46 | lyarwood | kashyap: https://github.com/openstack/nova/blob/9ecefeb836964c52a5a2969b15c82b11c51d32ab/nova/api/openstack/compute/migrate_server.py#L76-L77 | |
| 13:57:58 | artom | lyarwood, could you revisit https://review.opendev.org/#/c/729763/11 when you get a chance? I didn't actually address any of your feedback, but hopefully my reasons make sense to you | |
| 13:58:10 | artom | stephenfin, ^^ same request, though I did fix the nit | |
| 14:01:16 | lyarwood | artom: yup was waiting for my changes to actually land before I went back to it | |
| 14:01:23 | lyarwood | artom: I'll hit it this afternoon | |
| 14:10:07 | sean-k-mooney | anyone rememebr off the top of there head when placment became required? | |
| 14:10:13 | sean-k-mooney | was it pike? | |
| 14:11:21 | lyarwood | sean-k-mooney: ocata I think | |
| 14:12:02 | sean-k-mooney | cool and prefilters were added in rocky right | |
| 14:16:29 | sean-k-mooney | yes they were. | |
| 14:40:58 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: mypy: Add type annotations https://review.opendev.org/745162 | |
| 14:40:59 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: mypy: Extend type annotations to 'vif_plug_noop' https://review.opendev.org/745163 | |
| 14:45:42 | openstackgerrit | Merged openstack/os-vif master: support pyroute2 0.5.13 https://review.opendev.org/744809 | |
| 15:12:50 | suryasingh | Hello Team | |
| 15:12:50 | suryasingh | Sorry to interrupt in middle of your work. | |
| 15:12:50 | suryasingh | I have a query about downsizing the nova instance. | |
| 15:12:50 | suryasingh | When I downsize with —poll status comes out to be Finished. But confirming the the same fails with below error. | |
| 15:12:50 | suryasingh | ERROR (Conflict): Cannot 'confirmResize' instance eaff75ba-1e16-4096-8882-33028d8b97fd while it is in vm_state active (HTTP 409) (Request-ID: req-71f53b9c-d01d-4bfe-a22c-b62401545f3c) | |
| 15:12:51 | suryasingh | So I just wanted to confirm, if it’s feasible operation from nova side and if not, is there any historical reason of it to not support downsize operation ? | |
| 15:26:33 | suryasingh | melwitt: dansmith lyarwood gmann efried would be very thankful if any of you respond to my query. | |
| 15:27:01 | gibi | suryasingh: do you try to downsize the disk in the flavor? | |
| 15:27:56 | suryasingh | @gibi whole flavor including disk. | |
| 15:28:02 | lyarwood | yeah I didn't think we supported down sizing tbh | |
| 15:28:14 | lyarwood | we can't downsize the disk at least | |
| 15:28:35 | gibi | the resize action fails if you try to decrease the disk size with 'Resize error: Unable to resize disk down.' | |
| 15:28:46 | gibi | the cpu and memory can be downsized | |
| 15:29:08 | sean-k-mooney | you can downsize the ram or cpus | |
| 15:29:11 | sean-k-mooney | but not the disk | |
| 15:29:20 | sean-k-mooney | as that would loose data potentially | |
| 15:29:39 | sean-k-mooney | oh gibi just said that | |
| 15:30:48 | suryasingh | gibi: sean-k-mooney lyarwood understood. Thanks for responding to query. | |
| 15:37:33 | stephenfin | Does anyone care to sanity check this idea for me? I end up wanting the feature every time I have to use osc-placement (along with auto-translation of RP names to UUIDs, but that's more work) https://review.opendev.org/#/c/743976/ | |
| 15:38:21 | stephenfin | As an aside, there are a quite a few patches queued up for review at https://review.opendev.org/#/q/is:open+project:openstack/osc-placement | |