| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-11 | |||
| 08:24:08 | sean-k-mooney | ill start with just nova and see if i can get that to work | |
| 08:24:09 | sean-k-mooney | ill start with just nova and see if i can get that to work | |
| 08:27:02 | sean-k-mooney | only looking at failed jobs should keep the bandwith mangeable, the full ci generate maybe 4TB of logs a month which i cans store but my isp would not be happy with me downloadign that much | |
| 08:27:02 | sean-k-mooney | only looking at failed jobs should keep the bandwith mangeable, the full ci generate maybe 4TB of logs a month which i cans store but my isp would not be happy with me downloadign that much | |
| 08:33:26 | swp20 | stephenfin: i use admin context to show a project's vm by id and it's success, but by name it says no name or id exist. is this a bug? | |
| 08:33:26 | swp20 | stephenfin: i use admin context to show a project's vm by id and it's success, but by name it says no name or id exist. is this a bug? | |
| 08:40:29 | stephenfin | swp20: no, API resources are organized by ID. You need to list and manually search or use a specific search API. Look at the 'Resource.find' method in openstacksdk for an example | |
| 08:40:29 | stephenfin | swp20: no, API resources are organized by ID. You need to list and manually search or use a specific search API. Look at the 'Resource.find' method in openstacksdk for an example | |
| 08:50:29 | swp20 | stephenfin: yeah, i find the 'Resource.find' has no 'all_tenants' param, if i add the {'all_tenants':1} to kwargs, then i can get the vm by name. | |
| 08:50:29 | swp20 | stephenfin: yeah, i find the 'Resource.find' has no 'all_tenants' param, if i add the {'all_tenants':1} to kwargs, then i can get the vm by name. | |
| 08:54:36 | swp20 | so need we claim this when we exec 'nova help show' command, it's really a bit confused. | |
| 08:54:36 | swp20 | so need we claim this when we exec 'nova help show' command, it's really a bit confused. | |
| 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: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 | |