Tuesday, 2015-09-29

*** rosmaita has quit IRC00:47
*** rosmaita has joined #openstack-searchlight00:49
*** sballe has quit IRC02:06
*** nikhil_k has joined #openstack-searchlight04:16
*** nikhil has quit IRC04:18
*** openstackgerrit has quit IRC06:16
*** openstackgerrit has joined #openstack-searchlight06:17
*** lakshmiS has joined #openstack-searchlight06:46
*** lakshmiS_ has joined #openstack-searchlight08:45
*** lakshmiS has quit IRC08:49
*** lakshmiS_ has quit IRC11:15
*** lakshmiS has joined #openstack-searchlight11:20
*** sigmavirus24_awa is now known as sigmavirus2414:10
sjmc7lakshmiS - for the facet patch, you may need to recreate your nova server mapping14:18
sjmc7had to change the security groups mapping to match the data14:19
nikhil_kTravT: sjmc7: lakshmiS: some feedback was provided at the drivers meeting today on the notification patch14:32
sjmc7ah yes?14:32
nikhil_khttp://eavesdrop.openstack.org/meetings/glance_drivers/2015/glance_drivers.2015-09-29-14.01.html14:32
nikhil_kthey don't want it for RC backport14:32
lakshmiShmm14:32
sjmc7i guess it makes sense... as pointed out though, it sat around for a long time and the complexity of the change is because of shortcuts taken when image members were first done14:35
sjmc7there's not much we can do i guess14:35
lakshmiSgoing though the log. we will have to think about some alternative in searchlight for liberty backport14:38
sjmc7there really isn't one other than occasional reindexing14:39
sjmc7or backporting the change themselves, which is likely what we would do14:40
lakshmiSor not allow access to members at all - i guess travis was leaning towards that14:40
lakshmiSits better to not show than miss something14:41
sjmc7yeah. do you know how widely used that feature is?14:41
lakshmiSdont know14:41
sjmc7maybe taking it out is safest14:41
lakshmiSyes14:41
sjmc7though we have to do it today i guess :)14:41
lakshmiSi will take a look. might be simpler i think like just removing rbac filter14:42
sjmc7and the field, since the data can get stale14:42
lakshmiSyou mean remove members fields from image document?14:43
sjmc7yeah14:43
lakshmiSmakes sense to remove the mapping14:44
lakshmiSah its bad. i was about to land a patch on processing the image member notifications in searchlight14:45
lakshmiSi will go ahead and commit it but lets not merge it14:45
sjmc7yeah, no harm putting it up14:46
TravTsorry guys, had to take kids to school.15:09
TravTbasically, this makes images almost worthless for horizon15:09
lakshmiSthat doesnt sound good15:09
sjmc7without members?15:09
TravTyes15:10
sjmc7why?15:10
TravTwhen you are in the project view, you see three groups of images.15:10
TravTthose that are in you project, those that are public, and those that shared with you15:10
TravTso, you could provide searching and index view of your project and public, but now there's all those others15:10
TravTand the only way to get that is to either go back to API or we'll have to do re-index15:11
lakshmiSre-index periodically?15:11
TravTthat's another option, but doing a full reindex is problematic on its own.15:13
TravTunless we do it without the "clear" option15:13
lakshmiSbut what triggers the full index15:13
TravTmeaning we do a non-destructive re-indexing15:13
TravTand only update data.15:14
TravTjust reading the full logs.....15:15
openstackgerritLakshmi N Sampath proposed openstack/searchlight: Fix for updating image index on member notifications  https://review.openstack.org/22899815:16
TravTsjmc7: lakshmiS: nikhil_k: i think that if we can get it in on mitaka that will just have to be good enough.  we will put in some release notes somehow and let deployers know that to use this with liberty they have two options15:19
TravT1) pull the patch themselves and apply15:20
TravT2) set up a cron for periodic re-index.15:20
sjmc7yeah, i don't see any other options. we would go for option 1)15:20
TravTfor 2) we actually need to add back the option to the index sync job to not clear the data.15:20
sjmc7:)15:20
TravTmaybe update only (no adds)15:21
openstackgerritMerged openstack/searchlight: Rename GLANCE_TEST_SOCKET_FD_STR  https://review.openstack.org/22864215:21
TravTso, perhaps we should look at supporting that option anyway.15:22
lakshmiSyeah lets push for that in glance15:22
TravTlakshmiS: you mean push for landing it in Mitaka?15:23
lakshmiSyeah15:23
lakshmiSi added a patch on consuming the events in searchlight which we should merge for mitaka release too15:24
TravTif history is a guide, then we can expect actual reviews to come in milestone 315:24
lakshmiSits always is15:24
TravTso, i think we should jump over to the glance room now and see if flaper87 is around.  ask for actual reviews sooner than later.15:25
lakshmiSsure15:26
TravTok,15:26
TravTjust some crickets over there.15:27
lakshmiSthey were active until a while back15:27
TravTits around that time of day when guys in Europe transition from the office.15:28
sjmc7we prefer the term "go to the pub"15:28
lakshmiSeven though i did chat with flaper87 earlier and he was interested in landing it in mitaka sooner15:29
TravTwell, that's good.15:30
sjmc7it didn't sound like anyone was opposed to the change, just that it had sat there until too late15:31
lakshmiSit was borderline bug and feature and didnt want in RC15:32
sjmc7TravT - though it is our last day, i was planning to try to get patches up for https://bugs.launchpad.net/searchlight/+bug/1496464 and https://bugs.launchpad.net/searchlight/+bug/1500904. i didn't see anything more urgent, and neither of them are particularly crucial15:32
openstackLaunchpad bug 1496464 in OpenStack Search (Searchlight) "Admin rbac filtering too lenient " [Undecided,New]15:32
openstackLaunchpad bug 1500904 in OpenStack Search (Searchlight) "Add 'id' to metadef namespaces" [Undecided,New]15:32
TravTsounds good.  The admin rbac filtering is actually pretty important i think.15:33
TravTalthough, we have to think about what this means for glance and members.15:34
sjmc7yeah. i'm going to have it default to applying normal rbac even for admins unless all_projects is given15:34
TravTglance is actually a bit more tricky than just doing it or or not...  but i guess all_projects still kinda works.15:36
TravTsjmc7: lakshmiS: take a look at this: https://github.com/openstack/searchlight/blob/master/searchlight/elasticsearch/plugins/glance/images.py#L7915:37
TravTthat query won't ever return shared images anyway, i think15:37
TravToh wait, it is an OR15:38
TravTnvrmnd, i apparently need to get more cofee15:38
TravTcoffee15:38
lakshmiS:)15:38
lakshmiSwill log back after my dinner...15:39
TravTok. how's your review backlog looking?15:39
TravTgo get some food. :)15:40
lakshmiSthere are two more i was going to look at but lets reprioritize after i come back15:41
TravTi'm going to be working on horizon plugin right now to see if there are any more hidden issues that we haven't thought of from a use perspective.15:41
TravTmy plan is to cherry pick all the current patches for searchlight onto local and hit it15:41
lakshmiSok i am planning to be online tonight for any last minute patches15:42
TravTok. getting these ones properly reviewed and merged are highest priority15:43
TravThttps://launchpad.net/searchlight/+milestone/liberty-rc115:43
TravTbut should we target your notification listener patch for RC1 as well?15:43
lakshmiSthere's no harm since it will never recieve any notifications to act on15:43
lakshmiSwhen developers patch glance in liberty they will one less patch  to apply15:44
TravTok. I am thinking I should go ahead and put up a release notes doc or something.15:44
TravTor we'll just add a note to the glance rst.15:44
lakshmiSyeah15:44
lakshmiS:)15:45
lakshmiSi dont know if glance team will like that15:45
TravTno, i mean our glance plugin rst15:45
TravThttps://github.com/openstack/searchlight/blob/master/doc/source/plugins/glance.rst15:45
lakshmiSoh that one.15:45
TravTMaybe we'll add a release notes section to the bottom of each one to track differences from release to release...15:46
TravTok, go eat and see you shortly15:46
lakshmiSkragniz used to add that for glance similarly15:46
TravTyeah, i miss kragniz15:46
*** lakshmiS has quit IRC15:48
kragnizTravT: shhh15:50
TravTkragniz!15:50
TravThow are you doing?15:51
kragnizI'm in a lecture at the moment, being all studenty15:51
sjmc7i also prefer the term "at the pub"15:51
TravTstill doing some openstacky stuffs too?15:51
kragnizhere and there15:52
TravTsjmc7: i just opened one of my 9706 unread emails... looks like they are re-arranging our office and i am getting moved tomorrow. so i have to go in an pack things up.  I guess that's a couple hours down the drain today.15:52
kragnizI'll do more once I settle back in at uni15:52
sjmc7maybe someone'll do it for you if you don't show up15:53
TravTkragniz: well, hope you are having fun.15:53
TravTif you are ever looking to learn more about elasticsearch and apply your skills, i happen to know somewhere that you could do that15:53
TravTsjmc7: i'm pretty sure i'll never see most of it again... maybe that wouldn't be a bad thing15:54
sjmc7possessions are fleeting15:54
kragnizheh15:54
kragnizI miss all the openstack fun15:54
openstackgerritMerged openstack/searchlight: Add additional facet fields  https://review.openstack.org/22806315:54
kragnizI'll be in tokyo, though15:55
TravTwhoa!15:55
TravThow'd you swing that?15:55
TravTthat's cool15:55
kragnizthank the openstack foundation!15:55
TravTsweet15:56
openstackgerritMerged openstack/searchlight: Fix sorting behavior for common problematic fields  https://review.openstack.org/22637816:01
nikhil_kTravT: just saw the note on the notification patch. thanks, we are good for mitaka on that16:04
TravTyeah, thanks nikhil_k for keep on pursuing that.16:08
TravTas you see, i just had a quick conversations with flaper87 in glance room16:08
*** TravT has quit IRC16:11
*** sballe has joined #openstack-searchlight16:12
*** TravT has joined #openstack-searchlight16:13
*** GB21 has joined #openstack-searchlight16:14
nikhil_kTravT: absolutely, I think it can land soon16:25
TravTwe'll keep our fingers crossed. :)16:27
*** lakshmiS has joined #openstack-searchlight16:53
openstackgerritLakshmi N Sampath proposed openstack/searchlight: Fix for updating image index on member notifications  https://review.openstack.org/22899817:10
rosmaitaTravT: got a minute for a metadefs question?17:12
TravTrosmaita: sure.  lakshmiS is here as well.17:12
rosmaitacould you take a quick look at this: https://review.openstack.org/#/c/218869/4/etc/metadefs/compute-guest-memory-backing.json17:13
rosmaitaparticularly the resource_type_associations17:13
rosmaitait looks to me like the OS::Nova::Flavor element is repeated twice17:13
TravTyeah, looking now17:13
rosmaitajust wondering if that's normal or a mistake17:13
rosmaitathanks!17:13
TravTno, that looks like a mistake to me.17:14
rosmaitaok, cool17:14
TravTi wonder if he meant OS::Glance::Image17:14
TravTor OS::Glance::Aggregate17:14
TravTi meant OS::Nova::Aggregate17:14
rosmaitai'll put a note on the review (unless you want to)17:15
TravTsure, i can add a note17:15
rosmaitaok, thanks17:15
lakshmiSunless if they need a different prefix17:15
TravTyeah, but OS::Nova::Flavor is repeated17:17
rosmaitai am definitely going to buckle down and learn metadefs17:17
rosmaitaespecially since i have been proposing them as a solution to various problems17:18
TravT:)17:18
TravTrosmaita: recently somebody at yahoo was asking me more about them, because i think they are starting to use them.17:20
rosmaitacool17:20
TravTit takes awhile for things to start getting picked up and used17:20
openstackgerritSteve McLellan proposed openstack/searchlight: Apply RBAC for admins unless all_projects is given  https://review.openstack.org/22905817:24
*** lakshmiS has quit IRC17:56
*** lakshmiS has joined #openstack-searchlight17:59
ekarlsohttps://review.openstack.org/#/c/219271/ < doesn't work ?17:59
TravTekarlso?17:59
TravTit doesn't work for you?17:59
ekarlsoehm, I mean ready for merge ?18:00
TravTi think so,18:00
TravTi think lakshmiS was going to verify it as well18:00
ekarlsokk18:01
lakshmiSwill comment once done testing18:04
TravTsjmc7: ekarlso: i added this bug.  Take a look and let me know what you think about it18:10
TravThttps://bugs.launchpad.net/searchlight/+bug/150095818:10
openstackLaunchpad bug 1500958 in OpenStack Search (Searchlight) "Designate documents don't have updated_at default date" [Undecided,New]18:10
TravTbasically, that just makes it a little harder to search / sort18:10
sjmc7did i not add that in the patch a couple of days ago?18:11
TravThttps://review.openstack.org/#/c/226524/18:11
TravTsee my comments on it18:11
TravTand with latest master, same thing18:12
sjmc7ah.. i was bamboozled by them already being in the mapping18:14
TravTyeah...18:14
lakshmiSAn auth plugin is required to determine endpoint URL.  how do you create it18:20
sjmc7auth_plugin = password18:22
TravTsjmc7: phrase slop will make a nice demo... :)18:29
sjmc7yep!18:29
TravTeven if minor perf impact... just fun to play with.18:30
TravTi have now changed to have a facet called "Full Text Query"18:30
sjmc7i think we did that for the juno prototype18:30
TravTto get around magic search bug...18:30
TravTI put travis tripp in the image description18:30
TravTwith phrase slop, i can do "tripp travis"18:31
TravTand still match18:31
TravTpretty nice18:31
*** nikhil has joined #openstack-searchlight18:42
*** nikhil_k has quit IRC18:44
*** GB21 has quit IRC18:53
*** lakshmiS has quit IRC19:06
openstackgerritSteve McLellan proposed openstack/searchlight: Add 'id' to metadefs  https://review.openstack.org/22911019:12
*** lakshmiS has joined #openstack-searchlight19:25
openstackgerritNikhil Komawar proposed openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911919:34
nikhilTravT: ^19:34
sjmc7ha, oops! thanks nikhil19:40
sjmc7aww, you lowered the size for us? :)19:40
nikhilsjmc7: heh, that was not intentional19:42
sjmc7hack them! hack them!19:42
nikhilI think it won't matter for tests :P19:42
sjmc7:)19:42
*** lakshmiS_ has joined #openstack-searchlight19:43
*** lakshmiS has quit IRC19:46
*** lakshmiS has joined #openstack-searchlight19:47
openstackgerritNikhil Komawar proposed openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911919:47
sjmc7much better!19:48
sjmc7although looks like the x509 debug info isn't there19:48
*** lakshmiS_ has quit IRC19:48
openstackgerritNikhil Komawar proposed openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911919:48
nikhilsjmc7: haha yeah, realized and now it19:49
nikhilit's there!19:49
sjmc7third time's a charm19:49
nikhileh, a typo19:49
nikhilmay be not19:49
nikhil:P19:49
sjmc7:)19:49
openstackgerritNikhil Komawar proposed openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911919:52
nikhilthis stuff is more annoying than I thought19:52
openstackgerritNikhil Komawar proposed openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911919:53
nikhileh, just gunna leave it as is. It looks different on my screen for some reason :|19:55
nikhilsorry about the span19:55
nikhilspam*19:55
nikhiljust a bit antsy waiting for my presentation to start and they want me to stick to a IM that I don't use regularly19:56
sjmc7do some stretches19:56
* nikhil ack19:57
*** lakshmiS_ has joined #openstack-searchlight19:59
*** lakshmiS has quit IRC20:02
*** lakshmiS has joined #openstack-searchlight20:03
*** lakshmiS_ has quit IRC20:04
openstackgerritSteve McLellan proposed openstack/searchlight: Refactor clients handling to use Sessions.  https://review.openstack.org/21927120:07
*** lakshmiS has quit IRC20:23
openstackgerritTravis Tripp proposed openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911920:24
TravTnikhil_k: thanks for the cert fix!20:25
TravTi had a bug mentioning it, so I added that to the commit message20:25
sjmc7i've rebased the KS session patch20:25
TravTok, i'll try it out.20:26
*** lakshmiS has joined #openstack-searchlight20:32
TravTsjmc7 do you have a recommended way to force a manual trigger on this patch?20:35
TravThttps://review.openstack.org/#/c/228639/20:35
TravTi could fudge some bad mappings or something.20:36
openstackgerritMerged openstack/searchlight: Adds a new cert under Searchlight name  https://review.openstack.org/22911920:36
openstackgerritMerged openstack/searchlight: Refactor clients handling to use Sessions.  https://review.openstack.org/21927120:40
sjmc7TravT manual trigger?20:40
sjmc7oh.. yeah20:40
sjmc7delete the mapping20:41
*** lakshmiS has quit IRC20:50
TravTsjmc7: looks like recordsets have no facets options.21:06
TravTstatus / type21:06
sjmc7i can add them, but i didn't have time to add a test case for them21:06
sjmc7there wasn't one for zones or recordsets21:07
TravTyeah, one getting the facets via UI, it helps with symetery21:08
sjmc7i'll try and squeeze it in today21:08
TravTif not, let me or lakshmiS know and we'll try tomorrow.21:09
sjmc7TravT - i'll put up a patch for https://bugs.launchpad.net/searchlight/+bug/1500958 (designate updated_at) but i think it should be a bug against designate21:56
openstackLaunchpad bug 1500958 in OpenStack Search (Searchlight) "Designate documents don't have updated_at default date" [Undecided,New] - Assigned to Steve McLellan (sjmc7)21:56
*** sigmavirus24 is now known as sigmavirus24_awa22:19
openstackgerritSteve McLellan proposed openstack/searchlight: Default designate updated_at to created_at  https://review.openstack.org/22916922:25
TravTsjmc7: just in time for me to test22:25
sjmc7i can't keep up!22:26
TravThaha, that's how I feel22:26
TravTthat all_projects one made a big difference.22:27
sjmc7good or bad?22:28
TravTgood22:28
TravTbut now i need to add in an option the UI to allow user to specify it...22:29
sjmc7correcting the metadef id one now; i think it does need refactoring but it'll have to wait22:29
TravTi need to add in a little search options box like kelly had, i think.22:29
sjmc7yep22:29
sjmc7i'd be inclined to make the 'all projects' thing front and center22:29
sjmc7unless you have an 'admin' panel too22:29
openstackgerritSteve McLellan proposed openstack/searchlight: Add 'id' to metadefs  https://review.openstack.org/22911022:29
TravTit's kind of interesting...22:29
TravTbeen thinking about whether it should be admin panel and project panel22:30
TravTor just make it an option...22:30
TravTcurrent pattern definitely would mean there should be an admin panel22:30
TravTbut that is really easy to do22:30
TravTi just need to move a few files around22:30
sjmc7yep22:30
TravTfor the images angular panel22:31
TravTi put it under the app/core section22:31
TravTso it can just be linked in under a panel22:31
TravTin either place22:31
sjmc7right22:31
TravTthe updated_at patch looks pretty straightforward...22:32
TravTwill test it out now22:32
sjmc7yep!22:32
sjmc7but i think it should be a designate bug22:32
TravTyeah, its kinda the whole normalize / not normalize certain fields debate fur us.22:33
sjmc7bbiab, got to climb onto the roof22:33
sjmc7yeah22:33
TravTdon't fall off22:33
TravTsquirrels?22:33
sjmc7trees22:34
sjmc7raccoon22:34
TravTwell, if he asks for your wallet, i think you should just do it. don't try to be a hero or anything22:39
sjmc7the tree was no match for me22:59
TravThahaha22:59
sjmc7the wind's blowing about 30mph22:59
TravTi'm picturing you with a stick of dynamite22:59
sjmc7pair of scissors and some insulating foam23:00
sjmc7the racoon wasn't up there23:00
TravTall you need to do is this23:01
TravTdig a small hole23:01
TravTput a bar across the hole23:01
TravTdrop a crumpled piece of foil in it23:01
TravThe'll reach in but not be able to pull it out.23:01
TravTand won't let go.23:01
sjmc7:|23:01
sjmc7won't that make it mad?23:02
sjmc7plus it's got tonnes of garbage to eat23:02
TravTuntil you come near, then he'll maul your face23:02
sjmc7hahahaha23:02
openstackgerritSteve McLellan proposed openstack/searchlight: Default designate updated_at to created_at  https://review.openstack.org/22916923:25

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