| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-08 | |||
| 06:36:07 | openstackgerrit | Chris Friesen proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363 | |
| 06:37:20 | openstackgerrit | Chris Friesen proposed openstack/nova master: Add emulated TPM support to Nova -- testcase test only https://review.openstack.org/641932 | |
| 06:44:17 | openstackgerrit | Chris Friesen proposed openstack/nova master: Split up _is_storage_shared_with() in libvirt driver https://review.openstack.org/639933 | |
| 06:44:17 | openstackgerrit | Chris Friesen proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363 | |
| 06:44:18 | openstackgerrit | Chris Friesen proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.openstack.org/639934 | |
| 09:11:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use Selection object to fill request group mapping https://review.openstack.org/638711 | |
| 09:43:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use Selection object to fill request group mapping https://review.openstack.org/638711 | |
| 09:45:22 | aspiers | kashyap: I'm intrigued in which ways is the term "domain" Xen-specific? | |
| 09:45:52 | aspiers | AFAICS libvirt uses it everywhere | |
| 09:45:52 | kashyap | aspiers: libvirt's terminology of "domain" for a "guest" is a hold-over from Xen times. | |
| 09:46:14 | kashyap | aspiers: Can you explain what is a "host domain"? :-) | |
| 09:46:34 | kashyap | I mean, not being flippant. Just trying to put myself in an operator's shoes reading the log | |
| 09:46:34 | aspiers | Your point being that KVM doesn't have a dom0? | |
| 09:46:37 | kashyap | Yep | |
| 09:46:55 | kashyap | "host hypervisor" is more generic | |
| 09:47:01 | kashyap | Not sure you agree :-) | |
| 09:47:18 | aspiers | No, it's a good point that operators read logs and are less likely to understand arcane libvirt / Xen terminology | |
| 09:47:34 | aspiers | I do agree | |
| 09:48:04 | aspiers | I was trying to adhere to libvirt terminology since that's what the code aligns with, but that's not as good for operators so I'll change it. | |
| 09:49:43 | kashyap | aspiers: Nod, I did notice you were being careful to use the correct terminology | |
| 10:20:21 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add detection of SEV support from QEMU/AMD-SP/libvirt on AMD hosts https://review.openstack.org/633855 | |
| 10:25:14 | aspiers | kashyap: https://review.openstack.org/#/c/633855/10..11/nova/virt/libvirt/host.py | |
| 10:25:46 | kashyap | Thank you | |
| 10:26:06 | kashyap | aspiers: Also, once again kudos for the very nice doc string | |
| 10:26:12 | aspiers | thanks :) | |
| 10:37:33 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: qemu: Make disk image conversion dramatically faster https://review.openstack.org/640781 | |
| 10:37:33 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Use 'writeback' QEMU cache mode for Nova instances https://review.openstack.org/641981 | |
| 10:48:22 | kashyap | stephenfin: Commented here: https://review.openstack.org/#/c/640866/; it's a simple phrasing fix that removes any ambiguity whatsover, let's please do it? | |
| 11:08:02 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use Selection object to fill request group mapping https://review.openstack.org/638711 | |
| 11:28:20 | sean-k-mooney | did we kill force evacuate yesterday when we released python-novaclient | |
| 11:28:22 | sean-k-mooney | http://logs.openstack.org/65/607365/24/gate/nova-live-migration/0e40d39/job-output.txt.gz#_2019-03-08_07_40_10_653566 | |
| 11:30:42 | sean-k-mooney | looks like we broke the nova-live-migration job | |
| 11:31:03 | sean-k-mooney | lets see if i can figure out where to fix that | |
| 11:37:07 | bauzas | sean-k-mooney: that's why we provided a major version | |
| 11:37:17 | sean-k-mooney | ya | |
| 11:37:21 | sean-k-mooney | i found the issue | |
| 11:37:38 | sean-k-mooney | https://github.com/openstack/nova/blob/1554d35834a474514f827449bd7d4f1d2f0af1d6/gate/test_evacuate.sh#L64-L90 | |
| 11:37:52 | sean-k-mooney | not really sure if i should just remove that or what | |
| 11:37:57 | bauzas | the force flag is no longer accepted | |
| 11:38:02 | sean-k-mooney | i know | |
| 11:38:04 | bauzas | unless you provide a specific microversion | |
| 11:38:10 | sean-k-mooney | we removed the api | |
| 11:38:25 | sean-k-mooney | ha i could use the old microverion yes | |
| 11:39:03 | bauzas | AFAIR, we still need to use the force flag or just using a specific aggregate for forcing to use a compute | |
| 11:39:06 | sean-k-mooney | ya ok ill do that and ill see if we can de evacuate with osc while ime at it | |
| 11:39:14 | bauzas | sean-k-mooney: thanks | |
| 11:43:59 | openstackgerrit | Adam Spiers proposed openstack/nova master: Use assertXmlEqual() helper for all XML comparison tests https://review.openstack.org/641852 | |
| 11:44:35 | aspiers | stephenfin: https://review.openstack.org/#/c/641852/2..3/nova/test.py | |
| 11:48:12 | sean-k-mooney | dumb question but --os-compute-api-version is used to set the microverion right | |
| 11:48:31 | sean-k-mooney | wew dont use the term micro version in the help text | |
| 11:52:18 | openstackgerrit | sean mooney proposed openstack/nova master: update gate test for removal of force evacuate https://review.openstack.org/641986 | |
| 12:20:39 | kashyap | mdbooth: On the off-chance you have time today: https://review.openstack.org/#/c/641981/ | |
| 12:20:54 | kashyap | FWIW, the code comment I wrote in there, I got it reviewed by Kevin Wolf of QEMU. | |
| 12:25:20 | shilpasd | smcginnis: Hi, need review for openstack/releases patch for masakariclient https://review.openstack.org/#/c/641988/1 | |
| 12:25:43 | shilpasd | dtantsur: Hi, need review for openstack/releases patch for masakariclient https://review.openstack.org/#/c/641988/1 | |
| 12:28:52 | dtantsur | shilpasd: hi, why do you want a review from me on that? I have no relationship to either releases or masakari (or nova) | |
| 12:30:03 | shilpasd | dtantsur: ohh, thanks for confirmation, please ignore the request | |
| 12:31:52 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Re-approve AMD SEV support for Train https://review.openstack.org/641994 | |
| 12:33:28 | kashyap | aspiers: s/Stein/Train/ in the commit message :-) | |
| 12:35:30 | bauzas | sean-k-mooney: yup, that's the field to use | |
| 12:39:19 | aspiers | kashyap: doh :) | |
| 12:39:42 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Re-approve AMD SEV support for Train https://review.openstack.org/641994 | |
| 12:48:17 | shilpasd | smcginnis: Thank you for workflow +1 for masakariclient https://review.openstack.org/#/c/641988/1 | |
| 12:51:23 | smcginnis | shilpasd: No problem. Probably best not to use the nova channel to discuss release activities for masakari though. ;) | |
| 12:52:31 | shilpasd | smcginnis: ok, will take a note of it | |
| 13:11:47 | cdent | gibi: is the rest of the bandwidth resource provider expected to land or is it being kicked to train? | |
| 13:12:38 | sean-k-mooney | cdent: on monday i think there were only 3 patches left for stien | |
| 13:12:44 | sean-k-mooney | the rest was going to be in train | |
| 13:12:52 | sean-k-mooney | i dont know if we got those 3 in | |
| 13:13:44 | sean-k-mooney | https://review.openstack.org/#/c/636360/ | |
| 13:13:50 | cdent | thanks sean-k-mooney. trying to make some notes about it in the pupdate | |
| 13:13:55 | sean-k-mooney | cdent: ^ that was the patch that enabled the feature i think] | |
| 13:14:51 | sean-k-mooney | so if i was following correctly we have landed enough for spwaning vms with bandwith request | |
| 13:15:05 | sean-k-mooney | all move opertaions are puntted to train | |
| 13:16:03 | gibi | cdent: the heal_allocation part is train (but will be backportad as that is a nova-manage extension). starts here: https://review.openstack.org/#/c/637953 | |
| 13:16:24 | cdent | thanks gibi | |
| 13:16:40 | sean-k-mooney | gibi: we are going to make this as compelte right https://blueprints.launchpad.net/nova/+spec/bandwidth-resource-provider | |
| 13:16:41 | gibi | cdent: there is two bug like things https://review.openstack.org/#/c/639608/ | |
| 13:16:48 | gibi | cdent: https://review.openstack.org/#/c/638711/ | |
| 13:16:54 | sean-k-mooney | and add a new spec/blueprint for the final bits for train? | |
| 13:17:08 | gibi | sean-k-mooney: I'm not sure, the original spec talked about serve move operations which was not finished (not even started) in Stein | |
| 13:17:29 | gibi | sean-k-mooney: I have to talk to melwitt and mriedem about the administration | |
| 13:17:39 | sean-k-mooney | sure | |
| 13:17:54 | gibi | cdent: and there is test coverage extension and refactor here https://review.openstack.org/#/c/641679/ | |
| 13:18:03 | sean-k-mooney | but enough has lannded that wyou can boot a vm with a min bandwith request now? | |
| 13:18:04 | gibi | sean-k-mooney: I'm OK in both ways | |
| 13:18:19 | gibi | sean-k-mooney: yeas, boot, delete, and detach port works | |
| 13:18:27 | cdent | \o/ | |
| 13:19:00 | sean-k-mooney | cool :) i need to go test that i guess but that is good to see | |
| 13:20:05 | sean-k-mooney | gibi: we dont have any tempest tests for this right? | |
| 13:20:30 | sean-k-mooney | we can praobly add them to the numa/nfv test discussion at the ptg | |
| 13:26:47 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Delete orphaned RS and IM records of archived instances https://review.openstack.org/560042 | |
| 13:30:55 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Rework 'EBUSY' (SIGKILL) error handling code path https://review.openstack.org/639091 | |
| 13:34:29 | gibi | sean-k-mooney: there is a tempest test proposed https://review.openstack.org/#/c/629253/ | |
| 13:35:15 | gibi | sean-k-mooney: today I finally able too make an env with real SRIOV so I tested that the SRIOV part of the feature works | |
| 13:35:24 | gibi | so far so good | |
| 13:39:55 | mriedem | i heard the gate is broken? | |
| 13:42:12 | mriedem | ah shit i see | |
| 13:42:28 | sean-k-mooney | mriedem: i proposed a patch for the --force issue | |
| 13:42:32 | mriedem | link? | |