| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-05 | |||
| 21:40:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove redundant image GET call in _do_rebuild_instance https://review.openstack.org/600260 | |
| 21:40:19 | mriedem | cdent: i imagine the VC resource tracker is infinitely worse than nova's | |
| 21:40:24 | mriedem | which gives me some comfort | |
| 21:40:38 | mriedem | by worse, i mean complicated | |
| 21:41:08 | cdent | thankfully I get to stay far away from that stuff | |
| 22:13:05 | mriedem | fun, just saw a gate failure where rebuild creates the libvirt guest then while polling the domain to see if it's running, libvirt says it's not found | |
| 22:13:08 | mriedem | and we blow up | |
| 22:24:40 | openstackgerrit | Elancheran S proposed openstack/nova stable/pike: Add exact match aggregate image properties matcher/filter https://review.openstack.org/599870 | |
| 22:25:32 | openstackgerrit | Elancheran S proposed openstack/nova master: Add exact match aggregate image properties matcher/filter https://review.openstack.org/593167 | |
| 22:31:15 | openstackgerrit | Jonte Watford proposed openstack/nova master: Modified version of 0027-Numa-object-string-representations.patch with some updates from the current numa files for nova: numa.py instance_numa_topology.py https://review.openstack.org/600269 | |
| 23:31:02 | Sundar | melwitt: Please ping me when you can | |
| #openstack-nova - 2018-09-06 | |||
| 00:42:36 | openstackgerrit | fupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf https://review.openstack.org/592243 | |
| 00:49:52 | openstackgerrit | Merged openstack/nova master: Move str to six.string_types https://review.openstack.org/599493 | |
| 00:49:59 | openstackgerrit | Merged openstack/nova master: Fix a failure to format config sample https://review.openstack.org/597986 | |
| 01:02:19 | naichuans | mriedem: Got it, Matt. I can't got PTG this time, maybe bauzas: and efried: could take care about it with you? | |
| 01:22:23 | melwitt | Sundar: I got your email, I'll add info to our etherpad about cyborg/nova on monday from 2-3pm | |
| 01:23:46 | openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node https://review.openstack.org/582899 | |
| 01:33:02 | Sundar | melwitt: Thank you | |
| 01:35:27 | melwitt | thanks | |
| 03:33:28 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | |
| 04:56:44 | openstackgerrit | Merged openstack/nova stable/rocky: Don't use '_TransactionContextManager._async' https://review.openstack.org/597421 | |
| 05:47:04 | openstackgerrit | Merged openstack/nova master: Doc: PowerVM does support shelve https://review.openstack.org/593052 | |
| 06:10:24 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix a failure to format config sample https://review.openstack.org/600336 | |
| 08:01:03 | bauzas | naichuans: hi | |
| 08:01:24 | bauzas | naichuans: sure, I'll make sure I get you concerns | |
| 08:01:29 | bauzas | your* | |
| 08:10:23 | openstackgerrit | Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | |
| 08:29:04 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Remove noisy DEBUG log https://review.openstack.org/600372 | |
| 10:28:01 | shyamb | Hi | |
| 10:28:26 | shyamb | Is it expected or some issue? | |
| 10:28:26 | shyamb | all other directories in "/var/lib/nova" has nova:nova ownership | |
| 10:28:26 | shyamb | On compute node, I see ownership of "/var/lib/nova/instances/snapshots" as root:root | |
| 11:02:55 | openstackgerrit | Merged openstack/nova master: Fix nits: Compute: Handle reshaped provider trees https://review.openstack.org/598387 | |
| 11:17:46 | openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | |
| 12:15:13 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Fix resource tracker updates during instance evacuation https://review.openstack.org/600425 | |
| 13:23:29 | openstack | Launchpad bug 1790850 in OpenStack Compute (nova) "Xenapi test failure when building the Debian 18.0.0 package in Sid" [Undecided,New] | |
| 13:23:29 | mriedem | naichuans: looks like the xen driver has some unit tests that actually execute openssl? https://bugs.launchpad.net/nova/+bug/1790850 | |
| 13:27:36 | naichuans | mriedem: I don't familiar with this part of the code. But you are right, look we are using openSLL in xenapi. Is that a version conflict? | |
| 13:27:57 | mriedem | yes looks like it's generating an error on debian sid | |
| 13:28:41 | bauzas | I remember some old story about that | |
| 13:29:19 | naichuans | Is it urgent? Or I will look into it tomorrow. bauzs: anything could share with us? | |
| 13:29:28 | mriedem | naichuans: no not urgent | |
| 13:29:40 | naichuans | Thanks, Matt | |
| 13:30:22 | bauzas | naichuans: trying to remember the situation we had | |
| 13:30:40 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova stable/rocky: placement: use simple code paths when possible https://review.openstack.org/600447 | |
| 13:31:12 | naichuans | :) bauzas: | |
| 13:32:41 | openstack | Launchpad bug 1790847 in OpenStack Compute (nova) "Failing tests when building Rocky Debian package in Python 3.7" [Medium,Confirmed] | |
| 13:32:41 | mriedem | if anyone has a py37 system, we have a few unit test bugs https://bugs.launchpad.net/nova/+bug/1790847 https://bugs.launchpad.net/nova/+bug/1790849 | |
| 13:32:42 | openstack | Launchpad bug 1790849 in OpenStack Compute (nova) "Failng hacking tests when building Rocky Debian package under Python 3.7" [Medium,Confirmed] | |
| 13:33:23 | openstack | Launchpad bug 1771506 in OpenStack Compute (nova) "Unit test failure with OpenSSL 1.1.1" [Low,Confirmed] | |
| 13:33:23 | bauzas | naichuans: mriedem: yup, remember this https://bugs.launchpad.net/nova/+bug/1771506 | |
| 13:33:51 | bauzas | we're like super dependent on openssl because of xen | |
| 13:33:59 | bauzas | something we should probably mock out | |
| 13:35:10 | mriedem | ah i'll mark the dupe | |
| 13:35:49 | naichuans | Yes, it should be same. Thanks, bauzas | |
| 13:36:17 | naichuans | I will look into it tomorrow | |
| 13:41:47 | openstackgerrit | Eric Fried proposed openstack/nova master: Add an example to add more pci devices in nova.conf https://review.openstack.org/592243 | |
| 13:42:31 | efried | mriedem: I would feel more comfortable approving https://review.openstack.org/#/c/592243/ if I had your eyes on it. | |
| 13:42:42 | efried | stephenfin: wanna have a look at that one last time -^ | |
| 13:42:50 | efried | (hopefully last :) | |
| 13:47:27 | mriedem | efried: pci anything in nova is my biggest weakness | |
| 13:47:50 | mriedem | i added moshe and sahid | |
| 13:48:14 | efried | mriedem: I already shepherded this patch to not touch the pci code. It was originally going to change the logic so you *could* specify [{...}, ...] | |
| 13:48:22 | efried | mriedem: But okay, thanks. | |
| 13:50:12 | melwitt | nova meeting in 10 min | |
| 13:50:18 | mriedem | this reminds me to look at that persons other patch https://review.openstack.org/#/c/582899/ | |
| 13:50:25 | mriedem | with 0 explanation in the commit mesage | |
| 13:50:28 | mriedem | *message | |
| 13:55:22 | mriedem | big ol 01 | |
| 13:55:23 | mriedem | -1 | |
| 13:55:50 | mriedem | dansmith: heh i was just going to point you at ^ | |
| 13:56:01 | mriedem | and now see, "We should get Matt's read on this as well. However, I'm skeptical of how/why this would be happening and thus think we should require a healthy bug report showing the root cause. " | |
| 13:56:08 | mriedem | i said basically the same thing | |
| 13:56:30 | dansmith | is there more info now? | |
| 13:56:41 | mriedem | not in the commit message | |
| 13:57:05 | mriedem | the bug report says there are mq issues | |
| 13:57:08 | mriedem | that's about it | |
| 13:57:22 | dansmith | right, well, that's not enough, IMHO | |
| 13:57:36 | mriedem | yeah i can't tell if this is talking about resize or build reschedules or what | |
| 13:58:45 | mriedem | "Exception during message handling: MessageDeliveryFailure: Unable to connect to AMQP server on 10.127.3.64:5672 after None tries: 'NoneType' object has no attribute '__getitem__'" | |
| 13:59:12 | mriedem | doesn't really tell me anything | |
| 13:59:49 | dansmith | I'm just not interested in working too hard at it with so little detail | |
| 14:00:26 | mriedem | yeah me neither | |
| 14:00:37 | mriedem | i'm just not interested in working | |
| 14:03:08 | mriedem | damn gibi is gone | |
| 14:03:15 | mriedem | need a re-+2 on https://review.openstack.org/#/c/599744/ | |
| 14:03:41 | mriedem | alex_xu: or you can proxy gibi's +2 and approve ^ since he was +2 before i removed the try/except stuff | |
| 14:04:37 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 14:05:39 | dansmith | mriedem: so that online migration has to be removed when the split happens yeah? | |
| 14:07:20 | mriedem | dansmith: yup | |
| 14:07:30 | mriedem | i don't know how we're going to do those | |
| 14:07:31 | dansmith | ah, I guess this is all within placement tables anyway? | |
| 14:07:39 | mriedem | yeah | |
| 14:07:41 | dansmith | I was thinking this had to cross-reference nova and placement things | |
| 14:07:58 | dansmith | okay, so placement'll need a way to trigger this beyond that I suppose | |
| 14:08:35 | cdent | i've started some _very_ hand-wavey motions in the direction of a placement-manage | |
| 14:09:01 | mriedem | cdent: efried: the reshaper item ~L269 in https://etherpad.openstack.org/p/nova-ptg-stein should probably move to the retrospective etherpad? | |
| 14:09:06 | mriedem | it's more of a retrospective item right? | |
| 14:09:55 | efried | mriedem: okay | |
| 14:10:07 | sahid | efried: i shared my idea | |