Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-09
11:24:51 huanxie @johnthetubaguy Could you help review my patch for using os-xenapi lib https://review.openstack.org/#/c/406059/ https://review.openstack.org/#/c/411241/, thanks a lot
11:44:05 openstackgerrit sahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260
11:44:06 openstackgerrit sahid proposed openstack/nova: libvirt: ensure live-migration success https://review.openstack.org/409261
11:47:47 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Caches and synchronizes the traits from os_traits https://review.openstack.org/416007
11:47:48 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199
11:48:17 openstackgerrit Marco Voelz proposed openstack/nova-specs: Add port deletion on termination of server https://review.openstack.org/416997
11:51:42 openstackgerrit Jeremy Liu proposed openstack/nova: [3/3]Replace six.iteritems() with .items() https://review.openstack.org/415080
11:54:11 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Caches and synchronizes the traits from os_traits https://review.openstack.org/416007
11:54:12 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Import os_traits library https://review.openstack.org/377381
11:54:12 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199
11:54:13 openstackgerrit Alex Xu proposed openstack/nova: [POC][placement] Add Traits related table to the api database https://review.openstack.org/376198
12:04:07 openstackgerrit melanie witt proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775
12:17:02 melwitt superdan: FYI this change is needed in order to get use_neutron to default to False in devstack when using n-net for a local cells v1 environment https://review.openstack.org/#/c/416401/
12:18:03 melwitt so I had to rebase your devstack change locally to get a cells v1 env running
12:21:30 openstackgerrit Marco Voelz proposed openstack/nova-specs: Add port deletion on termination of server https://review.openstack.org/416997
12:33:16 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
12:56:45 openstackgerrit Li Yingjun proposed openstack/nova: Notifications for flavor operations https://review.openstack.org/398171
13:00:50 openstackgerrit Li Yingjun proposed openstack/nova: Update notification for flavor https://review.openstack.org/413846
13:19:49 openstackgerrit Pawel Koniszewski proposed openstack/nova: Add new configuration option live_migration_scheme https://review.openstack.org/410817
13:49:18 openstackgerrit Alex Szarka proposed openstack/nova: Transform instance.resize.error notifications https://review.openstack.org/417882
13:49:52 openstackgerrit Alex Szarka proposed openstack/nova: Transform instance.resize.error notifications https://review.openstack.org/417882
13:52:33 edleafe Scheduler subteam meeting in 7 minutes in #openstack-meeting-alt
13:58:38 openstackgerrit Alex Szarka proposed openstack/nova: Transform instance.resize.error notifications https://review.openstack.org/417882
14:28:01 superdan melwitt: ah cool, I just ended hacking around it, but that's better
14:30:31 diana_clarke superdan is going to be a sad panda when he realizes it's Monday, not Friday.
14:31:25 dims diana_clarke : LOL :)
14:33:24 kashyap diana_clarke: We should consider the corner case that it _is_ a "Yay! Monday" case for Dan?
14:33:55 diana_clarke kashyap: Only parents look forward to Mondays :)
14:34:28 kashyap :-)
14:44:57 openstackgerrit Gábor Antal proposed openstack/nova: Removed unnecessary parantheses and fixed formation https://review.openstack.org/370549
14:46:57 mtreinish mriedem: yeah, the logic for networks normally passes a specific network when creating a server
14:47:05 mtreinish we had to do that for the case with more than network being present
14:50:20 mtreinish mriedem: most of the tests call https://github.com/openstack/tempest/blob/master/tempest/test.py#L609 and use the network returned as the tenant_network arg in the create server call
14:54:36 mtreinish mriedem: there are tests that don't set that don't do that though. Like the auto allocate network test
15:03:58 lyarwood_ does anyone have an example stable/* local.conf to hand for devstack? It appears that the CI jobs on stable are using Zuul globals and not local.conf to pick this up.
15:21:57 openstackgerrit Christopher Martin proposed openstack/nova: Stop using mox in test_compute_cells.py https://review.openstack.org/389750
15:39:40 rfolco cdent, here?
15:40:13 cdent rfolco: am now, but seem to be having some network difficulty, so if I disappear, that's why
15:40:21 rfolco ack
15:41:00 rfolco cdent, do you think I should replace decorator with a custom version exception handler inside the new method ?
15:41:11 rfolco cdent, or returning 404 would be acceptable
15:41:27 cdent Yeah, I would just check the version and if it too low, raise a 405
15:41:57 cdent there's a few example of how to do that in other areas of the code
15:42:08 cdent if you can't find it let me now, I'll find a link
15:42:45 rfolco cdent, ok cool. Will remove the decorator then. Thanks for the tip.
15:43:42 openstackgerrit Radoslav Gerganov proposed openstack/nova: Add connection pool size to vSphere settings https://review.openstack.org/414085
15:44:14 cdent rfolco: sure, sorry for misleading you before. I was optimistic about being able to make the decorator more intelligent
15:46:57 openstackgerrit Julian proposed openstack/nova: Transform keypair.create notification https://review.openstack.org/408689
15:47:13 cdent but in the end too optimistic
15:47:27 bauzas gmrlrlbl, we don't have a ComputeNode.get_by_uuid() method...
15:47:42 bauzas and we don't have an host field for the ResourceProvider
15:47:45 bauzas graaaaah
15:48:06 rfolco cdent, well, I was able to hack how the decorator would return non-404, but that's ugly
15:48:20 bauzas so, no way to update the list of HostStates by keying (host, node) unless adding a new method :(
15:48:50 bauzas cdent: dansmith: for all the drivers but Ironic, host == node
15:49:37 bauzas cdent: dansmith: should I add a new DB call for getting the host name just because we have Ironic ? meh
15:51:53 cdent bauzas: I'm not understanding what you're trying to do? Are you saying: you have a list of compute nodes from placement api, now you want to make a collection of host states?
15:52:03 bauzas cdent: exactly
15:52:11 bauzas cdent: but we're keying per a tuple
15:52:18 bauzas cdent: which (host, node)
15:52:24 bauzas cdent: so I need to know the host value
15:52:47 cdent I believe jay has made some adjustment on that, at least in the resource tracker, but I'm not sure if it impacts where you're at
15:52:51 bauzas cdent: for 90% of our deployments, host == hypervisor_name
15:53:39 bauzas but given Ironic has host != hypervisor_hostname, I probably need to add a helper method in the ComputeNode object like compute_node.get_by_resource_provider(uuid)
15:53:56 bauzas or I could just get_by_uuid() because fortunately, cn.uuid == rp.uuid
15:54:15 bauzas but that sounds silly to DB lookup just because of Ironic
15:54:21 bauzas graaaah
15:55:25 bauzas cdent: and yeah, I know jay's cleanups but AFAICT, we're not changing yet that
15:56:24 cdent presumably he'll show up at some point today and we can get his input too.
16:24:33 cdent mriedem: could you remind me where you left the ptg planning etherpad?
16:24:45 mriedem cdent: https://etherpad.openstack.org/p/nova-ptg-pike
16:24:53 cdent thanks
16:27:42 openstackgerrit Christopher Martin proposed openstack/nova: Change soft delete resources to prevent DOS https://review.openstack.org/386756
16:30:43 mriedem ildikov: i won't be at the nova/cinder meeting probably, i'll be around but have about an hour today to get some stuff done before i have to catch a flight
16:31:00 mriedem ildikov: i'd say priority this week should be figuring out what's up with the endpoint 404 for using cinder v3
16:31:38 ildikov mriedem: tnx for the notice, safe travels!
16:31:47 ildikov mriedem: I didn;t have much time to play more with that
16:32:36 ildikov mriedem: I fixed the issue locally with hard coding the endpoint_template, but for instance the new attach calls in Cinder still acted weird while testing
16:32:55 smcginnis ildikov: Acted weird?
16:32:56 ildikov mriedem: like on call worked from the CLI, but not from Nova
16:33:02 ildikov *one
16:33:03 smcginnis Ah
16:33:17 mriedem well, which cli?
16:33:29 mriedem remember that latest novaclient doesn't proxy through to cinder anymore
16:33:34 ildikov I called attachment-create and I looked into the logs, the exact same looking call was 202 from the CLI and got 404 when Nova called it...
16:33:36 mriedem so things like listing attachments in novaclient aren't using cinder
16:33:41 ildikov Cinder CLI, sorry
16:33:55 ildikov the old Nova attach call worked fine
16:34:11 ildikov but the new call failed for at least seemingly no good reason
16:34:12 smcginnis ildikov: Well that's good. At least the existing stuff didn't break with it.
16:34:15 openstackgerrit Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961
16:34:15 openstackgerrit Sylvain Bauza proposed openstack/nova: WIP/DNM : Put all the magic in a PlacementClient https://review.openstack.org/417111
16:34:23 openstackgerrit Dan Smith proposed openstack/nova: WIP: Make metadata server know about cell mappings https://review.openstack.org/417962
16:34:23 openstackgerrit Dan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing https://review.openstack.org/396775
16:35:03 ildikov mriedem: smcginnis: in Devstack with admin cred I got 'Volume not found' and with demo cred I got 'Malformed request' error
16:35:44 ildikov it can still be some misconfig on my side, not saying it's not, but still looks odd
16:36:06 ildikov smcginnis: no, it seemed to work, but I didn't do any thorough testing on it either
16:36:18 mriedem idk, i guess if you're hitting just issues with using cinder (no nova involved) with v3 then i'd start opening bugs
16:36:22 openstackgerrit sean mooney proposed openstack/os-vif: introduces MTU support for vhost-user https://review.openstack.org/410135
16:36:35 ildikov smcginnis: and with just setting catalog_info my env failed like the logs on the gate for the switch to v3 patch

Earlier   Later