Monday, 2017-08-07

*** edmondsw has joined #openstack-nova00:02
*** yangyapeng has joined #openstack-nova00:04
*** edmondsw has quit IRC00:06
*** thorst has joined #openstack-nova00:08
*** felipemonteiro_ has joined #openstack-nova00:11
*** yangyapeng has quit IRC00:11
*** thorst has quit IRC00:12
*** yassine has joined #openstack-nova00:13
*** gmann has quit IRC00:18
*** gmann has joined #openstack-nova00:18
*** ircuser-1 has quit IRC00:26
*** slagle has joined #openstack-nova00:28
*** itlinux has quit IRC00:29
*** sdake_ has quit IRC00:32
*** sdake_ has joined #openstack-nova00:32
*** itlinux has joined #openstack-nova00:33
*** itlinux has quit IRC00:33
openstackgerritTetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':'  https://review.openstack.org/49128200:42
*** zhurong has joined #openstack-nova00:43
*** kiennt has joined #openstack-nova00:47
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/49085900:49
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/48808600:51
*** mdnadeem has joined #openstack-nova00:52
*** zhurong has quit IRC00:53
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/49129600:53
*** Shunli has joined #openstack-nova00:58
*** phuongnh has joined #openstack-nova00:59
*** dixiaoli has joined #openstack-nova01:02
*** slagle has quit IRC01:08
*** yangyapeng has joined #openstack-nova01:10
*** yangyapeng has quit IRC01:10
*** trungnv has joined #openstack-nova01:14
*** markvoelker has joined #openstack-nova01:15
*** yangyapeng has joined #openstack-nova01:17
*** hongbin has joined #openstack-nova01:20
*** mdnadeem has quit IRC01:21
*** litao__ has joined #openstack-nova01:28
*** mdnadeem has joined #openstack-nova01:31
*** dixiaoli_ has joined #openstack-nova01:37
*** dixiaoli has quit IRC01:37
*** crushil has joined #openstack-nova01:38
*** randomhack has joined #openstack-nova01:41
*** randomhack has quit IRC01:46
*** liusheng has joined #openstack-nova01:47
openstackgerritAlex Xu proposed openstack/nova master: placement: filtering the resource provider id when delete trait association  https://review.openstack.org/48920501:49
*** edmondsw has joined #openstack-nova01:50
*** edmondsw has quit IRC01:54
*** markvoelker has quit IRC02:02
*** markvoelker has joined #openstack-nova02:02
*** thorst has joined #openstack-nova02:13
*** thorst has quit IRC02:18
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945102:20
*** tuanluong has joined #openstack-nova02:26
*** huanxie has joined #openstack-nova02:30
*** markvoelker has quit IRC02:34
*** tbachman has joined #openstack-nova02:55
*** mingyu has quit IRC03:06
*** liusheng has quit IRC03:06
*** owalsh_ has joined #openstack-nova03:06
*** liusheng has joined #openstack-nova03:07
*** mingyu has joined #openstack-nova03:07
*** felipemonteiro__ has joined #openstack-nova03:08
*** dave-mccowan has quit IRC03:09
*** owalsh has quit IRC03:10
*** felipemonteiro_ has quit IRC03:11
*** baoli has joined #openstack-nova03:13
*** thorst has joined #openstack-nova03:14
*** tbachman has quit IRC03:16
*** mamandle has joined #openstack-nova03:17
*** felipemonteiro__ has quit IRC03:18
*** thorst has quit IRC03:19
*** crushil has quit IRC03:26
*** kfox1111 has quit IRC03:29
*** sridharg has joined #openstack-nova03:36
openstackgerritDanfly proposed openstack/nova master: update availablity_zone in the instances table  when live-migration successes  https://review.openstack.org/49122603:37
*** Nel1x has quit IRC03:37
*** edmondsw has joined #openstack-nova03:38
*** edmondsw has quit IRC03:42
*** mamandle has quit IRC03:43
*** ykuo_ has quit IRC03:45
*** links has joined #openstack-nova03:49
*** nicolasbock has joined #openstack-nova03:51
*** ykuo_ has joined #openstack-nova03:57
*** hongbin has quit IRC04:01
*** udesale has joined #openstack-nova04:06
*** udesale has quit IRC04:15
*** gbarros has quit IRC04:16
*** udesale has joined #openstack-nova04:18
*** ratailor has joined #openstack-nova04:33
*** mingyu has quit IRC04:33
*** PsionTheory has joined #openstack-nova04:37
*** david-lyle has quit IRC04:38
*** udesale has quit IRC04:40
openstackgerritDanfly proposed openstack/nova master: update availablity_zone in the instances table when live-migration successes  https://review.openstack.org/49122604:40
openstackgerritTetsuro Nakamura proposed openstack/nova master: fix nova accepting invalid availability zone name with ':'  https://review.openstack.org/49134004:41
*** udesale has joined #openstack-nova04:41
*** slaweq has joined #openstack-nova04:42
*** zerda3 has joined #openstack-nova04:44
*** slaweq has quit IRC04:46
*** itlinux has joined #openstack-nova04:54
*** zerda3 has quit IRC04:57
*** Shunli has quit IRC05:06
*** Shunli has joined #openstack-nova05:06
*** mingyu has joined #openstack-nova05:09
*** mingyu has quit IRC05:15
*** thorst has joined #openstack-nova05:15
*** sree has joined #openstack-nova05:20
*** thorst has quit IRC05:20
*** jaosorior has joined #openstack-nova05:24
*** trinaths has joined #openstack-nova05:26
*** zhurong has joined #openstack-nova05:28
*** mamandle has joined #openstack-nova05:28
*** itlinux has quit IRC05:32
*** psachin has joined #openstack-nova05:36
*** kiennt has quit IRC05:37
*** moshele has joined #openstack-nova05:40
*** shan has joined #openstack-nova05:41
*** randomhack has joined #openstack-nova05:42
*** randomhack has quit IRC05:47
*** xinliang has joined #openstack-nova05:47
*** xinliang has quit IRC05:47
*** xinliang has joined #openstack-nova05:47
openstackgerritAlex Xu proposed openstack/nova master: placement: the common method to filter resource providers by traits  https://review.openstack.org/48920605:50
*** mamandle has quit IRC05:52
*** ekuris has joined #openstack-nova05:54
*** mamandle has joined #openstack-nova05:54
*** yangyapeng has quit IRC06:01
*** yangyapeng has joined #openstack-nova06:02
*** kiennt has joined #openstack-nova06:05
*** dixiaoli_ has quit IRC06:06
*** Oku_OS-away is now known as Oku_OS06:09
*** sridharg has quit IRC06:14
*** mingyu has joined #openstack-nova06:16
*** sridharg has joined #openstack-nova06:18
*** kiennt has quit IRC06:19
*** mamandle has quit IRC06:19
*** mingyu has quit IRC06:21
*** mingyu has joined #openstack-nova06:27
*** sridharg has quit IRC06:35
*** sridharg has joined #openstack-nova06:35
*** ratailor has quit IRC06:35
*** mamandle has joined #openstack-nova06:36
*** mingyu has quit IRC06:39
*** udesale__ has joined #openstack-nova06:41
*** udesale has quit IRC06:43
*** sridharg has quit IRC06:43
*** yangyape_ has joined #openstack-nova06:45
*** sridharg has joined #openstack-nova06:45
*** yangyapeng has quit IRC06:45
*** vks1 has joined #openstack-nova06:46
*** udesale__ has quit IRC06:48
*** udesale has joined #openstack-nova06:49
*** sridharg has quit IRC06:53
*** mingyu has joined #openstack-nova06:53
*** sridharg has joined #openstack-nova06:56
openstackgerritAlex Xu proposed openstack/nova master: placement: the common method to filter resource providers by traits  https://review.openstack.org/48920606:59
openstackgerritAlex Xu proposed openstack/nova master: placement: add traits support to get resource_providers with shared  https://review.openstack.org/47846406:59
*** sshwarts has joined #openstack-nova07:00
*** bhagyashris has joined #openstack-nova07:00
*** sridharg has quit IRC07:02
*** nicolasbock_ has joined #openstack-nova07:06
*** yangyape_ is now known as yangyapeng07:07
*** nicolasbock_ has quit IRC07:07
*** alexchadin has joined #openstack-nova07:07
*** sridharg has joined #openstack-nova07:08
*** rcernin has joined #openstack-nova07:08
*** mamandle has quit IRC07:14
*** edmondsw has joined #openstack-nova07:14
*** kiennt has joined #openstack-nova07:15
*** thorst has joined #openstack-nova07:16
*** mamandle has joined #openstack-nova07:17
*** ratailor has joined #openstack-nova07:17
*** kiennt_ has joined #openstack-nova07:18
*** markus_z has joined #openstack-nova07:18
*** edmondsw has quit IRC07:18
*** tesseract has joined #openstack-nova07:20
*** kiennt has quit IRC07:20
*** thorst has quit IRC07:21
*** udesale__ has joined #openstack-nova07:22
*** udesale has quit IRC07:23
*** sahid has joined #openstack-nova07:25
*** priteau has joined #openstack-nova07:28
bauzasgood morning Nova07:36
*** udesale__ has quit IRC07:42
*** udesale has joined #openstack-nova07:42
*** gibi_chaosmonkey is now known as gibi07:46
gibigood morning07:46
*** mamandle has quit IRC07:59
*** alexchadin has quit IRC08:01
*** sree_ has joined #openstack-nova08:04
*** sree_ is now known as Guest6666108:05
*** sree has quit IRC08:07
*** alexchadin has joined #openstack-nova08:07
*** moshele has quit IRC08:11
*** priteau has quit IRC08:14
*** moshele has joined #openstack-nova08:16
*** priteau has joined #openstack-nova08:17
*** markvoelker has joined #openstack-nova08:17
*** randomhack has joined #openstack-nova08:18
*** markvoelker_ has joined #openstack-nova08:19
*** aarefiev_afk is now known as aarefiev08:21
*** markvoelker has quit IRC08:23
*** randomhack has quit IRC08:24
*** alexchadin has quit IRC08:26
*** alexchadin has joined #openstack-nova08:28
*** baoli has quit IRC08:30
*** mamandle has joined #openstack-nova08:40
*** fnordahl_ is now known as fnordahl08:42
*** kiennt_ is now known as kiennt08:48
*** amuresan has joined #openstack-nova08:51
*** alexchadin has quit IRC08:59
*** edmondsw has joined #openstack-nova09:02
*** moshele has quit IRC09:05
*** edmondsw has quit IRC09:06
*** moshele has joined #openstack-nova09:09
*** mingyu_ has joined #openstack-nova09:12
*** mingyu has quit IRC09:12
openstackgerritVladyslav Drok proposed openstack/nova master: [placement] Add api-ref for allocation_candidates  https://review.openstack.org/48111209:12
openstackgerritVladyslav Drok proposed openstack/nova master: [placement] Add api-ref for usages  https://review.openstack.org/48056309:12
openstackgerritVladyslav Drok proposed openstack/nova master: [placement] Make placement_api_docs.py failing  https://review.openstack.org/48092409:12
openstackgerritVladyslav Drok proposed openstack/nova master: [placement] Add api-ref for RP usages  https://review.openstack.org/45010509:12
openstackgerritVladyslav Drok proposed openstack/nova master: [placement] Add api-ref for RP allocations  https://review.openstack.org/45055009:12
*** slaweq has joined #openstack-nova09:16
openstackgerritDanfly proposed openstack/nova master: update availablity_zone in the instances table after live-migration  https://review.openstack.org/49122609:16
*** thorst has joined #openstack-nova09:17
*** slaweq has quit IRC09:21
*** sambetts_ is now known as sambetts09:21
*** thorst has quit IRC09:22
*** sahid has quit IRC09:24
*** lennyb has quit IRC09:25
*** alexchadin has joined #openstack-nova09:26
*** jaosorior has quit IRC09:27
*** cdent has joined #openstack-nova09:28
*** Shunli has quit IRC09:28
cdentgibi: 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/170895809:30
openstackLaunchpad bug 1708961 in OpenStack Compute (nova) "migration of single instance from multi-instance request spec fails with IndexError" [Undecided,New]09:30
openstackLaunchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Low,New]09:30
*** yangyape_ has joined #openstack-nova09:31
*** tuanluong has quit IRC09:32
*** trungnv has quit IRC09:32
*** phuongnh has quit IRC09:32
*** hieulq has quit IRC09:32
*** yangyapeng has quit IRC09:32
*** trungnv has joined #openstack-nova09:32
*** tuanluong has joined #openstack-nova09:33
*** phuongnh has joined #openstack-nova09:33
*** hieulq has joined #openstack-nova09:33
*** PsionTheory has quit IRC09:36
bauzascdent: gibi: that doesn't help but I'm pretty concerned by how we just missed https://bugs.launchpad.net/nova/+bug/170896109:37
openstackLaunchpad bug 1708961 in OpenStack Compute (nova) "migration of single instance from multi-instance request spec fails with IndexError" [Undecided,New]09:37
bauzascdent: gibi: that is a serious blocker to me09:38
cdentbauzas: yeah, I think we’ve got some gaps in testing.09:38
*** sahid has joined #openstack-nova09:38
bauzasit's more than that IMHO09:38
bauzasI 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
gibibauzas: nice findings09:39
bauzasbut we totally forgot to take care of move operations09:39
cdentcareful review is important, but it shouldn’t be a substitute for real tests09:40
bauzasgibi: cdent: about https://bugs.launchpad.net/nova/+bug/1708958 we have ComputeFilter for that09:40
openstackLaunchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Low,Confirmed]09:40
cdentif we have to rely on careful review, then we need better tests09:40
bauzascdent: I specifically asked for conservative testing about that09:41
bauzasanyway, it's done now09:41
cdentbauzas: 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 IRC09:42
*** yamamoto_ has quit IRC09:44
*** jaosorior has joined #openstack-nova09:45
bauzascdent: just made a comment in the bug report to clarify what I want09:45
bauzascdent: I don't want placement to have nova-specifisms09:45
*** sahid has joined #openstack-nova09:45
gibiI mean, cdent, nice findings09:45
cdentyes09:45
bauzascdent: but I'd love placement to consider inventory freshness09:45
gibiI'm sooo slow today09:45
cdentbauzas: I think the issue is simpler than that. The choice is really: should disabling a hypervisor cause (something in nova) to delete the resource provider09:47
cdentI think it should09:47
cdentsince if a service is diabled it is not providing resources09:47
*** sdague has joined #openstack-nova09:47
bauzascdent: that would be one option, I agree09:48
*** dtantsur|afk is now known as dtantsur09:48
gibicdent: it is not that simple. As disabling a host doesn't mean that the VMs running on that host freed up all the resources09:48
gibicdent: so that host still providing resources to existing VMs09:48
bauzasgibi: from a placement perspective, we don't care about the space that RP has09:48
cdentbauzas: actaully gibi’s right: we want to use allocations to indicate quota use09:49
bauzasgibi: unless *cough cough* we have shared resources from that honst :)09:49
cdentand we can’t delete an rp (or inventory) if there are allocations in use09:49
bauzasyet another can of worms spotted IMHO09:49
cdentsigh, what a twisted web we weave09:49
bauzasanyway, should only be optimization, since we have ComputeFilter09:50
bauzasthat's just we're providing a whole list that can be long of potential candidates that are eventually not selected because of the computefilter09:50
bauzasagain, low09:51
* cdent nods09:51
bauzasthe other bug is more concerning me09:51
cdentI _think_ the fix is relatively straightforward on that one09:51
bauzasI'll try to look at a proper resolution once I'm done with writing the prelude section explaining how great we are :)09:51
cdentbut given how many gears within gears we have, who knows what the cascades will be09:52
* cdent sighs09:52
gibicdent: you mean this is an interesting challenge? :)09:52
cdentgibi: heh, that’s very positive of you for a monday morning09:53
bauzascdent: I really wanted to help placement bugfixing, permission to work on the critical one ? :)09:53
bauzasthe resize ones were just killing my brain09:53
*** shan is now known as shan_afk09:53
cdentbauzas: 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 bugs09:54
bauzasroger.09:54
cdentI set up a 6 node devstack over the weekend so I can do some real experimentation. I fear what I might find.09:54
gibicdent: I just want to motivate you to dig into it :)09:54
bauzaspermission is more for making sure I'm not duplicating efforts09:54
bauzaslike, /me taking mutex on it09:55
cdentbauzas: godspeed09:55
bauzasvoilà.09:55
* cdent coffees, eats, etcs09:55
bauzasFWIW, I'm in a perfect place for chasing bugs09:56
bauzaslitterally 200m away from the meditarean sea09:56
* bauzas doesn't use imperial metrics09:56
*** [1]amuresan has joined #openstack-nova09:59
*** amuresan has quit IRC10:01
*** [1]amuresan is now known as amuresan10:01
*** yamamoto has joined #openstack-nova10:02
*** gcb has quit IRC10:03
*** thorst has joined #openstack-nova10:05
*** lpetrut has joined #openstack-nova10:08
*** kiennt has quit IRC10:09
ratailorCan anybody help me in debugging this code block.10:09
ratailorhttps://github.com/openstack/nova/blob/master/nova/objects/host_mapping.py#L90-L9310:09
ratailorHow can I check value of api_models.HostMapping.host in above code block.10:10
stephenfinsdague, 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 IRC10:11
stephenfinI ask because I think I might need to move this document https://review.openstack.org/#/c/490952/1/doc/source/admin/migration.rst10:12
stephenfinand this, for evacuate https://review.openstack.org/#/c/490952/1/doc/source/admin/evacuate.rst10:12
bauzasstephenfin: live migrations are only possible for users that are having admin keystone rights10:13
stephenfinbauzas: and evacuations?10:13
bauzasstephenfin: AFAIR, evacuations are only for admin-only too since it requires a knowledge of your host health10:14
bauzasstephenfin: but I can check the policy10:14
bauzasstephenfin: the thing is, we show those operations by the CLI10:14
stephenfinbauzas: So they both warrant staying in the admin guide. I'll keep them there so. Thanks! :)10:14
stephenfinwhat do you mean?10:14
stephenfin"by the CLI"?10:14
bauzasstephenfin: I mean a single user can see those operations10:15
openstackgerritSylvain Bauza proposed openstack/nova master: Add a prelude section for Pike  https://review.openstack.org/49142410:15
stephenfinbauzas: You mean regardless of whether they're an admin or not?10:15
bauzasstephenfin: yup10:15
bauzasit will just give'em a 401 if they try10:15
*** yamamoto has quit IRC10:16
bauzasthe fact is, I don't remember which specific move operation is either admin-only or regular user, depending if people provided a target or not10:16
bauzasbut anyway, those two tools (live-mig and evac) are aimed for admins anyway10:17
*** slaweq has joined #openstack-nova10:17
stephenfinbauzas: Right, they can stay where they are now so10:18
* bauzas is just trying to generate the policy file to double-check10:18
bauzasit'll take a while tho, under 1Mbps b/w10:19
*** slaweq has quit IRC10:21
bauzasstephenfin: okay, confirmed, live-migrations and evacuations are admin-only, and cold migrations depend on whether you specify a target or not10:24
*** mdnadeem has quit IRC10:25
*** claudiub has joined #openstack-nova10:28
stephenfinbauzas: Awesome. It stays in the admin guide so10:30
*** yangyape_ has quit IRC10:30
*** yangyapeng has joined #openstack-nova10:31
*** yangyapeng has quit IRC10:33
*** zenoway has joined #openstack-nova10:34
*** yangyapeng has joined #openstack-nova10:35
*** tuanluong has quit IRC10:35
sdaguestephenfin: right, with default policy they are both admin only10:35
*** phuongnh has quit IRC10:35
sdaguebauzas: I'm confused about your single user statement10:36
*** lennyb has joined #openstack-nova10:37
*** yangyapeng has quit IRC10:39
*** krtaylor has quit IRC10:41
*** zenoway has quit IRC10:42
vks1sdague: hi10:42
sdaguevks1: morning10:44
vks1sdague: good morning10:45
vks1sdague: for REST API we use iso format for timestamp, which oslo API is used to generate that ?10:46
*** mdnadeem has joined #openstack-nova10:47
*** markvoelker has joined #openstack-nova10:49
*** alexchadin has quit IRC10:49
*** zenoway has joined #openstack-nova10:50
bauzassdague: sorry, was disconnected, which statement in particular?10:50
*** edmondsw has joined #openstack-nova10:50
*** krtaylor has joined #openstack-nova10:51
*** Guest66661 has quit IRC10:52
*** markvoelker_ has quit IRC10:53
*** markvoelker_ has joined #openstack-nova10:54
*** markvoelker has quit IRC10:54
*** edmondsw has quit IRC10:54
*** trinaths has left #openstack-nova10:54
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add additional content to admin guide  https://review.openstack.org/49095210:56
*** markvoelker has joined #openstack-nova10:57
*** tbachman has joined #openstack-nova10:57
sdaguevks1: I'm not sure it's coming from oslo10:59
*** markvoelker has quit IRC10:59
*** markvoelker_ has quit IRC11:00
sdaguevks1: I think they are going through this - https://github.com/openstack/nova/blob/6e77e75993c5b431c68c963bb06494ae5d36b944/nova/utils.py#L1344-L135511:00
*** markvoelker has joined #openstack-nova11:01
*** sahid has quit IRC11:02
*** strigazi_OFF is now known as strigazi11:05
*** slaweq has joined #openstack-nova11:06
*** gszasz has joined #openstack-nova11:07
*** slaweq has quit IRC11:10
*** sridharg has quit IRC11:11
*** sridharg has joined #openstack-nova11:11
openstackgerritSylvain Bauza proposed openstack/nova master: Fix migrate single instance when it was created concurrently  https://review.openstack.org/49143911:12
*** markvoelker_ has joined #openstack-nova11:12
*** yamamoto has joined #openstack-nova11:13
bauzascdent: gibi: ^11:14
* bauzas bailing out for lunch11:14
gibibauzas: looking11:14
*** markvoelker has quit IRC11:15
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove 'contrib' directory  https://review.openstack.org/49144311:22
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove files from 'tools'  https://review.openstack.org/49144411:22
*** jaypipes has joined #openstack-nova11:25
*** eharney has quit IRC11:27
gibibauzas: I left some comments in https://review.openstack.org/#/c/49143911:27
*** catintheroof has joined #openstack-nova11:28
*** markvoelker_ has quit IRC11:28
*** thorst has joined #openstack-nova11:28
*** markvoelker has joined #openstack-nova11:29
*** slagle has joined #openstack-nova11:31
*** thorst has quit IRC11:33
stephenfinasettle_: 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 IRC11:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resize with too big flavor  https://review.openstack.org/49081411:34
asettle_stephenfin: I can do thoughts11:34
stephenfinIt's pretty...icky right now and I haven't even started dragging in the stuff from the old configuration guide11:34
stephenfinasettle_: thoughts are good11:35
asettle_Okay, lemme go have a look at the beautiful build11:36
asettle_Well, that's just stunning isn't it11:36
asettle_I like the uhh... raw... blob ... of it all11:36
stephenfinWhat can I say11:36
stephenfinI've a knack11:36
*** yangyapeng has joined #openstack-nova11:37
asettle_You truly do11:37
asettle_Hmm, okay. I'll have a lookie and comment11:37
*** catintheroof has quit IRC11:38
*** yangyapeng has quit IRC11:39
*** yangyapeng has joined #openstack-nova11:39
asettle_It'd be cool to have this as Administration Guides or something similar :) (cause it's for the tasks, mostly)11:40
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resize with too big flavor  https://review.openstack.org/49081411:40
*** markvoelker has joined #openstack-nova11:40
*** zenoway has quit IRC11:40
stephenfinasettle_: Can't argue with that11:40
*** markvoelker_ has joined #openstack-nova11:41
stephenfinThe tasks seem to be broken up into two types two: cloud-level tasks and instance-level ones11:42
stephenfin*two types too11:42
*** markvoelker_ has quit IRC11:42
stephenfinI'd guess most of the configuration guides docs, once added, would fall into the former11:42
*** markvoelker_ has joined #openstack-nova11:44
*** zhurong has quit IRC11:44
*** markvoelker has quit IRC11:44
asettle_I would imagine so, ya11:44
asettle_Give me the afternoon to poke around :) I just gotta scoot out for lunch before afternoon meetings11:44
*** amuresan has quit IRC11:46
*** markvoelker has joined #openstack-nova11:47
*** liverpooler has quit IRC11:47
*** markvoelker_ has quit IRC11:50
*** catintheroof has joined #openstack-nova11:50
*** markvoelker_ has joined #openstack-nova11:51
*** markvoelker has quit IRC11:52
*** markvoelker has joined #openstack-nova11:53
*** catintheroof has quit IRC11:55
*** markvoelker_ has quit IRC11:56
*** markvoelker_ has joined #openstack-nova11:56
*** ratailor has quit IRC11:58
*** markvoelker has quit IRC12:00
*** cdent has quit IRC12:02
*** markvoelker has joined #openstack-nova12:02
*** markvoelker_ has quit IRC12:03
*** moshele has quit IRC12:04
*** moshele has joined #openstack-nova12:04
*** vks1 has quit IRC12:04
sdaguestephenfin / asettle_ another one of those subpages is here - https://review.openstack.org/#/c/490994/12:05
sdagueI think I pushed that after you all had kicked off for the weekend12:05
*** thorst has joined #openstack-nova12:05
*** jangutter_ has joined #openstack-nova12:06
*** slaweq has joined #openstack-nova12:07
*** jangutter has quit IRC12:07
*** moshele has quit IRC12:08
*** thorst has quit IRC12:09
*** mgariepy has quit IRC12:10
*** thorst has joined #openstack-nova12:10
*** baoli has joined #openstack-nova12:11
*** slaweq has quit IRC12:11
*** cleong has joined #openstack-nova12:17
*** sahid has joined #openstack-nova12:19
stephenfinsdague: done12:24
openstackgerritStephen Finucane proposed openstack/python-novaclient master: tools: Remove dead script  https://review.openstack.org/48013812:24
openstackgerritStephen Finucane proposed openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks'  https://review.openstack.org/41652012:26
stephenfinCouple 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:open12:27
*** pchavva has joined #openstack-nova12:29
*** catintheroof has joined #openstack-nova12:30
*** cdent has joined #openstack-nova12:33
*** baoli has quit IRC12:34
cdentjaypipes: your hip/back/whatever any better?12:34
*** baoli has joined #openstack-nova12:34
*** gcb has joined #openstack-nova12:35
*** maciejjozefczyk has joined #openstack-nova12:35
*** baoli has quit IRC12:39
maciejjozefczykHello, 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 IRC12:41
*** mgariepy has joined #openstack-nova12:42
*** mikal has quit IRC12:44
cdentstephenfin: 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-nova12:49
bauzasgibi: looking12:50
bauzascdent: cdent: just to make sure, the -1s are about comments, right?12:54
*** shan_afk has quit IRC12:54
cdentbauzas: 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 fixed12:55
bauzascdent: okay12:55
*** edmondsw has joined #openstack-nova12:55
bauzascdent: tbc, we still need to use the ReqSpec record in case we don't have the instance list12:55
cdentyes, that’s what my rewrite says12:56
*** edmondsw_ has joined #openstack-nova12:57
*** edmondsw has quit IRC12:59
*** mikal has joined #openstack-nova13:00
*** edmondsw_ has quit IRC13:01
*** edmondsw has joined #openstack-nova13:02
*** esberglu has joined #openstack-nova13:03
*** sree_ has joined #openstack-nova13:04
openstackgerritSylvain Bauza proposed openstack/nova master: Fix migrate single instance when it was created concurrently  https://review.openstack.org/49143913:05
*** sree_ is now known as Guest1520913:05
*** liverpooler has joined #openstack-nova13:06
*** edmondsw has quit IRC13:07
*** slaweq has joined #openstack-nova13:08
jaypipescdent: yes, it is, thanks for asking. :)13:08
* cdent is relieved; didn’t want to have shoot jay13:08
jaypipescdent: well, you might still have to shoot me, just not for that.13:09
cdenthave you seen our new exciting bug(s) for today?13:09
cdentbauzas already fixed it, but it gives me dread13:09
jaypipescdent: the resize big flavor one from gibi?13:09
cdentjaypipes: no, https://bugs.launchpad.net/nova/+bug/170896113:10
openstackLaunchpad 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
cdentI started doing some by hand testing over the weekend, bumped into that13:10
*** lyan has joined #openstack-nova13:10
jaypipescdent: eww. :(13:10
cdentjaypipes: also bumped up against this question (not quite a bug, but an issue) https://bugs.launchpad.net/nova/+bug/170895813:11
openstackLaunchpad bug 1708958 in OpenStack Compute (nova) "disabling a compute service does not disable the resource provider" [Low,Confirmed]13:11
*** smatzek has joined #openstack-nova13:11
jaypipescdent: 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-nova13:11
bauzasjaypipes: remember the discussion we had in the review about being conservative with num_instances ? then, kaboom. :)13:11
cdentjaypipes: it’s not a bug for nova13:12
*** lucasxu has joined #openstack-nova13:12
jaypipescdent: hold up, I have a senior pug wandering around looking suspiciously prone to going the bathrooom..13:12
*** edmondsw has joined #openstack-nova13:12
bauzasjaypipes: about the compute disabling, like I said in the comment, we have ComputeFilter for this13:12
jaypipesfuck. too late.13:12
cdentjaypipes: but it implies a reality mismatch between available resources13:12
cdent:(13:12
bauzasjaypipes: but we could possibly reduce the number of passed RPs to the scheduler if we have a way to know if the RP is stale13:13
bauzasso, like 50% a bug, and 50% a feature to me13:13
*** slaweq has quit IRC13:13
*** edmondsw_ has joined #openstack-nova13:13
bauzascdent: actually, pushing the bug to Wishlist13:13
*** edmondsw has quit IRC13:13
cdentbauzas: 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 groovy13:14
*** baoli has joined #openstack-nova13:15
jaypipescdent, bauzas: sorry, back from picking up poop :(13:15
jaypipescdent, bauzas: lemme discuss one thing at a time.13:15
asettle_THanks sdague - looking now13:15
cdentsame day different poop13:15
bauzascdent: I'm fine too, I'm just putting it to Wishlist to make sure we remember it13:16
bauzasjaypipes: hah, fortunately for you that's a pug poop :)13:16
jaypipesbauzas: 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 :p13:17
jaypipesanyway, enough about poop.13:17
jaypipescdent, bauzas: so, which bug to discuss first?13:17
bauzasjaypipes: arf13:17
*** baoli has quit IRC13:17
cdent jaypipes of the two I mentioned, I think we’re done already13:18
bauzasjaypipes: honestly, just the critical one bug13:18
cdent1st is fixed, 2nd is not immediately relevant13:18
bauzas+113:18
cdentthe 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 down13:19
cdentbut progress is being made13:19
*** baoli has joined #openstack-nova13:19
jaypipescdent: ya.13:20
*** liverpooler has quit IRC13:27
*** liverpooler has joined #openstack-nova13:27
*** awaugama has joined #openstack-nova13:29
edleafeScheduler subteam meeting in 30 minutes in #openstack-meeting-alt13:30
*** jangutter has joined #openstack-nova13:30
openstackgerritBalazs Gibizer proposed openstack/nova master: Raise NoValidHost if no allocation candidates  https://review.openstack.org/49149113:30
gibicdent, jaypipes, bauzas: my naive bugfix for the bug 170863713:31
openstackbug 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
gibicdent, jaypipes, bauzas: https://review.openstack.org/49149113:31
bauzasgibi: looking13:31
* cdent looks13:31
*** jangutter_ has quit IRC13:33
bauzasgibi: unfortunately, that won't work13:34
bauzasgibi: just one word : CachingScheduler (actually, two)13:34
bauzasoh, fuck, nevermind13:35
gibibauzas: I might need more words than two to understand the reason13:35
bauzaswe're already under the USES_ALLOC_CANDIDATES conditional13:35
bauzasgibi: CachingScheduler isn't using Placement13:35
bauzasgibi: but on the other hand, it still runs the legacy filters13:36
*** lbragstad has joined #openstack-nova13:36
bauzasbut like I said, we're under the conditional that asserts us that we already use placement13:36
*** awaugama has quit IRC13:36
gibiOK, I think I see13:37
gibibut then CoreFilter is still needed for the CachingScheduler. isn't it?13:37
bauzascorrect13:39
bauzaswell, if of course the operator wants it13:40
bauzasany filter can be disabled13:40
gibisure13:40
gibiit just mean an infinite overallocation on such resource13:40
*** slagle has quit IRC13:40
openstackgerritBalazs Gibizer proposed openstack/nova master: Raise NoValidHost if no allocation candidates  https://review.openstack.org/49149113:41
*** slagle has joined #openstack-nova13:42
jaypipesgibi: did you run all the func tests for that patch?13:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Raise NoValidHost if no allocation candidates  https://review.openstack.org/49149113:42
gibijaypipes: yes, I did and passed for me13:44
gibijaypipes: most of the functional tests using CachingScheduler as far as I know13:44
jaypipesgibi: really? I don't think so...13:44
* gibi digging a referenc13:44
gibijaypipes: hm, my bad, it is the ChanceScheduler not the CacheScheduler13:46
gibis/CacheScheduler/CachingScheduler/13:46
jaypipesgibi: if that's the case, that's a very serious bug in our functional tests.13:46
cdentthe PlacementFixture was turned on functional test-wide recently13:46
gibijaypipes: I'm trying to find the place it is set...13:46
cdentsorry, meant to add a ? on that ^13:46
jaypipesgibi: the ChanceScheduler is worthless.13:47
*** gbarros has joined #openstack-nova13:48
*** sridharg has quit IRC13:49
bauzasjaypipes: people use chancescheduler for very simple functional tests involving the scheduler13:49
bauzasonly a very few use filterscheduler13:49
*** links has quit IRC13:50
jaypipesbauzas: since chance scheduler doesn't use placement, that's a mistake IMHO.13:51
jaypipeswe're not covering a large portion of our functional tests for no good reason.13:52
*** sridharg has joined #openstack-nova13:52
* gibi failed to find evidence for his claim about func tests using ChanceScheduler13:53
*** randomhack has joined #openstack-nova13:53
bauzasjaypipes: if we run only one compute and want to boot one instance, why should we run filterscheduler and placement ?13:53
gibibauzas: do you know where the ChanceScheduler is set in the functional test env?13:53
bauzasgibi: just look at self.flags(driver='chance_scheduler', group='scheduler')13:53
jaypipesbauzas: because placement is now required.13:53
jaypipesbauzas: 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
gibibauzas: thanks. I always forget about the fact that it is chance_scheduler in the conf not ChanceScheduler13:54
bauzasjaypipes: that's 2 different concerns IMHO13:54
bauzasjaypipes: the main concerning one is that not all our drivers use placement13:55
bauzasjaypipes: and that, I fully agree, should be fixed13:55
*** baoli has quit IRC13:55
dansmithwe're going to fix that by deprecating caching scheduler13:55
bauzasjaypipes: the second concern you express is that a large portion of our tests are not using filter_scheduler, but rather a dummy driver13:55
jaypipesbauzas: no, I don't think the chance scheduler should be fixed at all. it should be removed. it does nothing at all.13:55
bauzasjaypipes: that second concern is not a big deal to me13:56
*** owalsh_ is now known as owalsh13:56
dansmithI dunno about chance scheduler, but I'd kinda expect the same13:56
*** mamandle has joined #openstack-nova13:56
jaypipesbauzas: why is that second concern not a big deal to you? we're missing a huge % of coverage because of this.13:56
bauzasdansmith: jaypipes: well, I personnally think that having different drivers is good for nova13:56
jaypipesbauzas: yeah, I don't.13:56
bauzasdansmith: jaypipes: and I expressed my idea of having the same input data for all drivers13:56
bauzasjust the scheduler algorithm should be different13:57
dansmithhaving a completely ridiculous scheduler (chance) is not useful just to have an alternate option, IMHO13:57
*** baoli has joined #openstack-nova13:57
*** baoli has quit IRC13:57
* gibi is checking how big the kaboom if he removes chance_scheduler from the func env setup...13:57
bauzasdansmith: well, I know some operators that use it :)13:57
dansmithand the reason to have the cachingscheduler should go away when we're claiming in the scheduler, so... no real reason to keep tht either13:58
dansmithbauzas: and? :)13:58
jaypipesbauzas: 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
bauzasI mean, filter scheduler being O(n2), some people tend to prefer other in-tree13:58
*** asettle_ is now known as asettle13:59
bauzasjaypipes: I agree13:59
*** felipemonteiro has joined #openstack-nova13:59
*** felipemonteiro_ has joined #openstack-nova13:59
bauzasjaypipes: let me rephrase, I just think that all drivers should by default get the list of hosts to verify by calling placement13:59
jaypipesbauzas: what do you think the caching scheduler's big O notation is?13:59
bauzasthe same14:00
jaypipesright.14:00
bauzasbut chance isn't14:00
*** mriedem has joined #openstack-nova14:00
jaypipeschance doesn't do anything at all.14:00
edleafeScheduler subteam running now in #openstack-meeting-alt14:00
mriedemo/14:00
jaypipesbauzas: it doesn't even consider if a compute has room for the workload.14:00
bauzasjaypipes: it does ONE thing, picking a host14:00
bauzasjaypipes: the rest is done by the compute claim14:01
bauzasthat's a quick scheduler at least14:01
bauzasif you are enough spaced14:01
dansmithheh14:01
jaypipesbauzas: err, you and I have different ideas of efficiency I guess.14:01
bauzasyeah, I know, it's orthogonal to what we do14:01
jaypipesbauzas, dansmith: sched meeting in meeting-alt14:02
bauzasI'm just describing *why* people use it14:02
*** crushil has joined #openstack-nova14:02
dansmithbauzas: 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 IRC14:03
bauzasdansmith: unfortunately, only hallways talks14:03
dansmiththe 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-nova14:04
*** vks1 has joined #openstack-nova14:04
bauzasanyway, I'm not particularly attached to chance_scheduler14:04
bauzasit's even broken if you use ironic14:04
stephenfincdent:  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 process14:04
dansmithbauzas: oh okay, seems like you're arguing to keep maintaining it14:04
bauzasI'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 work14:05
*** baoli has joined #openstack-nova14:05
*** mingyu_ has quit IRC14:05
bauzasdansmith: sorry, I'm unclear, I'm just giving food for thoughts about why we have it in-tree14:05
bauzaswe could outsource chance_scheduler14:05
bauzasif we make clear the fact that it gets a list of RPs provided by Placement14:06
bauzasit == the scheduler driver interface14:06
mriedemthe chance scheduler shouldn't even be asking placement for anything14:06
mriedemit sets a flag for that (or doesn't set it rather)14:06
dansmithI don't know what outsource means in this context14:06
bauzasdansmith: out-of-tree14:06
dansmithmriedem: well, the problem is it won't claim14:06
dansmithmriedem: so if we have it in tree it needs to at least do that14:07
mriedemdansmith: claim where? the scheduler?14:07
mriedemneither does the caching scheduler14:07
dansmithmriedem: right, same issue14:07
mriedemwhy is that a problem? that was intentional14:07
dansmithmriedem: because if we remove the claiming on the compute, we have no way of knowing if something will fit before sending it14:07
mriedemwe aren't removing claims in the compute in pike14:08
dansmithwhich isn't so much a problem for chance since chance doesn't care about that at all,14:08
dansmithmriedem: I think we're talking about the long-term strategy here14:08
bauzasI really 'd love to have a whiteboard14:08
mriedemdansmith: ok - not sure why it's a problem or fire drill for this week14:08
mriedemmaybe it's not14:08
bauzasI just see drivers as a black box from a placement perspective14:08
mriedemsounds like a distraction14:08
dansmithmriedem: I don't think it is14:08
bauzas#1 scheduler manager passes a list of candidates to the driver14:09
*** marst has joined #openstack-nova14:09
bauzas#2 driver does its black magic to find the perfect candidate14:09
*** imacdonn has quit IRC14:09
bauzas#3 scheduler manager would claim the allocation14:09
bauzasthat is the loved interface I'd like14:09
bauzasso, one day, we could just do better things than just looping over the list of instances which loops over the list of hosts14:10
*** slaweq has joined #openstack-nova14:10
dansmithfilter scheduler with no filters or weights configured is O(n) right?14:12
dansmithand is effectively chance, but without the pathological sending of instances to full computes14:12
openstackgerritMerged openstack/nova-specs master: Amend spec for "Allow custom resource classes in flavor extra specs"  https://review.openstack.org/48174814:12
bauzasdansmith: in theory, we also loop over each filter14:13
bauzasdansmith: but since we have far less filters than hosts, I'm just keeping it O(n2)14:13
dansmithbauzas: well (a) that's O(1) and (b) that's irrelevant if there are no filters :)14:14
bauzasdansmith: the point is, what would be the interest of filter scheduler if we don't filter anything ?14:14
dansmithbauzas: there is zero interest of the chance scheduler, so I'm not sure what your point is :)14:14
bauzasdansmith: LOL14:14
*** slaweq has quit IRC14:15
dansmithfilter 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 set14:15
bauzasdansmith: I see your point14:15
*** jistr is now known as jistr|mtg14:16
bauzasif you want to deprecate chance, that would the solution, I agree14:16
dansmithI'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 POC14:16
bauzaswould be14:16
*** crushil has quit IRC14:17
gibi20% of func tests are failing after change chance_scheduler to filter_scheduler14:20
gibilet's see if it is something that easy to fix14:20
cdentgibi++14:21
*** awaugama has joined #openstack-nova14:21
*** crushil has joined #openstack-nova14:22
*** sridharg has quit IRC14:22
*** efried_WEEKEND is now known as efried14:25
mriedemmikal: http://eavesdrop.openstack.org/meetings/nova/2017/nova.2017-07-27-14.00.log.html#l-6714:26
mriedemmikal: if you'd read the meeting log14:27
*** sridharg has joined #openstack-nova14:28
*** ekuris has quit IRC14:31
*** jangutter has quit IRC14:34
*** jangutter has joined #openstack-nova14:34
*** alexchadin has joined #openstack-nova14:35
edleafemriedem: you'd be the longest-serving PTL that nobody voted for14:37
cdentlet’s get mdbooth to run for PTL14:38
cdentfor variety14:38
cdentmriedem: I went ahead and backported that at-least-one allocation fix: v14:39
cdenthttps://review.openstack.org/#/c/491487/14:39
bauzascdent: +2d14:40
cdentthanks14:41
*** baoli_ has joined #openstack-nova14:43
*** cdent has quit IRC14:44
*** baoli has quit IRC14:45
*** gcb_ has joined #openstack-nova14:45
*** vks1 has quit IRC14:46
*** gcb has quit IRC14:47
*** sridharg has quit IRC14:48
jaypipesmriedem: k, commented on https://review.openstack.org/#/c/491098/114:48
*** psachin has quit IRC14:51
*** jistr|mtg is now known as jistr14:52
*** gcb_ has quit IRC14:52
mriedemjaypipes: thanks, replied14:54
*** hongbin has joined #openstack-nova14:54
lennybdansmith: pls take a look #link https://bugs.launchpad.net/nova/+bug/170892014:55
openstackLaunchpad bug 1708920 in OpenStack Compute (nova) "Cold migration fails" [Undecided,New]14:55
*** alexchadin has quit IRC14:56
jaypipesmriedem: no, Matt, we *are* removing the call to put_allocations() in Pike.14:56
jaypipesmriedem: that's what this does. https://review.openstack.org/#/c/491012/4/nova/compute/resource_tracker.py14:56
*** jdillaman has joined #openstack-nova14:57
jaypipesmriedem: what it doesn't do is stop the heal allocations thing from happening when ocata computes are in the mix.14:57
jaypipesmriedem: because we'll need to continually correct the mistake that the ocata compute will make.14:57
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove dead function, variable  https://review.openstack.org/49151214:58
jaypipesmriedem: 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
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove some single use function from utils  https://review.openstack.org/49151314:59
jaypipesmriedem: I'd like dansmith's thoughts on that, though.14:59
mriedemjaypipes: i don't think we can claim support for shared storage at this point regardless14:59
mriedemwe're basically implementing a feature now14:59
*** alexchadin has joined #openstack-nova14:59
mriedemwhich is why i un-completed the shared storage bp14:59
dansmithlennyb: 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-nova15:00
jaypipesmriedem: 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
jaypipesmriedem: 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-nova15:01
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove files from 'tools'  https://review.openstack.org/49144415:02
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove "vif" script  https://review.openstack.org/49144315:02
lennybdansmith, thanks, I will recheck it.15:03
jaypipesmriedem: 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
mriedemjaypipes: the service version won't change until the actual code is upgraded though15:03
mriedemjaypipes: i agree with that15:03
dansmithis it alternate-ego mondays? why is jaypipes being "Frank" ?15:03
*** markvoelker has quit IRC15:03
jaypipesheh15:03
*** crushil has quit IRC15:04
*** david-lyle has joined #openstack-nova15:04
jaypipesmriedem: 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
jaypipesmriedem: the alternative is all the mess of conditionals and such that we need to add to the resource tracker and report client.15:06
dansmithjaypipes: 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
jaypipesmriedem: choose your poison I think?15:06
*** markvoelker has joined #openstack-nova15:06
jaypipesdansmith: yeah.15:06
sdaguemriedem (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 work15:06
dansmithjaypipes: that would be really unfortunate, IMHO15:06
dansmithjaypipes: some people take a looong time to do that upgrade15:06
jaypipesdansmith: as opposed to being out of whack repeatedly with pike computes fixing ocata data over and over again.15:06
*** PsionTheory has joined #openstack-nova15:08
mriedemsdague: i can look in a bit15:08
*** cdent has joined #openstack-nova15:08
jaypipesdansmith: do you think we should even attempt to fix the shared storage reporting in Pike?15:09
jaypipesdansmith: aka mriedem's https://review.openstack.org/#/c/491098/115:09
dansmithjaypipes: well, tbh, I thought that was out the window a while ago15:09
dansmiththe computes have no idea which shared storage uuid a given thing is using anyway right?15:09
jaypipesdansmith: right. and ocatas are just going to pummel the allocations either way.15:10
*** slaweq has joined #openstack-nova15:11
mriedemjaypipes: dansmith: i'm resigned to give up on shared storage for pike, i'll put a known issue in the release notes15:11
jaypipesdansmith, 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
mriedemkind of weird to say, "this feature doesn't yet work" in a known issues section though15:11
mriedemi don't know of omission is better than clearly saying it's not supported15:12
mriedemi'd rather be clear personally15:12
mriedemof what is supported and what's not15:12
jaypipesmriedem: yeah, agreed15:12
*** Oku_OS is now known as Oku_OS-away15:12
openstackgerritStephen Finucane proposed openstack/nova master: doc: Address review comments for contributor index  https://review.openstack.org/49151715:13
jaypipesmriedem: 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
jaypipesdansmith: agreed?15:13
dansmithI dunno why anyone would think it's supported unless you said it was, but okay15:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add additional content to admin guide  https://review.openstack.org/49095215:13
*** slaweq has quit IRC15:16
bauzasmmm, interesting, generation of relnotes fails if you have two reno files sharing the same explicit RST target15:16
*** moshele has quit IRC15:18
*** bswartz has quit IRC15:25
*** annegentle has joined #openstack-nova15:25
*** bswartz has joined #openstack-nova15:27
jaypipesdansmith: hey, did you catch my question above about agreeing/disagreeing with this:15:33
jaypipesmriedem: 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
dansmithjaypipes: yeah, but that's not a change in course right?15:34
dansmithI mean, that's been the plan?15:34
dansmithit'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 time15:34
dansmithmriedem: in support of that goal, and circling back to the discussion this morning, shouldn't we deprecate chance and caching in pike?15:35
jaypipesdansmith: correct, that was the plan./15:36
dansmithjaypipes: okay then, yea agreed :)15:36
jaypipesdansmith: 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 IRC15:37
bauzasjaypipes: dansmith: looks a good discussion for Queens, nope ?15:37
bauzasat least for CachingScheduler15:37
*** beekneemech is now known as bnemec15:37
dansmithbauzas: very specifically we need to deprecate both now, which is what I just said15:38
bauzasbecause we could ask operators to try out our placement nice features15:38
bauzasdansmith: if that's all but just sending a signal, I'm okay15:38
dansmithbauzas: it's about deprecating things before we break them entirely15:38
bauzasdansmith: if that's implying that we will remove that in Queens, I dunno15:38
dansmithI just said it was about removing this stuff in queens,15:39
dansmithor at least being able to15:39
jaypipesbauzas: it's about deprecating in Pike.15:39
jaypipesbauzas: and removing in Queens.15:39
dansmithyes.15:39
bauzastbh, I don't really care of chance scheduler15:39
bauzasI just gave explanation about why people use it15:39
gibijaypipes: I'm changing the func test env to use filter_scheduler as we speak15:39
gibijaypipes: it doesn't seems terribly hard to do15:39
efriedjamielennox yt?15:39
jaypipesgibi: that's good news indeed.15:40
dansmithgibi: chance with one (fake) host should be pretty much the same behavior I'd hope :)15:40
bauzasI'm a little more concerned by cachingscheduler, which is AFAIK heavely used somewhere15:40
bauzasgibi: I don't expect real breakages tbh15:40
bauzassince tests using chance are just single-host15:40
gibiI think I can push that patch today15:40
jaypipesdansmith: 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
dansmithjaypipes: ....right... did I say something wrong above?15:41
dansmithor you're just saying that changing that could break functional tests in weird ways?15:41
dansmithif the latter, that's why I said "pretty much the same behavior" :)15:42
dansmithideally, and hopefully15:42
jaypipesdansmith: 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
dansmithoh, indeed15:43
jaypipesanyway, sounds like we're in violent agreement, so I'll shut it.15:43
mriedemwe could always write a scheduler driver just used in tests - we have the fake driver15:43
mriedemwe could change the fake driver to be like the chance driver15:43
jaypipesmriedem: I hope you're kidding. :)15:43
mriedembut best to just use stripped down filter scheduler in functional tests15:43
mriedemnote that you can still plug in scheduler drivers15:43
dansmithjaypipes: I think he's kidding15:43
mriedemwhich won't use placement15:43
jaypipesdansmith: you never know with these South Dakatons.15:44
mriedemso if we deprecated caching and chance in pike b/c of removing allocations in the computes in queens, we should also deprecate the option15:44
dansmithif 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 decisions15:44
dansmithwhich is fine I guess15:44
mriedemi think it would be ok to deprecate caching and chance as a signaling mechanism that new deployments shouldn't use those15:45
dansmithlike compute manager being pluggable, also think it's unlikely anyone has written their own scheduler that doesn't break every release15:45
mriedemwe removed the ability for the compute manager to be pluggable, or do you mean the virt driver?15:46
dansmiththat's my point,15:46
dansmiththere's basically zero chance you've written your own compute manager that does everything properly15:46
dansmithand 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 today15:46
openstackgerritStephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group  https://review.openstack.org/46246915:47
mriedemwell, you just didn't upgrade from kilo L)15:47
mriedem:)15:47
mriedemwhich is like 50% of deployments probably15:47
bauzasmriedem: by default, we use filter scheduler, but people tend to use cachingscheduler because they just want to not run all filters15:47
bauzasmriedem: I mean, when writing func tests15:47
jaypipesbauzas: hmm?15:47
mriedemhuh?15:47
mriedembauzas: people use caching scheduler for performance15:47
dansmithlol15:47
dansmithhe means test writers I think15:48
dansmithI hope.15:48
bauzasmriedem: so if we provide them either a fixture or something easy like dansmith said about just running without filters, that would be sufficient I guess15:48
mriedemi 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 scale15:48
bauzasmriedem: jaypipes: huh, s/caching/chance15:48
dansmithmriedem: that means no removing the compute node claiming15:48
dansmithmriedem: which I think is really terrible15:48
*** lpetrut has quit IRC15:49
bauzassorry, 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 filters15:49
bauzas(c) dansmith15:49
dansmithall the complexity that is going to make jaypipes' head explode is because we're double-claiming with different rules in the RT15:49
bauzascachingscheduler is a totally different story15:49
mriedemdansmith: we should be able to test that before removal15:49
bauzaswe want to remove caching because we do claims only in filter, right?15:49
mriedemanyway, we're not removing those today, if we want to signal i think that's fine15:50
dansmithmriedem: who is going to test that at scale do you think?15:50
bauzasjaypipes: can't we imagine to have caching scheduler to claim as well ?15:50
dansmithOSIC would have been an option, but tbh, I dunno who is likely to do it by the time we need to know in queens15:50
bauzasjaypipes: you said you were blocked when untangling that, but do you think it's doable tho ?15:50
mriedemdansmith: 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 IRC15:50
mriedembecause huawei public cloud is using the caching scheduler15:50
jaypipesbauzas: 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
dansmithmriedem: okay well, that'd be cool15:51
bauzasjaypipes: caching has different input than filterscheduler, but we could still claim the result right?15:51
dansmithbauzas: not really15:52
dansmithbauzas: if it's not getting allocation candidates, it is missing some of what it needs15:52
dansmithbauzas: we could make it work, but it would be wonky15:52
bauzasI see15:52
jaypipesdansmith: and not worthwhile, imho.15:52
dansmithbauzas: or we'd have to sift through the results of /ac to find the thing we chose based on our cached data15:52
dansmithjaypipes: agreed15:52
bauzasso, yeah, benchmarks15:52
dansmithIIRC, 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 traffic15:53
bauzasI need to disappear for family business, but I will watch out thec convo15:53
bauzasdansmith: you're correct15:53
dansmithso 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 loss15:53
openstackgerritSylvain Bauza proposed openstack/nova master: Add a prelude section for Pike  https://review.openstack.org/49142415:53
*** markvoelker has quit IRC15:53
bauzasdansmith: right, I just mentioned that in the prelude ^15:53
bauzasdansmith: we can now run multiple scheduler workers15:54
bauzasTHAT is the big improvement15:54
dansmithI would expect to be able to brute-force scale the scheduler as high as you want with just that change alone15:54
openstackgerritBalazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests  https://review.openstack.org/49152915:54
*** tbachman has quit IRC15:54
bauzasscale out you mean ?15:54
*** aarefiev is now known as aarefiev_afk15:54
bauzasor scale up the load ?15:54
dansmithbauzas: scale it to as high of a throughput as you need by scaling out yes15:55
bauzask15:55
bauzasI agree15:55
gibiwe can easily remove chance from the func test. here is the patch: https://review.openstack.org/49152915:55
bauzasyeah15:56
bauzasnot a big deal15:56
bauzasanyway I need to leave15:56
bauzas++15:56
cdentgibi: nice15:56
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Flatten functions  https://review.openstack.org/36747015:56
openstackgerritStephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions  https://review.openstack.org/38507115:56
openstackgerritStephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions  https://review.openstack.org/38507215:56
openstackgerritStephen Finucane proposed openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list  https://review.openstack.org/38507415:56
bauzasFWIW we don't even need RetryFilter15:56
bauzasif we are singlehost15:57
sdagueso... is someone going deprecate chance completely?15:57
dansmithsdague: and caching15:58
*** ijw has joined #openstack-nova15:58
sdaguesure, that too15:58
dansmithwe need to deprecate both in pike15:58
dansmithwe can remove one or both when appropriate15:58
sdagueok, is that up for review? we're in rc week15:58
*** tbachman has joined #openstack-nova15:58
jaypipesgibi: why the change from 1 vcpu to 2 vcpus? that's weird..15:58
dansmithsdague: are you asking if you can do it?15:59
gibijaypipes: because there is resize to same host where the allocations are doubled up15:59
*** yamamoto has quit IRC15:59
gibijaypipes: so at least 2 vcpu is needed15:59
dansmithgibi: yeah15:59
jaypipesgibi: guh, gotcha.15:59
dansmithmakes sense, I had to do that in some of the tests as well15:59
dansmithjaypipes: this is the max_unit thing I was talking about with moves16:00
dansmithjaypipes: that is going to piss people off16:00
dansmithbecause allocation_ratio won't apply to single-host moves16:00
*** Apoorva has joined #openstack-nova16:00
dansmithmigration uuid will fix that16:00
jaypipesdansmith: yeah, understood. not sure it's the end of the world, though.16:00
cdentmigration uuid will fix everything!16:00
dansmitheff yeah it's the end of the world16:01
jaypipes:)16:01
dansmithI have a single-node cloud at home!16:01
gibidansmith: only one? ;)16:01
*** Apoorva has quit IRC16:01
dansmithgibi: many nodes, but only one in the openstack deployment :)16:01
mnaseri'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 free16:03
mnaserarguably.. shouldnt it be based on a percentage?16:03
mnaserhttps://github.com/openstack/nova/blob/master/nova/scheduler/weights/ram.py16:04
jaypipesmnaser: perhaps use the num instances weigher instead?16:04
dansmithor weigh the num_instances higher than the ram one16:04
dansmithmnaser: some people want to pack first16:04
jaypipesmnaser: nm, there is no such thing :(16:04
dansmithbecause windows licenses16:04
mnaserdansmith: packing would still work with -1.016:05
dansmithjaypipes: there's something like that pretty sure16:05
mnaserhttps://github.com/openstack/nova/tree/master/nova/scheduler/weights16:05
mnaseri dont see a num_instances on16:05
mriedemstephenfin: where does this come from? https://review.openstack.org/#/c/490952/3/doc/source/admin/security-groups.rst16:05
mnaserthere's a fitler but not a weighter16:05
stephenfinmriedem: cli-nova-manage-projects-security.rst16:05
stephenfin(I think16:06
jaypipesmnaser: yeah :(16:06
mriedemoh nvm https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-nova-manage-projects-security.rst16:06
mriedemyeah16:06
dansmithmnaser: yeah, that's what I'm thinking of, heh16:06
*** ircuser-1 has joined #openstack-nova16:06
mnasercause 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 memory16:06
*** aarefiev_afk has quit IRC16:06
*** mamandle has quit IRC16:07
mnaseri'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 am16:07
*** udesale has quit IRC16:08
*** itlinux has joined #openstack-nova16:08
dansmithman I really thought we had more weighers16:08
*** udesale has joined #openstack-nova16:08
mriedemstephenfin: so on https://review.openstack.org/#/c/490952/ I think we should have those various volume attach troubleshooting docs16:09
jaypipesmnaser: 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
jaypipesmnaser: else submit a wishlist bug for one of us to get to.16:10
jaypipesmnaser: like dansmith I'm kinda surprised such a thing doesn't already exist :(16:10
*** lyan has quit IRC16:11
mriedemstephenfin: and maybe leave todo comments for combining the quotas pages16:11
*** slaweq has joined #openstack-nova16:12
mriedemstephenfin: if you want we can just deal with those comments in a follow up patch16:13
*** Apoorva has joined #openstack-nova16:13
*** yangyapeng has quit IRC16:13
*** yangyapeng has joined #openstack-nova16:13
stephenfinmriedem: Nah, I might as well address them there now16:14
stephenfinmriedem: 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 IRC16:17
*** Qiming has quit IRC16:18
mriedemstephenfin: replied again16:19
mriedemi don't see why --version is different from --local_cell16:19
mriedemsince they are both options16:19
mriedemplus i feel like this is a distraction right now16:19
*** yangyapeng has quit IRC16:20
*** Qiming has joined #openstack-nova16:20
mriedemsdague: how is https://review.openstack.org/#/c/490644/ different from https://review.openstack.org/#/c/478485/ ?16:21
mriedemnote i can't get to docs-draft.openstack.org/44/490644/3/check/gate-nova-docs-ubuntu-xenial/bd5e01d//doc/build/html/ right now16:22
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005716:22
sdaguemriedem: the TOC approach16:23
sdaguemriedem: see commit message16:24
*** dtp has joined #openstack-nova16:24
*** mingyu has quit IRC16:28
*** hemna_ has joined #openstack-nova16:29
*** udesale has quit IRC16:31
openstackgerritMerged openstack/nova master: Add resource utilities to scheduler utils  https://review.openstack.org/49051416:39
*** slaweq has joined #openstack-nova16:39
*** ijw has quit IRC16:40
*** sshwarts has quit IRC16:42
*** openstackstatus has quit IRC16:42
*** lyan has joined #openstack-nova16:43
mriedemsdague: do we need to worry about redirects for anything in https://review.openstack.org/#/c/490644/ ?16:44
*** openstackstatus has joined #openstack-nova16:44
*** ChanServ sets mode: +v openstackstatus16:44
mriedemsdague: or is that handled by https://review.openstack.org/#/c/490644/3/doc/source/index.rst@186 ?16:44
*** mingyu has joined #openstack-nova16:45
mnaserjaypipes do you think proposing a fix to replace the existing ram weighter by percentage would be an acceptable change16:45
mnaseror introduce a new weighter instead?16:45
*** hoonetorg has quit IRC16:45
mnaserbecause i genuinely think a ram weighter based on free memory only is not correct16:45
mriedemsdague: nvm, looks like nothing actually moves16:47
mriedemso no redirects needed16:47
jaypipesmnaser: a new weigher would be better.16:49
openstackgerritEd Leafe proposed openstack/nova master: Fix migrate single instance when it was created concurrently  https://review.openstack.org/49143916:50
edleafebauzas: ^^ just fixed the pep8 issue16:51
*** gszasz has quit IRC16:52
sdaguemriedem: right, I did not move any source files16:55
sdagueI was just trying to build landing pages, and tame the TOC on the left sidebar which is a confusing mess right now16:55
mnaserehh16:56
*** hoonetorg has joined #openstack-nova16:58
dansmithjaypipes: mnaser agreed16:58
dansmithbehavioral change may surprise people16:58
mnaseryep, that's what i thought, i'm working on something now16:59
jaypipescool16:59
stvnoyesmriedem 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-nova17:00
jaypipesdansmith: correct.17:01
*** crushil has joined #openstack-nova17:02
*** yamamoto has quit IRC17:04
*** rybridges has quit IRC17:04
*** hemna_ has quit IRC17:05
*** mdnadeem has quit IRC17:06
*** tesseract has quit IRC17:12
*** rajathagasthya has joined #openstack-nova17:13
*** pcaruana has quit IRC17:15
stephenfinmriedem: Further replies left on https://review.openstack.org/#/c/453808/17:16
stephenfinI 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 depressing17:17
openstackgerritMerged openstack/nova master: Add Contributor Guide section page  https://review.openstack.org/49064417:18
mriedemstephenfin: why does --verbose block that but --local_cell does not?17:18
mriedemwhy doesn't cliff allow us to have options, in other words17:18
*** sahid has quit IRC17:19
*** Swami has joined #openstack-nova17:19
*** burt has joined #openstack-nova17:21
*** rcernin has quit IRC17:22
* mriedem goes to lunch17:22
*** mriedem has quit IRC17:22
*** hongbin has quit IRC17:25
*** hongbin has joined #openstack-nova17:26
*** Guest15209 has quit IRC17:33
*** randomhack has quit IRC17:35
*** sambetts is now known as sambetts|afk17:36
*** vks11 has quit IRC17:37
*** randomhack has joined #openstack-nova17:38
*** annegentle has quit IRC17:41
*** slaweq has quit IRC17:43
*** aarefiev has joined #openstack-nova17:48
*** hongbin has quit IRC17:50
*** markvoelker has joined #openstack-nova17:54
*** vks1 has joined #openstack-nova17:56
*** gouthamr has joined #openstack-nova17:57
openstackgerritMohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher  https://review.openstack.org/49156117:58
mnaserdansmith jaypipes i havent contributed to nova in a while so i welcome all critique whenever you have time ^17:58
openstackgerritMerged openstack/python-novaclient master: tools: Remove dead script  https://review.openstack.org/48013817:58
jaypipesmnaser: cheers17:58
*** itlinux has quit IRC17:59
*** vks1 has quit IRC18:03
*** slaweq has joined #openstack-nova18:04
*** randomhack has quit IRC18:04
*** itlinux has joined #openstack-nova18:05
*** slaweq has quit IRC18:09
*** yamahata has quit IRC18:09
*** baoli_ has quit IRC18:10
*** dtantsur is now known as dtantsur|afk18:12
*** lpetrut has joined #openstack-nova18:12
*** yangyapeng has joined #openstack-nova18:15
*** litao__ has quit IRC18:17
*** yangyapeng has quit IRC18:20
*** baoli has joined #openstack-nova18:21
*** itlinux has quit IRC18:21
*** itlinux has joined #openstack-nova18:22
*** ijw has joined #openstack-nova18:22
*** annegentle has joined #openstack-nova18:24
*** baoli has quit IRC18:25
*** slaweq has joined #openstack-nova18:25
*** baoli has joined #openstack-nova18:25
*** baoli has quit IRC18:25
openstackgerritEd Leafe proposed openstack/nova master: Handle ironicclient failures in Ironic driver  https://review.openstack.org/48792518:26
*** slaweq has quit IRC18:27
*** ijw has quit IRC18:27
*** slaweq has joined #openstack-nova18:27
*** gbarros has quit IRC18:28
*** baoli has joined #openstack-nova18:28
*** markvoelker has quit IRC18:28
*** randomhack has joined #openstack-nova18:30
*** edmondsw_ has quit IRC18:32
*** edmondsw has joined #openstack-nova18:32
*** annegentle has quit IRC18:34
*** kfarr has joined #openstack-nova18:34
*** edmondsw_ has joined #openstack-nova18:35
*** edmondsw has quit IRC18:37
jaypipesdansmith: what's the difference between rebuild and evacuate?18:37
dansmithjaypipes: rebuild is in place on the same compute node18:38
dansmithjaypipes: evacuate means "rebuild me somewhere else" and has to be run on a down compute node18:38
dansmithjaypipes: 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 link18:38
jaypipesdansmith: thx. sorry, just needed a reminder after looking at the code... :(18:38
dansmithnp18:38
*** edmondsw_ has quit IRC18:39
*** lpetrut has quit IRC18:41
*** yamahata has joined #openstack-nova18:44
mnaserdansmith jaypipes and evacuate when using ceph means start me somewhere else (not rebuilding)18:44
jaypipesmnaser: when using ceph *for the root disk*. :)18:45
dansmithmnaser: it's still a rebuild under the covers in nova,18:45
dansmithit just doesn't erase the disk18:45
dansmithwhich is covered in the above link18:45
*** mriedem has joined #openstack-nova18:45
mnaseryes, that's correct. when reading that link, volume-backed made me think cinder but indeed18:45
dansmithwell, it says shared storage in one place too, but yeah18:46
*** slaweq has quit IRC18:47
mriedem+2 on bauzas' IndexError fix https://review.openstack.org/#/c/491439/18:54
*** nicolasbock has quit IRC18:57
sdaguemriedem: also this doc page when you get a chance - https://review.openstack.org/#/c/490994/18:58
sdaguemriedem: +A18:59
mriedemyup the tab is open18:59
*** vishwanathj has quit IRC19:06
*** vishwanathj has joined #openstack-nova19:06
*** slaweq has joined #openstack-nova19:08
*** armax has joined #openstack-nova19:08
*** slaweq has quit IRC19:13
*** baoli has quit IRC19:15
*** rajathagasthya has quit IRC19:16
*** slaweq has joined #openstack-nova19:18
*** pchavva has quit IRC19:24
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/49129619:25
openstackgerritMatt Riedemann proposed openstack/nova master: Add a prelude section for Pike  https://review.openstack.org/49142419:26
openstackgerritMatt Riedemann proposed openstack/nova master: Mark max microversion for Pike in history doc  https://review.openstack.org/49158119:26
openstackgerritMatt Riedemann proposed openstack/nova master: Add release note for shared storage known issue  https://review.openstack.org/49158219:26
*** gbarros has joined #openstack-nova19:26
*** reedip_afk has quit IRC19:33
*** slaweq has quit IRC19:33
*** xyang1 has joined #openstack-nova19:33
*** randomhack has quit IRC19:34
*** edmondsw has joined #openstack-nova19:38
*** randomhack has joined #openstack-nova19:41
*** priteau has quit IRC19:43
*** priteau has joined #openstack-nova19:44
*** reedip_afk has joined #openstack-nova19:45
*** awaugama has quit IRC19:46
openstackgerritMatt Riedemann proposed openstack/nova master: Add release note for shared storage known issue  https://review.openstack.org/49158219:46
*** priteau has quit IRC19:48
openstackgerritMerged openstack/python-novaclient master: Fix reservation_id not supported by Nova API  https://review.openstack.org/46844419:49
*** annegentle has joined #openstack-nova19:52
*** slaweq has joined #openstack-nova19:53
mriedemstephenfin: 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.rst19:58
*** slaweq has quit IRC19:59
*** baoli has joined #openstack-nova19:59
*** gbarros has quit IRC20:00
*** thorst has quit IRC20:00
sdaguemriedem: 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 in20:00
mriedemi didn't know if it was intentional20:01
sdagueit's also about finding the right places for things to go20:01
sdagueyeh, 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 challenging20:02
openstackgerritIlya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory  https://review.openstack.org/49158920:03
*** thorst has joined #openstack-nova20:03
*** aarefiev has quit IRC20:04
*** liverpooler has quit IRC20:04
openstackgerritIlya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory  https://review.openstack.org/49158920:06
*** thorst has quit IRC20:07
mriedemsdague: ok questions in https://review.openstack.org/#/c/490994/20:10
*** slaweq has joined #openstack-nova20:14
*** yangyapeng has joined #openstack-nova20:16
*** gouthamr has quit IRC20:17
*** thorst has joined #openstack-nova20:18
*** slaweq has quit IRC20:19
sdaguelooking20:21
*** yangyapeng has quit IRC20:21
sdaguemriedem: hopefully answers20:23
openstackgerritEric Fried proposed openstack/nova master: WIP: Base _ContextAuthPlugin on BaseIdentityPlugin  https://review.openstack.org/49120320:23
*** gouthamr has joined #openstack-nova20:24
mriedemok reply to the reply20:25
sdaguemriedem / 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 flow20:25
sdagueand how things would chunk up the rest of the way20:25
mriedemif 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
sdaguemriedem: right, so that was next on the chopping block20:26
sdagueyou want to look at that etherpad quick?20:26
mriedemjust did20:26
sdagueso 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 index20:28
sdagueand if we get a general ack that's how we want the world to look, I'll help write those bits20:29
*** mingyu has quit IRC20:30
*** smatzek has quit IRC20:31
openstackgerritMohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher  https://review.openstack.org/49156120:32
*** slaweq has joined #openstack-nova20:35
*** rajathagasthya has joined #openstack-nova20:35
*** gouthamr has quit IRC20:36
*** slaweq has quit IRC20:39
*** randomha1k has joined #openstack-nova20:40
*** cleong has quit IRC20:41
*** slaweq has joined #openstack-nova20:41
*** randomhack has quit IRC20:43
mriedemi've already found some gems in the docs about stuff that's way old, like the nova-api service description only mentions xml requests20:44
mriedemand all the ec2 api stuff needs to go away20:44
*** randomha1k has quit IRC20:44
*** moshele has joined #openstack-nova20:46
dansmithmriedem: we're dropping CD support, we might as well go back to XML anyway right?20:47
openstackgerritMatt Riedemann proposed openstack/nova master: Mark max microversion for Pike in history doc  https://review.openstack.org/49158120:47
openstackgerritMatt Riedemann proposed openstack/nova master: Add release note for shared storage known issue  https://review.openstack.org/49158220:47
openstackgerritMatt Riedemann proposed openstack/nova master: Add a prelude section for Pike  https://review.openstack.org/49142420:47
mriedemdid we mark json as experimental and never unmark it?20:47
*** mingyu has joined #openstack-nova20:53
*** gjayavelu has joined #openstack-nova20:54
*** mriedem has quit IRC20:56
cdentdansmith: naw, backward to SOAP and forward to GRPC20:56
sdaguemriedem dropped already... eh?20:57
*** annegentle has quit IRC20:59
openstackgerritmelanie witt proposed openstack/nova master: Add functional test for local delete allocations  https://review.openstack.org/47057821:02
*** mingyu has quit IRC21:08
*** crushil has quit IRC21:09
*** lucasxu has quit IRC21:15
*** priteau has joined #openstack-nova21:16
*** kfarr has quit IRC21:17
*** slaweq has quit IRC21:18
*** catintheroof has quit IRC21:20
*** yamamoto has joined #openstack-nova21:20
*** yamamoto has quit IRC21:25
*** markvoelker has joined #openstack-nova21:26
*** annegentle has joined #openstack-nova21:30
openstackgerritMerged openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks'  https://review.openstack.org/41652021:31
*** slaweq has joined #openstack-nova21:39
openstackgerritEd Leafe proposed openstack/nova master: Raise NoValidHost if no allocation candidates  https://review.openstack.org/49149121:41
*** slaweq has quit IRC21:44
*** priteau has quit IRC21:44
*** edmondsw has quit IRC21:45
*** edmondsw has joined #openstack-nova21:45
openstackgerritMerged openstack/nova master: Detach device from live domain even if not found on persistent  https://review.openstack.org/48854521:47
openstackgerritMerged openstack/nova master: Stop using mox stubs in cast_as_call.py  https://review.openstack.org/47773021:48
*** edmondsw has quit IRC21:50
*** slaweq has joined #openstack-nova21:55
*** yamamoto has joined #openstack-nova21:56
cdentjaypipes, 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
dansmithcdent: waiting for jaypipes to refresh the patches, AFAIK21:59
jaypipesdansmith, cdent: yeah, I'm still on it :( I'm lost in unit test hell. :(21:59
* cdent sends cookies21:59
cdentokay, I’ll go looking for those in the morning22:00
*** markvoelker has quit IRC22:00
*** xyang1 has quit IRC22:00
openstackgerritMerged openstack/nova master: Use wsgi-intercept in OSAPIFixture  https://review.openstack.org/48682522:00
jaypipescdent: thx for the cookies. :)22:00
*** slaweq has quit IRC22:01
cdenthmmm, 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-nova22:09
openstackgerritEric Fried proposed openstack/nova master: WIP: Base _ContextAuthPlugin on BaseIdentityPlugin  https://review.openstack.org/49120322:09
*** esberglu has quit IRC22:09
*** esberglu has joined #openstack-nova22:10
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005722:11
efriedsdague yt?22:11
*** mingyu has quit IRC22:14
*** slaweq has joined #openstack-nova22:14
*** esberglu has quit IRC22:14
*** Apoorva_ has joined #openstack-nova22:15
*** yangyapeng has joined #openstack-nova22:17
*** Apoorva has quit IRC22:18
*** slaweq has quit IRC22:19
*** yangyapeng has quit IRC22:21
*** Apoorva_ has quit IRC22:24
*** ijw has joined #openstack-nova22:25
*** Apoorva has joined #openstack-nova22:25
*** esberglu has joined #openstack-nova22:25
*** gouthamr has joined #openstack-nova22:29
*** yamahata has quit IRC22:29
*** slaweq has joined #openstack-nova22:30
*** thorst has quit IRC22:32
*** markvoelker has joined #openstack-nova22:32
*** yassine has quit IRC22:33
*** yamahata has joined #openstack-nova22:33
*** sree has joined #openstack-nova22:34
*** yassine has joined #openstack-nova22:35
*** hongbin has joined #openstack-nova22:35
*** marst has quit IRC22:36
*** slaweq has quit IRC22:36
openstackgerritEd Leafe proposed openstack/nova master: replace chance with filter scheduler in func tests  https://review.openstack.org/49152922:38
*** sree has quit IRC22:38
*** moshele has quit IRC22:39
*** ijw has quit IRC22:46
*** slaweq has joined #openstack-nova22:49
*** itlinux has quit IRC22:51
*** markvoelker has quit IRC22:54
*** edmondsw has joined #openstack-nova22:54
*** slaweq has quit IRC22:54
*** markvoelker has joined #openstack-nova22:54
*** PsionTheory has quit IRC22:58
*** lyan has quit IRC22:58
*** edmondsw has quit IRC22:58
*** markvoelker has quit IRC22:59
*** slaweq has joined #openstack-nova23:02
*** lyan has joined #openstack-nova23:08
*** slaweq has quit IRC23:08
*** annegentle has quit IRC23:09
*** lyan has quit IRC23:11
*** jaypipes has quit IRC23:19
openstackgerritMohammed Naser proposed openstack/nova master: Introduce new percentage based RAM weigher  https://review.openstack.org/49156123:20
*** sdague has quit IRC23:21
cdentoh look at the time23:22
*** cdent has quit IRC23:22
*** slaweq has joined #openstack-nova23:24
*** catintheroof has joined #openstack-nova23:28
*** slaweq has quit IRC23:30
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org will be offline momentarily at 00:00 utc for a quick reconfiguration-related restart23:33
*** itlinux has joined #openstack-nova23:34
openstackgerritMerged openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager  https://review.openstack.org/48573423:37
*** itlinux has quit IRC23:37
*** dtp has quit IRC23:38
*** felipemonteiro_ has quit IRC23:39
*** catintheroof has quit IRC23:50
*** slaweq has joined #openstack-nova23:56
*** mingyu has joined #openstack-nova23:57
*** crushil has joined #openstack-nova23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!