| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-04 | |||
| 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 | |
| 08:48:12 | openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | |
| 08:49:51 | sfinucan | johnthetubaguy: Are we allowed to delete configuration without a deprecation cycle, even if they do nothing? https://review.openstack.org/#/c/459470/ | |
| 08:50:46 | johnthetubaguy | sfinucan: so cloudpipe only works with nova-network, and nova-network has been deprecated (on and off) for a long time | |
| 08:50:52 | johnthetubaguy | sfinucan: I would give that one a pass | |
| 08:51:51 | sfinucan | johnthetubaguy: Cool. Sounds fair | |
| 08:54:18 | openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | |
| 08:56:49 | johnthetubaguy | sfinucan: +1 deleting that already deprecated option that does nothing now | |
| 08:56:59 | abhi89 | johnthetubaguy: can you please review https://review.openstack.org/#/c/460305/ | |
| 08:57:28 | johnthetubaguy | abhi89: ah, I should take a look at that now | |
| 09:02:35 | johnthetubaguy | abhi89: I really like that regression test now, really nice proof that your change works | |
| 09:04:00 | abhi89 | johnthetubaguy: yes.. i have well tested the changes also | |
| 09:04:55 | johnthetubaguy | abhi89: I just love seeing the automated tests like that, really stops us breaking your fix by accident later | |
| 09:05:46 | abhi89 | johnthetubaguy: yes.. Thanks John | |
| 09:06:09 | johnthetubaguy | abhi89: thank you for sticking with it, great to get that fixed | |