| congnt | sean-k-mooney: Hi, what libvirt version fixed issue about Icelake without flag mpx? Thanks | 02:26 |
|---|---|---|
| congnt | And what file xml in new libvirt version I can use for Icelake no mpx? I saw in libvirt 8.0, x86_Icelake-Server-noTSX.xml still have mpx flags. Thank you | 02:40 |
| *** dasm|off is now known as Guest149 | 03:32 | |
| opendevreview | Nobuhiro MIKI proposed openstack/nova-specs master: Add maxphysaddr support for Libvirt https://review.opendev.org/c/openstack/nova-specs/+/861033 | 08:43 |
| opendevreview | Jorge San Emeterio proposed openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | 11:24 |
| *** kopecmartin|sick is now known as kopecmartin | 11:55 | |
| sean-k-mooney | i said this downstream but if im needed ping my matirx accout here sean-k-mooney[m] ill be disconencting my irc client until the new year when im back form pto | 12:22 |
| *** Guest149 is now known as dasm | 12:30 | |
| opendevreview | Jan Hartkopf proposed openstack/nova-specs master: Repropose spec for updating user data https://review.opendev.org/c/openstack/nova-specs/+/863884 | 12:41 |
| *** umbSublime is now known as Guest182 | 13:11 | |
| *** umbSubli1 is now known as umbSublime | 13:11 | |
| opendevreview | ribaudr proposed openstack/nova master: Adds a repoducer for post_live_migration_at_destination failures https://review.opendev.org/c/openstack/nova/+/868088 | 14:03 |
| opendevreview | ribaudr proposed openstack/nova master: Fix bug ... https://review.opendev.org/c/openstack/nova/+/868089 | 14:03 |
| Uggla | Hi gibi, could you have a look at the above patches ^ | 14:05 |
| opendevreview | ribaudr proposed openstack/nova master: Adds a repoducer for post_live_migration_at_destination failures https://review.opendev.org/c/openstack/nova/+/868088 | 14:22 |
| opendevreview | ribaudr proposed openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | 14:22 |
| gibi | Uggla: you were faster, I've just left comment in the fix https://review.opendev.org/c/openstack/nova/+/868089 | 14:32 |
| gibi | but for the previous ps | 14:33 |
| Uggla | gibi, yep I have seen them. And realize how dumb I am. | 14:33 |
| gibi | Uggla: you are not dump, I think your reproducer and fix is correct. I only have minor nits | 14:33 |
| gibi | I will be off after today, so if you can respin it still today then I can +2 it | 14:34 |
| gibi | and then we can pull in bauzas to +A it :) | 14:35 |
| Uggla | yep fixing it ASAP. Note: I have opened a nova bug on launchpad as well and assigned it to me. I think it is what is expected in this case. | 14:36 |
| gibi | yepp | 14:39 |
| opendevreview | ribaudr proposed openstack/nova master: Adds a repoducer for post_live_migration_at_destination failures https://review.opendev.org/c/openstack/nova/+/868088 | 14:40 |
| opendevreview | ribaudr proposed openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | 14:40 |
| opendevreview | ribaudr proposed openstack/nova master: Adds a repoducer for post_live_migration_at_destination failures https://review.opendev.org/c/openstack/nova/+/868088 | 14:56 |
| opendevreview | ribaudr proposed openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | 14:56 |
| Uggla | gibi, that should be ok now. | 15:00 |
| gibi | Uggla: I'm +2 on both | 15:02 |
| Uggla | gibi, cool ! hoping it will be ok for bauzas, but it is easier I can give him a chocolate bribe. :D | 15:04 |
| gibi | :) | 15:05 |
| bauzas | Uggla: gibi: sorry, focusing on finishing the cpu impl series | 15:11 |
| bauzas | but I can try to look | 15:11 |
| bauzas | Uggla: hmmm, I'm a bit torn about using an existing regression test | 15:14 |
| bauzas | because if you want to backport a bugfix, it's simplier to have a specific regression test file | 15:15 |
| Uggla | bauzas, I thought to create a new file, but it is more or less the same subject. That's the reason why I appended to the existing one. | 15:15 |
| bauzas | Uggla: the problem is about backporting | 15:16 |
| Uggla | bauzas, I understand if you wish I can split in another file. | 15:16 |
| bauzas | Uggla: https://review.opendev.org/c/openstack/nova/+/868088 | 15:17 |
| bauzas | Uggla: I don't know if you want to backport the fix down to Train or if the existing regression file is on Train (for example), but here my concern is not really about *this* fix but rather about the usage | 15:18 |
| bauzas | because as a reminder, people sometimes just looks at code and copy/paste what they want without thinking about any reason why we do or we don't this | 15:19 |
| Uggla | ok, I guess it can be backported down to train as soon as Amit patch will be merged. | 15:20 |
| *** dasm is now known as dasm|off | 23:40 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!