| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-11 | |||
| 09:34:23 | openstackgerrit | likui proposed openstack/osc-placement stable/ussuri: Update command help information https://review.opendev.org/c/openstack/osc-placement/+/790618 | |
| 09:41:17 | openstackgerrit | likui proposed openstack/osc-placement master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/osc-placement/+/790630 | |
| 09:41:17 | openstackgerrit | likui proposed openstack/osc-placement master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/osc-placement/+/790630 | |
| 09:57:35 | openstackgerrit | David Vallee Delisle proposed openstack/os-vif master: Creating oslo.config.opts entry_points for plugins https://review.opendev.org/c/openstack/os-vif/+/789645 | |
| 09:57:35 | openstackgerrit | David Vallee Delisle proposed openstack/os-vif master: Creating oslo.config.opts entry_points for plugins https://review.opendev.org/c/openstack/os-vif/+/789645 | |
| 10:21:38 | stephenfin | lyarwood: Could you stick this on your review queue for the week, if you've time? https://review.opendev.org/c/openstack/nova/+/676209 | |
| 10:21:38 | stephenfin | lyarwood: Could you stick this on your review queue for the week, if you've time? https://review.opendev.org/c/openstack/nova/+/676209 | |
| 10:22:55 | stephenfin | bauzas: Any chance you could keep going through the instance_type -> flavor series? If any of them are too difficult to review, I can try break them up more https://review.opendev.org/q/topic:%2522compute_rpc_6.0%2522+status:open | |
| 10:22:55 | stephenfin | bauzas: Any chance you could keep going through the instance_type -> flavor series? If any of them are too difficult to review, I can try break them up more https://review.opendev.org/q/topic:%2522compute_rpc_6.0%2522+status:open | |
| 10:23:18 | bauzas | sean-k-mooney: any reason why you try to plug vifs *after* we try to reboot the instance or restart it ? https://github.com/openstack/nova/blob/7953c0197d1a4466cb5b78070d47626c92f9db6e/nova/compute/manager.py#L1137 | |
| 10:23:18 | bauzas | sean-k-mooney: any reason why you try to plug vifs *after* we try to reboot the instance or restart it ? https://github.com/openstack/nova/blob/7953c0197d1a4466cb5b78070d47626c92f9db6e/nova/compute/manager.py#L1137 | |
| 10:23:29 | bauzas | stephenfin: yup, I started to look at those | |
| 10:23:29 | bauzas | stephenfin: yup, I started to look at those | |
| 10:23:51 | bauzas | stephenfin: today, I'll do some reviews, specs first but I can try to look at your series too | |
| 10:23:51 | bauzas | stephenfin: today, I'll do some reviews, specs first but I can try to look at your series too | |
| 10:24:04 | stephenfin | good enough for me, ty :) | |
| 10:24:04 | stephenfin | good enough for me, ty :) | |
| 10:24:14 | bauzas | stephenfin: btw. I'm off tonight until next week | |
| 10:24:14 | bauzas | stephenfin: btw. I'm off tonight until next week | |
| 10:24:29 | stephenfin | okay, gtk | |
| 10:24:29 | stephenfin | okay, gtk | |
| 10:24:33 | bauzas | (perpetual PTOs as mriedem said, that's it) | |
| 10:24:33 | bauzas | (perpetual PTOs as mriedem said, that's it) | |
| 10:24:43 | bauzas | or... just May | |
| 10:24:43 | bauzas | or... just May | |
| 10:30:56 | sean-k-mooney | bauzas: we plug before we start the instance | |
| 10:30:56 | sean-k-mooney | bauzas: we plug before we start the instance | |
| 10:31:32 | sean-k-mooney | althopugh i guess its after in some cases | |
| 10:31:32 | sean-k-mooney | althopugh i guess its after in some cases | |
| 10:31:50 | sean-k-mooney | hum | |
| 10:31:50 | sean-k-mooney | hum | |
| 10:31:51 | bauzas | sean-k-mooney: look at the above | |
| 10:31:51 | bauzas | sean-k-mooney: look at the above | |
| 10:31:57 | sean-k-mooney | ya i am | |
| 10:31:57 | sean-k-mooney | ya i am | |
| 10:31:59 | bauzas | sean-k-mooney: we try to reboot the instance first | |
| 10:31:59 | bauzas | sean-k-mooney: we try to reboot the instance first | |
| 10:32:09 | bauzas | and then we also try to restart it | |
| 10:32:09 | bauzas | and then we also try to restart it | |
| 10:32:19 | bauzas | *before we plug the vifs* | |
| 10:32:19 | bauzas | *before we plug the vifs* | |
| 10:32:27 | sean-k-mooney | we might be trying to plug wtice but i dont see any reason not to plug first | |
| 10:32:27 | sean-k-mooney | we might be trying to plug wtice but i dont see any reason not to plug first | |
| 10:32:32 | sean-k-mooney | that would be more correct i think | |
| 10:32:32 | sean-k-mooney | that would be more correct i think | |
| 10:32:55 | bauzas | ok, I'm asking this because I'd prefer to pass the allocations to the new driver method I created *before* the two above calls | |
| 10:32:55 | bauzas | ok, I'm asking this because I'd prefer to pass the allocations to the new driver method I created *before* the two above calls | |
| 10:33:23 | sean-k-mooney | ya i also think that proably makes sense | |
| 10:33:23 | sean-k-mooney | ya i also think that proably makes sense | |
| 10:33:29 | bauzas | if you reboot the instance with not the mdevs, then you got a qemu exception | |
| 10:33:29 | bauzas | if you reboot the instance with not the mdevs, then you got a qemu exception | |
| 10:33:50 | sean-k-mooney | yep so it should be create mdevs then plug vifs then reboot | |
| 10:33:51 | sean-k-mooney | yep so it should be create mdevs then plug vifs then reboot | |
| 10:33:53 | bauzas | * bauzas also needs to find a vGPU env, btw. :) | |
| 10:35:14 | sean-k-mooney | maybe see if you can borrow one form james if they are not using it for qe tests | |
| 10:35:14 | sean-k-mooney | maybe see if you can borrow one form james if they are not using it for qe tests | |
| 10:36:16 | sean-k-mooney | bauzas: so ya just checked the code | |
| 10:36:16 | sean-k-mooney | bauzas: so ya just checked the code | |
| 10:36:30 | sean-k-mooney | that is litgimatly a bug that plug vifs is after the vm start | |
| 10:36:30 | sean-k-mooney | that is litgimatly a bug that plug vifs is after the vm start | |
| 10:36:52 | sean-k-mooney | the reboot code path is assumign its done before | |
| 10:36:52 | sean-k-mooney | the reboot code path is assumign its done before | |
| 10:37:10 | bauzas | kk | |
| 10:37:10 | bauzas | kk | |
| 10:37:24 | sean-k-mooney | it wont break things entirly in that we will fix it after but we should move it earlier | |
| 10:37:24 | sean-k-mooney | it wont break things entirly in that we will fix it after but we should move it earlier | |
| 10:37:25 | bauzas | ok, so I'll write the code | |
| 10:37:25 | bauzas | ok, so I'll write the code | |
| 10:37:41 | bauzas | sean-k-mooney: do you want me to write the bug report for the vifs one ? | |
| 10:37:41 | bauzas | sean-k-mooney: do you want me to write the bug report for the vifs one ? | |
| 10:37:43 | bauzas | or you ? | |
| 10:37:43 | bauzas | or you ? | |
| 10:38:04 | sean-k-mooney | hehe i mean if you feel like it go for it | |
| 10:38:04 | sean-k-mooney | hehe i mean if you feel like it go for it | |
| 10:38:17 | sean-k-mooney | you can point to https://github.com/openstack/nova/blob/7953c0197d1a4466cb5b78070d47626c92f9db6e/nova/virt/libvirt/driver.py#L3773-L3775 | |
| 10:38:17 | sean-k-mooney | you can point to https://github.com/openstack/nova/blob/7953c0197d1a4466cb5b78070d47626c92f9db6e/nova/virt/libvirt/driver.py#L3773-L3775 | |
| 10:38:29 | sean-k-mooney | as evidnce that we expect ti to be alredy plugged | |
| 10:38:29 | sean-k-mooney | as evidnce that we expect ti to be alredy plugged | |
| 10:38:31 | bauzas | ok, gotta go lunching but I'll write it later | |
| 10:38:31 | bauzas | ok, gotta go lunching but I'll write it later | |
| 10:38:49 | bauzas | * bauzas already has a shit ton of open tabs | |
| 10:39:07 | bauzas | hopefully, my new laptop would arrive... | |
| 10:39:07 | bauzas | hopefully, my new laptop would arrive... | |
| 10:39:29 | sean-k-mooney | did you get a ship date or just hoping its on its way | |
| 10:39:29 | sean-k-mooney | did you get a ship date or just hoping its on its way | |
| 10:39:37 | bauzas | I asked | |
| 10:39:37 | bauzas | I asked | |
| 10:39:58 | bauzas | "I just checked with our vendor and the laptop is currently in the status 'Allocated', which means that it should be sent soon (in a matter of days)." | |
| 10:39:58 | bauzas | "I just checked with our vendor and the laptop is currently in the status 'Allocated', which means that it should be sent soon (in a matter of days)." | |
| 10:40:20 | bauzas | (I know we are in an upstream channel, but that's not a secret phrase ;) ) | |
| 10:40:20 | bauzas | (I know we are in an upstream channel, but that's not a secret phrase ;) ) | |
| 10:40:44 | bauzas | this was 7 days ago, so I'm not against a delivery surprise | |
| 10:40:44 | bauzas | this was 7 days ago, so I'm not against a delivery surprise | |
| 10:41:28 | bauzas | context : CPU shortages on vendors, leaving my refreshment order open for 6 months, craziness of the post-covid economy | |
| 10:41:28 | bauzas | context : CPU shortages on vendors, leaving my refreshment order open for 6 months, craziness of the post-covid economy | |
| 10:42:13 | bauzas | (also the fact that most of the vendors are fabless...) | |
| 10:42:13 | bauzas | (also the fact that most of the vendors are fabless...) | |
| 10:45:47 | sean-k-mooney | well hopefully you will get it sooner then your broadband upgrade | |
| 10:45:47 | sean-k-mooney | well hopefully you will get it sooner then your broadband upgrade | |
| 11:06:27 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP compute: Persist image id within the system_metadata of an instance https://review.opendev.org/c/openstack/nova/+/790659 | |