| *** edmondsw has joined #openstack-nova | 00:02 | |
| *** yangyapeng has joined #openstack-nova | 00:04 | |
| *** edmondsw has quit IRC | 00:06 | |
| *** thorst has joined #openstack-nova | 00:08 | |
| *** felipemonteiro_ has joined #openstack-nova | 00:11 | |
| *** yangyapeng has quit IRC | 00:11 | |
| *** thorst has quit IRC | 00:12 | |
| *** yassine has joined #openstack-nova | 00:13 | |
| *** gmann has quit IRC | 00:18 | |
| *** gmann has joined #openstack-nova | 00:18 | |
| *** ircuser-1 has quit IRC | 00:26 | |
| *** slagle has joined #openstack-nova | 00:28 | |
| *** itlinux has quit IRC | 00:29 | |
| *** sdake_ has quit IRC | 00:32 | |
| *** sdake_ has joined #openstack-nova | 00:32 | |
| *** itlinux has joined #openstack-nova | 00:33 | |
| *** itlinux has quit IRC | 00:33 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':' https://review.openstack.org/491282 | 00:42 |
|---|---|---|
| *** zhurong has joined #openstack-nova | 00:43 | |
| *** kiennt has joined #openstack-nova | 00:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/490859 | 00:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/488086 | 00:51 |
| *** mdnadeem has joined #openstack-nova | 00:52 | |
| *** zhurong has quit IRC | 00:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/491296 | 00:53 |
| *** Shunli has joined #openstack-nova | 00:58 | |
| *** phuongnh has joined #openstack-nova | 00:59 | |
| *** dixiaoli has joined #openstack-nova | 01:02 | |
| *** slagle has quit IRC | 01:08 | |
| *** yangyapeng has joined #openstack-nova | 01:10 | |
| *** yangyapeng has quit IRC | 01:10 | |
| *** trungnv has joined #openstack-nova | 01:14 | |
| *** markvoelker has joined #openstack-nova | 01:15 | |
| *** yangyapeng has joined #openstack-nova | 01:17 | |
| *** hongbin has joined #openstack-nova | 01:20 | |
| *** mdnadeem has quit IRC | 01:21 | |
| *** litao__ has joined #openstack-nova | 01:28 | |
| *** mdnadeem has joined #openstack-nova | 01:31 | |
| *** dixiaoli_ has joined #openstack-nova | 01:37 | |
| *** dixiaoli has quit IRC | 01:37 | |
| *** crushil has joined #openstack-nova | 01:38 | |
| *** randomhack has joined #openstack-nova | 01:41 | |
| *** randomhack has quit IRC | 01:46 | |
| *** liusheng has joined #openstack-nova | 01:47 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: filtering the resource provider id when delete trait association https://review.openstack.org/489205 | 01:49 |
| *** edmondsw has joined #openstack-nova | 01:50 | |
| *** edmondsw has quit IRC | 01:54 | |
| *** markvoelker has quit IRC | 02:02 | |
| *** markvoelker has joined #openstack-nova | 02:02 | |
| *** thorst has joined #openstack-nova | 02:13 | |
| *** thorst has quit IRC | 02:18 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool https://review.openstack.org/489451 | 02:20 |
| *** tuanluong has joined #openstack-nova | 02:26 | |
| *** huanxie has joined #openstack-nova | 02:30 | |
| *** markvoelker has quit IRC | 02:34 | |
| *** tbachman has joined #openstack-nova | 02:55 | |
| *** mingyu has quit IRC | 03:06 | |
| *** liusheng has quit IRC | 03:06 | |
| *** owalsh_ has joined #openstack-nova | 03:06 | |
| *** liusheng has joined #openstack-nova | 03:07 | |
| *** mingyu has joined #openstack-nova | 03:07 | |
| *** felipemonteiro__ has joined #openstack-nova | 03:08 | |
| *** dave-mccowan has quit IRC | 03:09 | |
| *** owalsh has quit IRC | 03:10 | |
| *** felipemonteiro_ has quit IRC | 03:11 | |
| *** baoli has joined #openstack-nova | 03:13 | |
| *** thorst has joined #openstack-nova | 03:14 | |
| *** tbachman has quit IRC | 03:16 | |
| *** mamandle has joined #openstack-nova | 03:17 | |
| *** felipemonteiro__ has quit IRC | 03:18 | |
| *** thorst has quit IRC | 03:19 | |
| *** crushil has quit IRC | 03:26 | |
| *** kfox1111 has quit IRC | 03:29 | |
| *** sridharg has joined #openstack-nova | 03:36 | |
| openstackgerrit | Danfly proposed openstack/nova master: update availablity_zone in the instances table when live-migration successes https://review.openstack.org/491226 | 03:37 |
| *** Nel1x has quit IRC | 03:37 | |
| *** edmondsw has joined #openstack-nova | 03:38 | |
| *** edmondsw has quit IRC | 03:42 | |
| *** mamandle has quit IRC | 03:43 | |
| *** ykuo_ has quit IRC | 03:45 | |
| *** links has joined #openstack-nova | 03:49 | |
| *** nicolasbock has joined #openstack-nova | 03:51 | |
| *** ykuo_ has joined #openstack-nova | 03:57 | |
| *** hongbin has quit IRC | 04:01 | |
| *** udesale has joined #openstack-nova | 04:06 | |
| *** udesale has quit IRC | 04:15 | |
| *** gbarros has quit IRC | 04:16 | |
| *** udesale has joined #openstack-nova | 04:18 | |
| *** ratailor has joined #openstack-nova | 04:33 | |
| *** mingyu has quit IRC | 04:33 | |
| *** PsionTheory has joined #openstack-nova | 04:37 | |
| *** david-lyle has quit IRC | 04:38 | |
| *** udesale has quit IRC | 04:40 | |
| openstackgerrit | Danfly proposed openstack/nova master: update availablity_zone in the instances table when live-migration successes https://review.openstack.org/491226 | 04:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':' https://review.openstack.org/491340 | 04:41 |
| *** udesale has joined #openstack-nova | 04:41 | |
| *** slaweq has joined #openstack-nova | 04:42 | |
| *** zerda3 has joined #openstack-nova | 04:44 | |
| *** slaweq has quit IRC | 04:46 | |
| *** itlinux has joined #openstack-nova | 04:54 | |
| *** zerda3 has quit IRC | 04:57 | |
| *** Shunli has quit IRC | 05:06 | |
| *** Shunli has joined #openstack-nova | 05:06 | |
| *** mingyu has joined #openstack-nova | 05:09 | |
| *** mingyu has quit IRC | 05:15 | |
| *** thorst has joined #openstack-nova | 05:15 | |
| *** sree has joined #openstack-nova | 05:20 | |
| *** thorst has quit IRC | 05:20 | |
| *** jaosorior has joined #openstack-nova | 05:24 | |
| *** trinaths has joined #openstack-nova | 05:26 | |
| *** zhurong has joined #openstack-nova | 05:28 | |
| *** mamandle has joined #openstack-nova | 05:28 | |
| *** itlinux has quit IRC | 05:32 | |
| *** psachin has joined #openstack-nova | 05:36 | |
| *** kiennt has quit IRC | 05:37 | |
| *** moshele has joined #openstack-nova | 05:40 | |
| *** shan has joined #openstack-nova | 05:41 | |
| *** randomhack has joined #openstack-nova | 05:42 | |
| *** randomhack has quit IRC | 05:47 | |
| *** xinliang has joined #openstack-nova | 05:47 | |
| *** xinliang has quit IRC | 05:47 | |
| *** xinliang has joined #openstack-nova | 05:47 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: the common method to filter resource providers by traits https://review.openstack.org/489206 | 05:50 |
| *** mamandle has quit IRC | 05:52 | |
| *** ekuris has joined #openstack-nova | 05:54 | |
| *** mamandle has joined #openstack-nova | 05:54 | |
| *** yangyapeng has quit IRC | 06:01 | |
| *** yangyapeng has joined #openstack-nova | 06:02 | |
| *** kiennt has joined #openstack-nova | 06:05 | |
| *** dixiaoli_ has quit IRC | 06:06 | |
| *** Oku_OS-away is now known as Oku_OS | 06:09 | |
| *** sridharg has quit IRC | 06:14 | |
| *** mingyu has joined #openstack-nova | 06:16 | |
| *** sridharg has joined #openstack-nova | 06:18 | |
| *** kiennt has quit IRC | 06:19 | |
| *** mamandle has quit IRC | 06:19 | |
| *** mingyu has quit IRC | 06:21 | |
| *** mingyu has joined #openstack-nova | 06:27 | |
| *** sridharg has quit IRC | 06:35 | |
| *** sridharg has joined #openstack-nova | 06:35 | |
| *** ratailor has quit IRC | 06:35 | |
| *** mamandle has joined #openstack-nova | 06:36 | |
| *** mingyu has quit IRC | 06:39 | |
| *** udesale__ has joined #openstack-nova | 06:41 | |
| *** udesale has quit IRC | 06:43 | |
| *** sridharg has quit IRC | 06:43 | |
| *** yangyape_ has joined #openstack-nova | 06:45 | |
| *** sridharg has joined #openstack-nova | 06:45 | |
| *** yangyapeng has quit IRC | 06:45 | |
| *** vks1 has joined #openstack-nova | 06:46 | |
| *** udesale__ has quit IRC | 06:48 | |
| *** udesale has joined #openstack-nova | 06:49 | |
| *** sridharg has quit IRC | 06:53 | |
| *** mingyu has joined #openstack-nova | 06:53 | |
| *** sridharg has joined #openstack-nova | 06:56 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: the common method to filter resource providers by traits https://review.openstack.org/489206 | 06:59 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: add traits support to get resource_providers with shared https://review.openstack.org/478464 | 06:59 |
| *** sshwarts has joined #openstack-nova | 07:00 | |
| *** bhagyashris has joined #openstack-nova | 07:00 | |
| *** sridharg has quit IRC | 07:02 | |
| *** nicolasbock_ has joined #openstack-nova | 07:06 | |
| *** yangyape_ is now known as yangyapeng | 07:07 | |
| *** nicolasbock_ has quit IRC | 07:07 | |
| *** alexchadin has joined #openstack-nova | 07:07 | |
| *** sridharg has joined #openstack-nova | 07:08 | |
| *** rcernin has joined #openstack-nova | 07:08 | |
| *** mamandle has quit IRC | 07:14 | |
| *** edmondsw has joined #openstack-nova | 07:14 | |
| *** kiennt has joined #openstack-nova | 07:15 | |
| *** thorst has joined #openstack-nova | 07:16 | |
| *** mamandle has joined #openstack-nova | 07:17 | |
| *** ratailor has joined #openstack-nova | 07:17 | |
| *** kiennt_ has joined #openstack-nova | 07:18 | |
| *** markus_z has joined #openstack-nova | 07:18 | |
| *** edmondsw has quit IRC | 07:18 | |
| *** tesseract has joined #openstack-nova | 07:20 | |
| *** kiennt has quit IRC | 07:20 | |
| *** thorst has quit IRC | 07:21 | |
| *** udesale__ has joined #openstack-nova | 07:22 | |
| *** udesale has quit IRC | 07:23 | |
| *** sahid has joined #openstack-nova | 07:25 | |
| *** priteau has joined #openstack-nova | 07:28 | |
| bauzas | good morning Nova | 07:36 |
| *** udesale__ has quit IRC | 07:42 | |
| *** udesale has joined #openstack-nova | 07:42 | |
| *** gibi_chaosmonkey is now known as gibi | 07:46 | |
| gibi | good morning | 07:46 |
| *** mamandle has quit IRC | 07:59 | |
| *** alexchadin has quit IRC | 08:01 | |
| *** sree_ has joined #openstack-nova | 08:04 | |
| *** sree_ is now known as Guest66661 | 08:05 | |
| *** sree has quit IRC | 08:07 | |
| *** alexchadin has joined #openstack-nova | 08:07 | |
| *** moshele has quit IRC | 08:11 | |
| *** priteau has quit IRC | 08:14 | |
| *** moshele has joined #openstack-nova | 08:16 | |
| *** priteau has joined #openstack-nova | 08:17 | |
| *** markvoelker has joined #openstack-nova | 08:17 | |
| *** randomhack has joined #openstack-nova | 08:18 | |
| *** markvoelker_ has joined #openstack-nova | 08:19 | |
| *** aarefiev_afk is now known as aarefiev | 08:21 | |
| *** markvoelker has quit IRC | 08:23 | |
| *** randomhack has quit IRC | 08:24 | |
| *** alexchadin has quit IRC | 08:26 | |
| *** alexchadin has joined #openstack-nova | 08:28 | |
| *** baoli has quit IRC | 08:30 | |
| *** mamandle has joined #openstack-nova | 08:40 | |
| *** fnordahl_ is now known as fnordahl | 08:42 | |
| *** kiennt_ is now known as kiennt | 08:48 | |
| *** amuresan has joined #openstack-nova | 08:51 | |
| *** alexchadin has quit IRC | 08:59 | |
| *** edmondsw has joined #openstack-nova | 09:02 | |
| *** moshele has quit IRC | 09:05 | |
| *** edmondsw has quit IRC | 09:06 | |
| *** moshele has joined #openstack-nova | 09:09 | |
| *** mingyu_ has joined #openstack-nova | 09:12 | |
| *** mingyu has quit IRC | 09:12 | |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | 09:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563 | 09:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924 | 09:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105 | 09:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550 | 09:12 |
| *** slaweq has joined #openstack-nova | 09:16 | |
| openstackgerrit | Danfly proposed openstack/nova master: update availablity_zone in the instances table after live-migration https://review.openstack.org/491226 | 09:16 |
| *** thorst has joined #openstack-nova | 09:17 | |
| *** slaweq has quit IRC | 09:21 | |
| *** sambetts_ is now known as sambetts | 09:21 | |
| *** thorst has quit IRC | 09:22 | |
| *** sahid has quit IRC | 09:24 | |
| *** lennyb has quit IRC | 09:25 | |
| *** alexchadin has joined #openstack-nova | 09:26 | |
| *** jaosorior has quit IRC | 09:27 | |
| *** cdent has joined #openstack-nova | 09:28 | |
| *** Shunli has quit IRC | 09:28 | |
| cdent | gibi: I decided to take a page from your testing and do random stuff and find bugs. Seen these: https://bugs.launchpad.net/nova/+bug/1708961 https://bugs.launchpad.net/nova/+bug/1708958 | 09:30 |
| openstack | Launchpad bug 1708961 in OpenStack Compute (nova) "migration of single instance from multi-instance request spec fails with IndexError" [Undecided,New] | 09:30 |
| openstack | Launchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Low,New] | 09:30 |
| *** yangyape_ has joined #openstack-nova | 09:31 | |
| *** tuanluong has quit IRC | 09:32 | |
| *** trungnv has quit IRC | 09:32 | |
| *** phuongnh has quit IRC | 09:32 | |
| *** hieulq has quit IRC | 09:32 | |
| *** yangyapeng has quit IRC | 09:32 | |
| *** trungnv has joined #openstack-nova | 09:32 | |
| *** tuanluong has joined #openstack-nova | 09:33 | |
| *** phuongnh has joined #openstack-nova | 09:33 | |
| *** hieulq has joined #openstack-nova | 09:33 | |
| *** PsionTheory has quit IRC | 09:36 | |
| bauzas | cdent: gibi: that doesn't help but I'm pretty concerned by how we just missed https://bugs.launchpad.net/nova/+bug/1708961 | 09:37 |
| openstack | Launchpad bug 1708961 in OpenStack Compute (nova) "migration of single instance from multi-instance request spec fails with IndexError" [Undecided,New] | 09:37 |
| bauzas | cdent: gibi: that is a serious blocker to me | 09:38 |
| cdent | bauzas: yeah, I think we’ve got some gaps in testing. | 09:38 |
| *** sahid has joined #openstack-nova | 09:38 | |
| bauzas | it's more than that IMHO | 09:38 |
| bauzas | I remember being super cautious during reviews about how we tried to make sure that num_instances was equal or not to the list of instances... | 09:39 |
| gibi | bauzas: nice findings | 09:39 |
| bauzas | but we totally forgot to take care of move operations | 09:39 |
| cdent | careful review is important, but it shouldn’t be a substitute for real tests | 09:40 |
| bauzas | gibi: cdent: about https://bugs.launchpad.net/nova/+bug/1708958 we have ComputeFilter for that | 09:40 |
| openstack | Launchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Low,Confirmed] | 09:40 |
| cdent | if we have to rely on careful review, then we need better tests | 09:40 |
| bauzas | cdent: I specifically asked for conservative testing about that | 09:41 |
| bauzas | anyway, it's done now | 09:41 |
| cdent | bauzas: yeah, 1708958, as I say on the bug report, isn’t a functionality bug, more a “in what way do we want this data to be correct" | 09:41 |
| *** sahid has quit IRC | 09:42 | |
| *** yamamoto_ has quit IRC | 09:44 | |
| *** jaosorior has joined #openstack-nova | 09:45 | |
| bauzas | cdent: just made a comment in the bug report to clarify what I want | 09:45 |
| bauzas | cdent: I don't want placement to have nova-specifisms | 09:45 |
| *** sahid has joined #openstack-nova | 09:45 | |
| gibi | I mean, cdent, nice findings | 09:45 |
| cdent | yes | 09:45 |
| bauzas | cdent: but I'd love placement to consider inventory freshness | 09:45 |
| gibi | I'm sooo slow today | 09:45 |
| cdent | bauzas: I think the issue is simpler than that. The choice is really: should disabling a hypervisor cause (something in nova) to delete the resource provider | 09:47 |
| cdent | I think it should | 09:47 |
| cdent | since if a service is diabled it is not providing resources | 09:47 |
| *** sdague has joined #openstack-nova | 09:47 | |
| bauzas | cdent: that would be one option, I agree | 09:48 |
| *** dtantsur|afk is now known as dtantsur | 09:48 | |
| gibi | cdent: it is not that simple. As disabling a host doesn't mean that the VMs running on that host freed up all the resources | 09:48 |
| gibi | cdent: so that host still providing resources to existing VMs | 09:48 |
| bauzas | gibi: from a placement perspective, we don't care about the space that RP has | 09:48 |
| cdent | bauzas: actaully gibi’s right: we want to use allocations to indicate quota use | 09:49 |
| bauzas | gibi: unless *cough cough* we have shared resources from that honst :) | 09:49 |
| cdent | and we can’t delete an rp (or inventory) if there are allocations in use | 09:49 |
| bauzas | yet another can of worms spotted IMHO | 09:49 |
| cdent | sigh, what a twisted web we weave | 09:49 |
| bauzas | anyway, should only be optimization, since we have ComputeFilter | 09:50 |
| bauzas | that's just we're providing a whole list that can be long of potential candidates that are eventually not selected because of the computefilter | 09:50 |
| bauzas | again, low | 09:51 |
| * cdent nods | 09:51 | |
| bauzas | the other bug is more concerning me | 09:51 |
| cdent | I _think_ the fix is relatively straightforward on that one | 09:51 |
| bauzas | I'll try to look at a proper resolution once I'm done with writing the prelude section explaining how great we are :) | 09:51 |
| cdent | but given how many gears within gears we have, who knows what the cascades will be | 09:52 |
| * cdent sighs | 09:52 | |
| gibi | cdent: you mean this is an interesting challenge? :) | 09:52 |
| cdent | gibi: heh, that’s very positive of you for a monday morning | 09:53 |
| bauzas | cdent: I really wanted to help placement bugfixing, permission to work on the critical one ? :) | 09:53 |
| bauzas | the resize ones were just killing my brain | 09:53 |
| *** shan is now known as shan_afk | 09:53 | |
| cdent | bauzas: I don’t think you need to ask my permission, if you want that one, take it. I haven’t started on it. I need to catch up on a bunch of reviews, and then later in the day I’ll go back to breaking stuff and reporting bugs | 09:54 |
| bauzas | roger. | 09:54 |
| cdent | I set up a 6 node devstack over the weekend so I can do some real experimentation. I fear what I might find. | 09:54 |
| gibi | cdent: I just want to motivate you to dig into it :) | 09:54 |
| bauzas | permission is more for making sure I'm not duplicating efforts | 09:54 |
| bauzas | like, /me taking mutex on it | 09:55 |
| cdent | bauzas: godspeed | 09:55 |
| bauzas | voilà. | 09:55 |
| * cdent coffees, eats, etcs | 09:55 | |
| bauzas | FWIW, I'm in a perfect place for chasing bugs | 09:56 |
| bauzas | litterally 200m away from the meditarean sea | 09:56 |
| * bauzas doesn't use imperial metrics | 09:56 | |
| *** [1]amuresan has joined #openstack-nova | 09:59 | |
| *** amuresan has quit IRC | 10:01 | |
| *** [1]amuresan is now known as amuresan | 10:01 | |
| *** yamamoto has joined #openstack-nova | 10:02 | |
| *** gcb has quit IRC | 10:03 | |
| *** thorst has joined #openstack-nova | 10:05 | |
| *** lpetrut has joined #openstack-nova | 10:08 | |
| *** kiennt has quit IRC | 10:09 | |
| ratailor | Can anybody help me in debugging this code block. | 10:09 |
| ratailor | https://github.com/openstack/nova/blob/master/nova/objects/host_mapping.py#L90-L93 | 10:09 |
| ratailor | How can I check value of api_models.HostMapping.host in above code block. | 10:10 |
| stephenfin | sdague, bauzas, cdent: Is live migration an operator or user-level operation, i.e. 'doc/source/admin' or 'doc/source/user'? | 10:10 |
| *** thorst has quit IRC | 10:11 | |
| stephenfin | I ask because I think I might need to move this document https://review.openstack.org/#/c/490952/1/doc/source/admin/migration.rst | 10:12 |
| stephenfin | and this, for evacuate https://review.openstack.org/#/c/490952/1/doc/source/admin/evacuate.rst | 10:12 |
| bauzas | stephenfin: live migrations are only possible for users that are having admin keystone rights | 10:13 |
| stephenfin | bauzas: and evacuations? | 10:13 |
| bauzas | stephenfin: AFAIR, evacuations are only for admin-only too since it requires a knowledge of your host health | 10:14 |
| bauzas | stephenfin: but I can check the policy | 10:14 |
| bauzas | stephenfin: the thing is, we show those operations by the CLI | 10:14 |
| stephenfin | bauzas: So they both warrant staying in the admin guide. I'll keep them there so. Thanks! :) | 10:14 |
| stephenfin | what do you mean? | 10:14 |
| stephenfin | "by the CLI"? | 10:14 |
| bauzas | stephenfin: I mean a single user can see those operations | 10:15 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add a prelude section for Pike https://review.openstack.org/491424 | 10:15 |
| stephenfin | bauzas: You mean regardless of whether they're an admin or not? | 10:15 |
| bauzas | stephenfin: yup | 10:15 |
| bauzas | it will just give'em a 401 if they try | 10:15 |
| *** yamamoto has quit IRC | 10:16 | |
| bauzas | the fact is, I don't remember which specific move operation is either admin-only or regular user, depending if people provided a target or not | 10:16 |
| bauzas | but anyway, those two tools (live-mig and evac) are aimed for admins anyway | 10:17 |
| *** slaweq has joined #openstack-nova | 10:17 | |
| stephenfin | bauzas: Right, they can stay where they are now so | 10:18 |
| * bauzas is just trying to generate the policy file to double-check | 10:18 | |
| bauzas | it'll take a while tho, under 1Mbps b/w | 10:19 |
| *** slaweq has quit IRC | 10:21 | |
| bauzas | stephenfin: okay, confirmed, live-migrations and evacuations are admin-only, and cold migrations depend on whether you specify a target or not | 10:24 |
| *** mdnadeem has quit IRC | 10:25 | |
| *** claudiub has joined #openstack-nova | 10:28 | |
| stephenfin | bauzas: Awesome. It stays in the admin guide so | 10:30 |
| *** yangyape_ has quit IRC | 10:30 | |
| *** yangyapeng has joined #openstack-nova | 10:31 | |
| *** yangyapeng has quit IRC | 10:33 | |
| *** zenoway has joined #openstack-nova | 10:34 | |
| *** yangyapeng has joined #openstack-nova | 10:35 | |
| *** tuanluong has quit IRC | 10:35 | |
| sdague | stephenfin: right, with default policy they are both admin only | 10:35 |
| *** phuongnh has quit IRC | 10:35 | |
| sdague | bauzas: I'm confused about your single user statement | 10:36 |
| *** lennyb has joined #openstack-nova | 10:37 | |
| *** yangyapeng has quit IRC | 10:39 | |
| *** krtaylor has quit IRC | 10:41 | |
| *** zenoway has quit IRC | 10:42 | |
| vks1 | sdague: hi | 10:42 |
| sdague | vks1: morning | 10:44 |
| vks1 | sdague: good morning | 10:45 |
| vks1 | sdague: for REST API we use iso format for timestamp, which oslo API is used to generate that ? | 10:46 |
| *** mdnadeem has joined #openstack-nova | 10:47 | |
| *** markvoelker has joined #openstack-nova | 10:49 | |
| *** alexchadin has quit IRC | 10:49 | |
| *** zenoway has joined #openstack-nova | 10:50 | |
| bauzas | sdague: sorry, was disconnected, which statement in particular? | 10:50 |
| *** edmondsw has joined #openstack-nova | 10:50 | |
| *** krtaylor has joined #openstack-nova | 10:51 | |
| *** Guest66661 has quit IRC | 10:52 | |
| *** markvoelker_ has quit IRC | 10:53 | |
| *** markvoelker_ has joined #openstack-nova | 10:54 | |
| *** markvoelker has quit IRC | 10:54 | |
| *** edmondsw has quit IRC | 10:54 | |
| *** trinaths has left #openstack-nova | 10:54 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add additional content to admin guide https://review.openstack.org/490952 | 10:56 |
| *** markvoelker has joined #openstack-nova | 10:57 | |
| *** tbachman has joined #openstack-nova | 10:57 | |
| sdague | vks1: I'm not sure it's coming from oslo | 10:59 |
| *** markvoelker has quit IRC | 10:59 | |
| *** markvoelker_ has quit IRC | 11:00 | |
| sdague | vks1: I think they are going through this - https://github.com/openstack/nova/blob/6e77e75993c5b431c68c963bb06494ae5d36b944/nova/utils.py#L1344-L1355 | 11:00 |
| *** markvoelker has joined #openstack-nova | 11:01 | |
| *** sahid has quit IRC | 11:02 | |
| *** strigazi_OFF is now known as strigazi | 11:05 | |
| *** slaweq has joined #openstack-nova | 11:06 | |
| *** gszasz has joined #openstack-nova | 11:07 | |
| *** slaweq has quit IRC | 11:10 | |
| *** sridharg has quit IRC | 11:11 | |
| *** sridharg has joined #openstack-nova | 11:11 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix migrate single instance when it was created concurrently https://review.openstack.org/491439 | 11:12 |
| *** markvoelker_ has joined #openstack-nova | 11:12 | |
| *** yamamoto has joined #openstack-nova | 11:13 | |
| bauzas | cdent: gibi: ^ | 11:14 |
| * bauzas bailing out for lunch | 11:14 | |
| gibi | bauzas: looking | 11:14 |
| *** markvoelker has quit IRC | 11:15 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove 'contrib' directory https://review.openstack.org/491443 | 11:22 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove files from 'tools' https://review.openstack.org/491444 | 11:22 |
| *** jaypipes has joined #openstack-nova | 11:25 | |
| *** eharney has quit IRC | 11:27 | |
| gibi | bauzas: I left some comments in https://review.openstack.org/#/c/491439 | 11:27 |
| *** catintheroof has joined #openstack-nova | 11:28 | |
| *** markvoelker_ has quit IRC | 11:28 | |
| *** thorst has joined #openstack-nova | 11:28 | |
| *** markvoelker has joined #openstack-nova | 11:29 | |
| *** slagle has joined #openstack-nova | 11:31 | |
| *** thorst has quit IRC | 11:33 | |
| stephenfin | asettle_: If you have time this week, I'd like your thoughts on what we can do with the admin guide going forward https://review.openstack.org/#/c/490952/ | 11:33 |
| *** markvoelker has quit IRC | 11:34 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with too big flavor https://review.openstack.org/490814 | 11:34 |
| asettle_ | stephenfin: I can do thoughts | 11:34 |
| stephenfin | It's pretty...icky right now and I haven't even started dragging in the stuff from the old configuration guide | 11:34 |
| stephenfin | asettle_: thoughts are good | 11:35 |
| asettle_ | Okay, lemme go have a look at the beautiful build | 11:36 |
| asettle_ | Well, that's just stunning isn't it | 11:36 |
| asettle_ | I like the uhh... raw... blob ... of it all | 11:36 |
| stephenfin | What can I say | 11:36 |
| stephenfin | I've a knack | 11:36 |
| *** yangyapeng has joined #openstack-nova | 11:37 | |
| asettle_ | You truly do | 11:37 |
| asettle_ | Hmm, okay. I'll have a lookie and comment | 11:37 |
| *** catintheroof has quit IRC | 11:38 | |
| *** yangyapeng has quit IRC | 11:39 | |
| *** yangyapeng has joined #openstack-nova | 11:39 | |
| asettle_ | It'd be cool to have this as Administration Guides or something similar :) (cause it's for the tasks, mostly) | 11:40 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with too big flavor https://review.openstack.org/490814 | 11:40 |
| *** markvoelker has joined #openstack-nova | 11:40 | |
| *** zenoway has quit IRC | 11:40 | |
| stephenfin | asettle_: Can't argue with that | 11:40 |
| *** markvoelker_ has joined #openstack-nova | 11:41 | |
| stephenfin | The tasks seem to be broken up into two types two: cloud-level tasks and instance-level ones | 11:42 |
| stephenfin | *two types too | 11:42 |
| *** markvoelker_ has quit IRC | 11:42 | |
| stephenfin | I'd guess most of the configuration guides docs, once added, would fall into the former | 11:42 |
| *** markvoelker_ has joined #openstack-nova | 11:44 | |
| *** zhurong has quit IRC | 11:44 | |
| *** markvoelker has quit IRC | 11:44 | |
| asettle_ | I would imagine so, ya | 11:44 |
| asettle_ | Give me the afternoon to poke around :) I just gotta scoot out for lunch before afternoon meetings | 11:44 |
| *** amuresan has quit IRC | 11:46 | |
| *** markvoelker has joined #openstack-nova | 11:47 | |
| *** liverpooler has quit IRC | 11:47 | |
| *** markvoelker_ has quit IRC | 11:50 | |
| *** catintheroof has joined #openstack-nova | 11:50 | |
| *** markvoelker_ has joined #openstack-nova | 11:51 | |
| *** markvoelker has quit IRC | 11:52 | |
| *** markvoelker has joined #openstack-nova | 11:53 | |
| *** catintheroof has quit IRC | 11:55 | |
| *** markvoelker_ has quit IRC | 11:56 | |
| *** markvoelker_ has joined #openstack-nova | 11:56 | |
| *** ratailor has quit IRC | 11:58 | |
| *** markvoelker has quit IRC | 12:00 | |
| *** cdent has quit IRC | 12:02 | |
| *** markvoelker has joined #openstack-nova | 12:02 | |
| *** markvoelker_ has quit IRC | 12:03 | |
| *** moshele has quit IRC | 12:04 | |
| *** moshele has joined #openstack-nova | 12:04 | |
| *** vks1 has quit IRC | 12:04 | |
| sdague | stephenfin / asettle_ another one of those subpages is here - https://review.openstack.org/#/c/490994/ | 12:05 |
| sdague | I think I pushed that after you all had kicked off for the weekend | 12:05 |
| *** thorst has joined #openstack-nova | 12:05 | |
| *** jangutter_ has joined #openstack-nova | 12:06 | |
| *** slaweq has joined #openstack-nova | 12:07 | |
| *** jangutter has quit IRC | 12:07 | |
| *** moshele has quit IRC | 12:08 | |
| *** thorst has quit IRC | 12:09 | |
| *** mgariepy has quit IRC | 12:10 | |
| *** thorst has joined #openstack-nova | 12:10 | |
| *** baoli has joined #openstack-nova | 12:11 | |
| *** slaweq has quit IRC | 12:11 | |
| *** cleong has joined #openstack-nova | 12:17 | |
| *** sahid has joined #openstack-nova | 12:19 | |
| stephenfin | sdague: done | 12:24 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: tools: Remove dead script https://review.openstack.org/480138 | 12:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks' https://review.openstack.org/416520 | 12:26 |
| stephenfin | Couple of trivial "remove dead files" patches here, were anyone looking for easy +2s https://review.openstack.org/#/q/topic:trivial+owner:%22Stephen+Finucane+%253Cstephenfin%2540redhat.com%253E%22+status:open | 12:27 |
| *** pchavva has joined #openstack-nova | 12:29 | |
| *** catintheroof has joined #openstack-nova | 12:30 | |
| *** cdent has joined #openstack-nova | 12:33 | |
| *** baoli has quit IRC | 12:34 | |
| cdent | jaypipes: your hip/back/whatever any better? | 12:34 |
| *** baoli has joined #openstack-nova | 12:34 | |
| *** gcb has joined #openstack-nova | 12:35 | |
| *** maciejjozefczyk has joined #openstack-nova | 12:35 | |
| *** baoli has quit IRC | 12:39 | |
| maciejjozefczyk | Hello, im trying to create own periodic task outside upstream nova code (like nova.compute.manager tasks). Shouldn't it be registered the way like custom nova scheduler filters are ( option scheduler_available_filters in nova.conf)? Is it even possible to use both custom and generic periodic tasks in compute manager? | 12:41 |
| *** mamandle has quit IRC | 12:41 | |
| *** mgariepy has joined #openstack-nova | 12:42 | |
| *** mikal has quit IRC | 12:44 | |
| cdent | stephenfin: do you consider https://review.openstack.org/#/c/490952/ done now? It’s sort of hard to tell/know and review other than “sure, lgtm” | 12:48 |
| *** dave-mccowan has joined #openstack-nova | 12:49 | |
| bauzas | gibi: looking | 12:50 |
| bauzas | cdent: cdent: just to make sure, the -1s are about comments, right? | 12:54 |
| *** shan_afk has quit IRC | 12:54 | |
| cdent | bauzas: yes, as I tried to say on the comment: the code fix looks right, but the comments are misleading enough that they ought to be fixed | 12:55 |
| bauzas | cdent: okay | 12:55 |
| *** edmondsw has joined #openstack-nova | 12:55 | |
| bauzas | cdent: tbc, we still need to use the ReqSpec record in case we don't have the instance list | 12:55 |
| cdent | yes, that’s what my rewrite says | 12:56 |
| *** edmondsw_ has joined #openstack-nova | 12:57 | |
| *** edmondsw has quit IRC | 12:59 | |
| *** mikal has joined #openstack-nova | 13:00 | |
| *** edmondsw_ has quit IRC | 13:01 | |
| *** edmondsw has joined #openstack-nova | 13:02 | |
| *** esberglu has joined #openstack-nova | 13:03 | |
| *** sree_ has joined #openstack-nova | 13:04 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix migrate single instance when it was created concurrently https://review.openstack.org/491439 | 13:05 |
| *** sree_ is now known as Guest15209 | 13:05 | |
| *** liverpooler has joined #openstack-nova | 13:06 | |
| *** edmondsw has quit IRC | 13:07 | |
| *** slaweq has joined #openstack-nova | 13:08 | |
| jaypipes | cdent: yes, it is, thanks for asking. :) | 13:08 |
| * cdent is relieved; didn’t want to have shoot jay | 13:08 | |
| jaypipes | cdent: well, you might still have to shoot me, just not for that. | 13:09 |
| cdent | have you seen our new exciting bug(s) for today? | 13:09 |
| cdent | bauzas already fixed it, but it gives me dread | 13:09 |
| jaypipes | cdent: the resize big flavor one from gibi? | 13:09 |
| cdent | jaypipes: no, https://bugs.launchpad.net/nova/+bug/1708961 | 13:10 |
| openstack | Launchpad bug 1708961 in OpenStack Compute (nova) "migration of single instance from multi-instance request spec fails with IndexError" [Critical,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | 13:10 |
| cdent | I started doing some by hand testing over the weekend, bumped into that | 13:10 |
| *** lyan has joined #openstack-nova | 13:10 | |
| jaypipes | cdent: eww. :( | 13:10 |
| cdent | jaypipes: also bumped up against this question (not quite a bug, but an issue) https://bugs.launchpad.net/nova/+bug/1708958 | 13:11 |
| openstack | Launchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Low,Confirmed] | 13:11 |
| *** smatzek has joined #openstack-nova | 13:11 | |
| jaypipes | cdent: that's definitely not a bug. and frankly, we cover that in the scheduler's integration with the "service group API". | 13:11 |
| *** moshele has joined #openstack-nova | 13:11 | |
| bauzas | jaypipes: remember the discussion we had in the review about being conservative with num_instances ? then, kaboom. :) | 13:11 |
| cdent | jaypipes: it’s not a bug for nova | 13:12 |
| *** lucasxu has joined #openstack-nova | 13:12 | |
| jaypipes | cdent: hold up, I have a senior pug wandering around looking suspiciously prone to going the bathrooom.. | 13:12 |
| *** edmondsw has joined #openstack-nova | 13:12 | |
| bauzas | jaypipes: about the compute disabling, like I said in the comment, we have ComputeFilter for this | 13:12 |
| jaypipes | fuck. too late. | 13:12 |
| cdent | jaypipes: but it implies a reality mismatch between available resources | 13:12 |
| cdent | :( | 13:12 |
| bauzas | jaypipes: but we could possibly reduce the number of passed RPs to the scheduler if we have a way to know if the RP is stale | 13:13 |
| bauzas | so, like 50% a bug, and 50% a feature to me | 13:13 |
| *** slaweq has quit IRC | 13:13 | |
| *** edmondsw_ has joined #openstack-nova | 13:13 | |
| bauzas | cdent: actually, pushing the bug to Wishlist | 13:13 |
| *** edmondsw has quit IRC | 13:13 | |
| cdent | bauzas: that’s fine with me, it was mostly me fishing for information on how we expect things to be represented. that it is not currently breaking anything is groovy | 13:14 |
| *** baoli has joined #openstack-nova | 13:15 | |
| jaypipes | cdent, bauzas: sorry, back from picking up poop :( | 13:15 |
| jaypipes | cdent, bauzas: lemme discuss one thing at a time. | 13:15 |
| asettle_ | THanks sdague - looking now | 13:15 |
| cdent | same day different poop | 13:15 |
| bauzas | cdent: I'm fine too, I'm just putting it to Wishlist to make sure we remember it | 13:16 |
| bauzas | jaypipes: hah, fortunately for you that's a pug poop :) | 13:16 |
| jaypipes | bauzas: well, it is diarrhea this morning since 5:45am. | 13:17 |
| * bauzas just try to explain that he's having a Eurasier (50cm height) dug :p | 13:17 | |
| jaypipes | anyway, enough about poop. | 13:17 |
| jaypipes | cdent, bauzas: so, which bug to discuss first? | 13:17 |
| bauzas | jaypipes: arf | 13:17 |
| *** baoli has quit IRC | 13:17 | |
| cdent | jaypipes of the two I mentioned, I think we’re done already | 13:18 |
| bauzas | jaypipes: honestly, just the critical one bug | 13:18 |
| cdent | 1st is fixed, 2nd is not immediately relevant | 13:18 |
| bauzas | +1 | 13:18 |
| cdent | the implication, however, of the 1st, matt’s comments about summing instead of maxing, and some of alex comments about evacuate, suggests we have a bit more work todo to nail it all down | 13:19 |
| cdent | but progress is being made | 13:19 |
| *** baoli has joined #openstack-nova | 13:19 | |
| jaypipes | cdent: ya. | 13:20 |
| *** liverpooler has quit IRC | 13:27 | |
| *** liverpooler has joined #openstack-nova | 13:27 | |
| *** awaugama has joined #openstack-nova | 13:29 | |
| edleafe | Scheduler subteam meeting in 30 minutes in #openstack-meeting-alt | 13:30 |
| *** jangutter has joined #openstack-nova | 13:30 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Raise NoValidHost if no allocation candidates https://review.openstack.org/491491 | 13:30 |
| gibi | cdent, jaypipes, bauzas: my naive bugfix for the bug 1708637 | 13:31 |
| openstack | bug 1708637 in OpenStack Compute (nova) "nova does not properly claim resources when server resized to a too big flavor" [High,In progress] https://launchpad.net/bugs/1708637 - Assigned to Balazs Gibizer (balazs-gibizer) | 13:31 |
| gibi | cdent, jaypipes, bauzas: https://review.openstack.org/491491 | 13:31 |
| bauzas | gibi: looking | 13:31 |
| * cdent looks | 13:31 | |
| *** jangutter_ has quit IRC | 13:33 | |
| bauzas | gibi: unfortunately, that won't work | 13:34 |
| bauzas | gibi: just one word : CachingScheduler (actually, two) | 13:34 |
| bauzas | oh, fuck, nevermind | 13:35 |
| gibi | bauzas: I might need more words than two to understand the reason | 13:35 |
| bauzas | we're already under the USES_ALLOC_CANDIDATES conditional | 13:35 |
| bauzas | gibi: CachingScheduler isn't using Placement | 13:35 |
| bauzas | gibi: but on the other hand, it still runs the legacy filters | 13:36 |
| *** lbragstad has joined #openstack-nova | 13:36 | |
| bauzas | but like I said, we're under the conditional that asserts us that we already use placement | 13:36 |
| *** awaugama has quit IRC | 13:36 | |
| gibi | OK, I think I see | 13:37 |
| gibi | but then CoreFilter is still needed for the CachingScheduler. isn't it? | 13:37 |
| bauzas | correct | 13:39 |
| bauzas | well, if of course the operator wants it | 13:40 |
| bauzas | any filter can be disabled | 13:40 |
| gibi | sure | 13:40 |
| gibi | it just mean an infinite overallocation on such resource | 13:40 |
| *** slagle has quit IRC | 13:40 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Raise NoValidHost if no allocation candidates https://review.openstack.org/491491 | 13:41 |
| *** slagle has joined #openstack-nova | 13:42 | |
| jaypipes | gibi: did you run all the func tests for that patch? | 13:42 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Raise NoValidHost if no allocation candidates https://review.openstack.org/491491 | 13:42 |
| gibi | jaypipes: yes, I did and passed for me | 13:44 |
| gibi | jaypipes: most of the functional tests using CachingScheduler as far as I know | 13:44 |
| jaypipes | gibi: really? I don't think so... | 13:44 |
| * gibi digging a referenc | 13:44 | |
| gibi | jaypipes: hm, my bad, it is the ChanceScheduler not the CacheScheduler | 13:46 |
| gibi | s/CacheScheduler/CachingScheduler/ | 13:46 |
| jaypipes | gibi: if that's the case, that's a very serious bug in our functional tests. | 13:46 |
| cdent | the PlacementFixture was turned on functional test-wide recently | 13:46 |
| gibi | jaypipes: I'm trying to find the place it is set... | 13:46 |
| cdent | sorry, meant to add a ? on that ^ | 13:46 |
| jaypipes | gibi: the ChanceScheduler is worthless. | 13:47 |
| *** gbarros has joined #openstack-nova | 13:48 | |
| *** sridharg has quit IRC | 13:49 | |
| bauzas | jaypipes: people use chancescheduler for very simple functional tests involving the scheduler | 13:49 |
| bauzas | only a very few use filterscheduler | 13:49 |
| *** links has quit IRC | 13:50 | |
| jaypipes | bauzas: since chance scheduler doesn't use placement, that's a mistake IMHO. | 13:51 |
| jaypipes | we're not covering a large portion of our functional tests for no good reason. | 13:52 |
| *** sridharg has joined #openstack-nova | 13:52 | |
| * gibi failed to find evidence for his claim about func tests using ChanceScheduler | 13:53 | |
| *** randomhack has joined #openstack-nova | 13:53 | |
| bauzas | jaypipes: if we run only one compute and want to boot one instance, why should we run filterscheduler and placement ? | 13:53 |
| gibi | bauzas: do you know where the ChanceScheduler is set in the functional test env? | 13:53 |
| bauzas | gibi: just look at self.flags(driver='chance_scheduler', group='scheduler') | 13:53 |
| jaypipes | bauzas: because placement is now required. | 13:53 |
| jaypipes | bauzas: we will never be able to get rid of the code in the compute node that does "claiming" if the chance scheduler is allowed to keep on existing. same for caching scheduler, frankly. | 13:54 |
| gibi | bauzas: thanks. I always forget about the fact that it is chance_scheduler in the conf not ChanceScheduler | 13:54 |
| bauzas | jaypipes: that's 2 different concerns IMHO | 13:54 |
| bauzas | jaypipes: the main concerning one is that not all our drivers use placement | 13:55 |
| bauzas | jaypipes: and that, I fully agree, should be fixed | 13:55 |
| *** baoli has quit IRC | 13:55 | |
| dansmith | we're going to fix that by deprecating caching scheduler | 13:55 |
| bauzas | jaypipes: the second concern you express is that a large portion of our tests are not using filter_scheduler, but rather a dummy driver | 13:55 |
| jaypipes | bauzas: no, I don't think the chance scheduler should be fixed at all. it should be removed. it does nothing at all. | 13:55 |
| bauzas | jaypipes: that second concern is not a big deal to me | 13:56 |
| *** owalsh_ is now known as owalsh | 13:56 | |
| dansmith | I dunno about chance scheduler, but I'd kinda expect the same | 13:56 |
| *** mamandle has joined #openstack-nova | 13:56 | |
| jaypipes | bauzas: why is that second concern not a big deal to you? we're missing a huge % of coverage because of this. | 13:56 |
| bauzas | dansmith: jaypipes: well, I personnally think that having different drivers is good for nova | 13:56 |
| jaypipes | bauzas: yeah, I don't. | 13:56 |
| bauzas | dansmith: jaypipes: and I expressed my idea of having the same input data for all drivers | 13:56 |
| bauzas | just the scheduler algorithm should be different | 13:57 |
| dansmith | having a completely ridiculous scheduler (chance) is not useful just to have an alternate option, IMHO | 13:57 |
| *** baoli has joined #openstack-nova | 13:57 | |
| *** baoli has quit IRC | 13:57 | |
| * gibi is checking how big the kaboom if he removes chance_scheduler from the func env setup... | 13:57 | |
| bauzas | dansmith: well, I know some operators that use it :) | 13:57 |
| dansmith | and the reason to have the cachingscheduler should go away when we're claiming in the scheduler, so... no real reason to keep tht either | 13:58 |
| dansmith | bauzas: and? :) | 13:58 |
| jaypipes | bauzas: the caching scheduler doesn't implement a different "algorithm". it implements a different storage mechanism and a different behaviour in responding to the instance update events. | 13:58 |
| bauzas | I mean, filter scheduler being O(n2), some people tend to prefer other in-tree | 13:58 |
| *** asettle_ is now known as asettle | 13:59 | |
| bauzas | jaypipes: I agree | 13:59 |
| *** felipemonteiro has joined #openstack-nova | 13:59 | |
| *** felipemonteiro_ has joined #openstack-nova | 13:59 | |
| bauzas | jaypipes: let me rephrase, I just think that all drivers should by default get the list of hosts to verify by calling placement | 13:59 |
| jaypipes | bauzas: what do you think the caching scheduler's big O notation is? | 13:59 |
| bauzas | the same | 14:00 |
| jaypipes | right. | 14:00 |
| bauzas | but chance isn't | 14:00 |
| *** mriedem has joined #openstack-nova | 14:00 | |
| jaypipes | chance doesn't do anything at all. | 14:00 |
| edleafe | Scheduler subteam running now in #openstack-meeting-alt | 14:00 |
| mriedem | o/ | 14:00 |
| jaypipes | bauzas: it doesn't even consider if a compute has room for the workload. | 14:00 |
| bauzas | jaypipes: it does ONE thing, picking a host | 14:00 |
| bauzas | jaypipes: the rest is done by the compute claim | 14:01 |
| bauzas | that's a quick scheduler at least | 14:01 |
| bauzas | if you are enough spaced | 14:01 |
| dansmith | heh | 14:01 |
| jaypipes | bauzas: err, you and I have different ideas of efficiency I guess. | 14:01 |
| bauzas | yeah, I know, it's orthogonal to what we do | 14:01 |
| jaypipes | bauzas, dansmith: sched meeting in meeting-alt | 14:02 |
| bauzas | I'm just describing *why* people use it | 14:02 |
| *** crushil has joined #openstack-nova | 14:02 | |
| dansmith | bauzas: can you find any reference to support your claim that some people use chance successfully and recommend it in anything other than toy environments? | 14:03 |
| *** felipemonteiro has quit IRC | 14:03 | |
| bauzas | dansmith: unfortunately, only hallways talks | 14:03 |
| dansmith | the only such mention on the first page of google is a RAX presentation from 2013 that says "filter scheduler is the only legit one" | 14:03 |
| *** mingyu has joined #openstack-nova | 14:04 | |
| *** vks1 has joined #openstack-nova | 14:04 | |
| bauzas | anyway, I'm not particularly attached to chance_scheduler | 14:04 |
| bauzas | it's even broken if you use ironic | 14:04 |
| stephenfin | cdent: https://review.openstack.org/#/c/490952/ is pretty much done, yeah. All that needs to be checked if if I've missed anything from the import process | 14:04 |
| dansmith | bauzas: oh okay, seems like you're arguing to keep maintaining it | 14:04 |
| bauzas | I'm just trying to explain that I want to make sure we keep a clear separation in between what placement gives to drivers, and how driver work | 14:05 |
| *** baoli has joined #openstack-nova | 14:05 | |
| *** mingyu_ has quit IRC | 14:05 | |
| bauzas | dansmith: sorry, I'm unclear, I'm just giving food for thoughts about why we have it in-tree | 14:05 |
| bauzas | we could outsource chance_scheduler | 14:05 |
| bauzas | if we make clear the fact that it gets a list of RPs provided by Placement | 14:06 |
| bauzas | it == the scheduler driver interface | 14:06 |
| mriedem | the chance scheduler shouldn't even be asking placement for anything | 14:06 |
| mriedem | it sets a flag for that (or doesn't set it rather) | 14:06 |
| dansmith | I don't know what outsource means in this context | 14:06 |
| bauzas | dansmith: out-of-tree | 14:06 |
| dansmith | mriedem: well, the problem is it won't claim | 14:06 |
| dansmith | mriedem: so if we have it in tree it needs to at least do that | 14:07 |
| mriedem | dansmith: claim where? the scheduler? | 14:07 |
| mriedem | neither does the caching scheduler | 14:07 |
| dansmith | mriedem: right, same issue | 14:07 |
| mriedem | why is that a problem? that was intentional | 14:07 |
| dansmith | mriedem: because if we remove the claiming on the compute, we have no way of knowing if something will fit before sending it | 14:07 |
| mriedem | we aren't removing claims in the compute in pike | 14:08 |
| dansmith | which isn't so much a problem for chance since chance doesn't care about that at all, | 14:08 |
| dansmith | mriedem: I think we're talking about the long-term strategy here | 14:08 |
| bauzas | I really 'd love to have a whiteboard | 14:08 |
| mriedem | dansmith: ok - not sure why it's a problem or fire drill for this week | 14:08 |
| mriedem | maybe it's not | 14:08 |
| bauzas | I just see drivers as a black box from a placement perspective | 14:08 |
| mriedem | sounds like a distraction | 14:08 |
| dansmith | mriedem: I don't think it is | 14:08 |
| bauzas | #1 scheduler manager passes a list of candidates to the driver | 14:09 |
| *** marst has joined #openstack-nova | 14:09 | |
| bauzas | #2 driver does its black magic to find the perfect candidate | 14:09 |
| *** imacdonn has quit IRC | 14:09 | |
| bauzas | #3 scheduler manager would claim the allocation | 14:09 |
| bauzas | that is the loved interface I'd like | 14:09 |
| bauzas | so, one day, we could just do better things than just looping over the list of instances which loops over the list of hosts | 14:10 |
| *** slaweq has joined #openstack-nova | 14:10 | |
| dansmith | filter scheduler with no filters or weights configured is O(n) right? | 14:12 |
| dansmith | and is effectively chance, but without the pathological sending of instances to full computes | 14:12 |
| openstackgerrit | Merged openstack/nova-specs master: Amend spec for "Allow custom resource classes in flavor extra specs" https://review.openstack.org/481748 | 14:12 |
| bauzas | dansmith: in theory, we also loop over each filter | 14:13 |
| bauzas | dansmith: but since we have far less filters than hosts, I'm just keeping it O(n2) | 14:13 |
| dansmith | bauzas: well (a) that's O(1) and (b) that's irrelevant if there are no filters :) | 14:14 |
| bauzas | dansmith: the point is, what would be the interest of filter scheduler if we don't filter anything ? | 14:14 |
| dansmith | bauzas: there is zero interest of the chance scheduler, so I'm not sure what your point is :) | 14:14 |
| bauzas | dansmith: LOL | 14:14 |
| *** slaweq has quit IRC | 14:15 | |
| dansmith | filter scheduler with no filters gives you at least selection by "would fit at all" in constant time (per instance) with proper claiming and random selection of a host within the possible set | 14:15 |
| bauzas | dansmith: I see your point | 14:15 |
| *** jistr is now known as jistr|mtg | 14:16 | |
| bauzas | if you want to deprecate chance, that would the solution, I agree | 14:16 |
| dansmith | I'd be willing to bet a number of people assume chance does at least "would fit" and then once they read more and realize it doesn't, move on to something sane as they work through their POC | 14:16 |
| bauzas | would be | 14:16 |
| *** crushil has quit IRC | 14:17 | |
| gibi | 20% of func tests are failing after change chance_scheduler to filter_scheduler | 14:20 |
| gibi | let's see if it is something that easy to fix | 14:20 |
| cdent | gibi++ | 14:21 |
| *** awaugama has joined #openstack-nova | 14:21 | |
| *** crushil has joined #openstack-nova | 14:22 | |
| *** sridharg has quit IRC | 14:22 | |
| *** efried_WEEKEND is now known as efried | 14:25 | |
| mriedem | mikal: http://eavesdrop.openstack.org/meetings/nova/2017/nova.2017-07-27-14.00.log.html#l-67 | 14:26 |
| mriedem | mikal: if you'd read the meeting log | 14:27 |
| *** sridharg has joined #openstack-nova | 14:28 | |
| *** ekuris has quit IRC | 14:31 | |
| *** jangutter has quit IRC | 14:34 | |
| *** jangutter has joined #openstack-nova | 14:34 | |
| *** alexchadin has joined #openstack-nova | 14:35 | |
| edleafe | mriedem: you'd be the longest-serving PTL that nobody voted for | 14:37 |
| cdent | let’s get mdbooth to run for PTL | 14:38 |
| cdent | for variety | 14:38 |
| cdent | mriedem: I went ahead and backported that at-least-one allocation fix: v | 14:39 |
| cdent | https://review.openstack.org/#/c/491487/ | 14:39 |
| bauzas | cdent: +2d | 14:40 |
| cdent | thanks | 14:41 |
| *** baoli_ has joined #openstack-nova | 14:43 | |
| *** cdent has quit IRC | 14:44 | |
| *** baoli has quit IRC | 14:45 | |
| *** gcb_ has joined #openstack-nova | 14:45 | |
| *** vks1 has quit IRC | 14:46 | |
| *** gcb has quit IRC | 14:47 | |
| *** sridharg has quit IRC | 14:48 | |
| jaypipes | mriedem: k, commented on https://review.openstack.org/#/c/491098/1 | 14:48 |
| *** psachin has quit IRC | 14:51 | |
| *** jistr|mtg is now known as jistr | 14:52 | |
| *** gcb_ has quit IRC | 14:52 | |
| mriedem | jaypipes: thanks, replied | 14:54 |
| *** hongbin has joined #openstack-nova | 14:54 | |
| lennyb | dansmith: pls take a look #link https://bugs.launchpad.net/nova/+bug/1708920 | 14:55 |
| openstack | Launchpad bug 1708920 in OpenStack Compute (nova) "Cold migration fails" [Undecided,New] | 14:55 |
| *** alexchadin has quit IRC | 14:56 | |
| jaypipes | mriedem: no, Matt, we *are* removing the call to put_allocations() in Pike. | 14:56 |
| jaypipes | mriedem: that's what this does. https://review.openstack.org/#/c/491012/4/nova/compute/resource_tracker.py | 14:56 |
| *** jdillaman has joined #openstack-nova | 14:57 | |
| jaypipes | mriedem: what it doesn't do is stop the heal allocations thing from happening when ocata computes are in the mix. | 14:57 |
| jaypipes | mriedem: because we'll need to continually correct the mistake that the ocata compute will make. | 14:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove dead function, variable https://review.openstack.org/491512 | 14:58 |
| jaypipes | mriedem: so, w.r.t. the problem of shared storage, I think instead of adding the type of code you added in your patch, maybe we should just wait for Queens when we can assume no compute hosts are attempting to heal allocations in the compute node. | 14:59 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove some single use function from utils https://review.openstack.org/491513 | 14:59 |
| jaypipes | mriedem: I'd like dansmith's thoughts on that, though. | 14:59 |
| mriedem | jaypipes: i don't think we can claim support for shared storage at this point regardless | 14:59 |
| mriedem | we're basically implementing a feature now | 14:59 |
| *** alexchadin has joined #openstack-nova | 14:59 | |
| mriedem | which is why i un-completed the shared storage bp | 14:59 |
| dansmith | lennyb: that's not a lost connection to the cell database, that's a lack of configuration of some node for the api database. are you running conductors on your second (non-allinone) node? | 15:00 |
| *** yamahata has joined #openstack-nova | 15:00 | |
| jaypipes | mriedem: another thing I've been thinking of is instead of further mucking with the resource tracker and/or making the scheduler report client that we do a nova-status or nova-manage audit function that cleans up allocations based on the cell DB states. | 15:00 |
| jaypipes | mriedem: and once done, we have all computes on service version 22 (or whatever) and after that point, we remove all allocation stuff from the compute node entirely (other than the aforementioned delete allocations pieces) | 15:01 |
| *** vks11 has joined #openstack-nova | 15:01 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove files from 'tools' https://review.openstack.org/491444 | 15:02 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove "vif" script https://review.openstack.org/491443 | 15:02 |
| lennyb | dansmith, thanks, I will recheck it. | 15:03 |
| jaypipes | mriedem: because, if I'm being frank, the report client is starting to look like the mess of spaghetti code in the resource tracker that placement was supposed to simplify. :( | 15:03 |
| mriedem | jaypipes: the service version won't change until the actual code is upgraded though | 15:03 |
| mriedem | jaypipes: i agree with that | 15:03 |
| dansmith | is it alternate-ego mondays? why is jaypipes being "Frank" ? | 15:03 |
| *** markvoelker has quit IRC | 15:03 | |
| jaypipes | heh | 15:03 |
| *** crushil has quit IRC | 15:04 | |
| *** david-lyle has joined #openstack-nova | 15:04 | |
| jaypipes | mriedem: no, I understand that about the service version. what I'm suggesting is that we add a patch that does the "reconcile allocations by looking at cell DB state", we remove all allocation healing from the nova-computes (with a service version bump) and then we have the upgrade process be a run of the nova-manage/nova-status command that audits allocation information. | 15:05 |
| jaypipes | mriedem: the alternative is all the mess of conditionals and such that we need to add to the resource tracker and report client. | 15:06 |
| dansmith | jaypipes: so that means allocations are just all out of whack until you finish upgrading all your ocata computes and then run this fixer-upper thing? | 15:06 |
| jaypipes | mriedem: choose your poison I think? | 15:06 |
| *** markvoelker has joined #openstack-nova | 15:06 | |
| jaypipes | dansmith: yeah. | 15:06 |
| sdague | mriedem (or others) anyone want to approve these new docs sub pages - https://review.openstack.org/#/c/490994/ ? I'd like to get those 2 in before doing more to prevent wasted work | 15:06 |
| dansmith | jaypipes: that would be really unfortunate, IMHO | 15:06 |
| dansmith | jaypipes: some people take a looong time to do that upgrade | 15:06 |
| jaypipes | dansmith: as opposed to being out of whack repeatedly with pike computes fixing ocata data over and over again. | 15:06 |
| *** PsionTheory has joined #openstack-nova | 15:08 | |
| mriedem | sdague: i can look in a bit | 15:08 |
| *** cdent has joined #openstack-nova | 15:08 | |
| jaypipes | dansmith: do you think we should even attempt to fix the shared storage reporting in Pike? | 15:09 |
| jaypipes | dansmith: aka mriedem's https://review.openstack.org/#/c/491098/1 | 15:09 |
| dansmith | jaypipes: well, tbh, I thought that was out the window a while ago | 15:09 |
| dansmith | the computes have no idea which shared storage uuid a given thing is using anyway right? | 15:09 |
| jaypipes | dansmith: right. and ocatas are just going to pummel the allocations either way. | 15:10 |
| *** slaweq has joined #openstack-nova | 15:11 | |
| mriedem | jaypipes: dansmith: i'm resigned to give up on shared storage for pike, i'll put a known issue in the release notes | 15:11 |
| jaypipes | dansmith, mriedem: so maybe the best we can hope for in Pike is to land https://review.openstack.org/#/c/491012/ (the "stop healing allocations if all Pike" patch), just accept shared storage doesn't work and tell operators not to create shared providers in Pike. | 15:11 |
| mriedem | kind of weird to say, "this feature doesn't yet work" in a known issues section though | 15:11 |
| mriedem | i don't know of omission is better than clearly saying it's not supported | 15:12 |
| mriedem | i'd rather be clear personally | 15:12 |
| mriedem | of what is supported and what's not | 15:12 |
| jaypipes | mriedem: yeah, agreed | 15:12 |
| *** Oku_OS is now known as Oku_OS-away | 15:12 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Address review comments for contributor index https://review.openstack.org/491517 | 15:13 |
| jaypipes | mriedem: and at least, first thing in Queens we can get rid of the auto-healing ocata code, assume all computes are no longer auto-healing allocations and fix shared providers properly. | 15:13 |
| jaypipes | dansmith: agreed? | 15:13 |
| dansmith | I dunno why anyone would think it's supported unless you said it was, but okay | 15:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add additional content to admin guide https://review.openstack.org/490952 | 15:13 |
| *** slaweq has quit IRC | 15:16 | |
| bauzas | mmm, interesting, generation of relnotes fails if you have two reno files sharing the same explicit RST target | 15:16 |
| *** moshele has quit IRC | 15:18 | |
| *** bswartz has quit IRC | 15:25 | |
| *** annegentle has joined #openstack-nova | 15:25 | |
| *** bswartz has joined #openstack-nova | 15:27 | |
| jaypipes | dansmith: hey, did you catch my question above about agreeing/disagreeing with this: | 15:33 |
| jaypipes | mriedem: and at least, first thing in Queens we can get rid of the auto-healing ocata code, assume all computes are no longer auto-healing allocations and fix shared providers properly. | 15:33 |
| dansmith | jaypipes: yeah, but that's not a change in course right? | 15:34 |
| dansmith | I mean, that's been the plan? | 15:34 |
| dansmith | it'd be really nice if we were to remove the compute allocations behavior by removing all the legacy RT stuff in the compute node at the same time | 15:34 |
| dansmith | mriedem: in support of that goal, and circling back to the discussion this morning, shouldn't we deprecate chance and caching in pike? | 15:35 |
| jaypipes | dansmith: correct, that was the plan./ | 15:36 |
| dansmith | jaypipes: okay then, yea agreed :) | 15:36 |
| jaypipes | dansmith: w.r.t. to deprecating the caching and chance scheduler, I'd be a-ok with that, but that's also why I was quite concerned with the earlier conversation between gibi, me and bauzas where it was posited that our func tests were using the chance scheduler :( | 15:37 |
| *** markus_z has quit IRC | 15:37 | |
| bauzas | jaypipes: dansmith: looks a good discussion for Queens, nope ? | 15:37 |
| bauzas | at least for CachingScheduler | 15:37 |
| *** beekneemech is now known as bnemec | 15:37 | |
| dansmith | bauzas: very specifically we need to deprecate both now, which is what I just said | 15:38 |
| bauzas | because we could ask operators to try out our placement nice features | 15:38 |
| bauzas | dansmith: if that's all but just sending a signal, I'm okay | 15:38 |
| dansmith | bauzas: it's about deprecating things before we break them entirely | 15:38 |
| bauzas | dansmith: if that's implying that we will remove that in Queens, I dunno | 15:38 |
| dansmith | I just said it was about removing this stuff in queens, | 15:39 |
| dansmith | or at least being able to | 15:39 |
| jaypipes | bauzas: it's about deprecating in Pike. | 15:39 |
| jaypipes | bauzas: and removing in Queens. | 15:39 |
| dansmith | yes. | 15:39 |
| bauzas | tbh, I don't really care of chance scheduler | 15:39 |
| bauzas | I just gave explanation about why people use it | 15:39 |
| gibi | jaypipes: I'm changing the func test env to use filter_scheduler as we speak | 15:39 |
| gibi | jaypipes: it doesn't seems terribly hard to do | 15:39 |
| efried | jamielennox yt? | 15:39 |
| jaypipes | gibi: that's good news indeed. | 15:40 |
| dansmith | gibi: chance with one (fake) host should be pretty much the same behavior I'd hope :) | 15:40 |
| bauzas | I'm a little more concerned by cachingscheduler, which is AFAIK heavely used somewhere | 15:40 |
| bauzas | gibi: I don't expect real breakages tbh | 15:40 |
| bauzas | since tests using chance are just single-host | 15:40 |
| gibi | I think I can push that patch today | 15:40 |
| jaypipes | dansmith: well, the problem is chance essentially disables the placement API in scheduler, which disables claims in the scheduler, which means everything relies on allocations done on the compute node. :( | 15:41 |
| dansmith | jaypipes: ....right... did I say something wrong above? | 15:41 |
| dansmith | or you're just saying that changing that could break functional tests in weird ways? | 15:41 |
| dansmith | if the latter, that's why I said "pretty much the same behavior" :) | 15:42 |
| dansmith | ideally, and hopefully | 15:42 |
| jaypipes | dansmith: well, I was just saying that's the reason I was dismayed to hear from gibi that the chance scheduler was used for almost all the functional tests... | 15:42 |
| dansmith | oh, indeed | 15:43 |
| jaypipes | anyway, sounds like we're in violent agreement, so I'll shut it. | 15:43 |
| mriedem | we could always write a scheduler driver just used in tests - we have the fake driver | 15:43 |
| mriedem | we could change the fake driver to be like the chance driver | 15:43 |
| jaypipes | mriedem: I hope you're kidding. :) | 15:43 |
| mriedem | but best to just use stripped down filter scheduler in functional tests | 15:43 |
| mriedem | note that you can still plug in scheduler drivers | 15:43 |
| dansmith | jaypipes: I think he's kidding | 15:43 |
| mriedem | which won't use placement | 15:43 |
| jaypipes | dansmith: you never know with these South Dakatons. | 15:44 |
| mriedem | so if we deprecated caching and chance in pike b/c of removing allocations in the computes in queens, we should also deprecate the option | 15:44 |
| dansmith | if we're moving the responsibility of resource tracking and claiming to the scheduler, then that just means a driver that doesn't use placement should use something else to make sure it's making reasonable decisions | 15:44 |
| dansmith | which is fine I guess | 15:44 |
| mriedem | i think it would be ok to deprecate caching and chance as a signaling mechanism that new deployments shouldn't use those | 15:45 |
| dansmith | like compute manager being pluggable, also think it's unlikely anyone has written their own scheduler that doesn't break every release | 15:45 |
| mriedem | we removed the ability for the compute manager to be pluggable, or do you mean the virt driver? | 15:46 |
| dansmith | that's my point, | 15:46 |
| dansmith | there's basically zero chance you've written your own compute manager that does everything properly | 15:46 |
| dansmith | and same with scheduler, as we continue to move responsibility and apis around, there's no chance you wrote a scheduler in kilo and are still using it successfully today | 15:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469 | 15:47 |
| mriedem | well, you just didn't upgrade from kilo L) | 15:47 |
| mriedem | :) | 15:47 |
| mriedem | which is like 50% of deployments probably | 15:47 |
| bauzas | mriedem: by default, we use filter scheduler, but people tend to use cachingscheduler because they just want to not run all filters | 15:47 |
| bauzas | mriedem: I mean, when writing func tests | 15:47 |
| jaypipes | bauzas: hmm? | 15:47 |
| mriedem | huh? | 15:47 |
| mriedem | bauzas: people use caching scheduler for performance | 15:47 |
| dansmith | lol | 15:47 |
| dansmith | he means test writers I think | 15:48 |
| dansmith | I hope. | 15:48 |
| bauzas | mriedem: so if we provide them either a fixture or something easy like dansmith said about just running without filters, that would be sufficient I guess | 15:48 |
| mriedem | i don't think we can remove the caching scheduler until we've shown that the filter scheduler + claims in the scheduler outperforms caching scheduler at scale | 15:48 |
| bauzas | mriedem: jaypipes: huh, s/caching/chance | 15:48 |
| dansmith | mriedem: that means no removing the compute node claiming | 15:48 |
| dansmith | mriedem: which I think is really terrible | 15:48 |
| *** lpetrut has quit IRC | 15:49 | |
| bauzas | sorry, was just explaining that we can I guess easily remove chance if we just provide a very standard fixture that would just run filterscheduler without all filters | 15:49 |
| bauzas | (c) dansmith | 15:49 |
| dansmith | all the complexity that is going to make jaypipes' head explode is because we're double-claiming with different rules in the RT | 15:49 |
| bauzas | cachingscheduler is a totally different story | 15:49 |
| mriedem | dansmith: we should be able to test that before removal | 15:49 |
| bauzas | we want to remove caching because we do claims only in filter, right? | 15:49 |
| mriedem | anyway, we're not removing those today, if we want to signal i think that's fine | 15:50 |
| dansmith | mriedem: who is going to test that at scale do you think? | 15:50 |
| bauzas | jaypipes: can't we imagine to have caching scheduler to claim as well ? | 15:50 |
| dansmith | OSIC would have been an option, but tbh, I dunno who is likely to do it by the time we need to know in queens | 15:50 |
| bauzas | jaypipes: you said you were blocked when untangling that, but do you think it's doable tho ? | 15:50 |
| mriedem | dansmith: well, i know mirantis + intel has done some scale testing in the past, i don't know if those resources are still available, otherwise i'd like to try and see if huawei can do that, | 15:50 |
| *** alexchadin has quit IRC | 15:50 | |
| mriedem | because huawei public cloud is using the caching scheduler | 15:50 |
| jaypipes | bauzas: caching scheduler would need to stop using the cached compute node information it has and instead call ComputeNodeList.get_all_by_uuid(), which would make the caching scheduler === the filter scheduler. | 15:50 |
| dansmith | mriedem: okay well, that'd be cool | 15:51 |
| bauzas | jaypipes: caching has different input than filterscheduler, but we could still claim the result right? | 15:51 |
| dansmith | bauzas: not really | 15:52 |
| dansmith | bauzas: if it's not getting allocation candidates, it is missing some of what it needs | 15:52 |
| dansmith | bauzas: we could make it work, but it would be wonky | 15:52 |
| bauzas | I see | 15:52 |
| jaypipes | dansmith: and not worthwhile, imho. | 15:52 |
| dansmith | bauzas: or we'd have to sift through the results of /ac to find the thing we chose based on our cached data | 15:52 |
| dansmith | jaypipes: agreed | 15:52 |
| bauzas | so, yeah, benchmarks | 15:52 |
| dansmith | IIRC, the reason to keep caching scheduler after we moved to placement was because we weren't claiming and thus still making bad (cached) decisions in high traffic | 15:53 |
| bauzas | I need to disappear for family business, but I will watch out thec convo | 15:53 |
| bauzas | dansmith: you're correct | 15:53 |
| dansmith | so claiming now should resolve that I think, and I would expect the gains of being able to run multiple schedulers would be more win than loss | 15:53 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add a prelude section for Pike https://review.openstack.org/491424 | 15:53 |
| *** markvoelker has quit IRC | 15:53 | |
| bauzas | dansmith: right, I just mentioned that in the prelude ^ | 15:53 |
| bauzas | dansmith: we can now run multiple scheduler workers | 15:54 |
| bauzas | THAT is the big improvement | 15:54 |
| dansmith | I would expect to be able to brute-force scale the scheduler as high as you want with just that change alone | 15:54 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529 | 15:54 |
| *** tbachman has quit IRC | 15:54 | |
| bauzas | scale out you mean ? | 15:54 |
| *** aarefiev is now known as aarefiev_afk | 15:54 | |
| bauzas | or scale up the load ? | 15:54 |
| dansmith | bauzas: scale it to as high of a throughput as you need by scaling out yes | 15:55 |
| bauzas | k | 15:55 |
| bauzas | I agree | 15:55 |
| gibi | we can easily remove chance from the func test. here is the patch: https://review.openstack.org/491529 | 15:55 |
| bauzas | yeah | 15:56 |
| bauzas | not a big deal | 15:56 |
| bauzas | anyway I need to leave | 15:56 |
| bauzas | ++ | 15:56 |
| cdent | gibi: nice | 15:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Flatten functions https://review.openstack.org/367470 | 15:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions https://review.openstack.org/385071 | 15:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions https://review.openstack.org/385072 | 15:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list https://review.openstack.org/385074 | 15:56 |
| bauzas | FWIW we don't even need RetryFilter | 15:56 |
| bauzas | if we are singlehost | 15:57 |
| sdague | so... is someone going deprecate chance completely? | 15:57 |
| dansmith | sdague: and caching | 15:58 |
| *** ijw has joined #openstack-nova | 15:58 | |
| sdague | sure, that too | 15:58 |
| dansmith | we need to deprecate both in pike | 15:58 |
| dansmith | we can remove one or both when appropriate | 15:58 |
| sdague | ok, is that up for review? we're in rc week | 15:58 |
| *** tbachman has joined #openstack-nova | 15:58 | |
| jaypipes | gibi: why the change from 1 vcpu to 2 vcpus? that's weird.. | 15:58 |
| dansmith | sdague: are you asking if you can do it? | 15:59 |
| gibi | jaypipes: because there is resize to same host where the allocations are doubled up | 15:59 |
| *** yamamoto has quit IRC | 15:59 | |
| gibi | jaypipes: so at least 2 vcpu is needed | 15:59 |
| dansmith | gibi: yeah | 15:59 |
| jaypipes | gibi: guh, gotcha. | 15:59 |
| dansmith | makes sense, I had to do that in some of the tests as well | 15:59 |
| dansmith | jaypipes: this is the max_unit thing I was talking about with moves | 16:00 |
| dansmith | jaypipes: that is going to piss people off | 16:00 |
| dansmith | because allocation_ratio won't apply to single-host moves | 16:00 |
| *** Apoorva has joined #openstack-nova | 16:00 | |
| dansmith | migration uuid will fix that | 16:00 |
| jaypipes | dansmith: yeah, understood. not sure it's the end of the world, though. | 16:00 |
| cdent | migration uuid will fix everything! | 16:00 |
| dansmith | eff yeah it's the end of the world | 16:01 |
| jaypipes | :) | 16:01 |
| dansmith | I have a single-node cloud at home! | 16:01 |
| gibi | dansmith: only one? ;) | 16:01 |
| *** Apoorva has quit IRC | 16:01 | |
| dansmith | gibi: many nodes, but only one in the openstack deployment :) | 16:01 |
| mnaser | i'm going over the scheduler code and i've realized that we're pretty much sending instances to our biggest hypervisors and the smaller ones remain empty. the ram weighter by default weighs based on free memory, which means that a hypervisor with 128gb memory and all of it free will have less priority than one that has 384gb with 192gb free | 16:03 |
| mnaser | arguably.. shouldnt it be based on a percentage? | 16:03 |
| mnaser | https://github.com/openstack/nova/blob/master/nova/scheduler/weights/ram.py | 16:04 |
| jaypipes | mnaser: perhaps use the num instances weigher instead? | 16:04 |
| dansmith | or weigh the num_instances higher than the ram one | 16:04 |
| dansmith | mnaser: some people want to pack first | 16:04 |
| jaypipes | mnaser: nm, there is no such thing :( | 16:04 |
| dansmith | because windows licenses | 16:04 |
| mnaser | dansmith: packing would still work with -1.0 | 16:05 |
| dansmith | jaypipes: there's something like that pretty sure | 16:05 |
| mnaser | https://github.com/openstack/nova/tree/master/nova/scheduler/weights | 16:05 |
| mnaser | i dont see a num_instances on | 16:05 |
| mriedem | stephenfin: where does this come from? https://review.openstack.org/#/c/490952/3/doc/source/admin/security-groups.rst | 16:05 |
| mnaser | there's a fitler but not a weighter | 16:05 |
| stephenfin | mriedem: cli-nova-manage-projects-security.rst | 16:05 |
| stephenfin | (I think | 16:06 |
| jaypipes | mnaser: yeah :( | 16:06 |
| mriedem | oh nvm https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-nova-manage-projects-security.rst | 16:06 |
| mriedem | yeah | 16:06 |
| dansmith | mnaser: yeah, that's what I'm thinking of, heh | 16:06 |
| *** ircuser-1 has joined #openstack-nova | 16:06 | |
| mnaser | cause if it was percentage based, the ability to pack vs distribute would still work, but would be based on usage percentage rather than just absolute free memory | 16:06 |
| *** aarefiev_afk has quit IRC | 16:06 | |
| *** mamandle has quit IRC | 16:07 | |
| mnaser | i'm just slightly annoyed because i remember very clearly reading somewhere online (ML?) that someone had this exact issue and they dealt with it somehow but i figured "we won't deal with that" and here i am | 16:07 |
| *** udesale has quit IRC | 16:08 | |
| *** itlinux has joined #openstack-nova | 16:08 | |
| dansmith | man I really thought we had more weighers | 16:08 |
| *** udesale has joined #openstack-nova | 16:08 | |
| mriedem | stephenfin: so on https://review.openstack.org/#/c/490952/ I think we should have those various volume attach troubleshooting docs | 16:09 |
| jaypipes | mnaser: can't believe I'm suggesting this, but you could adapt the existing RAM weigher and make it work on percentages instead of raw capacity numbers. | 16:10 |
| jaypipes | mnaser: else submit a wishlist bug for one of us to get to. | 16:10 |
| jaypipes | mnaser: like dansmith I'm kinda surprised such a thing doesn't already exist :( | 16:10 |
| *** lyan has quit IRC | 16:11 | |
| mriedem | stephenfin: and maybe leave todo comments for combining the quotas pages | 16:11 |
| *** slaweq has joined #openstack-nova | 16:12 | |
| mriedem | stephenfin: if you want we can just deal with those comments in a follow up patch | 16:13 |
| *** Apoorva has joined #openstack-nova | 16:13 | |
| *** yangyapeng has quit IRC | 16:13 | |
| *** yangyapeng has joined #openstack-nova | 16:13 | |
| stephenfin | mriedem: Nah, I might as well address them there now | 16:14 |
| stephenfin | mriedem: Also, what are your thoughts on https://review.openstack.org/#/c/453808 ? Can it still go in before RC1 if a FFE isn't required/suitable? | 16:15 |
| *** slaweq has quit IRC | 16:17 | |
| *** Qiming has quit IRC | 16:18 | |
| mriedem | stephenfin: replied again | 16:19 |
| mriedem | i don't see why --version is different from --local_cell | 16:19 |
| mriedem | since they are both options | 16:19 |
| mriedem | plus i feel like this is a distraction right now | 16:19 |
| *** yangyapeng has quit IRC | 16:20 | |
| *** Qiming has joined #openstack-nova | 16:20 | |
| mriedem | sdague: how is https://review.openstack.org/#/c/490644/ different from https://review.openstack.org/#/c/478485/ ? | 16:21 |
| mriedem | note i can't get to docs-draft.openstack.org/44/490644/3/check/gate-nova-docs-ubuntu-xenial/bd5e01d//doc/build/html/ right now | 16:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 16:22 |
| sdague | mriedem: the TOC approach | 16:23 |
| sdague | mriedem: see commit message | 16:24 |
| *** dtp has joined #openstack-nova | 16:24 | |
| *** mingyu has quit IRC | 16:28 | |
| *** hemna_ has joined #openstack-nova | 16:29 | |
| *** udesale has quit IRC | 16:31 | |
| openstackgerrit | Merged openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | 16:39 |
| *** slaweq has joined #openstack-nova | 16:39 | |
| *** ijw has quit IRC | 16:40 | |
| *** sshwarts has quit IRC | 16:42 | |
| *** openstackstatus has quit IRC | 16:42 | |
| *** lyan has joined #openstack-nova | 16:43 | |
| mriedem | sdague: do we need to worry about redirects for anything in https://review.openstack.org/#/c/490644/ ? | 16:44 |
| *** openstackstatus has joined #openstack-nova | 16:44 | |
| *** ChanServ sets mode: +v openstackstatus | 16:44 | |
| mriedem | sdague: or is that handled by https://review.openstack.org/#/c/490644/3/doc/source/index.rst@186 ? | 16:44 |
| *** mingyu has joined #openstack-nova | 16:45 | |
| mnaser | jaypipes do you think proposing a fix to replace the existing ram weighter by percentage would be an acceptable change | 16:45 |
| mnaser | or introduce a new weighter instead? | 16:45 |
| *** hoonetorg has quit IRC | 16:45 | |
| mnaser | because i genuinely think a ram weighter based on free memory only is not correct | 16:45 |
| mriedem | sdague: nvm, looks like nothing actually moves | 16:47 |
| mriedem | so no redirects needed | 16:47 |
| jaypipes | mnaser: a new weigher would be better. | 16:49 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Fix migrate single instance when it was created concurrently https://review.openstack.org/491439 | 16:50 |
| edleafe | bauzas: ^^ just fixed the pep8 issue | 16:51 |
| *** gszasz has quit IRC | 16:52 | |
| sdague | mriedem: right, I did not move any source files | 16:55 |
| sdague | I was just trying to build landing pages, and tame the TOC on the left sidebar which is a confusing mess right now | 16:55 |
| mnaser | ehh | 16:56 |
| *** hoonetorg has joined #openstack-nova | 16:58 | |
| dansmith | jaypipes: mnaser agreed | 16:58 |
| dansmith | behavioral change may surprise people | 16:58 |
| mnaser | yep, that's what i thought, i'm working on something now | 16:59 |
| jaypipes | cool | 16:59 |
| stvnoyes | mriedem mdbooth - I updated review https://review.openstack.org/#/c/389798/ - it's a small change. please take a look when you have a minute. Thanks. | 16:59 |
| *** yamamoto has joined #openstack-nova | 17:00 | |
| jaypipes | dansmith: correct. | 17:01 |
| *** crushil has joined #openstack-nova | 17:02 | |
| *** yamamoto has quit IRC | 17:04 | |
| *** rybridges has quit IRC | 17:04 | |
| *** hemna_ has quit IRC | 17:05 | |
| *** mdnadeem has quit IRC | 17:06 | |
| *** tesseract has quit IRC | 17:12 | |
| *** rajathagasthya has joined #openstack-nova | 17:13 | |
| *** pcaruana has quit IRC | 17:15 | |
| stephenfin | mriedem: Further replies left on https://review.openstack.org/#/c/453808/ | 17:16 |
| stephenfin | I know it's a distraction and would understand if it were ignored but it totally blocks the (undoubtedly fabulous) cliff integration for another cycle which would be rather depressing | 17:17 |
| openstackgerrit | Merged openstack/nova master: Add Contributor Guide section page https://review.openstack.org/490644 | 17:18 |
| mriedem | stephenfin: why does --verbose block that but --local_cell does not? | 17:18 |
| mriedem | why doesn't cliff allow us to have options, in other words | 17:18 |
| *** sahid has quit IRC | 17:19 | |
| *** Swami has joined #openstack-nova | 17:19 | |
| *** burt has joined #openstack-nova | 17:21 | |
| *** rcernin has quit IRC | 17:22 | |
| * mriedem goes to lunch | 17:22 | |
| *** mriedem has quit IRC | 17:22 | |
| *** hongbin has quit IRC | 17:25 | |
| *** hongbin has joined #openstack-nova | 17:26 | |
| *** Guest15209 has quit IRC | 17:33 | |
| *** randomhack has quit IRC | 17:35 | |
| *** sambetts is now known as sambetts|afk | 17:36 | |
| *** vks11 has quit IRC | 17:37 | |
| *** randomhack has joined #openstack-nova | 17:38 | |
| *** annegentle has quit IRC | 17:41 | |
| *** slaweq has quit IRC | 17:43 | |
| *** aarefiev has joined #openstack-nova | 17:48 | |
| *** hongbin has quit IRC | 17:50 | |
| *** markvoelker has joined #openstack-nova | 17:54 | |
| *** vks1 has joined #openstack-nova | 17:56 | |
| *** gouthamr has joined #openstack-nova | 17:57 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher https://review.openstack.org/491561 | 17:58 |
| mnaser | dansmith jaypipes i havent contributed to nova in a while so i welcome all critique whenever you have time ^ | 17:58 |
| openstackgerrit | Merged openstack/python-novaclient master: tools: Remove dead script https://review.openstack.org/480138 | 17:58 |
| jaypipes | mnaser: cheers | 17:58 |
| *** itlinux has quit IRC | 17:59 | |
| *** vks1 has quit IRC | 18:03 | |
| *** slaweq has joined #openstack-nova | 18:04 | |
| *** randomhack has quit IRC | 18:04 | |
| *** itlinux has joined #openstack-nova | 18:05 | |
| *** slaweq has quit IRC | 18:09 | |
| *** yamahata has quit IRC | 18:09 | |
| *** baoli_ has quit IRC | 18:10 | |
| *** dtantsur is now known as dtantsur|afk | 18:12 | |
| *** lpetrut has joined #openstack-nova | 18:12 | |
| *** yangyapeng has joined #openstack-nova | 18:15 | |
| *** litao__ has quit IRC | 18:17 | |
| *** yangyapeng has quit IRC | 18:20 | |
| *** baoli has joined #openstack-nova | 18:21 | |
| *** itlinux has quit IRC | 18:21 | |
| *** itlinux has joined #openstack-nova | 18:22 | |
| *** ijw has joined #openstack-nova | 18:22 | |
| *** annegentle has joined #openstack-nova | 18:24 | |
| *** baoli has quit IRC | 18:25 | |
| *** slaweq has joined #openstack-nova | 18:25 | |
| *** baoli has joined #openstack-nova | 18:25 | |
| *** baoli has quit IRC | 18:25 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Handle ironicclient failures in Ironic driver https://review.openstack.org/487925 | 18:26 |
| *** slaweq has quit IRC | 18:27 | |
| *** ijw has quit IRC | 18:27 | |
| *** slaweq has joined #openstack-nova | 18:27 | |
| *** gbarros has quit IRC | 18:28 | |
| *** baoli has joined #openstack-nova | 18:28 | |
| *** markvoelker has quit IRC | 18:28 | |
| *** randomhack has joined #openstack-nova | 18:30 | |
| *** edmondsw_ has quit IRC | 18:32 | |
| *** edmondsw has joined #openstack-nova | 18:32 | |
| *** annegentle has quit IRC | 18:34 | |
| *** kfarr has joined #openstack-nova | 18:34 | |
| *** edmondsw_ has joined #openstack-nova | 18:35 | |
| *** edmondsw has quit IRC | 18:37 | |
| jaypipes | dansmith: what's the difference between rebuild and evacuate? | 18:37 |
| dansmith | jaypipes: rebuild is in place on the same compute node | 18:38 |
| dansmith | jaypipes: evacuate means "rebuild me somewhere else" and has to be run on a down compute node | 18:38 |
| dansmith | jaypipes: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | 18:38 |
| * edleafe was just looking up that link | 18:38 | |
| jaypipes | dansmith: thx. sorry, just needed a reminder after looking at the code... :( | 18:38 |
| dansmith | np | 18:38 |
| *** edmondsw_ has quit IRC | 18:39 | |
| *** lpetrut has quit IRC | 18:41 | |
| *** yamahata has joined #openstack-nova | 18:44 | |
| mnaser | dansmith jaypipes and evacuate when using ceph means start me somewhere else (not rebuilding) | 18:44 |
| jaypipes | mnaser: when using ceph *for the root disk*. :) | 18:45 |
| dansmith | mnaser: it's still a rebuild under the covers in nova, | 18:45 |
| dansmith | it just doesn't erase the disk | 18:45 |
| dansmith | which is covered in the above link | 18:45 |
| *** mriedem has joined #openstack-nova | 18:45 | |
| mnaser | yes, that's correct. when reading that link, volume-backed made me think cinder but indeed | 18:45 |
| dansmith | well, it says shared storage in one place too, but yeah | 18:46 |
| *** slaweq has quit IRC | 18:47 | |
| mriedem | +2 on bauzas' IndexError fix https://review.openstack.org/#/c/491439/ | 18:54 |
| *** nicolasbock has quit IRC | 18:57 | |
| sdague | mriedem: also this doc page when you get a chance - https://review.openstack.org/#/c/490994/ | 18:58 |
| sdague | mriedem: +A | 18:59 |
| mriedem | yup the tab is open | 18:59 |
| *** vishwanathj has quit IRC | 19:06 | |
| *** vishwanathj has joined #openstack-nova | 19:06 | |
| *** slaweq has joined #openstack-nova | 19:08 | |
| *** armax has joined #openstack-nova | 19:08 | |
| *** slaweq has quit IRC | 19:13 | |
| *** baoli has quit IRC | 19:15 | |
| *** rajathagasthya has quit IRC | 19:16 | |
| *** slaweq has joined #openstack-nova | 19:18 | |
| *** pchavva has quit IRC | 19:24 | |
| openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/491296 | 19:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a prelude section for Pike https://review.openstack.org/491424 | 19:26 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark max microversion for Pike in history doc https://review.openstack.org/491581 | 19:26 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for shared storage known issue https://review.openstack.org/491582 | 19:26 |
| *** gbarros has joined #openstack-nova | 19:26 | |
| *** reedip_afk has quit IRC | 19:33 | |
| *** slaweq has quit IRC | 19:33 | |
| *** xyang1 has joined #openstack-nova | 19:33 | |
| *** randomhack has quit IRC | 19:34 | |
| *** edmondsw has joined #openstack-nova | 19:38 | |
| *** randomhack has joined #openstack-nova | 19:41 | |
| *** priteau has quit IRC | 19:43 | |
| *** priteau has joined #openstack-nova | 19:44 | |
| *** reedip_afk has joined #openstack-nova | 19:45 | |
| *** awaugama has quit IRC | 19:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for shared storage known issue https://review.openstack.org/491582 | 19:46 |
| *** priteau has quit IRC | 19:48 | |
| openstackgerrit | Merged openstack/python-novaclient master: Fix reservation_id not supported by Nova API https://review.openstack.org/468444 | 19:49 |
| *** annegentle has joined #openstack-nova | 19:52 | |
| *** slaweq has joined #openstack-nova | 19:53 | |
| mriedem | stephenfin: why didn't we migrate the hypervisor-specific config references? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/config-reference/source/compute/hypervisor-hyper-v.rst | 19:58 |
| *** slaweq has quit IRC | 19:59 | |
| *** baoli has joined #openstack-nova | 19:59 | |
| *** gbarros has quit IRC | 20:00 | |
| *** thorst has quit IRC | 20:00 | |
| sdague | mriedem: I think the answer to many things is going to be there are so many documents, and the code reviews are slow enough, that it's taking a while to get them in | 20:00 |
| mriedem | i didn't know if it was intentional | 20:01 |
| sdague | it's also about finding the right places for things to go | 20:01 |
| sdague | yeh, I don't know, but I do know in building some of these sub pages that giving this thing any narative cohesion starting with 100 pages of mostly random notes people made at different depths, is challenging | 20:02 |
| openstackgerrit | Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589 | 20:03 |
| *** thorst has joined #openstack-nova | 20:03 | |
| *** aarefiev has quit IRC | 20:04 | |
| *** liverpooler has quit IRC | 20:04 | |
| openstackgerrit | Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589 | 20:06 |
| *** thorst has quit IRC | 20:07 | |
| mriedem | sdague: ok questions in https://review.openstack.org/#/c/490994/ | 20:10 |
| *** slaweq has joined #openstack-nova | 20:14 | |
| *** yangyapeng has joined #openstack-nova | 20:16 | |
| *** gouthamr has quit IRC | 20:17 | |
| *** thorst has joined #openstack-nova | 20:18 | |
| *** slaweq has quit IRC | 20:19 | |
| sdague | looking | 20:21 |
| *** yangyapeng has quit IRC | 20:21 | |
| sdague | mriedem: hopefully answers | 20:23 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Base _ContextAuthPlugin on BaseIdentityPlugin https://review.openstack.org/491203 | 20:23 |
| *** gouthamr has joined #openstack-nova | 20:24 | |
| mriedem | ok reply to the reply | 20:25 |
| sdague | mriedem / sfinucan / asettle - https://etherpad.openstack.org/p/ideal-nova-docs-landing-page as I stare at things about how to get to the next slice here, this is kind of what I was thinking about for an ideal front page flow | 20:25 |
| sdague | and how things would chunk up the rest of the way | 20:25 |
| mriedem | if the cells v2 layout thing stays in "Architecture Evolution Plans" as an ops guide then we probably need to rename "Architecture Evolution Plans" | 20:25 |
| sdague | mriedem: right, so that was next on the chopping block | 20:26 |
| sdague | you want to look at that etherpad quick? | 20:26 |
| mriedem | just did | 20:26 |
| sdague | so I think we could get there in 3 more patches. One that was the For Operators and one For Users and one smoothing out the index | 20:28 |
| sdague | and if we get a general ack that's how we want the world to look, I'll help write those bits | 20:29 |
| *** mingyu has quit IRC | 20:30 | |
| *** smatzek has quit IRC | 20:31 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher https://review.openstack.org/491561 | 20:32 |
| *** slaweq has joined #openstack-nova | 20:35 | |
| *** rajathagasthya has joined #openstack-nova | 20:35 | |
| *** gouthamr has quit IRC | 20:36 | |
| *** slaweq has quit IRC | 20:39 | |
| *** randomha1k has joined #openstack-nova | 20:40 | |
| *** cleong has quit IRC | 20:41 | |
| *** slaweq has joined #openstack-nova | 20:41 | |
| *** randomhack has quit IRC | 20:43 | |
| mriedem | i've already found some gems in the docs about stuff that's way old, like the nova-api service description only mentions xml requests | 20:44 |
| mriedem | and all the ec2 api stuff needs to go away | 20:44 |
| *** randomha1k has quit IRC | 20:44 | |
| *** moshele has joined #openstack-nova | 20:46 | |
| dansmith | mriedem: we're dropping CD support, we might as well go back to XML anyway right? | 20:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark max microversion for Pike in history doc https://review.openstack.org/491581 | 20:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for shared storage known issue https://review.openstack.org/491582 | 20:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a prelude section for Pike https://review.openstack.org/491424 | 20:47 |
| mriedem | did we mark json as experimental and never unmark it? | 20:47 |
| *** mingyu has joined #openstack-nova | 20:53 | |
| *** gjayavelu has joined #openstack-nova | 20:54 | |
| *** mriedem has quit IRC | 20:56 | |
| cdent | dansmith: naw, backward to SOAP and forward to GRPC | 20:56 |
| sdague | mriedem dropped already... eh? | 20:57 |
| *** annegentle has quit IRC | 20:59 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add functional test for local delete allocations https://review.openstack.org/470578 | 21:02 |
| *** mingyu has quit IRC | 21:08 | |
| *** crushil has quit IRC | 21:09 | |
| *** lucasxu has quit IRC | 21:15 | |
| *** priteau has joined #openstack-nova | 21:16 | |
| *** kfarr has quit IRC | 21:17 | |
| *** slaweq has quit IRC | 21:18 | |
| *** catintheroof has quit IRC | 21:20 | |
| *** yamamoto has joined #openstack-nova | 21:20 | |
| *** yamamoto has quit IRC | 21:25 | |
| *** markvoelker has joined #openstack-nova | 21:26 | |
| *** annegentle has joined #openstack-nova | 21:30 | |
| openstackgerrit | Merged openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks' https://review.openstack.org/416520 | 21:31 |
| *** slaweq has joined #openstack-nova | 21:39 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Raise NoValidHost if no allocation candidates https://review.openstack.org/491491 | 21:41 |
| *** slaweq has quit IRC | 21:44 | |
| *** priteau has quit IRC | 21:44 | |
| *** edmondsw has quit IRC | 21:45 | |
| *** edmondsw has joined #openstack-nova | 21:45 | |
| openstackgerrit | Merged openstack/nova master: Detach device from live domain even if not found on persistent https://review.openstack.org/488545 | 21:47 |
| openstackgerrit | Merged openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730 | 21:48 |
| *** edmondsw has quit IRC | 21:50 | |
| *** slaweq has joined #openstack-nova | 21:55 | |
| *** yamamoto has joined #openstack-nova | 21:56 | |
| cdent | jaypipes, dansmith: i’m somewhat lost on the current scheduler/placement situation. What’s currently on the urgent to do list? (I’m trying to make my plan for tomorrow morning) | 21:58 |
| dansmith | cdent: waiting for jaypipes to refresh the patches, AFAIK | 21:59 |
| jaypipes | dansmith, cdent: yeah, I'm still on it :( I'm lost in unit test hell. :( | 21:59 |
| * cdent sends cookies | 21:59 | |
| cdent | okay, I’ll go looking for those in the morning | 22:00 |
| *** markvoelker has quit IRC | 22:00 | |
| *** xyang1 has quit IRC | 22:00 | |
| openstackgerrit | Merged openstack/nova master: Use wsgi-intercept in OSAPIFixture https://review.openstack.org/486825 | 22:00 |
| jaypipes | cdent: thx for the cookies. :) | 22:00 |
| *** slaweq has quit IRC | 22:01 | |
| cdent | hmmm, I wonder if we wanted to merget that ^^ right now. Will be interesting to see how/if that impacts the gate. | 22:01 |
| *** mingyu has joined #openstack-nova | 22:09 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Base _ContextAuthPlugin on BaseIdentityPlugin https://review.openstack.org/491203 | 22:09 |
| *** esberglu has quit IRC | 22:09 | |
| *** esberglu has joined #openstack-nova | 22:10 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 22:11 |
| efried | sdague yt? | 22:11 |
| *** mingyu has quit IRC | 22:14 | |
| *** slaweq has joined #openstack-nova | 22:14 | |
| *** esberglu has quit IRC | 22:14 | |
| *** Apoorva_ has joined #openstack-nova | 22:15 | |
| *** yangyapeng has joined #openstack-nova | 22:17 | |
| *** Apoorva has quit IRC | 22:18 | |
| *** slaweq has quit IRC | 22:19 | |
| *** yangyapeng has quit IRC | 22:21 | |
| *** Apoorva_ has quit IRC | 22:24 | |
| *** ijw has joined #openstack-nova | 22:25 | |
| *** Apoorva has joined #openstack-nova | 22:25 | |
| *** esberglu has joined #openstack-nova | 22:25 | |
| *** gouthamr has joined #openstack-nova | 22:29 | |
| *** yamahata has quit IRC | 22:29 | |
| *** slaweq has joined #openstack-nova | 22:30 | |
| *** thorst has quit IRC | 22:32 | |
| *** markvoelker has joined #openstack-nova | 22:32 | |
| *** yassine has quit IRC | 22:33 | |
| *** yamahata has joined #openstack-nova | 22:33 | |
| *** sree has joined #openstack-nova | 22:34 | |
| *** yassine has joined #openstack-nova | 22:35 | |
| *** hongbin has joined #openstack-nova | 22:35 | |
| *** marst has quit IRC | 22:36 | |
| *** slaweq has quit IRC | 22:36 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529 | 22:38 |
| *** sree has quit IRC | 22:38 | |
| *** moshele has quit IRC | 22:39 | |
| *** ijw has quit IRC | 22:46 | |
| *** slaweq has joined #openstack-nova | 22:49 | |
| *** itlinux has quit IRC | 22:51 | |
| *** markvoelker has quit IRC | 22:54 | |
| *** edmondsw has joined #openstack-nova | 22:54 | |
| *** slaweq has quit IRC | 22:54 | |
| *** markvoelker has joined #openstack-nova | 22:54 | |
| *** PsionTheory has quit IRC | 22:58 | |
| *** lyan has quit IRC | 22:58 | |
| *** edmondsw has quit IRC | 22:58 | |
| *** markvoelker has quit IRC | 22:59 | |
| *** slaweq has joined #openstack-nova | 23:02 | |
| *** lyan has joined #openstack-nova | 23:08 | |
| *** slaweq has quit IRC | 23:08 | |
| *** annegentle has quit IRC | 23:09 | |
| *** lyan has quit IRC | 23:11 | |
| *** jaypipes has quit IRC | 23:19 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher https://review.openstack.org/491561 | 23:20 |
| *** sdague has quit IRC | 23:21 | |
| cdent | oh look at the time | 23:22 |
| *** cdent has quit IRC | 23:22 | |
| *** slaweq has joined #openstack-nova | 23:24 | |
| *** catintheroof has joined #openstack-nova | 23:28 | |
| *** slaweq has quit IRC | 23:30 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org will be offline momentarily at 00:00 utc for a quick reconfiguration-related restart | 23:33 | |
| *** itlinux has joined #openstack-nova | 23:34 | |
| openstackgerrit | Merged openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager https://review.openstack.org/485734 | 23:37 |
| *** itlinux has quit IRC | 23:37 | |
| *** dtp has quit IRC | 23:38 | |
| *** felipemonteiro_ has quit IRC | 23:39 | |
| *** catintheroof has quit IRC | 23:50 | |
| *** slaweq has joined #openstack-nova | 23:56 | |
| *** mingyu has joined #openstack-nova | 23:57 | |
| *** crushil has joined #openstack-nova | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!