| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-08 | |||
| 16:47:14 | bauzas | dansmith: yeah, we can try | |
| 16:47:18 | efried | jaypipes Is there anything else needed to get that patch going? | |
| 16:47:26 | dansmith | bauzas: seems like the obvious thing to do, IMHO | |
| 16:47:33 | jaypipes | efried: nope. | |
| 16:47:33 | bauzas | dansmith: my point is, that run can last a while | |
| 16:47:35 | efried | jaypipes (All action items related to that discussion will happen in later/other patches) | |
| 16:47:41 | efried | jaypipes Cool. | |
| 16:47:41 | bauzas | dansmith: anyway, let's try | |
| 16:47:46 | jaypipes | efried: I just finishing up working on n-r-p in allocation candidates) | |
| 16:47:50 | efried | ++ | |
| 16:47:52 | dansmith | bauzas: if they aren't in the guest xml, then they weren't setup properly when we shut down the last time | |
| 16:48:11 | dansmith | bauzas: so if hard reboot blows the xml away, it was broken before reboot, thus init_host isn't making things worse | |
| 16:48:21 | bauzas | dansmith: that's also interesting to see that ovirt folks also had the approach of a deterministic UUID for the mdev | |
| 16:48:31 | bauzas | dansmith: FYI https://mpolednik.github.io/2017/05/21/vfio-mdev/ | |
| 16:48:52 | dansmith | bauzas: I think a deterministic uuid is a bad idea, personally | |
| 16:48:57 | ktibi | dansmith I have a private cloud plateform with ~1500 Vms, it's not very small for me :/ just a user create 200 instances at 4AM ^^ | |
| 16:49:17 | dansmith | ktibi: again, I'm not arguing | |
| 16:49:23 | bauzas | dansmith: I need to go to the doctor for 1 hour, but I'll work tonight | |
| 16:49:26 | mriedem | ktibi: can you find the related failure from the neutron logs? | |
| 16:49:32 | bauzas | let's wrap up that convo later | |
| 16:49:37 | mriedem | ktibi: all nova gets back from neutron is an event saying vif plugging failed | |
| 16:49:40 | dansmith | bauzas: okay | |
| 16:49:45 | mriedem | which doesn't tell us why it failed at all | |
| 16:49:55 | ktibi | dansmith yes no pb ;) just the issue can touch large cloud I think :/ | |
| 16:50:11 | bauzas | dansmith: anyway, the deterministic UUID thingy is separated from the other changes in my series, so if we see it's a non-sense, I'm fine dropping it | |
| 16:50:18 | dansmith | bauzas: ack | |
| 16:50:21 | mdbooth | lyarwood: Ok, that's slightly more complex, but still not a showstopper. We need to pull host volume operations out of _create_domain_and_network, and explicitly call _connect_volume before getting guest xml if required. | |
| 16:50:37 | mdbooth | Or just move it all into _get_guest_storage_config after all | |
| 16:50:38 | ktibi | mriedem I'll try to get logs from neutron | |
| 16:50:49 | mdbooth | Not what I had in mind, but would still be a consolidation | |
| 16:51:51 | mriedem | mdbooth: is this a WIP? https://review.openstack.org/#/c/531852/ | |
| 16:51:57 | mriedem | if so, can you WIPify it? | |
| 16:52:08 | mdbooth | mriedem: Yep, will do | |
| 16:52:12 | mriedem | thanks | |
| 16:52:20 | lyarwood | mdbooth: yeah context down to _get_guest_storage_config and then merge the encryptor stuff into _connect/disconnect_volume | |
| 16:52:42 | mdbooth | lyarwood: And *remove* it from _create_domain_and_network | |
| 16:53:02 | mdbooth | We can already be confident that nothing is relying on that to do volume connection, because it never has | |
| 16:53:11 | lyarwood | mdbooth: in one change? | |
| 16:53:34 | mdbooth | Was just thinking about that. I don't see why not. | |
| 16:53:50 | lyarwood | mdbooth: k, I'll reuse the refactor change | |
| 16:53:50 | mdbooth | You might want the swap_volume patch first, though | |
| 16:53:54 | mdbooth | And lxc | |
| 16:54:02 | mdbooth | My WIP above? | |
| 16:54:04 | lyarwood | ah true | |
| 16:54:11 | lyarwood | mdbooth: no my original | |
| 16:54:14 | mdbooth | If you're going to take it I'll just abandon that | |
| 16:54:17 | ktibi | mdbooth nothing in neutron-server.log :/ 0warning 0 error | |
| 16:54:21 | mdbooth | kk, cool | |
| 16:55:07 | ktibi | oups sorrty for mriedem | |
| 16:58:43 | mriedem | jaypipes: interested if adding an index is worth it for this https://review.openstack.org/#/c/531132/3/nova/db/sqlalchemy/migrate_repo/versions/376_add_migrations_updated_at_index.py - it's adding an index for listing migrations and filtering on updated_at (--changes-since); the query could have other columns/filters in it, so not sure if the index really helps or not | |
| 16:59:16 | jaypipes | mriedem: soon as I'm done with two other series reviews, will review. | |
| 16:59:20 | mriedem | although my guess is if you just did "nova migration-list --changes-since x" it would help | |
| 17:06:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | |
| 17:06:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | |
| 17:06:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | |
| 17:06:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add index(updated_at) on migrations table. https://review.openstack.org/531132 | |
| 17:06:51 | stephenfin | ralonsoh: Can we merge [1], given that [2] isn't working yet? [1] https://review.openstack.org/#/c/484386/ [2] https://review.openstack.org/#/c/530961/ | |
| 17:08:27 | mriedem | dtantsur: re: CI testing for ironic+bfv https://review.openstack.org/#/c/468353/ - is there at least an existing job that ironic uses for this? if so, we can add that to nova's experimental queue | |
| 17:09:46 | dtantsur | mriedem: yep ironic-tempest-dsvm-bfv | |
| 17:14:07 | dtantsur | wanna me propose a patch? | |
| 17:14:14 | efried | jaypipes (and another core, maybe stephenfin or bauzas?) would you please add https://review.openstack.org/#/c/518633/ to your list? The patch above that is already +W, and it'll be nice to have for your nrp test case (https://review.openstack.org/#/c/531443/) | |
| 17:14:50 | stephenfin | efried: Won't get to it myself today, but if it's still there tomorrow then I'll take a shot | |
| 17:14:57 | efried | stephenfin Thanks | |
| 17:15:15 | mriedem | dtantsur: sure | |
| 17:15:40 | mriedem | efried: i'll take a look | |
| 17:15:48 | efried | mriedem Thanks. | |
| 17:20:47 | ralonsoh | stephenfin: sorry for the delay. [2] is working now | |
| 17:20:50 | ralonsoh | stephenfin: openstack-tox-functional-with-sudoSUCCESS in 4m 00s | |
| 17:21:07 | stephenfin | ralonsoh: Oh, congrats! Nice work | |
| 17:21:10 | ralonsoh | stephenfin: Am I missing something? | |
| 17:21:15 | ralonsoh | stephenfin: oooook! | |
| 17:21:33 | ralonsoh | stephenfin: Sean's magic.... you know | |
| 17:21:41 | stephenfin | Oh, I know :) | |
| 17:21:55 | stephenfin | Cool. I'll review that tomorrow morning so | |
| 17:22:03 | ralonsoh | stephenfin: thanks! | |
| 17:22:08 | stephenfin | np | |
| 17:25:41 | jaypipes | efried: just finishing up a review on that, yes. | |
| 17:29:01 | jaypipes | efried: done | |
| 17:29:15 | EmilienM | mriedem: hey matt, I remember we had a thread about how to handles patches like https://review.openstack.org/531055 - I don't recall the outcome, do you have the link of the thread? | |
| 17:31:02 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 17:31:40 | mriedem | EmilienM: the eventual outcome was the first contact SIG | |
| 17:31:57 | EmilienM | ok | |
| 17:32:00 | mriedem | https://wiki.openstack.org/wiki/First_Contact_SIG | |
| 17:32:50 | mriedem | and http://superuser.openstack.org/articles/sig-first-contact-openstack/ | |
| 17:32:54 | mriedem | i guess point them at that? | |
| 17:33:05 | EmilienM | mriedem: ok, thx | |
| 17:33:10 | mriedem | should probably build something into the first-time contributor comment that people get on their patch too | |
| 17:33:15 | EmilienM | mwhahaha: ^ | |
| 17:33:49 | EmilienM | the thing is I'm not sure this is a new contributor: https://review.openstack.org/#/q/owner:shangxiaobj%2540inspur.com+status:merged | |
| 17:34:11 | EmilienM | the person is doing http to https a lot and it waste so much CI resources imho | |
| 17:34:18 | mriedem | yeah they aren't, | |
| 17:34:27 | mriedem | i'd probably ping huang zhipeng (howard) | |
| 17:34:38 | mriedem | he's in china and on the first contact sig group | |
| 17:35:12 | mriedem | i think a bunch of these contributors from chinese companies actually hang out in a wechat group for openstack, | |
| 17:35:28 | mriedem | so he sometimes proxies stuff to that group where people are actually listening (and speaking the same language) | |
| 17:35:34 | EmilienM | mriedem: ok thx for the advises | |
| 17:35:51 | mriedem | yw | |
| 17:42:52 | jaypipes | tox -efast8 would be fast if it didn't try to reinstall all the global requirements each time I run it. | |
| 17:43:20 | mriedem | it shouldn't try to re-install them | |
| 17:43:29 | mriedem | pip + wheels should have all of that cached | |