14:00:22 <SergeyLukjanov> #startmeeting sahara
14:00:26 <openstack> Meeting started Thu Nov 19 14:00:22 2015 UTC and is due to finish in 60 minutes.  The chair is SergeyLukjanov. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:27 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:28 <esikachev> hi
14:00:29 <SergeyLukjanov> #link https://wiki.openstack.org/wiki/Meetings/SaharaAgenda
14:00:30 <openstack> The meeting name has been set to 'sahara'
14:00:37 <SergeyLukjanov> let's wait a few mins for more attendees
14:01:21 <vgridnev_> hi
14:02:22 <NikitaKonovalov> o/
14:02:49 <sreshetnyak> o/
14:03:02 <SergeyLukjanov> #topic sahara@horizon status (crobertsrh, vgridnev)
14:03:07 <SergeyLukjanov> folks, please
14:03:36 <crobertsrh> Hoping to have our code back in sahara-dashboard sometime in the next week or so.
14:03:44 <vgridnev> I have nothing in this topic
14:03:50 <vgridnev> crobertsrh, great!
14:04:01 <crobertsrh> That will mean rebasing quite a few existing changes, of course.
14:04:09 <vgridnev> any news from david lyle?
14:04:18 <crobertsrh> That is the news from him :)
14:04:29 <SergeyLukjanov> cool
14:04:52 <SergeyLukjanov> are there any current issues preventing it?
14:05:27 <vgridnev> i've also noticed that we have new group in gerrit, sahara-dashboard core
14:06:04 <SergeyLukjanov> yup
14:06:16 <SergeyLukjanov> it currently includes sahara-core
14:06:31 <SergeyLukjanov> and I think that we should add crobertsrh and NikitaKonovalov to it
14:06:42 <crobertsrh> He mentioned that there are some integration test issues, but thinks that he might work those out separately.
14:06:47 <tmckay> +1
14:06:56 <SergeyLukjanov> do we need to vote for it? any objections?
14:07:14 <tmckay> do we even have to vote on the mailing list? Can we just do it?
14:07:19 <tmckay> heh :)
14:07:25 <tmckay> no objections
14:07:26 <SergeyLukjanov> I think we can just do it ;)
14:07:42 <tmckay> nobody who is not crazy would ever object to a dashboard core
14:08:10 <SergeyLukjanov> done :)
14:08:28 <SergeyLukjanov> anything else to chat about re dashboard?
14:08:50 <vgridnev> nothing from me
14:09:01 <crobertsrh> Nope
14:09:05 <huichun> Move to this channel?
14:09:11 <tmckay> crobertsrh, congrats, those patches are going to merge before they even cool off ;-)
14:09:28 <crobertsrh> heh
14:09:53 <SergeyLukjanov> #topic News / updates
14:10:14 <vgridnev> direct engine finally removed!
14:13:44 <huichun> tmckay:  Scheduled edp job has been long time reviewed,  and modified according to all review comments, wish to be reviewed quickly T_T
14:13:51 <tmckay> not much from me, I put up a few specs that I hope I will get to soon
14:14:15 <tmckay> huichun, ok.
14:14:45 <tmckay> is elmiko here?
14:15:49 * tmckay thought maybe he had something to add about secret store or V2 API
14:15:58 <SergeyLukjanov> vgridnev, please take a look on scheduled edp job patches
14:16:13 <vgridnev> huichun, are you going to add integration tests for scheduled jobs?
14:16:31 <huichun> Yes but another patch
14:16:45 <huichun> Because they are separated
14:17:02 <vgridnev> separated from what?
14:17:20 <tmckay> huichun, I will look today, too. I have not done many reviews since Tokyo.
14:17:46 <huichun> I will write another patch to submit
14:18:24 * tmckay goes to lots of meetings it seems :)
14:18:28 <elmiko> tmckay: hi
14:18:36 <elmiko> doh... dst...
14:18:47 <tmckay> elmiko, hey, any updates on secret store or v2 API or anytihng?
14:19:03 <elmiko> just posted a new patch for secret store last night, the config wrapping is in place
14:19:07 <huichun> vgridnev: I think one patch is for feature , one patch for interagration test will be clearly
14:19:12 <elmiko> api v2, still working on gathering changes for the spec
14:19:57 <SergeyLukjanov> elmiko, do we need separated topic on today's meeting re v2?
14:20:14 <elmiko> SergeyLukjanov: not this week, thanks
14:20:37 <elmiko> i'd like to keep it as a separate item though, so we can keep it current
14:21:22 <huichun> tmckay: thx Trevor
14:22:07 <SergeyLukjanov> #topic Open discussion
14:22:09 <vgridnev> huichun, it's clear but it all-in-one case we will be sure that everything works well
14:25:59 <vgridnev> huichun, ok, waiting for a change with scheduled tests
14:26:06 <tmckay> oh, related to protected/public
14:26:29 <tmckay> so it's possible now, but it came up when we were talking about allowing public to be changed on protected items
14:26:54 <huichun> vgridnev: for integration test, I will make resume suspend scheduled as one
14:27:00 <tmckay> we probably need to investigate what happens when a public ng or cluster template is made non-public when it is referenced in a cluster by another tenant
14:27:21 <tmckay> what might break?  Do we need to disallow public -> non-public for referenced templates?
14:27:26 <tmckay> I have no idea, haven't tried
14:27:31 <tmckay> not sure what will happen
14:27:39 <tmckay> db lookups will fail, that's for certain
14:27:57 <tmckay> but there might be enough cached in the cluster object that it doesn't matter
14:28:23 <tmckay> anybody ever try this?
14:28:26 <vgridnev> huichun, let's do scheduled first
14:29:21 <crobertsrh> tmckay:  I have not tried such a thing.
14:30:41 <tmckay> crobertsrh, I'll make a note to try it
14:31:52 <elmiko> is there any news about the ci lab with respect to the "Quota exceeded for port" errors we are seeing?
14:32:16 <vgridnev> esikachev, ^^
14:32:37 <esikachev> yes, this problem is resolved
14:33:08 <elmiko> \o/
14:33:10 <elmiko> thanks
14:33:26 <esikachev> there was a problem with reinstalling devstack on ci
14:33:50 <elmiko> i knew devstack was in there somehow ;)
14:36:09 <huichun> vgridnev:  maybe I add integration test as one patch followed this scheduled patch? Cause everytime I should resubmit and it will clear over the passed review
14:36:11 <AndreyPavlov> tmckay, i haven't tried this, but it's pretty much the same case as if referenced public template would be deleted
14:36:28 <AndreyPavlov> the problem is how to control such situations
14:37:25 <tmckay> AndreyPavlov, ack.  In the general case, we don't allow deletion of referenced templates, so maybe we just need those checks on public -> non-public
14:37:51 <tmckay> same logic would work, I think
14:38:04 <tmckay> I'll poke at it
14:39:16 <AndreyPavlov> actually this template can be referenced by private resource created in another tenant
14:39:32 <vgridnev> huichun, you can make the change based on scheduled patch, I think we will not have any issues with that
14:39:47 <AndreyPavlov> in this case it will be invisible for sahara when it comes to deletion
14:39:55 <AndreyPavlov> or update op public field
14:40:25 <AndreyPavlov> do we want to let sahara search across all tenants?
14:40:32 <vgridnev> huichun, you can ping me in case of problems by mail or in irc, I will help you
14:41:02 <tmckay> AndreyPavlov, why not?  It has admin credentials under the hood anyway, and superuser db access
14:41:12 <tmckay> filtering search by tenant is only a filter
14:41:35 <tmckay> it wouldn't expose any information, except that "someone is using the thing you shared, sorry!"
14:42:00 <tmckay> the only other thing I can think of is to auto-copy templates (we have talked about this kind of thing before)
14:42:24 <tmckay> since tenant/name is unique, it could just be automatically injected into the tenant that is consuming it
14:42:40 <tmckay> but, ids and tables and things would be messed up
14:42:48 <huichun> vgridnev: so I write another patch(integration test)depends on this scheduled patch, does this make sense? So you can keep review on this scheduled patch, because it really take long time(many times has two +2,nearly merged,but comes -1,then restart)
14:42:50 <tmckay> (which is why I think we have not tried such a thing in the past)
14:43:09 <tmckay> AndreyPavlov, I'll see what happens and write some kind of spec, we can talk about more there :)
14:43:11 <huichun> vgridnev:  thx for your help Vitaly
14:45:20 <vgridnev> Yes, that make sense, I think. If we would tests we can merge that as we will have two +2, I think)
14:45:31 <vgridnev> huichun, ^^
14:46:09 <huichun> vgridnev:  thx  Vitaly ^^
14:46:09 <AndreyPavlov> tmckay, i guess we need to think about concerns of across-tenant searching
14:46:36 <AndreyPavlov> cause if we will not find them, it would be much easier and clear to implement
14:47:03 <tmckay> AndreyPavlov, yes, good thought.  spec spec spec :)  We will beat it into submission :)
14:47:25 <tmckay> I'll plan to try it this week, and put something up for discussion
14:47:42 <AndreyPavlov> tmckay, ok :)
14:50:14 <vgridnev> oh, folks. We moved all bugs that lost activities for 180 days to Incomplete state, so, please if the bug that still should be fixed move that from incomplete state
14:51:57 <crobertsrh> Ok :)
14:55:12 <SergeyLukjanov> 5 mins left
14:55:13 <vgridnev> any topic to discuss?
14:55:29 <elmiko> nothing from me
14:57:19 <SergeyLukjanov> #endmeeting