Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-03
23:03:02 dansmith >>> 1 is True
23:03:03 dansmith False
23:04:38 dansmith mriedem: ah you said it above and I missed.. "same object in memory" is right
23:05:00 dansmith mriedem: but assertIsInstance is a subclass check
23:16:17 openstackgerrit Matthew Treinish proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470
23:16:20 mtreinish mriedem: ^^^ that should address your comments
#openstack-nova - 2017-05-04
00:00:25 openstackgerrit Matthew Treinish proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470
00:26:25 mriedem dansmith: yeah gawd
00:43:00 mriedem mtreinish: we shall see
00:53:36 mriedem mtreinish: hells yes
00:53:37 mriedem +2
00:53:49 mriedem alex_xu_: oomichi: https://review.openstack.org/#/c/459470/6
01:09:38 mtreinish mriedem: awesome, thanks
01:11:33 mtreinish mriedem: I think once that merges it'll be my first nova bp implemented since the coverage extension back in grizzly
01:11:45 mriedem mtreinish: oh it's not done yet,
01:11:49 mriedem novaclient changes buddy
01:12:37 mriedem i don't think it mentions it
01:12:47 mtreinish it doesn't
01:13:07 mriedem certs was already deprecated in novaclient https://github.com/openstack/python-novaclient/commit/b6aea669630dc88296bdded963051a35cc9f995e
01:13:21 mriedem but cloudpipe wasn't https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/cloudpipe.py
01:13:38 mriedem i can probably do that if you want, i've been doing a lot of that in novaclient lately
01:13:51 mtreinish mriedem: if you want, otherwise I'd push those up tomorrow
01:14:32 mriedem mtreinish: you should probably do it since it'll just enable me to procrastinate on summit prep
01:14:52 mtreinish mriedem: heh, what do you think it is for me?
01:14:54 mriedem don't use the certs one as an example, use the one that deprecated os-hosts in novaclient
01:15:07 mriedem use this as the example https://github.com/openstack/python-novaclient/commit/ea3b9f7fef28a7f06b51a3b176c85a1fc33cecaa
01:15:10 mtreinish ok
01:15:54 mriedem mtreinish: there is a nova onboarding session at the summit you can attend too
01:15:57 mriedem since you're a new contributor again
01:17:05 mtreinish mriedem: heh, I'll consider it
01:17:26 mtreinish I just looked my last nova commit before the cert removal was in feb. 2016
01:18:14 mriedem probably when you were doing the tenant isolation stuff
01:18:17 mriedem no, that was older
01:18:38 mtreinish mriedem: https://review.openstack.org/#/c/279799/
01:19:01 mriedem ah i remember that
01:22:36 mtreinish mriedem: at some point this cycle I'm also going to work on adding docs on how to install nova (or any project) from a tarball, because there were no instructions I could find for that (something from my dirty clouds talk)
01:23:42 mriedem ok
01:23:55 mriedem i noticed your talk was accepted but i think i have a conflict...
01:23:57 mriedem which sucks
01:26:20 openstackgerrit Vladik Romanovsky proposed openstack/nova master: neutron: retrieve physical network name from a multi-provider network https://review.openstack.org/430026
01:28:40 mtreinish mriedem: oh well, there will always be the video
01:28:51 mtreinish mriedem: I linked you the spoiler picture of what the cloud looked like right?
01:29:12 vladikr jaypipes, dansmith, thanks for the reviews. I've update the multi-segments patch (comments mostly), could you please take a look? https://review.openstack.org/430026
01:29:25 vladikr s/update/updated
01:30:50 mriedem mtreinish: i think so yeah
01:30:53 jaypipes vladikr: done!
01:31:17 vladikr jaypipes, thanks!
01:31:21 jaypipes np :)
01:32:53 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
01:35:15 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
01:41:34 mriedem Kevin_Zheng: do you want to rebase this? https://review.openstack.org/#/c/459593/
01:42:02 mriedem i suppose i can in gerrit
01:42:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
01:42:31 Kevin_Zheng mriedem sure, but why?
01:42:57 mriedem just because it messes up the patch set order when going through the changes in gerrit if it's not rebased
01:43:10 mriedem nothing major
01:43:35 Kevin_Zheng AH, OK, I was going to ask you to review this one if you have time
01:44:33 mriedem i can run through it quick
01:46:56 Kevin_Zheng One question
01:48:15 Kevin_Zheng if we are using placement, which is mandatory in O, it seems that we are unable to use AggregateCoreFilter properly
01:57:08 mriedem Kevin_Zheng: i'm not familiar with that filter; comments in your tags 3/4 patch btw
01:57:22 mriedem edleafe: bauzas: jaypipes: ^ re: placement + AggregateCoreFilter
01:58:03 jaypipes Kevin_Zheng: that filter wasn't touched by the placement service.
01:58:21 mriedem i don't think that's the question
01:58:48 jaypipes mriedem: I don't see a question...
01:58:53 mriedem this is a host aggregate with cpu_allocation_ratio meta set on it?
01:59:00 mriedem is that the idea?
01:59:34 mriedem jaypipes: so with placement in the filter scheduler we basically ignore the corefilter because we get compute node RPs via the VCPU resource class filtering right?
01:59:42 mriedem we don't ignore it, but it's redundant
01:59:42 openstackgerrit fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789
01:59:58 mriedem and i think we said you could/should disable the CoreFilter, RAMFilter and DiskFilter once you're using placement
02:00:20 jaypipes yeah...
02:00:34 jaypipes but aggregate metadata should still be processed, IIRC.
02:01:00 mriedem yeah i'm not sure why this would be a problem,
02:01:15 mriedem it's just extra filtering once we get the list of compute node resource providers filtered on VCPU to start
02:01:36 mriedem Kevin_Zheng: it's probably best to just report a bug with details
02:01:51 mriedem especially because i have a date with my couch and tv
02:02:54 Kevin_Zheng ha， i will
02:03:03 mriedem good night all
02:03:08 mriedem and day
02:05:47 Kevin_Zheng As described in aggregatecorefilter it will try to get ratio from aggregate first and if not possible then fall back to per host ratio
02:08:03 Kevin_Zheng If we use placement, then it directly selected hosts with per host ratio, if we have different configurations in aggregate meta, then the host selection might not be what we expected
02:10:04 Kevin_Zheng Say, if people only use aggregatecorefilter as it will fall back to per host ratio, it won't work with placement
02:22:49 openstackgerrit shaofeng cheng proposed openstack/nova master: Update nova version newton to ocata in upgrade.rst https://review.openstack.org/457197
03:18:35 openstackgerrit Jackie Truong proposed openstack/nova master: Add missing rootwrap filter for cryptsetup https://review.openstack.org/462348
03:33:09 openstackgerrit chenaidong1 proposed openstack/nova master: Fix a typo https://review.openstack.org/462349
03:38:41 openstackgerrit chenaidong1 proposed openstack/python-novaclient master: Fix a typo https://review.openstack.org/462351
04:03:22 openstackgerrit LiChunlin proposed openstack/nova master: update the description of hypervisor statistics response https://review.openstack.org/462355
04:52:56 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova master: Remove oslo.config deprecated parameter enforce_type https://review.openstack.org/462364
07:25:46 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log endpoint instead of stevedore https://review.openstack.org/461647
07:27:02 openstackgerrit jichenjc proposed openstack/nova master: use plain routes list for os-simple-tenant-usage https://review.openstack.org/460821
07:33:47 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-server-groups endpoint instead of stevedore https://review.openstack.org/462169
07:38:58 openstackgerrit Andreas Scheuring proposed openstack/nova master: DO NOT MERGE https://review.openstack.org/462421
07:48:04 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore https://review.openstack.org/461641
07:49:29 openstackgerrit jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458
07:52:26 openstackgerrit Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470
07:53:57 openstackgerrit Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470
08:13:21 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049
08:19:48 sfinucan mdbooth: Could you take a look at https://review.openstack.org/#/q/topic:disk_info_processing today? It's been around a long time and it seems like something you should look at first
08:26:32 openstackgerrit shaofeng cheng proposed openstack/nova master: Replace newton to release_name in upgrade.rst https://review.openstack.org/457197

Earlier   Later