=== kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [17:26] hey guys, jcastro wants to know why https://jujucharms.com/q/cloudfoundry is bring up snake eyes for lp:cf-charmers/cloudfoundry/trunk [17:26] * whit has to bounce [17:26] he's going to want to know if we pass proof, etc. [17:26] so I'm digging in the testing stuff [17:28] I'm not sure what snake eyes is :) [17:28] we can't find it in search [17:28] ~cf-charmers/cloudfoundry is the charm we're looking for [17:28] ahh right, it's not there [17:28] I'm guessing the push was >1h ago [17:29] and it is here: https://jujucharms.com/u/cf-charmers/cloudfoundry/trusty I am not sure why it is not in search results. [17:29] no it's been there a long while [17:29] was just going to say [17:29] jcastro: I think this is a bug with the search [17:30] well yeah, of course it is. :) [17:30] haha - I mean, because the charm is in the system [17:30] should I file one now [17:30] yes please [17:30] thanks! [17:30] because it's not promulgated [17:31] I filed that in the bug around it [17:31] eco needs to mark it as promulgated and it'll show up [17:31] but it's not showing up as search at all [17:31] yes, I konw [17:31] * rick_h_ looks for the bug with the full written out explanation [17:31] but you're showing other personal namespace results? [17:31] jcastro: yes, but nothing with charmers at the end of the usernmae shows [17:32] they're filtered due to a bug we're working on [17:32] as I said, I wrote it all out in a but on the CF charm to whit [17:32] Oh ok, so basically you meant to match "~charmers" but catch-alled "~cf-charmers" by mistake [17:32] no, we do it on purpoise because landscape-charmers openstack-charmers and so on are normally promulgated [17:33] so do reduce duplicate results we only show the promulgated charm and we filter out the one that ends in -charmers [17:33] ok, so basically, you want me to get a charmer to promulgate it [17:33] so if a -charmers owned charm isn't promulgated it'll be hidden [17:33] jcastro: yes please [17:33] so move it to the store? Or is there another definition? [17:33] no, it's in the store, just hidden [17:34] its there, right there: https://jujucharms.com/u/cf-charmers/cloudfoundry/trusty [17:34] jcastro: see https://bugs.launchpad.net/charms/+bug/1425069 [17:34] Bug #1425069: cloud foundry icon has svg error [17:34] jrwren: right but search filters [17:34] rick_h_: right. thanks for reminding me. [17:34] due to the dupe results bug we had [17:35] rick_h_: right. [17:35] while we fix promulgation in the charmstore which is in progress [17:35] do you have a timeline for a fix? Because promulgating it opens a new can of worms [17:35] soon we will have no dupes and search results unfiltered. [17:35] jcastro: I'm hoping it's in the next release scheduled for hte 16th [17:35] because then we'll need to make sure it has tests, is maintained, and all the other policy bits [17:35] jcastro: yep [17:36] ok, so like, I think we'll just wait then [17:36] jcastro: ok, that's fine [17:36] but atm we're working between the lesser of two bugs and this is a known corner case that gets bit [17:36] while we try to fix both bugs [17:39] while doing this refactoring I really wish we were using typescript, dart, or the like so that I could catch odd-man-out bugs without manual testing [17:42] hatch: how is one of those going to help? [17:43] jrwren: because you define the signature of methods with the systems so on compile if the data being passed to it doesn't match then it'll fail [17:43] similar to any typed language [17:43] hatch: cool. [17:44] yeah it really is - of course it adds yet another layer of abstraction but sometimes it's worth it [17:44] also saves from having to write that part of the integration tests :) [17:45] doesn't replace integration tests of course though [17:51] jrwren: just in case you're interested, it's like this https://github.com/Microsoft/TypeScriptSamples/blob/master/todomvc/js/todos.ts#L71 === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [18:04] uiteam lf a trivial review and qa for https://github.com/juju/juju-gui/pull/699 thx [18:04] uiteam I also need one more for https://github.com/juju/juju-gui/pull/698 === kadams54 is now known as kadams54-away [18:23] * rick_h_ goes to get food before next round of calls === alexpilotti_ is now known as alexpilotti === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [22:00] Morning === kadams54 is now known as kadams54-away