| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-16 | |||
| 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 | |
| 17:25:30 | efried | yeah, the getattr removal is in the subsequent patch, and I was able to follow that. But it's also dependent on this being right. | |
| 17:26:20 | sean-k-mooney | this code path is for netwroking-vpp or snabb switch | |
| 17:26:41 | sean-k-mooney | its very very rarly used but we still dont want to break people that use it | |
| 17:37:07 | openstackgerrit | Merged openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796 | |
| 17:53:27 | sean-k-mooney | efried: done. i have put a walk through of how the dead code would have been called in the past but it is never used after https://review.opendev.org/#/q/Ifab3006454708ab290b93f02d82b794c334c3946 was merged last cycle | |
| 17:53:44 | efried | thanks sean-k-mooney | |
| 17:54:31 | sean-k-mooney | no worries dinner just arrived so ill be away for a while. i might be back in an hour or two | |
| 18:01:06 | spatel | sean-k-mooney: yt? | |
| 18:01:49 | spatel | virsh list command showing instance-<uuid> is there anyway it can show hostname of instance | |
| 18:03:29 | spatel | In google people saying "virsh domhostname <domain>" but what is domain going to be? | |
| 18:03:51 | spatel | also i don't think kvm support that | |
| 18:20:18 | donnyd | Was there still interest in the NUMA for fortnebula? | |
| 18:20:49 | donnyd | LOL... interest in NUMA topologies | |
| 18:21:35 | donnyd | I have things to a more stable state with decent performance, so now I can move forward | |
| 18:25:31 | openstackgerrit | Mauricio Tavares proposed openstack/nova master: Increases the level of detail provided by the debug messages in pci_passthrough_filter.py. https://review.opendev.org/671106 | |
| 18:30:41 | efried | artom, sean-k-mooney: ---^ ? | |
| 18:39:58 | artom | donnyd, yes please, I would like 1 the NUMA ;) | |
| 18:40:24 | donnyd | LOL +2 | |
| 18:40:27 | artom | For serious though, yeah, flavors with >= 2 NUMA nodes would be awesome | |
| 18:40:34 | donnyd | you get 2 numas artom | |
| 18:41:26 | donnyd | I am not deep in the zuul or nodepool code, so how does the test work. Can you ask for a specific flavor? | |
| 18:41:53 | donnyd | I need a way to stay compatible with the other jobs | |
| 18:41:59 | artom | efried, I don't even know where to start with that. I guess look at the logic before drowning them with the formatting and whitespace nits? | |
| 18:42:24 | artom | donnyd, that's not my strong suite, I'm afraid | |
| 18:42:55 | efried | artom: I was pointing at donnyd, not at Mauricio's patch | |
| 18:43:06 | artom | efried, oh, hah | |