| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-22 | |||
| 11:08:47 | sean-k-mooney | sure that should work i have one or two meetsing around 4pm IST(UTC+1) | |
| 11:13:32 | adrianc | sean-k-mooney: great, can you set something at around 11AM (UTC+1) | |
| 11:14:36 | sean-k-mooney | that works for me. im not sure if there are any US based folks that should join but assumeing not then yes | |
| 11:16:42 | adrianc | great, no US folks from my side. | |
| 11:17:44 | sean-k-mooney | cool we may want to check in with artom who will not be online at that time but i can sync with him later | |
| 11:18:30 | artom | I'm here, barely | |
| 11:19:11 | sean-k-mooney | artom: its 6am for you now right? | |
| 11:19:17 | artom | 7:20 | |
| 11:19:28 | aspiers | ouch | |
| 11:20:04 | sean-k-mooney | artom: why? why would you be awake and on irc at this time? | |
| 11:21:00 | artom | Heh, I woke up on my own, 6-7 hours of sleep is usually enough for me | |
| 11:21:33 | artom | And I wanted to get a downstream escalation out of the way, hence laptop and IRC | |
| 11:22:07 | sean-k-mooney | artom: i was just talking to adrianc about having a call about sriov livemigtion at 11:00 utc+1 on wednesday. is that something you want to join i was sugessting it might be a bit too early for you but not sure you needed to attend either | |
| 11:22:19 | artom | Also, I need to literally go buy milk, otherwise 1. kids will complain that I'm a bad parent and 2. I'll have nothing to put in my coffee | |
| 11:22:58 | artom | sean-k-mooney, err, can you give me a time delta for the meeting? Like, "in 3 hours"? | |
| 11:22:59 | sean-k-mooney | artom: hehe no worres coff.. childern are importent | |
| 11:23:38 | sean-k-mooney | artom: it 12:22 now so it would be your 6 am on wednesday | |
| 11:23:49 | artom | sean-k-mooney, hah, no. | |
| 11:23:51 | artom | :) | |
| 11:34:17 | jangutter | sean-k-mooney: is https://review.openstack.org/#/c/608693/ now more or less in line with your idea of the "generic representor" thingy? If so, I'll update the spec again to reflect this. | |
| 11:46:42 | openstackgerrit | Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407 | |
| 11:50:27 | artom | sean-k-mooney, thanks for thinking of including me, but 6:00 is too damn early for a meeting | |
| 11:50:46 | artom | I'll happily review the specs/code in the normal fashion | |
| 12:03:03 | sean-k-mooney | artom: no worries ill can give you the clip notes if it impacts the numa aware migration in anyway | |
| 12:08:51 | sean-k-mooney | jangutter: sorry spinning a few plates currently ill comment on the review later today | |
| 12:09:34 | jangutter | sean-k-mooney: no rush at all, crockery comes first | |
| 12:09:58 | sean-k-mooney | jangutter: if i remember correctly it did but ill have to check it later to confirm | |
| 12:29:37 | jaosorior | hey folks, when configuring a vendordata plugin, from where is the vendordata plugin called? is it from nova-metadata? or from nova-api? or the compute itself? | |
| 12:38:31 | jaypipes | aspiers: ok, sorry for delay. https://review.openstack.org/#/c/609779/ is now reviewed. | |
| 12:38:44 | aspiers | thanks! | |
| 12:38:47 | jaypipes | np! :) | |
| 12:39:01 | jaypipes | aspiers: sorry, was out at a $work thing all last week :( | |
| 12:39:10 | aspiers | np at all | |
| 12:39:17 | aspiers | I had enough other feedback to keep me busy ;-) | |
| 12:39:23 | jaypipes | rock on :) | |
| 12:40:05 | sapd1 | Hi everyone, I get this errror when live migrate instance. http://paste.openstack.org/show/732581/ | |
| 12:40:18 | sapd1 | I upgraded from queens to rocky | |
| 12:40:46 | sapd1 | libvirt: 3.6 and qemu 2.11 | |
| 13:08:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629 | |
| 13:29:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect https://review.openstack.org/607400 | |
| 13:29:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Remove unnecessary redirect https://review.openstack.org/607402 | |
| 13:30:51 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779 | |
| 13:31:00 | aspiers | jaypipes: ^^^ | |
| 13:39:02 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174 | |
| 13:40:05 | jaypipes | aspiers: rock on. | |
| 13:43:19 | openstackgerrit | Markus Hentsch proposed openstack/nova-specs master: Spec for the Nova part of Image Encryption https://review.openstack.org/608696 | |
| 13:55:56 | efried | n-sch meeting in 5 minutes in #openstack-meeting-alt | |
| 14:03:06 | mriedem | alex_xu: you should probably take a look at this https://review.openstack.org/#/c/609591/ | |
| 14:03:55 | stephenfin | mriedem: Could you take a look at this bug fix series this week? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1784353 | |
| 14:04:10 | alex_xu | mriedem: got it, will take a look | |
| 14:04:44 | mriedem | stephenfin: yeah | |
| 14:04:49 | stephenfin | ta | |
| 14:05:44 | sapd1 | Does anyone get this problem when upgrade from queens to rocky? | |
| 14:10:42 | s10 | sapd1: looks like this issue is related to the upgrade to the qemu 2.11, not nova. | |
| 14:11:43 | sapd1 | s10: I have some compute nodes which are running qemu 2.11 and it gets same issue. | |
| 14:12:43 | sapd1 | Does nova-compute require higher qemu/libvirt version/ | |
| 14:13:09 | sapd1 | sorry s/2.11/2.10/ | |
| 14:13:52 | mriedem | johnthetubaguy: you might care about this https://review.openstack.org/#/c/609043/ | |
| 14:26:46 | mriedem | stephenfin: these are pretty easy https://review.openstack.org/#/q/topic:bug/1763051+(status:open+OR+status:merged) - first is a refactor, second is a try/catch/log but don't blow up sort of thing | |
| 14:27:33 | mriedem | oh sorry, latter is try/catch/rollback | |
| 14:27:45 | mriedem | was mixing that up with a similar live migration error handling patch | |
| 14:27:58 | stephenfin | mriedem: Sounds good | |
| 14:28:15 | stephenfin | mriedem: Also, I took a look at those debug logs you reworked for the issue awaugama and I are seeing. +2'd the first but do you think the second one should be merged now? Comments are confusing/misleading | |
| 14:28:26 | mriedem | i believe your employer cares about those live migration ones | |
| 14:29:23 | stephenfin | This one (efried got to it too) https://review.openstack.org/#/c/597560/ | |
| 14:29:33 | mriedem | what is confusing/misleading? | |
| 14:29:47 | mriedem | i also mostly restored/cleaned that up for awaugama to recreate the problem | |
| 14:29:59 | mriedem | if it's not helpful then i don't care if we don't use it | |
| 14:30:02 | stephenfin | "I don't really intend these overly verbose logs to get merged." doesn't apply anymore, right? | |
| 14:30:39 | mriedem | compare PS7 to PS1 | |
| 14:30:46 | mriedem | it's a much slimmer version | |
| 14:31:29 | stephenfin | ack | |
| 14:31:38 | efried | mriedem: See comments, I still think it's going to be excessive. | |
| 14:32:04 | efried | but yeah, I didn't vote because I wasn't sure if we were now intending to actually merge it or if it was still just for people to debug with. | |
| 14:33:25 | mriedem | i just replied | |
| 14:33:40 | mriedem | if you want to wait for awaugama to recreate and say, yes this helped, then let's wait | |
| 14:34:20 | awaugama | owalsh: can you help with pulling in a patch from upstream into a build for extra logging? | |
| 14:39:31 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: OVA and StrOpt images as VM templates https://review.openstack.org/609736 | |
| 15:04:40 | mriedem | stephenfin: lyarwood: https://review.openstack.org/#/c/587013/15/nova/tests/fixtures.py@1728 | |
| 15:04:43 | mriedem | am i missing something? | |
| 15:23:43 | melwitt | ||
| 16:02:22 | stephenfin | mriedem: Yeah, but I've confused you in addressing your nit here https://review.openstack.org/#/c/587013/12/nova/tests/fixtures.py@1721 | |
| 16:02:51 | stephenfin | mriedem: _attachment_id on that line is actually the two item tuple | |
| 16:03:13 | stephenfin | mriedem: See https://review.openstack.org/#/c/587013/15/nova/tests/fixtures.py@1711 | |
| 16:04:17 | openstackgerrit | sean mooney proposed openstack/nova master: harden placement init under wsgi https://review.openstack.org/610034 | |
| 16:04:31 | stephenfin | and two lines above that, where we explode it out into the two item tuple. I guess that should go back to be called 'attachment' | |
| 16:07:15 | mriedem | ok commented | |
| 16:07:26 | mriedem | i'd like that re-worked because the variable naming is really confusing | |
| 16:08:24 | stephenfin | Agreed. It was called 'attachment' before but I guess that isn't much better | |
| 16:20:54 | openstackgerrit | Merged openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088 | |
| 16:21:01 | openstackgerrit | Merged openstack/nova master: Log the operation when updating generation in ProviderTree https://review.openstack.org/597553 | |
| 16:21:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | |
| 16:21:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | |
| 16:21:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | |
| 16:39:12 | openstackgerrit | sean mooney proposed openstack/nova master: harden placement init under wsgi https://review.openstack.org/610034 | |
| 16:41:07 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9) | |
| 16:41:07 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733 | |
| 16:43:38 | mnaser | fyi -- if anyone from the nova team hasn't seen this (yet): http://lists.openstack.org/pipermail/openstack-dev/2018-October/135941.html | |
| 16:43:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix jsonutils.to_primitive UserWarning https://review.openstack.org/612445 | |
| 16:44:06 | spatel | sean-k-mooney: you there? | |