| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-28 | |||
| 11:06:59 | bauzas | this is for a PCI ID | |
| 11:07:01 | gibi | btw, bauzas, this trait was not mentioned in the spec so I added a comment to the merged spec now and I will push a follow up on the spec to sync it with the implementation reality | |
| 11:07:03 | bauzas | hence singular | |
| 11:07:20 | sean-k-mooney | gibi: we can update the spec later | |
| 11:07:25 | gibi | yeah I | |
| 11:07:29 | sean-k-mooney | based on the final merged implemeantion | |
| 11:07:30 | gibi | I'm planning to do so | |
| 11:07:32 | bauzas | gibi: ok, so I wasn't lazy then :D | |
| 11:07:39 | bauzas | :ô | |
| 11:07:39 | bauzas | :ô | |
| 11:07:40 | bauzas | :p | |
| 11:07:42 | gibi | I'm collecting the comments there to know what to update | |
| 11:07:51 | sean-k-mooney | we also move the logic to the compute manager | |
| 11:07:55 | gibi | sean-k-mooney: yepp | |
| 11:08:11 | sean-k-mooney | i dont know if we eplxitly said tiwoudl be in the driver before but its good to document | |
| 11:08:32 | bauzas | gibi: I'll try to look at your series before tuesday eob | |
| 11:09:06 | bauzas | oh wait | |
| 11:09:09 | gibi | bauzas: when you look try to focus on the high level direction and architecture please | |
| 11:09:18 | bauzas | gibi: https://github.com/openstack/os-traits/tree/master/os_traits/hw | |
| 11:09:34 | gibi | we discussed with sean-k-mooney that this is not a hw trait | |
| 11:09:37 | bauzas | could be hw | |
| 11:09:40 | gibi | but more like a capability trait | |
| 11:09:43 | gibi | of the compute service | |
| 11:09:50 | bauzas | meh ok | |
| 11:12:52 | opendevreview | Merged openstack/os-traits master: Add COMPUTE_MANAGED_PCI_DEVICE trait https://review.opendev.org/c/openstack/os-traits/+/850538 | |
| 11:12:59 | gibi | bauzas, sean-k-mooney: thanks! | |
| 11:13:35 | gibi | I'm going to push a os-traits release | |
| 11:15:39 | sean-k-mooney | lets wait a sec | |
| 11:15:40 | bauzas | damn | |
| 11:15:41 | bauzas | | nova | 11 | 22 | 50.0 | | |
| 11:15:47 | bauzas | 50% of blind rechecks | |
| 11:15:49 | bauzas | man ! | |
| 11:15:56 | sean-k-mooney | it would be nice to do https://review.opendev.org/c/openstack/os-traits/+/844336 too | |
| 11:16:38 | bauzas | don't people read our meeting ? :)= | |
| 11:16:40 | sean-k-mooney | ricolin: ^ coudl you respin that quickly and remvoe the none | |
| 11:17:23 | sean-k-mooney | well | |
| 11:17:31 | sean-k-mooney | ricolin: we could leave it i guess | |
| 11:17:45 | sean-k-mooney | ah actully no | |
| 11:18:08 | sean-k-mooney | vioummu=non shouuld not force libvirt | |
| 11:18:32 | sean-k-mooney | so we should remove the trait and not request a trait if you set the exptra spec to none | |
| 11:19:00 | sean-k-mooney | gibi: bauzas alernitivly i could adress my own nit in that patch and ye could review | |
| 11:19:09 | gibi | sean-k-mooney: go for it | |
| 11:19:13 | gibi | I can respin the releasenotes | |
| 11:19:22 | gibi | s/releasenotes/release/ | |
| 11:21:26 | opendevreview | sean mooney proposed openstack/os-traits master: Add traits for vIOMMU https://review.opendev.org/c/openstack/os-traits/+/844336 | |
| 11:22:30 | sean-k-mooney | oh the depends on is wrong | |
| 11:22:49 | opendevreview | sean mooney proposed openstack/os-traits master: Add traits for vIOMMU https://review.opendev.org/c/openstack/os-traits/+/844336 | |
| 11:23:54 | sean-k-mooney | ok ^ should be good to review | |
| 11:27:34 | gibi | done +2 | |
| 12:19:21 | gibi | stephenfin, sean-k-mooney: so far I fixed the duplicated mocking in libvirt.test_driver and all of them was either an unnecessary second mocking or can be fixed easily. So I think while it is manual work it can be done. I will try to fix them today and push a patch | |
| 12:19:49 | sean-k-mooney | sound promising | |
| 12:21:11 | gibi | I think it is worth to fix as there was really stupid mistakes caught by this. like mocking the same thing twice in two consecutive decorator and applying assert on the second mock :) | |
| 12:43:08 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 12:48:43 | bauzas | ok, will rebase my novaclient change on top of ^ | |
| 12:50:06 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 12:52:01 | Uggla | bauzas, hoping you will not have too many conflicts. | |
| 12:52:17 | bauzas | Uggla: nope, just one I guess | |
| 12:52:23 | bauzas | for the microversion number | |
| 13:01:28 | opendevreview | Sylvain Bauza proposed openstack/python-novaclient master: Add support for 2.92 : keypair import mandatory https://review.opendev.org/c/openstack/python-novaclient/+/851231 | |
| 13:01:41 | bauzas | Uggla: actually, no merge conflict I got \o/ | |
| 13:01:45 | bauzas | just a simple rebase | |
| 13:01:56 | Uggla | bauzas, \o/ | |
| 13:07:10 | opendevreview | Amit Uniyal proposed openstack/nova master: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/848886 | |
| 13:33:54 | stephenfin | gibi: nice work :) | |
| 14:12:47 | gibi | TIL: if there are both side_effect and return_value is defined on a mock then return_value is ignored | |
| 14:14:50 | gibi | bauzas: when you are around, could you hit https://review.opendev.org/c/openstack/os-traits/+/844336 ? | |
| 14:15:04 | bauzas | I am and I can | |
| 14:15:21 | gibi | thank you | |
| 14:15:32 | gibi | I will update the os-trait release patch once that lands | |
| 14:15:58 | bauzas | done | |
| 14:17:31 | gibi | awesome thanks | |
| 14:26:56 | opendevreview | Merged openstack/os-traits master: Add traits for vIOMMU https://review.opendev.org/c/openstack/os-traits/+/844336 | |
| 14:30:31 | gibi | sean-k-mooney, bauzas: I've updated the os-traits release patch https://review.opendev.org/c/openstack/releases/+/851390 | |
| 14:33:18 | efried | :* | |
| 14:33:30 | gibi | o/ | |
| 14:34:27 | gibi | efried: how is life? | |
| 14:35:38 | efried | What's new with y'all? | |
| 14:35:38 | efried | Living the dream. Working on https://github.com/openshift/hive/ mostly. Enjoy it a lot. | |
| 14:38:12 | gibi | good to hear that | |
| 14:39:25 | gibi | I got involved writing k8s operators to deploy OpenStack so I will look at hive at some point to learn some tricks :) | |
| 14:40:05 | gibi | while also trying to make PCI tracking in Placement happen | |
| 14:44:18 | efried | In theory hive will deploy OpenStack -- though I've never tried it myself. | |
| 14:44:18 | efried | Not gonna claim hive is the examplar of k8s operators, but it has some pretty cool stuff going on. | |
| 14:47:19 | whoami-rajat | hi #openstack-nova , wanted to request reviews on my feature, it's been sitting for 3 months without reviews and just wanted to get it in in the Zed cycle https://review.opendev.org/c/openstack/nova/+/831219 | |
| 14:47:33 | whoami-rajat | 1) https://review.opendev.org/c/openstack/nova/+/820368 | |
| 14:47:36 | whoami-rajat | 2) https://review.opendev.org/c/openstack/nova/+/831219 | |
| 14:47:42 | whoami-rajat | 3) https://review.opendev.org/c/openstack/nova/+/830883 | |
| 14:49:16 | dansmith | whoami-rajat: sorry I lost track of that.. tempest tests are now working? | |
| 14:49:38 | dansmith | whoami-rajat: the top patch is in merge conflict | |
| 15:01:14 | sean-k-mooney | i started reviewing on eof thos yesterday but was tied up with downstream stuff | |
| 15:01:49 | sean-k-mooney | oh it was the tempest patch | |
| 15:03:12 | sean-k-mooney | dansmith: im pretty sure that passwords are optional for rebuild even if you orginaly set one but im not sure about what tempest needs | |
| 15:03:18 | sean-k-mooney | im expecting it to use the ssh key it has | |
| 15:03:35 | sean-k-mooney | and fall back to the default cirros one otherwise based on its config | |
| 15:04:15 | dansmith | sean-k-mooney: yeah, but that code was copied from somewhere else, and it looked to me like it might be required in case they're using passwords instead of keys | |
| 15:04:29 | dansmith | but regardless, it's just not part of the test so the comment was irrelevant I think | |
| 15:05:31 | sean-k-mooney | ack | |
| 15:05:58 | sean-k-mooney | i havent looked at the nova change yet | |
| 15:06:18 | sean-k-mooney | its on my radar | |
| 15:06:55 | sean-k-mooney | whoami-rajat:^ | |