| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-31 | |||
| 16:58:22 | bauzas | ricolin: just said -1 for https://review.opendev.org/c/openstack/nova/+/830646 but if you don't have time today or tomorrow morning, lemme know | |
| 16:58:36 | bauzas | ricolin: we could just need a follow-up patch for my nits | |
| 16:58:47 | bauzas | and I'll +2 | |
| 17:15:40 | ricolin | bauzas: thanks, I will try to update it ASAP | |
| 17:16:19 | whoami-rajat | dansmith, do you have any idea about the nova multi cell breaking constantly or is it just flaky? | |
| 17:16:29 | dansmith | whoami-rajat: there's a fix in the queue | |
| 17:16:35 | dansmith | whoami-rajat: no point in rechecking until that lands | |
| 17:16:48 | whoami-rajat | ah ok | |
| 17:16:51 | dansmith | it has been in check for hours, not even in the gate yet | |
| 17:16:51 | whoami-rajat | wasn't aware | |
| 17:17:09 | whoami-rajat | doesn't sound good | |
| 17:30:36 | dansmith | whoami-rajat: it's in the gate now | |
| 17:30:48 | whoami-rajat | great | |
| 17:30:58 | whoami-rajat | can i have the link? | |
| 17:31:15 | dansmith | it's 855378 | |
| 17:31:24 | whoami-rajat | ack | |
| 17:56:48 | sean-k-mooney | whoami-rajat: sicne your change is approved even if its not merged by COB tommrow at m3 it can be rechecked to land it | |
| 17:56:59 | sean-k-mooney | so we will keep an eye on it until its landed | |
| 17:57:45 | sean-k-mooney | whoami-rajat: if you have a depends on form the nova client and osc patchs against the top patch we can review those in parralel | |
| 17:57:55 | whoami-rajat | sean-k-mooney, thanks, just wanted to update other project patches after this merged, guess it's safe to update the clients and tempest in parallel so will get to that | |
| 17:58:06 | whoami-rajat | sean-k-mooney, yep, i will just update them quickly and let you know | |
| 17:58:47 | sean-k-mooney | yep depends on will prevent them merging until the api change lands and then they can just be rechecked at that point | |
| 18:00:01 | whoami-rajat | ack, sounds good | |
| 18:14:14 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 18:14:15 | opendevreview | Rico Lin proposed openstack/nova master: Add traits for viommu model https://review.opendev.org/c/openstack/nova/+/844507 | |
| 18:15:12 | sean-k-mooney | ricolin: you still have not adress my feedback on https://review.opendev.org/c/openstack/nova/+/844507/15 | |
| 18:15:45 | sean-k-mooney | the way you are doing the traits reporting is wrong | |
| 18:16:54 | opendevreview | Rajat Dhasmana proposed openstack/python-novaclient master: Add support to rebuild boot volume 2.93 https://review.opendev.org/c/openstack/python-novaclient/+/827163 | |
| 18:18:08 | whoami-rajat | novaclient: https://review.opendev.org/c/openstack/python-novaclient/+/827163 | |
| 18:18:09 | whoami-rajat | osc: https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |
| 18:18:11 | whoami-rajat | sean-k-mooney, ^ | |
| 18:19:22 | sean-k-mooney | ack ill go review those now | |
| 18:20:02 | whoami-rajat | thanks! | |
| 18:28:35 | ricolin | sean-k-mooney: okay, checking the comments again | |
| 19:31:49 | dansmith | that skip is about to pop | |
| 19:32:04 | dansmith | next ten minutes or so | |
| 19:43:41 | sean-k-mooney | im going to go have dinner/breakfest | |
| 19:43:41 | sean-k-mooney | none of that is blocking the nova change but that just my feedback | |
| 19:43:41 | sean-k-mooney | whoami-rajat: i looked over the chagnes novaclinet looks good. minor issue with osc, and does the tempest change really need its own job? i would just add that test to tempest full if cinder is deployed and nova is new enough | |
| 19:50:33 | opendevreview | Merged openstack/nova master: Skip UnshelveToHostMultiNodesTest in nova-multi-cell https://review.opendev.org/c/openstack/nova/+/855378 | |
| 20:18:00 | opendevreview | Rico Lin proposed openstack/nova master: Add traits for viommu model https://review.opendev.org/c/openstack/nova/+/844507 | |
| 20:18:00 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 20:18:34 | ricolin | sean-k-mooney: can you check if I do make it right this time, thanks ^^^ | |
| 20:50:40 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 20:50:41 | opendevreview | Rico Lin proposed openstack/nova master: Add traits for viommu model https://review.opendev.org/c/openstack/nova/+/844507 | |
| 23:27:37 | opendevreview | Merged openstack/nova master: Add documentation and releasenotes for RBAC change https://review.opendev.org/c/openstack/nova/+/854882 | |
| #openstack-nova - 2022-09-01 | |||
| 05:10:33 | whoami-rajat | sean-k-mooney, it is already being tested in tempest-full but the problem is the image is same before/after rebuild so it's not testing the actual feature i.e. to reimage to a new image hence the new job, not sure if there is a better way to do it | |
| 07:44:40 | opendevreview | Merged openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 07:47:15 | gibi | good morning | |
| 07:51:31 | opendevreview | Merged openstack/nova master: Add conductor RPC interface for rebuild https://review.opendev.org/c/openstack/nova/+/831219 | |
| 07:53:15 | opendevreview | Merged openstack/nova master: Add API support for rebuilding BFV instances https://review.opendev.org/c/openstack/nova/+/830883 | |
| 07:58:53 | bauzas | good morning | |
| 08:16:52 | Uggla | gibi, bauzas good morning | |
| 08:17:18 | bauzas | Uggla: how was this morning ? :) | |
| 08:17:28 | bauzas | was Rose happy to see her friends ? | |
| 08:18:48 | Uggla | Yes, she's ok and happy because she has the teacher she wanted. | |
| 08:19:05 | gibi | :) | |
| 08:21:11 | Uggla | One little girl in her group was crying, and Rose gave her hand to go to the classroom. | |
| 08:21:47 | gibi | so nice | |
| 08:24:48 | bauzas | nice and bravo Rose | |
| 08:25:22 | bauzas | my oldest is still on vacations, she'll start tomorrow | |
| 08:25:37 | Uggla | in short no pb so far. I'll see this afternoon. | |
| 08:26:12 | bauzas | if you see me disappearing quickly, that's because I forgot about the time :p | |
| 08:32:06 | Uggla | 1st day on a Friday ? This is a warmup day for teachers ? :) | |
| 08:33:02 | bauzas | Uggla: no, for the "6e" chidren (10-11yo) they started today | |
| 08:33:20 | bauzas | for the older (12-14yo), they start tomorrow | |
| 08:33:55 | Uggla | yep strange to start on a Friday. (for your older one) | |
| 08:35:10 | bauzas | for the "6eme" folks, they only work today and not tomorrow | |
| 08:36:20 | bauzas | ("6eme" is first year of middle school) | |
| 08:46:43 | Uggla | ok so only 1 day the first week. Soft start. | |
| 09:19:26 | sean-k-mooney | its pretty common here | |
| 09:19:53 | sean-k-mooney | they give the students who are new to the school 1 day before the rest come back | |
| 09:20:06 | sean-k-mooney | so they can be shown around without all the other people | |
| 09:20:52 | sean-k-mooney | so normally they start on a wednesday here youngets class group, then first half then rest | |
| 09:21:13 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Add API for assisted volume extend https://review.opendev.org/c/openstack/nova-specs/+/855490 | |
| 09:21:24 | sean-k-mooney | melwitt: ^ | |
| 09:36:16 | sean-k-mooney | whoami-rajat: tempest has an alternitive image so we can have two test one that reimages to the same image and one the reimages to the alt image https://github.com/openstack/tempest/blob/master/tempest/config.py#L294-L297 | |
| 09:36:31 | sean-k-mooney | whoami-rajat: so you would boot with image_ref and rebuild to image_ref_alt | |
| 09:37:13 | sean-k-mooney | whoami-rajat: we really only care that the images are idfferent in glance they can both be cirros and its fine | |
| 09:38:03 | sean-k-mooney | with the old microversion you can jsut the requried alt image to assert its not allowed | |
| 09:38:03 | sean-k-mooney | the mecanics are the same if its the same image or differnt with the new microverion | |
| 09:38:09 | sean-k-mooney | and the same image to assert it preserved data | |
| 09:56:19 | whoami-rajat | sean-k-mooney, those parameters point to the same image in default tempest job, like see this tempest.conf from tempest-full run, both image_ref and image_ref_alt points to same image https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_83f/831018/24/check/tempest-full-py3/83f6b4a/controller/logs/tempest_conf.txt | |
| 09:56:37 | whoami-rajat | sean-k-mooney, I'm configuring different images for those parameters in my job, see tempest.conf for my new job https://a59fb54e72bca9b30065-6505ce46ad1aa88851df8d527471f9dc.ssl.cf1.rackcdn.com/831018/24/check/tempest-rebuild-volume-backed/c4755d7/controller/logs/tempest_conf.txt | |
| 09:57:13 | whoami-rajat | IMAGE_URLS parameter in devstack enables setting different images for those parameters | |
| 09:57:27 | whoami-rajat | by default only one image is created and set as both image_ref and image_ref_alt | |
| 09:58:31 | whoami-rajat | that's why i configured a new job with this parameter, wasn't sure if modifying existing jobs for my test run is a valid thing so created a new job | |
| 10:00:55 | gibi | stephenfin: hi! how do you feel, will you have time to review the PCI series up until https://review.opendev.org/c/openstack/nova/+/850468/20 before the FF deadline? | |
| 10:09:07 | sean-k-mooney | whoami-rajat: yes but it would be better to configure devstack to upload the cirrios image twich with differnt names and then use that | |
| 10:09:38 | sean-k-mooney | so i think yes modifying the exisitng josb is totally valid | |
| 10:10:06 | sean-k-mooney | and would get more coverage of the feature. that can be done in the futrue | |
| 10:10:20 | sean-k-mooney | whoami-rajat: also congrats its merged in nova this morning | |
| 10:10:26 | whoami-rajat | hmm, ack, will try that | |
| 10:10:29 | sean-k-mooney | whoami-rajat: sorry it took so long | |
| 10:10:54 | whoami-rajat | sean-k-mooney, thanks and thank you to all the nova team bauzas gibi dansmith for helping with that! | |
| 10:11:02 | whoami-rajat | no problem :) | |
| 10:11:14 | sean-k-mooney | did you see my comments on teh osc patch | |
| 10:11:22 | whoami-rajat | nope, looking | |
| 10:11:38 | sean-k-mooney | ack you duplicated a test but forgot to rename it in the copy paste | |
| 10:11:48 | sean-k-mooney | trivial fix | |