| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-06 | |||
| 13:25:40 | hrw | zuul... 4h from sending patch and still no results ;( | |
| 13:27:32 | sean-k-mooney | jaypipes: well not haveing to pay 23% vat on import certenly helps | |
| 13:29:23 | jaypipes | sean-k-mooney: heh, well I can't help you with that :) | |
| 13:29:47 | jaypipes | hrw: your patch hasn't even made it to the zuul birthing canal yet. | |
| 13:30:02 | jaypipes | hrw: contractions will likely begin in about another hour. | |
| 13:32:55 | hrw | jaypipes: looks like | |
| 13:47:22 | bauzas | hrw: so you had a point | |
| 13:47:38 | bauzas | about when nova is passing the XML to libvirt | |
| 13:47:46 | hrw | bauzas: yes | |
| 13:47:56 | bauzas | did you found it ? | |
| 13:48:07 | hrw | not yet. digged in other stuff | |
| 13:50:23 | hrw | bauzas: _create_domain() :D | |
| 13:50:54 | bauzas | there is another method AFAIR | |
| 13:51:35 | hrw | this one calls libvirt_guest.Guest | |
| 13:51:51 | bauzas | at least _create_domain_and_network() AFAIK | |
| 13:52:04 | hrw | _create_domain_and_network calls _create_domain | |
| 13:52:15 | bauzas | oh my bad yeah | |
| 13:52:37 | bauzas | for spawn at leasy | |
| 13:52:52 | hrw | bauzas: that's the only point I need :) | |
| 13:53:40 | bauzas | k | |
| 13:56:04 | openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | |
| 13:58:46 | ericyoung | We have started to see failures in the tempest.scenario.test_encrypted_cinder_volume tests with the ScaleIo backend. | |
| 13:58:57 | ericyoung | Any known issues with Luks encrypted volumes? | |
| 14:01:53 | ameeda | Hello, could you please give me another +2 here https://review.openstack.org/#/c/528385/ | |
| 14:03:51 | hrw | jaypipes: thanks for +2+W :) | |
| 14:04:45 | jaypipes | hrw: np. | |
| 14:05:11 | efried | Quick and easy approval: https://review.openstack.org/#/c/533237/ (just registering what PowerVM did in Q) | |
| 14:07:00 | jaosorior | anybody knows what actions trigger the compute.instance.update notification? | |
| 14:07:51 | gibi | jaosorior: hi | |
| 14:07:56 | gibi | jaosorior: let me dig a bit | |
| 14:08:26 | jaosorior | thanks gibi! | |
| 14:09:15 | ameeda | HELP: when I run .tox/functional/bin/python -m testtools.run nova.tests.functional.db.test_archive.TestDatabaseArchive.test_archive_deleted_rows | |
| 14:09:20 | ameeda | I got this error esttools.matchers._impl.MismatchError: 'instances' not in {'instance_system_metadata': 11, 'instance_actions_events': 2, 'block_device_mapping': 1, 'instance_info_caches': 1, 'instance_actions': 2, 'instance_extra': 1, 'security_group_instance_association': 1} | |
| 14:09:26 | ameeda | this caused after I wrote migration script for sqllite here https://review.openstack.org/#/c/526900/29/nova/db/sqlalchemy/migrate_repo/versions/378_instance_system_metadata_sqlite_upgrade.sql | |
| 14:09:38 | ameeda | what I did wrong to get that ? | |
| 14:13:16 | gibi | jaosorior: hard to answer in general. That notification is eventually emited from https://github.com/openstack/nova/blob/2c1874a0ecdd1b5ce7670cdfc42396e90e3a55aa/nova/notifications/base.py#L233 | |
| 14:14:02 | gibi | jaosorior: and tha is for example called for every action that changes a persistent attribute of an Instance object https://github.com/openstack/nova/blob/2c1874a0ecdd1b5ce7670cdfc42396e90e3a55aa/nova/objects/instance.py#L826 | |
| 14:14:11 | jaosorior | gibi: so, from what I can gather, updates are triggered by metadata changes (which is exactly what I need). but I wanted to know what else would trigger it | |
| 14:14:18 | jaosorior | I see | |
| 14:15:08 | gibi | jaosorior: yes, metadata update on the Instance also triggers this code path | |
| 14:17:44 | efried | stephenfin or gibi, care to push this? https://review.openstack.org/#/c/533237/ | |
| 14:18:37 | stephenfin | efried: Sure, done | |
| 14:18:42 | efried | Thanks! | |
| 14:20:48 | gibi | efried: I missed it :0 | |
| 14:20:49 | gibi | :) | |
| 14:21:36 | efried | gibi: Don't worry, I've got other patches to satisfy your cravings. | |
| 14:24:19 | bauzas | mriedem: are we considering https://bugs.launchpad.net/nova/+bug/1747582 as a Won't Fix ? | |
| 14:24:20 | openstack | Launchpad bug 1747582 in OpenStack Compute (nova) "delete a resize instance change quota_usages incorrectly" [Undecided,New] - Assigned to guolidong (guolidong) | |
| 14:24:34 | bauzas | I mean, we did a huge work on quotas thanks to melwitt after Newton | |
| 14:25:59 | openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: Provide support matrix and doc for VGPU https://review.openstack.org/540808 | |
| 14:28:58 | jianghuaw | bauzas, ^ | |
| 14:29:15 | bauzas | jianghuaw: cool, will review it as soon as I can | |
| 14:29:22 | jianghuaw | bauzas, thanks. | |
| 14:34:00 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 14:45:28 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec https://review.openstack.org/541290 | |
| 14:45:47 | stephenfin | sean-k-mooney: It's very rough, but fancy taking a look at ^ at some point? | |
| 14:46:06 | stephenfin | efried: You might be able to weight in from the generic device management aspect too ^ | |
| 14:46:18 | mriedem | bauzas: depends on if it's recreatable in master | |
| 14:46:22 | efried | ack | |
| 14:46:33 | bauzas | mriedem: just told that actually | |
| 14:46:34 | stephenfin | Try as I may, I can't think of a reason to need that, but I'm clearly missing something as both you and sean-k-mooney suggested it :) | |
| 14:46:36 | bauzas | hence Incomplete | |
| 14:48:41 | sean-k-mooney | stephenfin: reason to need ? | |
| 14:50:12 | sean-k-mooney | stephenfin: also first taught on https://review.openstack.org/541290 is it should proably be an neutron spec but i have only got to line 10 so ill keep reading | |
| 14:55:00 | sean-k-mooney | stephenfin: of were you refering to a need for numa-aware-vswitches when you said you can think why you need that? | |
| 15:05:15 | mriedem | bauzas: can you review https://review.openstack.org/#/c/539739/ please? it's an rc1 blocker due to a regression in the native encrypted luks stuff, blocking some cinder 3rd party CIs | |
| 15:05:33 | bauzas | mriedem: roger. | |
| 15:05:47 | bauzas | LUKS, I am your father. | |
| 15:11:31 | edleafe | bauzas: <groan!> | |
| 15:12:04 | bauzas | that's possibly one of the top said jokes, I know | |
| 15:16:31 | openstackgerrit | Lajos Katona proposed openstack/nova master: WIP: ServerMovingTests with custom resources https://review.openstack.org/497399 | |
| 15:31:19 | openstackgerrit | Ed Leafe proposed openstack/nova master: Make the InstanceMapping marker UUID-like https://review.openstack.org/539323 | |
| 15:41:31 | stephenfin | sean-k-mooney: Yup, keep reading. It seems like a nova thing in that current design :) | |
| 15:41:39 | stephenfin | though you may have left comments in the interim | |
| 15:52:22 | sean-k-mooney | stephenfin: well im commenting back. the current design will not work unfrotunetly for example have you taught about how you handel bonds. | |
| 15:58:14 | sean-k-mooney | stephenfin: sorry in advance https://review.openstack.org/#/c/541290/1 comments in line | |
| 15:58:23 | stephenfin | sean-k-mooney: :) | |
| 15:59:33 | sean-k-mooney | stephenfin: by the way where did the 50% performanc number come from? the numa effect on ovs-dpdk should be in the singel figure if you turn on the numa support in dpdk | |
| 16:00:14 | stephenfin | sean-k-mooney: I was wondering if anyone was going to call me on that. It's been quoted numerous times to me by fbaudin, but I think it's based on internal analysis | |
| 16:00:43 | stephenfin | I can root out the details and add them in some form to the spec | |
| 16:00:50 | stephenfin | ...which clearly needs more work :) | |
| 16:01:20 | sean-k-mooney | stephenfin: ya but i have mention to him in the past that we have messured much lower numbers in the past if you turn on the numa support in dpdk/ovs which is off by default | |
| 16:03:22 | stephenfin | sean-k-mooney: Thanks, I wasn't aware of that. There's some great info in there | |
| 16:03:33 | stephenfin | sean-k-mooney: You aren't, by any chance, going to be in Dublin, are you? | |
| 16:04:25 | sean-k-mooney | i proably will be. im not sure if i will be there on behalf of intel though as my travel has not been approved so i might not be the there the whole week if i go up myself | |
| 16:14:39 | sean-k-mooney | stephenfin: just looking at the at the dpdk-devel packages for centos and fedora and they appear to have the the numa support turned on. and the ovs packages depend on libnuma so they "should" have numa support but if the testing was not done with at least 1 pmd per numa node then ovs will not be able to compensate internally if the vm and nic are not on the same numa node | |
| 16:15:26 | stephenfin | sean-k-mooney: Could you stick that in the review too? This is all good data to use while working on this | |
| 16:15:48 | stephenfin | sean-k-mooney: Planning on setting up an AIO deployment this evening to start hacking on this and see what I can get working | |
| 16:16:04 | efried | sean-k-mooney: Are you from Dublin originally? | |
| 16:16:41 | sean-k-mooney | stephenfin: yep will do. if you use networking-ovs-dpdk/devstack just be aware we dont acally compile in the ovs support you will have to add it in but patches are welcome. | |
| 16:16:44 | sahid | mriedem can you ack this https://review.openstack.org/#/c/515008/, it's a bugfix | |
| 16:17:18 | sean-k-mooney | efried: no i lived there for a year im on the other cost in shannon but its only about a 3 hour drive or so | |
| 16:17:44 | efried | I thought pronouncing 'th' as 't' was a Dublin thing. | |
| 16:18:07 | stephenfin | sean-k-mooney: You mean you don't compile in DPDK support...right? | |
| 16:18:29 | stephenfin | If not, what about compiling in? | |
| 16:18:36 | stephenfin | efried: For example? | |
| 16:18:49 | efried | stephenfin: "taught" for "thought" | |
| 16:19:26 | sean-k-mooney | efried: they pronounce 'th' more like a 'd' but 'th' as a 't' is more of a tipperary thing that i generally avoid doing in speach | |
| 16:20:06 | stephenfin | Ah, no. My father, who's also from the west coast, does that too | |
| 16:20:07 | stephenfin | One, two, tree... | |