| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-09 | |||
| 10:06:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: fakelibvirt: Stop distinguishing between NUMA, non-NUMA https://review.opendev.org/675570 | |
| 10:06:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571 | |
| 10:07:19 | stephenfin | alex_xu, efried, sean-k-mooney: Okay, with the exception of the reshaper, which still has some todo's in it, I _think_ the PCPU stuff is ready for review. It might look big but the bulk of it is handling tech debt. Let me know if you need advice on any part of it | |
| 10:07:47 | alex_xu | stephenfin: cool~ | |
| 10:08:41 | stephenfin | alex_xu, efried, sean-k-mooney: I'm going to continue working on the reshaper and then start manual testing once that's done. I particularly want to check Train controller <-> Stein compute node compatibility and upgrades | |
| 10:10:50 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add database schema upgrade check https://review.opendev.org/667047 | |
| 10:37:41 | openstackgerrit | Adam Spiers proposed openstack/nova master: Rename 'map' variable to avoid shadowing keywords https://review.opendev.org/675578 | |
| 10:38:00 | openstackgerrit | Chris Dent proposed openstack/nova master: Add a document that describes profiling eventlet services https://review.opendev.org/674865 | |
| 10:47:57 | frickler | can someone help with fixing novnc multinode job config? see my latest comment in https://bugs.launchpad.net/nova/+bug/1669468 | |
| 10:47:57 | openstack | Launchpad bug 1669468 in OpenStack Compute (nova) "tempest.api.compute.servers.test_novnc.NoVNCConsoleTestJSON.test_novnc fails intermittently in neutron multinode nv job" [Medium,Confirmed] | |
| 10:49:45 | frickler | might actually be devstack's fault to properly set up the config, but I'm also not 100% sure how the correct config would look like | |
| 11:15:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow up for I491eb7abe1405b27c552cdef142c688a46e914f2 https://review.opendev.org/675589 | |
| 11:53:23 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Indroduce new LibvirtConfigGuestPMEM https://review.opendev.org/672956 | |
| 11:53:23 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Populates existing pmems in libvirt device manager https://review.opendev.org/672957 | |
| 11:53:24 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Report pmem namespaces resources by provider tree https://review.opendev.org/634551 | |
| 11:53:24 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Support instance creation and resize with pmems https://review.opendev.org/672958 | |
| 11:53:25 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add functional tests for persistent memory https://review.opendev.org/675592 | |
| 12:43:30 | belmoreira | cdent nice blogpost about placement performance. You may also be interested in these results: https://docs.google.com/document/d/1UE3jpukdoX45l5ghUg0AlcWBh95glRiGUv7E3YazXNk/edit | |
| 12:43:42 | belmoreira | I hope to publish it soon | |
| 12:44:27 | belmoreira | also ^ efried mriedem | |
| 12:44:28 | cdent | ah, excellent, I'll give that a read soon, just getting ready to go to lunch | |
| 12:47:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test heal port allocations in nova-next https://review.opendev.org/669879 | |
| 12:54:33 | openstackgerrit | Merged openstack/nova master: libvirt: Handle alternative UEFI firmware binary paths https://review.opendev.org/348394 | |
| 13:16:42 | mriedem | party people in the house say hey-o! | |
| 13:17:25 | cdent | hey? o? | |
| 13:18:11 | mriedem | that's the kind of enthusiasm i'm looking for | |
| 13:18:21 | mriedem | lyarwood: can you get this queens backport? https://review.opendev.org/672690 | |
| 13:18:30 | mriedem | dansmith: can you get this one? https://review.opendev.org/668123 | |
| 13:18:37 | mriedem | today is queens release day | |
| 13:21:11 | gibi | is it Friday yet? | |
| 13:22:58 | mriedem | yes | |
| 13:23:03 | mriedem | got another queens backport that needs +W https://review.opendev.org/#/c/669126/ | |
| 13:23:12 | artom | mriedem, didn't know Elizabeth was in jail... | |
| 13:24:46 | mriedem | i'm missing the reference | |
| 13:25:11 | cdent | artom: the life of a royal is constant imprisonment and suffering, don't ya know | |
| 13:26:09 | mriedem | oh queens release day, i see | |
| 13:27:47 | lyarwood | mriedem: ack done | |
| 13:27:54 | mriedem | can you hit these as well? https://review.opendev.org/#/q/topic:bug/1837955+branch:stable/queens | |
| 13:28:03 | lyarwood | sure thing | |
| 13:28:21 | gibi | based on the joke it is definitely Friday! | |
| 13:28:50 | mriedem | could use a stable core on these as well https://review.opendev.org/#/q/topic:bug/1764556+branch:stable/queens | |
| 13:28:53 | mriedem | johnthetubaguy: ^? | |
| 13:32:14 | mriedem | another queens backport https://review.opendev.org/#/c/672161/2 | |
| 13:35:08 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Fix type error on call to mount device https://review.opendev.org/669629 | |
| 13:36:11 | mriedem | and https://review.opendev.org/#/c/669629/ | |
| 13:38:03 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659247 | |
| 13:38:28 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659247 | |
| 13:46:24 | finucannot | aspiers: So apparently I can't access lxml documentation because it's Friday and that's not allowed (try navigating to lxml.de) | |
| 13:48:26 | mordred | the code is self-documenting? | |
| 13:48:46 | finucannot | I guess it _has_ to be | |
| 13:49:55 | mriedem | you can get to https://lxml.de/tutorial.html | |
| 13:50:21 | mriedem | https://lxml.de/api/index.html | |
| 13:50:29 | finucannot | Ah, sweet | |
| 13:50:32 | finucannot | mriedem++ | |
| 13:50:33 | mriedem | that'll be $5 please | |
| 13:51:48 | finucannot | Darn, I'm fresh out of dollar bills. Crying shame, that | |
| 13:52:34 | mriedem | i'll take 5 eu | |
| 13:53:00 | finucannot | Britain refuses to take even one | |
| 13:53:21 | mriedem | boris has it under control | |
| 13:54:14 | mriedem | polite muttering i'm sure | |
| 14:04:33 | aspiers | mriedem: no swear-words in this channel please | |
| 14:05:29 | frickler | mriedem: I commented on https://bugs.launchpad.net/nova/+bug/1669468 earlier with what I think would be the reason for about 50% failure rate, but I'm not sure at which place to fix it | |
| 14:05:29 | openstack | Launchpad bug 1669468 in OpenStack Compute (nova) "tempest.api.compute.servers.test_novnc.NoVNCConsoleTestJSON.test_novnc fails intermittently in neutron multinode nv job" [Medium,Confirmed] | |
| 14:15:02 | melwitt | frickler: I think your assessment sounds probable. I think I could find where to fix it, I've done a few changes to console settings in devstack in the past | |
| 14:16:10 | mriedem | sorry, on a call now but will take a look later | |
| 14:19:21 | cdent | good article belmoreira . I made a couple of suggested edits. It seems like "The provider_tree "deepcopy" should be re-evaluated" is the big takeaway. Based on my recent posting it _might_ make sense to add a __copy__ to provider tree | |
| 14:27:00 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659247 | |
| 14:38:25 | openstackgerrit | Merged openstack/nova stable/queens: Stabilize unshelve notification sample tests https://review.opendev.org/674636 | |
| 14:45:22 | mriedem | need a stable core for this queens one as well https://review.opendev.org/#/c/666862/ | |
| 14:47:44 | dansmith | mriedem: I can just +W that because the only conflict was a test yeah? | |
| 14:47:57 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Fix double word hacking test https://review.opendev.org/665790 | |
| 14:48:07 | dansmith | (as in without looking very hard) | |
| 14:58:29 | mriedem | dansmith: yeah | |
| 15:03:47 | finucannot | aspiers, sean-k-mooney: We're touching an awful lot of the same code between the amd-sev-libvirt-support, image-metadata-prefiltering and cpu-resources specs. I forsee a lot of merge conflicts, but also a chance to share best practices | |
| 15:04:03 | finucannot | aspiers, sean-k-mooney: Especially around the scheduler pre-filter stuff. Lots of comments left on both series | |
| 15:08:52 | mriedem | dansmith: since you just went through these for rocky, if you have a minute https://review.opendev.org/#/q/topic:bug/1764556+branch:stable/queens | |
| 15:08:55 | mriedem | i'm trying to flush queens for the release | |
| 15:13:08 | aspiers | finucannot: I feel like libvirt/driver.py and test_driver.py are horrendously big and could do with splitting up somehow | |
| 15:13:22 | melwitt | otherwise I can do those, I meant to but was watching the rocky change land first. I should have pre-reviewed | |
| 15:14:06 | finucannot | aspiers: Aye, it regularly crashes the flake8 process in syntastic (Vim plugin). I don't even know where to start though | |
| 15:14:10 | artom | mriedem, you're going for a royal flush? | |
| 15:14:21 | mriedem | no, 4 of a kind | |
| 15:14:33 | mriedem | shoot the moon | |
| 15:14:52 | artom | groaning \o/ | |
| 15:15:16 | mriedem | this is the current +2'ed list of queens changes https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+label:Code-Review=2 | |
| 15:15:18 | mriedem | if that helps | |
| 15:21:58 | dansmith | mriedem: that service too old set looked like a clean pick from the previous yeah? | |
| 15:22:28 | dansmith | makes sense I guess since most of the change is the added and then modified tests | |
| 15:23:34 | mriedem | yup no conflicts | |
| 15:29:37 | belmoreira | thanks cdent | |
| 15:32:19 | mriedem | melwitt: are you working on a patch for bug 1669468? otherwise i can throw one up quick to see if it explodes. | |
| 15:32:19 | openstack | bug 1669468 in OpenStack Compute (nova) "tempest.api.compute.servers.test_novnc.NoVNCConsoleTestJSON.test_novnc fails intermittently in neutron multinode nv job" [Medium,Confirmed] https://launchpad.net/bugs/1669468 | |
| 15:33:04 | melwitt | mriedem: I was planning to but haven't looked yet. if you have something to try, go ahead | |
| 15:33:39 | mriedem | btw i've got most of the tests in https://review.opendev.org/#/c/675218/ fixed but there are 2 failing for some yet unknown reason https://review.opendev.org/#/c/675218/ | |
| 15:34:33 | melwitt | ack | |
| 15:41:22 | openstackgerrit | Merged openstack/nova stable/queens: fix cellv2 delete_host https://review.opendev.org/672690 | |
| 15:41:29 | openstackgerrit | Merged openstack/nova stable/queens: Add functional regression test for bug 1837955 https://review.opendev.org/673567 | |
| 15:41:29 | openstack | bug 1837955 in OpenStack Compute (nova) queens "MaxRetriesExceeded sometime fails with messaging exception" [Medium,In progress] https://launchpad.net/bugs/1837955 - Assigned to Matt Riedemann (mriedem) | |
| 15:42:13 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/668123 | |
| 15:46:05 | mriedem | frickler: melwitt: this might have been the regression https://review.opendev.org/#/c/672493/6 | |