| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-17 | |||
| 14:13:32 | mdbooth | mriedem: I'm still banging on https://review.openstack.org/#/c/383859/ btw | |
| 14:13:32 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | |
| 14:14:41 | mdbooth | I just updated it. sfinucan had asked for some docs, and I noticed an additional potential race while I was writing them. I've fixed the race and added docstrings. | |
| 14:15:17 | mdbooth | Jenkins thinks it's good, and I've run the updated patch against devstack nfs, which still passes. | |
| 14:17:12 | lyarwood | mdbooth: quick nit on that review around your use of os.path.ismounted | |
| 14:17:39 | mdbooth | The race was: if we lose and regain our libvirt connection, when we re-initialise the mount state we couldn't be certain that there were no ongoing operations from before we lost the connection. | |
| 14:17:40 | lyarwood | mdbooth: still looking at the rest, started there given the other change I'm working on | |
| 14:18:06 | mdbooth | lyarwood: Thanks for looking! | |
| 14:19:46 | tdurakov | johnthetubaguy: hi, are you around? | |
| 14:20:19 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | |
| 14:23:06 | mdbooth | lyarwood: That raises a follow-on question about is_mounted, btw. | |
| 14:23:23 | mdbooth | I'm going to tread carefully around that rabbit hole :) I don't want to go down it. | |
| 14:24:49 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | |
| 14:25:14 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494 | |
| 14:25:38 | mriedem | mdbooth: ack. are you ok with the latest version of https://review.openstack.org/#/c/312488/ now? | |
| 14:26:00 | lyarwood | mdbooth: I assume your going to ask if it did that check before right? | |
| 14:26:06 | lyarwood | you're* | |
| 14:26:27 | mdbooth | mriedem: Just checking again. I was nearly ok with with it last time iirc. | |
| 14:27:15 | mdbooth | lyarwood: I responded to your comment. | |
| 14:28:23 | lyarwood | mdbooth: yup good point | |
| 14:30:28 | diana_clarke | mriedem: Let me know if there is anything you want help with, given ice day and all. In particular, is there anything I can help with wrt what tripleo needs from nova? | |
| 14:30:56 | diana_clarke | man pages etc | |
| 14:31:31 | mriedem | ok, hm | |
| 14:31:53 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | |
| 14:31:54 | mriedem | it would be nice to have man page entries for the cells v2 commands | |
| 14:32:16 | mriedem | in here http://docs.openstack.org/developer/nova/man/nova-manage.html | |
| 14:32:45 | diana_clarke | mriedem: yeah, I saw the ask in the nova-cells-v2 wiki, which is why I asked. I no one is doing that, I can take that on. | |
| 14:33:03 | diana_clarke | typo: *if* no one is doing that... | |
| 14:33:17 | mriedem | i don't think anyone is working on that so it's all yours | |
| 14:33:35 | mriedem | you might want to do them one at a time in a series just to make reviews easier | |
| 14:34:02 | diana_clarke | mriedem: okie, dokie. and I'll update the wiki, claiming that task | |
| 14:34:42 | mriedem | we might want to get https://review.openstack.org/#/c/420442/ merged to avoid conflicts too | |
| 14:34:50 | mriedem | sdague: super simple change https://review.openstack.org/#/c/420442/ | |
| 14:35:17 | sdague | mriedem: bam | |
| 14:37:36 | mriedem | thanks emeril | |
| 14:38:25 | mriedem | sean-k-mooney: do you know if we can re-use _get_vif_name in here https://review.openstack.org/#/c/410737/ ? i.e. is the device field set on that type of vif for vhostuser? | |
| 14:39:44 | mriedem | i guess the vif does have a devname, "devname":"tap96429488-f7" | |
| 14:39:50 | mriedem | but it's not what's needed here i guess | |
| 14:43:24 | markus_z | tdurakov: sorry, I missed the meeting | |
| 14:44:03 | tdurakov | markus_z: ping me once update the hook | |
| 14:45:06 | markus_z | tdurakov: I pushed the teardown pkoniszewski asked for in ps42. https://review.openstack.org/#/c/347471/ | |
| 14:45:21 | tdurakov | acked, let me check it again | |
| 14:45:28 | markus_z | thanks! | |
| 14:49:23 | sean-k-mooney | mriedem just looking now | |
| 14:50:45 | sean-k-mooney | mriedem: looking at the code the vhost-user socket name must = the name of the openvswitch port | |
| 14:51:27 | sean-k-mooney | mriedem: if we called _get_vif_name() we would have to modify neutron to generate the socket path differently | |
| 14:52:19 | sean-k-mooney | my intention was to modify neutron to pass the name of the device in the port binding | |
| 14:52:36 | sean-k-mooney | and then nave nova use that if present or fall back to generating it if not set | |
| 14:54:32 | mriedem | ok so in other words this is fine | |
| 14:55:23 | sean-k-mooney | mriedem: yes what is there is currently fine but i would like to decouple it in the future so that we dont have to ensure that neutron and nova generate the name with the same algoritim | |
| 14:56:06 | tdurakov | markus_z: done, thank you | |
| 14:56:24 | markus_z | tdurakov: yeah, thanks! | |
| 14:56:32 | tdurakov | mriedem, sdague hi, could you please take a look https://review.openstack.org/#/c/347471/43 | |
| 14:57:27 | jaypipes | sdague: well said, on the PWG thing. I won't say that publicly, of course, since I'd get my head chewed off internally. | |
| 14:58:43 | cdent | jaypipes: you must have very flexible teeth or jaw | |
| 15:03:18 | mriedem | sean-k-mooney: ok i've also gone through https://review.openstack.org/#/c/385061/ | |
| 15:10:29 | mriedem | alex_xu: your intel buddies might want to look at this https://review.openstack.org/#/c/420960/1/nova/objects/fields.py | |
| 15:10:36 | mriedem | pkoniszewski: ^ | |
| 15:12:33 | cdent | _gryf: I think that endpoint stuff is simply that I hadn't revisited that update_inventory code in a long time (perhaps 4 months?) | |
| 15:13:54 | _gryf | cdent, ok, cool | |
| 15:22:21 | alex_xu | mriedem: that sounds like that overlap with traits. in the future we can request specific trait in the image metadata, the requested trait can be cpu feature. there is the spec about request traits https://review.openstack.org/#/c/351063/3/specs/ocata/approved/request-standardized-capabilities.rst@66 | |
| 15:27:05 | pkoniszewski | thanks mriedem, i will take a look | |
| 15:31:50 | cdent | edleafe: I think you're on PTO, but if not I fixed that bug in gabbi and made a new one: https://pypi.python.org/pypi/gabbi | |
| 15:34:13 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: add cells commands prep https://review.openstack.org/421320 | |
| 15:48:26 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method https://review.openstack.org/418134 | |
| 15:48:26 | openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 15:48:58 | bauzas | mriedem: jaypipes: folks I need to bail out for a few, but you can bite ^ | |
| 15:49:09 | openstackgerrit | Vladyslav Drok proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | |
| 15:53:59 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 simple_cell_setup https://review.openstack.org/421333 | |
| 15:54:34 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | |
| 15:56:39 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | |
| 15:57:50 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | |
| 16:08:16 | mriedem | tdurakov: markus_z: for https://review.openstack.org/#/c/347471/ i see it depends on a tempest patch to enable the test, do we have a patch up anywhere that depends on the nova change to see it all working and running the serial console tests in the live migration job? | |
| 16:08:43 | mriedem | or is it self-testing? | |
| 16:09:02 | markus_z | mriedem: it's self testing | |
| 16:09:31 | mriedem | oh really http://logs.openstack.org/71/347471/43/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/c427a53/console.html#_2017-01-17_13_10_19_246017 | |
| 16:09:59 | markus_z | http://logs.openstack.org/71/347471/43/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/c427a53/console.html#_2017-01-17_13_10_22_840459 | |
| 16:10:45 | markus_z | We have to skip in in grenade. Let me search for the Ocata-Ocata case. | |
| 16:11:04 | mriedem | oh right | |
| 16:11:17 | mriedem | this is the o-o one http://logs.openstack.org/71/347471/43/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/0b45362/console.html#_2017-01-17_12_34_13_290332 | |
| 16:11:21 | mriedem | still skipped | |
| 16:11:52 | mriedem | melwitt: i'm struggling trying to get discover_hosts to work on the subnode in a multinode job http://logs.openstack.org/81/420981/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/6767a3a/logs/subnode-2/devstacklog.txt.gz#_2017-01-17_14_49_15_138 | |
| 16:11:59 | mriedem | melwitt: please tell me there is something you know about this that i don't | |
| 16:12:11 | mriedem | the subnode nova.conf has the api_database connection and we're getting the cell mappings from the api db ok, | |
| 16:12:23 | mriedem | it's the target_cell that's failing on the non-cell0 cell | |
| 16:12:37 | mriedem | for some reason it's trying to connect to 127.0.0.1 | |
| 16:13:13 | markus_z | mriedem: That was the first run before the hook enables the serial console. After the hook we have http://logs.openstack.org/71/347471/43/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/0b45362/console.html#_2017-01-17_12_35_19_716558 | |
| 16:13:56 | markus_z | mriedem: The teardown of the hook sets everything back, so we have http://logs.openstack.org/71/347471/43/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/0b45362/console.html#_2017-01-17_12_36_38_329078 | |
| 16:20:19 | shaner | hi all - any idea why live-migrations won't show up when issuing 'migrations-list'? | |
| 16:20:30 | shaner | (in kilo) | |
| 16:21:07 | shaner | only those that were migrated using 'migrate' show up .. vs 'live-migration' | |
| 16:25:31 | jaypipes | shaner: pls see /topic. best to ask on ask.openstack.org or mailing list for usage questions. | |
| 16:25:53 | shaner | jaypipes, thx, will do | |
| 16:26:23 | mriedem | migrate is cold migration | |
| 16:27:27 | openstackgerrit | int32bit proposed openstack/python-novaclient: Allow multiple tag add/delete from cli https://review.openstack.org/414635 | |
| 16:28:12 | shaner | mriedem, shouldn't 'migration-list' list all types of migrations though? | |
| 16:28:59 | mriedem | i don't think until 2.23 http://docs.openstack.org/developer/nova/api_microversion_history.html#id21 | |
| 16:29:23 | mriedem | fwiw this is the api code in kilo https://github.com/openstack/nova/blob/kilo-eol/nova/api/openstack/compute/contrib/migrations.py#L53 | |
| 16:30:00 | openstackgerrit | int32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space https://review.openstack.org/402632 | |
| 16:36:36 | mriedem | shaner: was the migration in progress or complete? if the migration was complete i wonder if the migration record was deleted | |