| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-04 | |||
| 09:13:44 | kashyap | cfriesen: Responded on the review | |
| 09:14:58 | pvc | gibi Filter PciPassthroughFilter returned 0 hosts | |
| 09:21:38 | kashyap | mdbooth: Morning | |
| 09:21:52 | kashyap | mdbooth: Was trying to debug this only failure: http://logs.openstack.org/58/567258/4/check/openstack-tox-py35/727a7d4/testr_results.html.gz | |
| 09:22:02 | kashyap | (Context patch: https://review.openstack.org/#/c/567258/) | |
| 09:23:58 | mdbooth | kashyap: Looking | |
| 09:24:19 | kashyap | Thank you | |
| 09:24:41 | kashyap | mdbooth: So, it is blowing up here: nova/tests/unit/virt/test_virt_drivers.py --> test_live_migration() | |
| 09:25:13 | mdbooth | kashyap: Ah, this is syntactic | |
| 09:25:24 | mdbooth | Hopefully an easy one :) | |
| 09:25:33 | kashyap | Oh, really? | |
| 09:26:06 | kashyap | Please enlighten, /me is not feeling his best today | |
| 09:26:08 | mdbooth | kashyap: Those words normally precede a severe humbling | |
| 09:26:14 | mdbooth | So we'll see | |
| 09:26:44 | kashyap | mdbooth: If you want to apply the patch locally, here is a convenient: https://kashyapc.fedorapeople.org/0001-libvirt-Remove-usage-of-migrateToURI-2-APIs.patch | |
| 09:26:47 | gibi | pvc: then something is wrong with your pci setup | |
| 09:27:00 | kashyap | You can `git am` it, if you like. | |
| 09:27:58 | pvc | gibi on what side | |
| 09:28:08 | mdbooth | kashyap: git review -d is fine | |
| 09:28:10 | gibi | pvc: that is a good question :) | |
| 09:28:27 | kashyap | mdbooth: Yep, I find it a tad slower, though. But only a few seconds :-) | |
| 09:28:29 | pvc | i already enabled VT-d and VT-x | |
| 09:33:50 | openstackgerrit | Chen proposed openstack/nova master: Fix a typo https://review.openstack.org/572061 | |
| 09:37:42 | openstackgerrit | tianhui proposed openstack/nova master: Fix bug for hypervisors https://review.openstack.org/572063 | |
| 09:47:33 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505 | |
| 09:50:00 | jangutter | sahid: would you like me to build a lower-level unit test specifically for _set_config_VIFGeneric (re https://review.openstack.org/571461 )? I can do that. | |
| 09:57:56 | mdbooth | kashyap: Found it. | |
| 09:58:03 | mdbooth | kashyap: Ok, that was obtuse :) | |
| 09:58:11 | kashyap | mdbooth: I am still wading through PDB | |
| 09:58:27 | kashyap | Thanks for taking time to debug :-) | |
| 09:58:48 | kashyap | Right now I have debugger here: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7347 | |
| 10:02:21 | mdbooth | kashyap: Commented | |
| 10:02:28 | kashyap | Thanks! | |
| 10:03:02 | mdbooth | kashyap: I eventually resorted to grep :/ | |
| 10:03:18 | mdbooth | Python was hiding that mock extremely well. | |
| 10:03:44 | kashyap | :-) Still reading | |
| 10:03:47 | mdbooth | I couldn't even work it out from the debugger, even though I was almost 100% sure that guest.migrate was being mocked. | |
| 10:04:54 | kashyap | mdbooth: If you couldn't work it out...then I had little hope :P | |
| 10:10:47 | kashyap | mdbooth: So, this part was dodgy: | |
| 10:10:48 | kashyap | 136 lambda self, destination, migrate_uri, params, flags, | |
| 10:10:48 | kashyap | 135 self.stub_out('nova.virt.libvirt.guest.Guest.migrate', | |
| 10:10:51 | kashyap | 137 domain_xml, bandwidth: None) | |
| 10:11:11 | mdbooth | kashyap: Yep. You need to pop the domain_xml argument from that lambda. | |
| 10:11:30 | kashyap | Yep, now that I've removed it | |
| 10:11:47 | kashyap | Huh, "obtuse" was the apt word I was looking for. Thanks | |
| 10:11:59 | kashyap | I'll also see about the 'params' dict comments | |
| 10:12:45 | mdbooth | It was buried in an unrelated file in a function called by the setUp of a Grandparent class. | |
| 10:15:46 | mdbooth | Might do better to convert those few to Mock with an autospec | |
| 10:16:24 | mdbooth | I believe that will behave the same, but will automatically copy the argument signature of the function being mocked | |
| 10:16:55 | kashyap | mdbooth: Probably in separate change? | |
| 10:17:09 | mdbooth | Although tbh it smells that we're looking so deep inside the libvirt driver in that test anyway | |
| 10:17:15 | kashyap | (Prefer to avoid fixing everything in this one change) | |
| 10:17:30 | mdbooth | kashyap: Indeed. It would obviously have to be a preceding change, though. | |
| 10:18:25 | kashyap | mdbooth: Yeah, noted. Thanks, you saved my Monday morning | |
| 10:23:50 | kashyap | mdbooth: You went looking for 'domain_xml' via `grep`? | |
| 10:24:07 | mdbooth | kashyap: Yep :( | |
| 10:24:23 | mdbooth | kashyap: Was looking for mocks including domain_xml | |
| 10:24:40 | kashyap | I see | |
| 10:24:50 | mdbooth | I was pretty sure that migrate() was being mocked and I just couldn't find it. | |
| 10:25:41 | mdbooth | kashyap: Disappointingly brute force. | |
| 10:26:02 | kashyap | Yeah, I was stupefied. Embarassed that I didn't do the `grep` :-) | |
| 10:26:03 | sahid | jangutter: i think you can just add an assertion on the vif_name value | |
| 10:26:10 | sahid | is that not enough? | |
| 10:28:37 | kashyap | sahid: Hey, thanks for the review | |
| 10:30:00 | kashyap | sahid: Are you really -1 on the 'params' thing? | |
| 10:30:50 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258 | |
| 10:31:06 | kashyap | sahid: Haven't addressed your comment yet, still thinking and playing locally. | |
| 10:31:12 | sahid | seems reasonable yes, it's convention we have. unfortunatly for that method we had two handle several prototypes but now that you are remove the old method that looks reasonable | |
| 10:31:26 | sahid | s/two/to | |
| 10:32:41 | sahid | kashyap: that should not be so difficult to change, i'm ok if you do that in a next patch | |
| 10:33:02 | kashyap | sahid: Yeah, I am not opposed to it. But you make a good point. So still considering. | |
| 10:33:19 | kashyap | sahid: Would prefer to change it in a next patch, if that's alright | |
| 10:34:26 | sahid | kashyap: ok so please make it to happen and i will review both patch. since that does not really make sence to generate the params dict for only bandwidth and migrate_uri | |
| 10:35:49 | sahid | ok... if you don't want do it now, adding a todo note seems the thing to do | |
| 10:36:01 | sahid | s/ok/or | |
| 10:36:09 | kashyap | sahid: No worries, I'll make a second patch | |
| 10:36:23 | kashyap | sahid: And post it as a two-patch series. | |
| 10:37:00 | kashyap | sahid: Thanks for looking | |
| 10:43:31 | kashyap | Maybe it makes more sense to do it all in one change. | |
| 11:21:07 | pvc | Filter PciPassthroughFilter returned 0 hosts | |
| 11:32:16 | openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add support for generic offload representors https://review.openstack.org/572081 | |
| 11:32:25 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | |
| 11:32:30 | openstackgerrit | Jan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif https://review.openstack.org/571325 | |
| 11:32:31 | openstackgerrit | Jan Gutter proposed openstack/nova master: [WIP] Add support for vrouter HW offloads https://review.openstack.org/572082 | |
| 11:49:39 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375 | |
| 11:49:40 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401 | |
| 11:49:41 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166 | |
| 11:49:42 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465 | |
| 11:49:43 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | |
| 12:07:48 | openstackgerrit | Damon Li proposed openstack/nova master: Remove minimum version check when save service https://review.openstack.org/572089 | |
| 12:13:37 | openstackgerrit | Damon Li proposed openstack/nova master: Remove minimum version check when save service https://review.openstack.org/572089 | |
| 12:26:55 | openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add support for generic offload representors https://review.openstack.org/572081 | |
| 12:32:24 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148 | |
| 13:28:53 | mike99201 | does nova-conductor call transport_url from anywhere other than nova.conf or is it possibly cached somewhere? | |
| 13:31:48 | naichuans | jaypipes: Hi, jay. I have a disscuss with Eric on a vgpu n-rp patch:https://review.openstack.org/#/c/520313/40/nova/virt/xenapi/driver.py. Could you review it and give some suggestion when you have the time? | |
| 13:32:15 | jaypipes | naichuans: yep! | |
| 13:32:36 | naichuans | jaypipes: Thanks! | |
| 13:33:03 | jaypipes | mike99201: it's cached in the API database, unfortunately: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L118 | |
| 13:33:44 | mike99201 | could that be a reason that conductor service starts find and connects to rabbitmq, but when i launch an instance i'm getting invalid credentials when connecting to rabbitmq? | |
| 13:33:57 | mike99201 | i've changed the password between the time the database was setup and now | |