Monday, 2017-05-01

*** esberglu has joined #openstack-nova00:03
*** esberglu has quit IRC00:07
*** edmondsw has joined #openstack-nova00:10
*** edmondsw has quit IRC00:15
*** ijw has joined #openstack-nova00:32
*** baoli has joined #openstack-nova00:33
*** nkorabli has joined #openstack-nova00:36
*** ijw has quit IRC00:38
*** amotoki has joined #openstack-nova00:38
*** baoli has quit IRC00:38
*** nkorabli has quit IRC00:41
*** amotoki has quit IRC00:42
*** imacdonn has quit IRC00:56
*** fragatina has quit IRC01:04
*** baoli has joined #openstack-nova01:06
*** thorst has joined #openstack-nova01:14
*** salv-orlando has joined #openstack-nova01:16
*** thorst has quit IRC01:19
*** salv-orlando has quit IRC01:22
*** cNilesh has joined #openstack-nova01:22
*** baoli has quit IRC01:24
*** baoli has joined #openstack-nova01:24
*** nicolasbock has joined #openstack-nova01:26
*** chatter29 has joined #openstack-nova01:26
chatter29hey guys01:26
chatter29allah is doing01:26
chatter29sun is not doing allah is doing01:26
*** chatter29 has quit IRC01:26
*** ijw has joined #openstack-nova01:30
*** baoli has quit IRC01:31
*** thorst has joined #openstack-nova01:33
*** slagle has joined #openstack-nova01:34
*** yamahata_ has joined #openstack-nova01:35
*** thorst has quit IRC01:41
*** iceyao has joined #openstack-nova01:42
*** fragatina has joined #openstack-nova01:42
*** brault has joined #openstack-nova01:57
*** edmondsw has joined #openstack-nova01:58
*** brault has quit IRC02:01
*** edmondsw has quit IRC02:03
*** mingyu has joined #openstack-nova02:11
*** salv-orlando has joined #openstack-nova02:18
*** salv-orlando has quit IRC02:22
*** yamamoto has joined #openstack-nova02:24
*** ssurana has joined #openstack-nova02:35
*** yangyapepeng has joined #openstack-nova02:37
*** thorst has joined #openstack-nova02:38
*** amotoki has joined #openstack-nova02:39
*** vladikr has quit IRC02:39
*** fragatina has quit IRC02:40
*** fragatina has joined #openstack-nova02:40
*** yangyapepeng has quit IRC02:41
*** amotoki has quit IRC02:43
*** fragatina has quit IRC02:54
*** thorst has quit IRC02:56
*** ijw has quit IRC03:01
*** ijw has joined #openstack-nova03:02
*** armax has joined #openstack-nova03:06
*** ijw has quit IRC03:06
*** psachin has joined #openstack-nova03:09
*** haplo37 has quit IRC03:10
*** slaweq has joined #openstack-nova03:13
*** psachin has quit IRC03:13
*** haplo37 has joined #openstack-nova03:15
*** slaweq has quit IRC03:17
*** nkorabli has joined #openstack-nova03:17
*** dave-mccowan has quit IRC03:22
*** yamamoto has quit IRC03:27
*** masber has quit IRC03:29
*** takashin has joined #openstack-nova03:30
*** nicolasbock has quit IRC03:31
*** fragatina has joined #openstack-nova03:38
*** psachin has joined #openstack-nova03:39
*** yamahata_ has quit IRC03:40
*** armax has quit IRC03:41
*** edmondsw has joined #openstack-nova03:46
*** fragatina has quit IRC03:49
*** dimtruck is now known as zz_dimtruck03:50
*** edmondsw has quit IRC03:51
*** thorst has joined #openstack-nova03:53
*** thorst has quit IRC03:58
*** armax has joined #openstack-nova04:00
*** ijw has joined #openstack-nova04:02
*** vks1 has joined #openstack-nova04:05
*** ssurana has quit IRC04:07
*** ijw has quit IRC04:07
*** yangyapepeng has joined #openstack-nova04:08
*** armax has quit IRC04:09
*** yamamoto has joined #openstack-nova04:09
*** yangyapepeng has quit IRC04:12
*** links has joined #openstack-nova04:12
*** iceyao has quit IRC04:14
*** armax has joined #openstack-nova04:14
*** adisky_ has joined #openstack-nova04:14
*** esberglu has joined #openstack-nova04:14
*** vks1 has quit IRC04:19
*** esberglu has quit IRC04:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a functional test for 'removeFloatingIp' action  https://review.openstack.org/45302604:22
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc  https://review.openstack.org/45456504:24
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Use 'note' directive  https://review.openstack.org/45871704:24
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in servers-action-console-output  https://review.openstack.org/45871304:27
openstackgerritTakashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls  https://review.openstack.org/41387604:27
openstackgerritTakashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases  https://review.openstack.org/39543604:28
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action  https://review.openstack.org/45605304:28
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix unnecessary description in servers-admin-action  https://review.openstack.org/46086604:28
*** sbezverk has quit IRC04:31
*** nkorabli has quit IRC04:35
*** nkorabli has joined #openstack-nova04:36
*** ijw has joined #openstack-nova04:38
*** nkorabli has quit IRC04:41
*** ijw has quit IRC04:42
*** armax has quit IRC04:46
*** thorst has joined #openstack-nova04:54
*** fragatina has joined #openstack-nova04:57
*** brault has joined #openstack-nova04:57
*** thorst has quit IRC04:58
*** armax has joined #openstack-nova05:00
*** brault has quit IRC05:01
*** armax has quit IRC05:04
*** esberglu has joined #openstack-nova05:08
*** esberglu has quit IRC05:12
*** trinaths has joined #openstack-nova05:18
*** salv-orlando has joined #openstack-nova05:20
*** trinaths has quit IRC05:22
*** salv-orlando has quit IRC05:25
*** mingyu has quit IRC05:26
*** kaisers has joined #openstack-nova05:32
*** iceyao has joined #openstack-nova05:32
*** mdnadeem has joined #openstack-nova05:34
*** ijw has joined #openstack-nova05:39
*** ijw has quit IRC05:46
*** trinaths has joined #openstack-nova05:51
*** arunman has joined #openstack-nova05:53
*** thorst has joined #openstack-nova05:54
*** fragatina has quit IRC05:55
*** thorst has quit IRC05:59
*** trinaths has left #openstack-nova06:00
*** yangyapepeng has joined #openstack-nova06:09
*** haplo37 has quit IRC06:10
*** yangyapepeng has quit IRC06:13
*** haplo37 has joined #openstack-nova06:16
*** salv-orlando has joined #openstack-nova06:22
*** arunman has quit IRC06:22
*** salv-orlando has quit IRC06:26
*** salv-orlando has joined #openstack-nova06:37
*** amotoki has joined #openstack-nova06:40
*** ekuris has quit IRC06:41
*** ijw has joined #openstack-nova06:43
*** amotoki has quit IRC06:45
*** ijw has quit IRC06:48
*** arunman has joined #openstack-nova06:52
*** thorst has joined #openstack-nova06:55
*** ekuris has joined #openstack-nova06:56
*** thorst has quit IRC07:00
*** tesseract has joined #openstack-nova07:06
*** mingyu has joined #openstack-nova07:07
*** ssurana has joined #openstack-nova07:08
*** salv-orlando has quit IRC07:08
*** arunman has quit IRC07:10
*** ssurana has quit IRC07:12
*** rcernin has joined #openstack-nova07:12
*** slaweq has joined #openstack-nova07:13
*** slaweq has quit IRC07:18
*** artom__ has quit IRC07:18
*** artom__ has joined #openstack-nova07:18
*** artom__ has quit IRC07:19
*** artom__ has joined #openstack-nova07:19
*** edmondsw has joined #openstack-nova07:22
*** haplo37 has quit IRC07:22
*** slaweq has joined #openstack-nova07:23
*** edmondsw has quit IRC07:27
*** rcernin has quit IRC07:32
*** artom__ has quit IRC07:32
*** haplo37 has joined #openstack-nova07:32
*** artom__ has joined #openstack-nova07:32
*** Qiming has joined #openstack-nova07:41
*** slaweq has quit IRC07:42
*** ijw has joined #openstack-nova07:44
*** masber has joined #openstack-nova07:46
*** slaweq has joined #openstack-nova07:50
*** esberglu has joined #openstack-nova07:51
*** esberglu has quit IRC07:55
*** mingyu has quit IRC07:55
*** thorst has joined #openstack-nova07:56
*** mingyu has joined #openstack-nova07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** ttx has quit IRC08:00
*** ttx has joined #openstack-nova08:01
*** ijw has quit IRC08:05
*** salv-orlando has joined #openstack-nova08:08
*** yangyapepeng has joined #openstack-nova08:09
*** cdent has joined #openstack-nova08:12
*** salv-orlando has quit IRC08:13
*** yangyapepeng has quit IRC08:14
*** thorst has quit IRC08:15
*** brault has joined #openstack-nova08:20
*** arunman has joined #openstack-nova08:28
*** arunman has quit IRC08:39
*** voelzmo has joined #openstack-nova08:53
*** owalsh_ has joined #openstack-nova08:58
*** mingyu has quit IRC08:58
*** owalsh has quit IRC08:59
*** owalsh_ is now known as owalsh08:59
*** cdent has quit IRC08:59
*** ijw has joined #openstack-nova09:01
*** ijw has quit IRC09:07
*** arunman has joined #openstack-nova09:09
*** salv-orlando has joined #openstack-nova09:09
*** thorst has joined #openstack-nova09:12
*** salv-orlando has quit IRC09:14
*** arunman has quit IRC09:16
*** thorst has quit IRC09:16
*** mingyu has joined #openstack-nova09:17
*** mingyu has quit IRC09:25
*** iceyao has quit IRC09:25
*** cdent has joined #openstack-nova09:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895509:26
*** voelzmo has quit IRC09:30
*** voelzmo has joined #openstack-nova09:31
*** iceyao has joined #openstack-nova09:35
*** ekuris has quit IRC09:36
*** esberglu has joined #openstack-nova09:38
*** iceyao has quit IRC09:41
*** esberglu has quit IRC09:43
*** voelzmo has quit IRC09:45
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896409:58
*** iceyao has joined #openstack-nova10:02
*** ijw has joined #openstack-nova10:03
*** nicolasbock has joined #openstack-nova10:04
*** iceyao has quit IRC10:07
*** cNilesh has quit IRC10:09
*** sdague has joined #openstack-nova10:09
*** yangyapepeng has joined #openstack-nova10:10
*** salv-orlando has joined #openstack-nova10:10
*** ijw has quit IRC10:12
*** yangyapepeng has quit IRC10:15
*** salv-orlando has quit IRC10:15
*** mingyu has joined #openstack-nova10:19
*** iceyao has joined #openstack-nova10:20
*** iceyao has quit IRC10:25
*** iceyao has joined #openstack-nova10:28
*** mdnadeem_ has joined #openstack-nova10:32
*** mdnadeem has quit IRC10:34
*** mdnadeem_ has quit IRC10:36
*** tbachman has quit IRC10:39
*** mingyu has quit IRC10:40
*** ericyoung has joined #openstack-nova10:41
*** mdnadeem_ has joined #openstack-nova10:50
*** edmondsw has joined #openstack-nova10:58
*** djohnsto has joined #openstack-nova10:59
*** jschlueter has joined #openstack-nova11:01
*** edmondsw has quit IRC11:03
*** ijw has joined #openstack-nova11:09
*** thorst has joined #openstack-nova11:14
*** ijw has quit IRC11:14
*** thorst has quit IRC11:18
*** smatzek_ has joined #openstack-nova11:23
*** dave-mccowan has joined #openstack-nova11:23
*** yangyapepeng has joined #openstack-nova11:26
*** esberglu has joined #openstack-nova11:27
*** yangyapepeng has quit IRC11:30
*** iceyao has quit IRC11:31
*** slaweq has quit IRC11:31
*** esberglu has quit IRC11:31
*** slaweq has joined #openstack-nova11:31
*** iceyao has joined #openstack-nova11:32
*** iceyao has quit IRC11:33
*** slaweq__ has joined #openstack-nova11:33
*** mingyu has joined #openstack-nova11:34
*** slaweq has quit IRC11:37
*** mingyu has quit IRC11:45
*** mvk has quit IRC11:46
*** tbachman has joined #openstack-nova11:49
*** thorst has joined #openstack-nova11:49
*** mdnadeem_ has quit IRC11:52
*** tbachman has quit IRC11:54
*** tbachman_ has joined #openstack-nova11:54
*** mingyu has joined #openstack-nova11:54
*** sdague has quit IRC11:55
*** haplo37 has quit IRC11:55
*** iceyao has joined #openstack-nova11:59
*** takashin has left #openstack-nova11:59
*** haplo37 has joined #openstack-nova12:00
*** jschlueter has quit IRC12:00
*** iceyao has quit IRC12:03
*** mvk has joined #openstack-nova12:04
*** ijw has joined #openstack-nova12:11
*** cdent has quit IRC12:11
*** mingyu has quit IRC12:11
*** mdnadeem has joined #openstack-nova12:12
*** sdague has joined #openstack-nova12:12
*** edmondsw has joined #openstack-nova12:15
*** ijw has quit IRC12:17
*** tbachman_ is now known as tbachman12:24
*** amotoki has joined #openstack-nova12:27
*** iceyao has joined #openstack-nova12:31
*** vks1 has joined #openstack-nova12:36
*** rwmjones has quit IRC12:37
*** mdrabe has joined #openstack-nova12:37
*** cdent has joined #openstack-nova12:38
*** rwmjones has joined #openstack-nova12:38
*** edmondsw_ has joined #openstack-nova12:45
*** edmondsw_ has quit IRC12:45
*** mingyu has joined #openstack-nova12:45
*** mingyu has quit IRC12:46
*** mingyu has joined #openstack-nova12:46
*** jaypipes has joined #openstack-nova12:54
*** dklyle has joined #openstack-nova12:57
*** david-lyle has quit IRC12:57
*** lyan has joined #openstack-nova12:58
jaypipescores: easy review: https://review.openstack.org/#/c/45767812:58
*** arunman has joined #openstack-nova12:59
*** gcb has joined #openstack-nova13:00
*** eharney has joined #openstack-nova13:03
*** gcb has quit IRC13:03
*** gcb has joined #openstack-nova13:04
*** arunman has quit IRC13:11
*** salv-orlando has joined #openstack-nova13:12
*** ijw has joined #openstack-nova13:13
*** baoli has joined #openstack-nova13:13
*** baoli has quit IRC13:13
cdentedleafe, jaypipes based on review noise there seem to be some of us around on this international workers holiday. will we have a scheduler meeting?13:13
*** baoli has joined #openstack-nova13:14
jaypipescdent: planning on it, yes.13:14
cdentk13:14
*** arunman has joined #openstack-nova13:16
*** salv-orlando has quit IRC13:17
*** arunman has quit IRC13:17
*** ijw has quit IRC13:18
*** hoonetorg has quit IRC13:19
*** hoonetorg has joined #openstack-nova13:20
*** vladikr has joined #openstack-nova13:23
*** baoli has quit IRC13:25
*** mingyu has quit IRC13:26
*** yangyapepeng has joined #openstack-nova13:27
*** links has quit IRC13:29
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276613:31
*** yangyapepeng has quit IRC13:31
*** mingyu has joined #openstack-nova13:32
*** iceyao has quit IRC13:32
*** mriedem has joined #openstack-nova13:32
*** smatzek_ has quit IRC13:32
* cdent does his monthly ritual ^13:32
*** ijw has joined #openstack-nova13:33
edleafeI was planning on it13:33
edleafeWorkers in the US don't get crap :)13:34
*** esberglu has joined #openstack-nova13:36
* cdent pages Upton Sinclair13:36
*** mingyu has quit IRC13:36
*** lbragstad_ is now known as lbragstad13:37
edleafecdent: https://twitter.com/EdLeafe/status/85790539731066060913:38
cdentyeah, I <3ed that one13:39
cdentit's _hilarious_13:39
cdentWTF are people thinking13:39
*** awaugama has joined #openstack-nova13:39
edleafeStupidity trumps all13:40
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/46138613:40
edleafe...or is it Trump's stupidity...13:40
* edleafe goes wildly off-topic13:41
cdentsymptom or disease?13:41
*** arunman has joined #openstack-nova13:41
mriedemsdague: alex_xu: if you want to remove nova-cert https://review.openstack.org/#/c/459460/13:42
*** baoli has joined #openstack-nova13:46
*** iceyao has joined #openstack-nova13:49
edleafeScheduler subteam meeting in 10 minutes in #openstack-meeting-alt13:50
*** tbachman has quit IRC13:52
*** gcb has quit IRC13:52
*** gcb has joined #openstack-nova13:53
mriedemefried: can you amend the service catalog spec for the glance.api_servers decision?13:53
*** iceyao has quit IRC13:53
efriedmriedem Sure thing.13:54
mriedemthanks13:54
*** gcb_ has joined #openstack-nova13:54
*** gcb_ has quit IRC13:56
*** smatzek_ has joined #openstack-nova13:56
*** gcb has quit IRC13:58
*** ijw has quit IRC13:59
sdaguewith the glance api_servers thing, it's unfortunate that we've got all these little bits of topology state sitting in config files, that you need to implement a topology management solution in config management13:59
*** beekneemech is now known as bnemec14:00
*** gcb has joined #openstack-nova14:00
*** crushil has joined #openstack-nova14:00
*** liverpooler has joined #openstack-nova14:00
*** mlavalle has joined #openstack-nova14:00
cdentsdague: there's been some discussion of that should be network and dns controlled, yeah?14:01
*** xyang1 has joined #openstack-nova14:01
*** ijw has joined #openstack-nova14:03
sdaguecdent: well, that mostly just moves a hard topology problem elsewhere, and I can definitely see not wanting to use networking to hide the addressing for the compute host <-> glance from a troubleshooting perspective14:04
mriedemsfinucan: i'm guessing you have opinions on anything in https://review.openstack.org/#/c/450125/ re-using the nova-manage common code14:04
cdentor even topological service catalogs?14:04
* cdent shrugs14:04
*** lucasxu has joined #openstack-nova14:04
sdaguewell, you actually need to put rack topology into the system somewhere for that to work14:06
*** tbachman has joined #openstack-nova14:06
sdagueanyway, it's unfortunate. The path out of it isn't super clear.14:08
efriedmriedem Is this blueprint an appropriate place to describe the changes in keystoneauth1 to load an Adapter from the conf?14:10
*** mingyu has joined #openstack-nova14:10
mriedemefried: i don't follow, do you need to add something to ksa?14:11
efriedmriedem https://review.openstack.org/#/c/460337/14:11
mriedemin laymans terms please14:12
mriedemwhy do we need this14:12
mriedemkeep in mind i don't have ksa memorized like mordred14:12
*** zz_dimtruck is now known as dimtruck14:13
efriedmriedem Yeah, been taking my lead from mordred here.  The Adapter is a thing with service_type, interface, etc. defined on it.  Adding a way to load one of _those_ guys from conf gives us a nice consistent way to get the new options into the appropriate conf groups from one centralized place, like we already do elsewhere for auth & session.14:13
*** salv-orlando has joined #openstack-nova14:14
*** artom__ is now known as artom14:14
efriedmriedem Initial patch set was going to add that centralized thingy in nova (https://review.openstack.org/#/c/458257/4/nova/conf/endpoint.py)14:14
mriedemefried: ok is that necessary *now*? i mean, can't we do that cleanup/refactor later after we're using ksa at all?14:14
mriedemi worry about building the perfect solution before we have any solution14:15
*** lucasxu has quit IRC14:15
efriedmriedem Well, see above.  It's... okaaayy... but it ain't pretty.14:15
*** mingyu has quit IRC14:15
efriedmriedem This is overall cleaner tbh.14:15
*** mingyu has joined #openstack-nova14:15
efriedHas traction from ksa cores.14:16
mriedemi thought we were basically going to do the same thing we already do for ironic and placement, but for neutron, cinder and glance?14:16
*** lucasxu has joined #openstack-nova14:16
efriedmriedem Yes, that's basically the idea.  But we were still going to want a centralized place to define the necessary conf options (service_type, interface, etc.)14:17
efriedksa Adapter was already that place - just needed some loading affordance.14:17
*** salv-orlando has quit IRC14:17
*** mdrabe has quit IRC14:17
efriedmriedem Will make more sense once you see the next nova patch set.  I get rid of that endpoint.py thing and just include the ksa loading in the relevant confs.14:18
efriedmriedem I have that code change done - was working on UT, but can post it now if you wanna see it.14:18
mriedemhonestly i haven't reviewed any of it14:18
mriedemi'm in a meeting too,14:18
*** mdrabe has joined #openstack-nova14:18
mriedemjay and monty have been helping out here, so if there is an ideal path forward that makes this work and is less ugly, and has sufficient buy in from the ksa team to actually make progress before the 3rd milestone, then sure14:19
mriedemgo nuts14:19
jaypipesmriedem: mostly mordred. :)14:19
efriedmriedem Roger that, going nuts. Since others have been engaged, I've been trying not to bug you with it, cause I know you're a busy dude.14:20
efriedmriedem So the question remains.  I can leave the spec fairly vague in terms of where the centralized conf options are defined if you prefer.  But at this point I feel like the design is pretty well hammered out, so I'd rather be explicit if possible.14:21
mriedemthe spec needs to be amended for the glance.api_servers thing anyway14:22
mriedemif you want to add something high-level in there about ksa changes that's fine14:22
efriedmriedem Roger wilco.14:24
openstackgerritJay Pipes proposed openstack/nova master: placement: get providers sharing capacity  https://review.openstack.org/46079814:26
jaypipesedleafe, cdent: ^14:26
edleafekewl14:26
*** tbachman has quit IRC14:27
*** gcb has quit IRC14:29
*** gcb has joined #openstack-nova14:29
jaypipescdent: since Andrey's on PTO for the next week, you think you could take over all the placement API ref patches and push fixes on them?14:30
*** ijw has quit IRC14:30
*** catintheroof has joined #openstack-nova14:30
cdenta) I had planned to that last week, but didn't reach it, b)I thought he was back today-ish?, c) but if not, yeah14:31
*** gcb has quit IRC14:31
*** gcb has joined #openstack-nova14:31
jaypipescdent: cheers, appreciated.14:32
*** eharney has quit IRC14:32
openstackgerritMatt Riedemann proposed openstack/nova master: Add compute_nodes_uuid_idx unique index  https://review.openstack.org/45488714:33
*** iceyao has joined #openstack-nova14:33
*** tbachman has joined #openstack-nova14:33
*** djohnsto has quit IRC14:33
*** djohnsto has joined #openstack-nova14:35
*** catintheroof has quit IRC14:36
*** iceyao has quit IRC14:37
*** catintheroof has joined #openstack-nova14:40
*** sree has joined #openstack-nova14:41
*** catintheroof has quit IRC14:42
cfriesen_I'm running into issues setting up devstack on Ubuntu...hitting an issue with keystone not starting.  Anyone else seen this?14:43
*** arunman has quit IRC14:44
sdaguecfriesen_: is this somewhere with accessable logs?14:45
cfriesen_sdague: yes, local dedicated mini-box14:45
sdaguecfriesen_: some changes were made in deployment defaults, the issue might be a service collision14:45
cfriesen_sdague: here's the output from stack.sh: http://paste.openstack.org/show/608527/14:49
cfriesen_I haven't found anything useful in /opt/stack/logs yet14:49
sdaguecfriesen_: what's in /etc/apache2/sites-enabled14:50
cfriesen_sdague: 000-default.conf  keystone.conf  keystone-wsgi-admin.conf  keystone-wsgi-public.conf  nova-placement-api.conf14:52
sdagueyeh, so that's the issue I expect14:52
sdagueone sec14:52
*** hongbin has joined #openstack-nova14:53
cdentjaypipes: left some comments on the slides. Overall _very_ good. Some minor nits on naming, and a minor freakout about inbound notifications (dear god no).14:54
*** markvoelker has joined #openstack-nova14:54
sdaguecfriesen_: if you pull https://review.openstack.org/461422 and do a ./clean.sh from there before your next ./stack.sh, I think you'll be good14:54
cfriesen_sdague: okay, will try it out14:54
sdaguecfriesen_: I think the issue is that keystone.conf is still configured the old way, and not being updated correctly14:55
*** chyka has joined #openstack-nova14:56
*** sree_ has joined #openstack-nova14:56
*** sree_ is now known as Guest1211514:57
*** salv-orlando has joined #openstack-nova14:57
*** sree has quit IRC14:58
*** dfisher has joined #openstack-nova15:01
*** cdent has quit IRC15:03
*** Guest12115 has quit IRC15:10
*** sree has joined #openstack-nova15:11
cfriesen_sdague: it got further, but now I get an issue running "openstack project show admin":  http://paste.openstack.org/show/608528/15:11
*** links has joined #openstack-nova15:13
*** sree has quit IRC15:15
*** hongbin has quit IRC15:15
*** links has quit IRC15:17
mriedemdansmith: in case you didn't see yet, i went through your series to make discovering host mappings faster - the last change is outstanding15:19
mriedemand i had some questions about upgrades in there15:19
dansmithmriedem: I saw some of them were in the gate on friday yeah, thanks for that15:19
dansmithmriedem: I'm still feeling really terrible and I'm probably only going to be around until I can get the doc to call in an rx for me15:20
sdaguecfriesen_: that apache dir?15:20
dansmithbut if I have a chance I'll look at that15:20
mriedemdansmith: np15:20
mriedemit's not urgent15:20
cfriesen_sdague: 000-default.conf  keystone-wsgi-admin.conf  keystone-wsgi-public.conf  nova-placement-api.conf15:21
mriedem10000mg of z-pack is probably more important15:21
sdaguecfriesen_: well, that should at least be right15:21
sdaguecfriesen_: pastbin your local.conf?15:21
sdaguecfriesen_: sanity checking locally to make sure I didn't bork a thing15:22
*** arunman has joined #openstack-nova15:22
cfriesen_sdague: http://paste.openstack.org/show/608529/15:22
*** tbachman has quit IRC15:22
cfriesen_sdague: the git repos are pointing at my dev box15:23
sdaguecfriesen_: yeh, all that should be sane15:24
*** djohnsto has quit IRC15:24
*** arunman has quit IRC15:25
sdaguecfriesen_: after a clean.sh are there any python processes still running that seem odd?15:25
sdaguelike uwsgi ones15:25
sdaguethe only thing I can imagine is there is an errant keystone, so it's magic token is different15:26
*** yangyapepeng has joined #openstack-nova15:28
*** arunman has joined #openstack-nova15:28
*** tbachman has joined #openstack-nova15:30
cfriesen_sdague: checking15:30
*** cdent has joined #openstack-nova15:30
*** arunman has quit IRC15:31
*** yangyapepeng has quit IRC15:32
*** voelzmo has joined #openstack-nova15:33
lbragstadjohnthetubaguy i took another stab at simplifying https://review.openstack.org/#/c/460344/15:33
openstackgerritMatt Riedemann proposed openstack/nova master: Add online data migration for populating services.uuid  https://review.openstack.org/45489915:34
openstackgerritMatt Riedemann proposed openstack/nova master: db api: add get_service_by_uuid  https://review.openstack.org/46142815:34
openstackgerritMatt Riedemann proposed openstack/nova master: objects: add Service.get_by_uuid method  https://review.openstack.org/46142915:34
lbragstadjohnthetubaguy I'm going to propose a series of follow-on patches with more detail15:34
openstackgerritMatt Riedemann proposed openstack/nova master: db api: add service_get_by_uuid  https://review.openstack.org/46142815:35
openstackgerritMatt Riedemann proposed openstack/nova master: objects: add Service.get_by_uuid method  https://review.openstack.org/46142915:35
cfriesen_on another note...who's the person to talk to around virt.hardware.vcpus_realtime_topology() currently?15:36
*** iceyao has joined #openstack-nova15:37
cfriesen_sdague: no python processes running after unstack.sh/clean.sh15:37
sdaguecfriesen_: hmm... I don't know, things are starting fine here. How old are your clones of all the projects? Maybe something is required from keystone that I didn't realized15:38
*** iceyao has quit IRC15:41
cfriesen_sdague: it's been a while...and the box itself was upgraded in place from 14.04 to 16.04.   maybe I should blow it away and start from scratch.15:41
jaypipescdent: thanks very much!15:41
cdentthe font is... concerning15:42
cdentbut you've made the most of it15:42
jaypipescdent: as I mentioned, it's the Mirantis Boston template...15:42
jaypipesnothing I can do about that.15:42
cdentyeah, I know15:42
cdentthat's why I think it is great that you've "made the most of it'15:43
sdaguecfriesen_: the 403 typically means there is a credentials mismatch with the keystone server15:43
sdaguecfriesen_: if you want to keep debugging it, that's the thread to follow15:43
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-floating-ips endpoint instead of stevedore  https://review.openstack.org/46143115:43
sdagueit must be part of the cleanup, because it works in clean environments15:44
cfriesen_jaypipes: you +2'd this https://review.openstack.org/#/c/356383/ last year....just wondering why we have the implied "0-<n-1>" as part of the mask.  This means if the admin specifies "hw:cpu_realtime_mask=1" they actually get all the cpus, which seems odd.15:45
*** artom has quit IRC15:46
*** artom has joined #openstack-nova15:46
jaypipescfriesen_: how DARE you question my judgment!15:46
jaypipescfriesen_: heh, j/k :) will take a look.15:46
cfriesen_jaypipes: thanks...I also notice that this option doesn't seem to be documented in the manuals anywhere15:47
cfriesen_so I might have to fix that15:47
jaypipescfriesen_: that would be awesome if you did. ++15:47
*** artom has quit IRC15:48
*** arunman has joined #openstack-nova15:48
*** artom has joined #openstack-nova15:48
cfriesen_jaypipes: I've got another manual fix under review for what happens when flavor/image specify different things for cpu policy, cpu thread policy, mempage size15:48
*** ericyoung has quit IRC15:48
jaypipescfriesen_: link me and I'll add it to my queue.15:48
cfriesen_jaypipes: cool, will do15:49
jaypipesdanke15:49
*** salv-orl_ has joined #openstack-nova15:52
*** Apoorva has joined #openstack-nova15:52
*** Apoorva has quit IRC15:52
*** Apoorva has joined #openstack-nova15:53
cdentjaypipes: the new hotness on placement+uwsgi can be teased out of the results of https://review.openstack.org/#/c/456717/ but it's not immediately apparent without doing some digging so I'm not sure if it is worth it. May be easier/more simple for the sake of the presentation to say "it's just a simple wsgi app, run it with anything!"15:54
*** salv-orlando has quit IRC15:55
*** amotoki has quit IRC15:55
*** crushil has quit IRC15:57
*** sbezverk has joined #openstack-nova15:57
*** dklyle has quit IRC16:00
*** dklyle has joined #openstack-nova16:00
*** gyee has joined #openstack-nova16:00
*** Sukhdev has joined #openstack-nova16:00
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-floating-ip-pools endpoint instead of stevedore  https://review.openstack.org/46143616:02
*** salv-orl_ has quit IRC16:05
*** mdnadeem has quit IRC16:06
*** Swami has joined #openstack-nova16:08
*** nic has joined #openstack-nova16:09
*** arunman has quit IRC16:09
dansmithmriedem: hmm, looks like I never uploaded the patch set that fixed the setting mapped=1 thing16:11
dansmithmriedem: and it's not in my local tree since I pulled from gerrit over this at some point16:11
dansmiththat method looked quite different in that version16:11
* dansmith goes hunting through reflog16:11
openstackgerritMatthew Treinish proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947016:13
mtreinishmriedem: ^^^ part 2 to close out the spec16:13
mriedemmtreinish: excellent16:13
*** Sukhdev has quit IRC16:14
openstackgerritMatthew Treinish proposed openstack/nova master: Remove cloudpipe APIs  https://review.openstack.org/45947016:15
jaypipesmriedem: looks like another unit test would be affected by the compute_nodes.uuid unique index: http://logs.openstack.org/87/454887/4/check/gate-nova-python27-ubuntu-xenial/e5b8b91/testr_results.html.gz16:15
jaypipesmriedem: I blame dansmith ;)16:15
mriedemgah16:15
mriedemok16:15
*** crushil has joined #openstack-nova16:18
*** haplo37 has quit IRC16:20
*** iceyao has joined #openstack-nova16:22
*** haplo37 has joined #openstack-nova16:23
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-floating-ips-bulk endpoint instead of stevedore  https://review.openstack.org/46143916:24
*** dklyle is now known as david-lyle16:24
*** iceyao has quit IRC16:26
*** ijw has joined #openstack-nova16:26
*** bmace has quit IRC16:27
*** bmace has joined #openstack-nova16:28
jaypipescfriesen_: reviewed. good stuff.16:28
*** tesseract has quit IRC16:29
*** openstack has joined #openstack-nova17:00
*** openstack has joined #openstack-nova17:10
*** openstack has joined #openstack-nova17:23
cfriesen_but anyone that actually cares should probably specify which they want, so I'm not too worried about it.17:23
jaypipescdent, edleafe: maybe it would help if I just push the remainder of the patch series so you can see the end result.17:23
openstackgerritDan Smith proposed openstack/nova master: Make discover_hosts only query for unmapped ComputeNode records  https://review.openstack.org/42790317:23
dansmithmriedem: I was thinking, we could make the scheduler call the host discovery on SIGHUP, which may be more straightforward (i.e. you have to SIGHUP the scheduler to notice new hosts17:24
edleafejaypipes: I sure hope so :)17:24
jaypipesedleafe: thanks for the encouragement.17:24
*** baoli has quit IRC17:24
cdentjaypipes: could be, but it is also very useful for the rest of us who are not in your brain to actually talk about it and figure out the larger context of what's going on. For example my latest comment is basically about what "typical" is.17:24
cdentthink of this as an educational opportunity™17:24
*** psachin has quit IRC17:25
cdenta union of souls over one page17:25
* cdent struggles for more cliches17:25
jaypipescdent, edleafe: are you essentially asking me to add a "agg_ids" argument to _get_providers_sharing_capacity() that would winnow the resulting provider ID list to only the aggregates that have been previously selected in ResourceProviderList.get_all_by_filters()?17:25
openstackgerritChris Friesen proposed openstack/nova master: Tweak the cpu_realtime_mask handling slightly  https://review.openstack.org/46145617:25
cfriesen_jaypipes: what would you say to something like this?17:26
edleafejaypipes: no, not if you are then going to use that in a future join expression.17:26
edleafejaypipes: there wasn't any indication in the code that this would be happening17:27
jaypipesedleafe: precisely my plan, yes.17:27
jaypipesedleafe: what do you mean there wasn't any indication in the code? you mean the code comments didn't say how I'd be using the result of _get_providers_sharing_capacity()?17:28
cdentjaypipes: I'm not asking for a specific action, I'm simply trying to understand the intent and more importantly the context within which the stuff is going to operate. without that I can't review very effectively17:28
edleafejaypipes: your since-corrected comments also made it sound like it was going to be "in the same aggregate"17:28
jaypipescdent: k, then lemme push more patches that show that.17:28
jaypipesedleafe: k, lemme push some patches.17:28
cdentmy new strategy for everything is to be 5yo and ask "why". Soon everyone will hate me.17:29
*** trinaths has joined #openstack-nova17:29
*** kylek3h has quit IRC17:30
edleafecdent: "soon"??17:30
*** ijw has quit IRC17:30
cfriesen_cdent: your punishment is to watch all episodes of "Paw Patrol"17:30
cdentedleafe: just because you already hate me17:30
cdentcfriesen_: my mom doesn't let me watch older kids tv17:30
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-agents endpoint instead of stevedore  https://review.openstack.org/46145917:33
*** salv-orlando has joined #openstack-nova17:36
*** sree has joined #openstack-nova17:38
openstackgerritJay Pipes proposed openstack/nova master: placement: get providers sharing capacity  https://review.openstack.org/46079817:39
jaypipescdent: ^^ removed the aggregate stuff from the test case..17:40
jaypipesworking on follow up patches now. gimme about half hour.17:40
*** salv-orlando has quit IRC17:40
*** tongli has quit IRC17:41
cdentthanks jaypipes, it wasn't a huge sin that stuff being in there, it was just that it was misleading with regard to the thing being tested (without more insight into what's going on). It really did make me think that I was missing something in your code.17:41
cdentwill watch out for the new stuff17:41
*** lucasxu has joined #openstack-nova17:41
*** sree has quit IRC17:42
dansmithmriedem: what do you want me to put as the message for the NovaException in that coding error case? I don't want to reveal too much to the user17:42
*** baoli has joined #openstack-nova17:43
dansmithjust "Internal Error" ?17:43
mriedemumm17:43
mriedemdefault is msg_fmt = _("An unknown exception occurred.")17:43
mriedemmaybe that's good enough17:43
dansmithack17:44
*** iceyao has joined #openstack-nova17:44
*** hongbin has joined #openstack-nova17:45
jaypipescdent: yep, agree with ya.17:45
openstackgerritDan Smith proposed openstack/nova master: re-Allow adding computes with no ComputeNodes to aggregates  https://review.openstack.org/46061617:46
openstackgerritChris Dent proposed openstack/nova master: devref and reno for nova-{api,metadata}-wsgi scripts  https://review.openstack.org/46128917:47
openstackgerritChris Dent proposed openstack/nova master: Add pbr-installed wsgi application for metadata api  https://review.openstack.org/45941317:47
cdentI think I got those nits mriedem ^17:47
cdentbut could very well have add more17:47
openstackgerritKaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code  https://review.openstack.org/43985517:48
*** iceyao has quit IRC17:48
*** toure_ is now known as toure17:52
*** dane-fichter has quit IRC17:52
*** marst has joined #openstack-nova17:59
*** voelzmo has joined #openstack-nova18:00
*** voelzmo has quit IRC18:01
*** voelzmo has joined #openstack-nova18:02
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-assisted-volume-snapshots endpoint  https://review.openstack.org/46147018:05
mriedemcdent: yup, one tiny issue18:05
cdentyay!18:05
cdentnice catch18:06
*** brad[] has joined #openstack-nova18:07
openstackgerritChris Dent proposed openstack/nova master: devref and reno for nova-{api,metadata}-wsgi scripts  https://review.openstack.org/46128918:08
*** voelzmo has quit IRC18:12
*** openstackstatus has joined #openstack-nova18:14
*** ChanServ sets mode: +v openstackstatus18:14
*** tbachman has quit IRC18:14
*** mingyu has quit IRC18:15
openstackgerritMatt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload  https://review.openstack.org/46147318:16
*** fragatina has quit IRC18:23
*** fragatina has joined #openstack-nova18:23
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-availability-zone endpoint instead of stevedore  https://review.openstack.org/46147518:24
*** felipemonteiro has joined #openstack-nova18:26
*** jdurgin has joined #openstack-nova18:27
openstackgerritDan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path  https://review.openstack.org/45863618:29
openstackgerritDan Smith proposed openstack/nova master: Move to proper target_cell calling convention  https://review.openstack.org/45863418:29
openstackgerritDan Smith proposed openstack/nova master: Make target_cell() yield a new context  https://review.openstack.org/45863518:29
openstackgerritDan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack  https://review.openstack.org/45853718:29
openstackgerritMatt Riedemann proposed openstack/nova master: Add uuid to service.update notification payload  https://review.openstack.org/46147318:29
*** deniz has joined #openstack-nova18:29
*** mvensky has joined #openstack-nova18:31
*** tbachman has joined #openstack-nova18:35
openstackgerritMathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change  https://review.openstack.org/45432218:36
*** salv-orlando has joined #openstack-nova18:37
*** ijw has joined #openstack-nova18:37
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-fixed-ips endpoint instead of stevedore  https://review.openstack.org/46147918:38
*** salv-orlando has quit IRC18:41
*** trinaths has left #openstack-nova18:44
*** iceyao has joined #openstack-nova18:49
*** mdrabe_ has quit IRC18:50
*** mdrabe_ has joined #openstack-nova18:51
*** fragatina has quit IRC18:52
jaypipesI think dansmith is going to add some vomit to his snot when he sees the SQL in my next patch.18:53
*** iceyao has quit IRC18:53
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore  https://review.openstack.org/46148018:56
*** slaweq has quit IRC18:57
*** jamesdenton has joined #openstack-nova18:57
cdentvomitty snot! \o/18:59
*** ijw has quit IRC19:01
edleafethat reminds me... lunchtime!19:01
*** voelzmo has joined #openstack-nova19:01
*** kfarr has joined #openstack-nova19:02
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-availability-zone endpoint instead of stevedore  https://review.openstack.org/46147519:04
openstackgerritEric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa  https://review.openstack.org/46148119:05
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-fixed-ips endpoint instead of stevedore  https://review.openstack.org/46147919:05
efriedmriedem mordred jaypipes ^^19:05
openstackgerritGhanshyam Mann proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore  https://review.openstack.org/46148019:05
*** fragatina has joined #openstack-nova19:08
*** gouthamr has joined #openstack-nova19:14
*** dtroyer has joined #openstack-nova19:14
*** david-lyle is now known as dklyle19:17
*** dklyle is now known as david-lyle19:17
*** fragatina has quit IRC19:17
*** gouthamr has quit IRC19:18
dtroyerI've got a question on the interaction between aggregates and cells…   I have a Mitaka DevStack configuration that has n-cell enabled; the hosts from "nova host-list" look like cell!region@vxd-mitaka, which aggregate-add-host does not like.  Is this not a valid thing?  Or am I missing something here?19:19
jaypipesefried: gonna be while...19:19
*** adisky_ has quit IRC19:19
mriedemdtroyer: cells v1 doesn't support aggregates19:20
efriedjaypipes Don't think there's a hurry.  mriedem asked me to update based on [glance]api_servers decision.19:20
dtroyermriedem: cool, that's what I needed to know.  is there a way from the API to know if I have cells v1 or v2?19:20
mriedemdtroyer: well, one thing i guess is GET /os-cells would return something for cells v1 if enabled https://developer.openstack.org/api-ref/compute/#cells-os-cells-capacities19:21
mriedemthat's admin-only by default19:21
mriedemthe api i mean19:21
mriedembut cells v1 runs the separate n-cell service, that's how devstack checks it19:21
mriedemdtroyer: cells v2 is baked in, not a separate service or API19:21
dtroyermriedem: thanks.  I'm trying to do graceful failures in OSC in the aggregate commands so admin-only is OK19:21
mriedemdtroyer: so starting in ocata, you're running cells v2 by design19:21
dtroyerah, gotcha19:22
mriedemeven if you're running cells v1, you have cells v2 :)19:22
* dtroyer is mostly cell-ignorant19:22
mriedemconsider your mind blown19:22
*** jamesdenton has quit IRC19:22
mriedemdtroyer: the other thing would probably be the /os-services API19:22
mriedemhttps://github.com/openstack/nova/blob/master/nova/cmd/cells.py#L4119:23
dtroyermriedem: one more question… I have a note that nova-net required cells v1, that doesn't seem right to me…19:23
mriedemif you're running cells, there should be a nova-cells service binary in the GET /os-services response19:23
mriedemi think anyway19:23
mriedemdtroyer: other way around19:23
mriedemwell,19:23
mriedemcells v1 support in-tree requires nova-net,19:24
dtroyerok, that explains a couple of things.  thanks19:24
mriedemdtroyer: in ocata we made nova-network only start under a cells v1 config19:24
mriedemthat's probably the confusion19:24
dtroyerI bet so19:24
mriedemwe basically just continued to disable nova-net in ocata,19:25
mriedemand switched all jobs to use neutron except for the cells v1 job19:25
*** yangyapepeng has joined #openstack-nova19:29
*** iceyao has joined #openstack-nova19:31
dansmithjaypipes: I don't hate your sql, I admire it.19:31
dansmithjaypipes: I just note that I shan't be debugging it if something goes wrong19:31
cdenttshirt19:31
*** slaweq has joined #openstack-nova19:32
*** awaugama has quit IRC19:34
*** yangyapepeng has quit IRC19:35
*** iceyao has quit IRC19:35
openstackgerritEric Fried proposed openstack/nova-specs master: Updates for [glance]api_servers and ksa  https://review.openstack.org/46148119:39
*** ijw has joined #openstack-nova19:49
*** ijw has quit IRC19:49
*** ijw has joined #openstack-nova19:49
*** ijw has quit IRC19:50
*** iceyao has joined #openstack-nova19:51
openstackgerritEd Leafe proposed openstack/nova master: Remove the can_host column  https://review.openstack.org/46149419:53
*** gcb has quit IRC19:53
*** gcb has joined #openstack-nova19:54
*** kfarr has quit IRC19:56
melwittjaypipes: fyi I pushed an update to the quotas series attempting to address your comments https://review.openstack.org/#/c/44623919:56
jaypipesmelwitt: yup, reviewing it now :)19:56
jaypipeswhile waiting for functional tests to complete..19:56
*** iceyao has quit IRC19:56
melwittjaypipes: cool, thanks :)19:57
*** kfarr has joined #openstack-nova19:57
*** Apoorva has quit IRC19:58
*** annegentle has joined #openstack-nova19:59
ildikovjaypipes: around for a meeting on #openstack-meeting-3? :)20:02
jaypipesildikov: I'm sorry, Ildiko, I'm right in the middle of debugging a nasty little feature and really need to focus on it.20:03
jaypipessorry :(20:03
jaypipesildikov: will read back from channel later.20:03
ildikovjaypipes: ok, let's catch up later then20:03
*** Apoorva has joined #openstack-nova20:03
*** mingyu has joined #openstack-nova20:04
jaypipesthx ildikov20:04
*** mingyu has quit IRC20:08
*** jamesdenton has joined #openstack-nova20:10
*** gcb has quit IRC20:13
*** gcb has joined #openstack-nova20:14
*** cfriesen_ has quit IRC20:14
*** liverpooler has quit IRC20:14
*** dimtruck is now known as zz_dimtruck20:25
*** fragatina has joined #openstack-nova20:26
*** voelzmo has quit IRC20:27
*** iceyao has joined #openstack-nova20:32
*** smatzek_ has quit IRC20:36
*** iceyao has quit IRC20:37
*** mdrabe_ is now known as mdrabe20:37
*** jamesdenton has quit IRC20:38
*** kfarr has quit IRC20:38
*** jamesdenton has joined #openstack-nova20:40
*** zz_dimtruck is now known as dimtruck20:45
*** ijw has joined #openstack-nova20:50
*** cfriesen_ has joined #openstack-nova20:51
*** haplo37 has quit IRC20:51
*** annegentle has quit IRC20:52
*** ijw has quit IRC20:55
*** thorst has quit IRC20:56
*** dtp has joined #openstack-nova20:57
*** samarendra has left #openstack-nova20:58
*** Sukhdev has joined #openstack-nova20:59
*** smatzek_ has joined #openstack-nova20:59
*** dfisher has left #openstack-nova21:03
*** ijw has joined #openstack-nova21:04
*** slaweq has quit IRC21:05
*** markvoelker has quit IRC21:06
*** sdague has quit IRC21:06
*** eharney has quit IRC21:07
*** ijw has quit IRC21:07
*** Sukhdev_ has joined #openstack-nova21:08
*** annegentle has joined #openstack-nova21:09
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_service_url()  https://review.openstack.org/45825721:10
*** artom has quit IRC21:12
*** artom has joined #openstack-nova21:12
*** iceyao has joined #openstack-nova21:13
*** crushil has quit IRC21:13
*** artom has quit IRC21:14
*** artom has joined #openstack-nova21:14
*** thorst has joined #openstack-nova21:15
*** dillaman has quit IRC21:15
*** smatzek_ has quit IRC21:16
mriedemoh boy, so my daughter got home and there is a note from the teacher in her backpack about inappropriate behavior,21:16
mriedeme.g. a worksheet of sight words to fille out, every entry has the word "butt" in it21:17
mriedemshe at least spelled it correctly...21:17
*** iceyao has quit IRC21:18
*** Qiming has quit IRC21:19
*** thorst has quit IRC21:20
*** READ10 has quit IRC21:20
*** krtaylor has quit IRC21:21
*** Qiming has joined #openstack-nova21:21
*** jdurgin has quit IRC21:22
cdent\o/21:23
*** edmondsw has quit IRC21:24
*** ijw has joined #openstack-nova21:25
*** ijw has quit IRC21:25
dansmithmriedem: sight words?21:25
*** ijw has joined #openstack-nova21:25
mriedemdansmith: yeah, like simple single-syllable words that they should recognize immediately21:26
openstackgerritDan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test  https://review.openstack.org/46151321:26
mriedemthey, here, were, are, them21:26
mriedembutt21:26
dansmithmriedem: and butt is inappropriate?21:26
dansmithmaybe the teacher should read "everybody poops"21:26
imacdonnI'd recognise a butt anywhere!21:26
mriedemwell, not only that, but the picture of a butt pooping21:26
dansmithmriedem: I dunno, I think I'd go argue with the teacher21:26
*** cfriesen_ has quit IRC21:27
* tbachman just misread dansmith’s patch subject as “delete-while-butting”. #powerofsuggestion21:28
cdenttbachman++21:28
dansmithtbachman: that would be inappropriate (apparently)21:28
*** dillaman has joined #openstack-nova21:29
*** yangyapepeng has joined #openstack-nova21:30
edleafemriedem: the apple doesn't fall far...21:30
*** ijw has quit IRC21:31
melwittwat is this "UnicodeDecodeError \'ascii\' codec can\'t decode byte 0xc3 in position 3575: ordinal not in range(128)" coming from the cells v1 tempest job21:32
mriedemedleafe: it's true, it reminds me of things i got in trouble for in school21:32
mriedemmelwitt: likely means we aren't using six.text_type somewhere21:32
mriedemmelwitt: link and stacktrace?21:33
* melwitt kicks dirt21:33
mriedemlike, LOG.warning('something %s' % unicode) will do that21:33
*** fragatin_ has joined #openstack-nova21:33
*** krtaylor has joined #openstack-nova21:34
*** ijw has joined #openstack-nova21:34
melwittmriedem: http://logs.openstack.org/39/446239/5/check/gate-tempest-dsvm-cells-ubuntu-xenial/7061c58/logs/screen-n-cell-region.txt.gz?#_May_01_20_22_15_32426421:34
*** ijw has quit IRC21:35
*** yangyapepeng has quit IRC21:35
*** ijw has joined #openstack-nova21:35
mriedemhmm https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L101121:35
cdentedleafe: I guess you didn't run all the functional tests before pushing that can_host change. I can no longer trust you!21:35
*** yassine has joined #openstack-nova21:35
mriedemmelwitt: i've seen something like that in n-cpu where we log an instance with plugging vifs21:36
mriedemit dumps the entire instance and there must be something in there that has unicode in it21:36
*** yassine is now known as Guest7383721:36
melwitthm, okay. thanks21:36
cdentmelwitt: saw some melwitt I've just started seeing that too21:36
mriedemmelwitt: https://bugs.launchpad.net/nova/+bug/158072821:36
openstackLaunchpad bug 1580728 in OpenStack Compute (nova) "UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 386: ordinal not in range(128) in nova.virt.libvirt.vif:unplug with unicode instance.display_name" [Medium,Triaged]21:36
melwittI don't think I did anything to cause that but maybe I did21:36
*** baoli has quit IRC21:37
*** fragatina has quit IRC21:37
mriedemin ^ it's unicode in the instance display name21:37
cdenthttp://logs.openstack.org/13/459413/5/check/gate-tempest-dsvm-cells-ubuntu-xenial/da51ac2/console.html#_2017-05-01_20_33_50_88786621:37
melwittcdent: ack. good it's not my patch maybe probably21:37
*** flaper87 has quit IRC21:38
melwittthat's weird, I wonder what made this start happening all of a sudden. implies something recently changed to unicode21:38
mriedemcheck logstash21:38
*** jamesdenton has quit IRC21:39
*** salv-orlando has joined #openstack-nova21:39
*** jdurgin has joined #openstack-nova21:40
mriedemi only see 2 hits in logstash21:40
melwittdoing. I also found earlier today that tempest.api.volume.admin.test_volume_quota_classes.VolumeQuotaClassesTest.test_update_default_quota fails intermittently. I'm guessing it's something to do with parallel tests racing21:41
*** lucasxu has quit IRC21:42
*** krtaylor has quit IRC21:42
cdentmriedem: what did you search on?21:42
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI._find_service  https://review.openstack.org/46151921:43
mtreinishmelwitt: heh, the default quota tests are an inherent race condition21:43
mriedemcdent: i deleted it :) sec21:43
melwittmessage:"UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 413: ordinal not in range(128)"21:43
mriedemmelwitt: the default quota compute api tests in tempest are locked to run serially21:43
cdentcuz i've got 259 in the last 6 hours on message:"decode byte 0xc3 in position" which is clearly too general, but...21:43
melwittis what I searched on and see the same, only two hits21:43
mtreinishbut I think we set it to have sane defaults so we don't ever decrease the default quotas beyond where they start (which would be a big problem)21:43
mriedemmessage:"UnicodeDecodeError" AND message:"instance_update_at_top" AND tags:"screen-n-cell-region.txt"21:44
*** salv-orlando has quit IRC21:44
*** jdurgin has quit IRC21:45
edleafecdent: I was meaning to mark it WIP and forgot. I knew it wasn't fully tested.21:45
melwittmtreinish, mriedem: so they're set to run serially ... the thing I see is e.g. (u'snapshots_tempest-QosSpecsTestJSON-volume-type-1219921907', -1) not in [(u'gigabytes', 1001), (u'backups', 11), (u'snapshots_lvmdriver-1', -1), (u'volumes_lvmdriver-1', -1), (u'volumes', 11), (u'per_volume_gigabytes', -1), (u'backup_gigabytes', 1001), (u'gigabytes_lvmdriver-1', -1), (u'snapshots', 11)]21:46
cdentedleafe: my bad really, I was being far too optimistic21:46
melwittmtreinish, mriedem: as though some other test had added a quota default and it gets picked up unexpectedly in another21:47
* cdent has run out of usefulness 21:48
cdentg'night all21:48
*** cdent has quit IRC21:48
mtreinishmelwitt: do you have a link to that failure?21:50
*** catintheroof has joined #openstack-nova21:50
melwittmtreinish: yeah, sorry I didn't include it http://logs.openstack.org/39/446239/5/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0b6acd6/logs/testr_results.html.gz21:51
*** markvoelker has joined #openstack-nova21:57
mtreinishmelwitt: that's really weird. It looks like tempest sends a PUT with the new default quota entries, cinder returns 200, and the tempest does a GET to list the default quotas and the stuff it just added isn't there21:57
*** jdurgin has joined #openstack-nova21:58
mtreinishmelwitt: oh, I think you might be right about a race21:58
mriedemdansmith: while looking at this https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4508 for using uuid rather than id, i realized this could have been a direct lookup if we stored the compute node uuid on the host mapping, but we only store the ComputeNode.host field...21:58
*** Swami has quit IRC21:59
melwittmtreinish: hm, what I thought is that update_body is returning an additional unexpected item that was added by a different test21:59
*** slaweq has joined #openstack-nova22:01
mtreinishmelwitt: the test does a GET to get all the default quotas, loops over that list increases each quota there by 1 with a big update, then does another list and loops over them to see everything was updated22:02
mtreinishthe failure looks like another test class is making a default quota set and cleaning up after itself between the first get and after the update22:03
mtreinishand tempest is failing on the verification that everything it asked to be updated is22:03
mtreinishbecause a bunch of stuff was deleted22:03
mtreinishat least that's what it looks like22:04
melwittokay. I'm looking at the code too but highly possible I've not understood it :)22:04
mtreinishI'm having a hard time figuring out where the other default quotas are coming from though. It says volumes_tempest-QosSpecsTestJSON-volume-type-1219921907 which should mean the QosSpecsTestJSON class created it22:05
mtreinishbut I don't see where that happens in the code22:05
mtreinishbut I could just not be seeing it22:05
*** iceyao has joined #openstack-nova22:05
melwittI was focusing on the part where it's asserting (new, old) and I thought it mean there was something unexpected in "new", the return from the PUT22:07
melwittmaybe I got that backward because I'm not familiar with the ContainsAll matcher22:08
*** slaweq has quit IRC22:09
*** iceyao has quit IRC22:10
mtreinishmelwitt: iirc the contains all matcher is basically a for loop that does: for x in arg[0]: self.assertIn(x, arg[1])22:11
mtreinishbut I'd have to dig up the code to be sure22:11
*** thorst has joined #openstack-nova22:11
melwittagreed that it should mean the QosSpecsTestJSON created it but all I see is e.g. "cls.create_test_qos_specs(...)" and I dunno what all that entails. if it could possibly include creating a quota default22:12
mtreinishmelwitt: but looking at the tempest log: http://logs.openstack.org/39/446239/5/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0b6acd6/logs/tempest.txt.gz#_2017-04-29_03_26_53_00022:12
mtreinishthe teardownclass is deleting volume types around the same time the default quotas tests are doing their thing22:12
mtreinishso I think we want to add an external lock between the classes to protect against that22:13
*** xyang1 has quit IRC22:15
*** thorst has quit IRC22:16
mtreinishmelwitt: can you open a bug for this so we can track it with e-r? I can work on a fix for it. It shouldn't be too hard22:16
melwittmtreinish: sure22:16
mtreinishunless you wanted to work on the patch :)22:17
*** ijw has quit IRC22:17
*** ijw has joined #openstack-nova22:17
melwittmtreinish: no thanks :)22:20
*** deniz has left #openstack-nova22:21
*** ijw has quit IRC22:22
dansmithmriedem: you mean we could have known which cells to check instead of iterating over all of them?22:22
*** mlavalle has quit IRC22:22
dansmithmriedem: the id isn't unique, so it only buys us not iterating over all the cells in the case where the compute id happens to be uniquer22:23
*** ijw has joined #openstack-nova22:23
dansmithbut on average, for similar-sized cells, you'll have nearly 100% overlap22:23
openstackgerritEd Leafe proposed openstack/nova master: Remove the can_host column  https://review.openstack.org/46149422:25
melwittmtreinish: https://bugs.launchpad.net/tempest/+bug/168748822:28
openstackLaunchpad bug 1687488 in tempest "VolumeQuotaClassesTest.test_update_default_quota fails intermittently" [Undecided,New]22:28
mtreinishmelwitt: thanks22:29
*** marst has quit IRC22:29
*** haplo37 has joined #openstack-nova22:31
mriedemdansmith: right i meant we could just take the compute node uuid to look up a host mapping to find out the cell it's in, but right now we'll have to iterate the cells (like we do if given the primary key id)22:31
mriedemnot a huge deal, just something i thought of when seeing your comment in that host api method22:32
*** browne has joined #openstack-nova22:32
dansmithyou mean once we have the uuid from the api, then we can use the uuid for the lookup, right?22:32
mriedemyes22:32
dansmithor do you mean if we had uuid on the hostmapping we could avoid the cell scan?22:32
mriedemthat's what i'm adding to HostAPI.compute_node_get now, but we still have to iterate the cells22:32
mriedemcorrect22:32
mriedemdansmith: https://gist.github.com/mriedem/013d1bcb9fd547a61adcccb99b9f0eae22:33
dansmithokay, since hostmapping is 1:1 to compute node that should be okay, I think.. but we don't want to do the same for service I think22:33
mriedemwe don't have a ServiceMapping in the api db22:33
mriedemso we can't22:33
dansmithwhat I meant was, we can't just tack the service uuid onto hostmapping too22:34
mriedemoh, ok yeah agree22:34
mriedemif we did add uuid to hostmappings, we'd have to have an online data migration22:34
mriedemso it becomes less fun22:34
dansmithyeah ;)22:34
mriedemi bet at some point this will be one of those things we wish we had though, like bdm.uuid22:35
mriedembut if we really need it at some point, we could add it later22:35
*** lyan has quit IRC22:37
*** marst has joined #openstack-nova22:39
*** catintheroof has quit IRC22:39
*** salv-orlando has joined #openstack-nova22:40
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_service_url()  https://review.openstack.org/45825722:41
*** fragatin_ has quit IRC22:42
*** fragatina has joined #openstack-nova22:43
*** browne has quit IRC22:43
*** salv-orlando has quit IRC22:45
*** thorst has joined #openstack-nova22:46
*** iceyao has joined #openstack-nova22:50
*** browne has joined #openstack-nova22:54
*** iceyao has quit IRC22:56
*** chyka has quit IRC22:56
*** chyka has joined #openstack-nova22:56
*** thorst has quit IRC22:59
*** thorst has joined #openstack-nova22:59
*** dimtruck is now known as zz_dimtruck23:01
*** edmondsw has joined #openstack-nova23:02
*** chyka has quit IRC23:03
*** cfriesen_ has joined #openstack-nova23:10
*** felipemonteiro has quit IRC23:11
*** crushil has joined #openstack-nova23:12
*** jamesdenton has joined #openstack-nova23:13
*** Sukhdev has quit IRC23:16
*** ijw has quit IRC23:19
efriedmordred jaypipes johnthetubaguy FYI ^^ is failing Jenkins because it Depends-On a keystoneauth patch which apparently isn't get picked up.  (Are there only certain projects Depends-On works with?)23:22
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuid in HostAPI.compute_node_get  https://review.openstack.org/46153223:22
*** ijw has joined #openstack-nova23:22
*** ijw has quit IRC23:23
*** ijw has joined #openstack-nova23:23
*** jamesdenton has quit IRC23:23
mriedemefried: depends-on doesn't work if the dependent change is in a library or client,23:23
mriedemunless you have a job that uses LIBS_FROM_GIT23:23
mriedemin devstack23:24
efriedmriedem Okay, good to know.23:24
efriedFWIW, it live-tests okay ;-)23:24
*** crushil has quit IRC23:26
cfriesen_for testing purposes is there a helper function to compare dicts where one outputs keys/values as strings and the other as unicode?23:27
efriedcfriesen_ map()?23:29
efriedOr this? http://stackoverflow.com/questions/1254454/fastest-way-to-convert-a-dicts-keys-values-from-unicode-to-str23:30
*** yangyapepeng has joined #openstack-nova23:31
cfriesen_efried:  I was wondering if nova (or oslo) already had something23:31
*** iceyao has joined #openstack-nova23:32
openstackgerritTakashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls  https://review.openstack.org/41387623:33
*** takashin has joined #openstack-nova23:33
*** yangyapepeng has quit IRC23:36
*** iceyao has quit IRC23:37
*** crushil has joined #openstack-nova23:38
*** salv-orlando has joined #openstack-nova23:41
*** viks has joined #openstack-nova23:42
*** cfriesen_ has quit IRC23:42
vikshi all, seeking attention on Nova libvirt volume driver for Veritas HyperScale at https://review.openstack.org/#/c/443951/ . Pre-req of OS-brick has already been merged(https://review.openstack.org/#/c/442754/ , https://review.openstack.org/#/c/458688/)23:45
*** salv-orlando has quit IRC23:45
mordredefried: sweet. I was travelling all today but will try to connect with keystone folks tomorrow to follow up on that23:48
*** rfolco has joined #openstack-nova23:54
*** jamielennox is now known as jamielennox|away23:56
*** edmondsw has quit IRC23:56
*** artom has quit IRC23:57
*** artom has joined #openstack-nova23:57
*** jamielennox|away is now known as jamielennox23:58
*** gcb has quit IRC23:59

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