| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-10 | |||
| 16:42:18 | artom | But reality and all that | |
| 16:42:22 | efried | aspiers: And if you're collecting a spec tweak fup patch, clarify in there. | |
| 16:42:55 | sean-k-mooney | for what its worth however i think we can encode it here in a new 3 line function https://github.com/openstack/nova/blob/master/nova/network/model.py#L461 | |
| 16:43:21 | artom | sean-k-mooney, yeah, that part's not hard | |
| 16:43:43 | artom | It's how we use it that I'm fuzzier on | |
| 16:43:55 | artom | Ie, split events to be waited for between compute and libvirt | |
| 16:44:26 | artom | I'm making an executing decision: ph00d, then hax. A patch, even broken WIP, is better than endless discussion | |
| 16:44:28 | mriedem | i'm not terribly comfortable in backporting something that gets this complicated especially since we don't test any of these vif types in the gate | |
| 16:44:34 | sean-k-mooney | e.g. def wait_in_manager(self): | |
| 16:44:36 | sean-k-mooney | retrun self.is_hybrid_plug_enabled or self.get("vnic_type") == "direct-physical" | |
| 16:45:00 | mriedem | artom: hacking something up that you do'nt have agreement on from the people reviewing it isn't going to speed this up | |
| 16:45:11 | sean-k-mooney | then in the manager an driver we just do if vif.wait_in_manager(): | |
| 16:45:13 | artom | mriedem, I thought we kinda did? | |
| 16:45:27 | mriedem | how about, 1. artom gets food, 2. sean comments on the change | |
| 16:45:30 | artom | mriedem, for backporting, if we split the patch and only backport the hybrid stuff? | |
| 16:45:30 | mriedem | 3. artom digests both | |
| 16:45:41 | artom | Leaving the SRIOV stuff for master only | |
| 16:46:37 | sean-k-mooney | artom: or we can backport this downstream only if we really have to | |
| 16:46:54 | sean-k-mooney | but ill comment on the review i think we can make this simpler | |
| 16:46:57 | artom | sean-k-mooney, it's always an option, but not my first one, obviously | |
| 16:47:22 | sean-k-mooney | i think if we only fix it for ovs we will get a bug report form a different custoemr for sriov | |
| 16:47:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document how to revert, confirm a cold migration https://review.opendev.org/664396 | |
| 16:47:33 | sean-k-mooney | so we will have to fix both eventually | |
| 16:47:42 | artom | sean-k-mooney, yeah, and that's fine, and we can address it later | |
| 16:47:56 | artom | Especially since it'd be as simple as adding a check to that 3-line method of yours :) | |
| 16:48:27 | artom | We could even do it now in a fup patch, and backport if/when some testing happens | |
| 16:48:31 | sean-k-mooney | its actully 2 lines :P untill we have pep8 it | |
| 16:48:32 | artom | *fup patch now | |
| 16:49:02 | artom | (Though realistically it shouldn't merge until if/when some testing happens :) | |
| 17:02:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate RetryFilter https://review.opendev.org/663953 | |
| 17:06:01 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Change new image property to hw_mem_encryption https://review.opendev.org/664397 | |
| 17:26:37 | aspiers | efried: should I hold off including doc change fragments covering SEV support until the whole series lands? | |
| 17:27:12 | efried | aspiers: "doc change fragments" in the spec or in the codebase? | |
| 17:27:17 | aspiers | codebase | |
| 17:27:20 | aspiers | otherwise there's a risk of documenting stuff which doesn't work yet | |
| 17:27:26 | aspiers | especially if it didn't all land before Train | |
| 17:27:37 | efried | Right, those go in in the patch that flips the switch that makes it work. | |
| 17:27:45 | aspiers | OK thanks | |
| 17:27:46 | efried | docs & reno etc. | |
| 17:27:49 | aspiers | Yup | |
| 17:45:53 | spatel | sean-k-mooney: is there anyway to stop migration instance? I have initiate block migration and i want to cancel it | |
| 17:46:21 | sean-k-mooney | spatel: as in cancel a livemigration | |
| 17:46:26 | sean-k-mooney | i think we have a way too | |
| 17:46:31 | sean-k-mooney | let me check | |
| 17:46:38 | sean-k-mooney | i know people have asked for it before | |
| 17:46:44 | spatel | i didn't find any command line option or GUI option | |
| 17:47:16 | sean-k-mooney | well what you can do with crul directly to the api is > then the clis enable | |
| 17:47:30 | sean-k-mooney | i dont know if we ever landed the force complete and force abourt apis however | |
| 17:47:32 | spatel | ? | |
| 17:48:05 | sean-k-mooney | we dont always get around to adding everyting that the api support to the command line clients | |
| 17:48:13 | sean-k-mooney | or somethimes it takes a while | |
| 17:49:33 | spatel | what if i kill TCP connection which is transferring disk | |
| 17:49:57 | sean-k-mooney | spatel: you can abort it via libvirt using virsh | |
| 17:50:58 | sean-k-mooney | spatel: so ya it looke like you can use the old nova cli | |
| 17:51:11 | sean-k-mooney | nova live-migration-abort INSTANCE_ID MIGRATION_ID or nova live-migration-force-complete INSTANCE_ID MIGRATION_ID | |
| 17:51:31 | sean-k-mooney | spatel: https://docs.openstack.org/nova/pike/admin/live-migration-usage.html#addressing-migration-timeouts | |
| 17:52:34 | sean-k-mooney | spatel: nova live-migration-abort is what you are looking for. | |
| 17:52:54 | spatel | let me see | |
| 17:52:55 | sean-k-mooney | you need the instance id + the migration_id for the currently running migration | |
| 17:53:26 | sean-k-mooney | you get the migration id by doing "nova server-migration-list <server/instance uuid>" | |
| 17:54:10 | spatel | how do i find MIGRATION_ID? | |
| 17:54:25 | sean-k-mooney | spatel: ^ :) | |
| 17:54:58 | sean-k-mooney | spatel: its covered in the monitoring migration part of that doc https://docs.openstack.org/nova/pike/admin/live-migration-usage.html#monitoring-the-migration | |
| 17:55:08 | spatel | reading it.. | |
| 17:57:26 | sean-k-mooney | does that make sense you do "nova server-migration-list <server id>" | |
| 17:57:38 | sean-k-mooney | then that give you the list of migrations. | |
| 17:58:20 | sean-k-mooney | then if you have more then one you use nova server-migration-show <server uuid > <migration id> | |
| 17:58:32 | sean-k-mooney | to see which one was the most recent | |
| 17:58:42 | sean-k-mooney | and then that one is the migration you want to canchel with | |
| 17:58:53 | sean-k-mooney | nova live-migration-abort INSTANCE_ID MIGRATION_ID | |
| 18:00:02 | spatel | nova server-migration-list <server id> this is showing nothing about migration except saying | OS-EXT-STS:task_state | resize_migrating | |
| 18:00:36 | spatel | i am not seeing any ID... | |
| 18:04:02 | sean-k-mooney | spatel: am can you past the output to paste.openstack.org. feel free to remove anything sensitive | |
| 18:04:46 | sean-k-mooney | nova server-migration list shoudl vice you an output that start with Id | Source Node | Dest Node .... | |
| 18:05:01 | sean-k-mooney | the Id colum is the migration id | |
| 18:06:21 | spatel | hmm | |
| 18:09:00 | spatel | http://paste.openstack.org/show/752693/ | |
| 18:09:03 | spatel | sean-k-mooney: ^^ | |
| 18:09:27 | sean-k-mooney | spatel: openstack server show not migration list | |
| 18:10:08 | sean-k-mooney | do "nova migration-list --instance-uuid 580e624f-d1c9-496c-a591-a36c2de92e62 | |
| 18:10:12 | sean-k-mooney | " | |
| 18:11:02 | spatel | I can see lots of info npw | |
| 18:11:04 | spatel | now | |
| 18:11:19 | sean-k-mooney | ok do you have more then 1 row | |
| 18:11:33 | sean-k-mooney | if not you just need the value of the ID colum | |
| 18:12:22 | sean-k-mooney | then you can do "nova live-migration-abort 580e624f-d1c9-496c-a591-a36c2de92e62 <id> | |
| 18:12:26 | sean-k-mooney | " | |
| 18:12:46 | spatel | http://paste.openstack.org/show/752695/ | |
| 18:12:56 | spatel | sean-k-mooney: ^6 | |
| 18:13:03 | sean-k-mooney | ya so it would be | |
| 18:13:12 | spatel | is that UUID is migration ID? | |
| 18:13:13 | sean-k-mooney | "nova live-migration-abort 580e624f-d1c9-496c-a591-a36c2de92e62 384" | |
| 18:13:21 | sean-k-mooney | mp Id | |
| 18:13:25 | sean-k-mooney | *no | |
| 18:13:32 | sean-k-mooney | it the first column | |
| 18:13:42 | sean-k-mooney | the UUID column is the instance uuid | |
| 18:14:10 | spatel | to abort it i need migration ID right ( that i not able to find :( ) | |
| 18:14:39 | sean-k-mooney | spatel: so as i said its call Id in the output | |
| 18:14:57 | sean-k-mooney | so in this case it 384 and the command would be "nova live-migration-abort 580e624f-d1c9-496c-a591-a36c2de92e62 384" | |
| 18:15:24 | spatel | 384 | |