Thursday, 2017-05-04

*** yangyape_ has quit IRC00:00
openstackgerritMatthew Treinish proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947000:00
*** jamesdenton has quit IRC00:00
*** yamahata has joined #openstack-nova00:02
*** ssurana has joined #openstack-nova00:04
*** eric_young has quit IRC00:04
*** armax has quit IRC00:05
*** eric_young has joined #openstack-nova00:06
*** amotoki has quit IRC00:07
*** yingjun has joined #openstack-nova00:08
*** jamesdenton has joined #openstack-nova00:17
*** xgerman has joined #openstack-nova00:20
*** test222 has joined #openstack-nova00:22
mriedemdansmith: yeah gawd00:26
*** jamesdenton has quit IRC00:28
*** gyee has quit IRC00:29
*** zhurong has joined #openstack-nova00:31
*** ijw has quit IRC00:34
*** coreywright has joined #openstack-nova00:35
mriedemmtreinish: we shall see00:43
*** winston-d_ has joined #openstack-nova00:46
*** tovin07_ has joined #openstack-nova00:46
*** artom has quit IRC00:48
*** artom has joined #openstack-nova00:49
*** artom has quit IRC00:50
*** artom has joined #openstack-nova00:50
*** Jack_Iv has quit IRC00:51
mriedemmtreinish: hells yes00:53
mriedem+200:53
mriedemalex_xu_: oomichi: https://review.openstack.org/#/c/459470/600:53
*** cfriesen has joined #openstack-nova00:58
*** phuongnh has joined #openstack-nova00:58
*** abhishek has joined #openstack-nova01:00
*** tuanluong has joined #openstack-nova01:02
*** dave-mccowan has joined #openstack-nova01:03
*** coreywright has quit IRC01:04
*** abhishek has quit IRC01:07
mtreinishmriedem: awesome, thanks01:09
*** yangyapepeng has joined #openstack-nova01:11
*** yangyapepeng has quit IRC01:11
mtreinishmriedem: I think once that merges it'll be my first nova bp implemented since the coverage extension back in grizzly01:11
mriedemmtreinish: oh it's not done yet,01:11
mriedemnovaclient changes buddy01:11
* mtreinish rereads the spec01:12
*** gjayavelu has quit IRC01:12
mriedemi don't think it mentions it01:12
mtreinishit doesn't01:12
mriedemcerts was already deprecated in novaclient https://github.com/openstack/python-novaclient/commit/b6aea669630dc88296bdded963051a35cc9f995e01:13
mriedembut cloudpipe wasn't https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/cloudpipe.py01:13
mriedemi can probably do that if you want, i've been doing a lot of that in novaclient lately01:13
mtreinishmriedem: if you want, otherwise I'd push those up tomorrow01:13
*** yolkfull__ has joined #openstack-nova01:13
mriedemmtreinish: you should probably do it since it'll just enable me to procrastinate on summit prep01:14
mtreinishmriedem: heh, what do you think it is for me?01:14
mriedemdon't use the certs one as an example, use the one that deprecated os-hosts in novaclient01:14
mriedemuse this as the example https://github.com/openstack/python-novaclient/commit/ea3b9f7fef28a7f06b51a3b176c85a1fc33cecaa01:15
mtreinishok01:15
*** nic has quit IRC01:15
*** thorst has joined #openstack-nova01:15
mriedemmtreinish: there is a nova onboarding session at the summit you can attend too01:15
mriedemsince you're a new contributor again01:15
*** david-lyle has joined #openstack-nova01:16
*** zhurong has quit IRC01:16
mtreinishmriedem: heh, I'll consider it01:17
mtreinishI just looked my last nova commit before the cert removal was in feb. 201601:17
*** coreywright has joined #openstack-nova01:17
mriedemprobably when you were doing the tenant isolation stuff01:18
mriedemno, that was older01:18
mtreinishmriedem: https://review.openstack.org/#/c/279799/01:18
mriedemah i remember that01:19
*** cNilesh has joined #openstack-nova01:21
*** Apoorva__ has quit IRC01:21
mtreinishmriedem: at some point this cycle I'm also going to work on adding docs on how to install nova (or any project) from a tarball, because there were no instructions I could find for that (something from my dirty clouds talk)01:22
*** Shunli has joined #openstack-nova01:23
*** gcb has joined #openstack-nova01:23
mriedemok01:23
*** Yangyape_ has joined #openstack-nova01:23
mriedemi noticed your talk was accepted but i think i have a conflict...01:23
mriedemwhich sucks01:23
*** kaisers_ has joined #openstack-nova01:25
openstackgerritVladik Romanovsky proposed openstack/nova master: neutron: retrieve physical network name from a multi-provider network  https://review.openstack.org/43002601:26
*** zhurong has joined #openstack-nova01:26
mtreinishmriedem: oh well, there will always be the video01:28
mtreinishmriedem: I linked you the spoiler picture of what the cloud looked like right?01:28
*** kaisers has quit IRC01:29
vladikrjaypipes, dansmith, thanks for the reviews. I've update the multi-segments patch (comments mostly), could you please take a look? https://review.openstack.org/43002601:29
vladikrs/update/updated01:29
mriedemmtreinish: i think so yeah01:30
jaypipesvladikr: done!01:30
vladikrjaypipes, thanks!01:31
jaypipesnp :)01:31
*** yamamoto has joined #openstack-nova01:31
*** cfriesen has quit IRC01:31
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore  https://review.openstack.org/46204901:32
*** dixiaoli has joined #openstack-nova01:33
*** david-lyle has quit IRC01:34
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore  https://review.openstack.org/46204901:35
*** yamamoto has quit IRC01:35
*** litao has joined #openstack-nova01:41
mriedemKevin_Zheng: do you want to rebase this? https://review.openstack.org/#/c/459593/01:41
*** mingyu has joined #openstack-nova01:41
mriedemi suppose i can in gerrit01:42
openstackgerritMatt Riedemann proposed openstack/nova master: Support tag instances when boot(3/4)  https://review.openstack.org/45959301:42
Kevin_Zhengmriedem sure, but why?01:42
mriedemjust because it messes up the patch set order when going through the changes in gerrit if it's not rebased01:42
mriedemnothing major01:43
Kevin_ZhengAH, OK, I was going to ask you to review this one if you have time01:43
mriedemi can run through it quick01:44
*** mingyu has quit IRC01:46
Kevin_ZhengOne question01:46
Kevin_Zhengif we are using placement, which is mandatory in O, it seems that we are unable to use AggregateCoreFilter properly01:48
*** gjayavelu has joined #openstack-nova01:48
*** ijw has joined #openstack-nova01:49
*** ijw has quit IRC01:54
*** masber has quit IRC01:56
mriedemKevin_Zheng: i'm not familiar with that filter; comments in your tags 3/4 patch btw01:57
mriedemedleafe: bauzas: jaypipes: ^ re: placement + AggregateCoreFilter01:57
jaypipesKevin_Zheng: that filter wasn't touched by the placement service.01:58
mriedemi don't think that's the question01:58
jaypipesmriedem: I don't see a question...01:58
mriedemthis is a host aggregate with cpu_allocation_ratio meta set on it?01:58
mriedemis that the idea?01:59
mriedemjaypipes: so with placement in the filter scheduler we basically ignore the corefilter because we get compute node RPs via the VCPU resource class filtering right?01:59
openstackgerritfengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf  https://review.openstack.org/45878901:59
mriedemwe don't ignore it, but it's redundant01:59
mriedemand i think we said you could/should disable the CoreFilter, RAMFilter and DiskFilter once you're using placement01:59
jaypipesyeah...02:00
jaypipesbut aggregate metadata should still be processed, IIRC.02:00
mriedemyeah i'm not sure why this would be a problem,02:01
mriedemit's just extra filtering once we get the list of compute node resource providers filtered on VCPU to start02:01
*** marst has joined #openstack-nova02:01
mriedemKevin_Zheng: it's probably best to just report a bug with details02:01
mriedemespecially because i have a date with my couch and tv02:01
Kevin_Zhengha, i will02:02
mriedemgood night all02:03
mriedemand day02:03
*** mriedem has quit IRC02:03
*** salv-orlando has joined #openstack-nova02:05
Kevin_ZhengAs described in aggregatecorefilter it will try to get ratio from aggregate first and if not possible then fall back to per host ratio02:05
*** cfriesen has joined #openstack-nova02:07
Kevin_ZhengIf we use placement, then it directly selected hosts with per host ratio, if we have different configurations in aggregate meta, then the host selection might not be what we expected02:08
*** salv-orlando has quit IRC02:09
Kevin_ZhengSay, if people only use aggregatecorefilter as it will fall back to per host ratio, it won't work with placement02:10
*** dave-mccowan has quit IRC02:10
*** gjayavelu has quit IRC02:12
*** yolkfull has joined #openstack-nova02:13
*** yolkfull_ has joined #openstack-nova02:15
*** yolkfull__ has quit IRC02:16
*** yolkfull__ has joined #openstack-nova02:17
*** yolkfull has quit IRC02:18
*** yolkfull_ has quit IRC02:19
*** dave-mccowan has joined #openstack-nova02:20
openstackgerritshaofeng cheng proposed openstack/nova master: Update nova version newton to ocata in upgrade.rst  https://review.openstack.org/45719702:22
*** yolkfull__ has quit IRC02:23
*** yolkfull__ has joined #openstack-nova02:23
*** zhurong has quit IRC02:34
*** yamamoto has joined #openstack-nova02:36
*** zhurong has joined #openstack-nova02:36
*** markvoelker has quit IRC02:40
*** ssurana has quit IRC02:40
*** thorst has quit IRC02:41
*** yamamoto has quit IRC02:46
*** fragatina has joined #openstack-nova02:51
*** dave-mccowan has quit IRC02:51
*** Jack_Iv has joined #openstack-nova02:51
*** masber has joined #openstack-nova02:52
*** fragatina has quit IRC02:53
*** david-lyle has joined #openstack-nova02:53
*** hongbin has joined #openstack-nova02:53
*** fragatina has joined #openstack-nova02:53
*** fragatin_ has quit IRC02:54
*** gjayavelu has joined #openstack-nova02:55
*** Jack_Iv has quit IRC02:56
*** lucasxu has joined #openstack-nova02:58
*** nicolasbock has quit IRC02:58
*** vladikr has quit IRC02:59
*** lucasxu has quit IRC03:00
*** fragatin_ has joined #openstack-nova03:01
*** john51 has joined #openstack-nova03:03
*** sbezverk_ has joined #openstack-nova03:04
*** manjeets__ has joined #openstack-nova03:04
*** fragatina has quit IRC03:04
*** dimtruck is now known as zz_dimtruck03:05
*** iceyao_ has joined #openstack-nova03:05
*** crushil_ has joined #openstack-nova03:05
*** fragatin_ has quit IRC03:05
*** salv-orlando has joined #openstack-nova03:06
*** raorn has joined #openstack-nova03:07
*** slagle__ has joined #openstack-nova03:07
*** belliott_ has joined #openstack-nova03:07
*** jgrassler has joined #openstack-nova03:07
*** gaurangt_ has joined #openstack-nova03:10
*** zsli_ has joined #openstack-nova03:10
*** irenab_ has joined #openstack-nova03:10
*** salv-orlando has quit IRC03:11
*** rha_ has joined #openstack-nova03:11
*** test222 has quit IRC03:11
*** iceyao has quit IRC03:11
*** slagle has quit IRC03:11
*** rodrigod` has quit IRC03:11
*** irenab has quit IRC03:11
*** mariojv_ has quit IRC03:11
*** Guest73837 has quit IRC03:11
*** jgrassle1 has quit IRC03:11
*** raorn_ has quit IRC03:11
*** crushil has quit IRC03:11
*** jaypipes has quit IRC03:11
*** john51_ has quit IRC03:11
*** gaurangt has quit IRC03:11
*** ekuris has quit IRC03:11
*** dillaman has quit IRC03:11
*** sbezverk has quit IRC03:11
*** hamzy has quit IRC03:11
*** Kevin_Zheng has quit IRC03:11
*** manjeets has quit IRC03:11
*** belliott has quit IRC03:11
*** mdavidson has quit IRC03:11
*** rha has quit IRC03:11
*** irenab_ is now known as irenab03:11
*** rodrigods has joined #openstack-nova03:11
*** Shunli has quit IRC03:12
*** tovin07_ has quit IRC03:16
*** tovin07_ has joined #openstack-nova03:17
*** trungnv has quit IRC03:18
*** Guest73837 has joined #openstack-nova03:18
*** mdavidson has joined #openstack-nova03:18
*** trungnv has joined #openstack-nova03:18
*** ekuris has joined #openstack-nova03:18
openstackgerritJackie Truong proposed openstack/nova master: Add missing rootwrap filter for cryptsetup  https://review.openstack.org/46234803:18
*** mariojv_ has joined #openstack-nova03:18
*** hamzy has joined #openstack-nova03:19
*** dillaman has joined #openstack-nova03:19
*** jaypipes has joined #openstack-nova03:19
*** jaypipes has quit IRC03:19
*** Kevin_Zheng has joined #openstack-nova03:24
*** masber has quit IRC03:24
*** dixiaoli has quit IRC03:30
*** armax has joined #openstack-nova03:31
*** mingyu has joined #openstack-nova03:32
*** gongysh has joined #openstack-nova03:32
openstackgerritchenaidong1 proposed openstack/nova master: Fix a typo  https://review.openstack.org/46234903:33
*** mingyu has quit IRC03:38
openstackgerritchenaidong1 proposed openstack/python-novaclient master: Fix a typo  https://review.openstack.org/46235103:38
*** thorst has joined #openstack-nova03:41
*** markvoelker has joined #openstack-nova03:44
*** thorst has quit IRC03:45
*** gouthamr has quit IRC03:58
*** iceyao_ has quit IRC03:59
*** armax has quit IRC03:59
openstackgerritLiChunlin proposed openstack/nova master: update the description of hypervisor statistics response  https://review.openstack.org/46235504:03
*** abhishek has joined #openstack-nova04:05
*** salv-orlando has joined #openstack-nova04:07
*** sridharg has joined #openstack-nova04:08
*** masber has joined #openstack-nova04:11
*** salv-orlando has quit IRC04:11
*** zhurong has quit IRC04:12
*** thorst has joined #openstack-nova04:12
*** ssurana has joined #openstack-nova04:12
*** adisky_ has joined #openstack-nova04:13
*** trinaths has joined #openstack-nova04:17
*** xinliang has quit IRC04:20
*** abhishek has quit IRC04:23
*** thorst has quit IRC04:30
*** xinliang has joined #openstack-nova04:33
*** zhurong has joined #openstack-nova04:35
*** yamahata has quit IRC04:35
*** mingyu has joined #openstack-nova04:36
*** Sukhdev has joined #openstack-nova04:36
*** fragatina has joined #openstack-nova04:39
*** fragatina has quit IRC04:40
*** fragatina has joined #openstack-nova04:40
*** mingyu has quit IRC04:41
*** slaweq has quit IRC04:46
*** ayogi has joined #openstack-nova04:47
*** salv-orlando has joined #openstack-nova04:52
openstackgerritChangBo Guo(gcb) proposed openstack/nova master: Remove oslo.config deprecated parameter enforce_type  https://review.openstack.org/46236404:52
*** hongbin has quit IRC04:56
*** iceyao has joined #openstack-nova04:56
*** ratailor has joined #openstack-nova04:57
*** cfriesen has quit IRC04:59
*** tjones has quit IRC04:59
*** iceyao has quit IRC05:03
*** Sukhdev has quit IRC05:05
*** arunman has joined #openstack-nova05:08
*** vks1 has joined #openstack-nova05:08
*** marst has quit IRC05:11
*** tjones has joined #openstack-nova05:11
*** baoli has quit IRC05:12
*** test222 has joined #openstack-nova05:14
*** prateek has joined #openstack-nova05:16
*** yamamoto has joined #openstack-nova05:18
*** links has joined #openstack-nova05:18
*** yamamoto has quit IRC05:21
*** diga has joined #openstack-nova05:22
*** trungnv has quit IRC05:38
*** Dinesh_Bhor has quit IRC05:41
*** abhi89 has joined #openstack-nova05:41
*** Dinesh_Bhor has joined #openstack-nova05:41
*** Dinesh_Bhor has quit IRC05:43
*** irenab has left #openstack-nova05:44
*** Dinesh_Bhor has joined #openstack-nova05:48
*** trungnv has joined #openstack-nova05:48
*** sree has joined #openstack-nova05:51
*** trungnv has quit IRC05:51
*** Jack_Iv has joined #openstack-nova05:52
*** Jack_Iv has quit IRC05:56
*** trungnv has joined #openstack-nova06:01
*** kiwi_rot has joined #openstack-nova06:02
*** salv-orlando has quit IRC06:04
*** amotoki has joined #openstack-nova06:05
*** mdnadeem has joined #openstack-nova06:07
*** rajathagasthya has joined #openstack-nova06:09
*** iceyao has joined #openstack-nova06:10
*** mingyu has joined #openstack-nova06:11
*** coreywright has quit IRC06:14
*** avolkov has joined #openstack-nova06:14
*** rajathagasthya has quit IRC06:15
*** rajathagasthya has joined #openstack-nova06:15
*** mingyu has quit IRC06:16
*** trinaths has quit IRC06:18
*** brault has quit IRC06:19
*** rajathagasthya has quit IRC06:21
*** yamamoto has joined #openstack-nova06:21
*** trinaths has joined #openstack-nova06:23
*** voelzmo has joined #openstack-nova06:23
*** sbezverk_ has quit IRC06:23
*** udesale has joined #openstack-nova06:24
*** mingyu has joined #openstack-nova06:25
*** yamamoto has quit IRC06:26
*** yamamoto has joined #openstack-nova06:26
*** thorst has joined #openstack-nova06:27
*** yamamoto has quit IRC06:28
*** andreas_s has joined #openstack-nova06:28
*** Oku_OS-away is now known as Oku_OS06:28
*** kiwi_rot has quit IRC06:30
*** kiwi_rot has joined #openstack-nova06:30
*** coreywright has joined #openstack-nova06:32
*** thorst has quit IRC06:32
*** ltomasbo|away is now known as ltomasbo06:37
*** mingyu has quit IRC06:41
*** mingyu has joined #openstack-nova06:42
*** rmart04_ has joined #openstack-nova06:42
*** pcaruana has joined #openstack-nova06:45
*** mnestratov has joined #openstack-nova06:45
*** slaweq has joined #openstack-nova06:47
*** rha_ is now known as rha06:51
*** rha has joined #openstack-nova06:51
*** slaweq has quit IRC06:52
*** arunman has quit IRC06:52
*** mingyu has quit IRC06:55
*** mingyu has joined #openstack-nova06:56
*** yamamoto has joined #openstack-nova06:59
*** Jack_Iv has joined #openstack-nova07:00
*** Jack_Iv has quit IRC07:00
*** Jack_Iv has joined #openstack-nova07:00
*** damien_r has quit IRC07:01
*** arunman has joined #openstack-nova07:02
*** damien_r has joined #openstack-nova07:04
*** Jack_Iv has quit IRC07:04
*** moshele has joined #openstack-nova07:04
*** jamielennox is now known as jamielennox|away07:06
*** yamamoto has quit IRC07:09
*** iceyao has quit IRC07:11
*** damien_r1 has joined #openstack-nova07:13
*** damien_r has quit IRC07:13
*** iceyao has joined #openstack-nova07:15
*** ralonsoh has joined #openstack-nova07:17
*** ijw has joined #openstack-nova07:19
*** iceyao has quit IRC07:20
*** iceyao has joined #openstack-nova07:24
*** aarefiev_afk is now known as aarefiev07:24
*** ijw has quit IRC07:24
*** salv-orlando has joined #openstack-nova07:24
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-instance-usage-audit-log  endpoint instead of stevedore  https://review.openstack.org/46164707:25
*** Jack_Iv has joined #openstack-nova07:26
openstackgerritjichenjc proposed openstack/nova master: use plain routes list for os-simple-tenant-usage  https://review.openstack.org/46082107:27
*** thorst has joined #openstack-nova07:28
*** tesseract has joined #openstack-nova07:29
*** thorst has quit IRC07:33
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for os-server-groups endpoint instead of stevedore  https://review.openstack.org/46216907:33
openstackgerritAndreas Scheuring proposed openstack/nova master: DO NOT MERGE  https://review.openstack.org/46242107:38
*** jpena|off is now known as jpena07:40
*** Yangyape_ has quit IRC07:45
*** Yangyape_ has joined #openstack-nova07:46
openstackgerritjichenjc proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore  https://review.openstack.org/46164107:48
*** ssurana has quit IRC07:48
openstackgerritjichenjc proposed openstack/nova master: refactory some code on get_password  https://review.openstack.org/44945807:49
*** karimb has joined #openstack-nova07:49
*** rmart04_ has quit IRC07:49
*** lpetrut has joined #openstack-nova07:51
openstackgerritAlex Xu proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947007:52
openstackgerritAlex Xu proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947007:53
*** markus_z has joined #openstack-nova07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** zsli_ has quit IRC08:06
*** zsli_ has joined #openstack-nova08:07
*** udesale has quit IRC08:11
*** gcb has quit IRC08:12
openstackgerritZhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore  https://review.openstack.org/46204908:13
*** lucas-afk is now known as lucasagomes08:13
*** udesale has joined #openstack-nova08:14
*** rajathagasthya has joined #openstack-nova08:14
*** jaosorior_away is now known as jaosorior08:14
*** rmart04 has joined #openstack-nova08:15
*** rajathagasthya has quit IRC08:15
sfinucanmdbooth: Could you take a look at https://review.openstack.org/#/q/topic:disk_info_processing today? It's been around a long time and it seems like something you should look at first08:19
*** brault has joined #openstack-nova08:24
*** trinaths has quit IRC08:25
openstackgerritshaofeng cheng proposed openstack/nova master: Replace newton to release_name in upgrade.rst  https://review.openstack.org/45719708:26
*** karimb has quit IRC08:28
*** thorst has joined #openstack-nova08:29
*** gjayavelu has quit IRC08:31
*** derekh has joined #openstack-nova08:36
*** adreznec has quit IRC08:45
*** adreznec has joined #openstack-nova08:47
openstackgerritjichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes  https://review.openstack.org/46244108:48
*** slaweq has joined #openstack-nova08:48
*** thorst has quit IRC08:49
sfinucanjohnthetubaguy: Are we allowed to delete configuration without a deprecation cycle, even if they do nothing? https://review.openstack.org/#/c/459470/08:49
*** gcb has joined #openstack-nova08:50
johnthetubaguysfinucan: so cloudpipe only works with nova-network, and nova-network has been deprecated (on and off) for a long time08:50
johnthetubaguysfinucan: I would give that one a pass08:50
*** karimb has joined #openstack-nova08:50
sfinucanjohnthetubaguy: Cool. Sounds fair08:51
*** ratailor_ has joined #openstack-nova08:52
*** slaweq has quit IRC08:53
openstackgerritjichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes  https://review.openstack.org/46244108:54
*** ratailor has quit IRC08:55
*** salv-orl_ has joined #openstack-nova08:56
johnthetubaguysfinucan: +1 deleting that already deprecated option that does nothing now08:56
abhi89johnthetubaguy: can you please review https://review.openstack.org/#/c/460305/08:56
johnthetubaguyabhi89: ah, I should take a look at that now08:57
*** salv-orlando has quit IRC08:59
*** karimb has quit IRC09:00
*** sridharg has quit IRC09:00
*** sree_ has joined #openstack-nova09:00
*** sree_ is now known as Guest9755009:01
*** zsli_ has quit IRC09:01
johnthetubaguyabhi89: I really like that regression test now, really nice proof that your change works09:02
*** karimb has joined #openstack-nova09:02
*** Shunli has joined #openstack-nova09:03
*** sree has quit IRC09:03
*** Shunli has quit IRC09:03
abhi89johnthetubaguy: yes.. i have well tested the changes also09:04
johnthetubaguyabhi89: I just love seeing the automated tests like that, really stops us breaking your fix by accident later09:04
*** Guest7024 has joined #openstack-nova09:05
abhi89johnthetubaguy: yes.. Thanks John09:05
johnthetubaguyabhi89: thank you for sticking with it, great to get that fixed09:06
*** zhurong has quit IRC09:06
sfinucanjohnthetubaguy: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today?09:10
johnthetubaguysfinucan: yeah, I want to sit down with the quota code again at some point though09:11
openstackgerritfalseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message  https://review.openstack.org/46198509:11
*** djohnsto has joined #openstack-nova09:11
sfinucanjohnthetubaguy: ah, at some point then :)09:14
*** zhurong has joined #openstack-nova09:15
*** yamamoto has joined #openstack-nova09:15
*** Guest7024 is now known as Frecy209:17
*** Shunli has joined #openstack-nova09:18
*** moshele has quit IRC09:18
*** tovin07 has quit IRC09:18
*** yamamoto has quit IRC09:19
sfinucanjohnthetubaguy: Ta :)09:24
*** markvoelker has quit IRC09:25
*** cdent has joined #openstack-nova09:29
*** rmart04 has quit IRC09:29
*** purplerbot has joined #openstack-nova09:29
*** yingjun has quit IRC09:31
*** iceyao has quit IRC09:36
*** aweeks has quit IRC09:38
*** aweeks has joined #openstack-nova09:38
*** test222 has quit IRC09:39
*** test222 has joined #openstack-nova09:41
*** lpetrut_ has joined #openstack-nova09:41
*** lpetrut has quit IRC09:41
*** lpetrut_ has quit IRC09:42
*** lpetrut_ has joined #openstack-nova09:42
*** links has quit IRC09:45
*** karimb has quit IRC09:47
*** karimb has joined #openstack-nova09:50
*** cNilesh has quit IRC09:57
openstackgerritStephen Finucane proposed openstack/nova master: Fix default_availability_zone docs  https://review.openstack.org/43888709:58
openstackgerritStephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group  https://review.openstack.org/46246909:58
*** moshele has joined #openstack-nova09:59
*** iceyao has joined #openstack-nova10:02
*** mvk has quit IRC10:03
*** Jack_Iv has quit IRC10:04
*** tovin07_ has quit IRC10:04
*** Jack_Iv has joined #openstack-nova10:05
*** iceyao has quit IRC10:06
*** zhurong has quit IRC10:07
*** nicolasbock has joined #openstack-nova10:08
*** iceyao has joined #openstack-nova10:10
*** sdague has joined #openstack-nova10:11
openstackgerritStephen Finucane proposed openstack/nova master: Add PCIWeigher  https://review.openstack.org/37952410:13
openstackgerritStephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances  https://review.openstack.org/37962510:13
*** mingyu has quit IRC10:13
openstackgerritStephen Finucane proposed openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks'  https://review.openstack.org/41652010:14
sfinucanbauzas: Fancy taking a look at two availability zone changes? https://review.openstack.org/438887 https://review.openstack.org/46246910:15
*** mingyu has joined #openstack-nova10:15
*** adisky_ has quit IRC10:19
*** adisky_ has joined #openstack-nova10:20
*** zhurong has joined #openstack-nova10:24
*** markvoelker has joined #openstack-nova10:26
*** tuanluong has quit IRC10:28
*** gongysh has quit IRC10:30
*** mvk has joined #openstack-nova10:30
*** arunman has quit IRC10:31
*** markvoelker has quit IRC10:33
*** arunman has joined #openstack-nova10:33
*** phuongnh has quit IRC10:36
*** mingyu has quit IRC10:36
*** phuongnh has joined #openstack-nova10:38
*** tbachman has quit IRC10:43
*** stian__ has quit IRC10:44
*** ratailor_ has quit IRC10:46
*** thorst has joined #openstack-nova10:46
*** wxy has quit IRC10:48
*** brault_ has joined #openstack-nova10:48
*** wxy has joined #openstack-nova10:48
*** mrhillsman has quit IRC10:48
*** phuongnh has quit IRC10:48
*** smatzek has joined #openstack-nova10:48
*** brault has quit IRC10:49
*** mrhillsman has joined #openstack-nova10:49
*** slaweq has joined #openstack-nova10:49
*** mingyu has joined #openstack-nova10:49
*** thorst has quit IRC10:51
*** slaweq has quit IRC10:53
*** salv-orl_ has quit IRC10:55
*** ijw has joined #openstack-nova10:56
*** yolkfull__ has quit IRC10:58
*** ijw has quit IRC11:01
*** Frecy2 has left #openstack-nova11:04
*** jpena is now known as jpena|lunch11:04
*** dave-mccowan has joined #openstack-nova11:08
*** nicolasbock has quit IRC11:12
*** ociuhandu has joined #openstack-nova11:12
*** abhi89 has quit IRC11:14
*** Jack_Iv has quit IRC11:15
*** edmondsw has quit IRC11:16
openstackgerritMatthew Booth proposed openstack/nova master: Fix scope of instance error context manager in resize_instance  https://review.openstack.org/46249911:16
mdboothHopefully pretty simple bugfix ^^^11:16
sean-k-mooneyhi11:20
sean-k-mooneyis there a way to make devstack not use systemd still11:21
mdboothsean-k-mooney: That got pushed yesterday, I think.11:21
mdboothhttps://review.openstack.org/#/c/462163/311:21
sean-k-mooneyswapping to systemd or keeping the legacy way11:21
mdboothKeeps systemd option, but defaults to legacy11:22
*** diga has quit IRC11:22
sean-k-mooneyi pulled devstack today and it defaulted to systemd instead of screen11:22
sean-k-mooneybut ok ill look into it. i hate systemd... this might actully be the thing that makes me stop using devstack for my development11:23
*** dpawlik has joined #openstack-nova11:24
sean-k-mooneyah i have to set USE_SCREEN=True to get the old behavior11:24
dpawlikHello. Can someone of you tell me if I should add DocImpact to the commit message in this PS: https://review.openstack.org/#/c/459753/ ?11:25
sean-k-mooneydpawlik: you will need to update the docs for the new image metadata item but i belive that would be in glance's docs not novas11:27
dpawliksean-k-mooney: hey. So better when I add here DocImpact and ask glance team to add it in its docs11:28
*** karimb has quit IRC11:29
sean-k-mooneydpawlik: https://wiki.openstack.org/wiki/Documentation/DocImpact11:30
dpawliksean-k-mooney: thx11:30
*** thorst has joined #openstack-nova11:33
*** vks1 has quit IRC11:33
*** Shunli has quit IRC11:34
*** nicolasbock has joined #openstack-nova11:34
*** gszasz has joined #openstack-nova11:36
*** sbezverk has joined #openstack-nova11:40
openstackgerritDaniel Pawlik proposed openstack/nova master: Added kvm_hidden feature  https://review.openstack.org/45975311:44
sean-k-mooneysdague: hi, 2 quick quetstions regarding the new systemd workflow for devstack will the screen workflow be tested and maintined going forword? also will devstack now work across reboots again?11:45
*** mingyu has quit IRC11:45
*** mingyu has joined #openstack-nova11:45
*** catintheroof has joined #openstack-nova11:47
sdaguesean-k-mooney: screen goes away in Queens11:49
sdaguehttp://lists.openstack.org/pipermail/openstack-dev/2017-May/116301.html11:50
sdagueI'm not sure what works across reboots means11:50
* mdbooth wonders if the library which writes to the systemd journal drops messages if it would block11:50
sdaguemdbooth: that's entirely possible...11:50
mdboothI don't think the journal itself drops messages11:50
sdaguemdbooth: we did disable it now, so we're out of that woods for the moment11:51
mdboothYeah, makes sense11:51
mdboothOne to investigate though, for sure11:51
*** catintheroof has quit IRC11:51
sean-k-mooneysdague: ok i might need to make changes to our devstack plugins/CIs.11:53
*** Yangyape_ has quit IRC11:53
sean-k-mooneyi think they will work but they may not11:54
sdaguesean-k-mooney: yeh, that thread has some common issues people hit11:54
*** salv-orlando has joined #openstack-nova11:54
sean-k-mooneysdague: i guess il have to give it a fair shot but honestly the fact that the process ran in screen sessions was one of the things i liked about devstack as it made it easy to wrap your head around when you were new to openstack11:56
kashyapsean-k-mooney: It isn't any more?  I haven't run a DevStack run in a couple of weeks11:57
sdaguesean-k-mooney: ok, an equally large number of people have stated that screen was the most confusing thing ever11:57
*** jpena|lunch is now known as jpena11:57
*** kiwi_rot has quit IRC11:57
sdaguebut, regardless, there are other reasons we need this change11:57
kashyapI've seen the systemd / 'screen' related change whoosh by.  /me looks closer...11:57
kashyapHmm, it's this patch: "make USE_SCREEN=False imply USE_SYSTEMD=True"11:57
*** Guest97550 has quit IRC11:57
*** mingyu has quit IRC11:58
*** tbachman has joined #openstack-nova11:58
*** sree has joined #openstack-nova11:58
sean-k-mooneyyes i swapped the flag back and it still failed to stack so my local failure wasnt related to the systemd change so im going to start with a clean environment again11:58
sean-k-mooneykashyap: ^11:59
kashyapsean-k-mooney: What is the implication of that patch?  Still reading this - https://review.openstack.org/#/c/460062/11:59
sean-k-mooneysdague: screen not being maintianed for one11:59
sean-k-mooneykashyap: the openstack service deployed by devstack are now executed via systemd instead of in screen with there logs sent to the journal12:00
kashyapsean-k-mooney: Hmm.  I like systemd journal logs (for its nice 'query-ability'), but I miss firing up a quick `screen -r` to flip through all the services12:01
sean-k-mooneykashyap: that and quickly running a service in a debugger or viewing the live logs though journalctl supports that too.12:02
kashyapHmm.  What is the rationale for the "feature"?12:02
kashyapI'd agree with mtreinish there, where he says:12:03
*** sree has quit IRC12:03
kashyap"I think expanding this comment a bit to say the intent here would have been nice, since I got confused by this at first."12:03
sean-k-mooneykashyap: make test closer to production12:03
* kashyap nods; that sounds reasonable12:04
sdaguekashyap: all sorts of stuff on the mailing list over the last month, I'd rather not just reiterate all that here12:04
kashyapThen we can _really_ claim: "If it works on DevStack, it's all kosher" :P12:04
*** yamahata has joined #openstack-nova12:04
sean-k-mooneykashyap: haha sure...12:04
kashyapsdague: Yeah, sorry, I'd go catch up the thread.  I did see the mail, but spent all of 2 seconds on it.12:04
sean-k-mooneysdague: if devstack works across reboots again this would be cool sideffect but it might cause some people to swap to other tools for dev.12:06
openstackgerritChangBo Guo(gcb) proposed openstack/nova master: Use check_string_length from oslo_utils  https://review.openstack.org/28639012:07
kashyapsean-k-mooney: But _does_ it work cope with reboots with this patch?  If so, that'd be nice12:07
sean-k-mooneysdague: anyway thanks for trying to keep it working for plugins. at least 1 of your plugins seeam to be working fine with the change ill check the otheres when i get time or delegate to the rest of my team :)12:07
*** udesale has quit IRC12:07
sean-k-mooneykashyap: dont know havent tried it12:07
sean-k-mooneyi miss rejoin stack for that abiltiy it was really useful for demoing stuff but it also had lots of issues if you didnt know how it worked12:08
*** mingyu has joined #openstack-nova12:09
openstackgerritMatthew Booth proposed openstack/nova master: WIP: Automatically revert resize which fails on destination  https://review.openstack.org/46252112:10
*** abhi89 has joined #openstack-nova12:11
*** mingyu has quit IRC12:14
*** lucasagomes is now known as lucas-hungry12:14
sdaguesean-k-mooney: you are going to need to explain what your reboot workflow is, because I only reboot my devstack environments to get rid of cruft. :) So I honestly am not sure what "works with reboots" means12:15
*** trinaths has joined #openstack-nova12:17
sean-k-mooneysdague: before rejoin_stack.sh was removed you just had to start ovs,mysql,apache and rabbit then run rejoin_stack.sh and your devstack enviorment came back up12:17
*** edmondsw has joined #openstack-nova12:17
*** smatzek has quit IRC12:17
sdaguesudo systemctl start devstack@* post boot might do that12:17
sean-k-mooneysdague: this was useful for demos as you could prepera an enviorment and snapshot it in a vm and give it to someone to try12:17
sean-k-mooneysdague: yep you are enabling the service by default i think so it might just work aslong as you dont run unstack.sh12:18
sean-k-mooneysdague: or clean.sh i guess12:18
sdagueyep12:18
*** pchavva has joined #openstack-nova12:18
sean-k-mooneyanyway it might just be a nice sideffect to get that fucntionality back12:18
sean-k-mooneyi used to jsut run in offline mode instad but that ment you still had to recreated the vms/netwoks/flavour ...12:19
sean-k-mooneythats not hard jsut extra steps12:19
*** Jack_Iv has joined #openstack-nova12:20
*** mingyu has joined #openstack-nova12:20
alex_xu_johnthetubaguy: sdague sfinucan appreciate if you can help this two easy api-ref changes https://review.openstack.org/461998 https://review.openstack.org/45873812:21
sdaguesean-k-mooney: yeh, let me know how it works. If we need some tweaks to make that use can solid for you, that would be cool12:22
*** liverpooler has joined #openstack-nova12:22
sdaguethere was so much hacktastic around getting screen to be reliable, I look forward to that code being deleted with fire12:22
*** sshnaidm has joined #openstack-nova12:25
*** kiwi_rot has joined #openstack-nova12:25
sshnaidmhas anybody seen nova errors like: AttributeError: 'Instance' object has no attribute '_save_tags'  ?12:26
*** zhurong has quit IRC12:30
*** mingyu has quit IRC12:33
johnthetubaguysshnaidm: I think I saw a bug and patch up for that somewhere12:33
johnthetubaguysshnaidm: I think gibi has this patch up: https://review.openstack.org/#/c/450265/12:34
johnthetubaguydansmith: I wasn't sure about the object change in this one actually https://review.openstack.org/#/c/450265/12:34
*** lucas-hungry is now known as lucasagomes12:34
*** mingyu has joined #openstack-nova12:35
*** ijw has joined #openstack-nova12:36
sshnaidmjohnthetubaguy, thanks, so as I understand it's not critical, right?12:36
*** ratailor has joined #openstack-nova12:37
johnthetubaguysshnaidm: I believe so12:37
*** abalutoiu has joined #openstack-nova12:38
*** eharney has joined #openstack-nova12:38
*** artom has quit IRC12:39
*** slagle__ is now known as slagle12:40
*** ijw has quit IRC12:41
*** abalutoiu has quit IRC12:41
*** ayogi has quit IRC12:44
*** sou has joined #openstack-nova12:46
souHey, I am stuck with an issue for past 2 days12:47
souI am trying a nova rebuild12:47
souand it fails all the time12:47
souwith error: ERROR oslo_messaging.rpc.dispatcher [req-b5730d47-3c34-4949-9836-01139238f829 8a205b2aa31c4c1185a417c6e99e852c fcb05e141ede4164ab6db9ad0724a08e - - -] Exception during message handling: id12:47
sou nothing else in logs12:48
southe exception is generated from /usr/lib/python2.7/site-packages/glanceclient/openstack/common/apiclient/base.py12:48
souline 494, in __getattr__12:48
souany one else has faced similar issue?12:49
souI am using Kilo version12:49
soucompute nodes uses KVM and images are lvm12:49
*** lyan has joined #openstack-nova12:50
*** slaweq has joined #openstack-nova12:50
*** cfriesen has joined #openstack-nova12:52
johnthetubaguysou: please try #openstack, see channel topic12:52
*** ratailor_ has joined #openstack-nova12:52
souThanks @johnthetubadguy12:53
*** Yangyapeng has joined #openstack-nova12:53
*** kiwi_rot has quit IRC12:54
*** yamamoto has joined #openstack-nova12:54
*** slaweq has quit IRC12:54
*** ratailor has quit IRC12:56
*** brault has joined #openstack-nova12:56
*** brault_ has quit IRC12:57
*** thorst is now known as thorst_afk12:59
*** vladikr has joined #openstack-nova13:12
*** toure|gone is now known as toure13:13
*** jamesdenton has joined #openstack-nova13:13
*** karimb has joined #openstack-nova13:13
*** mingyu has quit IRC13:14
*** iceyao_ has joined #openstack-nova13:15
*** iceyao has quit IRC13:15
*** mingyu has joined #openstack-nova13:16
*** rfolco has joined #openstack-nova13:20
*** ratailor_ has quit IRC13:21
*** yamamoto has quit IRC13:22
*** swebster_ has quit IRC13:23
*** markvoelker has joined #openstack-nova13:23
*** yamamoto has joined #openstack-nova13:24
*** mriedem has joined #openstack-nova13:24
*** mriedem has left #openstack-nova13:25
*** mriedem has joined #openstack-nova13:25
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for server-tags instead of stevedore  https://review.openstack.org/46164513:26
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-interface endpoint instead of stevedore  https://review.openstack.org/45997913:26
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for server-migrations endpoint instead of stevedore  https://review.openstack.org/46164613:26
abhi89johnthetubaguy: can we back port this fix https://review.openstack.org/#/c/460305/ to stable/ocata.. we were looking to have this change in this release in our product..13:30
johnthetubaguyabhi89: I think that should be OK13:30
dansmithjohnthetubaguy: yes, as you said, and as convention in the code, the empty save handler is the right approach13:31
abhi89johnthetubaguy: grt.. i will cherry pick the fix to ocata..13:33
gibimriedem: hi! I could not attend on the nova meeting today, I left small notes on the agenda13:33
mriedemgibi: ok thanks13:33
*** crushil_ has quit IRC13:36
*** gouthamr has joined #openstack-nova13:38
*** kiwi_rot has joined #openstack-nova13:41
*** zhurong has joined #openstack-nova13:41
*** Jack_Iv has quit IRC13:43
*** Jack_Iv has joined #openstack-nova13:43
*** esberglu has joined #openstack-nova13:44
mriedemgibi: do we need to call out https://review.openstack.org/#/c/401992/ when it's approved?13:45
mriedemmelwitt: sdague: dansmith: looks like the use_journal=True thing was causing the cells job failures http://status.openstack.org/elastic-recheck/#158072813:46
mriedemwe should still get https://review.openstack.org/#/c/461912 in though13:47
*** mingyu has quit IRC13:47
*** mingyu has joined #openstack-nova13:48
dansmithmriedem: okay, I've yet to see my devstack patch fail with journald remoed13:48
dansmithwell, fail for a related reason anyway13:48
abhi89johnthetubaguy: please review cherry picked fix https://review.openstack.org/#/c/462561/ for stable/ocata13:49
*** xyang1 has joined #openstack-nova13:49
*** zhurong_ has joined #openstack-nova13:50
*** zhurong has quit IRC13:51
*** yamahata has quit IRC13:51
*** lucasxu has joined #openstack-nova13:52
*** mlavalle has joined #openstack-nova13:53
*** baoli has joined #openstack-nova13:55
*** sou has quit IRC13:55
*** awaugama has joined #openstack-nova13:55
mriedemnova meeting in 4 minutes in #openstack-meeting13:56
*** bkopilov has quit IRC13:56
*** takashin has joined #openstack-nova13:56
*** vks1 has joined #openstack-nova13:56
*** jamielennox|away is now known as jamielennox13:58
*** crushil_ has joined #openstack-nova13:59
*** mariojv_ is now known as mariojv14:00
*** jaypipes has joined #openstack-nova14:00
*** cleong has joined #openstack-nova14:03
*** sam___ has joined #openstack-nova14:06
*** brault has quit IRC14:06
*** sam___ has left #openstack-nova14:09
*** ekuris has quit IRC14:09
*** prateek has quit IRC14:09
*** zhurong_ has quit IRC14:12
*** armax has joined #openstack-nova14:12
*** moshele has quit IRC14:14
*** liuyulong has joined #openstack-nova14:14
*** coreywright has quit IRC14:16
*** zz_dimtruck is now known as dimtruck14:18
*** catintheroof has joined #openstack-nova14:20
*** catinthe_ has joined #openstack-nova14:23
*** catintheroof has quit IRC14:24
*** coreywright has joined #openstack-nova14:28
*** markvoelker has quit IRC14:31
*** iceyao_ has quit IRC14:32
*** peter-hamilton has joined #openstack-nova14:32
mtreinishkashyap: fwiw, I was talking about the grenade line there. It's not 100% obvious that it's there for making sure systemd isn't used on grenade14:32
mtreinishI had to think it through out loud to understand the logic in that if14:33
kashyapmtreinish: Noted. :-)14:33
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.updateprop notification  https://review.openstack.org/46257614:33
*** kylek3h has joined #openstack-nova14:33
*** nammo has joined #openstack-nova14:34
nammoHi How can I remove a RetryFilter?14:34
nammo I removed it from nova.conf but it still exists14:35
bauzasmanjeets__: pull it out from enabled_filters14:37
bauzasoops14:38
bauzasnammo: ^14:38
* bauzas isn't drunk14:38
*** takashin has left #openstack-nova14:38
*** stvnoyes has quit IRC14:38
*** armax has quit IRC14:38
openstackgerritGábor Antal proposed openstack/nova master: [WIP] Transform aggregate.update_metadata notification  https://review.openstack.org/46062514:38
nammobauzas: Thanks for your response14:38
nammoI did it but it still show up in logs14:38
*** tbachman_ has joined #openstack-nova14:38
openstackgerritJay Pipes proposed openstack/nova master: placement: tests for non-shared with shared  https://review.openstack.org/46189914:39
openstackgerritJay Pipes proposed openstack/nova master: placement: test for agg association not sharing  https://review.openstack.org/46190114:39
openstackgerritJay Pipes proposed openstack/nova master: placement: test non-shared out of inventory  https://review.openstack.org/46190014:39
openstackgerritJay Pipes proposed openstack/nova master: placement: shared resources when finding providers  https://review.openstack.org/46157414:39
openstackgerritJay Pipes proposed openstack/nova master: placement: get providers sharing capacity  https://review.openstack.org/46079814:39
jaypipesdansmith: fixed that silly copy/paste error in the functional test ^^14:39
jaypipesdansmith: thx for your reviews!14:39
dansmithjaypipes: did you see the rest? I think I had a -1 on all of them14:40
*** stvnoyes has joined #openstack-nova14:40
nammobauzas: I restarted n-sch, n-cond14:40
nammoIs that sufficient?14:40
bauzasnammo: only n-sch14:40
*** tbachman has quit IRC14:40
*** tbachman_ is now known as tbachman14:40
nammobauzas: No it is not working for me :(14:41
*** voelzmo has quit IRC14:42
*** aloga has quit IRC14:42
*** aloga has joined #openstack-nova14:43
*** gongysh has joined #openstack-nova14:44
*** yamamoto has quit IRC14:44
*** annegentle has joined #openstack-nova14:44
*** lpetrut_ has quit IRC14:44
*** voelzmo_ has joined #openstack-nova14:44
nammobauzas: http://paste.openstack.org/show/608874/14:44
*** crushil_ has quit IRC14:44
jaypipesdansmith: I've fixed some issues. with regards to the long code comment, where do you want me to put that stuff?14:46
dansmithjaypipes: I dunno, it was just an idea.. I'd like to avoid moving the actual db query code elsewhere14:47
bauzasjaypipes: FWIW, I was reviewing your bottom change, but I was thinking about using Workbench14:47
jaypipesdansmith: right... I moved that code into the module-level scope per your request.14:47
bauzasjaypipes: because $sql14:48
dansmithjaypipes: yep, I saw, and I appreciate that. I'm referring to edleafe and cdent's desire to see it moved farther away14:48
jaypipesdansmith: oh. sorry, do you agree with that?14:48
*** claudiub has quit IRC14:49
dansmithjaypipes: agree with what? I just said I didn't want to see that code move out, so if we need to make things smaller, maybe we should move the comments out14:49
*** tbachman has quit IRC14:49
*** adisky_ has quit IRC14:49
*** gongysh has quit IRC14:49
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.update_metadata notification  https://review.openstack.org/46062514:50
cdentI'm not wed to the idea of moving it out, is fine in the same file, but I find huge python files difficult to think about14:51
*** slaweq has joined #openstack-nova14:51
jaypipesdansmith: right, and I was asking where you'd like to move the SQL comments.14:51
openstackgerritJay Pipes proposed openstack/nova master: placement: tests for non-shared with shared  https://review.openstack.org/46189914:51
openstackgerritJay Pipes proposed openstack/nova master: placement: test for agg association not sharing  https://review.openstack.org/46190114:51
openstackgerritJay Pipes proposed openstack/nova master: placement: test non-shared out of inventory  https://review.openstack.org/46190014:51
openstackgerritJay Pipes proposed openstack/nova master: placement: shared resources when finding providers  https://review.openstack.org/46157414:51
dansmithjaypipes: and I said "I dunno" :P14:52
dansmithjaypipes: my preference would be to not move things out of that file14:52
*** tongli has joined #openstack-nova14:52
jaypipesdansmith: ah, sorry, I missed that. :)14:52
bauzascdent: jaypipes: dansmithwhat needs to move ?14:52
bauzasI'm fine with leaving ORM code as it is14:52
bauzasif that's the point14:53
*** smatzek has joined #openstack-nova14:53
*** markvoelker has joined #openstack-nova14:53
*** slaweq has quit IRC14:55
*** salv-orl_ has joined #openstack-nova14:56
*** armax has joined #openstack-nova14:56
jaypipesbauzas: the discussion is not around the ORM code but rather there is a discussion about how long the code comment with SQL is on that function.14:58
bauzaswell14:58
bauzasokay14:58
bauzasnot sure it's a problme14:58
bauzaswhen I wrote the long docstring, I also fixed the comment when I changed the calls14:58
bauzasthat's why I'd like to at least pdb your change for verifying the SQL query14:59
*** salv-orlando has quit IRC14:59
mriedemwon't we want the big ass sql comment for the complicated orm code?14:59
mriedemas dansmith said earlier in the week, this is all fine but i won't be debugging it :)14:59
bauzasit's just part of a reviewer duty /me shrugs14:59
dansmiththere is no question about ORM stuff AT ALL15:00
dansmithit's about where to put these comments about the sql15:00
*** hongbin has joined #openstack-nova15:00
bauzastbc, I'm fine with the current situation15:00
bauzashaving large docstrings is good wit hme15:00
*** yingjun has joined #openstack-nova15:00
bauzasanyway, /me needs to take my daughter \o15:00
*** dillaman has quit IRC15:01
nammojaypipes: dansmith Hi guys! Can you please help me with removing a filter?15:05
peter-hamiltonmriedem: just wanted to put this on your radar - https://review.openstack.org/#/c/357202/15:05
nammoI removed it from enabled_filters, restarted n-sch but still it exists15:06
jaypipesnammo: give us 15-25 minutes. we're currently on a call.15:06
nammoOh ok sorry.15:06
jaypipesno worries :)15:06
peter-hamiltonmriedem: danefichter and i want more nova eyes on it before we go any further15:06
*** jerrygb has joined #openstack-nova15:07
*** claudiub has joined #openstack-nova15:07
*** crushil_ has joined #openstack-nova15:08
mtreinishjohnthetubaguy: what were you looking for on: https://review.openstack.org/#/c/459470 ? The comments are a bit vague on what's missing (its more of a discussion)15:10
johnthetubaguymtreinish: good question, I have a feeling I got distracted and moved to a different review15:10
*** Oku_OS is now known as Oku_OS-away15:11
*** kiwi_rot has quit IRC15:12
*** kaisers has joined #openstack-nova15:13
johnthetubaguymtreinish: you know, the only nit was that config that I think we can just remove, but I have a feeling mriedem wasn't 100% sure about the remove, lets worry about that in a follow up patch15:14
*** dillaman has joined #openstack-nova15:15
*** hshiina has joined #openstack-nova15:15
*** karimb has quit IRC15:17
mtreinishjohnthetubaguy: yeah, that's why I added the additional information to the help instead of deleting it15:18
*** kaisers has quit IRC15:18
*** iceyao has joined #openstack-nova15:18
*** brault has joined #openstack-nova15:19
*** karimb has joined #openstack-nova15:20
*** tbachman has joined #openstack-nova15:20
*** stvnoyes has quit IRC15:22
*** iceyao has quit IRC15:22
*** claudiub|2 has joined #openstack-nova15:23
*** stvnoyes has joined #openstack-nova15:23
*** ijw has joined #openstack-nova15:23
*** brault has quit IRC15:24
*** abhi89 has quit IRC15:24
*** kfarr has joined #openstack-nova15:24
*** armax has quit IRC15:25
*** tbachman has quit IRC15:25
*** aarefiev is now known as aarefiev_afk15:27
*** claudiub has quit IRC15:27
*** tbachman has joined #openstack-nova15:27
openstackgerritMatthew Treinish proposed openstack/python-novaclient master: Remove the deprecated certs interfaces  https://review.openstack.org/46259815:28
mtreinishmriedem: ^^^15:28
mtreinishI'll work on the cloudpipe client deprecation later15:28
*** moshele has joined #openstack-nova15:30
*** marst has joined #openstack-nova15:31
*** vks1 has quit IRC15:33
*** mdnadeem has quit IRC15:33
*** dpawlik has quit IRC15:34
*** chyka has joined #openstack-nova15:35
jaypipesnammo: so, which filter did you try removing?15:35
*** liuyulong has quit IRC15:36
*** moshele has quit IRC15:37
*** iceyao has joined #openstack-nova15:39
openstackgerritRoman Podoliaka proposed openstack/nova master: service: use restart_method='mutate' for all services  https://review.openstack.org/46260015:41
*** andreas_s has quit IRC15:42
*** yamamoto has joined #openstack-nova15:45
*** shaohe_feng has quit IRC15:46
*** voelzmo_ has quit IRC15:46
*** alex_xu_ has quit IRC15:46
*** heyongli has quit IRC15:46
*** voelzmo has joined #openstack-nova15:46
*** manjeets__ is now known as manjeets15:48
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/44925715:49
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177715:49
*** Jack_Iv has quit IRC15:49
*** gyee has joined #openstack-nova15:50
*** claudiub|2 has quit IRC15:51
*** damien_r1 has quit IRC15:51
mriedempeter-hamilton: it's going to be hard today, lots of meetings15:52
dansmithedleafe: think you could address some of the feedback that has been sitting on these patches for a few days? https://review.openstack.org/#/c/46017715:52
mriedemmtreinish: you can't remove the nova cert CLI stuff until queens15:52
mriedemmtreinish: we need to deprecate the cloudpipe API bindings and CLIs in novaclient like i did for the os-hosts stuff though15:53
*** Jack_Iv has joined #openstack-nova15:53
*** lucasxu has quit IRC15:53
mriedemjohnthetubaguy: oh the vpn_ip?15:53
dansmithedleafe: I haven't seen any replies from bauzas so I assume he's busy with other stuff15:53
mriedemyeah i wasn't sure15:53
*** jaosorior has quit IRC15:54
johnthetubaguymriedem: yeah, I figured mtreinish updated the description, we can always rip it out later if that turns out to be the right call.15:54
*** Apoorva has joined #openstack-nova15:54
edleafedansmith: I know he was on holiday earlier this week15:55
dansmithedleafe: yeah15:55
edleafebauzas: still around?15:55
*** lucasxu has joined #openstack-nova15:56
mriedemhe's picking up his daugher from school i think15:56
edleafedansmith: I mean, I'd be happy to work on this, but didn't want to step on bauzas's toes15:56
edleafemriedem: what, are they walking? :)15:56
*** lucasxu has quit IRC15:56
dansmithedleafe: yeah, it's been days though so I figured he's just busy, but cool to check15:56
mriedemedleafe: probably an old fashioned bike would be my guess15:56
*** lucasxu has joined #openstack-nova15:56
mriedemwith cigs and croissants15:57
*** yamamoto has quit IRC15:57
dansmithbut if you're willing I would think some of it could be shared at least15:57
edleafedansmith: that's fine with me15:57
*** cfriesen has quit IRC15:57
*** cfriesen has joined #openstack-nova15:57
edleafedansmith: I'll start on it after the api-wg meeting, unless bauzas speaks up first15:58
nammojaypipes: RetryFilter15:58
dansmithedleafe: cool, thanks15:58
jaypipesnammo: that cannot be removed, AFAIK.15:59
*** voelzmo has quit IRC15:59
edleafenammo: why would you want to remove that?15:59
*** lucasxu has quit IRC15:59
melwittit used to be able to be removed15:59
nammojaypipes: edleafe Any consecutive request to boot are failing due to RetryFilter15:59
*** slaweq has joined #openstack-nova16:00
edleafenammo: RetryFilter only comes into play when a request to build on a host has already failed16:00
jaypipesmelwitt: really? TIL...16:01
nammoedleafe: I deleted that VM and tried to boot again16:01
melwittwe had removed it because of the integration with other systems we had done, vast majority of failures were deterministic and retrying a different compute wasn't going to help and would waste time, so we didn't have RetryFilter in our list and set scheduler_max_attempts = 116:01
*** lucasxu has joined #openstack-nova16:02
nammomelwitt: Is it intended for me?16:02
melwittnammo: no. just trivia on one reason someone would remove it16:03
dansmithyay for more reasons to not do scheduler retries16:04
nammomelwitt: scheduler_max_attempts  This should be under default section?16:04
mtreinishmriedem: oh I thought the deprecation timer started long enough ago. Well I'll just wip it to block the merge until after the release16:05
*** iceyao has quit IRC16:06
melwittnammo: these days it's [scheduler]/max_attempts. but the fact that you see "Previously tried hosts" in your log still means the RetryFilter is still there, which doesn't make sense if you removed it from enabled_filters and restarted n-sch16:06
nammomelwitt: So is there any other solution to launch VM on the same host?16:07
nammomelwitt: host is up and available16:07
melwittnammo: what are you trying to do? the RetryFilter just retries on a different host if the selected host fails. you're trying to get multiple VMs to land on the same host?16:08
*** nic has joined #openstack-nova16:08
nammomelwitt: I just have one node that satifies my VM spec16:09
nammomelwitt: I will see the sched logs to see why RetryFilter failed16:10
melwittthat's usually what you want with a vanilla nova deployment, is to have retries. because if the host the scheduler selected fails to spawn the instance, it's usually best to try a different host next16:10
mriedemmtreinish: it's going to be sitting quite awhile if you do that, butok16:10
nammomelwitt: Agree16:10
nammoBut for me it just one host that my VM can launch on16:10
melwittnammo: I see. you should check the sched logs to determine why the first attempt failed. usually means not enough resources or extra_specs didn't match or something like that. if you turn on debug log level for n-sch you will see all the detail of which filters it tried and which one failed16:11
*** mvk has quit IRC16:11
*** lucasxu has quit IRC16:15
*** iceyao has joined #openstack-nova16:18
nammomelwitt: http://paste.openstack.org/show/608889/16:21
nammomelwitt: I see no error here16:25
nammomelwitt: Can you please help?16:25
openstackgerritDan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test  https://review.openstack.org/46151316:26
*** lucasagomes is now known as lucas-afk16:26
*** yamamoto has joined #openstack-nova16:26
*** salv-orl_ has quit IRC16:26
dansmithnammo: please see topic, this channel is for development16:26
openstackgerritDan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path  https://review.openstack.org/45863616:27
openstackgerritDan Smith proposed openstack/nova master: Move to proper target_cell calling convention  https://review.openstack.org/45863416:27
openstackgerritDan Smith proposed openstack/nova master: Make target_cell() yield a new context  https://review.openstack.org/45863516:27
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853716:27
nammodansmith: I see. Thanks16:28
dansmithjaypipes: is the reason that first test worked because of the trait?16:28
melwittnammo: to close the loop on this, agree there's no problem in n-sch and you need to check your n-cpu logs for why it failed to spawn the instance on the selected host16:29
mriedemdansmith: your access has been...granted16:29
dansmithmriedem: thanks16:29
*** ijw has quit IRC16:30
nammomelwitt: Thanks :)16:30
*** ijw has joined #openstack-nova16:31
*** iceyao has quit IRC16:32
*** iceyao has joined #openstack-nova16:32
*** iceyao has quit IRC16:33
*** mrhillsman has quit IRC16:35
*** wxy has quit IRC16:35
*** aweeks has quit IRC16:35
*** test222 has quit IRC16:35
*** ijw has quit IRC16:35
*** yamamoto has quit IRC16:37
*** pcaruana has quit IRC16:40
*** aweeks has joined #openstack-nova16:41
*** ijw has joined #openstack-nova16:41
mriedembauzas: jaypipes: edleafe: you guys will likely want to see this https://review.openstack.org/#/c/394857/ since it has to do with overhead calculation and overrides in the RT and virt drivers16:42
*** test222 has joined #openstack-nova16:43
edleafemriedem: just reading the commit message on that seems like it's a misunderstanding16:43
mriedemi'm -1ing now16:44
*** stvnoyes has quit IRC16:45
mriedemedleafe: done - you can proofread it16:45
mriedem:)16:45
*** wxy has joined #openstack-nova16:46
*** mrhillsman has joined #openstack-nova16:46
*** armax has joined #openstack-nova16:47
mriedembauzas: replied in https://review.openstack.org/#/c/454899/ - that's how the online data migrations were done for adding uuid to aggregates and compute_nodes tables,16:48
mriedemnot sure that's worth a -116:48
mriedembut maybe others have strong opinions16:48
*** erlon has joined #openstack-nova16:49
dansmithmriedem: I kinda like them in the object file, but I don't think you should change it just for that16:49
*** lucasxu has joined #openstack-nova16:49
*** imacdonn has quit IRC16:50
edleafemriedem: added my own -1 and explanation16:50
*** imacdonn has joined #openstack-nova16:50
*** moshele has joined #openstack-nova16:52
jaypipesdansmith: yes. first test passed because of the trait.16:52
dansmithjaypipes: yeah, ack16:52
*** fragatina has quit IRC16:54
*** moshele has quit IRC16:55
openstackgerritJackie Truong proposed openstack/nova master: Encode encryption key passed to dmcrypt  https://review.openstack.org/46267416:58
*** psachin has joined #openstack-nova16:58
*** baoli has quit IRC16:59
*** stvnoyes has joined #openstack-nova16:59
openstackgerritJay Pipes proposed openstack/nova master: placement: tests for non-shared with shared  https://review.openstack.org/46189917:00
openstackgerritJay Pipes proposed openstack/nova master: placement: test for agg association not sharing  https://review.openstack.org/46190117:00
openstackgerritJay Pipes proposed openstack/nova master: placement: test non-shared out of inventory  https://review.openstack.org/46190017:00
openstackgerritJay Pipes proposed openstack/nova master: placement: shared resources when finding providers  https://review.openstack.org/46157417:00
openstackgerritJay Pipes proposed openstack/nova master: placement: get providers sharing capacity  https://review.openstack.org/46079817:00
jaypipesdansmith, edleafe, cdent: ok, I believe I've responded or fixed all your review comments on above series.17:01
*** baoli has joined #openstack-nova17:01
*** kfarr has quit IRC17:01
jaypipesother than the moving of the long SQL code comment somewhere...17:01
dansmithjaypipes: gdi jay, you pushed too fast, I wanted to claim that review credit for my commit message nit :P17:01
jaypipeswe can bikeshed about where to move that later, ok?17:01
*** derekh has quit IRC17:01
jaypipesheh, sorry dansmith17:02
*** ralonsoh has quit IRC17:02
dansmith*shakes fist*17:02
dansmithmriedem is calling me out BY NAME in public so I have to up my game17:02
edleafejaypipes: for the record, I like the comment within the method. I just don't like a module-level method in the middle of the object class definitions17:02
*** karimb has quit IRC17:02
*** arunman has quit IRC17:03
jaypipesedleafe: noted. :)17:03
cdentblue17:03
dansmithheh17:03
*** hieulq_ has joined #openstack-nova17:05
*** hshiina has quit IRC17:07
* cdent goes to the polls to do his civic duty17:07
*** litao has quit IRC17:09
*** crushil_ has quit IRC17:10
*** jpena is now known as jpena|off17:12
*** markus_z has quit IRC17:12
dansmithjaypipes: in SQLandia, if I to a SELECT * WHERE longstring="uuidthing",17:13
dansmithis that much more efficient than doing an IN check for a list of one?17:13
*** fragatina has joined #openstack-nova17:13
openstackgerritEd Leafe proposed openstack/nova master: WIP: Don't delete allocation if instance being scheduled  https://review.openstack.org/46017717:13
openstackgerritEd Leafe proposed openstack/nova master: Add a convenience attribute for reportclient  https://review.openstack.org/46267917:13
edleafedansmith: ^^17:13
dansmithmeaning SELECT * WHERE longstring IN ("uuidthing") or whatever the syntax is17:13
dansmithedleafe: woot17:13
edleafedansmith: just addressed the comments. Will work on the rest now17:13
mriedemon an indexed column17:13
dansmithmriedem: ah, yeah, good point the index17:13
*** ltomasbo is now known as ltomasbo|away17:14
mriedemdansmith: well, after https://review.openstack.org/#/c/454887/ that is17:14
jaypipesdansmith: all optimizers I know of will rewrite the latter to the former.17:14
dansmithjaypipes: ah, okay17:14
dansmithactually, I think I remember seeing warnings about doing that17:14
*** gjayavelu has joined #openstack-nova17:17
mriedemdansmith: if you see the patch after it, i could have ComputeNode.get_by_uuid call ComputeNodeList.get_all_by_uuids and internally handle the [uuid] and [0], but i just wondered about the IN performance17:18
*** baoli has quit IRC17:18
dansmithmriedem: yeah17:19
*** baoli has joined #openstack-nova17:19
mriedembut maybe this is a small enough difference that it doesn't really matter and we could just squash those two changes, and just hide it all in the object code17:20
dansmithyeah, seems like it might be17:20
*** psachin has quit IRC17:20
mriedemhttp://stackoverflow.com/questions/9926433/mysql-performance-in-clause-vs-equals-for-a-single-value17:21
mriedem"There is no difference between the MySQL statements, and the MySQL  optimiser will transform the IN to the = when IN is just one element.  Don't bother."17:21
mriedemso what jaypipes said17:21
openstackgerritDan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path  https://review.openstack.org/45863617:22
openstackgerritDan Smith proposed openstack/nova master: Move to proper target_cell calling convention  https://review.openstack.org/45863417:22
openstackgerritDan Smith proposed openstack/nova master: Make target_cell() yield a new context  https://review.openstack.org/45863517:22
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853717:22
*** jaypipes has left #openstack-nova17:24
*** jaypipes has joined #openstack-nova17:24
*** lpetrut has joined #openstack-nova17:24
peter-hamiltonmriedem: sorry, i was out - not a problem, i just wanted to put it in the queue :)17:25
*** trinaths has left #openstack-nova17:26
*** ijw has quit IRC17:27
*** igordcar1 has quit IRC17:27
*** igordcard has joined #openstack-nova17:27
*** tesseract has quit IRC17:30
*** hieulq_ has quit IRC17:37
*** hieulq_ has joined #openstack-nova17:39
*** crushil_ has joined #openstack-nova17:41
*** mingyu has quit IRC17:41
*** gyee has quit IRC17:45
*** baoli_ has joined #openstack-nova17:47
*** yingjun has quit IRC17:48
*** karthiks has joined #openstack-nova17:48
*** baoli has quit IRC17:49
*** smatzek has quit IRC17:51
*** fragatina has quit IRC17:54
*** fragatina has joined #openstack-nova17:54
openstackgerritMatt Riedemann proposed openstack/nova master: db api: add service_get_by_uuid  https://review.openstack.org/46142817:55
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service  https://review.openstack.org/46151917:55
*** moshele has joined #openstack-nova17:55
openstackgerritMatt Riedemann proposed openstack/nova master: objects: add Service.get_by_uuid method  https://review.openstack.org/46142917:55
openstackgerritMatt Riedemann proposed openstack/nova master: objects: add ComputeNode.get_by_uuid method  https://review.openstack.org/46144817:55
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get  https://review.openstack.org/46153217:55
openstackgerritMatt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid  https://review.openstack.org/45489917:55
openstackgerritMatt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload  https://review.openstack.org/46147317:55
*** Apoorva has quit IRC17:56
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service  https://review.openstack.org/46151917:57
openstackgerritMatt Riedemann proposed openstack/nova master: objects: add ComputeNode.get_by_uuid method  https://review.openstack.org/46144817:57
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get  https://review.openstack.org/46153217:57
openstackgerritMatt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload  https://review.openstack.org/46147317:57
dansmithmriedem: pushing the same set three times in a row is fooling nobody17:58
mriedem2 times in a row17:58
dansmithN times in a row17:58
*** Sukhdev has joined #openstack-nova18:03
*** lpetrut has quit IRC18:03
mriedemgah, plus i screwed up the topic branch18:04
efriedmriedem mordred jaypipes [service catalog bp] I'd like to settle this issue of determining interface if not otherwise specified.18:05
mriedempistols at dawn it is18:05
efriedAt some point in the early (at least in my involvement) discussions, there was talk of trying the known ones in order.18:05
*** moshele has quit IRC18:05
efriedThis is a dict lookup, so that's cheap.18:06
mriedemmaybe the idea with starting using internal first is to avoid defaulting to public in ksa?18:06
efriedBut on the other side, there's "use the default from ksa"18:06
efried...which is "public", for whatever that's worth.18:06
efriedCan any of you remember the rationale behind the "loop through 'em" suggestion?18:06
mriedemefried: that was mordred or jaypipes' idea i thought, but presumably to hit the internal/admin urls before public18:07
efriedmriedem Yeah, starting with "internal" seemed like a more... uh... "secure" idea.18:07
mriedemsince today ops are probably using internal/admin18:07
efriedOkay, right now I've got internal->public->admin; that's an easy swap.  But question remains: do we want to do the loop thing at all?18:08
mriedemi don't really feel strongly either way, looping over a hard-coded list which might not match what's actually in the service catalog seems weird to me18:09
*** yonglihe has quit IRC18:09
efriedmriedem Yeah, that makes me a tad uncomfortable too.  Maybe mordred can point us to a way to get that list from ksa.18:09
*** abhi89 has joined #openstack-nova18:10
mriedemhttps://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L17318:10
mriedemthe docs say one thing but i'm pretty sure interface is open ended18:11
mriedemalthough at one point i thought i found json schema in the keystone api for v3 that restricted the interfaces18:11
mriedemto a known set18:11
efriedOne approach is to say, we try these three, and if it ain't one of those, they better specify it.18:13
efriedThey can always specify it.18:13
mriedemefried: https://github.com/openstack/keystone/blob/master/keystone/catalog/schema.py#L9418:14
efriednoyce.18:14
mriedemso the interface types are validated via jsonschema in the keystone api, at least for the v3 api i think18:14
mriedemso iterating through admin/internal/public probably seems fine18:15
*** dtp has joined #openstack-nova18:15
*** zerda3 has quit IRC18:15
efriedWhat order would you like?18:15
*** zerda3 has joined #openstack-nova18:15
mriedemdon't ask me :)18:16
mriedemask operators18:16
mriedem#openstack-operators18:16
abhi89dansmith, jaypipes: please review https://review.openstack.org/#/c/462561/.. its for stable/ocata merge..18:17
*** slaweq has quit IRC18:17
*** slaweq has joined #openstack-nova18:18
*** ociuhandu has quit IRC18:18
*** harlowja has quit IRC18:19
*** Sukhdev has quit IRC18:20
*** nammo has quit IRC18:21
*** smatzek has joined #openstack-nova18:22
*** slaweq has quit IRC18:23
*** smatzek has quit IRC18:30
*** cdent has quit IRC18:35
*** baoli_ has quit IRC18:37
*** baoli has joined #openstack-nova18:38
*** gszasz has quit IRC18:38
*** mingyu has joined #openstack-nova18:41
mriedemmelwitt: looks like we have a KeyError when the tests run now because of ^ https://review.openstack.org/#/c/460305/6/nova/tests/unit/compute/test_compute.py18:43
mriedemhttps://bugs.launchpad.net/nova/+bug/168836618:47
openstackLaunchpad bug 1688366 in OpenStack Compute (nova) "KeyError: u'overs' in unit test runs" [Low,Triaged]18:47
melwittmriedem: sigh ... I miss that config option that used to make format errors fatal for tests18:47
mriedemi don't know why that was removed18:47
*** cdent has joined #openstack-nova18:47
mriedemmaybe it was too useful18:47
melwittyeah, really. it was removed as part of the centralize-config-options work, and I tried to recreate the behavior with wonky monkey patching and wasn't successful https://review.openstack.org/#/c/41161518:48
*** Apoorva has joined #openstack-nova18:50
dansmith18448918:52
dansmithheh sorry18:52
melwittthere's been something in the unit tests spewing garbled text into the output for months now, I assume that's another KeyError some place in a test exception18:53
*** peter-hamilton has quit IRC18:54
*** amotoki has quit IRC18:55
openstackgerritEric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa  https://review.openstack.org/46148118:56
efriedmriedem ^^18:56
openstackgerritSteve Noyes proposed openstack/nova master: Add Cinder v3 detach to shutdown_instance  https://review.openstack.org/45687719:05
dansmithmelwitt: I wonder if I'm failing this test because of legit lack of cells support in consoleauth? http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/testr_results.html.gz19:08
* dansmith goes hunting19:08
melwittdansmith: I would guess so. looking to see if we do a target cell for getting the vnc console19:12
melwittI can't remember what the state of the compute rpcapi mq switching is19:13
dansmithyeah, I think it might actually be in the proxy thing19:13
*** gouthamr has quit IRC19:14
melwittin nova/api/openstack/compute/remote_consoles.py it gets an instance and passes it down to get_vnc_console, so if the context has already been targeted by the get_instance, it should work19:16
*** ociuhandu has joined #openstack-nova19:17
dansmithmelwitt: right, but that's to get the url, right? then we actually connect through the novncproxy to get to the rfb data,19:17
dansmithand it has to know where to proxy to, which is what is failing I think19:18
dansmithit makes a consoleauth rpc call, which may be going to the wrong place, I'm guessing19:18
melwittoh, yeah. must be19:18
*** abhi89 has quit IRC19:18
dansmithalthough I don't see anything barfing about an rpc timeout or anything19:18
melwittyeah, dtp's change made the consoleauth calls stay at the API and that call shouldn't try to mq switch19:19
melwittI can never remember how the consoles work. get the url first, now looking for where it does the consoleauth part19:20
dansmithyeah I have to re-learn each time19:21
melwittokay, the consoleauth is in compute/api, in get_vnc_console, so that should be fine I think19:22
dansmiththat's the thing that authorizes, not what checks right?19:23
dansmithyeah19:23
dansmithso we go there, create a token, return a url19:23
*** kaisers_ has quit IRC19:23
dansmiththen we connect to the proxy, do a websocket and expect rfb data19:23
*** kaisers has joined #openstack-nova19:23
dansmithbut we have to check the token in there again, and we do a rpc call to do it19:23
dansmithhttps://github.com/openstack/nova/blob/master/nova/console/websocketproxy.py#L105-L10619:24
*** cfriesen has quit IRC19:24
*** gouthamr has joined #openstack-nova19:24
melwittokay, proxy is running at the API, so is consoleauth, seems like it should be okay19:25
dansmithbut the tokens are in the cell db, no?19:25
* dansmith forgets19:25
melwitttokens are in memcached19:25
melwittwhy are there no errors in the logs anywhere :|19:26
dansmithoh, heh, and so the consoleauth service is just there to ask memcache for us for checking tokens?19:27
dansmithyeah, that's the weird thing19:27
melwittthat's my understanding, yeah19:27
*** hieulq_ has quit IRC19:28
dansmithyeah, looks like it19:28
dansmithwtf19:28
dansmithoh no,19:28
dansmithit also calls to compute19:28
*** pchavva has quit IRC19:28
dansmithto validate a port19:28
dansmithbut it does the mapping lookup and the target19:29
dansmithso yeah seems like maybe no logs because it's mostly working19:29
melwittI dunno if this is relevant but I don't see a log in consoleauth for the test you're failing http://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/screen-n-cauth.txt.gz?#_May_04_18_01_45_09178219:29
*** dave-mccowan has quit IRC19:30
dansmithI do see an authorize console report though19:30
dansmithnot for that test though yeah19:31
melwittyeah, based on the project/user names from the context19:31
*** smatzek has joined #openstack-nova19:31
dansmithso, I see the corresponding things in n-api.log for that server actions test,19:33
dansmithbut nothing for the vnc test19:33
dansmithdub tee eff19:33
*** kaisers_ has joined #openstack-nova19:33
melwittI can't even find that server uuid in n-api log19:34
dansmith        Body: {"console": {"url": "http://198.72.124.114:6080/vnc_auto.html?token=8bfe6807-a8be-44de-bca7-d9087fb2da4a", "type": "novnc"}}19:34
dansmithsure looks like it was called by tempest though19:34
dansmithhmm, yeah that's super bizarre19:34
melwittyeah, like, wtf19:34
dansmiththat server uuid is nowhere in the logs19:34
dansmiththis finished at noon, so it has to be without the journal usage19:35
melwittI found it a few times in n-cpu, but nothing about it spawning19:35
*** kaisers has quit IRC19:35
melwittjust accounting stuff19:36
dansmitho.O19:36
dansmithI also noticed what looked like a few restarts of the api service at the top of the log19:36
dansmithlike in the tripleo logs, but that's because of puppet I think19:37
dansmithso I wonder if we have an api worker that isn't actually logging19:37
*** baoli has quit IRC19:37
bauzasdansmith: edleafe: jaypipes: FWIW, I was writing some UTs for the scheduler-claims19:37
*** baoli has joined #openstack-nova19:38
dansmithmriedem: maybe those are the two api worker threads starting actually19:38
jaypipesbauzas: cool. will review promptly when you push.19:38
bauzasedleafe: could you pease upload your last point for https://review.openstack.org/#/c/460177/ for leaving me providing UTs ?19:38
dansmithsdague: can you think of any reason why we might be missing logs from one of the api workers or something like that?19:39
melwittdansmith: I dunno, on the surface it looks like we have to be missing some logs19:39
dansmithmelwitt: yeah, seems like impossible that we're not missing logs19:39
mriedemdansmith: haven't been following - did you pull the journal down and check there?19:40
dansmithmriedem: was just about to19:40
*** brault has joined #openstack-nova19:40
*** avolkov has quit IRC19:41
*** hieulq_ has joined #openstack-nova19:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add BDM to InstancePayload  https://review.openstack.org/44877919:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add keypairs field to InstancePayload  https://review.openstack.org/41973019:41
dansmithI only see it in nova-compute logs19:42
*** kfarr has joined #openstack-nova19:43
dansmithin the journal I mean19:43
*** dave-mccowan has joined #openstack-nova19:43
dansmithI see it in scheduler too19:44
dansmithbut no api or anywhere else19:44
melwittdansmith: since you're depending on devstack change, did you rebase that to get the use_journal change. it looks like you did already19:44
dansmithand I see it in neutron-server19:45
*** brault_ has joined #openstack-nova19:45
*** dave-mcc_ has joined #openstack-nova19:45
dansmithmelwitt: yep19:45
sdaguedansmith: what's going on?19:45
dansmithsdague: we think we're missing nova-api logs19:45
dansmithwell, and nova-compute logs actually19:45
dansmithwe see instances that were booted and used by tempest,19:45
dansmithbut zero evidence of that uuid in nova api logs, very little in compute (not even the boot part)19:46
dansmithin scheduler, and neutron19:46
dansmithso it clearly exists19:46
dansmithbut it's like shockingly absent from the other services19:46
*** crushil_ has quit IRC19:46
dansmithsdague: that's the case in the text logs and in the raw journal I pulled down19:47
dansmith(unified journal ftmfw by the way)19:47
sdagueright, they'll be the same19:47
sdagueok, give me the example in question19:47
dansmithwell, I know they should be, but.. checking :)19:47
dansmithhttp://logs.openstack.org/37/458537/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fe108a8/logs/19:47
dansmithinstance db767e42-2dbe-48d7-868d-b766d9879eb619:47
*** harlowja has joined #openstack-nova19:48
*** dave-mccowan has quit IRC19:48
*** brault has quit IRC19:48
dansmithand if you look for the NoVNCConsoleTestJSON tenant/user,19:48
dansmithyou get one hit in scheduler and nothing else19:48
sdagueso, it is entirely possible that the default rate limitters for journald are getting tripped19:49
sdagueI wonder if we can find out about that19:49
dansmithoh interesting19:49
sdaguedansmith: you have the journals all down already?19:49
dansmithsdague: just one journal, right?19:49
sdaguesearch for -k systemd19:49
dansmithdevstack.journal.gz?19:49
sdaguewell, there are usual multiples of them because they roll over19:50
sdaguebut, yeh, only looks like one in there19:50
dansmith-k is dmesg, which you can't do on a file19:50
dansmithunless you mean something else19:50
dansmith-u?19:50
dansmithjournalctl | grep systemd yields not much of relevance19:51
sdague-t19:51
sdaguesorry19:51
dansmithsame as the grep19:51
edleafebauzas: should be ready to upload soon. I've already created a unit test.19:52
*** salv-orlando has joined #openstack-nova19:52
sdagueok, so default seems to be 1000 messages per 30s19:52
sdagueI am sure we are exceeding that19:52
dansmithsurely at startup19:52
sdaguelet me put in a fix for that19:54
dansmithcool thanks19:54
dansmithlink me and I'll depend on it and recheck19:54
*** cdent has quit IRC19:54
*** hieulq_ has quit IRC19:55
melwittdansmith: so the websocketproxy goes direct to the instance? it doesn't need to know the compute host MQ right19:56
dansmithit goes direct to the host19:56
dansmithconsoleauth validate_token does talk to the compute mq, but it does the looku19:56
melwittokay. we'll just have to see the logs then, nothing is jumping out at me19:57
dansmithI think the token in memcache must have the ip/port that the proxy needs to connect to19:57
dansmithyeah19:57
dansmithhopefully something dumb19:57
dansmithsimple and dumb19:57
dansmithlike me.19:57
sdaguedansmith: I think this is it - https://review.openstack.org/46272519:57
melwittlol :P19:57
*** cdent has joined #openstack-nova19:58
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853719:58
dansmithsdague: ack thanks19:58
dansmithsdague: if that's really happening we probably want to merge that like.. yesterday :P19:58
mriedemgibi: i rebased and commented in https://review.openstack.org/#/c/419730/ for adding keypairs to InstancePayload - summary is i don't think we should lazy load instance.keypairs for every instance action, for the same reason I didn't want to do that for instance.tags19:59
sdaguedansmith: yep agreed, I want to make sure it looks sane20:00
sdaguehey, it's all about learning new things :)20:00
*** lbragstad_alt has joined #openstack-nova20:02
dansmithsdague: I'm not complaining :)20:02
sdaguealso, what's up with the linuxbridge job?20:03
sdaguethat seem things to be hating me today20:03
*** Jack_Iv has quit IRC20:04
dansmithsdague: that was an impressively incomprehensible exclamation20:04
*** Jack_Iv has joined #openstack-nova20:04
openstackgerritEd Leafe proposed openstack/nova master: Don't delete allocation if instance being scheduled  https://review.openstack.org/46017720:04
edleafebauzas: Here ya go ^^20:04
*** Apoorva_ has joined #openstack-nova20:05
dansmithwhat is this nonsense? http://logs.openstack.org/79/462679/1/check/gate-nova-tox-functional-ubuntu-xenial/c760042/testr_results.html.gz20:05
bauzasedleafe: okay, I'll update the next top changes20:05
melwittdansmith: socket_options, duh20:07
* dansmith glares at melwitt 20:07
melwitt:D20:07
*** iceyao has joined #openstack-nova20:08
*** eharney has quit IRC20:09
*** Apoorva has quit IRC20:09
*** liverpooler has quit IRC20:09
cdentdansmith: might be mismatched versions of urllib3 and wsgi-intercept20:10
cdentI saw that a while back but thought it was better.20:11
dansmithcdent: it's broken is what it is20:11
dansmithlooks rather fundamental, so wondering if it's worth a recheck20:11
cdentyessir, but I think it got better. are your deps maybe stale?20:11
dansmiththat patch is new from today20:11
cdenthmmm20:11
dansmithI guess edleafe already rechecked it20:12
cdenti'll mess with it a bit20:12
cdent(locally)20:12
*** iceyao has quit IRC20:12
mriedemcan't provide new key_name on rebuild?20:12
melwittmriedem: I don't think you can specify anything other than the image. there have been requests to allow new userdata and such, but the answer has been "just boot a new instance" if you're looking at re-imaging it anyway20:17
*** hongbin has quit IRC20:17
mriedemdo we rebuild the config drive during rebulid?20:18
melwittI don't think so20:18
melwittbut not sure, not a config drive expert20:18
mriedemi'm looking, just this keypair notification patch has me thinking about weird stuf20:19
mriedemhttps://review.openstack.org/#/c/419730/20:19
mriedemb/c i don't want to lazy-load instance.keypairs for every gd instance action notification20:19
melwittwe definitely don't change anything on it. I don't know if it is "recreated" at all though20:19
mriedemjaypipes: dansmith: ^ same predicament as with the instance.tags patch20:20
mriedemfor notifications that is20:20
mriedemwhich would include every vm/task state change if you enabled that option...20:21
mriedemCONF.notifications.notify_on_state_change i mean20:21
melwittpreload all the things20:22
*** slaweq has joined #openstack-nova20:22
melwittjk20:22
mriedemwell someone proposed that for instance.tags,20:22
mriedempreload by default20:22
jrolljust keep the whole db in RAM, wcpgw20:22
mriedemwe nacked that20:22
melwittsee, jroll knows what I'm talkin about20:23
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366020:23
mriedemi worry that some of the perf issues people are reporting in newton/ocata are due to all of the versioned notifications doing lazy-loads like crazy20:23
*** damien_r has joined #openstack-nova20:23
jrollmelwitt: :D20:24
jrollfeels like state change notifications shouldn't need tags, keypairs, etc20:24
*** Jack_Iv has quit IRC20:24
mriedemthat's what i've said20:25
mriedemand pushed back on20:25
*** Jack_Iv has joined #openstack-nova20:25
mriedemnor bdms...20:25
mriedemwe're basically throwing the kitchen sink at the notification consumer20:25
mriedembecause (1) we can and (2) we don't know what notification consumers even want/need20:26
jaypipesmriedem: currently reviewing dansmith's target_cell patches. will get to those a little later (all the notification ones)20:26
mriedemjaypipes: you don't really need to probably, just thinking out loud20:26
mriedemi've already -1ed20:26
jrollmriedem: and I just realized (3) notifications are considered lossy, so we can't assume consumers have state n-1 to diff properly20:26
melwitt:\ a lot of people use notifications, I thought. so one would think we would have some idea what notification consumers want20:26
jrollis the most sensible reason to me20:26
mriedemmelwitt: rebuild calls virt driver destroy and then spawn, so i'd think we're rebuilding the config drive...20:27
*** jose-phillips has joined #openstack-nova20:27
mriedembut i guess you just get the same key as before20:27
*** ociuhandu has quit IRC20:28
melwittmriedem: yeah, I don't know what the term "rebuild" means in the context of config drive. I know we don't change it20:28
mriedemmelwitt: the only consumers i know of really are ceilometer, mistral and searchlight20:28
*** kfarr has quit IRC20:28
mriedemnot sure how much ceilometer is used, and since they didn't get the stuff they needed in the notifications years ago, they just hit the rest api20:28
mriedemnot sure about mistral or searchlight20:28
melwittnotifications were supposed to be the answer to instance actions records. I have used those before in a previous life. and notifications, but just to know the history of what happened to an instance when the customer comes calling20:29
*** Jack_Iv has quit IRC20:29
mriedemmelwitt: what i meant by rebuild was really recreate, since destroy_disks=True when driver.destroy is called during rebuild20:33
mriedemso i think that will delete the config drive for the instance from the host,20:33
mriedemand then spawn would recreate it20:33
*** dave-mcc_ has quit IRC20:34
cdentedleafe, dansmith: I narrowed down the sitch on https://review.openstack.org/#/c/462679/ and left a comment there20:34
cdentI can push the fix if desired20:34
melwittmriedem: I see what you mean. that makes sense with the requests too since it's being recreated anyhow and people think, why not allow metadata changes20:35
*** liangy has joined #openstack-nova20:35
mriedemright so i for whatever reason thought you could provide a new key during a rebuild,20:36
mriedemand since i was -1ing that review saying scope the full keypair object to just a couple of notifications, like instance.create,20:36
mriedemi was thinking instance.update too, but the key doesn't actually change, so we shouldn't need the full keypair in there20:36
mriedemso i'm thinking we just put the full keypair in the instance.create notification and call it good20:37
melwittI see20:37
mriedemand put instance.key_name in everything else20:37
mriedemsince that's just stored on the instance record and not lazy-loaded20:37
melwittIMHO it seems like notifications are a lot more fine-grained than I would have expected or thought needed20:37
*** lyan has quit IRC20:37
mriedemfine grained as in providing more detail than you'd expect?20:38
mriedemor fine-grained as in different payloads for different notifications?20:38
melwittyeah, I guess on create you already have the key_name from the request, what else do you need20:38
mriedemwe also set instance.keypairs in the api during create20:38
mriedemso we don't need to lazy-load it20:38
jaypipesdansmith: if you have time: https://review.openstack.org/#/c/457782/20:39
melwittmriedem: I meant fine-grained like rich subobjects for everything20:39
*** lyan_ has joined #openstack-nova20:40
melwittI didn't even know keypair was a field on instance20:40
mriedemyeah that's the other argument is that the instance action payloads already have the full flavor and instance info_cache20:40
melwittI thought there was only the key name if anything20:40
mriedemwe started storing instance.keypairs when we moved keypairs to the instance_extras table20:41
mriedemand by we i mean dan20:41
mriedemwell,20:41
melwittah, right. I forgot about that even though it was mentioned to me recently20:41
mriedemor maybe that was the move to the api db20:41
mriedemi get confused20:41
*** ociuhandu has joined #openstack-nova20:41
melwittyeah I can't remember why. probably to avoid having to do a lookup in compute or something20:42
*** hamzy has quit IRC20:42
*** liangy has quit IRC20:43
*** cleong has quit IRC20:44
*** cdent has quit IRC20:47
*** hongbin has joined #openstack-nova20:50
*** moshele has joined #openstack-nova20:55
*** gouthamr has quit IRC20:56
*** salv-orl_ has joined #openstack-nova20:56
edleafecdent: dansmith: I'll push the fix.20:57
dansmithah20:57
dansmiththat was an odd failure20:57
*** salv-orlando has quit IRC20:59
*** thorst_afk has quit IRC20:59
mriedemhemna: so looking at https://review.openstack.org/#/c/382097/47/etc/cinder/rootwrap.d/volume.filters and https://review.openstack.org/#/c/443951/10/etc/nova/rootwrap.d/compute.filters i thought we were supposed to push that common root helper stuff into os-brick?21:00
mriedemdoesn't doing a rootwrap filter in cinder and nova kind of defeating the purpose?21:00
*** lucasxu has quit IRC21:01
mriedemheh it's not even used in the volume driver code in nova21:02
melwittmriedem: I found there is a bug in oslo.log too regarding the unicode thing https://review.openstack.org/#/c/46273521:02
mriedemmelwitt: nice catch21:03
*** thorst_afk has joined #openstack-nova21:03
*** brault_ has quit IRC21:03
melwitt:)21:04
johnsomHi nova folks.  Is there a known issue where cirros instances come up ACTIVE in nova show, but there is no console.log content?21:04
*** Apoorva_ has quit IRC21:04
mtreinishjohnsom: when I hit that locally on my cloud it was because the glance image was empty...21:05
*** Apoorva has joined #openstack-nova21:05
* mtreinish was just writing a slide about that exact issue for his talk next week21:05
johnsomYeah, two of us just restacked today and cirros and our image both won't come up21:06
mtreinishjohnsom: I would check the image size in glance. If it's zero bytes that's probably it21:07
johnsomWell, glance is showing content for the cirros image...  hmmm21:07
mtreinishhmm, ok maybe it's something else :)21:07
johnsomThe console logs aren't in this new fanlged journalctl logs are they?21:07
edleafecrap, looks like cdent has left21:07
mtreinishjohnsom: that's independent. THat's just service logs, because the services are running under systemd21:07
*** thorst_afk has quit IRC21:08
mtreinishconsole logs are grabbed from the virt driver21:08
edleafeI don't see where he is referring to in his comment. dansmith?21:08
johnsomYeah, I wouldn't have expected it.  Especially because there is a zero byte console.log file sitting there21:08
*** damien_r1 has joined #openstack-nova21:08
mtreinishjohnsom: I would check the n-cpu log21:08
*** damien_r2 has joined #openstack-nova21:08
edleafeoh, nvm21:09
edleafeI had the unit test test_resource_tracker.py open, not the functional21:09
openstackgerritAlex Szarka proposed openstack/nova master: Refactor instance-exists comment and test  https://review.openstack.org/46273821:09
*** bmace has quit IRC21:09
*** bmace has joined #openstack-nova21:09
*** awaugama has quit IRC21:09
dansmithedleafe: I didn't look close, I just read what he said and thought "mocks, okay, complicated"21:10
edleafeheh21:11
*** eric_young has quit IRC21:11
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366021:11
openstackgerritAlex Szarka proposed openstack/nova master: Refactor instance-exists comment and test  https://review.openstack.org/46273821:11
johnsomYeah, I started there.  No ERROR records and it shows 2 vcpus allocated.  It looks normal to me.  Ok, I will dig in.  I just didn't want to burn a bunch of time if this was a known issue.21:11
dansmithedleafe: assume he means this: https://pastebin.com/x6MJut7P21:12
openstackgerritEd Leafe proposed openstack/nova master: Don't delete allocation if instance being scheduled  https://review.openstack.org/46017721:12
openstackgerritEd Leafe proposed openstack/nova master: Add a convenience attribute for reportclient  https://review.openstack.org/46267921:12
*** annegentle has quit IRC21:12
edleafedansmith: yep ^^21:12
*** damien_r1 has quit IRC21:12
*** damien_r has quit IRC21:13
*** kiwi_rot has joined #openstack-nova21:13
*** kiwi_rot has quit IRC21:18
johnsommtreinish Ok, I am suspecting the new qemu packages from ubuntu.  Those processes are sitting at 120% in top21:18
*** dimtruck is now known as zz_dimtruck21:20
*** damien_r2 has quit IRC21:22
*** marst has quit IRC21:23
*** gouthamr has joined #openstack-nova21:26
*** smatzek has quit IRC21:27
*** Guest73837 has quit IRC21:28
*** marst has joined #openstack-nova21:30
*** moshele has quit IRC21:31
*** Guest73837 has joined #openstack-nova21:33
*** jerrygb has quit IRC21:35
mtreinishjohnsom: that could be it21:37
johnsomYeah, digging, will try a rollback21:37
*** karthiks has quit IRC21:38
*** annegentle has joined #openstack-nova21:42
*** mingyu has quit IRC21:43
*** baoli has quit IRC21:44
*** lbragstad has quit IRC21:45
*** mingyu has joined #openstack-nova21:46
openstackgerritmelanie witt proposed openstack/nova master: DNM: Revert "Use six.text_type() when logging Instance object"  https://review.openstack.org/46274421:47
openstackgerritEd Leafe proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries  https://review.openstack.org/46043321:48
openstackgerritEd Leafe proposed openstack/nova master: WIP: Allocate instance by the conductor  https://review.openstack.org/46045521:48
mriedemmelwitt: that's not going to work21:49
mriedemunless we have a ci job that uses LIBS_FROM_GIT=oslo.log21:49
mriedemthe depends-on won't work21:50
melwittmriedem: it won't make a new oslo.log? oh. I wasn't sure which things Depends-On works with21:50
mriedemefried had the same question earlier this week21:50
mriedembasically,21:50
mriedemcan't depends-on to project-config changes or libraries/clients21:50
mriedemthe latter requires LIBS_FROM_GIT21:50
*** salv-orl_ has quit IRC21:51
*** mvk has joined #openstack-nova21:51
efriedGood news is that it seems fairly straightforward to get a new release cut and through requirements.21:52
melwittmriedem: does Depends-On work on changes in other projects like cinder for example?21:52
mriedemwtf are we not running the releasenotes job anymore?21:52
mriedemmelwitt: yes21:52
melwittokay. oh, right because devstack always pulls those from git21:53
melwittI was just curious what the log would look like with the Instance object if we didn't do anything to convert it, with the oslo.log change21:54
mriedemmelwitt: you could do a devstack change that sets LIBS_FROM_GIT=oslo.log,21:55
mriedemand have that depends on the nova and oslo.log changes21:55
mriedemi think21:55
melwittbecause the ovo repr encodes to ascii, which has the chars with the \x escapes in it. I guess it would just look like in my unit test, with \x in it21:55
johnsommtreinish  Yep, downgrade packages work.21:56
johnsomapt-get install seabios=1.8.2-1ubuntu1 qemu-system-sparc=1:2.5+dfsg-5ubuntu10.11 qemu-system-ppc=1:2.5+dfsg-5ubuntu10.11 qemu-system-arm=1:2.5+dfsg-5ubuntu10.11 qemu-kvm=1:2.5+dfsg-5ubuntu10.11 qemu-system-x86=1:2.5+dfsg-5ubuntu10.11 qemu-system-mips=1:2.5+dfsg-5ubuntu10.11 qemu-system-misc=1:2.5+dfsg-5ubuntu10.1121:56
johnsomrestart nova and life is better21:56
*** hongbin has quit IRC22:03
*** annegentle has quit IRC22:03
*** annegentle has joined #openstack-nova22:04
*** annegentle has quit IRC22:04
*** Apoorva_ has joined #openstack-nova22:05
*** hongbin has joined #openstack-nova22:05
*** artom has joined #openstack-nova22:06
mriedemsdague: in https://review.openstack.org/#/c/435432/11/releasenotes/notes/project_id_validation-568d31c13c3ef735.yaml what does "If the user has enough permissions in user" mean?22:06
mriedemthe 'in user' part22:06
*** jamesdenton has quit IRC22:08
*** Apoorva has quit IRC22:09
*** mingyu has quit IRC22:09
*** mingyu has joined #openstack-nova22:09
*** edmondsw has quit IRC22:12
*** mingyu has quit IRC22:13
mriedemnvm i'll clean it up22:14
*** edmondsw has joined #openstack-nova22:15
*** edmondsw has quit IRC22:20
*** slaweq has quit IRC22:22
openstackgerritMatt Riedemann proposed openstack/nova master: Verify project id for flavor access calls  https://review.openstack.org/43543222:28
mriedemoomichi: ^ i cleaned that one up22:30
*** xyang1 has quit IRC22:31
*** hongbin_ has joined #openstack-nova22:33
*** hongbin has quit IRC22:33
oomichimriedem: thanks, lets close the bug22:34
dansmithsdague: melwitt: shockingly, I now have logs for that instance22:36
*** jerrygb has joined #openstack-nova22:37
melwittyou mean with sdague's change you do?22:37
dansmithmelwitt: checkit, like I thought: http://logs.openstack.org/37/458537/10/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0601a5e/logs/screen-n-cauth.txt.gz#_May_04_20_47_48_62078322:38
dansmithmelwitt: yeah22:38
melwittvalidate_console_port22:39
dansmithdinnuh I say?22:40
melwittyeah22:40
dansmithI shall flog that tomorrow22:40
melwittcool. somehow instance must not be targeted to the right MQ even though there's a target_cell call in there22:41
*** jerrygb has quit IRC22:41
dansmithsomething.22:44
*** mlavalle has quit IRC22:45
melwittstrange. not seeing what's wrong there22:47
mnaseri've been flying around the openstack api but this might be a little crazy.  is it possible to remove an existing ephermal drive and replace it by cinder volume... on an existing instance?22:48
melwittthe context gets wiped out after the context manager exits, but all of the calls are under it22:48
mnaserso converting to boot from cinder for an existing VM (bdm stuff relevant at all here?)22:48
*** tbachman has quit IRC22:49
*** salv-orlando has joined #openstack-nova22:51
*** winston-d_ has quit IRC22:53
*** gouthamr has quit IRC22:55
*** Apoorva has joined #openstack-nova22:56
*** Apoorva_ has quit IRC22:56
*** salv-orlando has quit IRC22:58
*** artom has quit IRC23:02
*** brault has joined #openstack-nova23:04
*** catinthe_ has quit IRC23:08
*** tbachman has joined #openstack-nova23:08
*** brault has quit IRC23:09
*** lyan_ has quit IRC23:14
jaypipesmriedem, dansmith, edleafe, alex_xu: your prompt attention to https://review.openstack.org/#/q/topic:cleanups+project:openstack/os-traits would be appreciated. I'd like to cut a 0.3 os-traits release ASAP because I need to use it in some of the new placement work...23:16
dansmithjaypipes: I -1'ed the fsck out of that last patch23:18
dansmithutter garbage!23:18
jaypipesdansmith: fixed. :)23:20
*** sdague has quit IRC23:20
dansmithjaypipes: that was almost a very embarrassing event23:23
jaypipes:)23:23
*** baoli has joined #openstack-nova23:24
*** baoli has quit IRC23:24
*** hamzy has joined #openstack-nova23:24
*** erlon has quit IRC23:24
*** baoli has joined #openstack-nova23:25
*** edmondsw has joined #openstack-nova23:25
mriedemha23:29
mriedemgot you both23:29
*** edmondsw has quit IRC23:30
dansmithmriedem: luckily I don't care about my ratio on os-traits, so impede progress all you want buddy :)23:30
mriedemi'll always remember23:32
mriedemlike your love of pizza and cookies and complete sentences23:32
dansmithheh23:32
mriedemjaypipes: on the last one,23:33
mriedemthat regex isn't enforced in the api schema from what i can tell23:33
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L2923:33
mriedemor does that just mean for custom traits?23:33
jaypipesyeah, that's just custom traits.23:34
mriedemwell, https://review.openstack.org/#/c/462759/2/os_traits/hw/cpu/x86.py is a standard trait no?23:34
jaypipesno23:35
jaypipesthat's a module.23:35
jaypipeshw.cpu.x86.SSE42 is a standard trait *symbol*, but the trait string is HW_CPU_X86_SSE4223:35
mriedembut based on the api schema,23:36
mriedemi could create a trait that's named HW_CPU_X86_AES-NI right?23:36
*** chyka has quit IRC23:37
mriedemor are you only allowed to create custom traits via the API? https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/trait.py#L9923:38
mriedemand standard traits come from the lib?23:38
*** chyka has joined #openstack-nova23:38
*** chyka has quit IRC23:43
jaypipesmriedem: correct.23:44
jaypipesmriedem: meh, fixed that README typo.23:47
*** artom has joined #openstack-nova23:50
jaypipesmriedem: I just want to keep the standard trait naming consistent.23:51
mriedemalright all done23:52
*** Apoorva has quit IRC23:52
*** hongbin_ has quit IRC23:53
*** salv-orlando has joined #openstack-nova23:54
*** Apoorva has joined #openstack-nova23:54
*** salv-orlando has quit IRC23:58

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