| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-25 | |||
| 16:05:14 | gibi | I'm torn, I would like to help but my priorities are elswhere | |
| 16:05:40 | gibi | I'm looking at the actual detach failure now to at least confirm it is the same issue | |
| 16:23:51 | gibi | updated https://review.opendev.org/c/openstack/tempest/+/842240/ , it is a long shot as I'm not really know what I am doing in tempest :) | |
| 16:24:08 | gibi | if it greens out then I will push a test nova patch depending on it | |
| 18:45:25 | gmann | gibi: hi, sorry I also missed to track that. let me check it today | |
| 19:03:59 | gmann | bauzas: i have created the LP project for placement and assigned maintainer to 'nova-drivers' https://launchpad.net/placement | |
| 19:05:12 | gmann | bauzas: as I am not part of nova-drivers group in LP, i cannot re-assign the 'Driver' from me to 'nova-drivers' or add it as part of 'OpenStack' | |
| 19:09:49 | gmann | or if either of you bauzas gibi dansmith melwitt can change back the Maintainer to me then I can finish the things there and assigned back to 'nova-drivers' | |
| 19:10:11 | gmann | did not that after changing maintainer to nova-driver it would not allow me to do the things | |
| 19:10:25 | gmann | that could have been last step | |
| 19:20:44 | gmann | * did not know that | |
| #openstack-nova - 2022-11-26 | |||
| 02:32:37 | opendevreview | Ghanshyam proposed openstack/nova master: DNM:testing lvm job https://review.opendev.org/c/openstack/nova/+/865765 | |
| #openstack-nova - 2022-11-27 | |||
| 15:25:58 | opendevreview | Andrei Yachmenev proposed openstack/nova-specs master: Add audio device to libvirt XML by nova libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/865634 | |
| 19:16:43 | opendevreview | Rafael Weingartner proposed openstack/nova master: Nova to honor "cross_az_attach" during server(VM) migrations https://review.opendev.org/c/openstack/nova/+/864760 | |
| #openstack-nova - 2022-11-28 | |||
| 09:46:20 | bauzas | gmann: thanks for having incepted the placement LP resource https://launchpad.net/placement | |
| 09:46:38 | bauzas | gmann: since I don't know exactly what's left, I just set the maintainer back to you, sure | |
| 09:46:47 | bauzas | gmann: ping me once you're done | |
| 10:14:49 | darkhorse | Hi team, In the compute_nodes table, there are 'host' column and 'hypervisor_hostname' column, which looks the same. Will you explain the difference between these fields? | |
| 10:24:38 | gibi | bauzas: so https://review.opendev.org/c/openstack/tempest/+/842240 is not green yet, but gmann pushed a test patch for nova-lvm with it https://review.opendev.org/c/openstack/nova/+/865765 it is also red but with one less failed test case (test_attach_detach_volume is green there now) | |
| 10:25:40 | bauzas | ack | |
| 10:26:26 | bauzas | darkhorse: 'host' is about the name of the nova-compute service | |
| 10:26:36 | bauzas | which is opted in by the conf option | |
| 10:27:00 | bauzas | darkhorse: while "hypervisor_hostname" is provided by the virt driver | |
| 10:30:42 | darkhorse | bauzas: thank you for clarification! | |
| 10:30:50 | bauzas | np | |
| 10:32:06 | bauzas | gibi: sean-k-mooney: could you please review my spec https://review.opendev.org/c/openstack/nova-specs/+/861591 before 3pm UTC today ? :) | |
| 10:34:20 | opendevreview | Sylvain Bauza proposed openstack/nova master: Skip live_migration_with_trunk tempest test on ovs-hybrid and nova-next https://review.opendev.org/c/openstack/nova/+/865658 | |
| 10:35:32 | bauzas | gibi: sean-k-mooney: gmann: ^ fwiw, added depends-on gmann's tempest patch on ^ to eventually get it accepted | |
| 10:42:12 | gibi | bauzas: lets wait for gmann to get up and discuss what are our options about the gate regarding the detach issue. If gmann sees a possible fix then I would wait for it, but if he doesn't then I cannot offer any better way forward now than disabling the tests given the amount of time I can spend on this. | |
| 10:43:17 | bauzas | gibi: I need to catch my daughter at school but let's revisit this once gmann's up | |
| 10:43:24 | gibi | yepp | |
| 10:43:55 | gibi | bauzas: I'm +2 on https://review.opendev.org/c/openstack/nova-specs/+/861591 | |
| 10:49:25 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 11:11:41 | bauzas | gibi: I very briefly looked at the failing tests and yeah, we need to continue working on it if we want to merge it | |
| 11:11:53 | bauzas | I can try to help but I don't have a lot of context about it | |
| 11:12:17 | bauzas | also, I wonder *why* we now have a larger number of issues with nova-lvm | |
| 11:12:46 | bauzas | anyway, /me goes afk for lunch | |
| 11:13:59 | kashyap | bauzas: I think not many (if any at all) production users use LVM | |
| 11:14:06 | kashyap | (With Nova, I mean) | |
| 11:14:55 | sean-k-mooney | it used to be popular for thost that need the better write prefromance | |
| 11:15:17 | sean-k-mooney | but its much less common then it used to be | |
| 11:15:42 | kashyap | Yeah | |
| 11:17:12 | sean-k-mooney | i previously used it for my home cloud in the past. i never really had issues with it. im currently using cinder lvm with bfv instead | |
| 11:19:14 | gibi | bauzas: when we change dependency version, ie changing to the next ubuntu image, the timings change and that teases out new races | |
| 11:19:35 | gibi | it think the jammy change triggered different timings in our tests showing more this issue | |
| 11:27:09 | opendevreview | Merged openstack/nova-specs master: Proposes cpu power managment in libvirt https://review.opendev.org/c/openstack/nova-specs/+/861591 | |
| 12:38:10 | opendevreview | Merged openstack/nova-specs master: Re-propose per process healthchecks https://review.opendev.org/c/openstack/nova-specs/+/864150 | |
| 12:53:21 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/864006 | |
| 12:53:22 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864007 | |
| 13:06:29 | bauzas | fwiw, I got f***d by Zuul b/c of nova-lvm again even if Depends-on on the Tempest patch https://review.opendev.org/c/openstack/nova/+/865658 | |
| 13:07:02 | sean-k-mooney | well the tempest change was still failing | |
| 13:07:07 | bauzas | yup | |
| 13:07:11 | bauzas | but the issue is still present | |
| 13:07:19 | sean-k-mooney | right becaues qemu is broken | |
| 13:07:36 | bauzas | other jobs are failing to due the flakey tempest test I agree | |
| 13:08:06 | sean-k-mooney | so we can either disabel the volume tests or we can make it non voting but we cant really do anythign else in onva | |
| 13:08:17 | bauzas | but for nova-lvm, we get the same exception than previously without depending on the tempest changhe | |
| 13:08:18 | sean-k-mooney | the nova-lvm job is intended to test the lvm image backend | |
| 13:08:25 | bauzas | yeah I know | |
| 13:08:27 | sean-k-mooney | so we dont striclty need to test cinder there | |
| 13:09:12 | bauzas | I don't disagree | |
| 13:10:27 | bauzas | but the failure is also for the rescue tests | |
| 13:10:48 | bauzas | in tearDownClass fixture ending | |
| 13:11:14 | gokhani | hello folks, some of my instances image backing file is deleted unexpectedly. And unfortunately glance image is also deleted. I only see a disk file under /var/lib/nova/instances. is there any way rescue this instances ? I tried using guestfish but it also tries to reach backing file. | |
| 13:12:34 | gibi | bauzas, sean-k-mooney: lets sync with gmann, but sadly I have to agree to disable those tests temprorarily | |
| 13:12:58 | bauzas | gibi: tbh, I don't know which exact cables to unplug | |
| 13:13:11 | bauzas | I mean, I know the cables | |
| 13:13:38 | bauzas | but I wonder if by unplugging them, it would either shutdown the Big Thing or even just Do Nothing | |
| 13:14:25 | sean-k-mooney | you woudl have to disabel all volume related tests more or less | |
| 13:14:51 | gibi | sean-k-mooney: no we don't have to, I think some of those has the wait for sshable thing implemented | |
| 13:14:55 | sean-k-mooney | there is one other tempest serise that might be related | |
| 13:15:05 | sean-k-mooney | gibi: it failed with that | |
| 13:15:30 | sean-k-mooney | i mean the current patch might not work | |
| 13:15:34 | sean-k-mooney | and making it work might be enough | |
| 13:15:36 | gibi | sean-k-mooney: it failed with a tempest patch that adds some sshable flag to some tests. But we already merged a lot of sshable flag patches | |
| 13:15:40 | sean-k-mooney | but the current patch still failed | |
| 13:16:06 | sean-k-mooney | oh i know but bauzas added a depend-on ot the last one in that seriese | |
| 13:16:21 | bauzas | yup | |
| 13:16:23 | sean-k-mooney | so we seam to need somethign else on top of that | |
| 13:16:28 | gibi | yes, taht depends on was for adding sshable for hopefully the last set of volume tests | |
| 13:16:35 | bauzas | yup | |
| 13:16:42 | gibi | that tempest test is not corret yet | |
| 13:16:55 | gibi | but gmann hopefully knows more about it | |
| 13:17:15 | gibi | so I think we need to disable those volume tests that are failing currently, not all volume tests | |
| 13:17:15 | bauzas | I'm just wrapping up all the LP bug reports | |
| 13:17:18 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/840410 | |
| 13:17:26 | sean-k-mooney | this one might also be interesting ^ | |
| 13:17:42 | gibi | yes, we need that too to disable trunk | |
| 13:17:49 | gibi | there we know ralonsoh working on a fix | |
| 13:18:11 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/840410 fixes a issue witht he cleanup function ordering | |
| 13:18:17 | sean-k-mooney | related to volumes | |
| 13:18:57 | gibi | sorry i mixed up things | |
| 13:19:03 | gibi | https://review.opendev.org/c/openstack/tempest/+/840410 is new to me | |
| 13:19:06 | sean-k-mooney | bauzas: could you perhaps also add a depend on for that | |
| 13:19:46 | gibi | probably we need a single nova patch that disables trunk (as in https://review.opendev.org/c/openstack/nova/+/865658) and disable the failing volume tests | |
| 13:20:15 | sean-k-mooney | ya that would be simpler to merge | |
| 13:20:24 | bauzas | sean-k-mooney: sec, writing an email to inform the world we're busted | |
| 13:28:37 | bauzas | I'm here | |
| 13:28:40 | bauzas | so | |
| 13:29:19 | sean-k-mooney | joing an internal call shortly but https://review.opendev.org/c/openstack/tempest/+/840410 may help with the volume issues | |