| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-16 | |||
| 13:52:05 | sean-k-mooney | i can create a version that will drop the unneeded jobs and just have teh two we need and propsoe the backpor if ye want | |
| 13:52:29 | artom | sean-k-mooney, I'd really appreciated if you handled covering all the test cases, yeah :) | |
| 13:53:31 | sean-k-mooney | so we need 3 or 4 test jobs | |
| 13:54:37 | artom | sean-k-mooney, 3, for cases A, B, D | |
| 13:54:46 | artom | A = neutron-tempest-iptables_hybrid | |
| 13:55:20 | artom | D = first half of https://review.opendev.org/#/c/664442/ (run revert tests in nova-live-migration) | |
| 13:55:26 | sean-k-mooney | b and d is the neutorn multinode job with and without iptables? | |
| 13:55:42 | artom | B = second half of https://review.opendev.org/#/c/664442/ (iptables in nova-live-migration/nova-multinode) | |
| 13:55:46 | sean-k-mooney | ok i can go do that now | |
| 13:56:24 | sean-k-mooney | *nova-multinode which was nova-live-migration | |
| 13:58:50 | mdbooth | Folks, I'm undecided on https://review.opendev.org/#/c/671023/ . It looks to me like a customer actually hit this race in practise while running a heat stack. However, in general I suspect it's fairly uncommon. My preference is generally to fix this stuff, but this kind of cleanup also tends to go ignored. Is a fix for this likely to be of interest? | |
| 14:00:27 | openstackgerrit | Merged openstack/nova master: Remove Rocky-era min compute trusted certs compat check https://review.opendev.org/669539 | |
| 14:01:05 | sean-k-mooney | assuming your runit test actully repoduced the race then it should be fairly staitforwad to show your patch is correct | |
| 14:01:37 | sean-k-mooney | so as long as the fix is not too invasive i dont see why this would be contovertial | |
| 14:01:45 | sean-k-mooney | but i dont know | |
| 14:03:10 | sean-k-mooney | its a race that is proably not hard to debug so if we dont fix it now it will proably bite us later | |
| 14:04:05 | mdbooth | sean-k-mooney: It will require a refactor of a method in libvirt.guest. Nobody likes refactors. | |
| 14:04:29 | sean-k-mooney | mdbooth: can you do it without changing the api signiture | |
| 14:04:42 | mdbooth | Of course, it's internal to the libvirt driver. | |
| 14:05:08 | sean-k-mooney | by api signiture i ment the signiture of the funtion | |
| 14:05:30 | mdbooth | Yes, probably, although the function signature does also need to be fixed. | |
| 14:06:51 | sean-k-mooney | as you said the guest stuff shoudl be internal to the libvirt driver so your e not modifying the virt direver api so it should be fine to backport | |
| 14:07:27 | artom | efried, hah, see where else that error popped up: http://logs.openstack.org/09/666409/8/check/tempest-full-py3/38bf84e/job-output.txt#_2019-07-14_17_19_10_677555 | |
| 14:07:42 | mdbooth | sean-k-mooney: That's fine. I'm more interested to know if anybody cares. If nobody cares I'm not going to fix it. | |
| 14:07:53 | yonglihe | efried: per the run queue, there is 2.75 ahead my patch, seems i better moving to 2.76 in advance? | |
| 14:07:56 | efried | artom: okay, so we should sort this out. | |
| 14:08:01 | mdbooth | sean-k-mooney: It's bad for my mental health ;) | |
| 14:08:08 | sean-k-mooney | mdbooth: well at least we have the bug and a repoducer unit test | |
| 14:08:19 | efried | yonglihe: You mean in the runway queue? | |
| 14:08:23 | sean-k-mooney | so if you dont fix it someone else can at a later date | |
| 14:08:33 | artom | efried, there aren't that many hits, but yeah, our theory from last night is pretty much confirmed | |
| 14:08:43 | yonglihe | efried, yeah. | |
| 14:09:19 | efried | yonglihe: Tough call. Just because it's ahead of you in the runway queue doesn't mean it'll get reviewed first. If you go to 2.76, you'll also need to stack yours on top of the 2.75 one, which is pretty weird since there's no actual dependency. | |
| 14:09:19 | yonglihe | 'power-update' might take the 2.75 | |
| 14:09:31 | efried | probably just best to ride it out. | |
| 14:09:47 | yonglihe | ok, let's see what happens then. | |
| 14:10:08 | artom | efried, in my experience it was basically a race to see which one lands first, and then the other person is stuck rebasing | |
| 14:10:10 | efried | artom: test_delete_server_while_in_building_state sounds like a test designed to expose this bug. | |
| 14:10:16 | mdbooth | artom: Is that holding up your revert resize change, btw? | |
| 14:10:18 | artom | efried, for serious | |
| 14:10:23 | artom | mdbooth, what is? | |
| 14:10:34 | mdbooth | artom: I don't know, that's what I'm asking. | |
| 14:10:36 | sean-k-mooney | mdbooth: the revert resize change has merged | |
| 14:10:45 | yonglihe | btw, the corresponding novaclent patch realse released: https://review.opendev.org/#/c/670790/ | |
| 14:10:46 | mdbooth | sean-k-mooney artom: \o/ | |
| 14:10:47 | artom | mdbooth, no, this is unrelated, something that efried noticed last night | |
| 14:11:06 | sean-k-mooney | mdbooth: on master. im working on creating a DNM patch to test stable | |
| 14:11:18 | artom | sean-k-mooney, patch_ES_ :) | |
| 14:11:26 | sean-k-mooney | artom has the backport alreay up | |
| 14:11:42 | sean-k-mooney | artom: im going to do 1 patch with 3 jobs | |
| 14:12:05 | mdbooth | [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate is not yet valid | |
| 14:12:21 | sean-k-mooney | its a dnm patch so i have no issue with copy pasting the job definiton and just not seting iptabels in one of them | |
| 14:12:22 | mdbooth | I bet I'm the only person to see that regularly :) | |
| 14:13:32 | sean-k-mooney | mdbooth: are you doing multi node but have not syncted your clock on each of the hosts | |
| 14:14:03 | openstack | Launchpad bug 1836754 in OpenStack Compute (nova) "Conflict when deleting allocations for an instance that hasn't finished building" [Undecided,New] | |
| 14:14:03 | artom | efried, https://bugs.launchpad.net/nova/+bug/1836754 Should I also propose something for elastic-recheck? | |
| 14:14:05 | mdbooth | sean-k-mooney: Nah, I'm using Hyper-V, which stops the clock while my laptop is suspended. It was still early last week. | |
| 14:14:17 | artom | sean-k-mooney, ack, that works too | |
| 14:14:32 | mdbooth | The guest tools are *supposed* to auto sync the time from the HV, but I've never made that work. | |
| 14:14:37 | sean-k-mooney | mdbooth: right but do you not have ntpd installed to fix that | |
| 14:15:23 | mdbooth | Yeah, I have to fix it with chronyd manually, which is a hack. It has to be manual because it won't auto-fix a jump that large. | |
| 14:16:57 | sean-k-mooney | ah i see. i have only ever used hyperv on a desktop or server so never had that issue | |
| 14:17:59 | sean-k-mooney | that reminds me i need to order an ssd so i dont need to keep dual booting my workstation | |
| 14:20:41 | mdbooth | sean-k-mooney: For an SSD? That seems like a weird distinction, but I'm out of touch with such things. | |
| 14:21:46 | sean-k-mooney | no i was wondering if my dell t7610 is technically a server or a workstation | |
| 14:22:32 | sean-k-mooney | the ssd boot drive died so im using the one out of my windows gaming pc as a stop gap but that means i have to dual boot it currently | |
| 14:22:59 | sean-k-mooney | im planning to get a seccond ssd and install linux then pass the windows ssd into a vm | |
| 14:23:02 | artom | ... do I need something special to propose patches to elastic-recheck? Gerrit's refusing me | |
| 14:23:16 | efried | artom: elastic-recheck would be nice, though I'm not sure who approves those when mriedem isn't around. | |
| 14:23:27 | sean-k-mooney | artom: no you shouldnt | |
| 14:23:33 | sean-k-mooney | artom: whats the error? | |
| 14:24:28 | artom | oh, no one had updated .gitreview to point to opendev/ instead of openstack-infra | |
| 14:24:44 | sean-k-mooney | ah ya that would break it | |
| 14:24:55 | sean-k-mooney | i think the redirects dont work proably in all cases | |
| 14:29:32 | artom | Oh, I think it's because I cloned from github and no one's re-setup the mirroring | |
| 14:30:02 | sean-k-mooney | i dont think the infra projects are going to be mirror going forward | |
| 14:33:50 | efried | stephenfin: you around to give https://review.opendev.org/#/c/657796/ a bump? | |
| 14:34:45 | stephenfin | Looking | |
| 14:36:03 | stephenfin | and done. That was nice and easy | |
| 14:36:43 | gibi | efried, stephenfin: thanks | |
| 14:37:17 | efried | gibi: I only regret I can't send "heal port allocations" to the gate. | |
| 14:37:26 | efried | half a dozen +As stacked up behind that one. | |
| 14:38:05 | gibi | efried: no worries. we will wait for Matt to to +A that after he is back | |
| 14:38:33 | gibi | meanwhile I can finally refocus on the server move operations + bandwidth series | |
| 15:01:56 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Nova object changes for forbidden aggregates request filter https://review.opendev.org/671072 | |
| 15:02:38 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074 | |
| 15:03:19 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Add a new request filter to forbidden aggregates https://review.opendev.org/671075 | |
| 15:03:58 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Enable request filter forbidden_aggregates https://review.opendev.org/667952 | |
| 15:22:23 | openstackgerrit | sean mooney proposed openstack/nova stable/stein: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/671079 | |
| 15:22:35 | artom | That was fast | |
| 15:23:28 | sean-k-mooney | it took longer then i planned but i think ^ is right | |
| 15:25:02 | sean-k-mooney | i guess not | |
| 15:26:15 | sean-k-mooney | oh i think im missing a new line | |
| 15:27:22 | sean-k-mooney | and a space | |
| 15:27:52 | openstackgerrit | sean mooney proposed openstack/nova stable/stein: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/671079 | |
| 15:34:48 | sean-k-mooney | ok zull accpeted that so it should report back eventually | |
| 16:34:19 | efried | shilpasd: still around? | |
| 16:34:40 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | |
| 16:34:41 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 16:34:41 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |