| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-10 | |||
| 06:13:33 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 06:24:47 | arshad777 | I have created a sync replicated volume with peer persistence enabled. Attached this volume | |
| 07:18:55 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 07:18:56 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | |
| 07:36:57 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: doc: Add support microversions for options https://review.opendev.org/681174 | |
| 08:21:11 | bauzas | stephenfin: once you're there, see dansmith's comments | |
| 08:21:27 | bauzas | stephenfin: he wants to squash https://review.opendev.org/#/c/680983/ | |
| 08:21:44 | bauzas | stephenfin: so please try to provide the new revisions this morning | |
| 08:51:03 | stephenfin | sure thing | |
| 09:02:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140 | |
| 09:11:36 | openstackgerrit | weibin proposed openstack/nova master: Add support for using ceph RBD ereasure code https://review.opendev.org/681188 | |
| 09:19:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972 | |
| 09:21:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980 | |
| 09:23:48 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497 | |
| 09:24:40 | stephenfin | bauzas: trivial patch with no conflicts here https://review.opendev.org/#/c/679339/ | |
| 09:24:45 | stephenfin | (if you'd be so kind) | |
| 09:25:02 | bauzas | reminder : I'm French, I'm never kind | |
| 09:25:41 | stephenfin | bauzas: I'm going to rebase my the cpu-resources series on top of the SEV series to head off the incoming merge conflicts too | |
| 09:25:50 | stephenfin | while I rework it, that is | |
| 09:26:05 | bauzas | stephenfin: +Wd FWIW | |
| 09:26:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827 | |
| 09:26:17 | bauzas | stephenfin: cool, I'll look at gibi's series then | |
| 09:28:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019 | |
| 09:30:40 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Extract pf$N literals as constants from func test https://review.opendev.org/680991 | |
| 09:30:41 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Improve dest service level func tests https://review.opendev.org/680998 | |
| 09:34:41 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023 | |
| 09:38:26 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023 | |
| 09:48:51 | bauzas | gibi: I'm almost done with https://review.opendev.org/#/c/676140 but I'll need to do some checks later today | |
| 09:57:18 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023 | |
| 10:09:57 | aspiers | stephenfin: I'm here in case you have any last minute questions about https://review.opendev.org/#/c/644565/ | |
| 10:21:31 | gibi | bauzas: thanks | |
| 10:31:48 | openstackgerrit | Alexandra Settle proposed openstack/nova master: Fixing broken links https://review.opendev.org/681206 | |
| 10:32:22 | openstackgerrit | Alexandra Settle proposed openstack/nova master: Fixing broken links https://review.opendev.org/681206 | |
| 11:00:15 | aspiers | Is there a way to get a guest's libvirt XML via the nova api? I'm guessing no | |
| 11:01:09 | aspiers | Would be nice if it was available from https://docs.openstack.org/api-ref/compute/?expanded=show-server-diagnostics-detail#show-server-diagnostics | |
| 11:01:58 | aspiers | Without the XML I am struggling to see how Tempest can verify that an SEV guest was actually booted with SEV enabled | |
| 11:07:23 | gmann | aspiers: same case for NFV use case testing from artom L180 -https://etherpad.openstack.org/p/qa-train-ptg | |
| 11:07:31 | aspiers | hmm | |
| 11:07:49 | aspiers | gmann: which line? | |
| 11:08:08 | gmann | in QA PTG, we accepted the idea of adding that tempest plugin into QA but it was action item for artom to propose that. | |
| 11:08:11 | gmann | L180 | |
| 11:08:15 | aspiers | thanks | |
| 11:08:36 | aspiers | oh nice | |
| 11:08:45 | aspiers | yeah "white box plugin" sounds like a good concept | |
| 11:09:09 | gmann | yeah and it can be expanded with more use case which is out of scope from Tempest | |
| 11:09:22 | aspiers | right | |
| 11:10:31 | aspiers | gmann: although I wonder if adding libvirt XML to the show-server-diagnostics API call might be a simpler solution | |
| 11:11:13 | aspiers | showing the XML could be an admin-only thing | |
| 11:12:51 | gmann | but even admin-only does it expose more info than nova should ? (from security point of view.) | |
| 11:13:38 | gmann | I think it was discussed previously also but sean-k-mooney or artom might know more on that. | |
| 11:16:31 | artom | gmann, yeah, I still want to do that, I was meant to propose a spec for Train but only have a very WIP up | |
| 11:17:39 | artom | The code exists in RDO project's gerrit, and a Red Hat QE and myself wanted to clear outstanding reviews on there and merging its tests for NUMA live migration | |
| 11:17:47 | artom | But that didn't happen yet | |
| 11:17:54 | gmann | i see. | |
| 11:18:19 | sean-k-mooney | gmann: sorry i was not following chat | |
| 11:18:27 | sean-k-mooney | what were we talking about | |
| 11:18:48 | gmann | sean-k-mooney: question from aspiers on exposing the libvirt XML to the show-server-diagnostics API | |
| 11:18:56 | sean-k-mooney | aspiers: no there is not a way to get the xml form the api | |
| 11:19:00 | sean-k-mooney | and there nerver will be | |
| 11:19:10 | aspiers | that's a bold statement :) | |
| 11:19:35 | sean-k-mooney | it completely violates the cloud abstration to expose that level of detail via the api | |
| 11:19:49 | sean-k-mooney | a non admin is not even ment to know the hypervior that is in use | |
| 11:20:00 | aspiers | sean-k-mooney: as admin-only diagnostics there is no violation | |
| 11:20:17 | aspiers | sean-k-mooney: we're not talking about non-admins | |
| 11:20:26 | sean-k-mooney | as admin only technically be they could jsut ssh into the host and look at the xml | |
| 11:20:37 | aspiers | sean-k-mooney: not from tempest they can't | |
| 11:20:49 | aspiers | plus that's a lot less convenient | |
| 11:20:53 | sean-k-mooney | tempest shoudl not be asserting behavior of the xml generation | |
| 11:20:59 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023 | |
| 11:21:13 | aspiers | sean-k-mooney: please first read the use case above to understand the need ^^^ | |
| 11:21:13 | sean-k-mooney | that is what functional or white box testing is for | |
| 11:21:14 | artom | aspiers, yeah, that's quite explicitly out of scope for tempest | |
| 11:21:20 | sean-k-mooney | tempest if for blackbox testing | |
| 11:21:25 | artom | But... quite explicitly *in* scope for whitebox :) | |
| 11:21:35 | gmann | yeah | |
| 11:21:44 | artom | So now you've landed yourself on the list of people interested, and will be poked mercilessly once it's ready ;) | |
| 11:21:44 | aspiers | again, this is repeating discussion from a few minutes ago when we talked about the white box plugin | |
| 11:22:16 | aspiers | if there is a white box plugin for tempest, then that means white box testing *is* in scope for the tempest ecosystem, even if not the core | |
| 11:22:16 | sean-k-mooney | aspiers: so why cant you boot a vm and ssh into an detect that sev is configured from within the vm? | |
| 11:22:20 | sean-k-mooney | or at least available | |
| 11:22:30 | aspiers | sean-k-mooney: how would I detect that? | |
| 11:22:35 | sean-k-mooney | lscpu? | |
| 11:22:45 | sean-k-mooney | is there not an msr or cpu flag for sev | |
| 11:22:45 | aspiers | sean-k-mooney: have you tested that? | |
| 11:22:55 | sean-k-mooney | no i dont have sev hardware | |
| 11:23:20 | gmann | aspiers: within scope of QA ecosystem not Tempest ecosystem. Tempest is just a tool under QA :) | |
| 11:23:24 | artom | aspiers, yeah, we're not saying "don't do it", we're saying "don't propose patches for it to Tempest" | |
| 11:23:46 | aspiers | artom: OK, it sounded like the former before :) | |
| 11:24:02 | aspiers | gmann: the white box tempest plugin isn't in the tempest ecosystem? ;-) | |
| 11:24:10 | sean-k-mooney | aspiers: whitebox and the intel nfv test repo use tempest as a framework | |
| 11:24:16 | sean-k-mooney | to do this type of testing | |
| 11:24:30 | sean-k-mooney | but its not in tempest as its out of scope fo tempest | |
| 11:24:47 | sean-k-mooney | but a whitebox style tempest plugin for sev would be fine | |
| 11:24:53 | sean-k-mooney | or add it to whitebox | |
| 11:25:04 | artom | sean-k-mooney, unrelated, but we had some discussion around saving the new NUMA topology in https://review.opendev.org/#/c/634606/75/nova/compute/manager.py@7223 - and in func tests at least, that instance.refresh() isn't necessary (yes, the tests check the InstanceNUMATopology) | |
| 11:25:21 | gmann | aspiers: It will be QA ecosystem. it can be done via tempest plugin or separate testing framework like extreme-testing( which never got progress ). but it will be separate project under QA with separate team. | |
| 11:25:34 | artom | sean-k-mooney, I'll try to get to the office later this morning, to see what's up with my machine, would you have the bandwidth to play around with that in the meantime in your env? | |
| 11:26:16 | sean-k-mooney | artom: since this is aparently our highest priority i can make time | |
| 11:26:23 | sean-k-mooney | what exactly do you want me to test | |
| 11:26:35 | sean-k-mooney | remove the instace.refresh | |