| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-25 | |||
| 14:44:12 | kashyap | - Antagonized-by: | |
| 14:44:12 | kashyap | - Bitten-by-and-tested-by: | |
| 14:44:38 | dansmith | yup, nice | |
| 14:45:11 | artom|gmtplus3 | Drive-by: Shoo Ting | |
| 14:45:31 | kashyap | mnaser: Live migrating level-1 guest (level-2 can be migrated fine) is the remaining gnarly problem that is being worked on (both in KVM and QEMU) | |
| 14:46:22 | sean-k-mooney | so Most-Definitely-Acked-by: | |
| 14:46:24 | sean-k-mooney | Mostly-acked-by: | |
| 14:46:26 | sean-k-mooney | NACKed-by: | |
| 14:46:33 | sean-k-mooney | are used as +2 +1 and -1 | |
| 14:46:46 | sean-k-mooney | also i like More-or-less-tested-by | |
| 14:47:35 | mriedem | kashyap: what are the chances GCE has forked everything to make nested KVM work for just their cloud? | |
| 14:48:14 | dansmith | *gasp* | |
| 14:48:19 | dansmith | they would NEVER do such a thing, surely | |
| 14:49:14 | kashyap | mriedem: The chances are 100% sunny :D I see what you mean, they do have their kernel; that said ... they contribute most KVM-related stuff back to the upstream, and post results of security audit, too | |
| 14:50:52 | sean-k-mooney | mriedem: from my past experience with kvm it works beeter if you use a vanial upstream kernel then a disto packaged one although that has change in the last 12-18 months or so as i think distors have got better at testing nested virt or just break it less | |
| 14:52:43 | mriedem | hmm, me checks notes...sean says things from a distro are better since about the time he started working for a distro...yup it checks out | |
| 14:53:11 | sean-k-mooney | goes back to his ubunut based laptop... | |
| 14:53:20 | dansmith | mmm, ubunut | |
| 14:53:47 | dansmith | that has to be about the best ubuntu type I've seen | |
| 14:53:54 | sean-k-mooney | my devstack vms are centos | |
| 14:53:55 | dansmith | *typo, hah | |
| 14:54:32 | mriedem | did you know a cashew is not a nut? | |
| 14:54:36 | dansmith | artom|gmtplus3: like a fancy brazilian walnut laptop, or the *really* fancy "ubunut" laptop | |
| 14:54:48 | sean-k-mooney | :) | |
| 14:55:50 | sean-k-mooney | mriedem: that was most based on the fact that centos finally started to wrok with nested vert although canonica broke it in 18.04 which made me sad | |
| 14:55:51 | mriedem | lyarwood: just so you're aware, stable/stein and rocky are essentially blocked until these devstack changes land https://review.opendev.org/#/q/I0d7f9f2baaee2836a719f199939156bd4f53f778 | |
| 14:56:24 | artom|gmtplus3 | Oh, not the whole laptop, just the CPU: Hazwellnut architecture :D | |
| 14:56:58 | sean-k-mooney | haha i would totally by that | |
| 14:57:44 | lyarwood | mriedem: was that behind tempest-slow etc timing out due to the network tests taking ages? | |
| 14:58:35 | lyarwood | ah nvm it doesn't look like it is | |
| 15:00:28 | mriedem | the agent on the tempest-slow jobs blows up | |
| 15:01:19 | lyarwood | ah I see | |
| 15:02:46 | lyarwood | well thanks for the heads up, I've had a tab open to look at that all day, I'll get on with some reviews once that has started to land in stable. | |
| 15:04:59 | sean-k-mooney | dansmith: im always amazed by how many grammer and spelling issue my brain just completely ignores | |
| 15:05:46 | dansmith | sean-k-mooney: and they make it really difficult for me to concentrate on the content.. diff'rent strokes. | |
| 15:07:28 | sean-k-mooney | ya, it just show how differently we read. not that you have pointed them out they are obvious but without your commets my brain just fills in the gaps/corrects them on the fly unless i force myself to read word by word really slowly | |
| 15:08:01 | artom|gmtplus3 | Yeah, there's a "severity" boundary where it switches from "distracting but harmless" to "active sabotage" | |
| 15:10:14 | jangutter | artom|gmtplus3: the late, great Iain M Banks had one of the most frustratingly good stories that tripped that balance in "Feersum Endjinn" | |
| 15:10:43 | mriedem | yay everything i had in the gate last night failed with RETRY_LIMIT and now i get to recheck everything | |
| 15:10:53 | artom|gmtplus3 | jangutter, will read | |
| 15:12:16 | sean-k-mooney | jangutter: im kind of tempted to read that | |
| 15:14:45 | jangutter | I think he's much more known for the Culture work, but FE was a good diversion. Just be warned that some chapters are physically hard to read, intentionally. | |
| 15:15:14 | sean-k-mooney | i have no issue reading "Woak up. Got dresd. Had brekfast. Spoke wif Ergates thi ant who sed itz juss been wurk wurk wurk 4 u lately master Bascule, Y dont u ½ a holiday? & I agreed & that woz how we decided we otter go 2 c Mr Zoliparia in thi I-ball ov thi gargoyle Rosbrith. " | |
| 15:19:32 | efried | mriedem: would you please give this a quick look for sanity https://review.opendev.org/655985 before I invest in test? | |
| 15:23:09 | artom|gmtplus3 | sean-k-mooney, your brain is weird, I can barely parse that | |
| 15:29:08 | sean-k-mooney | artom|gmtplus3: "Woke up. Got dressed. Had breakfast. spoke with Ergates the ant, who said, it's just been work work work for you lately master Bascule, why dont you have a holiday? And i aggred, an that was how we decied we ought to go to see Mr Zoliparia in the eye ball of the gargoyle Rosbrith." :) | |
| 15:29:49 | artom|gmtplus3 | sean-k-mooney, aha, thi = the didn't register at all | |
| 15:29:59 | artom|gmtplus3 | I was thinking "thy", the old English possessive | |
| 15:30:39 | sean-k-mooney | o/ | |
| 15:34:36 | mriedem | efried: comments inline | |
| 15:42:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: (Re-)enable vnc console tests in nova-multi-cell job https://review.opendev.org/667388 | |
| 15:51:11 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/stein: libvirt: Rework 'EBUSY' (SIGKILL) error handling code path https://review.opendev.org/667389 | |
| 16:09:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667396 | |
| 16:09:30 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667397 | |
| 16:09:30 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857 | |
| 16:09:30 | openstack | bug 1833581 in OpenStack Compute (nova) "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 16:11:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552 | |
| 16:11:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add functional test coverage for bug 1724172 https://review.opendev.org/512553 | |
| 16:11:25 | openstack | bug 1724172 in OpenStack Compute (nova) rocky "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,Confirmed] https://launchpad.net/bugs/1724172 | |
| 16:12:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667396 | |
| 16:12:07 | openstack | bug 1833581 in OpenStack Compute (nova) "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 16:13:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165 | |
| 16:13:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164 | |
| 16:14:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857 | |
| 16:16:31 | stephenfin | dansmith: Want to take another look through this? https://review.opendev.org/#/c/666629/ | |
| 16:29:53 | dansmith | stephenfin: will in a bit | |
| 16:30:00 | stephenfin | no rush | |
| 16:31:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Privsepify ipv4 forwarding enablement. https://review.opendev.org/635431 | |
| 16:31:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove no longer required "inner" methods. https://review.opendev.org/655282 | |
| 16:31:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Privsep the ebtables modification code. https://review.opendev.org/635435 | |
| 16:31:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove unused FP device creation and deletion methods. https://review.opendev.org/635433 | |
| 16:31:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.opendev.org/636508 | |
| 16:31:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move adding vlans to interfaces to privsep. https://review.opendev.org/635436 | |
| 16:31:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.opendev.org/639281 | |
| 16:31:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.opendev.org/639280 | |
| 16:31:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.opendev.org/639282 | |
| 16:31:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.opendev.org/639581 | |
| 16:31:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move final bridge commands to privsep. https://review.opendev.org/639580 | |
| 16:31:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move setting of device trust to privsep. https://review.opendev.org/639283 | |
| 16:31:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.opendev.org/639826 | |
| 16:31:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: We no longer need rootwrap. https://review.opendev.org/554438 | |
| 16:33:21 | efried | thanks mriedem | |
| 16:41:35 | mriedem | efried: how do you feel about having a mini personal goal of making https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L985-L997 obsolete for in-tree drivers so we could deprecate or remove it in Train - out of tree drivers could be hit so if we wanted to be nice we'd just log a deprecation warning in T and remove in U | |
| 16:41:47 | mriedem | the only in-tree drivers that would hit those paths are xenapi and hyperv, | |
| 16:41:53 | mriedem | the former implements get_inventory, | |
| 16:41:59 | mriedem | the latter implements neither get_inventory or upt | |
| 16:43:27 | mriedem | oh looky here the hyperv out of tree driver has upt implemented https://opendev.org/openstack/compute-hyperv/src/branch/master/compute_hyperv/nova/driver.py#L464 | |
| 16:44:49 | efried | mriedem: I'm big in favor of this. At some point I looked into doing it myself, but it (possibly hyperv) was going to be hairier than I was ready to deal with at that time. | |
| 16:46:17 | efried | mriedem: by "personal" you mean me or you? :P | |
| 16:46:45 | efried | stephenfin: I should wait to re-look at rootwrap series, yes? | |
| 16:47:14 | mriedem | efried: as in something we call each other about late on a friday night to talk about yes | |
| 16:47:16 | efried | stephenfin: The 'done' on the ipv4 patch is still forthcoming? | |
| 16:47:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix update_provider_tree signature in reference docs https://review.opendev.org/667408 | |
| 16:47:21 | stephenfin | efried: 2 secs | |
| 16:47:31 | efried | mriedem: ack, I like it. | |
| 16:48:08 | stephenfin | Actually, I want to test this more so I'll leave it til the morning | |
| 16:48:46 | stephenfin | efried: Sort of. I'm going to drop the first patch and then address the open TODOs in the remainder. If you want to take a look at any you haven't reviewed yet though, that could be helpful | |
| 16:49:38 | efried | something weird just happened | |
| 16:50:04 | stephenfin | Ohh, that's interesting | |
| 16:50:18 | stephenfin | I Ctrl + C'd git-review during the upload once I realized I'd more work to do | |