18:01:22 #startmeeting cue 18:01:23 Meeting started Tue Jul 21 18:01:22 2015 UTC and is due to finish in 60 minutes. The chair is vipul. Information about MeetBot at http://wiki.debian.org/MeetBot. 18:01:24 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 18:01:27 The meeting name has been set to 'cue' 18:01:37 same time.. new day 18:01:41 roll call.. 18:01:51 0/ 18:02:06 hey o/ 18:02:26 sputnik13, abitha[tab][tab] 18:02:56 #agenda https://wiki.openstack.org/wiki/Meetings/Cue 18:03:38 #topic action items 18:03:42 #link http://eavesdrop.openstack.org/meetings/cue/2015/cue.2015-07-13-18.01.txt 18:03:46 o/ 18:04:14 ok i'll skip davide's actions 18:04:32 http://eavesdrop.openstack.org/meetings/cue/2015/cue.2015-07-13-18.01.html 18:04:36 this one has links :) 18:04:55 html pfft 18:04:58 sputnik13 to fill in details for https://blueprints.launchpad.net/cue/+spec/kafka 18:05:10 * sputnik13 kicks the can further down 18:05:19 #action sputnik13 to fill in details for https://blueprints.launchpad.net/cue/+spec/kafka 18:05:20 I couldn't get kafka to work at all 18:05:39 I'm trying to pass a message to kafka and consume it when it's clustered 18:05:45 and it just kept breaking 18:05:57 hmm.. that's kind of foundational stuff.. 18:05:58 sputnik13: is this just a standalone kafka? 18:05:59 I think there's more investigation to be done, and I need to get help from someone who's done this before 18:06:09 esmute_ clustered kafka 18:06:13 or are you trying to get it in cue? 18:06:28 ok we should understand how to stand it up properly.. and see if that impacts cue in any way.. 18:06:35 in terms of our api, etc. 18:06:44 esmute_: standalone 18:06:53 vipul: agreed, I'll continue banging away at it 18:07:13 ok cool 18:07:13 esmute__ to follow up with patch (https://review.openstack.org/#/c/194324/) and ensure it gets merged 18:07:22 why does esmute_ have an underscore 18:07:30 vipul: that merged long ago 18:07:51 the int-tests are now voting and they are run during gating 18:07:52 oh yes.. its voting now 18:07:59 they seem to be pretty consistent too.. 18:08:09 vipul I think you're looking at the wrong list 18:08:25 the one at the top are the topics from last week, we talked about those actions last week 18:08:33 oh damn 18:08:36 what's the link 18:08:36 the actions from last week are half way down 18:08:55 got it.. 18:08:55 it so happens that we've had the same two issues for davide and I for the past 2 weeks :) 18:09:15 ok so only other one is sputnik13 to follow up on confirming https://bugs.launchpad.net/cue/+bug/1429304 18:09:16 Launchpad bug 1429304 in Cue "Error 400 Bad Request when adding a limit to the cluster list URL" [Medium,Incomplete] - Assigned to Steve Leon (steve-leon) 18:09:18 Launchpad bug 1429304 in cue "Error 400 Bad Request when adding a limit to the cluster list URL" [Medium,Incomplete] 18:09:19 Launchpad bug 1429304 in cue "Error 400 Bad Request when adding a limit to the cluster list URL" [Medium,Incomplete] https://launchpad.net/bugs/1429304 18:09:35 still to be done 18:09:49 sputnik13: going back to kafka, doesnt monasca use kafka for their checks? perhaps you could go ask them 18:09:53 I'll work with esmute_ to get it confirmed 18:10:07 esmute_: I'm using monasca's kafka playbook 18:10:26 yeah I'm planning to talk with them 18:10:53 #action sputnik13 to follow up on confirming https://bugs.launchpad.net/cue/+bug/1429304 18:10:55 Launchpad bug 1429304 in Cue "Error 400 Bad Request when adding a limit to the cluster list URL" [Medium,Incomplete] - Assigned to Steve Leon (steve-leon) 18:10:56 Launchpad bug 1429304 in cue "Error 400 Bad Request when adding a limit to the cluster list URL" [Medium,Incomplete] 18:10:57 Launchpad bug 1429304 in cue "Error 400 Bad Request when adding a limit to the cluster list URL" [Medium,Incomplete] https://launchpad.net/bugs/1429304 18:11:47 Ok that's it for actions.. 18:11:51 we don't have any discussion topics 18:12:04 Anyone have anything they wnat to bring up before we go to bugs/ 18:12:38 * vipul needs to do a better job at adding stuff to agenda prior to meeting 18:13:03 #topic bugs 18:13:21 let's squash them bugs 18:13:51 get ready.. this link is going to be awesome 18:13:52 https://bugs.launchpad.net/cue/+bugs?field.searchtext=&orderby=-importance&field.status%3Alist=NEW&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.importance%3Alist=CRITICAL&field.importance%3Alist=HIGH&field.importance%3Alist=MEDIUM&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_commenter=&field.subscriber=&field.structural_subscriber=&field.tag=&field.tags_combinator=ANY&fiel 18:13:52 d.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on&search=Search 18:13:58 damn that didn't work 18:14:07 doesnt work 18:14:15 http://bit.ly/1MChDwJ 18:14:27 ^ much better 18:14:31 hah, beat me to it 18:14:40 same link as mine 18:14:55 consistent hashing :) 18:15:09 yeah but the has is so small, I wonder how they make it unique 18:15:17 whatever... bugz 18:15:34 are there bugs on that list that need to be scrubbed? 18:15:47 there are 2 High 18:15:53 in New state 18:15:58 so we should look at them and triage them at minimum 18:16:05 ok let's start with https://bugs.launchpad.net/cue/+bug/1452959 18:16:06 Launchpad bug 1452959 in Cue "cluster does not ERROR on missing broker metadata" [High,New] 18:16:06 use a tiny url 18:16:06 or bitly 18:16:07 Launchpad bug 1452959 in cue "cluster does not ERROR on missing broker metadata" [High,New] 18:16:09 Launchpad bug 1452959 in cue "cluster does not ERROR on missing broker metadata" [High,New] https://launchpad.net/bugs/1452959 18:16:19 esmute_: we did, keep up with the convo man :) 18:16:29 15 lines up :P 18:16:30 i cant access those link for some reason 18:16:44 my internet is shitty 18:17:24 so this one should be easy to fix.. 18:17:35 is it High? 18:17:58 It's unlikely that we will have missing broker metadata.. if you've followed the deployment docs though 18:18:03 so i wouldn't think it's a high pri 18:18:21 there's also no loss of data and no complete failure 18:18:32 it's a deployment error, not something the user can influence 18:18:53 right but, the workaround is just delete it 18:18:53 how can the metadata get missing? 18:19:13 oh wait I get it 18:19:27 esmute_: this is broker metadata, it's a cue control plane setup issue 18:19:35 esmute_: it can't.. unless you've set things up incorrectly 18:20:04 ok changing status to confirmed / priority to medium? 18:20:08 on the positive side cue doesn't die :) 18:20:19 so its a deployment issue... but this could be a small issue of something larger 18:20:21 confirmed and I'd say low 18:20:35 esmute_: how so 18:20:43 we should introduce a timeout so that the cluster get to ERROR if something is wrong 18:21:06 Medium = Failure of a significant feature, with workaround or Failure of a fringe feature, no workaround 18:21:12 there will be cases of random issues when deployment doesnt go as planned.. 18:21:17 is this a case where we don't actually begin to invoke the flow? 18:21:31 there may be some stuff on the API side that isn't catching errors correctly 18:21:44 wait, let's not talk about how to do it 18:21:46 but i agree esmute_ we need to be do a catch all to error 18:22:03 what I mean is "timeout" is a specific implementation 18:22:07 and in this instance it's the wrong one 18:22:24 what we want is for the indication to be ERROR instead of BUILDING 18:22:42 sputnik13: why didn't the flow revert handle thigs 18:22:47 s/thigs/this 18:22:48 sputnik13: can cue detect that the broker metadata is missing? 18:23:21 that's what the bug is also indicating as the failure, so the resolution is for the flow to fail and set ERROR rather than leaving things in BUILDING 18:23:59 vipul: I think this might be related to an issue we had in the past where failure in subflows was not triggering a rollback of the surrounding flow 18:24:18 although you fixed that in taskflow 18:24:18 we addressed that bug, and this could be resolved, it might not 18:24:25 sputnik13: is the task retrying or does it just fail due to metadata missing and it doesnt report the status to the flow? 18:24:32 ok.. the other thing i was thinking is maybe this is a api side issue.. 18:24:36 I think this bug predates that fix 18:24:40 where we haven't actually submited a job 18:24:54 that's another possibility 18:25:04 would be nice to have a stacktrace :P 18:25:20 yes *somebody* filed an incomplete bug report 18:25:27 so are we trying to come up with a fix here or are we debating whether this bug should be fixed and its priority? 18:25:33 *somebody*... not going to say who 18:25:44 updated status to incomplete 18:25:49 let's find out if this is really an issue 18:25:53 some guy whose name is 'min pae' 18:25:54 esmute_: we should be debating the bug and its priority 18:25:56 or if taskflow fixed things 18:26:21 vipul: it should also be reprioritized, either medium or low 18:26:31 it's set to low 18:26:55 if someone gets a deployment without broker metadata.. try to build a cluster and capture the errors 18:27:13 ok 18:27:16 it should be low because it is a deployment issue.... in happy path deployment, this should not happen right? 18:27:26 that's the theory 18:27:26 esmute_: correct 18:27:30 https://bugs.launchpad.net/cue/+bug/1469823 18:27:31 Launchpad bug 1469823 in Cue "Delete Cluster fails when Cluster contains VM's which have already been deleted" [High,New] 18:27:33 next up 18:27:41 Launchpad bug 1469823 in cue "Delete Cluster fails when Cluster contains VM's which have already been deleted" [High,New] 18:27:42 Launchpad bug 1469823 in cue "Delete Cluster fails when Cluster contains VM's which have already been deleted" [High,New] https://launchpad.net/bugs/1469823 18:28:42 ok this is a hard one to repro 18:28:49 so does this happen because the list-port task is not checking whether the vms exist? 18:28:56 vipul: davideagnello has a fix for it already 18:29:03 esmute_: yes 18:29:07 well in that case.. :D 18:29:17 is the priority correct though? 18:29:29 does it leak resources? 18:29:34 sputnik13: are ports associated with VMs? ie can a port exist on its own? 18:29:43 esmute_: yes and yes 18:29:59 vipul: not sure, I guess it's a possibility 18:30:23 sputnik13: so why does it need to check whether the VM exists? Shouldnt it check only if the port exists and delete them if they do. 18:30:30 i guess i can look at davide's patch 18:30:31 i would say if there is a potential of things being left around and the user can't clean them up.. then it probably is at the right priority 18:30:40 esmute_: there's a nova port list prior to the port delete 18:31:02 vipul: ok 18:31:02 you mena a neutron port list? 18:31:21 vipul: no I think it's a nova port list 18:31:26 to get the ports that belong to the vm 18:31:31 #link https://review.openstack.org/#/c/196332/ 18:31:36 davide uses hotmail :P 18:31:39 its this one right? 18:32:00 davideagnello is of soft micro 18:32:55 ok other ones are all medium 18:33:04 should we go through the mediums? 18:33:18 we do have some time let's go quick 18:33:33 ok https://bugs.launchpad.net/cue/+bug/1425130 18:33:34 Launchpad bug 1425130 in Cue "Refactor Cue TaskFlow tasks" [Medium,New] 18:33:35 Launchpad bug 1425130 in cue "Refactor Cue TaskFlow tasks" [Medium,New] 18:33:36 Launchpad bug 1425130 in cue "Refactor Cue TaskFlow tasks" [Medium,New] https://launchpad.net/bugs/1425130 18:33:53 that's a wishlist at best 18:34:10 it's related to the os_tasklib stuff as well, there's no bug 18:34:13 it's a refactor 18:34:13 i dont understand this bug 18:34:23 sputnik13: can you ELI5? 18:34:28 yea it's more of a nice to have 18:34:33 it's refactoring, we should make it wishlist 18:34:36 ELI5? 18:34:54 Explain Like I'm 5? 18:35:02 ELI5 18:35:06 you redditors 18:35:14 http://bfy.tw/vFY 18:35:18 upvote 18:35:25 power of the google 18:35:40 esmute_: it's a refactor 18:35:45 next bug 18:36:00 https://bugs.launchpad.net/cue/+bug/1426103 18:36:01 Launchpad bug 1426103 in Cue "Validate UUID API parameter's at API layer instead of DB interface" [Medium,New] 18:36:01 Launchpad bug 1426103 in cue "Validate UUID API parameter's at API layer instead of DB interface" [Medium,New] 18:36:03 Launchpad bug 1426103 in cue "Validate UUID API parameter's at API layer instead of DB interface" [Medium,New] https://launchpad.net/bugs/1426103 18:36:34 that looks like another refactor to me 18:36:49 are we returning the wrong error code? 18:37:03 not sure... so it's incomplete as well 18:37:09 ok i'll put a comment 18:37:14 Does this make it more difficult to debug an invalid UUID issue? 18:37:31 Or is the current output still obvious? 18:37:36 this seems like a validate needs to happen at the API level? 18:38:17 dkalleg: i think we should be doing input validation.. instead of relying on a lower level subsystem to catch the input error 18:38:20 esmute_: that may be true but if the functionality is correct then it's a refactor not a bug 18:38:33 in which case it would be more low or wishlist, not medium 18:38:59 +1 if we should be returning a 500 or something else.. and we're not.. then it's a higher priority and a real bug 18:39:05 vipul: agreed. I ask if it is currently made obvious when a invalid UUID is passed in to help prioritize 18:39:15 vipul: 500 sounds scary 18:39:19 it should be 404 18:39:29 500 internal server error 18:39:50 404 not found ? 18:40:09 yes that makes sense actually on a GET 18:40:18 vipul: yes.. 404 not found... the user is sending an invalid UUID 18:40:39 in that case this is a bug 18:40:50 If it doesn't currently say ": you passed in an invalid UUID dum dum", then it seems important to correct that more quickly. 18:41:19 sounds like you just get a 400 bad request 18:41:31 sputnik13: are you updating the ubg? 18:41:35 s/ubg/bug 18:41:48 I asked the question "is the response correct?" 18:41:59 "Is the response code correct? What is the anomalous behavior?" 18:42:07 we should be returning a 404 sputnik13 18:42:24 I think he's saying we're returning 400 18:42:42 yea that sounds like we should just confirm it and prioritize 18:42:53 400 vs 404 seems not that bad to me 18:43:03 it's not accurate.. 18:43:28 true, but other functionality is fine 18:43:43 REST is all about response codes.. and if we can't tell the user the right response code then our api sucks 18:43:56 cue is doing what it's supposed to 18:44:20 no argument there, but that's more a design change than a defect, it's doing things as designed 18:44:26 that's subjective.. you could say Cue should be telling me that ID doesn't exist 18:44:28 tomato tomato :) 18:45:21 i think it's at least a Medium priority confirmed bug.. 18:45:28 i think it is a simple enought fix that we can take care of it fast 18:45:29 I can see it either way, medium sounds fine 18:45:38 +1 18:45:39 like a low hanging fruit 18:45:41 is it confirmed? 18:46:12 If davide has seen it.. then probably 18:46:34 ok let's move on to the next one 18:46:36 https://bugs.launchpad.net/cue/+bug/1429649 18:46:37 Launchpad bug 1429649 in Cue "Consolidate cue.conf and worker.conf" [Medium,New] 18:46:37 Launchpad bug 1429649 in cue "Consolidate cue.conf and worker.conf" [Medium,New] 18:46:39 Launchpad bug 1429649 in cue "Consolidate cue.conf and worker.conf" [Medium,New] https://launchpad.net/bugs/1429649 18:47:48 Seems like another refactor 18:47:52 that's definitely wishlist 18:47:55 this is a simple copy and paste to cue.conf 18:48:02 and a rm of woker.conf right? 18:48:09 and verify and test, yada yada 18:48:20 yeah 18:48:24 Is Medium too high? 18:48:26 why not just have separate conf for both? 18:48:33 dkalleg: there's no failure 18:48:36 cuz they 99% the same 18:48:55 low hanging fruit.. 18:48:58 sputnik13: Right. Seems like a refactor. So how are we prioritizing refactors? 18:49:01 although probably in the refactor category 18:49:13 dkalleg: https://wiki.openstack.org/wiki/Bugs 18:49:22 https://wiki.openstack.org/wiki/Bugs#Importance 18:49:31 sputnik13: gracias 18:49:35 so this is for the 1%? 18:50:05 esmute_: so we're saying we shouldn't maintain 2 separate confs for the 1% case 18:50:07 Wishlist +1 18:50:12 it's a refactor, let's push it to wishlist and move on to other bugs, the finer point of how to get it done can be offline 18:50:43 ok last bug https://bugs.launchpad.net/cue/+bug/1436116 18:50:44 Launchpad bug 1436116 in Cue "Cluster create API should accept username/password for rabbit endpoints" [Medium,New] 18:50:46 Launchpad bug 1436116 in cue "Cluster create API should accept username/password for rabbit endpoints" [Medium,New] 18:50:47 Launchpad bug 1436116 in cue "Cluster create API should accept username/password for rabbit endpoints" [Medium,New] https://launchpad.net/bugs/1436116 18:51:14 that's also not a defect 18:51:14 This is a feature.. so should be a BP and not a bug no? 18:51:18 that's a new feature 18:51:23 BP sounds right 18:51:32 sputnik13: you're filing one right 18:51:46 that BP should include also work in the client and dashboard 18:51:54 esmute_: I thought I did already 18:52:10 you probably did sputnik13 18:52:11 hmm guess not 18:52:11 we need to review the api changes this would have 18:52:28 and see if those make sense for qpid, kafka, etc 18:52:40 oh, right 18:52:43 yes bp :) 18:52:59 regardless this is wishlist, there's no "bp" importance 18:53:24 put a comment saying it shoudl be a BP 18:53:30 and link it when we have one 18:53:34 this is coming up soon though 18:53:53 its at the top in the backlog 18:53:54 yep.. something we want to get to soon 18:54:17 we shouldn't confirm this a s a bug right? 18:54:22 it should be "won't fix" 18:54:25 it's not a bug 18:54:36 err sorry "Invalid" 18:54:37 #action sputnik13 to file a BP for user defined credentials for brokers 18:54:52 marked it Invalid and Wishlist 18:54:55 ok 18:55:00 done with bugs for today 18:55:08 #topic open discussion 18:55:08 yay we're done 5 minutes early 18:55:12 instead of 5 minutes late 18:55:26 mid cycle! 18:55:31 oh yes! 18:55:33 we need a mid cycle 18:55:36 it will be held in seattle 18:55:39 at the HP offices 18:55:46 booooo 18:55:48 details to come 18:55:49 ;D 18:56:03 we need to get some non-HP contributors 18:56:17 indeed 18:56:45 somebody should go start a Cue company 18:56:56 so we can have a Cue day? 18:56:59 a Tesora for Cue 18:57:11 they're essentially a Trove company right? 18:57:15 pretty much 18:57:22 we need the equivalent for Cue, so we can do mid cycles :-D 18:57:47 how many talks have we submitted for Tokyo? 18:57:48 brb.. getting some funding 18:58:05 i believe around 5 18:58:10 I can invest $100 in your company, where does that put us in funding? 18:58:11 :) 18:58:23 and you want 10% i bet 18:58:33 no, I'm not that greedy, just 9% 18:58:39 ok with that.. 18:58:41 #endmeeting