Earlier  
Posted Nick Remark
#openstack-nova - 2017-03-22
09:11:44 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/cells/test_cells_rpc_driver.py https://review.openstack.org/448457
09:19:35 gmann alex_xu: nice but few comment please check - https://review.openstack.org/#/c/338051/13
09:20:05 gmann min_version not shown on generated doc
09:20:19 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add description to policies in keypairs.py https://review.openstack.org/448461
09:20:40 alex_xu gmann: thanks, let me check
09:26:46 openstackgerrit Alex Xu proposed openstack/nova master: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051
09:27:10 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/cells/test_cells_rpc_driver.py https://review.openstack.org/448457
09:35:42 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add description to policies in lock_server.py https://review.openstack.org/448468
09:38:38 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992
09:42:21 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
09:42:21 openstackgerrit Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
09:47:10 lyarwood mdbooth / johnthetubaguy ; morning, time to pester you both for reviews on https://review.openstack.org/#/c/439520/ - FYI I'm out for a week after today.
09:47:28 johnthetubaguy lyarwood: dang, I remember you mentioning that now
09:47:50 johnthetubaguy clearly I am jealous and blanked that.
09:52:21 lyarwood johnthetubaguy: haha, I'm going to be out for at least one day each week for the next 7 weeks btw. I'm sure I'll be the jealous one soon enough once I have no holiday left to take in the summer :|
09:53:46 johnthetubaguy lyarwood: ouch, yeah, thats rough.
09:57:43 lyarwood jgriffith: re https://review.openstack.org/#/c/438750/ - did you want to open this back up? AFAIK the plan was to use this change to introduce the new detach logic on top of I7a53e08f3fad6abb27a1d8ad425b4f916341cab3 & Ieb43099a31689e1b509cbffb9aeb7cc9948f1a18 right?
10:01:05 gmann alex_xu: sorry i gave again -1
10:01:52 gmann alex_xu: 1 sec
10:04:46 gmann alex_xu: if on last then we need it separate heading as currently it is under volume APIs.
10:07:01 alex_xu gmann: thanks
10:07:10 alex_xu gmann: is it ok move under crash-dump?
10:09:14 openstackgerrit Alex Xu proposed openstack/nova master: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051
10:15:23 gmann alex_xu: yea, may be ok
10:33:14 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova/tests/unit/cells/test_cells_rpc_driver.py https://review.openstack.org/448457
10:35:28 Jack_Iv Hey Folks! What postgresql version is compatible with Trove?
10:36:19 Jack_Iv 9.5 does not work due to wrong options in the config template
10:37:31 Jack_Iv LOG: unrecognized configuration parameter "checkpoint_segments" in file "/etc/postgresql/9.5/main/postgresql.conf" line 7
11:16:55 sdague alex_xu: I've got daddy daycare duties this morning, so I won't be around for the api meeting
11:52:27 alex_xu sdague: got it, enjoy the daddy daycare duties :)
12:01:54 VAhl When creating a network. The documentation names "physical_network". Where can I find the name of the "physical network"? neutron net-create --shared --provider:physical_network provider --provider:network_type flat provider.
12:07:07 openstackgerrit Stephen Finucane proposed openstack/nova master: netutils: Ignore 'use_ipv6' for network templates https://review.openstack.org/430910
12:07:08 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'allow_same_net_traffic' https://review.openstack.org/430913
12:07:08 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'use_ipv6' https://review.openstack.org/430911
12:07:09 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'allow_same_net_traffic' for port filters https://review.openstack.org/430912
12:07:09 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Final cleanups in conf/network https://review.openstack.org/431023
12:09:27 sfinucan dansmith: thanks for the reviews. Fixed up that errant whitespace in the base patch above
12:09:49 sfinucan Looks like the tempest failure was a CI bug. I couldn't reproduce locally
12:10:23 sfinucan sdague: Also put in the full commit ref in https://review.openstack.org/#/c/430912/, if you fancy taking a second look at it, please :)
12:39:13 openstackgerrit Alex Xu proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625
12:39:14 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200
12:39:14 openstackgerrit Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201
12:39:15 openstackgerrit Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199
12:44:04 cdent alex_xu++ on making the dependency
12:44:13 alex_xu cdent: \o/
12:45:16 alex_xu nova api meeting is in 15 mins at #openstack-meeting-4
12:47:40 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512
12:47:58 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707
12:48:08 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types https://review.openstack.org/430839
12:52:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
12:54:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
12:59:36 jgriffith lyarwood yep, thanks! I'll get that refactored with your patches now and back up
13:06:59 sfinucan mriedem, johnthetubaguy: Could you take a look at [1] again? Think I've all your feedback addressed [1] https://review.openstack.org/#/c/364468/
13:09:40 mriedem sfinucan: yeah, and jaypipes
13:09:43 mriedem after api meeting
13:09:51 sfinucan (y)
13:19:33 lyarwood jgriffith: cool thanks
13:23:17 mriedem dansmith: looks like in the services/hypervisors uuids spec, i missed the os-hosts api https://review.openstack.org/#/c/447149/ which is basically the same as the os-services api, except some other actions can be performed like stop/start/reboot of a host
13:23:36 mriedem but i'm guessing when you made os-services tolerate multiple cells, we got os-hosts support for free
13:23:41 mriedem since they work on the same resource
13:24:28 jaypipes mriedem: unfortunately.
13:25:47 mriedem jaypipes: right :)
13:25:52 mriedem makes me want to merge the APIs
13:26:01 jaypipes mriedem: err, no
13:26:08 mriedem err no?
13:26:12 jaypipes mriedem: compute node != the RPC service worker.
13:26:17 mriedem this isn't that
13:26:21 mriedem os-hypervisors == compute nodes
13:26:24 jaypipes one is a communication target, the other is a resource.
13:26:25 mriedem os-hosts == services
13:26:28 mriedem os-services == services
13:26:29 jaypipes oh...
13:26:35 mriedem i'm talking about the resource representation in those APIs
13:26:36 jaypipes shit, sorry Matt.
13:26:41 mriedem os-hosts and os-services are both services resources
13:26:50 mriedem both can list and show details, and enable/disable
13:26:50 jaypipes hmm.
13:26:59 mriedem but one has some more actions and the other has some other actions
13:27:03 mriedem but separate dumb APIs
13:27:16 jaypipes interesting. lemme finish review on sfinucan's PCI NUMA reserve bp and then will look at it.
13:27:40 dansmith mriedem: yeah, I dunno how we ended up with those
13:27:57 mriedem extensions?
13:28:01 mriedem i blame extensions
13:28:03 jaypipes mb
13:35:37 dansmith sfinucan: hmm, it didn't look like a random failure, but okay
13:36:16 sfinucan All I know is it worked on my machine 🙃
13:39:13 openstackgerrit Vladik Romanovsky proposed openstack/nova master: neutron: retrieve physical network name from a multi-provider network https://review.openstack.org/430026
13:39:24 dansmith sfinucan: this one seems like it might should have a reno about removing the thing, especially since it's firewall/security related no? https://review.openstack.org/#/c/430912/8
13:40:04 sfinucan dansmith: Actually, yeah - I promised that at the PTG. I'll whip one up now
13:40:13 dansmith thanks
13:40:31 sfinucan fwiw, I posted a question on "who uses this" to the operators-list and got zilch back
13:46:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove aggregate uuid generation on load from DB https://review.openstack.org/448579
13:46:48 bauzas mriedem: had a discussion with ttx this FR morning
13:46:54 bauzas mriedem: about the Forum
13:47:02 bauzas mriedem: not sure you saw that
13:47:12 dansmith sfinucan: cool
13:47:28 bauzas tl,dr: I just added a new section for our nova etherpad
13:47:37 bauzas mriedem: about things we could be discussing ad-hoc
13:48:12 sfinucan or not - looks like it got dropped :/ I'll add the reno then put that in -W for a week or so
13:48:46 openstackgerrit Danfly proposed openstack/nova master: remove mox from unit/api/openstack/compute/test_flavor_manage.py https://review.openstack.org/448581

Earlier   Later