Tuesday, 2018-01-23

*** r-daneel has quit IRC00:01
*** itlinux has quit IRC00:03
*** gcb has quit IRC00:05
*** hongbin has quit IRC00:06
mriedemi'm +2 on the OSC CLI change for show/set/delete allocations https://review.openstack.org/#/c/457534/ - i think that's a good one to get in before the final client release freeze this week (same day as FF)00:07
mriedembest way to review these i've found is pull them down to your devstack and kick the tires00:08
*** yasemin has joined #openstack-nova00:08
melwittmriedem: is there something special that has to be done to use the osc-placement plugin with OSC? I tried to use it last week and failed to figure it out. there were no 'openstack placement' or 'openstack resource' commands available00:10
mriedemmelwitt: git clone the repo,00:10
mriedemthen pip install it00:10
mriedemthen just: openstack resource provider -?00:11
mriedemfor the list of commands00:11
melwittokay. in my devstack it was installed and I even tried pip removing and pip installing it again with no luck. I'll try it again00:11
melwittthat is, the osc-placement package got installed by devstack by itself (I didn't do anything to install it manually)00:12
mriedemhmm, did you have the latest version of the osc-placement repo?00:12
*** Hazelesque has quit IRC00:12
mriedemoh...00:12
melwittno, it took whatever was on pypi00:12
mriedemi always git clone it into /opt/stack/00:12
*** robcresswell has quit IRC00:12
melwittgood to know00:12
mriedemthe pypi version doesn't have any actual CLIs yet00:12
melwittoh, no wonder. hah00:12
mriedemi'm waiting to get at least this last allocations one merged before we release for queens00:13
mriedemideally we could get to https://review.openstack.org/#/c/511182/ yet for queens00:14
mriedembecause not being able to work with resource classes in the CLI makes the other existing CLIs can't of hard00:14
mriedem*kind of hard00:14
*** chyka_ has joined #openstack-nova00:14
melwittack00:14
*** Hazelesque has joined #openstack-nova00:15
*** sdague has quit IRC00:16
*** AlexeyAbashkin has joined #openstack-nova00:16
*** chyka has quit IRC00:18
*** AlexeyAbashkin has quit IRC00:21
*** yangyapeng has quit IRC00:26
*** gongysh has joined #openstack-nova00:35
*** dikonoor has quit IRC00:38
*** chyka_ has quit IRC00:40
*** Tom-Tom has joined #openstack-nova00:43
openstackgerritGao Fei proposed openstack/nova master: Redirects to a valid https link from the old one  https://review.openstack.org/53524400:45
*** tetsuro_ has joined #openstack-nova00:47
*** Tom-Tom has quit IRC00:47
*** zhurong has joined #openstack-nova00:48
*** sambetts|afk has quit IRC00:54
*** sambetts_ has joined #openstack-nova00:54
*** Dinesh_Bhor has joined #openstack-nova00:57
*** lbragstad has quit IRC00:57
*** Dinesh_Bhor has quit IRC00:59
*** crushil has joined #openstack-nova01:01
*** zhaochao has joined #openstack-nova01:05
*** hiro-kobayashi has joined #openstack-nova01:06
*** phuongnh has joined #openstack-nova01:06
*** mdnadeem has joined #openstack-nova01:06
*** mdnadeem has quit IRC01:07
*** mdnadeem has joined #openstack-nova01:07
*** sree has joined #openstack-nova01:11
*** Tom-Tom has joined #openstack-nova01:12
*** zhurong has quit IRC01:13
*** yangyapeng has joined #openstack-nova01:13
*** sree has quit IRC01:15
*** tiendc has joined #openstack-nova01:17
*** gongysh has quit IRC01:21
*** liverpooler has quit IRC01:23
*** Dinesh_Bhor has joined #openstack-nova01:23
*** sree has joined #openstack-nova01:27
*** sree has quit IRC01:31
*** Yikun has quit IRC01:36
*** Yikun has joined #openstack-nova01:37
*** sree has joined #openstack-nova01:40
*** pooja has quit IRC01:45
*** sree has quit IRC01:45
*** hshiina has joined #openstack-nova01:46
*** chyka has joined #openstack-nova01:48
*** sree has joined #openstack-nova01:53
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref  https://review.openstack.org/52291801:54
*** sree has quit IRC02:02
*** sree has joined #openstack-nova02:03
*** chyka has quit IRC02:06
*** efried_back_wed has quit IRC02:06
*** hongbin has joined #openstack-nova02:08
*** sree has quit IRC02:08
*** yangyapeng has quit IRC02:08
*** robcresswell has joined #openstack-nova02:09
openstackgerritZhenyu Zheng proposed openstack/nova master: Use neutron port_list when filtering instance by ip  https://review.openstack.org/52550502:12
*** yangyapeng has joined #openstack-nova02:13
*** lbragstad has joined #openstack-nova02:14
*** gongysh has joined #openstack-nova02:15
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948302:15
*** efried_back_wed has joined #openstack-nova02:19
Kevin_Zhengmriedem https://review.openstack.org/525505 <- rebased due to conflict in test_compute_api.py02:19
*** liverpooler has joined #openstack-nova02:24
openstackgerritMerged openstack/osc-placement master: CLI for allocations  https://review.openstack.org/45753402:24
*** sree has joined #openstack-nova02:26
*** harlowja has quit IRC02:27
*** annp has joined #openstack-nova02:28
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref  https://review.openstack.org/52291802:30
*** sree has quit IRC02:31
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types  https://review.openstack.org/43083902:32
*** suresh12 has quit IRC02:39
*** yingjun has joined #openstack-nova02:41
*** tbachman has quit IRC02:41
*** suresh12 has joined #openstack-nova02:42
*** tianhui has quit IRC02:45
*** suresh12 has quit IRC02:47
*** bkopilov has quit IRC02:53
openstackgerritMerged openstack/nova master: Fix update_cell to ignore existing identical cells  https://review.openstack.org/53654602:59
*** vladikr has quit IRC02:59
openstackgerritMerged openstack/nova master: Start moving users of parted to privsep.  https://review.openstack.org/51901103:00
*** vladikr has joined #openstack-nova03:00
*** dikonoor has joined #openstack-nova03:03
*** vladikr has quit IRC03:05
*** hieulq has joined #openstack-nova03:09
*** smatzek has joined #openstack-nova03:11
*** dave-mcc_ has quit IRC03:16
*** gongysh has quit IRC03:18
*** hiro-kobayashi has quit IRC03:24
*** gongysh has joined #openstack-nova03:24
*** mgariepy has quit IRC03:24
*** fragatina has quit IRC03:25
*** hieulq_ has joined #openstack-nova03:25
*** sree has joined #openstack-nova03:26
*** phuongnh has quit IRC03:27
*** tiendc has quit IRC03:27
*** mgariepy has joined #openstack-nova03:27
*** hieulq has quit IRC03:28
*** annp has quit IRC03:29
*** sree has quit IRC03:31
*** tiendc has joined #openstack-nova03:32
*** phuongnh has joined #openstack-nova03:32
*** abhishekk has joined #openstack-nova03:34
*** smatzek has quit IRC03:34
*** amodi has quit IRC03:35
*** suresh12 has joined #openstack-nova03:37
*** suresh12 has quit IRC03:42
*** hshiina has quit IRC03:43
*** annp has joined #openstack-nova03:47
*** hshiina has joined #openstack-nova03:47
*** armax has quit IRC03:48
*** suresh12 has joined #openstack-nova03:49
*** armax has joined #openstack-nova03:55
*** gongysh has quit IRC03:55
*** liverpooler has quit IRC03:55
*** sree has joined #openstack-nova03:57
*** yamahata has joined #openstack-nova03:57
openstackgerritMerged openstack/nova master: Remove redundant swap_volume tests  https://review.openstack.org/53117903:59
*** fragatina has joined #openstack-nova04:01
*** sapd_ has quit IRC04:02
*** sapd_ has joined #openstack-nova04:02
*** suresh12 has quit IRC04:02
*** links has joined #openstack-nova04:02
*** fragatina has quit IRC04:05
*** Tom-Tom has quit IRC04:07
*** links has quit IRC04:07
*** Tom-Tom has joined #openstack-nova04:07
*** Tom-Tom has quit IRC04:07
*** Tom-Tom_ has joined #openstack-nova04:07
*** mlavalle has joined #openstack-nova04:08
*** fragatina has joined #openstack-nova04:10
*** moshele has joined #openstack-nova04:11
*** diga has joined #openstack-nova04:11
*** fragatina has quit IRC04:11
*** suresh12 has joined #openstack-nova04:13
*** Tom-Tom_ has quit IRC04:18
*** Tom-Tom has joined #openstack-nova04:19
*** links has joined #openstack-nova04:19
*** moshele has quit IRC04:20
*** bkopilov has joined #openstack-nova04:24
*** gongysh has joined #openstack-nova04:24
*** annp has quit IRC04:28
*** tiendc has quit IRC04:28
*** annp has joined #openstack-nova04:28
*** tiendc has joined #openstack-nova04:28
*** david-lyle has quit IRC04:30
*** dklyle has joined #openstack-nova04:31
*** dikonoor has quit IRC04:33
*** suresh12 has quit IRC04:36
*** hongbin has quit IRC04:37
*** Tom-Tom has quit IRC04:38
*** bhujay has joined #openstack-nova04:41
*** psachin has joined #openstack-nova04:42
*** sree has quit IRC04:42
*** sree has joined #openstack-nova04:42
*** Tom-Tom has joined #openstack-nova04:44
*** fragatina has joined #openstack-nova04:45
*** gongysh has quit IRC04:47
*** Tom-Tom has quit IRC04:48
*** armax_ has joined #openstack-nova04:49
*** armax has quit IRC04:49
*** armax_ has quit IRC04:53
*** armax has joined #openstack-nova04:56
*** ratailor has joined #openstack-nova05:01
*** dikonoor has joined #openstack-nova05:02
*** vish_18 has left #openstack-nova05:02
*** armax has quit IRC05:03
*** links has quit IRC05:06
*** links has joined #openstack-nova05:08
*** suresh12 has joined #openstack-nova05:11
*** claudiub has joined #openstack-nova05:11
*** harlowja has joined #openstack-nova05:12
*** harlowja has quit IRC05:13
*** moshele has joined #openstack-nova05:16
takashin05:19
*** crushil has quit IRC05:21
*** pcaruana has joined #openstack-nova05:24
*** trinaths has joined #openstack-nova05:25
*** ratailor has quit IRC05:25
*** ratailor has joined #openstack-nova05:27
*** zhurong has joined #openstack-nova05:29
*** ratailor has quit IRC05:34
*** smatzek has joined #openstack-nova05:35
*** pcaruana has quit IRC05:36
*** sidx64 has joined #openstack-nova05:37
*** sidx64 has quit IRC05:38
*** smatzek has quit IRC05:39
*** sidx64 has joined #openstack-nova05:40
*** gongysh has joined #openstack-nova05:40
*** ratailor has joined #openstack-nova05:41
*** ratailor_ has joined #openstack-nova05:45
*** dikonoor has quit IRC05:46
*** ratailor has quit IRC05:47
*** artom has quit IRC05:51
*** artom has joined #openstack-nova05:51
fanzhanghi anyone here05:52
fanzhangI hit a H202 error while running tox pep8 check, it's because my unit test using something like self.assertRaises(Exception,... I was wondering is H202  a mandatory rule or not? Please, if anyone knows about that, please do me a favor :)05:56
*** jaosorior has quit IRC05:57
*** jaosorior has joined #openstack-nova05:57
*** dikonoor has joined #openstack-nova05:58
*** mlavalle has quit IRC06:00
jianghuawfanzhang, it's mandatory I think. You should define a specific exception instead of using the general Exception.06:02
*** mlavalle has joined #openstack-nova06:03
*** mlavalle has quit IRC06:03
fanzhangcool, thanks a lot. jianghuaw :)06:03
*** amotoki has quit IRC06:03
jianghuawnp06:03
*** amotoki has joined #openstack-nova06:04
*** diga has quit IRC06:05
*** Dinesh_Bhor has quit IRC06:06
*** Dinesh_Bhor has joined #openstack-nova06:06
*** liuzz has joined #openstack-nova06:08
*** janki has joined #openstack-nova06:13
*** Eran_Kuris has joined #openstack-nova06:14
*** yamahata has quit IRC06:15
*** stakeda has joined #openstack-nova06:17
*** mdnadeem has quit IRC06:17
*** lpetrut has joined #openstack-nova06:21
*** Tom-Tom has joined #openstack-nova06:24
*** hshiina has quit IRC06:25
*** sidx64 has quit IRC06:27
*** trinaths has left #openstack-nova06:29
*** Yikun has quit IRC06:30
SpazmoticYou're actually here jianghuaw? :D  Didn't think you hung out on IRC06:30
SpazmoticI saw you put some XenAPI notes on the PTG, Did you plan on attending Dublin?06:31
*** Yikun has joined #openstack-nova06:31
SpazmoticHoping to get Corporate approval to attend, if so, would be nice to talk to another person interested in the XenAPI drivers06:31
jianghuawSpazmotic, yes. I'm planning to attend the PTG.06:31
jianghuawSpazmotic, thanks.06:32
*** mdnadeem has joined #openstack-nova06:32
jianghuawSpazmotic, hope I can see you in Dublin.06:32
SpazmoticWell wonderful to meet you.  If you ever want to talk i'm around this time of night. I remain on US Night shift / Seoul time :p06:32
*** sidx64 has joined #openstack-nova06:33
jianghuawnice to know:-)06:33
*** ratailor__ has joined #openstack-nova06:36
*** salv-orlando has joined #openstack-nova06:36
*** ratailor_ has quit IRC06:38
*** sridharg has joined #openstack-nova06:41
*** damien_r has quit IRC06:47
*** sidx64 has quit IRC06:53
*** sidx64 has joined #openstack-nova06:55
*** itlinux has joined #openstack-nova06:56
*** psachin has quit IRC06:56
SpazmoticSome Tuesday morning KPOP before we start waking up and working on actual serious things.  Enjoy your days friends, i'll be around.  https://www.youtube.com/watch?v=Ri6wvGjuoOg06:59
*** itlinux has quit IRC06:59
*** sidx64 has quit IRC07:00
*** tuanla____ has joined #openstack-nova07:01
*** trinaths has joined #openstack-nova07:03
*** sidx64 has joined #openstack-nova07:03
*** tovin07 has joined #openstack-nova07:04
*** gszasz has quit IRC07:05
*** sidx64 has quit IRC07:06
*** sidx64 has joined #openstack-nova07:07
*** sidx64_ has joined #openstack-nova07:12
*** sidx64 has quit IRC07:14
*** sidx64 has joined #openstack-nova07:15
*** sidx64_ has quit IRC07:17
*** fragatina has quit IRC07:18
*** itlinux has joined #openstack-nova07:27
*** edand has joined #openstack-nova07:28
*** pcaruana has joined #openstack-nova07:33
*** lpetrut has quit IRC07:33
*** vipul has joined #openstack-nova07:35
*** slaweq has joined #openstack-nova07:38
*** andreas_s has joined #openstack-nova07:38
*** lpetrut has joined #openstack-nova07:41
*** rha has quit IRC07:44
*** suresh12 has quit IRC07:47
*** andreas_s has quit IRC07:49
*** tbachman has joined #openstack-nova07:50
*** suresh12 has joined #openstack-nova07:51
ameedaMorning07:51
*** ralonsoh has joined #openstack-nova07:51
ameedawhat can I do with this ERROR log "http://paste.openstack.org/show/650903/"07:52
*** psachin has joined #openstack-nova07:53
*** suresh12 has quit IRC07:56
*** itlinux has quit IRC07:58
*** tbachman has quit IRC07:58
*** pcaruana has quit IRC07:58
*** rcernin has quit IRC08:00
*** sidx64 has quit IRC08:02
*** rushiagr has quit IRC08:04
*** AlexeyAbashkin has joined #openstack-nova08:05
*** sahid has joined #openstack-nova08:05
*** tesseract has joined #openstack-nova08:09
*** matrohon has joined #openstack-nova08:09
*** rushiagr has joined #openstack-nova08:10
*** pcaruana has joined #openstack-nova08:13
*** damien_r has joined #openstack-nova08:14
fanzhanghi ameeda, I met this error some days earlier, updating your sqlite to sqlite-3.11.0 and your sqlite-libs to sqlite-libs-3.11.0 fixed my problem08:15
fanzhangI mean it could help :)08:15
*** sapd_ has quit IRC08:16
*** alexchadin has joined #openstack-nova08:16
*** sapd_ has joined #openstack-nova08:16
*** takashin has left #openstack-nova08:16
*** lajoskatona has joined #openstack-nova08:16
*** hiro-kobayashi has joined #openstack-nova08:19
*** lpetrut has quit IRC08:21
*** sahid has quit IRC08:23
*** sahid has joined #openstack-nova08:26
*** salv-orlando has quit IRC08:29
*** salv-orlando has joined #openstack-nova08:29
openstackgerritTetsuro Nakamura proposed openstack/nova master: tests for alloc candidates with nested and traits  https://review.openstack.org/53189908:31
*** Eran_Kuris has quit IRC08:33
*** bkopilov has quit IRC08:33
*** Eran_Kuris has joined #openstack-nova08:34
*** salv-orlando has quit IRC08:34
*** gszasz has joined #openstack-nova08:34
*** annp has quit IRC08:38
*** tiendc has quit IRC08:38
*** annp has joined #openstack-nova08:38
*** tuanla____ has quit IRC08:38
*** tovin07 has quit IRC08:38
*** phuongnh has quit IRC08:38
*** tiendc has joined #openstack-nova08:38
*** phuongnh has joined #openstack-nova08:39
*** tuanla____ has joined #openstack-nova08:39
*** tovin07 has joined #openstack-nova08:39
ameedafanzhang: thanks , but how can I update that ?08:39
*** mdnadeem has quit IRC08:42
*** gouthamr has joined #openstack-nova08:44
*** jafeha__ is now known as jafeha08:44
*** bkopilov has joined #openstack-nova08:46
*** itlinux has joined #openstack-nova08:46
fanzhangameeda: this part is where you should google it. :) I'll suggest that you get the right rpm packages and just install them.08:46
*** jpena|off is now known as jpena08:49
*** cdent has joined #openstack-nova08:52
*** Tom-Tom_ has joined #openstack-nova08:57
ameedafanzhang: hehe, ok thank you very much !08:58
*** gouthamr has quit IRC08:58
fanzhangameeda: np08:59
ameedafanzhang: but you know, when I add my code the error appears, is that caused by my code ?08:59
*** salv-orlando has joined #openstack-nova08:59
fanzhangameeda: I'm not sure. Have you tested before adding your codes? I don't know, you can surely try to find out. :)09:00
*** Tom-Tom has quit IRC09:01
*** mdnadeem has joined #openstack-nova09:01
bauzasgood morning Novza09:01
ameedayes I test it without my code, I didn't see the error.09:01
*** Eran_Kuris has quit IRC09:01
*** bkopilov has quit IRC09:02
*** amorin[away] is now known as amorin09:02
*** priteau has joined #openstack-nova09:02
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector  https://review.openstack.org/46835309:04
*** gouthamr has joined #openstack-nova09:06
fanzhangameeda: well, actually, I never test it before addming my codes, but updating sqlite related rpms solved my problem.09:06
ameedafanzhang: so let me check that, thanks !09:06
fanzhangameeda: :)09:07
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status  https://review.openstack.org/53672209:08
*** sidx64 has joined #openstack-nova09:11
*** avolkov has joined #openstack-nova09:12
*** itlinux has quit IRC09:12
SpazmoticHmm I like that09:13
*** avolkov has quit IRC09:13
SpazmoticWould help with scheduler failures as well.09:13
*** avolkov has joined #openstack-nova09:13
*** Eran_Kuris has joined #openstack-nova09:14
*** bkopilov has joined #openstack-nova09:15
ameedajaypipes: are you around?09:20
*** artom has quit IRC09:20
*** claudiub|2 has joined #openstack-nova09:21
*** artom has joined #openstack-nova09:21
openstackgerritGao Fei proposed openstack/nova master: Replace Chinese punctuation with English punctuation  https://review.openstack.org/53672709:21
ameedaany work-flow can check this for me ? "https://review.openstack.org/#/c/526900/"09:22
*** claudiub has quit IRC09:24
openstackgerritStephen Finucane proposed openstack/nova master: Replace curly quotes with straight quotes  https://review.openstack.org/53672709:25
*** mvk has quit IRC09:26
openstackgerritRadoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree()  https://review.openstack.org/53634809:27
*** dikonoo has joined #openstack-nova09:31
bauzasjianghuaw: around ?09:33
*** vivsoni has quit IRC09:33
bauzasjianghuaw: I don't know if you tested your vGPU patches with all the instance actions09:33
*** vivsoni has joined #openstack-nova09:33
*** tesseract-RH has joined #openstack-nova09:33
*** xinliang has quit IRC09:34
bauzasjianghuaw: but see my relnote for libvirt https://review.openstack.org/#/c/533818/5/releasenotes/notes/add-support-for-vgpu-libvirt-91d2983e643f5ff1.yaml09:34
*** tesseract-RH has quit IRC09:34
*** dikonoor has quit IRC09:34
bauzasjianghuaw: IMHO, you should be impacted by possibly the same problems09:34
bauzaslike for migrating09:34
bauzasor resizing09:34
*** tesseract-RH has joined #openstack-nova09:35
gibiesberglu_: thanks for the update, I'm +2 on the SEA patch09:35
*** tesseract has quit IRC09:37
openstackgerritAlex Xu proposed openstack/nova master: placement: using the dict format for the allocations  https://review.openstack.org/53608309:37
openstackgerritAlex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs  https://review.openstack.org/53608509:37
*** yingjun has quit IRC09:37
*** derekh has joined #openstack-nova09:39
SpazmoticMorning UK Nova09:40
Spazmoticgood to see you all :)09:40
*** alexchadin has quit IRC09:40
alex_xugibi: bauzas ^, I just fix the functional tests09:40
Spazmoticjianghuaw had some meetings, fyi bauzas09:40
alex_xubauzas: also there need one more +2 https://review.openstack.org/#/c/535642/4 :) appreciate you can help it09:41
*** alexchadin has joined #openstack-nova09:41
*** alexchadin has quit IRC09:41
*** alexchadin has joined #openstack-nova09:41
*** alexchadin has quit IRC09:42
SpazmoticIf anyone gets a chance with XenAPI knowledge, would love a look at https://review.openstack.org/#/c/533168/.  Been sitting idle for about 12 days09:42
*** alexchadin has joined #openstack-nova09:42
*** alexchadin has quit IRC09:42
*** alexchadin has joined #openstack-nova09:43
*** alexchadin has quit IRC09:43
*** alexchadin has joined #openstack-nova09:44
*** alexchadin has quit IRC09:44
*** alexchadin has joined #openstack-nova09:44
*** alexchadin has quit IRC09:45
bauzasSpazmotic: ok, thanks09:46
bauzasSpazmotic: you working on Xen ?09:46
bauzasalex_xu: looking09:46
alex_xubauzas: thanks!09:46
*** xinliang has joined #openstack-nova09:46
maciejjozefczykHello everyone!09:47
bauzasalex_xu: I promised today to be a review day09:47
bauzasunfortunately, I have some nits to modify and then I'm looking09:47
bauzasthe placement API for traits is something really crucial so yeah09:48
alex_xubauzas: got it, appreciate for that!09:48
*** Dinesh_Bhor has quit IRC09:48
*** Dinesh_Bhor has joined #openstack-nova09:49
*** Dinesh_Bhor has quit IRC09:49
Spazmoticyes sir bauzas09:51
gibialex_xu: looking as well09:51
*** amoralej has joined #openstack-nova09:52
SpazmoticBack and forth planting my pepper seeds for the season, sorry :)09:53
*** tuanla____ has quit IRC09:55
*** tovin07 has quit IRC09:55
*** phuongnh has quit IRC09:55
*** jaosorior has quit IRC09:55
*** phuongnh has joined #openstack-nova09:56
*** tuanla____ has joined #openstack-nova09:56
*** tovin07 has joined #openstack-nova09:56
SpazmoticSuperhot Peppers and XenAPI.. essentially the same thing when caring for them.09:56
cdenthandle with care09:56
SpazmoticIndeedie heh09:57
*** dtantsur|afk is now known as dtantsur09:58
*** markvoelker has quit IRC09:59
*** mvk has joined #openstack-nova09:59
amoralejmelwitt, we are hitting a problem in last periodic in a tempest test https://bugs.launchpad.net/tripleo/+bug/174490710:01
openstackLaunchpad bug 1744907 in tripleo "Tempest test: "test_create_second_image_when_first_image_is_being_saved" failing in featureset020 periodic job" [Critical,New]10:01
amoraleji'm not sure if it's nova or glance related10:01
amoralejany idea?10:02
*** jcosmao has joined #openstack-nova10:02
amoralejsecond image is being properly created when error is expected10:02
alex_xugibi: thanks10:04
*** Eran_Kuris has quit IRC10:04
*** sree has quit IRC10:06
mdboothlyarwood: Have you seen melwitt 's review of the luks patch?10:06
*** Eran_Kuris has joined #openstack-nova10:06
*** sree has joined #openstack-nova10:06
mdboothSpazmotic: Expect severe pain the morning after use?10:07
mdboothlyarwood: I think you're still good, but you are missing 2 tests.10:07
jianghuawbauzas, sorry for late response. Just ended a meeting. thanks Spazmotic:-)10:09
bauzasno worries, see my earlier pings10:09
jianghuawbauzas, yes. XenAPI doesn't support suspend also.10:09
lyarwoodmdbooth: yup working through it now10:09
jianghuawXenAPI has precheck on it.10:09
bauzasjianghuaw: and resize I guess ?10:09
jianghuawI've not verified resizing.10:10
mdboothlyarwood: I've replied to most things if you refresh (and you're interested)10:10
jianghuawThe suspend will be rejected by XenServer XAPI if there is vGPU attached in the VM.10:11
*** Eran_Kuris has quit IRC10:11
*** zhurong has quit IRC10:11
*** sree has quit IRC10:11
*** sahid has quit IRC10:12
jianghuawbauzas, resizing works well from my side.10:14
*** lpetrut has joined #openstack-nova10:14
jianghuawXAPI handles more things beyond the virt driver than libvirt.10:15
*** itlinux has joined #openstack-nova10:15
bauzasjianghuaw: so XenAPI persists which vGPUs are used ?10:16
jianghuawbauzas, XAPI handle it. it has records in Db.10:16
jianghuawSo nova needn't worry about status.10:17
jianghuaw:-)10:17
bauzask10:17
*** alexchadin has joined #openstack-nova10:19
jianghuawbauzas, actually you need handle more things which may be more suitable be handled by hypervisor.10:21
*** sdague has joined #openstack-nova10:21
jianghuawAnyway you're super bauzas:-)10:21
openstackgerritMerged openstack/osc-placement master: CLI for usages  https://review.openstack.org/45753510:23
*** Eran_Kuris has joined #openstack-nova10:23
Spazmoticsweet sweet state.db10:24
Spazmoticpeppers all planted.. hopefully none of them will kill me10:24
*** dikonoo has quit IRC10:26
SpazmoticThanks bob, if you're in here ;)10:29
*** zhurong has joined #openstack-nova10:30
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest  https://review.openstack.org/53381810:30
openstackgerritSylvain Bauza proposed openstack/nova master: Avoid suspending guest with attached vGPUs  https://review.openstack.org/53569310:30
*** ralonsoh_ has joined #openstack-nova10:31
*** stakeda has quit IRC10:31
bauzasjianghuaw: just provided all the comments in the relnote $10:31
bauzas^10:31
*** sambetts_ is now known as sambetts10:32
*** abhishekk has quit IRC10:32
bauzasgibi: you were +2 on https://review.openstack.org/53381810:32
*** Tom-Tom_ has quit IRC10:32
bauzasgibi: I actually just modified the relnote+ rebased10:33
kashyapmdbooth: I'm still duking aroud the XML generation locally.  As there 4 different classes I have to navigate:10:33
kashyap     - LibvirtConfigCPUFeature - base class for defining CPU features10:33
kashyap     - LibvirtConfigCPU - base class for defining CPU models10:33
kashyap     - LibvirtConfigGuestCPUFeature - extension for setting the guest10:33
kashyap       specific feature policy10:33
kashyap     - LibvirtConfigGuestCPU - extension for setting the guest specific10:33
kashyap       match policy, and allowing use of host CPU model passthrough10:33
* kashyap just wanted to note that he hasn't dropped working on that. 10:33
jianghuawbauzas, sure. I will look at the patches.10:33
kashyapContext: It was in reference to https://review.openstack.org/#/c/534384/10:34
mdboothkashyap: Yep, guessed.10:34
*** ralonsoh has quit IRC10:34
mdboothkashyap: Incidentally, it sounded from that ml discussion that we might want to propose adding PCID automatically in Nova if it's not present in the explicit cpu model?10:35
*** lucas-afk is now known as lucasagomes10:35
*** diga has joined #openstack-nova10:36
mdboothOr were we still erring towards triple-o?10:36
kashyapmdbooth: It's the latter.10:36
mdboothI guess the advantage of doing it in Nova is that you can query libvirt to expand the cpu flags from the specified model10:36
kashyapmdbooth: Because, adding it automatically indeed is a bit too magic10:36
mdboothi.e. You don't need to duplicate the list10:37
* mdbooth wonders if ibrs is around for the long haul10:37
gibibauzas: I put back my +210:37
kashyapYes, it should be.10:37
*** gouthamr_ has joined #openstack-nova10:38
*** sidx64 has quit IRC10:39
kashyapmdbooth: Do you have a few more minutes to confirm I'm reading code correctly?  /me begins to type10:39
*** mdnadeem has quit IRC10:39
bauzasgibi: <310:39
kashyapmdbooth: So if you look at the class LibvirtConfigGuestCPUFeature() in libvirt/config.py.  The infrastrucutre already _exists_10:40
kashyapTo generate something like:10:40
kashyap    <cpu match='exact'>10:40
kashyap      <model fallback='forbid'>IvyBridge</model>10:40
kashyap      <vendor>Intel</vendor>10:40
kashyap      <feature policy='require' name='pcid'/>10:40
kashyap    </cpu>10:40
kashyapmdbooth: Also see corresponding test: LibvirtConfigGuestCPUFeatureTest() in test_libvirt_config.py.10:41
*** gouthamr has quit IRC10:41
kashyapSo since I'm introducing a config variable, I need to generate the objects for LibvirtConfigGuestCPUFeature() -- for every CPU flag configured in nova.conf10:42
* kashyap goes back to tinkering10:42
mdboothkashyap: Yep10:43
ameedaWARNING: when I pull the last version of nova "DevStack" , the system doesn't work anymore .10:45
kashyapmdbooth: Okido.  So my change here is wrong -- https://review.openstack.org/#/c/534384/2/nova/virt/libvirt/config.py.  Because the root element 'feature' is already handled in the existing class LibvirtConfigCPUFeature()10:45
mdboothkashyap: Yeah, that whole bit of your patch looked confused, but as it was a WIP and you hadn't written any tests yet I figured you'd get there on your own.10:45
* kashyap nods; sorry if I wasted review time10:46
mdboothNot at all. I've learned loads.10:46
*** itlinux has quit IRC10:47
mdboothThe motivation is much more interesting than some libvirt config wrangling code.10:47
*** sidx64 has joined #openstack-nova10:49
kashyapmdbooth: Hmm.  I'm wondering if I should make a single class that will handle the complete root element:10:51
kashyap    <feature policy='require' name='pcid'/>10:51
kashyap   <feature policy='require' name='mtrr'/>10:51
kashyap  ...10:51
kashyapBecause, for now, the 'name' attribute is handled in the class LibvirtConfigCPUFeature()10:52
kashyapAnd the 'policy' in LibvirtConfigGuestCPUFeature().10:52
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for encrypted volumes  https://review.openstack.org/52395810:52
mdboothkashyap: LibvirtConfigGuestCPUFeature already does that, no?10:52
mdboothlyarwood: Looking :)10:53
kashyapmdbooth: This is what it does currently:10:53
kashyap    486 class LibvirtConfigGuestCPUFeature(LibvirtConfigCPUFeature):10:53
kashyap    48710:53
kashyap    488     def __init__(self, name=None, **kwargs):10:53
kashyap    489         super(LibvirtConfigGuestCPUFeature, self).__init__(name, **kwargs)10:53
kashyap    49010:53
*** itlinux has joined #openstack-nova10:53
kashyap    491         self.policy = "require"10:53
*** jaosorior has joined #openstack-nova10:53
kashyap    49210:53
kashyap    493     def format_dom(self):10:53
kashyap    494         ft = super(LibvirtConfigGuestCPUFeature, self).format_dom()10:53
kashyap    49510:53
kashyap    496         ft.set("policy", self.policy)10:53
kashyap    49710:53
kashyap    498         return ft10:53
kashyapOkay, it inherits from the LibvirtConfigCPUFeature()...10:54
*** ralonsoh__ has joined #openstack-nova10:58
*** gszasz has quit IRC10:59
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Add request_id field to migrations object  https://review.openstack.org/53676511:01
*** ralonsoh_ has quit IRC11:01
SpazmoticI need to maek my first nova-spec.. need to read up on how to do that.11:02
*** mvk has quit IRC11:02
*** annp has quit IRC11:03
*** AlexeyAbashkin has quit IRC11:06
Spazmoticsounds like a job for tomorrowwwwwman11:06
*** itlinux has quit IRC11:07
*** gouthamr_ has quit IRC11:09
openstackgerritLee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes  https://review.openstack.org/53635011:09
cdentgibi, bauzas: this would be a useful debugging improvement to have for the post-freeze cleanup: https://review.openstack.org/#/c/519462/ Kenichi has a concern about a private member, but I think that's just the way it goes for now. Already has +2 from stephenfin11:12
*** bkopilov has quit IRC11:12
*** tuanla____ has quit IRC11:12
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status  https://review.openstack.org/53672211:14
*** mvk has joined #openstack-nova11:15
kashyapstephenfin or anyone: Do we have a precendence in Nova where we allow comma-separated values for a config attribute?11:17
stephenfinkashyap: Look at oslo_config.types11:17
stephenfinI'm guessing ListOpt is what you want11:17
*** dikonoo has joined #openstack-nova11:18
stephenfinI _think_ ListOpt uses CSV, but it should be easy enough to verify (look for existing usages in the nova tree)11:18
kashyapstephenfin: I see, will look once I make quick lunch11:19
kashyapstephenfin: So there is some precedence:11:19
kashyap$> git grep ListOpt nova/conf/libvirt.py11:19
kashyapnova/conf/libvirt.py:    cfg.ListOpt('disk_cachemodes',11:19
kashyapnova/conf/libvirt.py:    cfg.ListOpt('hw_machine_type',11:19
kashyapnova/conf/libvirt.py:    cfg.ListOpt('uid_maps',11:19
kashyapnova/conf/libvirt.py:    cfg.ListOpt('gid_maps',11:19
kashyapnova/conf/libvirt.py:    cfg.ListOpt('enabled_perf_events',11:19
kashyapnova/conf/libvirt.py:    cfg.ListOpt('vzstorage_mount_opts',11:19
stephenfinkashyap: Then you're golden. That's the approach I'd take11:20
stephenfinOffloads all the hard work (parsing) to oslo.config11:21
*** hiro-kobayashi has quit IRC11:22
kashyapI see, noted.11:22
kashyapstephenfin: So, what I'm looking to do is something like:11:22
kashyapIf you specify:11:23
kashyap    [libvirt]11:23
kashyap    cpu_model=IvyBridge11:23
kashyap    cpu_model_extra_flags="+pcid,-mtrr,pdpe1gb"11:23
kashyapWhich means: Add the PCID feature and 'pdpel1gb' feature (assume '+' is11:23
kashyapgiven if it's not specified), and *remove* the 'mtrr' feature.11:23
kashyapThen Nova should be able to generate an XML something like:11:23
*** AlexeyAbashkin has joined #openstack-nova11:23
kashyap     <cpu match='exact'>11:23
kashyap       <model fallback='forbid'>IvyBridge</model>11:23
kashyap       <vendor>Intel</vendor>11:23
kashyap       <feature policy='require' name='pcid'/>11:23
kashyap       <feature policy='require' name='pdpe1gb'/>11:23
kashyap     </cpu>11:23
kashyapstephenfin: Makes sense?11:23
stephenfinSounds sane. Why would you remove something though?11:24
kashyapThe +/- in nova.conf will be optional.  If nothing is specified, assume '+'.11:24
stephenfinI assume 'mtrr' would be added by default if you didn't explicitly add '-mtrr'?11:24
kashyapstephenfin: If an advanced operator wants to generate a baseline CPU config, and a certain CPU flag is causing performance issue or other nuisance11:25
kashyapAnd if she isolated the problem to _that_ specific CPU flag11:25
kashyapThens she may want to remove it.11:25
kashyapReasonable?11:25
kashyapstephenfin: Yep, your assumption about 'mtrr' is true.11:26
*** phuongnh has quit IRC11:26
stephenfinYum, that sounds reasonable to me11:26
stephenfin*Yup11:27
stephenfin:)11:27
kashyapI.e. if you specified: 'mtrr', then it will be '+mtrr'.   (Additional note: some named libvirt CPU models _already_ included 'mtrr')11:27
kashyapHeh, noted.11:27
kashyapThanks11:27
kashyapNow to figure out the XML spaghetti11:27
*** tiendc has quit IRC11:31
*** sidx64 has quit IRC11:32
*** gongysh has quit IRC11:36
*** diga has quit IRC11:37
*** sahid has joined #openstack-nova11:38
*** trinaths has left #openstack-nova11:43
*** ratailor__ has quit IRC11:44
*** itlinux has joined #openstack-nova11:45
*** salv-orlando has quit IRC11:48
*** sree has joined #openstack-nova11:48
*** salv-orlando has joined #openstack-nova11:48
*** gouthamr has joined #openstack-nova11:49
*** Guest78703 has quit IRC11:49
*** dtantsur is now known as dtantsur|brb11:50
*** tesseract-RH has quit IRC11:52
*** gouthamr_ has joined #openstack-nova11:52
*** salv-orlando has quit IRC11:53
*** mvenesio has joined #openstack-nova11:54
*** gouthamr has quit IRC11:55
*** jcosmao has left #openstack-nova11:57
*** jcosmao has joined #openstack-nova11:58
*** takedakn has joined #openstack-nova11:59
*** markvoelker has joined #openstack-nova12:00
*** alexchadin has quit IRC12:00
openstackgerritChris Dent proposed openstack/nova master: Fix nits in allocation candidate limit handling  https://review.openstack.org/53678412:00
cdentstephenfin: that ^ gets some cleanups you asked for12:00
ameedaalex_xu : can you please check this for me ? https://review.openstack.org/#/c/526900/12:02
*** liuzz has quit IRC12:03
*** gszasz has joined #openstack-nova12:05
*** zhurong has quit IRC12:05
*** takedakn has quit IRC12:07
*** jaypipes has joined #openstack-nova12:14
*** alexchadin has joined #openstack-nova12:16
*** smatzek has joined #openstack-nova12:19
*** lpetrut has quit IRC12:19
*** gouthamr_ has quit IRC12:19
*** nicolasbock has joined #openstack-nova12:22
*** dtantsur|brb is now known as dtantsur12:23
*** tetsuro_ has quit IRC12:29
*** jpena is now known as jpena|lunch12:33
*** markvoelker has quit IRC12:33
*** yamahata has joined #openstack-nova12:34
*** alexchadin has quit IRC12:36
*** alexchadin has joined #openstack-nova12:36
openstackgerritint32bit proposed openstack/nova master: Set server status to ERROR if rebuild failed  https://review.openstack.org/53626812:39
openstackgerritint32bit proposed openstack/nova master: Refix disk size during live migration with disk over-commit  https://review.openstack.org/53635112:43
*** kwathore__ has joined #openstack-nova12:43
*** kwathore_ has quit IRC12:46
*** alexchadin has quit IRC12:46
*** tesseract-RH has joined #openstack-nova12:47
*** salv-orlando has joined #openstack-nova12:49
ilyashakhathi! may I have cores attention to https://review.openstack.org/#/c/519664/ ? the patch is approved, but Zuul did not merge it12:49
ilyashakhatmost probably needs another +W to trigger the gate12:49
*** yangyapeng has quit IRC12:50
*** yangyapeng has joined #openstack-nova12:51
alex_xuilyashakhat: done12:51
openstackgerritLee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes  https://review.openstack.org/53635012:51
ilyashakhatalex_xu: thanks!12:51
*** psachin has quit IRC12:51
*** alexchadin has joined #openstack-nova12:52
alex_xuameeda: sorry, really busy on freeze, will get chance after freeze12:52
alex_xuilyashakhat: np12:52
*** salv-orlando has quit IRC12:54
ameedaalex_xu: np :)12:54
*** tesseract-RH has quit IRC12:54
*** alexchadin has quit IRC12:54
*** yangyapeng has quit IRC12:55
*** tesseract has joined #openstack-nova12:55
*** ralonsoh__ is now known as ralonsoh12:55
*** yassine has joined #openstack-nova12:57
*** zhurong has joined #openstack-nova12:57
*** yassine is now known as Guest8701112:57
*** sidx64 has joined #openstack-nova12:58
*** alexchadin has joined #openstack-nova13:00
*** sree has quit IRC13:04
*** yamahata has quit IRC13:04
*** sree has joined #openstack-nova13:04
*** bkopilov has joined #openstack-nova13:08
*** sree has quit IRC13:09
*** eharney has joined #openstack-nova13:11
*** sdague has quit IRC13:11
*** zhurong has quit IRC13:14
*** bhujay has quit IRC13:16
*** amoralej is now known as amoralej|lunch13:16
*** psachin has joined #openstack-nova13:17
openstackgerritsahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes  https://review.openstack.org/48499713:17
*** liverpooler has joined #openstack-nova13:19
*** Tom-Tom has joined #openstack-nova13:19
*** sidx64 has quit IRC13:20
*** gouthamr has joined #openstack-nova13:21
*** sidx64 has joined #openstack-nova13:21
*** dims has quit IRC13:22
*** pchavva has joined #openstack-nova13:24
*** alex_xu has quit IRC13:24
*** dims has joined #openstack-nova13:25
*** yangyapeng has joined #openstack-nova13:26
*** alex_xu has joined #openstack-nova13:27
*** sree has joined #openstack-nova13:28
lyarwoodmdbooth: *facepalm* how did I screw up that if volume_id block *sigh*13:29
*** jpena|lunch is now known as jpena13:29
mdboothlyarwood: Hehe, it totally doesn't matter.13:29
*** markvoelker has joined #openstack-nova13:31
*** gouthamr has quit IRC13:35
*** sree has quit IRC13:36
*** kwathore_ has joined #openstack-nova13:38
*** esberglu_ has quit IRC13:39
* mdbooth sighs. *Another* merge conflict in my local disk serial numbers series :(13:40
mdboothlyarwood: Bet ^^^ is your fault :P13:41
*** kwathore__ has quit IRC13:42
lyarwoodmdbooth: you're welcome! :)13:42
lyarwoodmdbooth: pretty sure your swap volume test removal patch also caused a conflict this morning13:42
lyarwoodmdbooth: so it works both ways :)13:42
SpazmoticThis I will say is the one benefit from working the XenAPI drivers13:43
mdboothIndeed.13:43
SpazmoticNo merge conflicts :p13:43
mdboothSpazmotic: And also the curse :) How do you find getting core reviews on it?13:43
SpazmoticNot sure if that burn was incidental or on purpose hehehehe13:43
* Spazmotic needs aloe 13:43
* mdbooth used to have terrible trouble in the vmware driver.13:44
SpazmoticMy last and first commit ac tually went through pretty quick, but that was back in Liberty13:44
SpazmoticNow that i'm trying to get more active, struggling ot get any love with Xenapi, hence why i'm trying to learn placement13:44
bauzascdent: all cool13:45
bauzasreviewing https://review.openstack.org/#/c/519462/13:45
*** jaypipes has quit IRC13:46
mdboothSpazmotic: It's a perennial problem that it's harder to get core reviews in areas of the codebase cores are less familiar with. Doesn't even help to become a core, because you can't core review your own stuff :/13:47
SpazmoticHahah nah I certainly understand.. I know XenAPI isn't the hotness right now, so i try to be patient and wait :)13:47
*** edmondsw has joined #openstack-nova13:49
cdentefried_back_wed: you're not supposed to be here. I haven't rechecked the things you just rechecked yet to try to make less noise while the stuff before it is still being chewed on13:49
*** salv-orlando has joined #openstack-nova13:50
efried_back_wedcdent It's a toss-up whether it'll be slower to check them serially (and have to wait for N to merge before rechecking N+1) or keep 'em all in the pipe in parallel so that maybe, just maybe, more than one will merge at once.13:50
efried_back_wedcdent And I promise, I'll bugger off to my appointment in a few minutes.13:51
cdentefried_back_wed: given that most of the failures are timeouts I tend to not want to parellelize13:51
cdentwhich is probably illogical since something else will just take the spot, but meh13:52
efried_back_wedcdent It had occurred to me to ask the infra folks whether we could increase whatever timeouts those are.  I imagine they've already weighed the pros and cons of that, though.13:52
cdent'spect so13:52
cdentthis is going to merge very soon: https://review.openstack.org/#/c/526540/13:52
*** salv-orlando has quit IRC13:55
efried_back_wedNow you've jinxed it13:57
*** esberglu has joined #openstack-nova13:58
SpazmoticFeels good to have a +1 on my code finally, and from someone at citrix with that.. makes me breathe easier hehe14:02
*** markvoelker has quit IRC14:04
*** kwathore_ has quit IRC14:05
*** links has quit IRC14:05
ildikov#success OpenStack now supports to attach a Cinder volume to multiple VM instances managed by Nova.14:05
openstackstatusildikov: Added success to Success page14:05
ildikovMiracles exist when we have amazing and dedicated people to make them happen. :)14:05
*** lyan has joined #openstack-nova14:05
gibiildikov: \o/14:06
rgerganovefried_back_wed, no luck trying to register a nested RP with the provider_tree14:06
ildikovgibi: :)14:06
efried_back_wedrgerganov Okay; leave some words on the patch, or send me an email, and I can dig in some more tomorrow (or maybe for a few minutes tonight)14:07
*** kwathore has joined #openstack-nova14:07
rgerganovefried_back_wed, sure, I am digging into this right now, will post all findings that I have at the end of the day14:07
efried_back_wedrgerganov ++14:07
openstackgerritRadoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree()  https://review.openstack.org/53634814:08
*** abhishekk has joined #openstack-nova14:08
openstackgerritSylvain Bauza proposed openstack/nova master: Avoid suspending guest with attached vGPUs  https://review.openstack.org/53569314:09
*** psachin has quit IRC14:09
mriedemandreykurilin: can you take a look at the novaclient change to support 2.59 https://review.openstack.org/#/c/516168/ and the small patch after it for 2.60?14:10
andreykurilinlooking14:11
*** dave-mcc_ has joined #openstack-nova14:12
*** melwitt has quit IRC14:12
cdentrgerganov: is your code that's not working viewable somewhere?14:12
rgerganovcdent: https://review.openstack.org/#/c/536348/314:13
cdentoh never mind, actually read the log14:13
cdentthanks14:13
mriedemavolkov: i had some questions in the osc-placement patch for aggregates, which is the first one to add microversion support https://review.openstack.org/#/c/505643/14:14
*** melwitt has joined #openstack-nova14:14
mriedemavolkov: i'm not sure how to expose in the help output that --os-placement-api-version 1.1 needs to be used14:14
mriedembesides just adding it directly into the help14:14
*** lucasagomes is now known as lucas-hungry14:15
*** moshele has quit IRC14:17
Roamer`mriedem, sorry if I'm starting to annoy you, I guess you didn't have time yesterday to take a look at 140733 - the StorPool driver?  It's just that it seems that there might not be much time for us to make changes if something should appear to be not quite right...  and yes, I do realize that you have a lot to look at, and an exception for a new driver is kind of low priority, I know14:18
*** crushil has joined #openstack-nova14:18
jrollmriedem: you didn't secretly fix the segfault thing yesterday, did you? seems to be passing now.14:19
*** smatzek has quit IRC14:20
openstackgerritAlex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs  https://review.openstack.org/53608514:21
cdentrgerganov: two things: a) what kind of error are you getting with regard to "no luck", b) if you're putting some vcpus (and other inventory classes)  in resource pools child providers, then they can't also be in the parent. it appears that's what your code is currently doing?14:22
alex_xugibi: I added the release note, thanks for the review14:22
*** salv-orlando has joined #openstack-nova14:22
alex_xubauzas: efried_back_wed I will address the nits tommorow, thanks for the review14:22
*** dklyle has quit IRC14:22
alex_xugibi: also the functional test!14:22
gibialex_xu: I'm totally OK to have the functional test in a separate followup14:23
rgerganovcdent, you mean that we can't have VCPU in both parent and childs?14:23
mriedemjroll: nope14:23
alex_xugibi: thanks, will try to submit a separate followup tomorrow14:24
jrollyay14:24
cdentrgerganov: it can be in both places, but they cannot overlap. Any inventory of the same type associated with the child provider needs to be subtracted from the parent.14:24
mriedemRoamer`: i still have the tab open14:25
rgerganovcdent, this is not the problem that I have now14:25
rgerganovcdent, when you start nova-compute udpate_provider_tree is being called twice14:25
cdentrgerganov: yeah, I get that, was just noting it for future reference14:25
cdentfirst in init and then in the periodic job?14:26
*** edand has quit IRC14:26
rgerganovcdent, both calls come from _update_available_resource in the RT14:26
*** rgerganov has left #openstack-nova14:27
* cdent nods14:27
*** rgerganov has joined #openstack-nova14:27
* cdent nods again14:27
rgerganovcdent, on the first call the provider tree seems to be correct and both parent and child RP are updated in placement14:28
rgerganovcdent, on the second call I get 409 from placement14:28
gibialex_xu: I'm +2 on the last patch too, thanks for the reno14:28
alex_xugibi: thanks for the review!14:28
rgerganovcdent, because it tries to create the nested RP again14:29
cdentrgerganov: how close together are those jobs, by timestamp?14:29
rgerganovcdent, they are far away because I am tracing the code14:30
rgerganovcdent, like couple of seconds away14:30
cdentIt seems like the provider tree isn't getting a chance to update its cache of the rp generation14:30
rgerganovcdent, correct14:31
cdentIt behaves the same way with the debugger not involved?14:31
rgerganovyes14:31
rgerganovon the first call the provider tree has both parent and child and the call to placement succeds14:32
*** vladikr has joined #openstack-nova14:32
rgerganovon the second call it has only the parent and when the driver adds the child and placement is called, we get 40914:32
*** sidx64 has quit IRC14:33
cdentrgerganov: and the message with the 409 is what?14:34
*** sidx64 has joined #openstack-nova14:35
*** sidx64 has quit IRC14:35
rgerganovcdent, Failed to create resource provider record in placement API for UUID 4be176f1-b09f-48dd-82d7-d9c1c86fb68b. Got 409: {"errors": [{"status": 409, "request_id": "req-69d31b34-bc50-4178-a569-ebaef4734cbd", "detail": "There was a conflict when trying to complete your request.\n\n Conflicting resource provider name: vdc already exists.  ", "title": "Conflict"}]}.14:35
cdentrgerganov: thanks, looking at the calling code now14:37
*** mvk has quit IRC14:37
*** eharney has quit IRC14:37
*** crushil has quit IRC14:38
*** markvoelker has joined #openstack-nova14:41
*** edand has joined #openstack-nova14:42
*** cleong has joined #openstack-nova14:42
*** bhujay has joined #openstack-nova14:43
edmondswgibi tx for reviewing the powervm SEA patch! Are you planning to look at the powervm vscsi patch as well?14:44
cdentrgerganov: on the second request, when the provider tree doesnt have the child, have you traced what's happening in _ensure_resource_provider. It seems like in there is where it could be going awry and not creating a full tree14:45
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs object  https://review.openstack.org/48940814:45
*** tbachman has joined #openstack-nova14:45
gibiedmondsw: I cannot promise that it will fit before the feature freeze14:46
*** alexchadin has quit IRC14:46
edmondswgibi ok. If you get to it, thanks. If not, maybe someone else will14:47
*** fragatina has joined #openstack-nova14:47
gibiedmondsw: please ping me daily, that will help me not to forget14:48
*** ralonsoh_ has joined #openstack-nova14:48
edmondswgibi you asked for it... ;)14:49
edmondswthanks14:49
*** bhujay has quit IRC14:49
edmondswstephenfin do you also want daily reminders? ;)14:50
gibi:)14:50
*** ralonsoh has quit IRC14:51
stephenfinedmondsw: Nope, it'll be done :)14:51
* stephenfin is working through this https://bugzilla.redhat.com/show_bug.cgi?id=153466914:51
openstackbugzilla.redhat.com bug 1534669 in openstack-nova "emulator_threads_policy needs improvement when hyper threading is enabled" [Medium,New] - Assigned to sfinucan14:51
edmondswstephenfin thanks!14:51
*** bhujay has joined #openstack-nova14:51
* stephenfin should probably create a Launchpad bug for that14:51
rgerganovcdent, correct, this method doesn't fetch the full tree on the second call14:51
rgerganovcdent, self._refresh_associations doesn't do what we expect14:52
*** hongbin has joined #openstack-nova14:52
*** rgerganov is now known as rgerganov|away14:53
*** dave-mcc_ has quit IRC14:55
*** amoralej|lunch is now known as amoralej14:55
edmondswbauzas johnthetubaguy any chance you could look at https://review.openstack.org/#/c/526094/ adding powervm cinder volume support?14:58
*** matrohon has quit IRC15:01
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620415:02
mriedemmelwitt: want to hit this 3 LOC patch for 2.60 support in novaclient? https://review.openstack.org/#/c/536621/15:03
*** claudiub has joined #openstack-nova15:05
*** dave-mccowan has joined #openstack-nova15:05
*** slaweq has quit IRC15:06
*** slaweq has joined #openstack-nova15:06
*** mdnadeem has joined #openstack-nova15:06
*** smatzek has joined #openstack-nova15:07
*** awaugama has joined #openstack-nova15:07
*** bauzas_ has joined #openstack-nova15:08
*** claudiub|2 has quit IRC15:08
*** amodi has joined #openstack-nova15:09
*** bauzas_ has quit IRC15:09
melwittmriedem: got it15:09
bauzasmriedem: I can look15:09
bauzasheh, snap15:09
mriedemthanks, that'll likely be it for queens15:10
*** slaweq has quit IRC15:10
*** gaoyan has joined #openstack-nova15:13
*** jackie-truong has joined #openstack-nova15:13
*** janki has quit IRC15:14
*** gaoyan has quit IRC15:14
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement  https://review.openstack.org/53685815:14
openstackgerritint32bit proposed openstack/nova master: Set server status to ERROR if rebuild failed  https://review.openstack.org/53626815:16
*** ralonsoh__ has joined #openstack-nova15:17
*** moshele has joined #openstack-nova15:18
*** bauzas_ has joined #openstack-nova15:18
*** itlinux has quit IRC15:19
*** mlavalle has joined #openstack-nova15:20
*** dikonoo has quit IRC15:20
*** lucas-hungry is now known as lucasagomes15:20
*** ralonsoh_ has quit IRC15:20
*** links has joined #openstack-nova15:21
*** bauzas_ has quit IRC15:22
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates  https://review.openstack.org/50039615:22
*** bauzas has quit IRC15:23
*** esberglu_ has joined #openstack-nova15:23
*** bauzas has joined #openstack-nova15:24
*** esbergl__ has joined #openstack-nova15:24
*** moshele has quit IRC15:24
*** armax has joined #openstack-nova15:25
*** felipemonteiro has joined #openstack-nova15:25
alex_xumriedem: in case you have time https://review.openstack.org/#/q/topic:bp/request-traits-in-nova+(status:open+OR+status:merged), I have last two patches waiting for one more +2, appreciate if you can help it15:25
*** esberglu has quit IRC15:26
*** lajoskatona has quit IRC15:27
*** eharney has joined #openstack-nova15:27
mriedemalex_xu: ok, also looks like https://review.openstack.org/#/c/536083/4/nova/scheduler/client/report.py overlaps with something dansmith was working on15:27
mriedemto use the 1.12 allocation candidates format for the move operations in the report client15:28
*** esberglu_ has quit IRC15:28
dansmithmriedem: I think that patch just has two remaining failures to look at but I've been kinda absorbed with other things lately15:29
melwittmriedem: on the set status to ERROR upon rebuild failure patch, do you think it would be worth adding a fault message to the instance along with the ERROR state? I was thinking since the instance is sitting with the wrong imageref, it might be helpful to give some guidance in the fault message in case someone doesn't know how to recover https://review.openstack.org/#/c/536268/15:31
*** esberglu has joined #openstack-nova15:31
*** tidwellr has quit IRC15:32
sean-k-mooneystephenfin: for the record that refactor is annoying. i have all but 1 test passing now howerver the pci request spec object has like 8 required field not just the 4 in the alias schema definition15:32
mriedemyeah a fault is probably necessary, i thought we'd have one on there but apparently we only use wrap_instance_fault in the compute manager15:32
stephenfinsean-k-mooney: Really? What are the other ones?15:33
*** suresh12 has joined #openstack-nova15:33
dansmithmelwitt: so can you push up a change to add a fault so we can get moving on that?15:33
sean-k-mooneystephenfin: http://paste.openstack.org/show/650978/15:34
*** corvus has quit IRC15:34
melwittdansmith: yep, let me do that15:34
sean-k-mooneystephenfin: the commented out ones should be deleted15:34
*** esbergl__ has quit IRC15:35
dansmithmelwitt: cool15:35
sean-k-mooneystephenfin: also i found that some test use device_id instead of product id so i have to support both and while the alias uses device_type the db uses dev_type so i have to convert when i create the spec15:35
*** damini has joined #openstack-nova15:36
stephenfinsean-k-mooney: You can probably drop capability_type15:37
sean-k-mooneystephenfin: anyway when i get the last test passing ill push up the new revision.15:37
stephenfinPer my comment in nova/pci/request.py15:37
stephenfinBut otherwise, yeah, I figured it wouldn't be super easy but it is a much better design long-term, IMO15:37
sean-k-mooneystephenfin: nope you will get an invalid key exception unless i specificlally add code to drop it15:37
stephenfinsean-k-mooney: I'm working through the CPU pinning code again. Every time I go through this, I have to relearn it15:38
stephenfinsean-k-mooney: But nothing except tests use it15:38
stephenfin*set it15:38
stephenfinand we could just filter it out of any '[pci]spec' options with a warning that it must be manually removed15:39
damini111Pls can anyone tell me how to write tempest test for shelve instance and unshelve .I am new to openstack.15:39
sean-k-mooneyhaha well now you mention it the code that is failing is in  consume_requests a pop from an empty list.15:39
sean-k-mooneystephenfin: ya its easy to ignore it and log a message the way i have it set up instead of a exception15:40
*** david-lyle has joined #openstack-nova15:40
*** slaweq has joined #openstack-nova15:40
stephenfindamini: Tempest has a helper function for that. You might want to start there https://github.com/openstack/tempest/blob/master/tempest/common/compute.py#L26315:41
*** jeblair has joined #openstack-nova15:41
*** jeblair is now known as corvus15:42
stephenfinsean-k-mooney: (y) that would be my recommendation15:42
alex_xudansmith: do you have the link? I didn't found in the gerrit15:42
stephenfinNo point adding something we're going to immediately deprecate15:42
dansmithalex_xu: link for what?15:42
alex_xudansmith: the 1.12 which mriedem mentioned, I guess you hit 1.12 by this patch https://review.openstack.org/#/c/531517/215:43
mriedemyeah that's the one15:43
*** damini has quit IRC15:44
*** slaweq has quit IRC15:44
alex_xumriedem: ah, I got it, dansmith already said he is working on other things. sorry, i misunderstand few words previously15:46
mriedemnp, i'm just pointing out you were both working on part of the same problem i think15:46
*** nicolasbock has quit IRC15:47
dansmithalex_xu: why are you only bumping to 1.12 instead of to current?15:47
dansmithalex_xu: because we're just going to have to bump right after this one15:47
alex_xudansmith: I have followup patch to support traits https://review.openstack.org/536085, which is 1.1715:47
dansmithalex_xu: okay seems like you could just start calling at 1.17 in the first patch, but okay15:48
dansmithI guess I'll rebase mine on your last change15:49
*** gaoyan has joined #openstack-nova15:50
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Address comments from original inventory patch  https://review.openstack.org/52157815:50
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Usage docs and initial release note for osc-placement  https://review.openstack.org/53685815:50
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add missing runtime requirements  https://review.openstack.org/53687015:50
alex_xudansmith: ok, thanks15:50
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Address review comments from allocations patch  https://review.openstack.org/53687115:50
*** gaoyan has quit IRC15:52
*** slaweq has joined #openstack-nova15:53
*** suresh12 has quit IRC15:55
*** Eran_Kuris has quit IRC15:55
*** gszasz has quit IRC15:58
*** r-daneel has joined #openstack-nova15:59
openstackgerritMerged openstack/nova master: Add reference to policy sample  https://review.openstack.org/53171316:00
*** abhishekk has quit IRC16:00
openstackgerritMerged openstack/nova master: Raise on API errors getting aggregates/traits  https://review.openstack.org/52654016:00
openstackgerritMerged openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination  https://review.openstack.org/51616816:01
openstackgerritmelanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed  https://review.openstack.org/53626816:02
*** openstackgerrit has quit IRC16:03
*** openstackgerrit has joined #openstack-nova16:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI for aggregates (v1.1)  https://review.openstack.org/50564316:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI for resource classes (v1.2)  https://review.openstack.org/51118216:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4)  https://review.openstack.org/51118316:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5)  https://review.openstack.org/51464216:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6)  https://review.openstack.org/51464316:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7)  https://review.openstack.org/51464416:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9)  https://review.openstack.org/51464616:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10)  https://review.openstack.org/51464716:04
openstackgerritMatt Riedemann proposed openstack/osc-placement master: [WIP] Get resource provider by uuid or name  https://review.openstack.org/52779116:04
*** janki has joined #openstack-nova16:05
bauzassorry folks, I just migrated my IRC bouncer16:07
bauzasin case some pinged me, lemme know16:08
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates  https://review.openstack.org/50039616:09
*** zhaochao has quit IRC16:10
openstackgerritMatthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix  https://review.openstack.org/52634616:10
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634716:10
openstackgerritMatthew Booth proposed openstack/nova master: Expose driver_block_device fields consistently  https://review.openstack.org/52836216:10
openstackgerritMatthew Booth proposed openstack/nova master: Add local_root to block_device_info  https://review.openstack.org/52902916:10
openstackgerritMatthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume  https://review.openstack.org/52836316:10
openstackgerritMatthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info  https://review.openstack.org/52932816:10
openstackgerritMatthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info  https://review.openstack.org/52932916:10
openstackgerritMatthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config  https://review.openstack.org/53078616:10
openstackgerritMatthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume  https://review.openstack.org/53078716:10
*** tssurya has quit IRC16:10
openstackgerritMatthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver  https://review.openstack.org/52938016:10
openstackgerritmelanie witt proposed openstack/nova master: Set server status to ERROR if rebuild failed  https://review.openstack.org/53626816:16
mriedemmelwitt: suggestion in https://review.openstack.org/#/c/536268/16:16
mriedemright before you pushed16:16
mriedemwe can assert the fault is set in the functional test that is also being updated16:17
melwittI think maybe I did what you asked16:17
*** tidwellr has joined #openstack-nova16:17
*** suresh12 has joined #openstack-nova16:17
melwittoh, nope, different func test16:17
*** edand_ has joined #openstack-nova16:17
mriedemah that works too16:17
*** gszasz has joined #openstack-nova16:17
melwittk, lemme add that16:18
mriedemthat's fine,16:18
mriedemthis is better actually because a rebuild can be done for a non-admin and we want to make sure the non-admin can see the fault16:18
mriedemi don't know if we want to say something specific in the fault message about attempting to rebuild with a different image, that could be a slippery slope of assumptions about why we failed16:18
melwittokay, cool16:18
mriedemlike, image might be fine, but some other filter failed because a resource is unavailable or something16:19
melwittyeah, I was thinking about that too16:19
mriedem+216:19
mriedemheh stable/newton is still around16:20
*** edand has quit IRC16:20
melwitt:)16:21
*** tssurya has joined #openstack-nova16:22
*** tesseract has quit IRC16:22
*** suresh12 has quit IRC16:22
mriedemthis is probably not worth yet another newton release before eol though16:22
mriedemtonyb: when you gonna pull that newton-eol trigger?16:22
melwittwah wah16:23
*** sahid has quit IRC16:27
*** edand__ has joined #openstack-nova16:29
*** penick has joined #openstack-nova16:29
*** tidwellr has quit IRC16:29
bauzaswait, Newton is not EOL'd yet ?16:30
bauzaswoah16:30
mriedemmelwitt: want to start on the backports?16:30
melwittmriedem: yes16:30
bauzasmelwitt: mriedem: dansmith: I think we should document a relnote with https://review.openstack.org/#/c/536268/616:30
bauzasI was about to say that we will provide a new behaviour16:31
dansmithbauzas: it's really the behavior that every other action has though16:31
bauzasI'm not asking for a microversion, but we need to explain for operators I think16:31
*** tidwellr has joined #openstack-nova16:31
bauzasdansmith: I know16:31
bauzasdansmith: I'm just thinking that it's a game changer16:31
bauzasand documenting it would be IMHO a small thing for a better point16:31
mriedem*shrug* i thought about a "fixes" release note just to point out that if scheduling fails during rebuild the instance will go to ERROR state and a fault will be recorded16:32
bauzasI just don't want to see rants here :p16:32
bauzasmriedem: exactly my point16:32
bauzasnot more than that16:32
mriedemi'd be fine with that16:32
bauzasso we can point out the relnote if someone is aking that16:32
dansmithmeh.16:32
*** edand_ has quit IRC16:32
bauzasbackporting that means also that we will change the logic16:33
bauzashopefully the stable version will be not a .z one16:33
*** sdague has joined #openstack-nova16:33
mriedemumm16:33
mriedemit's not a new feature16:34
mriedemit's a bug fix16:34
bauzassure, but we're changing the API right?16:34
mriedemwe didn't bump the .y for the change that made the API fail fast if you tried to rebuild a volume-backed instance and change the image16:34
bauzasagain, I'm not asking for a microversion, please16:34
bauzasokay16:34
bauzaswhatever the semver will be, I just think we need to explain that to ops16:34
bauzasos-instance-actions API is admin by default ?16:35
mriedemno16:35
*** bhujay has quit IRC16:35
bauzasI don't remember, I need to look at the default policy16:35
bauzasok, cool then16:35
mriedemos-instance-actions events with the traceback is admin by default16:35
*** matrohon has joined #openstack-nova16:35
bauzashah, right16:36
mriedemi think you can't see the events before some microversion if you're not an admin either16:36
bauzasI can test on my box16:36
mriedemdon't bother: https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id4616:36
bauzascool cool16:37
*** moshele has joined #openstack-nova16:37
openstackgerritMerged openstack/python-novaclient master: Add support for microversion 2.60 - volume multiattach  https://review.openstack.org/53662116:38
bauzasmriedem: dansmith: melwitt: looks like my proposal to add a relnote didn't get a huge consensus, still okay if I'm providing it ?16:39
melwittmriedem, bauzas: so, should I add a fixes relnote to the patch then?16:39
bauzashah, jinxed16:40
mriedemincoming16:40
openstackgerritMatt Riedemann proposed openstack/nova master: Set server status to ERROR if rebuild failed  https://review.openstack.org/53626816:40
bauzaswho's doing that ? I can take my pen if nobody does16:40
bauzasoh, a new rev16:40
bauzasmriedem: on it16:40
bauzasmriedem: +216:41
*** moshele has quit IRC16:41
bauzasI can fast-approve or dansmith can +W16:41
*** pcaruana has quit IRC16:41
mriedemstvnoyes: ildikov: first victim of the multiattach swap volume test http://logs.openstack.org/89/532689/5/check/nova-multiattach/094e46b/job-output.txt.gz#_2018-01-23_12_43_58_79034016:43
*** dmsimard has joined #openstack-nova16:44
mriedemthis is where the swap starts http://logs.openstack.org/89/532689/5/check/nova-multiattach/094e46b/job-output.txt.gz#_2018-01-23_12_43_59_31222616:44
openstackgerritmelanie witt proposed openstack/nova stable/pike: Set server status to ERROR if rebuild failed  https://review.openstack.org/53689716:45
dmsimardmriedem, kashyap: hi o/ It would be nice if we could land these in time for queens: https://review.openstack.org/#/c/524748/ && https://review.openstack.org/#/c/526748/ (and understand why the guestfs.set_backend_settings patch is breaking things when testing it here https://review.openstack.org/#/c/524750/ )16:45
dmsimardI suppose these are stable backport candidates as well16:46
*** wxy has quit IRC16:46
mriedemildikov: stvnoyes: this is on a rax-dfw node and those have been really slow lately, so maybe it's just related to a slow node timeout since the swap volume tests do take awhile16:47
mriedemdmsimard: i can't really say why things don't work with that stuff,16:48
mriedemneed some more libguestfs/qemu people16:48
dmsimardmriedem: I don't know these people, I'm a mere mortal :(16:48
mriedemdmsimard: i think https://review.openstack.org/#/c/526748/ is pretty straight forward though - don't do logging in a thread16:49
mriedemb/c eventlet can explode16:49
*** avolkov has quit IRC16:50
stvnoyesmriedem, ildikov, I finally got my new env working but I am seeing some failures in the 'normal' tempest multiattach tests. I am looking to see why that is. It is likely something unique to my env.  that's what I'm doing atm.16:51
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620416:52
mriedemit's doing the swap here http://logs.openstack.org/89/532689/5/check/nova-multiattach/094e46b/logs/screen-n-cpu.txt.gz#_Jan_23_11_16_02_65413516:55
mriedemsdague: do you ever remember anything about losing a requestid when using an elevated context? or people talking about resolving that?16:56
mriedemi was tracing a swap volume operation through the logs and lose some of the logging once the context gets elevated16:57
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates  https://review.openstack.org/50039616:58
openstackgerritRadoslav Gerganov proposed openstack/nova master: fix _ensure_resource_provider()  https://review.openstack.org/53690216:58
*** sahid has joined #openstack-nova16:59
*** Eran_Kuris has joined #openstack-nova17:01
openstackgerritRadoslav Gerganov proposed openstack/nova master: WIP: VMware: implement update_provider_tree()  https://review.openstack.org/53634817:01
*** tidwellr has quit IRC17:02
*** AlexeyAbashkin has quit IRC17:02
*** tidwellr has joined #openstack-nova17:02
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed  https://review.openstack.org/53690417:03
*** mvenesio has quit IRC17:06
*** mvenesio has joined #openstack-nova17:07
*** chyka has joined #openstack-nova17:07
*** Tom-Tom has quit IRC17:10
openstackgerritChris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports  https://review.openstack.org/53379717:13
*** Guest87011 has quit IRC17:13
*** matrohon has quit IRC17:14
*** mvenesio_ has joined #openstack-nova17:14
*** armax has quit IRC17:14
*** moshele has joined #openstack-nova17:14
*** jackie-truong has quit IRC17:14
gibimriedem: btw, do you need me to run the weekly nova meeting on Thrusday?17:15
mriedemgibi: maybe...17:16
mriedemi'll let you know in a bit17:16
gibimriedem: OK, I can do that if needed17:16
mriedemgibi: i should be able to do it, assuming i can remember :)17:17
*** mvenesio has quit IRC17:17
gibimriedem: OK17:17
*** felipemonteiro has quit IRC17:25
*** gszasz has quit IRC17:29
*** salv-orlando has quit IRC17:32
*** penick has quit IRC17:32
*** salv-orlando has joined #openstack-nova17:32
*** armax has joined #openstack-nova17:33
*** smatzek_ has joined #openstack-nova17:34
*** penick has joined #openstack-nova17:36
artommelwitt, no newton backport for the rebuild imageref fix? There was some confusion whether newton is EOL or not17:36
*** salv-orlando has quit IRC17:37
dansmithartom: it's EOL, just not deleted yet17:38
melwittartom: indeed the stable/newton branch is still around coincidentally. but earlier mriedem said it's probably not worth another newton release for this17:38
dansmithI agree17:39
artomSure, not gonna argue with that17:40
artom(I mean, I probably would, in theory)17:40
artomBut... I'm also realistic17:40
*** salv-orlando has joined #openstack-nova17:45
*** Eran_Kuris has quit IRC17:46
*** sahid has quit IRC17:47
*** gyee has joined #openstack-nova17:47
lyarwoodmelwitt: thanks for your review of https://review.openstack.org/#/c/523958/ btw, are you going to be able to hit it again today?17:55
*** sridharg has quit IRC17:56
melwittlyarwood: yes, will re-review today17:56
lyarwoodmelwitt: cool, thanks17:57
*** derekh has quit IRC18:01
*** gouthamr has joined #openstack-nova18:01
*** ralonsoh__ has quit IRC18:03
*** edand__ has quit IRC18:03
*** david-lyle has quit IRC18:07
*** jackie-truong has joined #openstack-nova18:07
openstackgerritmelanie witt proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest  https://review.openstack.org/53381818:08
*** tidwellr_ has joined #openstack-nova18:13
*** tidwellr has quit IRC18:13
SpazmoticWhew busy day18:22
SpazmoticIf any core's are able to spare some cycles today for a XenAPI review, would love it.  I am getting ready to head to bed but let me know if you have any questions. https://review.openstack.org/#/c/533168/18:25
*** janki has quit IRC18:26
*** links has quit IRC18:26
*** smatzek_ has quit IRC18:27
*** AlexeyAbashkin has joined #openstack-nova18:31
*** dtruong has quit IRC18:34
*** lucasagomes is now known as lucas-afk18:35
*** AlexeyAbashkin has quit IRC18:35
*** mdnadeem has quit IRC18:39
*** links has joined #openstack-nova18:40
ildikovmriedem: stvnoyes: sorry, partially off today18:41
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.61 - Add trusted_image_certificates  https://review.openstack.org/50039618:47
*** fragatina has quit IRC18:48
*** jpena is now known as jpena|off18:48
*** dtantsur is now known as dtantsur|afk18:48
*** fragatina has joined #openstack-nova18:49
*** gszasz has joined #openstack-nova18:50
*** smatzek has quit IRC18:54
mriedemildikov: np, i reported a bug against cinder, seems to be intermittent timing issue anyway18:54
*** cdent has quit IRC18:55
*** smatzek has joined #openstack-nova18:56
*** smatzek_ has joined #openstack-nova18:58
ildikovmriedem: that does not sound good19:00
*** smatzek has quit IRC19:00
ildikovmriedem: the timing issue I mean19:00
ildikovmriedem: Tnx for looking into it and reporting the big19:01
ildikov*bug19:01
*** eharney has quit IRC19:03
*** david-lyle has joined #openstack-nova19:04
*** jackie-truong has quit IRC19:05
*** smatzek_ has quit IRC19:09
*** moshele has quit IRC19:10
*** acormier has joined #openstack-nova19:12
*** liusheng has quit IRC19:14
*** liusheng has joined #openstack-nova19:15
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Make sure that functional test triggered on sample changes  https://review.openstack.org/53693419:16
*** links has quit IRC19:19
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Make sure that functional test triggered on sample changes  https://review.openstack.org/53693619:19
*** felipemonteiro has joined #openstack-nova19:19
*** smatzek has joined #openstack-nova19:23
*** smatzek has quit IRC19:29
*** penick has quit IRC19:29
*** gszasz has quit IRC19:31
*** gszasz has joined #openstack-nova19:31
*** jackie-truong has joined #openstack-nova19:33
*** gouthamr_ has joined #openstack-nova19:34
*** harlowja has joined #openstack-nova19:34
*** gouthamr has quit IRC19:36
mriedemalex_xu: remember to update the placement API reference docs with the new 'required' parameter to GET /allocation_candidates https://review.openstack.org/#/c/535642/19:36
melwittI was looking at this bug and it looks like the auto-disabling of compute can be used to DoS the cloud https://bugs.launchpad.net/nova/+bug/174210219:36
openstackLaunchpad bug 1742102 in OpenStack Compute (nova) "Simple user can disable compute" [Undecided,Confirmed]19:36
mriedemmelwitt: dansmith and i talked with the person that reported it a couple of weeks ago19:37
mriedemproblem is they are getting a super generic error on the nova side (VirtualInterfaceException due to the vif plugging failing)19:37
*** smatzek has joined #openstack-nova19:37
mriedemall that nova gets back from neutron is the vif plug failed event, with no details about why it failed19:37
mriedemso in that case we don't know if it's specific to that host or what19:37
dansmithwhich is normally a thing worth disabling the compute over19:38
mriedemwe could whitelist a set of exceptions that we ignore for the auto-disable, but that gets tricky19:38
melwittthe latest comment demonstrates it can be done via OverQuota, which is something I think we could avoid19:38
*** fragatina has quit IRC19:38
mriedemi think they said it wasn't a quota issue,19:38
mriedembecause we check port quota in the api before casting to compute to build the instance and the port,19:38
*** amoralej is now known as amoralej|off19:38
mriedemthe issue was someting to do with no more fixed IPs in the pool for the network backend19:39
mriedemif there is a way to preemptively know about that fixed IP thing, yeah we could check for that ahead of time19:39
melwittagreed it's tricky. the latest comment is an example involving volume quota, if the volume quota is 10 and they try to create 100 instances, 90 fail and they got 9 computes disabled19:40
dansmithwe should totally be able to distinguish something like that and not count that against the compute19:41
mriedemyeah i know we definitely don't check volume quota, even in the api19:41
mriedemhttps://review.openstack.org/#/c/520122/19:41
melwittit feels like maybe auto-disable shouldn't be the default, it looks like this is easy to exploit19:41
*** smatzek has quit IRC19:42
melwittI'm thinking of if a public cloud is running openstack, someone could do this and get all the computes disabled. unless I'm missing something19:42
dansmithin a public cloud it's less likely19:42
dansmithbecause a single successful boot will reset the counter19:42
dansmithI mean less likely IMHO19:42
*** smatzek has joined #openstack-nova19:43
mriedemthe fact we have no limit on the multiple instance create is a problem since you can just say give me 1000 instances, which can cause other issues like the rpc call to select_destinations to timeout19:43
mriedembut much harder to mess with that when i spec'ed up some ideas19:43
mriedemhttps://review.openstack.org/#/c/510235/19:44
dansmithwe could do something like record the user-id of the last failure and not count subsequent fails by a single user, although that would fail for single-tenant clouds (i.e. NFV)19:44
melwittthinking ... I think all you'd need is a burst of requests to pretty well ensure the counter won't reset because usually the scheduler will pack. but yeah, maybe it's not as easy to exploit in a public cloud as I was thinking19:46
dansmithI'm definitely in favor of catching and excluding quota-type failures if and where we can19:47
*** smatzek has quit IRC19:47
*** Guest87011 has joined #openstack-nova19:47
melwittyeah, that one is at least obvious. so any OverQuota that came come from compute (neutron/cinder)19:48
*** smatzek has joined #openstack-nova19:49
*** salv-orlando has quit IRC19:51
*** salv-orlando has joined #openstack-nova19:51
*** smatzek has quit IRC19:53
*** smatzek has joined #openstack-nova19:55
*** gszasz has quit IRC19:55
*** Guest87011 has quit IRC19:56
*** salv-orlando has quit IRC19:56
*** smatzek has quit IRC19:59
*** jackie-truong_ has joined #openstack-nova20:01
*** smatzek has joined #openstack-nova20:01
*** jackie-truong has quit IRC20:01
*** eharney has joined #openstack-nova20:02
*** smatzek has quit IRC20:06
*** sambetts is now known as sambetts|afk20:06
*** itlinux has joined #openstack-nova20:07
*** eharney_ has joined #openstack-nova20:07
*** eharney_ has quit IRC20:07
mriedemcellsv2 map_instances usage bug thingy https://bugs.launchpad.net/nova/+bug/174264920:08
openstackLaunchpad bug 1742649 in nova (Ubuntu) "map_instances default batch size is too small." [Undecided,New]20:08
mriedemmapping 200K instances took a long time20:08
*** edmondsw has quit IRC20:08
*** penick has joined #openstack-nova20:08
*** edmondsw has joined #openstack-nova20:09
melwittthat's a lot of instances20:10
mriedembecause it maps all of them in a single shot but in batches of 5020:10
*** smatzek has joined #openstack-nova20:13
melwitturgh. so really it would be best to have it similar to what we have in archive_deleted_rows now where you can give the max batch size but also say --until-complete20:13
*** edmondsw has quit IRC20:13
mriedemthat's what he asked for in there yeah20:14
mriedem"Actually, it would be useful to be able to say map all the instances using this batch size, eg: nova-manage --config-file /etc/nova/nova.conf cell_v2 map_instances --cell_uuid <uuid> --max_count all --batch_size 50000              "20:14
melwittyeah, I was thinking that sounded familiar and saw we have it for archive_deleted_rows20:14
mriedemthe thing about archive_deleted_rows is you're going to run that many times throughout the life of the deployment20:15
mriedemmap_instances is a done time deal when you're upgrading to cells v220:15
mriedem*one time20:15
melwittyeah20:15
melwittI dunno, it could be a low-hanging-fruit thing to do20:15
*** edmondsw has joined #openstack-nova20:15
mriedemi guess20:16
mriedemi'm not sure i'd want to review it though20:16
mriedemhigh hanging review fruit20:16
melwittI wasn't thinking it would be hard to review. maybe I'm underestimating it20:17
*** smatzek has quit IRC20:17
mriedemthe only thing you're underestimating is my laziness at the moment20:18
*** smatzek has joined #openstack-nova20:19
melwittheh. fair enough :)20:19
mriedemi'll put something in the docs and someone else can wizbang up a fancy new batch-size option or whatever20:19
*** edmondsw has quit IRC20:19
*** amodi has quit IRC20:20
melwittokay. I'll mention it at the cells meeting and see if tssurya might be interested20:21
mriedemalright. already threw it on the meeting agenda20:21
mriedemin the bugs list20:21
melwittcoolness20:21
mriedembtw, have we heard any updates from belmiro on the great cellsv2ification at cern?20:21
mriedemand how it's going?20:22
melwittI haven't heard anything yet. just implied from the fixes they've been doing for other nova-manage cell_v2 commands, that they've been working on it20:22
melwittI think they'll need an improvement to map_instances themselves lest it take hours to run, heh20:23
*** smatzek has quit IRC20:23
melwittif they haven't already gone through it20:23
*** smatzek has joined #openstack-nova20:24
*** Jeffrey4l has quit IRC20:25
*** Jeffrey4l has joined #openstack-nova20:25
*** edmondsw has joined #openstack-nova20:26
openstackgerritHongbin Lu proposed openstack/nova master: Handle RPC timeout on get_console_output  https://review.openstack.org/52533520:27
*** smatzek has quit IRC20:29
*** amodi has joined #openstack-nova20:32
*** smatzek has joined #openstack-nova20:32
*** mvenesio_ has quit IRC20:32
openstackgerritMatt Riedemann proposed openstack/nova master: Expand on when you might want to set --max-count for map_instances  https://review.openstack.org/53695920:33
openstackgerritMatt Riedemann proposed openstack/nova master: Expand on when you might want to set --max-count for map_instances  https://review.openstack.org/53695920:34
*** awaugama has quit IRC20:35
*** penick has quit IRC20:36
*** penick has joined #openstack-nova20:36
*** mvenesio has joined #openstack-nova20:39
*** salv-orlando has joined #openstack-nova20:39
*** mvenesio has quit IRC20:43
*** stvnoyes1 has quit IRC20:48
*** stvnoyes has quit IRC20:48
*** jackie-truong_ has quit IRC20:50
melwittwhat is with "libvirtError: Cannot recv data: Connection reset by peer" on stable, it's been flipping out, and specifically on the cells v1 job20:50
*** jackie-truong has joined #openstack-nova20:51
mriedemis libvirt crashing?20:52
mriedemcheck the libvirtd logs?20:52
mriedemRoamer`: done https://review.openstack.org/#/c/140733/20:52
* melwitt looks20:53
*** moshele has joined #openstack-nova20:54
Roamer`mriedem, just got the e-mail, thanks!  Looking at it...20:58
*** itlinux has quit IRC20:59
*** nicolasbock has joined #openstack-nova20:59
*** moshele has quit IRC20:59
tonybmriedem: my plan is to do the double checking today and then ask infra.21:01
tonybmriedem: (re tagging nova newton-eol)21:01
melwittdoesn't look like a crash but I'm not too familiar with libvirtd logging. I see a message '"kvm" accelerator not found' several times http://logs.openstack.org/97/536897/1/check/legacy-tempest-dsvm-cells/1f42e60/logs/libvirt/libvirtd.txt.gz#_2018-01-23_18_37_23_87321:01
*** moshele has joined #openstack-nova21:01
*** sdague has quit IRC21:03
*** pramodrj07 has joined #openstack-nova21:03
*** mvk has joined #openstack-nova21:04
*** damien_r has quit IRC21:05
*** damien_r has joined #openstack-nova21:05
*** lyan has quit IRC21:05
mriedemdoesn't look like it21:06
mriedemtonyb: ok21:06
melwittcomparing libvirtd.log with master change21:07
tonybmelwitt: I wouldn't think that was a problem in a dsvm test21:08
*** lyan has joined #openstack-nova21:08
melwittI don't find the kvm accelerator message in the master change libvirtd log21:08
*** moshele has quit IRC21:09
*** penick has quit IRC21:09
mriedemmelwitt: what's the virt_type in nova.conf?21:09
mriedemshould be qemu21:09
mriedemnote that the version of libvirt is probably different between stable and master job runs21:10
mriedemwe use the pike UCA in master21:10
melwitthuh, the nova-cpu.conf is very empty. http://logs.openstack.org/97/536897/1/check/legacy-tempest-dsvm-cells/1f42e60/logs/etc/nova/nova-cpu.conf.txt.gz21:10
melwittbut in nova.conf virt_type = qemu21:10
*** penick has joined #openstack-nova21:11
*** smatzek_ has joined #openstack-nova21:15
*** smatzek_ has quit IRC21:15
*** gszasz has joined #openstack-nova21:16
*** penick has quit IRC21:16
melwitttonyb: what do you mean? the kvm accelerator thing shouldn't hurt?21:22
mriedemmelwitt: the cellsv1 job is probably not running in superconductor mode,21:23
mriedemso doesn't use nova-cpu.conf21:23
melwittoh, okay21:23
*** smatzek has quit IRC21:23
tonybmelwitt: just that in dsvm I'd expect kvm accelerator to me missing (as we should be using qemu), but it it strange that it's even looking for it21:24
melwittbah, there's a legit fail in the functional test_server_group tests on stable too. working on that21:24
*** penick has joined #openstack-nova21:24
melwitttonyb: oh, I see21:24
*** moshele has joined #openstack-nova21:27
*** penick has quit IRC21:27
*** slaweq has quit IRC21:28
*** penick has joined #openstack-nova21:29
openstackgerritMerged openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075621:30
*** jackie-truong has quit IRC21:30
*** slaweq has joined #openstack-nova21:31
*** jackie-truong has joined #openstack-nova21:32
*** edmondsw has quit IRC21:33
openstackgerritMerged openstack/nova master: Replace curly quotes with straight quotes  https://review.openstack.org/53672721:33
*** edmondsw has joined #openstack-nova21:33
melwittcurly quotes eh?21:34
openstackgerritHongbin Lu proposed openstack/nova master: Handle RPC timeout on get_console_output  https://review.openstack.org/52533521:35
jackie-truongIf any core reviewer has a chance to take a look at this patch, it would be very much appreciated: https://review.openstack.org/#/c/486204/21:35
*** cleong has quit IRC21:35
*** slaweq has quit IRC21:36
jackie-truongjaypipes was looking at the patch series, but he's out at a work conference at the moment. He had taken away +2 from its dependent patch (https://review.openstack.org/#/c/489408/) due to the patch series not being complete. But patch series is complete now :-)21:37
jackie-truongUsage and testing instructions can be found here: https://etherpad.openstack.org/p/queens-nova-certificate-validation21:38
*** edmondsw has quit IRC21:38
jackie-truongAnd we also wrote a barbican-tempest-plugin test for it: https://review.openstack.org/#/c/515210/21:38
*** penick has quit IRC21:41
*** gouthamr_ has quit IRC21:42
*** dave-mccowan has quit IRC21:43
*** penick has joined #openstack-nova21:43
openstackgerritmelanie witt proposed openstack/nova stable/pike: Set server status to ERROR if rebuild failed  https://review.openstack.org/53689721:46
openstackgerritmelanie witt proposed openstack/nova stable/pike: Fix false positive server group functional tests  https://review.openstack.org/53698121:46
*** fragatina has joined #openstack-nova21:47
*** tidwellr_ has quit IRC21:54
*** pchavva has quit IRC21:57
*** felipemonteiro_ has joined #openstack-nova22:01
*** dave-mccowan has joined #openstack-nova22:03
openstackgerritPeter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver.  https://review.openstack.org/14073322:03
*** slaweq has joined #openstack-nova22:03
*** priteau has quit IRC22:03
*** slaweq has quit IRC22:04
mriedemdansmith: on alex_xu's 1.12 allocation candidate change, i assume you ran into the same sticky part here https://review.openstack.org/#/c/536083/4/nova/scheduler/client/report.py@115322:05
*** felipemonteiro has quit IRC22:05
mriedemwith assuming that the client is getting a 1.12 format request,22:05
mriedemwe could be in the middle of a server create / cold migrate where the format is actually 1.10, you upgrade and restart conductor, and now the client code assumes it's going to get a 1.12 but it's 1.10 and kablammo22:06
mriedemvia reschedule from the compute22:06
mriedembecause we don't have a versioned object facade for this payload22:06
*** amodi has quit IRC22:09
Roamer`mriedem, new patchset up; thanks a lot for the very useful comments!22:10
Roamer`(I guess now we wait for 12 hours or so for the checks to pass...)22:10
*** rcernin has joined #openstack-nova22:15
*** moshele has quit IRC22:16
*** penick has quit IRC22:16
mriedemRoamer`: do you have a 3rd party CI that will eventually comment on this change to say if it's working?22:17
Roamer`mriedem, actually, yes, we do; the logs are at http://logs.ci-openstack.storpool.com/ but the run it's doing right now is not with this change yet, the next one will be22:19
Roamer`so I guess in about two hours or so there'll be a run with this one22:19
Roamer`(yeah, I should really look into speeding the CI runs up a bit)22:19
Roamer`it's just Cinder though, not Nova22:20
Roamer`I mean, it's the Tempest volume tests, not all the Nova-related tests22:20
mriedembut at some point it's going to at least run a tempest compute api test to attach and detach a volume to a server instance yeah?22:20
Roamer`yes, it will22:20
mriedemok22:21
Roamer`I may not be awake when it does (just after midnight here), but I'll post a link in about seven hours22:21
Roamer`then you won't be awake :)22:21
*** penick has joined #openstack-nova22:21
mriedemoh i'll be awake22:21
mriedemsitting by the phone22:21
mriedemanyone know when jaypipes is going to be back?22:24
jackie-truongmriedem: All I heard is "at a work conference this week"22:24
mriedemalright22:25
Roamer`hopefully you won't have to rescue the mailman again... here in Sofia we have a suburb up in the mountain where I used to live for like 20 years, and the snow is a bit of a problem sometimes, like public transport going "nuh-uh, not today"22:25
mriedemi was unable to rescue the mailman - he had to get a tow truck22:25
mriedemrear-wheel drive with go-kart tires on an inline in a foot of snow isn't going to cut it22:26
mriedem*incline22:26
Roamer`oh, I hear you... there was this New Year's Eve when a neighbor had a party, he specifically warned people not to drive if they don't have the tires, and as a result there were three taxi cabs stuck in the snow on our street22:27
mriedemi did enjoy watching the neighbor across the street, who is new to snow, try to drive his bmw sedan over the 3 foot wall of ice that the plow left in front of his driveway, and of course get it stuck halfway22:27
*** yamahata has joined #openstack-nova22:28
melwittI have read that the most important thing is to have winter tires. and that 4x4 isn't necessarily needed?22:28
mriedemi've never had winter tires22:28
mriedemor 4x322:28
mriedem*4x422:28
mriedemjust don't be a jackhole22:28
mriedemand put a shovel in the trunk22:28
melwittheh, good to know22:29
mriedemlaura's got the 4x4 in our family22:29
jrollmelwitt: 4wd helps you go, winter tires help you stop :P22:29
*** penick has quit IRC22:29
mriedempump those brakes22:29
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed  https://review.openstack.org/53690422:29
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Fix false positive server group functional tests  https://review.openstack.org/53699522:29
jrolldon't be a jackhole is definitely the best advice though22:30
* melwitt records info for if she's ever driving in snow22:30
mriedemha, that reminds me - saw a minivan racing down the hill towing like 6 kids on a sled yesterday,22:30
mriedemif they would have had to stop short, all of those kids could have been destoryed22:30
mriedem*destroyed22:30
Roamer`well, they *might* have been able to roll to the side, fall off the sled, but still... no, not a good image at all22:31
*** jackie-truong has quit IRC22:31
mriedemthe ones that are destined to pass on their genes, yes they might have had the instinct22:31
*** Guest87011 has joined #openstack-nova22:31
mriedemor, be in the back22:32
*** felipemonteiro_ has quit IRC22:32
melwitt*facepalm*22:32
dansmithmriedem: hadn't even gotten that far actually, but yeah, newer code will probably need to handle the case and transform older ones to newer ones22:32
dansmithmriedem: and yes, that's related enough to what I was saying about passing this dict of allocations/resources around to need some wrapage22:33
mriedemi guess you could determine format based on whether or not the 'allocations' entry within is a list or dict, and swaparoo if it's not a dict22:34
*** acormier has quit IRC22:34
*** acormier has joined #openstack-nova22:34
dansmithfor this case I think it can be done, yeah, but it's not a good precedent22:35
*** acormier has joined #openstack-nova22:35
mriedemagree22:36
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Fix false positive server group functional tests  https://review.openstack.org/53699522:36
openstackgerritmelanie witt proposed openstack/nova stable/ocata: Set server status to ERROR if rebuild failed  https://review.openstack.org/53690422:36
*** dave-mccowan has quit IRC22:37
*** acormier has quit IRC22:37
*** acormier has joined #openstack-nova22:37
*** esberglu has quit IRC22:38
*** gszasz_ has joined #openstack-nova22:38
*** gszasz has quit IRC22:40
*** fragatina has quit IRC22:50
*** fragatina has joined #openstack-nova22:50
*** cburgess has quit IRC23:00
*** cburgess has joined #openstack-nova23:01
*** mvk has quit IRC23:02
*** artom has quit IRC23:03
openstackgerritHongbin Lu proposed openstack/nova master: [WIP] Use revision_number on adding/removing security group  https://review.openstack.org/53551023:10
*** acormier has quit IRC23:12
*** amodi has joined #openstack-nova23:12
*** acormier has joined #openstack-nova23:13
*** salv-orlando has quit IRC23:13
*** masber has quit IRC23:14
*** salv-orlando has joined #openstack-nova23:14
*** mvk has joined #openstack-nova23:17
*** acormier has quit IRC23:18
*** salv-orlando has quit IRC23:19
*** hongbin has quit IRC23:25
*** acormier has joined #openstack-nova23:26
*** acormier has quit IRC23:30
Roamer`mriedem, FTR, the StorPool CI is now doing a run with the latest patchset, and it just passed some tempest.api.compute.* tests; I'm going to sleep now, but I'll send you a link to the logs when I wake up, or you could check http://logs.ci-openstack.storpool.com/ for 534854/4 in an hour - hour and a half23:41
*** oanson has quit IRC23:42
*** jackie-truong has joined #openstack-nova23:42
*** oanson has joined #openstack-nova23:43
*** dave-mccowan has joined #openstack-nova23:51
*** takashin has joined #openstack-nova23:52
*** smatzek has joined #openstack-nova23:53
*** gszasz_ has quit IRC23:55
*** smatzek has quit IRC23:55

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