| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-19 | |||
| 12:33:45 | lyarwood | mriedem: ack will do | |
| 12:33:48 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 7 https://review.openstack.org/560748 | |
| 12:34:08 | openstackgerrit | sahid proposed openstack/nova-specs master: fix blueprint related to the spec overhead-pin-set https://review.openstack.org/561916 | |
| 12:40:00 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add request_id to instance action notifications https://review.openstack.org/553288 | |
| 12:44:36 | openstackgerrit | Merged openstack/nova stable/queens: Document how to disable notifications https://review.openstack.org/561954 | |
| 12:44:44 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: Report the allocated size of preallocated file based disks https://review.openstack.org/562074 | |
| 12:45:58 | sahid | mriedem: about your comment on the trusted tag we are going to erase, i let a comment in the review | |
| 12:46:02 | sahid | https://review.openstack.org/#/c/562565/ | |
| 12:46:31 | sahid | i think there are some condition where that could be an issue | |
| 12:52:49 | mriedem | sahid: wrong patch? | |
| 12:57:47 | sahid | mriedem: yep sorry https://review.openstack.org/#/c/458820/ | |
| 13:08:00 | openstackgerrit | Matthew Booth proposed openstack/nova master: Rename recreate to evacuate in driver signatures https://review.openstack.org/560900 | |
| 13:12:46 | openstackgerrit | Lei Zhang proposed openstack/nova master: Improve check capacity sql https://review.openstack.org/561862 | |
| 13:20:09 | maciejjozefczyk | Guys, maybe you remember any problem related to metadata-agent and cloud-init during openstack upgrade? I heard about bugs like cleaning of instance ssh service keys during upgrades but I'm not able to track any details. | |
| 13:25:45 | mriedem | sahid: replied | |
| 13:25:57 | mriedem | cdent: i guess we already had a postgresql job in nova's experimental queue | |
| 13:26:00 | mriedem | but that doesn't help you | |
| 13:27:52 | cdent | mriedem: yeah, looks like both jay and eric are happy to merge my fix without test proof | |
| 13:28:11 | cdent | which would require a functional environment running pg or a new tempest test | |
| 13:28:42 | cdent | I explored doing a functional pg environment locally but it quickly fell apart for reasons I couldn't care to figure out (at least not last night) | |
| 13:29:21 | sahid | mriedem: ok so you may have right, i thought about that hard-reboot thing but it seems i'm wrong i will build that env and re-put my head in that whole serie | |
| 13:29:50 | jaypipes | what did I mess up now? | |
| 13:32:01 | cdent | jaypipes: you're safe this time | |
| 13:32:05 | openstackgerrit | Chris Dent proposed openstack/nova master: Add root and parent provider uuid to group by clause https://review.openstack.org/562379 | |
| 13:38:34 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Skip placement on rebuild in same host https://review.openstack.org/561015 | |
| 13:54:17 | mriedem | cdent: as in functional tests in-tree? you have to have the pg env setup, there is a script for it | |
| 13:54:43 | mriedem | nova/tools/test-setup.sh | |
| 13:54:53 | mriedem | sets up the openstack_citest creds for mysql and pg | |
| 13:55:45 | bauzas | efried: jaypipes: thanks for providing the explanations about granular groups | |
| 13:56:01 | bauzas | efried: jaypipes: fine with me about defaulting 400 if you don't provide the qparam | |
| 13:56:17 | bauzas | mriedem: I guess you running the meeting ? | |
| 13:56:20 | bauzas | or gibi? | |
| 13:56:34 | efried | bauzas: ack. Does the "global" any|isolate option give you enough control to do what you need to do for NUMA? | |
| 13:56:44 | bauzas | efried: I think so yeah | |
| 13:56:56 | gibi | bauzas: I talked to melwitt yesterday and she said she will run it today | |
| 13:56:58 | bauzas | efried: I provided two qpery parameters | |
| 13:56:58 | efried | bauzas: It seems like it should: user who doesn't care about topo uses `any`; user who does care uses `isolate` | |
| 13:57:00 | jaypipes | efried: should be any|isolate|same, right? | |
| 13:57:09 | bauzas | efried: one would be the one you discussed | |
| 13:57:18 | melwitt | nova meeting in 3 minutes | |
| 13:57:20 | efried | jaypipes: If you want two granular groups to go into the same provider, you should combine them into a single numbered group. | |
| 13:57:21 | bauzas | efried: the second was 'in_tree' | |
| 13:57:37 | bauzas | efried: so I think it answers one of two | |
| 13:57:40 | efried | bauzas: We need to discuss that one further. | |
| 13:57:41 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/557930 | |
| 13:58:08 | cdent | mriedem: how do the functional tests become aware of that stuff? as in how does the new database string get connected to the code? that's the part I was struggling to make go. (simply overrdingn the default db connection string fell on its face) | |
| 13:58:15 | bauzas | efried: jaypipes: fine if I'm discussing that qpuery param by my own spec ? | |
| 13:58:26 | bauzas | efried: jaypipes: I tihnk we don't need a separate one | |
| 13:58:28 | efried | bauzas: As I stated somewhere or other, we can't use the keyword "in_tree" because that already has implications for GET /resource_providers to return *all* the providers in the same tree as the specified RP. | |
| 13:58:49 | bauzas | efried: I don't remember which exact param name I used but got it | |
| 13:58:53 | mriedem | cdent: i've always been confused by the framework for this, but you'd have to figure out how TestNovaMigrationsPostgreSQL works | |
| 13:58:55 | efried | bauzas: Also, it's not sufficient to say "same subtree" in any form, because we need an indication of where that subtree "starts". | |
| 13:59:09 | bauzas | efried: mmm k | |
| 13:59:10 | mriedem | the oslo.db opportunistic tests only run if a given backend is configured | |
| 13:59:27 | efried | bauzas: I did explain this further in one of the ML notes, but it was kinda buried. Let me find it... | |
| 13:59:31 | cdent | mriedem: i'll see if I can wrap my brain around that while vagueing out during the meeting | |
| 14:01:27 | cdent | wooph, some serious magic going on here | |
| 14:02:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Suggest when to propose notification changes https://review.openstack.org/562265 | |
| 14:03:07 | efried | bauzas: this one, scroll about 3/4 of the way down: http://lists.openstack.org/pipermail/openstack-dev/2018-April/129511.html | |
| 14:03:34 | Nisha_Agarwal | hi, I have stable pike setup where i was trying nova boot for ironic nodes. It always give "OrphanedObjectError" as soon as it spawns the thread for deploy. This happens due to the context getting set to None at some point. After debugging we found context is set to None here https://github.com/openstack/nova/blob/stable/pike/nova/objects/instance.py#L824 | |
| 14:03:49 | bauzas | efried: sorry, was probably not enough caffeinated | |
| 14:03:54 | bauzas | or my French | |
| 14:04:09 | bauzas | -ETOOMUCHWORDS | |
| 14:04:10 | Nisha_Agarwal | When i just comment out this line ironic deploy goes through successfully, | |
| 14:04:32 | Nisha_Agarwal | i tried on two Pike setups, i see this behaviour consistently | |
| 14:04:33 | efried | Dis-moi si tu veux que je l'explique en français :) | |
| 14:05:53 | Nisha_Agarwal | should i go ahead and raise bug against nova? | |
| 14:07:29 | Nisha_Agarwal | mriedem, ^^^^ | |
| 14:15:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add request_id to instance action notifications https://review.openstack.org/553288 | |
| 14:15:59 | sahid | dansmith: arg... unsupported configuration: Unknown interface <driver name='vhost-user'> | |
| 14:16:38 | mriedem | Nisha_Agarwal: do you have the latest fixes for pike? | |
| 14:16:58 | Nisha_Agarwal | mriedem, yes | |
| 14:17:03 | sahid | it's not really like libvirt ignores the driver name... only way to make it working is to avoid passing name attribute | |
| 14:17:25 | dansmith | sahid: that's really unfortunate... I guess I should push to see more real testing... | |
| 14:17:32 | Nisha_Agarwal | i have pulled all the patches for stable/pike | |
| 14:17:50 | sahid | dansmith: nothing is merged | |
| 14:18:17 | dansmith | sahid: I know, I mean before +2ing.. I expected you were testing these other configs locally before review | |
| 14:18:30 | sahid | i expected to have that env with DPDK ready before the final +2+W | |
| 14:19:36 | sahid | i'm going to fix the issue in libvirt but for nova i guess we should just avoid using name attribute for vhostuser interface | |
| 14:24:26 | dansmith | sean-k-mooney[m]: are there jobs anywhere in neutron or experimental that use vhost/vhostuser that we can depends-on changes like this to get at least a smoke test? | |
| 14:28:58 | jaypipes | sahid: quick update needed on https://review.openstack.org/#/c/562565/ commit message and then I'm +2 on the whole series. | |
| 14:30:54 | sahid | jaypipes: cool, i will fix that, thanks | |
| 14:30:57 | Nisha_Agarwal | mriedem, then i will go ahhead n raise the defect? | |
| 14:32:01 | mriedem | Nisha_Agarwal: i'm in a meeting so don't have time to investigate this, nor the ironic background, so for tracking it yes if you want to report a bug that's fine | |
| 14:32:37 | openstackgerrit | Merged openstack/nova stable/ocata: Avoid unnecessary lazy-loads in mutated_migration_context https://review.openstack.org/515581 | |
| 14:32:45 | openstackgerrit | Merged openstack/nova stable/ocata: Only attempt a rebuild claim for an evacuation to a new host https://review.openstack.org/550560 | |
| 14:49:05 | openstackgerrit | zhangyangyang proposed openstack/nova master: Deprecate auth_uri option https://review.openstack.org/562537 | |
| 15:00:47 | stephenfin | jaypipes, bauzas: Tell me more about https://review.openstack.org/#/c/557065/2/specs/rocky/approved/vgpu-rocky.rst | |
| 15:01:13 | stephenfin | jaypipes, bauzas: That's a very similar issue we have for NUMA-aware vSwitch | |
| 15:01:26 | bauzas | it's about configuring things | |
| 15:01:43 | stephenfin | I've been trying to get dynamic '[physnet_$PHYSNET_NAME]' groups created on the fly but oslo.config isn't designed that way | |
| 15:01:57 | bauzas | stephenfin: yeah I remember your spec | |
| 15:02:19 | stephenfin | So what I've done is ugly enough to not even push to Gerrit, heh https://github.com/stephenfin/nova/commit/b2c0421e64d627aaad2cf9ac71c6e1c79357a970 | |
| 15:02:23 | bauzas | looks like we're limited for describing resources or topologies | |
| 15:02:51 | bauzas | stephenfin: the problem is that oslo.config is pretty freeform | |
| 15:03:06 | bauzas | stephenfin: and IMHO the pci whitelist is a terrible interface | |
| 15:03:50 | stephenfin | Right, well I'd be interested in figuring out what you do. I don't think what I have is good enough so I'm open to ideas | |
| 15:04:09 | stephenfin | Until then, I'll probably revert back to a JSON option just so I can proceed | |
| 15:05:08 | jaypipes | stephenfin: dansmith is gonna hate that. | |
| 15:05:16 | mriedem | stephenfin: as in StrOpt with serialized json? | |