| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-06 | |||
| 10:45:07 | hrw | but pci(e) is added by libvirt not nova ;( | |
| 10:45:43 | bauzas | I need to disappear because $gym | |
| 10:45:52 | bauzas | but let's wrap that up when I'm back | |
| 11:04:51 | openstackgerrit | nalini proposed openstack/nova master: Modify show aggregate to display 'updated_at' value https://review.openstack.org/537334 | |
| 11:12:58 | openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: Provide support matrix and doc for VGPU https://review.openstack.org/540808 | |
| 11:29:13 | openstackgerrit | Chris Dent proposed openstack/nova-specs master: Add generation support in aggregate association https://review.openstack.org/540447 | |
| 11:30:19 | stvnoyes | mriedem: ok I'll look into it. Have you seen any problems or is your concern theoretical at the moment? | |
| 11:42:22 | openstackgerrit | Chris Dent proposed openstack/nova master: Remove microversion fallback code from report client https://review.openstack.org/528794 | |
| 11:43:25 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make bdms querying in multiple cells use scatter-gather https://review.openstack.org/541246 | |
| 12:39:10 | nsingh | andymccr: is there any framework to develop ansible plugin??? | |
| 12:52:54 | nsingh | evrardjp:is there any framework to develop ansible plugin??? | |
| 12:53:41 | evrardjp | what do you mean? | |
| 12:53:45 | evrardjp | nsingh: ^ | |
| 12:54:07 | evrardjp | ansible is the framework, you can simply use it for writing your own plugins... | |
| 12:54:26 | evrardjp | if you need stuff to be included in ansible and/or openstack-ansible, we can help you indeed | |
| 12:54:31 | evrardjp | depending on the time ofc | |
| 12:54:55 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | |
| 12:54:56 | openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 12:54:56 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | |
| 12:55:46 | nsingh | evrardjp:ok.. https://github.com/ansible/ansible is this the frmework you talking about? | |
| 12:56:16 | gibi | alex_xu, mriedem: I've fixed your comments ^^ | |
| 12:56:45 | evrardjp | nsingh: what are you trying to achieve? | |
| 12:58:17 | nsingh | evrardjp: i am new to ansible. i want to develop ansible plugin for project(masakari). | |
| 12:58:59 | evrardjp | to do what? | |
| 12:59:29 | evrardjp | I think we are probably not on the right channel :) | |
| 12:59:44 | nsingh | ok.. thanks | |
| 13:12:43 | sean-k-mooney | o/ jaypipes how is your new laptop treating you? | |
| 13:13:24 | sean-k-mooney | jaypipes: its rather early for you no? | |
| 13:17:08 | jaypipes | sean-k-mooney: no, 8am. new lappie is fantastic. :) | |
| 13:18:24 | sean-k-mooney | let me know how the battery life is that the main thing that has been putting me off getting one. | |
| 13:20:17 | sean-k-mooney | im looking at the new lg gram mainly for battery but i like system76 as a company so plus the galago looks like the perfect dev laptop if you travel even a little | |
| 13:23:52 | jaypipes | sean-k-mooney: it is excellent. and the damn keys are in the right place! /me shakes fist at Macbook. | |
| 13:24:04 | jaypipes | sean-k-mooney: also, the screen is fantastic. | |
| 13:24:29 | jaypipes | sean-k-mooney: and for $1100 including a 3-year warranty, you can't beat the price IMHO. | |
| 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 | |