Thursday, 2018-05-03

*** vladikr__ has quit IRC00:00
arvindn05i think either of the 2 options we boiled down to work for me....i put this in open dicussion for tommorow's meeting...hopefully we get concensus then :)00:01
*** gjayavelu has quit IRC00:04
*** gjayavelu has joined #openstack-nova00:09
*** Nel1x has joined #openstack-nova00:09
*** arvindn05 is now known as arvindn05_away00:13
*** lbragstad has quit IRC00:17
*** mriedem_inlaws has quit IRC00:23
openstackgerritJake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows  https://review.openstack.org/55675100:25
*** yamamoto has joined #openstack-nova00:36
*** BrinZhang has quit IRC00:38
*** BrinZhang has joined #openstack-nova00:39
*** mriedem has joined #openstack-nova00:41
*** yamamoto has quit IRC00:41
*** fragatin_ has quit IRC00:42
*** hoangcx has joined #openstack-nova00:43
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add nova-manage placement heal_allocations CLI  https://review.openstack.org/56588600:46
*** r-daneel has joined #openstack-nova00:46
*** r-daneel_ has joined #openstack-nova00:49
*** r-daneel has quit IRC00:50
*** r-daneel_ is now known as r-daneel00:50
*** jichen has joined #openstack-nova00:52
*** phuongnh has joined #openstack-nova01:04
*** threestrands has joined #openstack-nova01:05
*** r-daneel has quit IRC01:06
*** gjayavelu has quit IRC01:06
*** yamahata has quit IRC01:08
openstackgerritZhenyu Zheng proposed openstack/nova master: WIP new migration threads control  https://review.openstack.org/56350501:13
*** zhaochao has joined #openstack-nova01:16
*** Guest10014 is now known as jlvillal01:19
*** fragatina has joined #openstack-nova01:22
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies  https://review.openstack.org/54692501:23
*** Tom-Tom has quit IRC01:28
*** Tom-Tom has joined #openstack-nova01:28
*** namnh has joined #openstack-nova01:31
*** namnh has quit IRC01:31
*** Tom-Tom has quit IRC01:33
*** Nil_ has quit IRC01:34
*** yamamoto has joined #openstack-nova01:38
*** dpawlik has joined #openstack-nova01:44
*** yamamoto has quit IRC01:44
*** lei-zh has joined #openstack-nova01:47
*** dpawlik has quit IRC01:48
*** Tom-Tom has joined #openstack-nova01:49
*** yamamoto has joined #openstack-nova01:50
*** r-daneel has joined #openstack-nova01:53
*** bkopilov has quit IRC01:57
*** bkopilov_ has quit IRC01:57
*** gcb has joined #openstack-nova02:00
*** hongbin has joined #openstack-nova02:12
*** annp has joined #openstack-nova02:12
*** tiendc has joined #openstack-nova02:16
*** psachin has joined #openstack-nova02:18
*** lei-zh has quit IRC02:19
*** lei-zh has joined #openstack-nova02:19
*** gjayavelu has joined #openstack-nova02:21
*** lbragstad has joined #openstack-nova02:24
*** dougshelley66 has quit IRC02:30
*** moshele has joined #openstack-nova02:35
*** moshele has quit IRC02:42
*** tbachman has quit IRC02:46
openstackgerritMatt Riedemann proposed openstack/nova master: Fix the request context in ServiceFixture  https://review.openstack.org/55816002:52
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-manage placement heal_allocations CLI  https://review.openstack.org/56588602:52
*** mriedem has quit IRC02:52
openstackgerritMatt Riedemann proposed openstack/nova master: Add multi-cell negative test for cold migration with target host  https://review.openstack.org/52402702:54
*** spsurya has joined #openstack-nova02:54
*** moshele has joined #openstack-nova02:55
*** tbachman has joined #openstack-nova02:57
*** moshele has quit IRC03:02
*** zhangbailin_ has joined #openstack-nova03:04
*** BrinZhang has quit IRC03:07
*** dpawlik has joined #openstack-nova03:10
*** bkopilov has joined #openstack-nova03:13
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765803:14
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424003:14
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334003:14
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334403:14
*** dpawlik has quit IRC03:14
*** bkopilov_ has joined #openstack-nova03:16
*** bkopilov__ has joined #openstack-nova03:16
*** nicolasbock has quit IRC03:21
*** liverpooler has quit IRC03:31
*** hamzy has joined #openstack-nova03:34
*** udesale has joined #openstack-nova03:36
*** Gorian has quit IRC03:36
*** anquan_wu has joined #openstack-nova03:37
*** namnh has joined #openstack-nova03:38
*** liverpooler has joined #openstack-nova03:40
*** Gorian has joined #openstack-nova03:41
*** hongbin has quit IRC03:42
*** fragatina has quit IRC03:43
*** lpetrut has joined #openstack-nova03:44
*** jogo has quit IRC03:45
*** janki has joined #openstack-nova03:47
*** jogo has joined #openstack-nova03:50
*** jchhatbar has joined #openstack-nova04:00
*** germs has quit IRC04:02
*** liverpooler has quit IRC04:03
*** janki has quit IRC04:03
*** gjayavelu has quit IRC04:05
*** fragatina has joined #openstack-nova04:12
*** fragatin_ has joined #openstack-nova04:13
*** fragatina has quit IRC04:17
*** lpetrut has quit IRC04:18
*** hoonetorg has quit IRC04:21
*** lpetrut has joined #openstack-nova04:21
*** jpena|off has quit IRC04:29
*** weshay has quit IRC04:29
*** markmc has quit IRC04:30
*** rnoriega_ has quit IRC04:31
*** jpena|off has joined #openstack-nova04:31
*** ltomasbo has quit IRC04:31
*** markmc has joined #openstack-nova04:32
*** rnoriega has joined #openstack-nova04:33
*** abhishekk has joined #openstack-nova04:34
*** weshay has joined #openstack-nova04:35
*** Tom-Tom has quit IRC04:41
*** Nel1x has quit IRC04:41
*** gyee has quit IRC04:41
*** Nel1x has joined #openstack-nova04:41
*** lei-zh has quit IRC04:42
*** lpetrut has quit IRC04:43
*** Tom-Tom has joined #openstack-nova04:43
*** links has joined #openstack-nova04:46
*** Tom-Tom has quit IRC04:47
*** anquan_wu has quit IRC04:48
*** jichen has quit IRC04:48
*** Nel1x has quit IRC04:54
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334004:55
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334404:55
*** sree_ has joined #openstack-nova04:56
*** sree_ is now known as Guest4188804:57
*** Guest41888 has quit IRC04:57
*** Zames has joined #openstack-nova05:06
*** yamahata has joined #openstack-nova05:06
*** markvoelker has quit IRC05:08
*** Zames has quit IRC05:08
*** anquan_wu has joined #openstack-nova05:10
*** dpawlik has joined #openstack-nova05:11
*** dpawlik has quit IRC05:15
*** moshele has joined #openstack-nova05:16
*** Eran_Kuris has joined #openstack-nova05:24
*** armaan has joined #openstack-nova05:28
*** Tom-Tom has joined #openstack-nova05:29
*** Tom-Tom has quit IRC05:33
*** sree_ has joined #openstack-nova05:34
*** sree_ is now known as Guest2225805:34
*** wolverineav has joined #openstack-nova05:34
*** evin has quit IRC05:39
*** sridharg has joined #openstack-nova05:47
*** wolverineav has quit IRC05:52
openstackgerritVishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder  https://review.openstack.org/56570305:55
*** hoonetorg has joined #openstack-nova05:55
openstackgerritVishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder  https://review.openstack.org/56570306:08
*** markvoelker has joined #openstack-nova06:09
*** evin has joined #openstack-nova06:12
*** andreas_s has joined #openstack-nova06:16
*** Tom-Tom has joined #openstack-nova06:16
*** mingyu has joined #openstack-nova06:24
*** lpetrut has joined #openstack-nova06:29
*** anquan_wu_ has joined #openstack-nova06:32
*** anquan_wu has quit IRC06:36
*** anquan_wu_ has quit IRC06:36
*** kholkina has joined #openstack-nova06:38
*** lei-zh has joined #openstack-nova06:39
*** damien_r has joined #openstack-nova06:40
*** markvoelker has quit IRC06:44
*** hoonetorg has quit IRC06:44
*** bhagyashris has joined #openstack-nova06:46
*** zhuli has quit IRC06:50
*** hoonetorg has joined #openstack-nova06:50
*** pcaruana has joined #openstack-nova06:53
*** do3meli has joined #openstack-nova06:55
*** rcernin has quit IRC06:58
*** threestrands has quit IRC06:58
*** ccamacho has joined #openstack-nova06:58
*** aloga has joined #openstack-nova07:03
*** dpawlik has joined #openstack-nova07:12
*** moshele has quit IRC07:14
openstackgerritZhenyu Zheng proposed openstack/nova master: WIP new migration threads control  https://review.openstack.org/56350507:16
*** dpawlik has quit IRC07:16
*** tesseract has joined #openstack-nova07:18
*** Tom-Tom has quit IRC07:27
*** sahid has joined #openstack-nova07:29
*** ragiman_ has joined #openstack-nova07:33
*** jpena|off is now known as jpena07:35
*** tssurya has joined #openstack-nova07:35
*** markvoelker has joined #openstack-nova07:41
*** alexchadin has joined #openstack-nova07:43
sahidstephenfin: if you can have a look to the serie https://review.openstack.org/#/c/561912/07:43
*** Tom-Tom has joined #openstack-nova07:45
*** amoralej|off is now known as amoralej07:50
*** anquan_wu has joined #openstack-nova07:58
bauzasmorning stackers08:00
*** anquan_wu_ has joined #openstack-nova08:00
gibigood morning nova08:02
*** mgoddard has joined #openstack-nova08:02
*** anquan_wu has quit IRC08:04
*** moshele has joined #openstack-nova08:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared  https://review.openstack.org/55645008:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates  https://review.openstack.org/55651408:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs  https://review.openstack.org/56184208:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries  https://review.openstack.org/55804508:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree  https://review.openstack.org/55948008:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories  https://review.openstack.org/55955408:08
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement  https://review.openstack.org/56548708:08
*** markvoelker has quit IRC08:13
*** yamahata has quit IRC08:14
*** wolverineav has joined #openstack-nova08:15
*** tuanla____ has joined #openstack-nova08:17
*** lucas-afk is now known as lucasagomes08:17
*** mdnadeem has joined #openstack-nova08:21
gibibauzas: could you look at a small spec template update that already has +2 from mriedem? https://review.openstack.org/#/c/562265/08:23
bauzassure08:24
gibibauzas: thanks08:24
bauzasdone08:25
bauzaswith a comment but +Wd08:25
*** wolverineav has quit IRC08:27
gibibauzas: I can quickfix your nit then you can fastapprove it08:27
*** alexchad_ has joined #openstack-nova08:27
*** wolverineav has joined #openstack-nova08:28
bauzasgibi: nah, no worries08:28
bauzasI understand what's an API entity08:28
bauzasno need for a followup08:28
bauzasand I'm not a API SIG expert ;)08:28
bauzasbut I use a vocabulary08:28
bauzasREST verbs : GET, PUT, etc.08:29
gibibauzas: OK then let's merge it as is :)08:29
bauzasREST resource : /servers, etc.08:29
gibibauzas: I think your vocabulary is the correct one08:29
bauzasactions are not RESTful08:29
*** mdbooth has joined #openstack-nova08:30
bauzasbut let's call it a subcollection08:30
kashyapThose who can +2 backports, can you put this through please: https://review.openstack.org/#/c/565668/08:30
kashyap(It already has a +2.)08:30
*** anquan_wu__ has joined #openstack-nova08:30
*** alexchadin has quit IRC08:31
bauzaskashyap: just did08:31
*** mdbooth_ has joined #openstack-nova08:31
kashyapbauzas: Fantastic.  I've got two more, mind looking there too?08:31
*** liuzz_ has joined #openstack-nova08:31
bauzaskashyap: see matt's comment08:31
* kashyap looks08:31
bauzaskashyap: https://review.openstack.org/#/c/565670/108:32
* kashyap clicks08:32
openstackgerritMerged openstack/nova-specs master: Suggest when to propose notification changes  https://review.openstack.org/56226508:32
kashyapbauzas: Actually, that's what I did08:32
kashyapBut let me double-check08:32
*** wolverineav has quit IRC08:32
gibibauzas: subcollection sound weird. as PUT and GET is called verb, the 'action' term for our unRESTful actions makes more sense to me08:33
*** liuzz has quit IRC08:33
*** anquan_wu_ has quit IRC08:34
*** mdbooth has quit IRC08:34
*** derekh has joined #openstack-nova08:36
kashyapbauzas++08:36
bauzasgibi: fair enoguh08:38
bauzasgibi: let's call it an API action08:38
gibibauzas: action on an API resource :)08:38
bauzasgibi: but not a REST action, which is an oxymore08:38
bauzasanyway, we're furiously ratholing08:39
gibia good morning rathole to get us up to speed08:39
*** links has quit IRC08:43
*** links has joined #openstack-nova08:44
*** Guest22258 has quit IRC08:45
*** hemna_ has quit IRC08:46
*** anquan_wu_ has joined #openstack-nova08:47
*** namnh has quit IRC08:48
*** trungnv has quit IRC08:48
*** tiendc has quit IRC08:48
*** tiendc has joined #openstack-nova08:48
*** namnh has joined #openstack-nova08:48
*** trungnv has joined #openstack-nova08:48
*** anquan_wu__ has quit IRC08:51
*** slaweq has quit IRC08:56
*** slaweq has joined #openstack-nova08:57
*** markvoelker has joined #openstack-nova09:10
*** dpawlik has joined #openstack-nova09:13
*** yingjun has joined #openstack-nova09:16
*** dpawlik has quit IRC09:17
openstackgerritMerged openstack/nova-specs master: Document Rocky review priorities  https://review.openstack.org/55779609:23
openstackgerritVishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder  https://review.openstack.org/56570309:29
*** khappone has joined #openstack-nova09:32
*** khappone_ has quit IRC09:36
openstackgerritjichenjc proposed openstack/nova master: [doc] enhance admin/configuration/api.rst  https://review.openstack.org/56600809:38
*** blkart has quit IRC09:40
*** blkart has joined #openstack-nova09:41
*** yassine has joined #openstack-nova09:42
*** yassine has quit IRC09:42
*** yassine has joined #openstack-nova09:43
*** yassine is now known as Guest1223609:43
*** markvoelker has quit IRC09:44
*** yingjun has quit IRC09:44
openstackgerritTetsuro Nakamura proposed openstack/nova master: Add tests for alloc_cands with member_of  https://review.openstack.org/56139909:46
openstackgerritTetsuro Nakamura proposed openstack/nova master: Fix member_of with sharing providers  https://review.openstack.org/56140009:46
openstackgerritTetsuro Nakamura proposed openstack/nova master: Expand member_of functional test cases  https://review.openstack.org/56601109:46
openstackgerritjichenjc proposed openstack/nova master: [doc]remove nova-cert leftover in doc  https://review.openstack.org/56601309:48
openstackgerritStephen Finucane proposed openstack/nova master: doc: Don't use single backticks in man pages  https://review.openstack.org/54088709:49
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension  https://review.openstack.org/54088809:49
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Fix file permissions  https://review.openstack.org/56601409:49
*** mdbooth_ has quit IRC09:50
openstackgerritMerged openstack/nova stable/queens: libvirt: Make `cpu_model_extra_flags` case-insensitive for real  https://review.openstack.org/56566809:51
*** lei-zh has quit IRC09:52
*** Eran_Kuris has quit IRC09:54
*** mdbooth has joined #openstack-nova10:01
*** rdopiera has joined #openstack-nova10:06
*** rdopiera has left #openstack-nova10:06
*** alexchad_ has quit IRC10:07
*** namnh has quit IRC10:11
*** ragiman_ has quit IRC10:13
*** bkopilov has quit IRC10:13
*** bkopilov_ has quit IRC10:14
*** bkopilov__ has quit IRC10:14
*** sahid has quit IRC10:17
*** anquan_wu_ has quit IRC10:18
*** yamamoto has quit IRC10:21
*** hoangcx has quit IRC10:21
*** jmccarthy has joined #openstack-nova10:25
*** nicolasbock has joined #openstack-nova10:31
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path  https://review.openstack.org/49720110:31
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647510:31
*** abhishekk has quit IRC10:31
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307710:31
*** edmondsw has joined #openstack-nova10:31
*** mdbooth has quit IRC10:35
*** edmondsw has quit IRC10:36
*** phuongnh has quit IRC10:36
*** yamamoto has joined #openstack-nova10:37
*** Tom-Tom has quit IRC10:37
*** annp has quit IRC10:40
*** markvoelker has joined #openstack-nova10:41
*** alexchadin has joined #openstack-nova10:44
*** mdbooth has joined #openstack-nova10:48
*** alexchadin has quit IRC10:49
*** dtantsur|afk is now known as dtantsur10:50
*** Eran_Kuris has joined #openstack-nova10:51
*** fragatin_ has quit IRC10:52
*** dougshelley66 has joined #openstack-nova10:53
*** fragatina has joined #openstack-nova10:53
*** jmccarthy has left #openstack-nova10:54
kashyapbauzas: and stephenfin Can you put this through: https://review.openstack.org/#/c/565707/10:54
kashyapQuite mechanical change10:54
*** alexchadin has joined #openstack-nova10:54
kashyapOr gibi even :-)10:54
stephenfinkashyap: done10:55
kashyapSplendid10:55
*** jpena is now known as jpena|lunch10:58
openstackgerritMerged openstack/nova master: Cleanup patch for the cell-disable series  https://review.openstack.org/55929910:58
*** tbachman has quit IRC10:58
*** slaweq has quit IRC10:58
openstackgerritKashyap Chamarthy proposed openstack/nova stable/pike: libvirt: Make `cpu_model_extra_flags` case-insensitive for real  https://review.openstack.org/56567010:59
gibikashyap: done :)10:59
kashyapThanks!11:05
kashyapgibi: I can't really figure why this is the only one that is failing: https://review.openstack.org/#/c/564012/11:07
kashyap(Where you did the 'recheck')11:07
moshelestephenfin: hi can you review this pike backport https://review.openstack.org/#/c/563501/?11:07
gibikashyap: based on the fail tempest test case names it feels unrelated to me11:08
gibikashyap: if the recheck fails with similar result then we have to dig11:08
kashyapIndeed.  Anyway, let's see if the new 'recheck' helps11:08
* kashyap is reasonably pessimistic that it will fail11:08
kashyapAnyway, I have other irons in the fire...So onto them11:09
*** lucasagomes is now known as lucas-hungry11:10
*** slaweq has joined #openstack-nova11:10
*** mingyu has quit IRC11:11
*** kotra03_2 has joined #openstack-nova11:12
*** markvoelker has quit IRC11:14
*** slaweq has quit IRC11:15
*** dougshelley66 has quit IRC11:21
openstackgerritMerged openstack/nova master: Include only required fields in ironic node cache  https://review.openstack.org/53950811:21
*** opetrenko has quit IRC11:26
*** tuanla____ has quit IRC11:26
*** opetrenko has joined #openstack-nova11:27
*** pchavva has joined #openstack-nova11:35
*** bkopilov has joined #openstack-nova11:39
*** bkopilov_ has joined #openstack-nova11:40
*** bkopilov__ has joined #openstack-nova11:41
jaypipesstephenfin: answered your query on the query on https://review.openstack.org/#/c/561315 :)11:46
*** yamamoto has quit IRC11:48
*** eharney has quit IRC11:49
*** salv-orlando has joined #openstack-nova11:53
*** amoralej is now known as amoralej|off11:53
*** amoralej|off is now known as amoralej|lunch11:54
*** yamamoto has joined #openstack-nova11:55
*** yamamoto_ has joined #openstack-nova11:56
*** yamamoto_ has quit IRC11:57
*** mingyu has joined #openstack-nova11:58
*** salv-orlando has quit IRC11:58
*** yamamoto_ has joined #openstack-nova11:59
*** yamamoto has quit IRC12:00
*** armaan has quit IRC12:00
*** armaan has joined #openstack-nova12:01
*** wolverineav has joined #openstack-nova12:01
*** yamamoto_ has quit IRC12:02
*** gcb has quit IRC12:09
*** jchorin has joined #openstack-nova12:10
*** jchorin has left #openstack-nova12:11
*** jpena|lunch is now known as jpena12:11
*** markvoelker has joined #openstack-nova12:11
*** tbachman has joined #openstack-nova12:13
*** yamamoto has joined #openstack-nova12:13
*** tiendc has quit IRC12:14
*** edmondsw has joined #openstack-nova12:16
*** lucas-hungry is now known as lucasagomes12:16
*** markvoelker has quit IRC12:20
*** markvoelker has joined #openstack-nova12:20
*** yamamoto has quit IRC12:20
*** yamamoto has joined #openstack-nova12:21
*** abhishekk has joined #openstack-nova12:23
*** yamamoto has quit IRC12:26
*** moshele has quit IRC12:27
openstackgerritJim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources  https://review.openstack.org/56584112:28
jrolldtantsur: fyi, rebased ^ on vdrok's stuff with a depends-on for the scheduler changes instead12:28
*** moshele has joined #openstack-nova12:29
dtantsurthanks!12:31
*** liverpooler has joined #openstack-nova12:31
stephenfinjaypipes: Makes sense. Mind addressing these two comments and I can ack https://review.openstack.org/#/c/561315/15/nova/api/openstack/placement/util.py@533 https://review.openstack.org/#/c/561315/15/nova/api/openstack/placement/handlers/resource_provider.py@20012:32
*** liverpoo1er has joined #openstack-nova12:32
stephenfinbauzas, gibi: Easy doc win here, if you fancy taking a look https://review.openstack.org/#/c/540888/12:33
gibistephenfin: looking12:33
*** liverpooler has quit IRC12:36
*** zhangbailin_ has quit IRC12:36
gibistephenfin: you missed one link :) https://review.openstack.org/#/c/540888/3/api-guide/source/server_concepts.rst@89512:47
gibistephenfin: fix it quickly and I'm +212:47
*** lyan has joined #openstack-nova12:47
*** lyan is now known as Guest4972912:47
*** armaan has quit IRC12:54
*** mvk has quit IRC12:55
*** nicolasbock has quit IRC12:57
*** liuzz has joined #openstack-nova12:58
*** psachin has quit IRC12:59
*** liuzz_ has quit IRC13:00
*** READ10 has joined #openstack-nova13:02
*** jchhatbar has quit IRC13:02
openstackgerritJay Pipes proposed openstack/nova master: support multiple member_of qparams  https://review.openstack.org/56131513:02
jaypipesstephenfin: righty-o. all done ^13:02
*** mchlumsky has joined #openstack-nova13:07
* gibi is wondering: "do I need to run the nova meeting today?"13:09
*** lionx has joined #openstack-nova13:09
*** slaweq has joined #openstack-nova13:10
*** yamamoto has joined #openstack-nova13:11
*** lionx has quit IRC13:11
openstackgerritMerged openstack/nova master: trivial: Fix file permissions  https://review.openstack.org/56601413:11
*** dpawlik has joined #openstack-nova13:12
*** nicolasbock has joined #openstack-nova13:14
*** slaweq has quit IRC13:16
*** dpawlik has quit IRC13:17
*** yamamoto has quit IRC13:17
*** yamamoto has joined #openstack-nova13:17
*** slaweq has joined #openstack-nova13:19
*** mriedem has joined #openstack-nova13:20
*** yamamoto has quit IRC13:22
*** Eran_Kuris has quit IRC13:24
*** sahid has joined #openstack-nova13:25
*** amoralej|lunch is now known as amoralej13:26
*** mvk has joined #openstack-nova13:28
*** moshele has quit IRC13:31
*** eharney has joined #openstack-nova13:32
*** tetsuro has joined #openstack-nova13:33
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension  https://review.openstack.org/54088813:35
stephenfingibi: There were actually a few more. Think I've addressed everything that uses a versioned URL now13:35
gibistephenfin: thanks, looking13:36
sahidstephenfin: can you look at this serie https://review.openstack.org/#/c/561912/9, jaypipes already +2ed most of the patches13:37
stephenfinsahid: Can do13:37
gibistephenfin: is the new link at https://review.openstack.org/#/c/540888/3..4/doc/source/admin/manage-the-cloud.rst@25 more general than the original was?13:37
stephenfingibi: Possibly. I couldn't find an equivalent for the former13:38
openstackgerritStephen Finucane proposed openstack/nova-specs master: trivial: Fix file permissions  https://review.openstack.org/56606413:38
gibistephenfin: ahh so the original was a dead link already13:39
gibistephenfin: then it is OK for me13:39
stephenfinYup. A couple of them were13:39
gibistephenfin: then this fix is better than I thought :)13:39
openstackgerritMatt Riedemann proposed openstack/nova master: Remove [scheduler]/host_manager config option  https://review.openstack.org/56580613:39
*** Eran_Kuris has joined #openstack-nova13:39
*** vladikr_ is now known as vladikr13:39
*** awaugama has joined #openstack-nova13:48
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-manage placement heal_allocations CLI  https://review.openstack.org/56588613:52
gibistephenfin: why do we need this ref? https://review.openstack.org/#/c/540888/3..4/nova/api/openstack/compute/rest_api_version_history.rst@76113:52
*** fragatina has quit IRC13:52
*** fragatina has joined #openstack-nova13:53
stephenfingibi: for this https://review.openstack.org/#/c/540888/3..4/doc/source/admin/manage-volumes.rst@3713:53
mriedemjohnthetubaguy: want to re-approve this? https://review.openstack.org/#/c/524027/ had to rebase13:54
gibistephenfin: cool, thanks13:54
johnthetubaguymriedem: sorry missed that one, done13:54
melwittnova meeting in 5 minutes13:55
mriedemjohnthetubaguy: thanks13:56
mriedemjohnthetubaguy: it's also your mandatory weekly bugging for https://review.openstack.org/#/c/55207813:57
*** takashin has joined #openstack-nova13:59
johnthetubaguymriedem: so I managed to loose track of what we are doing with specs re freeze, although this one should get approved eiher way13:59
mriedemjohnthetubaguy: spec freeze is r-213:59
mriedemjune 7 i think13:59
*** rajinir has joined #openstack-nova14:00
johnthetubaguymriedem: ah, I had in my head r-1 for some reason14:00
mriedemit usually is the 1st milestone,14:01
*** kotra03_krk has joined #openstack-nova14:01
mriedembut because of runways we decided to move out the spec freeze14:01
johnthetubaguyyeah, makes good sense14:01
johnthetubaguyI am just out of the loop (again), too many stupid hospital trips, but thankfully that is all done now!14:01
Shilpastephenfin: hi, did you gone through http://paste.openstack.org/show/720231/, is it okay to discuss now?14:01
*** kotra03_2 has quit IRC14:02
openstackgerritMatt Riedemann proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399014:02
openstackgerritMatt Riedemann proposed openstack/nova master: Update layout docs for running console proxies  https://review.openstack.org/55748914:02
mriedemjohnthetubaguy: hopefully everything is ok, didn't know that was happening14:02
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate the nova-consoleauth service  https://review.openstack.org/56536714:02
mriedemstephenfin: rebased and fixed up https://review.openstack.org/#/c/565367/ - should be happy now14:03
* kashyap wonders if mriedem has a special dashboard that you check to keep on top of "right stuff".14:03
johnthetubaguymriedem: no worries, not really mentioned it, wasn't too bit a deal, just took time I didn't have!14:03
*** evin has quit IRC14:03
kashyapBecause when I just wanted to ping on something, Matt was already on it & addressed what needs addressing.14:04
kashyap(Not the first time it happened, though...)14:04
mriedemjohnthetubaguy: babies are just so gd needy!14:04
mriedemkashyap: the right stuff? https://www.youtube.com/watch?v=tbIEwIwYz-c14:05
* kashyap gingerly clicks on it14:05
efriedwithout clicking, /me assumes nkotb video.14:05
kashyapHaha14:05
johnthetubaguymriedem: its true14:05
*** Guest12236 has quit IRC14:06
*** esberglu has joined #openstack-nova14:06
kashyapmriedem: Didn't know of this vintage song; thanks for the education.14:06
*** hemna_ has joined #openstack-nova14:06
kashyapefried: You know your guess was right.14:08
* kashyap wonders will mriedem be doing that dance sequence at Vancouver (or Berlin - likely I'll be there)14:09
kashyapAs it's closer to me14:09
*** andreas_s has quit IRC14:10
*** mlavalle has joined #openstack-nova14:11
mriedemtssurya: question in https://bugs.launchpad.net/nova/+bug/1767303 for you14:11
openstackLaunchpad bug 1767303 in OpenStack Compute (nova) "Scheduler connects to all cells DBs to gather compute nodes info" [Undecided,New] - Assigned to Surya Seetharaman (tssurya)14:11
mriedemtssurya: maybe you just want to propose a backport of https://review.openstack.org/#/c/539617/ for queens?14:12
*** READ10 has quit IRC14:13
bauzasstephenfin: do we really need RsT for a docstring ? https://review.openstack.org/#/c/564214/2/nova/conf/devices.py14:14
*** andreas_s has joined #openstack-nova14:14
stephenfinbauzas: It's not a docstring. That's help text that's parsed by oslo_config.sphinext to generate this14:15
stephenfinhttps://docs.openstack.org/nova/queens/configuration/config.html14:15
bauzasshit14:15
bauzasbecause yes, it's a docstring14:15
bauzasit's just we use the docstring elsewhere14:15
stephenfinI think our definition of docstring differs :)14:15
bauzasIMHO, help text is for nova.conf14:16
bauzasbut meh14:16
stephenfinUm, that is nova.conf?14:16
bauzasa *python* docstring14:16
bauzas3 brackets14:16
bauzasthat's what I call "docstring"14:16
edleafebauzas: that's just a string14:16
stephenfinThat's a mult-line string14:16
stephenfinYeah14:16
edleafeA docstring is the comment that follows the method definition14:17
stephenfinhttps://www.python.org/dev/peps/pep-0257/#id1514:17
bauzaswell, if you should name something because of its usage, I disagree14:18
bauzasbut meh14:18
bauzas:p14:18
edleafebauzas: for example:14:19
edleafedef some_method():14:19
edleafe    "this is a docstring"14:19
edleafe    return True14:19
edleafeDoesn't need triple quotes14:19
edleafeunless it's multi-line14:19
bauzasI should write my own language14:19
*** yamamoto has joined #openstack-nova14:20
stephenfinYeah, theoretically you could even avoid using triple quotes then by escaping the new lines but no one would be so evil14:20
stephenfinbauzas: http://logs.openstack.org/14/564214/2/check/build-openstack-sphinx-docs/a23348b/html/configuration/config.html#devices.enabled_vgpu_types14:20
bauzasgrmblblb :p14:21
stephenfinEventually I'll have you all writing perfect rST14:21
*** evin has joined #openstack-nova14:21
stephenfinAt which point I can happily retire to Spain or something14:21
mriedem*perfect* rST?14:21
stephenfin(TM)14:22
*** Eran_Kuris has quit IRC14:22
bauzasthat's glorious14:22
*** esberglu has quit IRC14:23
*** hongbin has joined #openstack-nova14:24
mriedemstephenfin: speaking of https://review.openstack.org/#/c/565526/2..3/nova/conf/compute.py@a89414:25
mriedemyou should appreciate that14:25
stephenfinI do :) oslo_config.sphinxext should soon start reporting those warnings too (though it won't raise an error, to avoid breaking all the stuff)14:26
*** mikal has quit IRC14:27
*** esberglu has joined #openstack-nova14:34
*** esberglu_ has joined #openstack-nova14:36
*** r-daneel has quit IRC14:36
*** Eran_Kuris has joined #openstack-nova14:38
*** germs has joined #openstack-nova14:38
*** germs has quit IRC14:38
*** germs has joined #openstack-nova14:38
*** kaisers has quit IRC14:38
*** germs has quit IRC14:39
*** kaisers has joined #openstack-nova14:39
*** germs has joined #openstack-nova14:39
*** esberglu has quit IRC14:40
*** esberglu_ is now known as esberglu14:40
*** moshele has joined #openstack-nova14:40
*** mdbooth has quit IRC14:41
*** mvk has quit IRC14:42
*** moshele has quit IRC14:42
*** mdbooth has joined #openstack-nova14:43
*** takashin has left #openstack-nova14:45
*** tetsuro has quit IRC14:45
*** arvindn05_away is now known as arvindn0514:45
*** yamamoto has quit IRC14:46
*** _d34dh0r53_ is now known as d34dh0r5314:46
*** alexchadin has quit IRC14:49
*** kotra03_krk has quit IRC14:50
*** mikal has joined #openstack-nova14:50
mriedemefried: https://bugs.launchpad.net/nova/+bug/176713914:52
openstackLaunchpad bug 1767139 in OpenStack Compute (nova) "TypeError in _get_inventory_and_update_provider_generation" [Undecided,New]14:52
mriedemremember https://review.openstack.org/#/c/562739/ ?14:52
mriedemapparently people are hitting a variant of that in pike starting in 16.1.1 which is when the pike version of that backport was released14:52
mriedemlooking at the code, i can't figure out what might be causing that14:52
mriedemthey are saying in the bug report that it happens if they start the compute before the control plane is up, but i'm not sure what difference that would make - but placement would have to be running14:53
efriedmriedem: Sorry, is it the backport of the original or the backport of the revert that's breaking?14:53
mriedembackport of the original14:54
*** afazekas_ has quit IRC14:54
mriedemthere is no backport of the revert14:54
mriedemso this https://review.openstack.org/#/c/524618/14:54
efried^ needs to be reverted is what you're saying?14:55
mriedemin the bug they said this happens if compute starts before controller, so i have to assume that includes placement isn't up yet, so we're hitting a flow where the RT is posting inventory for the first time and the compute node RP can't be created14:55
mriedemi would like to avoid a revert if possible14:56
mriedemi'm trying to inspect the change and flow to see if there is something we're missing14:56
efriedmriedem: Is it weird that https://review.openstack.org/#/c/562739/ is a revert in ocata, but pike was skipped?14:57
mriedemwe only saw the ci failures in ocata14:57
mriedemi haven't seen those in pike14:58
efriedokay, I just thought that wasn't allowed.14:58
*** Nil_ has joined #openstack-nova14:58
*** afazekas has joined #openstack-nova14:58
mriedemi bet i know what it is14:58
mriedembazinga!14:58
efriedI guess it wasn't a cherry-pick of a revert; it was a revert of a cherry-pick.14:58
mriedemset_inventory_for_provider -> _ensure_resource_provider -> _create_resource_provider -> safe_connect returns None because it can't talk to placement yet14:59
mriedemhttps://review.openstack.org/#/c/524618/2/nova/scheduler/client/report.py@51614:59
mriedemso we put None in the cache14:59
*** r-daneel has joined #openstack-nova14:59
efriedThat sounds totally right.14:59
mriedemkafuckingboom14:59
melwittmriedem: do you have any idea if this bug is still a thing? it's ancient https://bugs.launchpad.net/nova/+bug/136616814:59
* efried looks up "safe"14:59
openstackLaunchpad bug 1366168 in OpenStack Compute (nova) "delete private flavor will cause access problem with existed instance" [Medium,Confirmed]15:00
*** abhishekk has quit IRC15:00
*** AlexeyAbashkin has joined #openstack-nova15:00
*** yamahata has joined #openstack-nova15:00
mriedemmelwitt: would have to look later, we need a fix for this placement thing in stable15:00
melwittk, mah bad15:00
mriedemmp15:00
mriedem*np15:00
mriedemI'll wip up a thing15:02
mriedemwhy we hit this in ocata ci and not pike, i'm not sure15:02
mriedemmaybe devstack changed the order of install stuff in pike for the fleetify15:02
*** AlexeyAbashkin has quit IRC15:03
*** links has quit IRC15:06
*** kholkina has quit IRC15:09
openstackgerritMerged openstack/nova stable/pike: libvirt: Make `cpu_model_extra_flags` case-insensitive for real  https://review.openstack.org/56567015:11
*** ejat has quit IRC15:12
*** dpawlik has joined #openstack-nova15:13
*** ejat has joined #openstack-nova15:15
melwittfrickler: hi, could you please lift the -2 on this devstack change https://review.openstack.org/484973? the change that depends on it has been approved https://review.openstack.org/33399015:16
kashyapmriedem: Thanks for the review here, I wrote that warning in a hurry (hence Workflow -1): https://review.openstack.org/#/c/565242/1/nova/virt/libvirt/driver.py15:16
*** slaweq has quit IRC15:17
melwittfrickler: and then the follow up patch (that depends-on the nova change) can be approved to re-enable the novnc tests https://review.openstack.org/52781215:17
kashyapI want get the wording right.  I will update that, and address the unit tests.15:17
*** dpawlik has quit IRC15:17
*** ejat has quit IRC15:19
*** ejat has joined #openstack-nova15:19
*** janki has joined #openstack-nova15:24
*** ccamacho has quit IRC15:24
*** ccamacho1 has joined #openstack-nova15:24
efriedgibi: o notification deity supreme...15:26
openstackgerritMatt Riedemann proposed openstack/nova master: Handle @safe_connect returns None side effect in _ensure_resource_provider  https://review.openstack.org/56609615:26
efriedgibi: Do you know of any examples you can point to where an op/admin wrote a script/daemon that listens for notifications and triggers some action?15:26
gibiefried: there should be a couple of services listening to notifications, let me dig up links15:27
*** andreas_s has quit IRC15:27
*** andreas_s has joined #openstack-nova15:28
gibiefried: searchlight: https://github.com/openstack/searchlight/blob/master/searchlight/elasticsearch/plugins/nova/notification_handler.py15:29
mriedemdesignate sink15:29
mriedemceilometer15:29
mriedemmistral15:29
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: Configure multiple vGPU types  https://review.openstack.org/56421415:29
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info  https://review.openstack.org/56230415:29
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: Get inventories per vGPU type  https://review.openstack.org/56421515:29
efriedgibi: Thanks.15:30
gibiefried: https://github.com/openstack/watcher/blob/master/watcher/decision_engine/model/notification/nova.py15:31
gibiefried: I mriedem listed the others in the meanwhile15:32
efriedgibi, mriedem: Thanks y'all.15:32
*** ccamacho1 has quit IRC15:35
*** exarlos has joined #openstack-nova15:36
*** andreas_s has quit IRC15:37
mriedemTheJulia: left some comments in https://bugs.launchpad.net/nova/+bug/176666115:38
openstackLaunchpad bug 1766661 in OpenStack Compute (nova) "'host' property is missing for Cinder volume connector when boot from volume" [Medium,Confirmed]15:38
mriedemtl;dr nova knows how to get the host connector from the db if needed15:38
openstackgerritHongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity  https://review.openstack.org/52024715:39
openstackgerritBalazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider  https://review.openstack.org/50230615:39
*** slaweq has joined #openstack-nova15:40
*** felipemonteiro__ has joined #openstack-nova15:41
*** andreas_s has joined #openstack-nova15:42
exarlosHi all. I had a user attempt a resize of a VM (horizon) and it has failed and reduced the VM to an error state15:42
exarlosocata / ubuntu / lvm  storage15:43
exarlosIt mentions not being able to stat the /var/lib/nova/instances folder15:43
mriedemhttps://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L741215:44
*** yamamoto has joined #openstack-nova15:44
mriedemis it volume-backed or not?15:44
exarlosmriedem: local lvm disks as the backend15:45
mriedemthen you'll eventually be failing because ^15:45
*** felipemonteiro_ has quit IRC15:45
mriedemi think cfriesen might have a patch floating around for that15:45
mriedemhttps://review.openstack.org/#/c/337334/15:46
exarlosmriedem: It has worked many times in the past. :-(15:46
melwittso, you're saying regression from newton -> ocata?15:46
mriedemhave you patched nova to make that happen?15:46
*** gyee has joined #openstack-nova15:47
exarlosI hadn't patched. There is a allow_resize_to_same_host option in nova.conf that permitted this to happen.15:48
mriedemgiven https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L7412 i don't see how that's possible15:50
exarlosLet me check local code.15:50
*** pcaruana has quit IRC15:50
exarlosHonestly though :-)15:50
*** liuzz has quit IRC15:52
*** liuzz has joined #openstack-nova15:53
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Remove support for monitoring Intel CMT `perf` events  https://review.openstack.org/56524215:53
bauzasjaypipes: efried: naichuans (I know it's late for you): I have a thought about providing traits for vGPU types15:53
efriedbauzas: Listening.  Though I confess I'm behind on those reviews.15:53
bauzasjaypipes: efried: naichuans: the problem is that vGPU types differ in between virt drivers, and even can be depending on the GPU host driver (eg. nvidia vs. intel)15:53
bauzasefried: no worries, it's unrelate15:54
*** damien_r has quit IRC15:54
efriedbauzas: The thing doing the dev discovery == the thing setting up the resource providers == the thing doing the provisioning == the virt driver.15:55
bauzasjaypipes: efried: naichuans: so, say, I'd like to provide a flavor asking for *this* type, should we provide specific traits like https://docs.nvidia.com/grid/6.0/grid-vgpu-user-guide/index.html#supported-gpus-grid-vgpu ?15:55
efriedThat's how it boils down for me.15:55
bauzasefried: ok, so you're not concerned by the virt driver setting for example a trait like "nvidia-11" while Xen would set a different one ?15:55
*** yamamoto has quit IRC15:55
*** sahid has quit IRC15:55
efriednot at all.15:55
bauzasif so, that's super cool, it prevents me to write a mapping table15:56
efriedwell15:56
exarlos15.0.7 is the version of nova components i'm running.15:56
efriedbauzas: If we can standardize some traits that all virt drivers could reasonably use when talking about the same device, that's the best thing.15:56
*** hamzy has quit IRC15:56
*** andreas_s has quit IRC15:56
bauzasefried: see for example https://docs.nvidia.com/grid/6.0/grid-vgpu-user-guide/index.html#vgpu-types-tesla-m6015:56
exarlosLooks like the code is different, so probably I am a bit behind.15:56
efriedbauzas: IOW, the value the virt driver discovers may be different from one virt driver to the next, but as long as it can map it to the standard trait, we're good.15:56
bauzasefried: the GPU type named "M60-0B" is named "nvidia-11" by the nvidia gpu driver15:57
bauzasefried: so the mediated devices etc. use that "nvidia-11" name15:57
bauzasefried: but if you use xen, it's totally named differently15:57
efriedbauzas: But is it still a 1-1 mapping?15:57
bauzasefried: https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#how-to-discover-a-gpu-type15:58
efriedbauzas: eek.15:58
bauzasefried: in libvirt, yes, a 1:115:58
efriedLet's talk through the use case real quick.15:59
exarlosmriedem: So, whilst I look at a few things, how best do I pull this back from an error state, and set it so it's bootable again? :-/15:59
bauzasefried: blame the gpu driver. :p15:59
efriedbauzas: From the user pov, is it likely that he's going to put together a flavor that asks for a vgpu and then not care whether he gets a libvirt or xen instance to satisfy it?15:59
bauzasefried: that's a flavor, right? so I don't expect at all end users requesting that16:00
efriedsorry, I never remember the right term.16:00
bauzasefried: I rather see operators setting conf file accordingly with flavors16:00
bauzasthey could name the flavor GRID M60-0Q I'm fine16:00
efriedbauzas: So what I'm trying to figure out is whether there's such a thing as a flavor that would include a VGPU, that's possible to use to deploy instances on different hypervisors?16:01
bauzasbut the extra spec would be "requested=nvidia-11"16:01
bauzasefried: mixing environments is highly discouraged16:01
efriedWell then I think it doesn't matter whether the traits are named the same from one hypervisor to the next.16:01
bauzasyou can do that but then split your cloud16:01
*** andreas_s has joined #openstack-nova16:01
bauzasefried: that said, there is a flaw16:01
exarlosmriedem: and also, should I expect each "server" to have a respective folder in /var/lib/nova/instances/<uuid> ?16:01
bauzasefried: the mdev GPU type name depends on the nvidia driver16:02
bauzasefried: since it depends on a vendor driver, we have no guarantee that 'nvidia-11' will still correspond to a specific GPU type in the future16:02
efriedbauzas: Right, so that's what I meant by 1-116:03
bauzasefried: if nvidia bumps a new driver that passes types that are not backwards compatible, then operators need to set again their flavors16:03
bauzasthat's not a public API, and I hate that16:03
bauzasthe kernel just leaks out what the VFIO device tells16:04
efriedbauzas: So wait, it's possible that 'nvidia-11' today corresponds to M60-0B and tomorrow will correspond to M99-0X ??16:04
bauzasefried: possibly yeah16:04
bauzascrazy, isn't it ?16:04
efriedthat's bullshit, yo.16:04
bauzasthat's a vendor API16:04
*** liuzz has quit IRC16:04
bauzasvendors can do crazy  things16:04
bauzasincluding breaking changes16:04
mriedemexarlos: without knowing where this actually failed and how it failed, it's hard to say16:05
mriedemif the guest is still running on the source host, which you could verify via virsh (assuming libvirt), then you could probably just reset the state of the instance16:05
efriedbauzas: Well, we can only do so much.  I think this mapping is something the virt driver will need to be able to keep track of.  And if the stoopid vendor driver makes a breaking change, the virt driver will just have to scramble to accomodate.16:05
bauzasefried: I think I reached the same conclusion16:05
mriedemhttps://developer.openstack.org/api-ref/compute/#reset-server-state-os-resetstate-action16:05
bauzasefried: for some reason, xen is better designed than libvirt16:06
efriedbauzas: But I think as much as we possibly can, we should create standard traits and ask the virt drivers to map to them.16:06
bauzasefried: it already makes the translation I guess16:06
efriedbauzas: Which one is doing translation, though?16:06
bauzasefried: because if you look the Xen GPU types, those look very identical to the nvidia doc16:06
bauzasefried: the xen API16:06
efriedbauzas: Right, so it's libvirt that's translating.16:06
bauzasefried: no16:06
efriedxen is just using 'em as is16:06
efriedno?16:07
bauzasefried: libvirt is straight setting what the kernel module tells16:07
bauzashttps://www.kernel.org/doc/Documentation/vfio-mediated-device.txt16:07
efriedoh, so xen is mapping those back to the model names in the literature16:07
exarlosmriedem: Let me go have a read of that.16:08
exarlosIt is on the same host still.16:08
bauzasefried: that's what I guess, yes16:08
efriedbauzas: so the point is that we need to choose one or the other, and then the docs for a given hypervisor have to tell me how to discover and determine the appropriate corresponding trait name.16:08
bauzasefried: ok, I take the challenge16:08
bauzasefried: I'm in favor of an unified trait name16:09
efriedbauzas: Which needs to happen regardless, because at the very least 'nvidia-11' will need to be translated to 'trait:HW_VGPU_TYPE_NVIDIA_11=required'16:09
bauzasyup16:09
efriedbauzas: What would be neat is if each hypervisor provided a script to discover the devs and list them along with their trait names.16:10
efriedthat might be asking too much, though :)16:10
bauzasefried: what I wonder if whether the mapping should be hardcoded in the virt driver code, or if we should leave operators setting that16:10
bauzasefried: heh, I have the same wish16:10
bauzasefried: xen does that16:10
*** andreas_s has quit IRC16:10
bauzasefried: but see my docs on libvirt usage16:10
bauzasthat's crazy16:11
bauzasyou have to lookup sysfs16:11
efriedbauzas: The operator mapping thing - that's going back to the yaml config we started brainstorming in Denver.16:11
bauzaswe don't have any libvirt API that shows you that16:11
bauzasefried: I know16:11
efriedbauzas: BUT that mapping is going to be more for device aliasing, white/blacklisting, etc.  I don't think we should be using it to define traits, unless maybe they're CUSTOM_*16:12
bauzasefried: just a thought16:12
bauzasefried: about custom traits16:12
openstackgerritStephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec  https://review.openstack.org/54129016:12
bauzasefried: can you set those with nested RPs ?16:12
*** yamahata has quit IRC16:12
bauzasefried: because then, I wouldn't care a bit about that16:12
bauzasefried: I could just ask operators to provide custom traits16:13
efriedbauzas: You can set traits on any RP.  We originally talked about traits "inheriting" or "propagating" in various directions, but we're not doing that.16:13
efriedbauzas: "on any RP" including child, sharing, etc.16:13
bauzasefried: I'm a bit rusty on traits16:13
bauzasefried: but I guess operators have to play with placement client to set the trait ?16:13
bauzasif so, that's perfect16:13
* jaypipes reads back...16:13
bauzasjaypipes: for your sanity, don't read the vendor crazypants I wrote16:14
efriedbauzas: Well, what we talked about in Dublin is that each of various entities (nova-compute, neutron, operator) will have a set of traits it owns/controls for a given provider.  It's allowed to set/unset those traits, but the other entities are not.16:15
efriedbauzas: So in this example, I would expect nova-compute (specifically the virt driver) to "own" the trait indicating the GPU type.  The operator doesn't get to set that.  That wouldn't make sense.16:15
bauzasefried: keep in mind we will have one inventory per vGPU type16:16
efriedbauzas: Same entity owns the inventory.16:16
bauzasefried: what if I want to set custom traits ?16:16
*** felipemonteiro__ has quit IRC16:17
*** felipemonteiro_ has joined #openstack-nova16:17
bauzasefried: say the virt driver now provides a tree of a root RP and two children RP, each one being a vGPU type16:17
bauzasefried: as an operator, I'd like to pick only one type based on a custom trait16:17
efriedbauzas: From this perspective, we don't really make a distinction between standard and custom traits.  But I would say the op is allowed to invent and assign a random custom trait as she sees fit.16:17
bauzasefried: exactly what we wrote in the spec, actually16:17
bauzashttps://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/add-support-for-vgpu.html16:18
efriedbauzas: Legit.  Basically the op just has to be careful not to stomp on a trait owned by compute.16:18
efriedbauzas: beyond that, sky's the limit.16:18
efriedbauzas: which is exactly why we made this a requirement in Dublin.16:18
efried(dansmith and I almost came to blows over it)16:18
bauzasyeah I remember16:18
*** zhaochao has quit IRC16:18
efriedI still believe it's going to bite us, but time will tell.16:19
bauzasefried: ok, so what's the way to set custom traits as of now ?16:19
bauzasthru placement API ? (and the client)16:19
efriedbauzas: Sure, that works.16:19
*** moshele has joined #openstack-nova16:20
bauzasefried: ok, what I need is testing then16:20
bauzasefried: my patch series is close to provide inventories using update_provider_tree()16:20
*** andreas_s has joined #openstack-nova16:20
efriedbauzas: There's no change to the placement API for any of that; the only thing we did was update the u-p-t spec/docs to make it clear that a dance is required to make sure you're not stepping on each other.16:20
efriedbauzas: Oh, are you aware of https://review.openstack.org/#/c/560444/ ?16:21
bauzasefried: ok, will look16:21
*** andrewbogott has quit IRC16:21
*** andrewbogott has joined #openstack-nova16:21
bauzasefried: no, I wasn't16:21
*** lpetrut_ has joined #openstack-nova16:21
bauzasefried: my top patch in the series returns a dict of dicts16:21
bauzasefried: I'll use that dict of dicts to populate the tree16:22
bauzasefried: but I was considering to implement by a change the new public interface for libvirt16:22
openstackgerritMerged openstack/nova master: Get anchors for sharing providers  https://review.openstack.org/56527916:22
openstackgerritMerged openstack/nova master: Make association_refresh configurable  https://review.openstack.org/56552616:22
efriedbauzas: Have you been following the xen driver work for vgpus?16:23
bauzasefried: barely but yes16:23
efriedbauzas: I think they're the furthest along with using upt in their driver.16:23
bauzasefried: well, I'm pretty close too16:23
efriedbauzas: Of course, they're based on code that hasn't merged and/or won't work until the nrp-in-alloc-cands series lands.16:23
bauzasefried: last change in my series is https://review.openstack.org/#/c/564215/16:23
bauzasefried: I know, and I was about to propose your series for a runway slot16:24
bauzasI'm a bit out of review as of those days due to my patch series, but I can switch to reviewing if I'm blocked due to that series :p16:24
*** lpetrut has quit IRC16:25
bauzasanyway, I need to drop for today16:25
efriedbauzas: Granular is ready as well. (Except both series are going to bounce on merge conflict soon)16:25
bauzasgranular is already a runway candidate16:25
jaypipesbauzas: I'm a strong -2 for having standard traits (or really, any traits) representing GPU vendor types16:25
efriedI guess I could put both of those series into the runway queue.16:25
efriedjaypipes: eh??16:25
bauzasjaypipes: I'm cool with that16:25
exarlosmriedem: pastebin.com/SPpHu9Tu is the current error state of the VM.16:25
bauzasjaypipes: I guess your concern is the os-traits explosion16:26
melwittlooks like the gate is timing out a lot in response to me saying it had been good in the nova meeting16:26
efriedjaypipes: Oh, you're looking for GPU *features* as opposed to types.16:26
bauzasmelwitt: yup, I noticed that in the meeting too16:26
melwitt:( sorry everyone16:26
*** lpetrut_ has quit IRC16:26
exarlosmriedem: I would think it's recoverable, but i'm still learning.16:26
bauzasjaypipes: efried: anyway, I need to drop16:26
efriedbauzas: à bientôt16:27
bauzasthe first step is to make nested RPs a thing16:27
bauzasthe second step will be to test my patches using custom traits16:27
mriedemmelwitt: excellent catch on https://review.openstack.org/#/c/541420/ - i reproduced in one of the unit tests16:27
bauzasso I'm super not concerned by having standard traits as of now, in particular for vendor purposes16:28
bauzasjaypipes: hope that clarifies ^16:28
*** tesseract has quit IRC16:28
* bauzas calls it a day16:28
jaypipesbauzas: you don't need custom traits for any of this.16:28
*** hongbin has quit IRC16:28
jaypipesbauzas: you don't need GPU types to be traits either.16:29
bauzaswhat I want is a pony16:29
*** hongbin has joined #openstack-nova16:29
mriedemexarlos: so you're hitting https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L745016:29
jaypipesbauzas: we had long conversations with Jianghua on the original VGPU spec about this.16:29
*** slaweq has quit IRC16:29
bauzasjaypipes: what do you suggest ?16:29
mriedemexarlos: which comes after https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L741316:29
melwittmriedem: woot (though concerned the unit test was covering it up?)16:29
mriedemexarlos: are you sure this isn't a volume-backed instance? and by volume-backed, i mean cinder volume, not local lvm disk16:29
*** slaweq has joined #openstack-nova16:29
*** slaweq has quit IRC16:30
jaypipesbauzas: use the traits for GPUs that are already in os-traits for describing the *capabilities* of the GPU.16:30
mriedemmelwitt: well, the unit test was missing a check for the bdm turducken16:30
melwitta-ha16:30
jaypipesbauzas: "nvidia-11" isn't a capability.16:30
mriedemmelwitt: the DriverVolumeBlockDevice wraps BlockDeviceMapping and proxies the fields to save on the internal BDM object16:30
jaypipesbauzas: support for, say, OpenCL 2.0 is a capability.16:30
bauzasgot it, good idea16:30
jaypipesbauzas: support for 4 max display heads is a capability (weirdly, but correct)16:30
jaypipesbauzas: support for a particular resolution size is a capability.16:30
openstackgerritMerged openstack/os-vif stable/queens: Fix VF-rep lookup routine to use parent PF number  https://review.openstack.org/55491716:30
melwittmriedem: super turducken16:31
exarlosmriedem: Ohhh. So, yes, I use cinder! Sorry.16:31
exarlosI thought you mean ceph or some other storage backend.16:31
*** do3meli has left #openstack-nova16:32
exarlosmriedem: Again, i'll mention that i'm running package version 15.0.7 - so could well be some way behind on the code.16:34
*** andreas_s has quit IRC16:35
exarlos(I don't see locally the lines you're suggesting on github)16:35
mriedemexarlos: ah ok then that is why you get past this check https://github.com/openstack/nova/blob/15.0.7/nova/virt/libvirt/driver.py#L729416:37
mriedemand you fail here https://github.com/openstack/nova/blob/15.0.7/nova/virt/libvirt/driver.py#L733116:37
mriedemexarlos: and your "instances_path" config option value must be /var/lib/nova/instances right?16:38
mriedemyou're hitting a ProcessExecutionError on the source node during the resize, and the instance is getting put into ERROR state here https://github.com/openstack/nova/blob/stable/ocata/nova/compute/manager.py#L679216:40
mriedemthe guest is otherwise fine on the source node, and you could reset it's state to ACTIVE16:40
mriedemyou'll have to debug why you're getting "Stderr: u"mv: cannot stat \'/var/lib/nova/instance'"16:41
mriedemthis is just trying to rename the directory mv /var/lib/nova/instances/97346b0a-71a5-4a66-a966-a41565cf3c80  /var/lib/nova/instances/97346b0a-71a5-4a66-a966-a41565cf3c80_resize16:41
exarlosmriedem: yeah, that folder does not exist. Hence my question as to whether all VMs should have a folder in this respective /vat/lib/nova/instances folder.16:44
exarlosThey only contain the console.log file.16:44
exarlosAnd only half of the folders are present, if I should expect all "servers" to have a folder here :-|16:44
exarlosinstances_path is default: $state_path/instances16:45
mriedemand state_path is /var/lib/nova16:46
exarlosmriedem: correct.16:47
mriedemyes all instances should have that directory, spawn() eventually calls this https://github.com/openstack/nova/blob/15.0.7/nova/virt/libvirt/driver.py#L310416:48
*** andreas_s has joined #openstack-nova16:49
mriedemhard rebooting the instance should recreate that path16:49
*** yamahata has joined #openstack-nova16:49
*** burt has quit IRC16:50
*** burt has joined #openstack-nova16:51
*** wwriverrat has joined #openstack-nova16:51
*** mdbooth has quit IRC16:53
exarlosmriedem: ah, okay. Concerns me a little bit that this folder isn't already there :-|16:54
mriedemyeah, i don't know why it wouldn't be16:54
mriedemalthough,16:55
mriedemhttps://github.com/openstack/nova/blob/15.0.7/nova/virt/libvirt/driver.py#L7229 might have done something naughty16:55
*** andreas_s has quit IRC16:55
mriedemwell nvm, probably not because if utils.execute('mv', inst_base, inst_base_resize) failed then inst_base_resize shouldn't exist16:56
exarlosit doesn't exist. I check my other compute nodes, and they do seem to have the folders present to match their VMs, but only show a console.log16:57
*** tesseract has joined #openstack-nova16:57
exarlosokay, i'll try and reset the state and fire her up.16:57
mriedemthey only have a console.log because the actual image data for the guest is in the root cinder volume16:58
mriedemyou can also just reboot the instance in ERROR state16:58
mriedemand verify that it recreates the instance dir16:59
*** derekh has quit IRC16:59
mriedemit will likely have to be a HARD reboot though16:59
mriedemhttps://developer.openstack.org/api-ref/compute/#reboot-server-reboot-action16:59
*** itlinux has joined #openstack-nova17:01
*** jpena is now known as jpena|off17:01
*** liuzz has joined #openstack-nova17:02
*** dtantsur is now known as dtantsur|afk17:02
exarlosmriedem: It's starting up :-)17:03
*** hamzy has joined #openstack-nova17:03
*** mgoddard has quit IRC17:03
exarlosAnd that folder exists now in /var/lib/nova/instances/ for the server.17:04
exarlosI guess that using local storage for openstack is really being left behind?17:05
exarlosIs ceph the go-to storage option of choice?17:07
*** lucasagomes is now known as lucas-afk17:08
larsksWill nova permit someone to set up PCI passthrough for a device that device is already bound to an inactive (e.g. powered off) instance?17:08
exarlos(for those of us who can't afford/have space at home for a real storage solution) :-)17:08
mriedemexarlos: ceph is the most widely used cinder backend, i don't know how much people also use it for local ephemeral storage, but i assume if you're doing it with persistent block devices then you might as well also use it for local ephemeral17:10
*** slaweq has joined #openstack-nova17:11
mriedemlarsks: we don't support hot plugging pci devices17:11
mriedemcyborg is the long-term solution for that i believe17:11
mriedemexarlos: i'm not sure why that local disk was gone, or if it has anything to do with your storage option of choice17:11
exarlosmriedem: Thankyou for your time today. I suspect you're rather busy.17:12
mriedemunless you had a hard drive failure on that host or something17:12
larsksmriedem: I wasn't asking about hot plugging...or was I? I didn't think so. I just meant you have instance A configure w/ pci passthrough, you shut down A and then try to create B with the same configuration...would nova allow that?17:12
mriedemlarsks: i hope not17:12
larsksFair enough.17:12
mriedemexarlos: np, thanks and good luck17:12
*** tssurya has quit IRC17:14
*** dpawlik has joined #openstack-nova17:14
*** damien_r has joined #openstack-nova17:15
*** slaweq has quit IRC17:16
*** dpawlik has quit IRC17:18
moshelemriedem: hi, what are the chances to approve this pike backport https://review.openstack.org/#/c/563501/ ?  the packport wan't trivial at all17:20
*** damien_r has quit IRC17:23
*** janki has quit IRC17:24
mriedemmoshele: that enables the nova side of a neutron feature added in pike correct?17:24
mriedemand the change on master was originally reverted https://review.openstack.org/#/c/518833/17:25
mriedemmaster/queens17:25
*** tesseract has quit IRC17:25
moshelemeridem: the first version was revert because it broken heat, but the second one was merged https://review.openstack.org/#/c/519066/17:27
*** damien_r has joined #openstack-nova17:27
*** mvk has joined #openstack-nova17:28
mriedemsure, but i'm still not sure if we should backport something to nova pike to enable a feature in neutron pike17:28
mriedemafk for lunch17:28
*** gjayavelu has joined #openstack-nova17:29
moshelemeridem: this is the feature enablement in pike https://review.openstack.org/#/c/398265/. the other patch was a bug fix (https://review.openstack.org/#/c/519066/ ) which was introduce by something I change in neutron17:30
moshelemerdem: this is the neutron change https://review.openstack.org/#/c/499203/17:31
moshelemriedem:  this is the neutron change https://review.openstack.org/#/c/499203/17:31
moshelemriedem:  ok I will try to catch you after lunchת bon appetit!17:32
*** damien_r has quit IRC17:34
*** spsurya has quit IRC17:42
*** mgoddard has joined #openstack-nova17:44
exarlosThanks again, ciao for now.17:44
*** exarlos has quit IRC17:44
*** udesale has quit IRC17:50
*** mdnadeem has quit IRC17:51
TahvokHey guys!17:56
*** sridharg has quit IRC17:57
TahvokI'm having some boot issues with kvm (using ocata): After creating an instance, I configured some volumes on it, and after shutdown, I cannot bring it up, it's getting stuck in seabios with 'no bootable device' message. If I remove all other volumes I've created, it manages to boot, but fails later as my configuration needs the volumes available17:58
TahvokSo I've checked the bootindex configuration, and it looks like it's completely fine, bootindex is set only on the bootable volume.17:58
TahvokI should also mention that I'm using ceph17:58
mriedemTahvok: do you have this https://review.openstack.org/#/c/561613/ ?18:00
mriedemjust released in 15.1.118:00
TahvokNova,version=15.1.118:00
TahvokLet me check the file just to be sure18:01
TahvokSo, I've just checked this, and not I do not have it18:03
melwittif you do a 'virsh dumpxml' on the instance, you might see the boot_index set incorrectly on your bootable volume (it should be 0). and if it's not, you need the bug fix from 5.1.118:06
melwittthis was the bug https://bugs.launchpad.net/nova/+bug/172958418:06
openstackLaunchpad bug 1729584 in OpenStack Compute (nova) queens "boot from volume + configdrive with virtio-scsi broken (regression)" [High,Fix committed] - Assigned to Artom Lifshitz (notartom)18:06
fricklermriedem: melwitt: removed the -2, will do a closer review tomorrow18:08
melwittcool, thanks frickler18:08
Tahvokmelwitt: yes, it's set as 1 and not 018:09
TahvokI'm restarting nova-compute now, to check the patch18:09
TahvokAnd it's working!18:10
melwittsweet18:10
TahvokI wonder why it says that I have nova 15.1.1 when I definitely do not have this patch18:10
melwitthm, yeah, it should be in there18:11
melwitt5.1.1 was released yesterday, very recent18:11
TahvokSo I definitely do not have this18:12
TahvokAs we rerun ansible a week ago18:12
melwittdo you build packages yourself from stable/ocata? or where did you get the packages?18:13
TahvokI'm using openstack-ansible project18:13
melwittokay. I'm not familiar with how it works but I'm guessing maybe it builds packages off stable as the "next version" which would have been 5.1.118:14
melwittthat is, if you were to run it today it would give you 5.1.2, just a guess18:14
TahvokYes, it's uses hashes from git18:15
TahvokOk, so that might be why I see 5.1.118:15
eanderssonRandom question but why does the scheduler need to log every time it does an instance sync?18:15
mriedemeandersson: need to?18:15
TahvokI will request to bump the hashes so I'll get this fix 'officially'18:16
mriedembecause if thinks you care18:16
eandersson:D18:16
mriedem*it18:16
eanderssonIt's just with 800+ nodes it gets a bit spammy :D18:16
melwittTahvok: ah, k. yeah. so when we released 5.1.1, that means it was tagged as 15.1.1 and pushed to pypi etc as that version18:16
mriedemi had a patch for oslo.log at one point to be able to drop stuff like that in the scheduler to AUDIT log18:16
mriedembelow DEBUG18:16
mriedemeandersson: is it debug today?18:16
eanderssonINFO18:16
melwittTahvok: this was the release patch. I'm not sure if/how openstack-ansible consumes that https://review.openstack.org/56404418:16
eanderssonbut we are still on mitaka (so might have changed)18:17
mriedemeandersson: which specific message?18:17
*** moshele has quit IRC18:17
mriedemSuccessfully synced instances from host ?18:17
mriedemyeah those should probably be debug18:18
eanderssonYea18:18
mriedemthe "received <operation> from unknown host" is probably ok at INFO18:18
mriedembut all the other "OH HI I DID SOMETHING I"M SUPPOSED TO DO!!!" is debug18:18
mriedem*should be debug18:18
eanderssonI can throw up a quick commit on gerrit18:18
mriedemack18:18
mriedemjust don't get any on me18:18
openstackgerritMatt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute  https://review.openstack.org/54142018:20
mriedemmelwitt: this time with 10x more bdmturducken ^ call in the next 20 minutes and we'll double your bdms18:20
melwittand the peasants rejoiced18:20
Tahvokmelwitt: mriedem thanks a lot for helping with the issue! And didn't know what was going on...18:21
TahvokWe've had it for a few months now...18:21
mriedemi can't say that's it, but it sounds right18:21
mriedemyeah it was a regression in ocata i believe18:21
melwittnp, I'd been chasing that bug for a long time to get the fix out18:21
*** tssurya has joined #openstack-nova18:35
*** felipemonteiro__ has joined #openstack-nova18:39
*** moshele has joined #openstack-nova18:39
*** amoralej is now known as amoralej|off18:42
*** felipemonteiro_ has quit IRC18:42
mriedemso uh, i don't think this is ever going to be true https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L319618:49
mriedemis it possible to reboot a rescued instance?18:50
mriedemthe api doesn't allow it18:50
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/vm_states.py#L69-L7218:50
mriedemis there some other place in nova where we internally reboot an instance when it's rescued?18:51
melwittI know not18:52
*** fragatina has quit IRC18:53
mriedemgit blame history here i go18:53
*** mgoddard has quit IRC18:53
*** fragatina has joined #openstack-nova18:53
*** moshele has quit IRC18:53
mriedemhttps://github.com/openstack/nova/commit/39ffe806cdfe97721fee5e48c8591c4c3b52766b18:54
openstackgerritMerged openstack/nova master: support multiple member_of qparams  https://review.openstack.org/56131518:54
mriedemhttps://github.com/openstack/nova/blob/39ffe806cdfe97721fee5e48c8591c4c3b52766b/nova/compute/api.py#L170218:54
*** imacdonn has quit IRC18:54
mriedemwtf, you couldn't reboot a rescue vm18:54
*** imacdonn has joined #openstack-nova18:54
*** eharney has quit IRC18:54
melwittwhat? that patch is about the vm_state, just making sure it stays "rescued" after the reboot. right?18:55
mriedemhttps://review.openstack.org/#/q/topic:bug/1170237+(status:open+OR+status:merged)18:56
mriedemyeah, but if you look at the reboot api code in that same hash, we don't allow rebooting a rescued vm18:56
melwittoh :\18:56
mriedemi mean, we don't allow rebooting an instance with vm_state=RESCUE18:56
mriedemhttps://review.openstack.org/#/q/topic:bug/1170237+(status:open+OR+status:merged)18:56
mriedemlooks like some rax/xen thing18:56
mriedemthat likely never made the api change upstream18:56
mriedemgd rax18:56
melwittguh, weird. so we have patches that say "now you can reboot a rescued instance" but you can't because the API kicks you out18:57
mriedemthe rax api code probably lets you18:57
melwittright18:57
mriedemwhere was that one guy that still works at rax?18:57
mriedemthe guy always in korea..18:58
melwitttbh, I don't know whether we're suppose to be able to reboot rescued instances. I'm not that familiar with the "rescue" function18:58
mriedemneither am i,18:58
mriedembut i can read code :)18:58
mriedemthat's how i found this18:58
melwittwell, yeah. I mean whether or not it makes sense. I would guess from the lack of user complaints that it's not usual to try to reboot one18:58
mriedemmikal: johnthetubaguy: when you're around, maybe you can sort this out - did rax have a proprietary change to allow rebooting rescued instances? https://review.openstack.org/#/q/topic:bug/1170237+(status:open+OR+status:merged)18:59
mriedembecause the upstream api doesn't allow that18:59
mriedemsee https://github.com/openstack/nova/blob/master/nova/compute/vm_states.py#L69-L7218:59
*** openstackgerrit has quit IRC19:05
*** moshele has joined #openstack-nova19:06
*** damien_r has joined #openstack-nova19:07
*** openstackgerrit has joined #openstack-nova19:09
openstackgerritMatt Riedemann proposed openstack/nova master: Fix being able to hard reboot a pausing instance  https://review.openstack.org/56614319:09
*** damien_r has quit IRC19:13
*** dpawlik has joined #openstack-nova19:15
*** damien_r has joined #openstack-nova19:15
*** mgoddard has joined #openstack-nova19:16
*** sapd_ has joined #openstack-nova19:19
*** dpawlik has quit IRC19:19
*** felipemonteiro_ has joined #openstack-nova19:21
*** felipemonteiro__ has quit IRC19:21
*** slaweq has joined #openstack-nova19:21
*** slaweq has quit IRC19:22
*** slaweq has joined #openstack-nova19:22
*** sapd has quit IRC19:22
*** mgoddard has quit IRC19:23
openstackgerritOliver Walsh proposed openstack/nova master: Fix handling of connect issues in _ensure_resource_provider  https://review.openstack.org/56614819:26
mriedemowalsh: i already have a patch up for that19:28
mriedembeat you by a few hours19:28
mriedemhttps://review.openstack.org/#/c/566096/19:28
owalshmriedem: snap19:29
owalshmriedem: worse on pike, it cache None19:32
mriedemyeah it was a backport regression19:32
melwittoh, so _that's_ where the None RP comes from. geesh19:33
melwittI remember we reverted a thing while we were trying to get backports done for the stable releases19:34
mriedemyeah that was ocata19:34
mriedemi suspect the fleetify devstack stuff in pike+ was hiding it for us19:35
mriedemsomehow19:35
mriedemalthough even in ocata devstack i thought we started the compute last19:35
*** zzzeek_ has quit IRC19:35
*** zzzeek has joined #openstack-nova19:37
*** mgoddard has joined #openstack-nova19:38
*** fragatina has quit IRC19:41
*** awaugama has quit IRC19:43
mriedemdansmith: what do you think a safe batch size is for this heal allocations CLI? at first i default to CONF.api.max_limit but that's 1000 which seems way too big https://review.openstack.org/#/c/565886/5/nova/cmd/manage.py@1776 - map_instances defaults to 5019:44
mriedemso was thinking about using 5019:44
dansmithyeah 1000 is too much19:44
dansmith50 is probably good19:44
mriedemonline_data_migrations also does 5019:44
dansmithyup19:44
melwittdid we do an audit of other uses of safe_connect where None can be returned?19:44
mriedemmelwitt: jaypipes has been working on untangling that19:45
melwittk, cool19:45
*** eharney has joined #openstack-nova19:49
*** mgoddard has quit IRC19:52
*** damien_r has quit IRC19:53
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix wrong arguments for 'detach_volume'  https://review.openstack.org/56615219:59
*** gjayavelu has quit IRC20:02
*** liverpoo1er has quit IRC20:04
*** damien_r has joined #openstack-nova20:04
*** mingyu has quit IRC20:06
openstackgerritMatt Riedemann proposed openstack/nova master: Add nova-manage placement heal_allocations CLI  https://review.openstack.org/56588620:06
*** moshele has quit IRC20:08
*** tbachman has quit IRC20:09
mriedemowalsh: if you want to do the backport to queens and pike for https://review.openstack.org/#/c/566096/ that would speed things along so i can +2 the backports20:11
mriedemowalsh: beware: the provider tree stuff in rocky will likely mean merge conflicts for the backports20:11
arvindn05dansmith: looks like there was agreement on the rebuild instance with traits thread. Can you send out an update on ML on the final approach?20:16
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.openstack.org/56615320:17
*** mchlumsky has quit IRC20:17
dansmitharvindn05: I started it a bit ago but got distracted.. if there is agreement you're not blocked right?20:17
*** wolverineav has quit IRC20:19
*** wolverineav has joined #openstack-nova20:20
openstackgerritMatt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute  https://review.openstack.org/54142020:23
arvindn05dansmith: i am blocked at this point because my approved patch https://review.openstack.org/#/c/560596/ is also holding based on the decision on the rebuild issue20:24
*** mingyu has joined #openstack-nova20:24
*** wolverineav has quit IRC20:24
arvindn05my next patches would be update the spec with decision on rebuild and propose the code patch for the same20:25
dansmitharvindn05: you mean you're blocked because your patches don't do what mriedem wants yeah?20:25
dansmitharvindn05: if you'd do what we said in the meeting this morning, then he'd remove his -W and everything would move along, AFAICT20:25
arvindn05dansmith: yup. but i was unfortunately not in the meeting and not entirely sure what approach was decided20:26
dansmithif only there was a log...20:27
arvindn05dansmith: is it fair to summarize it as we want to go check the allocations route20:27
arvindn05(8:25:17 AM) efried: arvindn05: That was the impression I got.  But yeah, let's see what dansmith has to say.20:27
dansmitharvindn05: yes20:27
arvindn05dansmith: great. Thanks for confirming....i will start with the spec and the code patch. glad the deadlock was resolved :)20:28
efried++20:28
*** moshele has joined #openstack-nova20:28
*** damien_r has quit IRC20:29
mriedemarvindn05: fyi http://eavesdrop.openstack.org/meetings/nova/2018/nova.2018-05-03-14.00.log.html#l-16120:30
*** weshay is now known as weshay|rover20:30
openstackgerritMatt Riedemann proposed openstack/nova master: Fix being able to hard reboot a pausing instance  https://review.openstack.org/56614320:31
owalshmriedem: sure20:31
mriedemthanks, hopefully it's not too bad, the patch is pretty isolated20:32
arvindn05mriedem: thanks got that from gibi as well :) last statement was <dansmith> I shall commentificate upon the threadage and reviewage so wanted to confirm :)20:32
*** moshele has quit IRC20:33
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup placement policy generator docs  https://review.openstack.org/56522520:34
arvindn05dansmith: one other clarification, we want to validate the allocations vs traits always correct? Even when we are rebuilding with the same image and no traits have changed20:41
*** evin has quit IRC20:42
*** arvindn05 has quit IRC20:46
*** arvindn05 has joined #openstack-nova20:48
melwittI would think so -- while the image hasn't changed, the underlying resource provider traits could have20:48
mriedemarvindn05: that gets tricky depending on where you do the validation,20:50
mriedemif the image doesn't change, i'm not sure if you get to the point in conductor where we'd be calling the scheduler20:50
mriedemi realze you're not calling the scheduler to do the validation, but i assumed it would be in the same block20:50
arvindn05mriedem: i am thinking it would be in the conductor20:51
arvindn05conductor already has the placement client and all its dependencies need to make the validation20:51
arvindn05conductor.manager.ComputeTaskManager#rebuild_instance somewhere in here is where the validation logic would lie20:52
*** andreas_s has joined #openstack-nova20:53
*** hamzy has quit IRC20:53
mriedemarvindn05: you'd have an else block here https://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/conductor/manager.py#L944 when recreate is False20:55
arvindn05in case of rebuild we would add logic here https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L89720:55
arvindn05mriedem: sorry...looking at your code pointer now20:56
mriedemmelwitt: so i'm looking at https://review.openstack.org/#/c/540258/ and something is bugging me,20:56
mriedemwith an affinity group policy, the group members would all have to be in the same cell because they have to be on the same host20:57
mriedemalso soft-affinity might screw with that...20:57
*** andreas_s has quit IRC20:57
mriedemfor an anti-affinity group, we could have instances in different cells i'd thinkg20:57
mriedem*think20:57
melwittyeah, I think so20:58
melwittare you suggesting that if policy == affinity then limit to same cell?20:58
mriedemso if i'm doing a move operation on an instance in an anti-affinity group, we have a targeted context for the cell that instance being moved is in,20:58
arvindn05mriedem: from the comment in line 916 that whole block executes only in case we are doing a rebuild with a new image....20:58
mriedemand we'll only get group hosts in that cell to make sure the instance doesn't go to a host that another member of the same group in the same cell is in, since we can't move across cells20:59
melwittright20:59
mriedemwhich is correct20:59
arvindn05if we want to run the validation always...we would have to do outside the block, correct?20:59
mriedemarvindn05: well, rebuild + new image or evacuate21:00
mriedemevacuate is a rebuild on another new host21:00
mriedembut same image21:00
mriedemin that case, the scheduler will go through GET /allocation_candidates and do a claim,21:00
mriedemso we don't need to do any image validation for traits in conductor for evacuate b/c that would be redundant21:00
arvindn05yup...i got that for evacuate...how about the case of rebuild without the image being changed21:01
mriedemmelwitt: ok and if we're not targeted (not a move, just instance create), then we need to get group hosts from all cells because we don't know in which cell the instance is going to land in21:01
melwittmriedem: right21:02
mriedemand this happens before we ever run the affinity/anti-affinity filters right?21:02
mriedemyeah called from schedule_and_build_instances21:03
melwittyes, this is the setup actually pretty early on in compute/api, I think21:03
melwittokay maybe not (sorry I forgot)21:03
mriedemok yeah so if i'm creating an instance an in anti-affinity group then the other members of the group could be in any cell and i need to iterate all cells to find which hosts those other members are in,21:04
melwittsetup_instance_group is called in some places and that's what does it (if group.hosts is not yet populated)21:04
melwittit will do that the first time it's called in a code path21:04
mriedemand if i'm creating an instance in an affinity group, if that group already has one member in it, we should know exactly which cell the new instance and new member of that group should get scheduled to21:04
mriedemso in that case, we wouldn't really need to scatter/gather all cells,21:05
mriedemi don't think...21:05
melwitthm, well, I don't know actually. affinity just means once the cell is selected, affine with an instance in the group if there is one. I think that's how it would behave21:05
melwittor does it actually choose the cell based on where the members are? I would have doubted that21:06
melwittyou could have group members in different cells even with the affinity policy, I think21:06
melwittlike, they affine with each other on a per cell basis21:07
melwittand if you're the first group member to land in a cell, you can build anywhere. it's just after that, new members have to affine if they land in that cell21:07
*** fragatina has joined #openstack-nova21:09
openstackgerritOliver Walsh proposed openstack/nova stable/queens: Handle @safe_connect returns None side effect in _ensure_resource_provider  https://review.openstack.org/56616121:09
mriedemi left some comments,21:10
*** fragatina has quit IRC21:11
mriedembut the strict affinity policy should filter such that members of the same group are on the same host21:11
mriedemwhich implies same cell21:11
mriedemif you built 2 instances in concurrent requests to the same strict affinity group, we don't guarantee that those end up on the same host via the scheduler b/c it doesn't have that information; that's why we have the late affinity check in the compute,21:11
mriedemHOWEVER,21:11
*** tssurya has quit IRC21:11
mriedemwith multiple cells, if instance A goes on host 1 in cell1 and instance B goes on host 2 in cell2, i'm not sure the late affinity check will catch that21:12
melwittyeah, that's a good question. I don't think this has been brought up before. I hadn't been thinking to target an instance to a cell where its affinity member lives21:12
*** wolverineav has joined #openstack-nova21:12
mriedemalthough it should.....the late check will ask the instance group for hosts that the other members are in, and if they are in another cell, it will fail21:12
mriedemhttps://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/compute/manager.py#L128721:13
*** dtruong_ has quit IRC21:13
mriedemyeah so i think we'll fail the late affinity check, and reschedule, however, we'll never find a host to fit that policy b/c it's going to be in the wrong cell at that point21:13
mriedemanti-affinity is much easier, different host in same cell, or different cell, doesn't matter21:14
melwittyeah ... I think my patch makes it so that check would do a "all cells" query and maybe it shouldn't ... getting confused21:14
mriedemwell,21:14
mriedemi think there are two cases here:21:14
mriedemfor affintiy21:14
mriedem1. a group member is already in a host (so a cell) and your lookup finds it, we know our new instance has to go in that cell to affine to the existing member21:15
mriedemworst case there is we're needlessly iterating multiple cells when we already know which one we want21:15
mriedemso, perf optimization for later maybe,21:15
*** edmondsw has quit IRC21:15
*** dpawlik has joined #openstack-nova21:16
mriedem2. we're creating multiple instances concurrently where there are no existing members in the affinity group - iterating the cells is fine but we can still have the scheduler put the members in the wrong cells21:16
*** edmondsw has joined #openstack-nova21:16
mriedemi.e. it's the same affinity race for concurrent creates that we've always had,21:16
mriedembut the late check in the compute can't fix that one21:16
mriedembecause we don't reschedule across cells, only within the same cell we're already in21:16
melwittI see, yeah21:16
*** mingyu has quit IRC21:17
melwittI think we have to check all cells though right? we have to pull instances to find out what host they're on21:17
mriedemso in case 2 you get a novalidhost / maxretriesexceeded and have to delete and recreate the instance, which at that point should work21:17
*** fragatina has joined #openstack-nova21:18
openstackgerritOliver Walsh proposed openstack/nova stable/pike: Handle @safe_connect returns None side effect in _ensure_resource_provider  https://review.openstack.org/56616421:18
melwittthat is, I don't think there's a way to avoid having to look in all cells unless you mean do it iteratively instead of a scatter-gather and stop early when found21:18
mriedemmelwitt: yeah i think the latter,21:18
openstackgerritkarim proposed openstack/nova master: Pushing image traits to ironic node  https://review.openstack.org/56562021:18
melwittgotcha21:18
mriedemlike, if policy == 'affinity' lookup the instance mappings for the group members, and they should all be in the same cell, so just get hosts from that one21:19
mriedemor just get one instance mapping for one member in the affinity group and pick it's host, because the new instance *has* to go on that same host21:19
mriedemanyway, like i said, perf optimization for later if we care21:19
mriedemi just wanted to make sure i was thinking through this correctly21:20
*** dpawlik has quit IRC21:21
*** edmondsw has quit IRC21:21
melwittoh, I see what you mean now, get the group members and choose a cell based on that21:21
openstackgerritJay Pipes proposed openstack/nova master: rework how we pass candidate request information  https://review.openstack.org/56616621:21
*** arvindn05 has left #openstack-nova21:22
melwittyeah, I think the only potential wrinkle there is, that would mean affinity always means one cell forever21:22
melwittand I'm not 100% sure that's how it's supposed to work considering cells are transparent, that is, if you imagine you have some application that will run faster if its instances are affined to a host, does that mean you want your application to only be able to run in one cell? or do you want it to run in multiple cells where each cell will have its instances running on the same host21:23
dansmithif you want the latter you need two groups regardless of cells right?21:24
melwittand have multiple instances of your application running from multiple cells? maybe that's not a thing that makes sense but just for the sake of thinking about it21:24
dansmithif you asked for host affinity, then "same cell" is kinda implied no?21:24
melwittmaybe, I'm not that much of an affinity expert21:24
dansmithaffinity means "same host" currently21:24
melwittif it is, then that's cool. I just didn't know21:24
dansmithanti-affinity means "any other host"21:24
dansmithor rather "not the same host"21:25
melwittright, okay. that makes it clear then. I don't know why I was thinking about multiple per cell affinity21:25
*** esberglu has quit IRC21:26
mriedemlooking at https://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/compute/manager.py#L1289 again,21:32
*** arvindn05 has joined #openstack-nova21:32
mriedemi think it's possible that you could have 2 affinity group members in different cells and we wouldn't fail21:32
mriedembecause if the group has instA and instB in cell1 and cell2 respectively, group_hosts = group.get_hosts(exclude=[instance.uuid]) for instA won't return the host for instB because it's in another cell, and we're not targeted for that other cell21:33
mriedemso group_hosts would be empty and we'll say all is clear21:33
* mriedem has a hard time thinking with jesus christ super star blaring from laura's phone in the other room21:34
melwittyeah ... you're talking about the current state of affinity with multiple cells?21:34
mriedemyeah regardless of this change21:34
melwittyeah, what I found is that since we're not targeted, we just get an empty hosts list every time21:34
mriedemhmm, is the group targeted when we get down to compute? https://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/compute/manager.py#L128921:35
melwittno because the group is pulled early before we target anything. I hope I wrote comments in the patch about that, I'm having trouble remembering it even now21:36
melwittoh, that in compute manager, that will be local to a cell21:37
dansmiththat'd be a bug, of course,21:37
*** gjayavelu has joined #openstack-nova21:37
dansmithif two instances in an affinity group were in different cells21:37
dansmithwhich maybe could have happened if we did something dumb in scheduler21:37
dansmith(maybe related to melwitt's patch)21:37
dansmithbut that would be, like, bad, not a legit arrangement21:38
melwittyeah. well, it definitely happens today because the query for hosts will always be empty21:38
mriedemdansmith: yeah it's totally a bug but i think it's something that could happen today with multiple cells and concurrent requests to create instances in the same affinity group21:38
dansmithack, okay21:38
melwittit doesn't have to be concurrent because no matter what, when you go to boot an instance and say "give me the hosts that are in this group" it will hand you an empty list because it's an untargeted DB access by nova-api21:39
*** esberglu has joined #openstack-nova21:39
dansmithprobably worth backporting that fix then and calling it out in a reno that people should audit their groups if they are running multiple cells21:39
dansmithmelwitt: yeah21:39
openstackgerritJulia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity  https://review.openstack.org/56372221:40
mriedemso i still can't tell,21:40
mriedemduring an instance create,21:41
mriedemwhen we get to the late affinity check in compute https://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/compute/manager.py#L128821:41
mriedemis that group going to have a cell-targeted context in it?21:41
dansmithno,21:41
dansmithbecause it's local to the cell21:41
dansmithif it's compute manager21:41
dansmithit can't go anywhere other than the local cell21:41
mriedemso it just reads the [database]/connection21:41
mriedemah right21:41
dansmithright21:41
melwittthe thing I'm concerned about is whether my patch will make that go to all cells, which would be wrong?21:42
dansmithdoes compute manager use setup_instance_group?21:42
dansmithI didn't think so21:42
melwittoh, no it doesn't. okay, my bad21:43
mriedemsetup_instance_group is only called from conductor i think, prior to sending the request spec to the scheduler to pick a host21:43
dansmithyou know that there's still a race though, right?21:44
dansmiththat's what the late-binding check is for,21:44
dansmithand we can't do that from the cell anyway21:44
dansmithso melwitt's patch will close the race by not just getting an empty list every time,21:44
dansmithbut if we're concurrently booting several instances, you can still sprinkle them among the cells21:45
dansmithalthough, that makes me wonder...21:45
mriedemright i know for the concurrent boot scenario, we're already in trouble21:45
dansmithinstead of doing the check late like we do now,21:45
dansmithwhat if we create the instance mapping (which means we've destined it to a cell) and then do the affinity check right there to make sure that the group hasn't spread across cells,21:46
dansmiththen once we've done that, we can let the late check happen in computemanager like normal, which will only need to consider the local cell anyway21:46
mriedemthere is still a race there though isn't there?21:47
mriedemit's the same race as for getting on different computes21:47
dansmithno, because if we know everything is in the same cell, then the late check as it is today works fine yeah?21:47
dansmithoh, although we can't look up the server group members there21:48
dansmiththat's why21:48
mriedemmy brain hurts21:48
mriedemso to summarize, i think with the concurrent affinity group create race, before multi-cell, you could race past the scheduler and late affinity check in the compute has to fix the race,21:49
mriedemwith multiple cells, you could race past the scheduler and the late affinity check in compute might not fail because it can't "see" that the group has members in another cell21:49
mriedemto which it's not affined21:49
mriedemand you end up with an affinity group with members in multiple cells21:49
dansmithno,21:49
dansmiththe late check is just for anti-affinity21:50
dansmithright?21:50
mriedemno21:50
* mriedem links again https://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/compute/manager.py#L128821:50
dansmithoh, I guess it does check both21:50
dansmiththe comment only says anti-affinity21:50
dansmithmriedem: so yes, your second comment is right, except that:21:51
mriedemso before multi-cell, you'd reschedule until you find the right host or fail,21:51
dansmith1. for anti-affinity it doesn't matter since anything it doesn't find is clearly not on the same host and,21:51
dansmith2. the problem is we can't upcall to get the members, not just that we can't see them in our db21:51
mriedemafter multi-cell, we might not even see the problem21:51
dansmithright but with multi-cell #2 gets you anyway21:51
*** liverpooler has joined #openstack-nova21:51
mriedemagree with 1 yes, anti-affinity is totes fine21:51
dansmithbecause of the upcall21:51
*** Guest49729 has quit IRC21:52
mriedemif the cell isn't configured to hit the api db, then yes we can't upcall to iterate the cells to find hosts for all members in the group21:52
*** yamamoto has joined #openstack-nova21:53
mriedemthis https://github.com/openstack/nova/blob/0ef3c685b9d2e0049f38fcf1a268870e69a5b9cf/nova/objects/instance_group.py#L47321:53
mriedemyeah?21:53
*** moshele has joined #openstack-nova21:53
dansmithwell,21:53
*** brault has quit IRC21:53
dansmithwe won't iterate cells anyway, but yeah21:53
*** itlinux_ has joined #openstack-nova21:53
*** wolverineav has quit IRC21:55
*** itlinux has quit IRC21:56
*** moshele has quit IRC21:57
*** yamamoto has quit IRC21:57
*** itlinux_ has quit IRC21:58
mriedemso the only way to really fix this is for the scheduler to be aware of the location of the group members at the time of picking a host, and that has to be global, so placement,21:58
mriedemand we likely need to model affinity (distance) for the group members using....provider aggregates? i know this is a rathole,21:58
mriedemi keep thinking of cells as a provider aggregate21:59
dansmithwell, this is why I've been punting the problem because placement affinity is better for lots of reasons21:59
mriedemmelwitt: anyway, i think the functional change in your patch is ok, my main issue / concern is with the test22:00
melwittmriedem: yeah, in the middle of replying now22:00
mriedemmelwitt: a functional test with real services would be ideal22:00
melwittwell, I think we need the unit test to cover all the weird paths, unless I'm just not thinking creatively enough22:00
mriedemthe majority of the work in a functional test for stuff like this is in the setup,22:01
melwittI'm gonna try to take a step back and see if I can come up with a real world scenario(s) that will take all of the paths that we can verify22:01
mriedemafter that it's just running scenarios and asserting stuff22:01
mriedemhttps://review.openstack.org/#/c/565886/6/nova/tests/functional/test_nova_manage.py@357 is an example with 2 working cells22:02
melwittthis is weird because there are two steps: 1) look up the hosts for members of the requested group 2) look up the hosts for members of the group the instance is a member of22:03
melwittand I was trying to test that both of those queries do the multi-cell thing. but yeah, probably could set this up for a functional test. I'll try it22:04
mriedemwith complicated changes like this, i find it's easier to write the functional test to setup the environment like the user would run a use case22:05
mriedemusing the actual APIs22:05
mriedemto create the groups and add members to them and such22:05
*** eharney has quit IRC22:06
*** mingyu has joined #openstack-nova22:06
mriedemotherwise it's too easy to fake things out in the db that aren't accurate22:06
melwittfor whatever reason, I did not expect it would be easy to do a functional test. I agree it would be a lot better to reason about too22:07
mriedemmostly just copy/paste the setup22:07
mriedempretty easy22:07
*** rcernin has joined #openstack-nova22:07
mriedemthe one thing with multi-cell functional and having different hosts in different cells, you'll need https://review.openstack.org/#/c/558160/22:08
mriedemotherwise the computes all get created in the default cell122:08
mriedembut that's approved now so shouldn't be a problem - you just need to specify the cell you want the compute in when you create it22:08
melwittI know, I tried to solve that problem with my CellDatabases patch months ago but people weren't okay with it because I did the ServiceWrapper thing22:09
mriedemi've been told i need to go workout because i've become somewhat of a troll, so ttyl22:09
*** mriedem is now known as mrjazzercise22:09
*** gjayavelu has quit IRC22:20
openstackgerritJay Pipes proposed openstack/nova master: process groups individually and merge candidates  https://review.openstack.org/56618022:23
*** slaweq has quit IRC22:23
*** moshele has joined #openstack-nova22:29
*** slaweq has joined #openstack-nova22:35
*** pchavva has quit IRC22:36
*** vladikr has quit IRC22:36
*** moshele has quit IRC22:38
*** felipemonteiro__ has joined #openstack-nova22:43
*** felipemonteiro__ has quit IRC22:44
*** liverpooler has quit IRC22:44
*** felipemonteiro__ has joined #openstack-nova22:44
*** felipemonteiro_ has quit IRC22:47
openstackgerritMerged openstack/nova master: Handle @safe_connect returns None side effect in _ensure_resource_provider  https://review.openstack.org/56609622:52
*** moshele has joined #openstack-nova22:53
*** damien_r has joined #openstack-nova22:55
*** gjayavelu has joined #openstack-nova22:57
openstackgerritMerged openstack/nova master: Fix the request context in ServiceFixture  https://review.openstack.org/55816022:58
*** threestrands has joined #openstack-nova22:59
*** damien_r has quit IRC22:59
*** felipemonteiro__ has quit IRC23:04
*** hongbin has quit IRC23:06
openstackgerritArvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image  https://review.openstack.org/56071823:07
*** gouthamr has quit IRC23:08
*** moshele has quit IRC23:08
*** rajinir has quit IRC23:09
*** gouthamr has joined #openstack-nova23:10
*** slaweq has quit IRC23:13
*** dpawlik has joined #openstack-nova23:14
*** dpawlik has quit IRC23:18
*** gouthamr has quit IRC23:18
*** masber has joined #openstack-nova23:23
idlemindhttp://paste.openstack.org/show/720326/23:33
idlemindI need those database connections to be updated to .9 not .11 ... what's the best way to do that?23:34
*** mlavalle has quit IRC23:34
idlemindcan i just drop the cells (delete) and they'll get recreated?23:34
idlemindor should i update them23:34
openstackgerritOliver Walsh proposed openstack/nova stable/pike: Handle @safe_connect returns None side effect in _ensure_resource_provider  https://review.openstack.org/56616423:46
openstackgerritArvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image  https://review.openstack.org/56071823:48
*** gouthamr has joined #openstack-nova23:51
*** yamamoto has joined #openstack-nova23:59

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