| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-16 | |||
| 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 | yonglihe | 'power-update' might take the 2.75 | |
| 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: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 | artom | efried, https://bugs.launchpad.net/nova/+bug/1836754 Should I also propose something for elastic-recheck? | |
| 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: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: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 16:34:41 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 16:34:42 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Add |
|
| 16:34:42 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 16:34:43 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 16:35:59 | efried | shilpasd: I'm going to edit the commit message of https://review.opendev.org/#/c/671075 real quick so I can be +2 on the whole series. | |
| 16:37:03 | openstackgerrit | Eric Fried proposed openstack/nova master: Add a new request filter to forbidden aggregates https://review.opendev.org/671075 | |
| 16:37:36 | openstackgerrit | Eric Fried proposed openstack/nova master: Enable request filter forbidden_aggregates https://review.opendev.org/667952 | |
| 16:38:20 | efried | dansmith: I'm +2 all the way up on the forbidden aggs series ^ | |
| 16:40:01 | efried | the biggish final patch is mostly doc | |
| 16:44:07 | openstackgerrit | Raubvogel proposed openstack/nova master: Author: Mauricio Tavares |
|
| 16:45:53 | openstackgerrit | Raubvogel proposed openstack/nova master: Author: Mauricio Tavares |
|
| 17:03:04 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 17:03:04 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 17:20:06 | efried | sean-k-mooney: you around? | |
| 17:20:49 | sean-k-mooney | kind of | |
| 17:20:54 | efried | I try and I try, but I just can't follow this stuff. Would you please review https://review.opendev.org/#/c/670588/ for me? I'll proxy your +1. | |
| 17:21:06 | sean-k-mooney | im wait on food to be delivered in 20 mins or so | |
| 17:21:18 | efried | I think it was your suggestion anyway from the subsequent patch | |
| 17:21:54 | sean-k-mooney | oh right. in the non ovs case we added a noop pluging to os vif to handel generic vhost-user plugging | |
| 17:22:07 | sean-k-mooney | and for the ovs case its already handeled by os-vif | |
| 17:23:09 | efried | yeah, see, I can read those words, but I've never been able to follow the code around that area | |
| 17:23:11 | sean-k-mooney | so those functions should never get called. but ill trace through the code and confim | |
| 17:23:16 | efried | thanks | |
| 17:24:27 | sean-k-mooney | well it used to do an indrect function call by appending the vif type to a prefix and using gettr to look up the function | |
| 17:24:39 | sean-k-mooney | so it was not exactly the simplest code | |
| 17:25:27 | sean-k-mooney | that actully predates me working on openstack but i never really understood why we did it that way | |