Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-20
22:10:33 openstackgerrit Merged openstack/nova master: Move the dac_admin privsep code to a new location. https://review.openstack.org/504194
22:12:13 openstackgerrit Eric Fried proposed openstack/nova master: Restore [vnc]vnc*; remove [DEFAULT] equivalents https://review.openstack.org/505831
22:12:26 efried mriedem stephenfin ^
22:14:30 openstackgerrit Merged openstack/nova master: Squash dac_admin privsep context. https://review.openstack.org/504195
22:14:55 efried cdent too, as a reviewer of the original.
22:15:02 openstackgerrit Merged openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
22:15:40 cdent efried: I’m too asleep to give that a good review now, and I guess must have been too asleep when I reviewed it the first time
22:15:50 efried You're not alone :)
22:16:11 mriedem efried: commented
22:16:38 cdent I’ve put it on my list for tomorrow
22:17:35 mriedem dansmith: you hate methods with 100 kwargs?!
22:17:45 dansmith mriedem: yes, yes I do
22:17:52 dansmith mriedem: especially when they're ALL optional
22:17:55 mriedem like _provision_instances, build_instances, build_and_run_instance, rebuild_instance
22:17:56 mriedem ETC?!
22:18:03 dansmith yeah, all of ours
22:18:36 mriedem does java allow default values yet?
22:19:15 dansmith heh idk
22:19:18 mriedem guess not
22:19:45 mriedem i remember missing that when going from c++ to java, and then being pleasantly surprised to see it again when going from java to python
22:20:07 mriedem now, what would go do
22:20:44 tasker require an interface.
22:21:22 openstackgerrit Merged openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/501343
22:21:53 dansmith tasker: I dunno if you're making fun of python or go with that statement, but... don't tell me :)
22:21:56 openstackgerrit Merged openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025
22:22:15 tasker . D
22:22:30 openstackgerrit Merged openstack/nova master: Squash dacnet_admin privsep context. https://review.openstack.org/504805
22:22:46 mriedem now someone needs to backport all of these https://review.openstack.org/#/q/topic:ironic-offline-migration+(status:open+OR+status:merged)
22:25:54 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
22:26:08 efried mriedem Replied :(letting you know since we don't seem to be getting emails):
22:28:01 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
22:28:04 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
22:49:44 mriedem i haven't read my gerrit email in a looong time
22:49:49 mriedem couldn't keep up with it
22:57:39 dansmith yeah, gerrit emails are borked today, known issue
23:03:27 dansmith here comes
23:03:42 openstackgerrit Dan Smith proposed openstack/nova master: Make instance_list honor global query limit https://review.openstack.org/504984
23:03:42 openstackgerrit Dan Smith proposed openstack/nova master: Add base implementation for efficient cross-cell instance listing https://review.openstack.org/504983
23:03:43 openstackgerrit Dan Smith proposed openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986
23:03:43 openstackgerrit Dan Smith proposed openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985
23:03:44 openstackgerrit Dan Smith proposed openstack/nova master: Add tests to validate instance_list handles faults correctly https://review.openstack.org/505392
23:03:44 openstackgerrit Dan Smith proposed openstack/nova master: Add fault-filling into instance_get_all_by_filters_sort() https://review.openstack.org/505391
23:03:45 openstackgerrit Dan Smith proposed openstack/nova master: Copy some tests to a cellsv1 mixin https://review.openstack.org/505442
23:03:45 openstackgerrit Dan Smith proposed openstack/nova master: Add get_instance_objects_sorted() https://review.openstack.org/505417
23:03:46 openstackgerrit Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418
23:03:46 openstackgerrit Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661
23:03:47 openstackgerrit Dan Smith proposed openstack/nova master: Remove legacy fault-loading routines https://review.openstack.org/505456
23:12:02 cfriesen from what I understand nova-api handles both the normal API endpoint and the metadata endpoint...how does the multiple calls to launcher.launch_service() end up using the same process(es)?
23:31:15 tasker did review.openstack just break?
23:34:00 tasker nevermind - i can get to it again.
23:56:51 openstackgerrit Chris Dent proposed openstack/nova master: Move project_id and user_id to Allocation object https://review.openstack.org/500410
23:56:52 openstackgerrit Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
23:56:52 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
#openstack-nova - 2017-09-21
00:39:31 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
00:41:48 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
00:41:52 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
01:04:58 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146
01:06:41 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient stable/pike: Updated from global requirements https://review.openstack.org/493187
01:23:37 mriedem owalsh: whenever you get a sec, if you know how to configure nova and cells v2 for clustered rabbit with the cell transport_url, it would be cool if you could help sort this out https://bugs.launchpad.net/nova/+bug/1717915
01:23:38 openstack Launchpad bug 1717915 in oslo.messaging "nova services and transport_url, cannot connect to vhost if specified" [Undecided,New]
01:30:01 openstackgerrit jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390
01:33:08 openstackgerrit jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492
01:36:21 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Split out the core of the ironic flavor migration https://review.openstack.org/505901
01:36:22 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/505903
01:36:22 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/505902
01:45:44 openstackgerrit jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485
02:01:43 openstackgerrit Merged openstack/nova master: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829
02:01:59 openstackgerrit jichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID https://review.openstack.org/505471
02:36:32 masber good morning all, I would like to run docker containers through nova, my environment is based on Newton. Is this the guide I need to follow? https://wiki.openstack.org/wiki/Docker
03:07:31 gmann jichen: sorry about api-ref comment which is not valid - https://review.openstack.org/#/c/505485/
03:07:45 gmann jichen: your PS3 was good to go
03:08:19 jichen gmann: ok, got it, you are right, I need keep 400 as validation stuff, thanks :)
03:09:48 gmann jichen: thanks
03:10:55 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
03:13:05 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
03:13:08 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
03:15:30 openstackgerrit jichenjc proposed openstack/nova master: Remove 400 as expected error https://review.openstack.org/505485
03:27:33 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
03:29:38 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
03:29:42 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
03:45:56 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/505839
03:48:02 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
03:48:05 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/502708
03:54:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
04:05:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
05:27:32 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029
05:36:26 openstackgerrit Hironori Shiina proposed openstack/nova-specs master: Ironic: Resize and cold migration support https://review.openstack.org/449155
05:50:47 jichen gmann: could you please help check https://review.openstack.org/#/c/505485/ ? I think gerrit has some issue
05:51:08 jichen so PS5's update didn't take effect and your PS4 -1 is still there..
05:52:29 gmann jichen: done.
05:52:38 jichen gmann: thanks~
05:53:46 gmann jichen: about your mail for PUT metadata/key . i thought you are asking about PUT metadata by key and will it allow to update more than 1 key or not
05:54:14 gmann jichen: if you see my reply actually that is handled in schema
05:55:40 gmann the schema and delete flag is actually very kind of hidden in that and does all magic
05:56:04 jichen yes, that's also my question , I just saw the mail, yes, exactly, it's hidden in the code logic and I missed that
05:56:33 jichen gmann: thanks for point that out and I think that's a little bit hack, we have update_all which can handle one key situation
05:57:11 jichen I mean we have update_all already and update one key like the API implemented looks to me seems useless and confusing
05:57:51 gmann jichen: but both have separate purpose, update_all remove the existing keys and update by key does not
05:58:18 jichen gmann: ah...ok , you are right
05:59:11 jichen thanks for the info, I will check the code again , thanks~

Earlier   Later