| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-09 | |||
| 15:49:48 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Interim follow ups for cpu-resources series https://review.opendev.org/680983 | |
| 15:49:49 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add 'NUMACell.pcpuset' field https://review.opendev.org/680108 | |
| 15:49:49 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Use sane indent https://review.opendev.org/680229 | |
| 15:49:50 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800 | |
| 15:49:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add support for translating CPU policy extra specs, image meta https://review.opendev.org/671801 | |
| 15:49:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571 | |
| 15:49:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895 | |
| 15:50:14 | mriedem | artom: i want an ack on the patch | |
| 15:50:23 | mriedem | to show he's actually looked at your code change | |
| 15:50:45 | bauzas | stephenfin: oh heh | |
| 15:52:55 | artom | mriedem, understood | |
| 15:54:27 | bauzas | gibi: still looking at the cpu-resource series but you're next :) | |
| 15:54:46 | bauzas | not sure I'll have time before EOB, but tomorrow you're up :) | |
| 15:54:55 | gibi | bauzas: no worries. I will end my day soon too | |
| 15:55:56 | bauzas | gibi: the check pipeline is a bit late but if tomorrow we can send you some patches, it should be on time | |
| 15:56:20 | gibi | fingers crossed :) | |
| 15:58:12 | openstackgerrit | Merged openstack/nova master: doc: Fix a broken reference link https://review.opendev.org/680892 | |
| 16:00:38 | mriedem | maybe melwitt can jump into that series | |
| 16:00:49 | mriedem | those bottom 6 or so changes are min version validation and functional testing | |
| 16:00:53 | mriedem | so there isn't a ton of context required | |
| 16:01:13 | mriedem | losing a day this week with the gate backed up likely means missing feature freeze | |
| 16:01:53 | bauzas | mriedem: I can work a bit overlate | |
| 16:02:26 | mriedem | or we can have more cores reviewing more code for FF | |
| 16:02:52 | melwitt | gibi's series? I can help | |
| 16:03:10 | mriedem | yeah, the bottom 6 changes are +2'ed, | |
| 16:03:20 | gibi | melwitt: series starts here https://review.opendev.org/#/c/680394 | |
| 16:03:22 | mriedem | the runtime code is just about min compute service version compat | |
| 16:03:26 | gibi | melwitt: thanks in advance | |
| 16:03:28 | melwitt | ok, thanks | |
| 16:03:32 | mriedem | and then there are functional test changes on top of those | |
| 16:04:16 | mriedem | melwitt: for context, compute rpc api 5.2 (service version 39) is what enables the compute side stuff https://review.opendev.org/#/c/655721/18/nova/objects/service.py | |
| 16:04:22 | mriedem | so that's what those min compute service version checks are looking for | |
| 16:05:49 | melwitt | cool thanks | |
| 16:06:31 | zbr | any chance to get pre-commit merged? https://review.opendev.org/#/c/665518/ | |
| 16:16:46 | sean-k-mooney | mriedem: artom i think donnyd said this earlier but FN is set to 0 capastiy currently as donnyd was having electical work done over the weekend. so the numa jobs are currenly not going to run | |
| 16:17:52 | mriedem | sean-k-mooney: couldn't we use vexxhost? | |
| 16:18:51 | artom | mriedem, sean-k-mooney, FWIW, the passing job ran with the allocations_cpu_tuning stuff | |
| 16:19:30 | artom | I did the recheck at Sep 7 9:52 AM | |
| 16:19:50 | sean-k-mooney | mriedem im going to reach out to mnaser to see if we can in the future but we need multi numa flaovr whic i dont think vexxhost currenly provide to the gate | |
| 16:20:08 | artom | Having uploaded PS60 of https://review.opendev.org/#/c/635229/60/ at Sep 7 9:26 AM | |
| 16:20:16 | mriedem | artom: did you confirm things in the logs as you'd expect? | |
| 16:20:28 | mriedem | like xml you'd expect to see? | |
| 16:20:30 | sean-k-mooney | mriedem: if vexxhost would be will to add one like the ones FN have added then yes we could use both vexxhost and FN for this | |
| 16:20:32 | artom | The changes since then are basically comments: https://review.opendev.org/#/c/635229/60..61/nova/tests/unit/virt/libvirt/test_driver.py | |
| 16:20:48 | artom | mriedem, yep, I even added debug logging to the _update_numa_xml method | |
| 16:20:54 | mriedem | artom: yeah i saw and commented on PS61 | |
| 16:22:02 | sean-k-mooney | mnaser: not to put you on the spot but is creating a custom flavor for the upstream ci to consume possible/something you/vexxhost would be willing to consider? | |
| 16:22:15 | artom | So for example input: https://zuul.opendev.org/t/openstack/build/dd0f5dad770d40a2afb3c506327d1b3e/log/compute/logs/screen-n-cpu.txt.gz#2081 | |
| 16:22:24 | sean-k-mooney | mnaser: i can provide the detail later if so. | |
| 16:22:34 | artom | And then output: https://zuul.opendev.org/t/openstack/build/dd0f5dad770d40a2afb3c506327d1b3e/log/compute/logs/screen-n-cpu.txt.gz#2195 | |
| 16:23:17 | artom | We see the vcpupin go from 0-3 and 4-6 to 1-3 and 4-7 | |
| 16:23:49 | artom | And then for the allocations_cpu_tuning, I have https://review.opendev.org/#/c/680806/ | |
| 16:24:00 | openstackgerrit | Chris Dent proposed openstack/nova master: Don't duplicate PlacementFixture in libvirt func tests https://review.opendev.org/681059 | |
| 16:24:04 | cdent | mriedem, efried ^ | |
| 16:24:20 | artom | Again, input: https://zuul.opendev.org/t/openstack/build/f79144ae28114cea96e708d0bb5ebfb1/log/logs/screen-n-cpu.txt.gz#2565 | |
| 16:24:30 | artom | And output: https://zuul.opendev.org/t/openstack/build/f79144ae28114cea96e708d0bb5ebfb1/log/logs/screen-n-cpu.txt.gz#2666 | |
| 16:24:33 | donnyd | sean-k-mooney: artom correct FN is at zero for the moment | |
| 16:24:50 | artom | <vcpu placement="static" cpuset="0">1</vcpu> in both cases | |
| 16:25:02 | artom | Because I couldn't figure out how to set vcpu_pin_set per-node | |
| 16:25:08 | artom | But at least it runs through the code | |
| 16:25:09 | donnyd | The electrical work is all done.... Now just waiting on propane and FN will be back in business | |
| 16:25:28 | sean-k-mooney | artom: i have set vcpu pin set per node in one of them | |
| 16:25:59 | sean-k-mooney | artom: https://review.opendev.org/#/c/679805/ | |
| 16:26:18 | sean-k-mooney | artom: and that also passed on saturday | |
| 16:26:27 | artom | sean-k-mooney, what's with a NUMA flavor though, right? | |
| 16:26:33 | mriedem | artom: ok can you link those into the patch for reference later? | |
| 16:26:34 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/43f69768e18a4e1594810e606f101b6a | |
| 16:26:41 | sean-k-mooney | artom: oh yes | |
| 16:26:50 | artom | My DNM thing hits the other, non-NUMA case, to show that allocation_cpu_pins works | |
| 16:27:06 | sean-k-mooney | well we could do it without a numa flavor later or i can stack and check it by hand now | |
| 16:27:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: fakelibvirt: Make 'Connection.getHostname' unique https://review.opendev.org/681060 | |
| 16:27:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895 | |
| 16:27:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Mock 'libvirt_utils.file_open' properly https://review.opendev.org/681061 | |
| 16:29:08 | stephenfin | artom: https://review.opendev.org/#/c/681060/ | |
| 16:30:58 | artom | stephenfin, way to steal credit brah ;) | |
| 16:31:15 | stephenfin | desperate times /o\ | |
| 16:31:21 | mriedem | co-authored-by | |
| 16:31:25 | artom | stephenfin, you proll'y want to explain in more details what those lambda scoping issues were | |
| 16:31:35 | stephenfin | I should have set the authorship now that I'm thinking about it | |
| 16:31:49 | cdent | mriedem: I did a quick survey to look for other duplicate placement fixtures and did not find | |
| 16:32:00 | artom | mriedem, to be fair, I didn't write *that* solution | |
| 16:32:00 | bauzas | melwitt: gibi: I'm +W with the first 3 changes | |
| 16:32:05 | artom | I just found the problem | |
| 16:32:06 | cdent | that's me done for today I reckon | |
| 16:32:10 | mriedem | cool | |
| 16:32:12 | bauzas | melwitt: gibi: see you tomorrow | |
| 16:32:12 | mriedem | thanks | |
| 16:32:34 | artom | stephenfin, because they're really not obviousy | |
| 16:32:36 | artom | *obvious | |
| 16:33:41 | sean-k-mooney | i just clicked on it and no ideay lambda were related to this other then th efact i saw lambdas in the code so ya | |
| 16:44:15 | artom | sean-k-mooney, btw, Matt's expecting a vote from you on that NUMA LM patch - just making sure everyone's on the same page :) | |
| 16:44:37 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673485 | |
| 16:47:32 | mriedem | ^ i'm +2 on that novaclient change for 2.79 now | |
| 16:47:37 | mriedem | need to get that merged and released by thursday, | |
| 16:47:41 | mriedem | the osc change is also dependent o nit | |
| 16:48:52 | sean-k-mooney | artom: ok i keep getting distracted by jumping between your stuff stephns patach and email. | |
| 16:49:26 | sean-k-mooney | ill kick off a local devstack run with your latest patches then finish my review of stephns code and then review your code | |
| 16:49:35 | artom | sean-k-mooney, I'll stop harassing you - just making sure expectations were clear | |
| 16:49:51 | sean-k-mooney | sorry i ment to finish reviewing stephens code already | |
| 16:50:35 | sean-k-mooney | no worries i just need to focus. i started late today so ill be around for another few hours today | |