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