/srv/irclogs.ubuntu.com/2013/11/06/#juju-gui.txt

bacrick_h_: you around?02:06
=== gary_poster|away is now known as gary_poster
gary_posterhey huwshimi, I'm running away, but I'd like your review of https://codereview.appspot.com/22200045 sometime today when you get a chance.  Would like your opinion of the effect in particular, but code thoughts are more than welcome as well.02:28
huwshimigary_poster: No problems, I'll take a look02:28
gary_posterthanks!02:28
gary_posterbye :-)02:28
=== gary_poster is now known as gary_poster|away
huwshimigary_poster|away: Bye02:29
hatchhey Makyo great video :D02:33
Makyohatch, thanks; working on finding a transcoding that doesn't artifact so much before I submit it proper.02:34
hatchit sounds so 'official' :D02:34
MakyoThat maaaaay have been the goal? :P02:35
hatchhaha - well I'm glad to be back at work tomorrow - I'll try and put together my notes for everyone else02:36
MakyoRock on~02:41
rick_h_huwshimi: sorry, been out03:10
rick_h_huwshimi: I'd say that it can just have a very generic initial template.03:10
huwshimirick_h_: It's all good, I've made progress :)03:10
rick_h_huwshimi: just a div container to stuff content into and other can deal with it/run with it03:10
rick_h_huwshimi: ah, cool stuff then. Going ok then?03:10
huwshimirick_h_: Yeah, just writing tests at the moment.03:11
huwshimirick_h_: It'll need a good review as there is stuff that is probably wrong, but I wasn't sure how else to do it :)03:11
rick_h_huwshimi: sure thing03:11
huwshimirick_h_:  One question however, to add a new test file do I just add the name to test/index.html? Do I need to do anything else to get the test runner to see them?03:13
rick_h_huwshimi: yea, just add to the test/index.html03:13
huwshimihmmm...03:13
rick_h_huwshimi: make sure there are no syntax errors or otherwise in the console or it'll look like the file just doesn't load03:14
huwshimiAh right, that could be it03:14
huwshimioh, lint is not happy :)03:15
rick_h_lol03:15
huwshimiYep, that was it03:19
rick_h_party03:19
=== rogpeppe1 is now known as rogpeppe
BradCrittendenhi frankban10:04
=== BradCrittenden is now known as ba
=== ba is now known as bac
frankbanhi bac 10:07
bachi frankban. i was going to ask you for a charmworld review, but it was simple and urgent so i self-reviewed. bad me.10:07
frankbanbac: np10:10
rick_h_bac: thanks for the link fix. I assumed urls would be generated with request.route_url and not need updating. Should have looked :/11:22
rick_h_morning frankban, welcome back to the party :) if any of the cards don't make sense let me know. I wrote them up as we were hitting things yesterday so might not be all that clear headed. 11:24
frankbanrick_h_: cool, I have to go for a while now, I'll ask when back, thank you11:43
rick_h_frankban: have fun11:43
rick_h_benji: started making cards out of the todo list and marked our 'debug' card as done. I've got to run the boy to day care this morning so will be in/out for a sec this morning. 11:50
benjirick_h_: k11:50
bacrick_h_: i'm just glad that the fix was easy.  after applying my bundle proofing stuff i was paniced when i saw everything produce 404s12:05
bacrick_h_: it occurs to me we should give jcastro and others a chance to fix their bundles before we roll the proofing onto production.  on staging only two bundles make it through proofing, none of jorge's12:06
rick_h_bac: well, we need it in production so when he runs proof locally it finds all the issues to fix12:35
rick_h_bac: imo, the only way to fix it all is to run charm proof on them and get the list of issues12:35
bacrick_h_: i agree.  my thought was he'd want a chance before they all disappear on production12:41
rick_h_bac: meh, I think with release pushed we'll just bribe him to push changes today/tomorrow :)12:41
rick_h_doh, he's not around atm for me to bug him about bribery12:41
bacrick_h_: i just don't want to get grief like "i had this big demo and there was nothing there".12:42
baclike the recent demo mark did without warning us12:42
rick_h_bac: ah, yea that I can get behind. 12:42
rick_h_bac: good point12:42
rick_h_luca___: I now feel like I have permission to reply to all questions with "Use the force luca"12:51
rick_h_luca___: and why did that email make me very very afraid? lol12:52
luca___rick_h_: lol12:52
luca___rick_h_: you'll find out haha12:52
* rick_h_ goes to get his antacids12:52
luca___rick_h_: a 4 hour meeting 1-2-1 meeting with Mark only leads to one thing12:52
rick_h_CHANGE!12:53
luca___rick_h_: bingo12:53
* rick_h_ is one smart cookie12:53
rick_h_ok, so as long as I'm right to be afraid12:53
rick_h_benji: lol, looks like I should read the comments more. See #3 in http://bazaar.launchpad.net/~hazmat/python-jujuclient/trunk/view/head:/jujuclient.py#L4112:57
benjihmm12:59
=== gary_poster|away is now known as gary_poster
frankbanrick_h_: ping13:11
rick_h_frankban: pong13:11
gary_posterfrankban, welcome back :-)13:11
frankbanrick_h_: I see you are working on the python jujuclient. I was thinking that, while we are there, we could also add an easy fix to deploy() to support ToMachineSpec (required by quickstart), but that's not a blocker now13:12
frankbanrick_h_: (and currently quickstart just overrides deploy)13:13
rick_h_frankban: ok, if it's not for the bundle release I say add a bug/card and happy to go back to it but right now it's release blockers or bust13:13
frankbanrick_h_: cool, ok13:14
rick_h_there's all these lovely disclaimers that jujuclient is "pre-alpha software...use at your own risk" but it's kind of in our productin story now so I assume we'll have to be spending some time getting it out of pre-alpha stage :/13:14
bachi gary_poster.  staging looks good and i think it is ready to roll out to production.  but none of jorge's bundles pass the proof test.  so, should we give him a chance to fix them before rolling out to production and having them disappear?  (only two bundles are on staging, hatch's and my fake one.)13:16
frankbanrick_h_: yeah. do you have a minute to talk about those charm cards?13:16
rick_h_frankban: sure thing13:16
bacfrankban: how's the new laptop?13:16
gary_posterbac, no.  They are not a loss until they work. :-)13:16
gary_posterbac, IOW, fire away13:17
rick_h_frankban: https://plus.google.com/hangouts/_/7ecpihg9purkm4gn74bkdgttpg?hl=en13:17
bacgary_poster: ok, i've already composed a message.  i'll just change the wording to be more "you're currently screwed, here's how to fix it."13:17
frankbanrick_h_: Charm tests do not install requirements.pip to be able to run make test. how to reproduce? it seems to me that test calls unitttest and ftest, and both depend on setup13:17
gary_posterbac, lol ok13:17
gary_posterbac, do you disagree with me?13:17
frankbanbac: the new laptop is enjoying its long trip from shangai, in venice now, hpefully it will arrive tomorrow13:18
bacgary_poster: given the way high-level presentations are being done with no warning, i worry that someone will discover no bundles are available at an inopportune moment.13:18
gary_poster:-)13:18
bacgary_poster: i guess maarten is mostly at risk13:18
bacsabdfl has already done his.13:19
gary_posterbac, but they are already using comingsoon, which is tied to staging, which already has the characteristic you are worried about13:19
bacohhhh13:19
gary_poster(and they were using the file dnd functionality)13:19
bacok, non-issue then13:19
bacoff to RT13:19
gary_postercool13:19
gary_poster:-) thanks13:19
bacwill release r44613:20
gary_posteryay!13:21
bacrt filed, jjo ack'ed13:26
gary_postercool.  thank you very much for pushing this through (at all hours of the night), bac13:27
bacwas no problem.13:28
bacgary_poster: there is another 'make jujucharms release' card.  if it is need could you add a description?13:38
gary_posterbac, yeah.  the intent is "make [non-manage] jujucharms release"13:39
gary_postersuggestion on spelling that? :-)13:39
gary_posterrick_h_, do you have some time to review https://codereview.appspot.com/22200045/ ?  You would be obvious choice, but can ask others if you are putting out fires or otherwise busy13:40
rick_h_gary_poster: sure thing. Looking now. I saw someone looking at it so moved on. 13:41
gary_postercool thanks13:41
gary_posterrick_h_, would you like a liberal dosing of comments?  If so, can do.  Was just wondering about that myself.13:54
gary_posterrick_h_, and thank you for review13:54
rick_h_gary_poster: yea maybe. I mean it's pick-apartable just read dense. 13:54
rick_h_gary_poster: I thought about trying to move the function out of linkify, but you're using the array in that scope so I guess you can't13:55
gary_posterrick_h_, right.  ok cool.  I'll give a whirl with some comments and  then land it.13:55
rick_h_gary_poster: maybe just whitespace around the inner function would help?13:55
gary_posterrick_h_, lol ok13:55
rick_h_gary_poster: cool, thanks. 13:55
hatchmorning all14:07
rick_h_morning hatch 14:12
bacwow, with 99.92% reporting the VA attorney general race is separated by 53 votes out of 2.2M cast.14:12
rick_h_ouch14:13
hatchwait for the recount....lol14:14
hatchrick_h_: did you get the lastpass v3 update? It not longer looks like someone threw up it's UI14:25
rick_h_hatch: yea, I went to ping you on it but you were afk14:25
rick_h_hatch: figured you'd be happy14:25
hatchyup, it's a little slow now but oh well14:28
rick_h_hatch: meh, new release. I'm sure i'll pick up as they get it going14:28
hatchI got the popup during my talk yesterday lol14:29
hatchthat was pretty comical14:29
rick_h_nice14:29
rick_h_incognito mode ftw14:29
rick_h_I've tried to start using that for talks since the great 'google voice text has arrived' debacle :)14:29
hatchhaha14:29
gary_posterrick_h_, beddah? thoughts? http://pastebin.ubuntu.com/6370735/ http://pastebin.ubuntu.com/6370754/14:31
rick_h_gary_poster: lol, nice14:32
gary_poster:-)14:32
rick_h_gary_poster: hatch appreciate it when he gets sent in to fix something or add another bit of parsing :)14:32
gary_poster:-) cool14:33
* hatch goes to look14:33
hatchhaha14:33
hatchthems a lot of comments14:33
gary_posterheh14:33
hatchugh so many emails14:50
gary_posterjujugui if you hard reload you will see I changed comingsoon to point back to manage14:59
rick_h_gary_poster: thanks!14:59
gary_posterwelcome14:59
hatchI also learnt yesterday that you removed the charmworld flag during the demo....lol14:59
rick_h_lol14:59
rick_h_hatch: what? You didn't get a local running copy to demo with?!14:59
rick_h_hatch: boy you leaned a lot of lessons in one talk15:00
hatch"and as you can see here when I do a search for 'hatch' I don't get....oh I do....well looks like they removed that flag"15:00
hatchhaha15:00
gary_posterheh15:00
gary_postersorry15:00
hatchrick_h_: ahh it was totally fine15:00
gary_posterhey rick_h_ I have to head out, but "home" link is missing from charm browser in comingsoon.  investigate15:00
hatchI deployed to ec2 the trunk15:00
gary_posterplease?15:00
hatchto demo how it worked15:01
rick_h_gary_poster: ugh, looking. 15:01
gary_posterrick_h_, wait I may be stupid15:01
rick_h_gary_poster: hmm, I see it /me hard reloads15:01
jcastrohey fellas15:01
bachi jcastro15:01
jcastroso the errors on "blah blah is not a type of string" and stuff15:01
gary_posterrick_h_, nevermind me.  sorry for the heart attack.  as you were. ;-)15:02
jcastroare the only errors in my bundles15:02
rick_h_gary_poster: ugh, but I'm hitting an issue where I can't get from browse to env and back :/15:02
rick_h_ugh, and I'm out of it as well. Environment isn't a button oops15:02
gary_posterrick_h_, that's a UX thing, yeah15:02
rick_h_jcastro: also remove constraints for now please. 15:02
gary_posterrick_h_, however, if you want to see a bug, try this. ;-)15:02
jcastroI only have constraints in one15:03
rick_h_ruh roh...15:03
jcastrobut the other ones, those aren't my fault right?15:03
rick_h_jcastro: yea, we chased down a lot of bug fun with it. Appreciate it pointing out the issue15:03
rick_h_jcastro: 'other ones'?15:03
jcastro"blah blah is not a type of string" and stuff15:03
rick_h_jcastro: yea, the config values must be of the right type. That should be good to go now15:04
jcastrook, so I need to update something then15:04
rick_h_jcastro: so if it says it's an integer it has to be a interger not "9" or anything15:04
jcastroalso, if that's good to go then how come brad's tests came back with all of them flunking?15:04
rick_h_jcastro: there was a bug in the gui exporting them all as strings, but should be fixed15:04
gary_posterrick_h_, (1) in quicksearch, choose "applications" (2) in quicksearch again, type a string to search for and then press return to make it happen.  In query string you'll see you are now constrained by category.  not cool because you can't escape category anymore15:04
jcastrorick_h_, ugh, so basically15:05
rick_h_gary_poster: yea, we hit that at the sprints. 15:05
jcastroI have to redo all my bundles15:05
rick_h_jcastro: no, small tweaks. Remove quotes if the config is an int15:05
gary_posterjujugui, need to head to do fam stuff for a bit (middle school thing I mentioned).  back in 1.5 hours hopefully15:05
rick_h_jcastro: should be small/easy. Which one are you looking at?15:05
bacjcastro: can you proof lp:~jorge/charms/bundles/mediawiki-scalable/bundle ?15:05
jcastrowordpress15:05
jcastrobac, ah, that one is crashing something15:06
bacyeah, i'm looking at it15:06
bacmay just be indentation15:06
jcastrorick_h_, constraints removed from discourse15:07
rick_h_jcastro: and wordpress please15:07
jcastrohuh weird, that one had blank ones15:08
jcastrodone15:08
rick_h_jcastro: yea, and they were of the wrong names to work 15:08
jcastrooh they all do15:09
jcastrook so how do I fix these bundles15:09
jcastroif it's a string it looks like this:15:09
jcastro"binlog-format": MIXED15:09
rick_h_jcastro: which fields are failing?15:10
jcastro"block-size": 5 <-- and this one looks like an int15:10
rick_h_jcastro: it shold list all the ones it doesn't like15:10
jcastroa bunch15:10
rick_h_dude, helpful, pastebin or something15:10
jcastroworking it!15:10
rick_h_:)15:10
jcastrohttp://paste.ubuntu.com/6370941/15:10
jcastrohttp://paste.ubuntu.com/6370942/ for wordpress15:11
rick_h_jcastro: kthx, will look into it. Looks like something is still borked15:11
jcastroyeah15:11
jcastrothat's why I'm like wait a minute, it's not my fault I'm not passing tests! :p15:12
jcastrobac, oppressor!15:12
bacha15:12
bacrick_h_: is this an issue that should stop our release?15:13
rick_h_bac: yes, the type checking in proof is bad still if that's the latest code on mjc15:14
rick_h_bac: did the release with the type check fixes go out?15:14
bacrick_h_: manage has not been updated.  i don't know what rev you're talking about, on staging i assume.15:14
rick_h_bac: ah so maybe it's not an issue then15:14
bacmanage is very old15:15
rick_h_bac: ah ok then. That would explain it. Ok, I thought I had fixed it. Will try to do a local test then15:15
bacrick_h_: actually i say old but i mean it has not been updated as part of this effort.  not sure what version of proof is on it.15:15
rick_h_bac: looking for the branch where I fixed the type checks15:16
* bac wishes heartbeat showed rev15:16
bacrick_h_: i don't know how to tell what is running on manage15:16
rick_h_bac: yea, well if heartbeat isn't showing rev then it's old enough I think. abel had that added I thought15:16
jcastrohmm, should I update my charm tools now?15:16
rick_h_jcastro: yea, need to do that. bac's email had notes on the rev, but it sounds like the server isn't updated with the latest atm either15:18
* jcastro nods15:18
bacrick_h_: no, abel's follow-on branch for revno in heartbeat has not landed even in trunk yet15:18
jcastrorick_h_, I was already in the car ready to drive over and kill you if you were going to make me remake every bundle15:18
rick_h_jcastro: heh, trying not to man. Trying15:19
rick_h_bac: I could have sworn I saw that go by with a LGTM, man I'm not keeping up today15:19
* rick_h_ isn't seeing the changes I expect to see15:20
rick_h_bac: ok, I'm looking for changes in r43715:22
rick_h_bac: I thought there was a deploy in progress yesterday after the rollback so had expected that to be on mjc?15:22
bacno15:22
bacmjc has not been touched.  an RT is waiting15:22
rick_h_ok, so jcastro then ignore proof until the server gets updated sorry. 15:22
rick_h_bac: rgr, thanks. Sorry for the confusion15:23
bacrick_h_: i had a disconnect and thought charm-proof was hitting staging15:23
bacactually i just didn't think too hard about it15:23
rick_h_bac: no, I thought proof hit mjc once he released charm-tools 1.015:23
rick_h_err 1.1 that is15:23
baci'm sure you're right15:23
baci just tried changing my /etc/hosts to point mjc to sjc but get SSL errors15:24
rick_h_yea, ugh15:24
rick_h_best thing is to hit proof with curl or httpie15:24
rick_h_err, the staging proof endpoint15:24
rick_h_bac: http://paste.mitechie.com/show/1058/15:25
baccool15:25
rick_h_that's it, going back to bed. I give up. "bzr shelve --all ... bzr: ERROR: bzrlib.errors.MalformedTransform: Tree transform is malformed [('missing parent', 'new-6')]"15:26
frankbanguihelp: I need two reviews and one QA for https://codereview.appspot.com/22300044 . Is anyone available? Thanks!15:49
Makyofrankban, on it.15:49
frankban(quickstart + python)15:49
frankbanthanks Makyo 15:49
Makyojujugui call in 1015:50
bacjujugui: manage.jujucharms.com updated.  heartbeat shows failure on bundles but ingest just started at :45.  http://manage.jujucharms.com/heartbeat15:50
rick_h_bac: cool, thanks!15:50
rick_h_jcastro: re-run your proof please. When I charm proof your wordpress bundle I get no output15:53
bac\o/15:54
bacrick_h_: this one still has issues ~jorge/charms/bundles/mediawiki-scalable/bundle/15:55
jcastrorick_h_, CONFIRMED! \o/15:55
bacjcastro: ^^15:56
jcastrook15:56
rick_h_bac: k, looking15:56
rick_h_bac: oh, it's got like blow up and die kind of issues :/15:56
bacrick_h_: did you say having 'constraints' with defaults causes problems?15:56
bacyeahs15:56
rick_h_bac: not with proof, but in the real world yes15:56
bacoh, so worst case scenario15:57
bacyay for false positives15:57
rick_h_bac: well we don't proof constraints right now. and if they're not 100% correct for Go the deployer will die in a fire15:57
rick_h_bac: this was the big result of yesterday's barn burning joy15:58
Makyojujugui call in 215:58
rick_h_bac: oh hmm, that json error is out of the remote proof call. So something isn't happy in our end after all perhaps15:59
benjirick_h_: I'll look at your branch and then we can touch base as to what to do next.16:11
rick_h_benji: did you want to catch up then and coordinate. Did you still have the diff we had going on in the charm stuff, the branch for the deployer and such?16:11
benjirick_h_: yep, I have all those diffs around16:11
rick_h_benji: cool, sorry about that. You had most of the other stuff so i took the 'easy' one right off. :/16:11
frankbanmacbook users, any suggestion for installing saucy on a mac? I've found http://blog.kylebarlow.com/2013/05/installing-ubuntu-1304-raring-ringtail.html but each guide is different16:12
hatchfrankban: are you going to do it on metal or vm?16:12
bachi evilnickveitch did you see the comments jcastro made in the Google doc on bundles we delivered to you in SF?  i was wondering if that was something i needed to pursue or if you were adding the extra info.16:13
frankbanhatch: metal16:13
hatchhmm sorry I don't have any input then :)16:13
bacrick_h_: how shall we pursue the proof issue jcastro's branch raises?16:14
frankbanhatch: any reason to prefer fusion?16:14
hatchfrankban: I just find it removes any driver issues16:15
bacfrankban: i prefer fusion b/c managing vms is much easier and less stress when upgrading16:15
hatchso things like the touchpad, camera, etc work as expected16:15
bacand it just works16:15
hatchbac: you're using fusion? I've been using parallels but it doesn't seem to support new Ubuntu well at all16:16
bacsnapshot the vm, upgrade to the next release, and if it doesn't work just rollback or restore from backup16:16
bacyep, i'm running mavericks, fusion 6, and saucy16:16
baconly issue is i have to manually set my display size when rebooting16:16
hatchahh16:16
baci like 1920x1200 and it doesn't stick and picks another16:17
hatchmy issue is that parallels doesn't release the parallels tools for Ubuntu very fast so the display drivers get corrupted16:17
frankbanbac: interesting, yeah, snapshots are cool16:17
hatchit's kind of a mess16:17
hatchso now I'm running it w/o X16:17
rick_h_bac: sorry, drink refill. Need to trace what's going on. I'm debugging a local charm-proof instance to see what mjc is returning right now16:17
bacfrankban: when running metal i'd have to mentally clear the next four hours before upgrading in case something went crazy.16:17
bacrick_h_: ok, let me know if i can help.16:18
rick_h_bac: ok, mjc is throwing a 500 internal server error at that bundle16:18
rick_h_bac: so the thing to do from here would be to toss it at a local charmworld instance like that pastebin I showed you and see what is failing16:19
bacfrankban: the touchpad is also a big issue.  everything works mac-like even when in fusion.  on metal i found it unusable as brushing it would jump the cursor.  may have improved by now.16:19
bacrick_h_: i wish proof had a hidden option for specifying the server16:20
rick_h_bac: +1 16:21
rick_h_bac: so I pdb;d int the proof and this is the string it's sending up http://paste.mitechie.com/show/1063/16:21
rick_h_bac: so I'll run charmworld locally and use that + httpie to test it and see what's up16:22
rick_h_bac: strange, I get a "could not parse the yaml file" error returned when I test locally :/16:23
rick_h_bac: ok, it passes on mjc as well16:24
bacrick_h_: he has a 'services' option on one of his services that looks dubious to me.  removing it didnt' help though16:24
rick_h_http://paste.mitechie.com/show/1064/ <- bac16:24
rick_h_bac: so it must be something in how the charm proof is sending the values that it doesn't like ?16:24
bacyep that's the unhelpful error i've seen16:24
rick_h_bac: because if I pdb stop and print the bundle_file contents then I get the correct error message16:25
rick_h_bac: well, it's what we've got to show though. It's unparseable yaml. It doesn't know why though. 16:26
rick_h_bac: but the proof code is getting a 500 server error. I wonder if sinzui is still setup to get emails of those16:26
bacrick_h_: yes16:26
rick_h_bac: yea, then we should try to get a copy of that email then. I can't dupe it locally so would be helpful to see the traceback in the email16:29
rick_h_bac: ooh, changed the url in the proof library to my localone and got the error16:29
rick_h_bac: http://paste.mitechie.com/show/1065/16:30
rick_h_bac: can you add type support in proof for float then?16:30
rick_h_bac: looks like I just missed that as a valid type for a config value. I didn't see that in the docs. :/16:31
bacoh16:32
bacbrb16:33
MakyoFfffffff16:34
MakyoForgot to destroy environment.16:34
MakyoI'm so bad at this.16:34
hatchlol16:35
hatchMakyo:  maybe you should create a script to destroy the environment after a 24h timeout lol16:35
MakyoI'm seriously pondering that, yeah, just a cron job that runs nightly with something like `juju destroy-environment -e aws-goju -y`16:37
hatchhaha16:39
bacMakyo: i've found my little notification-based watcher very helpful.16:44
MakyoOh?16:44
baci put something on our blog ages ago16:45
bacnot super smart but gives me enough of a nudge to know when things are running16:45
bacMakyo: http://jujugui.wordpress.com/2013/04/09/alerts-for-running-ec2-instances/16:46
MakyoOh!  Right!16:47
MakyoWill look into that.  Thanks bac16:47
rick_h_thanks benji 16:54
hatchrick_h_: http://bgrins.github.io/devtools-snippets/#console-save16:54
rick_h_bac: so added a card for the float type not supported in the bundles section. If you get a chance to grab it before me yay if not I'll try to get to it. It's not killer atm, because it'll fail to ingest, but needs fixing16:54
rick_h_hatch: cool16:55
bacrick_h_: ok. i'm doing a low priority feel good branch right now but will be free after my late lunch16:55
rick_h_hatch: my console's been messed up a while actually. I need to look into it16:55
rick_h_bac: rgr, thanks. 16:55
evilnickveitchbac, its okay, i am taking care of it...16:56
bacevilnickveitch: ok, that's wonderful16:57
benjigrr, I somehow destroyed my entire bzr config17:11
bacrick_h_: when you have a moment i think you'll appreciate this:  https://codereview.appspot.com/2193004617:11
rick_h_bac: just LGTM'd :)17:11
rick_h_bac: ty!17:11
rick_h_benji: :(17:11
bac:)17:12
gary_posterok, so after consuming the backlog, this is my understanding.  Could people please confirm or correct?  (1) frankban has Makyo reviewing https://codereview.appspot.com/22300044 and needs a second reviewer. (2) bac got m.j.c updated.  It mostly works except we discovered that proof needs to bless floats, and this breaks one of jorge's bundles.  bac and/or rick_h_ are working to fix. (3) rick_h_ and bac are also worki17:20
gary_posterng with jorge to make his bindles ingestable otherwise. (4) benji is working alone on the constraint whitelist charm deployer integration work, and I'd love to know status & expected timeline.17:20
gary_posters/bindles/bundles17:20
gary_postersound right?17:20
rick_h_gary_poster: somewhat. rick is trying to punt to bac to do the float fix and that should take care of jorge's bundles. Rick messed up and did the whitelist bit and it's up for review to hazmat and benjie has reviewed it. 17:21
gary_posterah cool17:21
rick_h_gary_poster: and benji and rick are scheduled to sit down and go over what's next since rick grabbed the whitelist owrk on accident17:21
benjigary_poster: I am working on getting all the branches squared away for proposing17:21
rick_h_but benji seems to be having bzr issues atm17:21
gary_posterbenji, ok great17:21
benjiI am, but have decided to ignore it as much as possible for the moment.17:22
gary_posterbac, are you, uh, catching rick_h_'s punt of the float issue?  I'm assuming that means a fix to proof, an upgrade to charmworld, and another deployment?17:22
rick_h_gary_poster: yea, if benji and I are in good shape I can grab it, but since it's only blocking one charm atm figured it was lower priority. 17:23
gary_posterrick_h_, agreed17:23
gary_posterfrankban, giveing you second review of branch.17:24
gary_posterMakyo, how is video?17:25
gary_postergiving17:25
Makyogary_poster, http://www.youtube.com/watch?v=vbRkfejThkk I'm investigating a way to get less artifacts in the video.17:25
hatchMakyo: yeah I was wondering why the HD versions looked SD17:26
MakyoThe video capture is HD, but transcoding screwed it up.17:27
hatchhmm interesting17:27
MakyoRecordMyDesktop records at 15fps and kdenlive hates that :P17:28
hatchcan you upload the raw file to youtube?17:28
frankbangary_poster: great thanks17:29
Makyohatch, the raw file has no sound.17:29
frankbanMakyo: thanks for the review, destroy the environment!17:30
Makyofrankban, haha, thanks :D17:30
gary_posterMakyo, cool!  Did you send to mramm?  Getting something sooner ratheer than later might be important17:31
hatchMakyo: ahhh gotcha17:31
Makyogary_poster, will do, it's uploading to u1 now17:31
gary_posterMakyo, awesome.  please cc me17:31
gary_posterfrankban, you have two LGTMs and a QAOK :-)17:41
frankbangary_poster: thanks cool17:41
hatchMakyo: ie link branch is proposing so fire-up-that-ie!!!!!!!!!17:41
MakyoIE is NEVER worth nine exclamation points >:/17:42
hatchbut...but....17:42
MakyoGear down there, big rig.17:43
hatchERMAHGERD IE!!!!!!!!!17:43
frankbangary_poster: nice suggestion thanks, I can include an example in the command line, but the charm version is required, e.g. cs:~juju-gui/precise/juju-gui-116. juju-core does not accept an unversioned CharmURL 17:44
gary_posterfrankban, ah ok, cool17:44
gary_posterand lol hatch17:45
hatchhaha17:45
* Makyo boots ie17:46
hatchhmm I'm getting a number of errors in IE on comingsoon17:49
hatch'all' is undefined and 'modules' is undefined17:49
gary_poster:-( 17:49
hatchinvestigating...17:49
hatchoh they are the source map url errors17:50
hatchMakyo: while you're in your IE can you also see if you can repro https://bugs.launchpad.net/juju-gui/+bug/1246971 (I cannot)17:51
_mup_Bug #1246971: Connecting service fails the first time in IE10 <juju-gui:Triaged> <https://launchpad.net/bugs/1246971>17:51
hatchohh17:56
hatchour source maps use //@ which looks like cause errors in IE1017:56
hatchwas this switched recently or has it always been like this and noone has noticed"17:57
rick_h_hatch: I thought that came up before. /me is trying to recall. 17:57
rick_h_hatch: does it only effect sourcemaps though? 17:57
rick_h_didn't we turn off sourcemaps at one point because of this?17:57
gary_posterhatch no change afaik17:57
benjihatch: I don't know of any recent source map work (I did the original work but I don't remmeber if the //@ was there from the start)17:58
rick_h_I remember the //@ 17:58
hatchrick_h_: yes teh application works as expected17:58
rick_h_hatch: ok, so maybe it was just ignored. I do recall the //@ causing IE issues though17:58
hatchok this must be a recent change in IE10 then17:58
hatchwe should create a high/urgent card to fix17:58
hatchapparently it being in the source will break all debugging in IE1117:59
hatchaccording to some of the VS blogs17:59
rick_h_IE11? ugh17:59
hatchI'm going to create an urgent card for this as it'll block release for ie1018:01
Makyohatch, can't repro.18:01
hatchMakyo: thanks, closing18:01
rick_h_hatch: I'm confused. App works as expected but blocks release?18:01
gary_posterhatch, same q :-)18:01
hatchrick_h_: gary_poster: IE10 pops up alerts() asking to debug the error if debugging is turned off18:02
hatchbut if you ignore those, it works fine :)18:02
gary_posterhatch ugh :-(18:02
gary_posterok hatch, you tackling that? :-)18:02
hatchyeah sure18:03
gary_posterok thanks.  hatch, would really like to release tomorrow.  if expedient solution is possible, lemme know.  For example, in theory, we don't serve sourcemaps for IE.  Probably not easy to do, but if it were, I'd be fine with it for tomorrow.18:04
gary_posterFix would be nicer, of course18:04
gary_posterand something we'd need soon after18:04
hatchoh yippee Makefile18:05
hatch;)18:05
gary_posterheh18:05
gary_posterhatch can pair if you'd like18:06
hatchI should be ok - I was just in a talk yesterday about Grunt and how nice the Gruntfile was to read :D18:08
gary_poster:-P18:08
rick_h_hatch: http://uploads.mitechie.com/books/Managing_Projects_with_GNU_Make_Third_Edition.pdf enjoy18:09
gary_posterlol18:09
hatchhaha yeah I have that already18:09
hatchdoesn't make Make any nicer to read :P18:09
rick_h_come on, make shell your new language to learn this year :)18:09
hatchontop of Go and Python eh?18:10
rick_h_sounds like a plan #moretolifethanjs18:10
hatchpretty soon I'll have to forget how to speak English to fit any more languages in there18:10
rick_h_benji: heads up, uploading a dist for the updated jujuclient here: http://uploads.mitechie.com/lp/dist/jujuclient-0.14.tar.gz so we can pull it down and test things out in the charm since it's a local dep. 18:11
benjirick_h_: cool, I'll test it soon18:12
rick_h_benji: ok, I'm going to go fix the charmworld bug then. Let me know if you need a hand with anything on the other side.18:13
benjik18:13
hatchoh nice the source map code is in node not bash18:16
hatchhaha18:17
hatchgo benji18:17
benjiI don't always write JS, but when I do it is to avoid writing shell script.18:17
hatchhahaha ^518:17
benji^518:17
benjiwe need an IRC symbol for a fist bump... or not18:18
hatch=B ᗺ=18:19
hatchlol18:19
benjiheh18:22
hatchMakyo: here is the link to the branch if you didn't already have it https://codereview.appspot.com/21430043/18:34
bacrick_h_, benji: we seem to have a bit of a problem: http://manage.jujucharms.com/heartbeat  look at the charm/basket queue18:51
* benji looks18:51
benjihmm18:51
rick_h_bac: yea, was just going to ask on that.18:52
baci don't really know how to proceed.  we have no visibility into prodstack18:52
rick_h_bac: yea, we've got to ask is for log files. 18:52
rick_h_bac: and hope something jumps out from there. 18:53
rick_h_bac: no vangaurd right now :/18:53
bachi sinzui.  we did a release to manage.jujucharms.com today.  it seemed to well but then i noticed on http://manage.jujucharms.com/heartbeat the charm queue has grown crazy18:54
sinzuiwell is ingest running?18:57
sinzuihmm, I think Abels unlanded branch would tell us a little more18:57
rick_h_looks like no vanguard right now and it's not dead in the water to go emergency on it. 18:58
rick_h_I've got a branch that needs a deploy up for review so we can watch it, get another deploy and get logs from it at that time?18:58
rick_h_bac: have time to review https://codereview.appspot.com/2237004418:59
rick_h_bac: for the float/config validation fix in proof18:59
bacsinzui: i'm not sure if ingest is running18:59
sinzuibac, how long has it been since the deploy?19:01
sinzuiand if someone did the deploy, do we know the heartbeat was sane afterwords19:02
baca good while.  let me look19:02
bacsinzui: 15:27UTC.  no, the heartbeat page showed the two failures b/c the bundles were removed by the migration and are to be repopulated by ingest19:03
sinzuiyep, I expected that. Ingest takes about 15 minutes, but we are hours later19:04
bacsinzui: so queue runs will just keep adding on?19:06
bacif ingest were started now it may not be able to empty the queue19:06
bacrick_h_: jjo is not the vanguard but he is around.19:07
bache did the deploy19:08
baci'm not sure what to ask for19:08
bacrick_h_: i did your review but don't understand why an unknown type is not an error19:08
rick_h_bac: we'd ask for the logs. There's app.log I think in the charm directory var/app.log (there's a few othersin there)19:08
rick_h_bac: yea, I wasn't sure what to do on that so went with ignore, but updating it to throw an error. 19:08
bacrick_h_: yeah, the log dir is a confusing jumble to me19:09
rick_h_I think that's the best way to keep that up to date19:09
rick_h_bac: yea, we'll figure it out though19:09
* bac looks at staging19:09
sinzuibac, The last item in the heartbeat indicates we are queue work, but not ingesting, webops can empty queues easily enough 19:10
rick_h_bac: sinzui ping'd jjo but looks like he ran away. 19:13
rick_h_bac: sinzui will keep an eye on vanguard19:13
sinzuibaskets are going down19:13
bacrick_h_: thanks19:13
bacsinzui: you mean the basket count?19:13
sinzuiyes19:13
rick_h_sinzui: oh, interesting19:13
bacyes, berry berry19:13
rick_h_crazy queue19:13
sinzuiWe can see that something is pulling19:13
rick_h_yea, we expect most bundles to fail so maybe it's got backed up and taking a few to get to one of the good ones :/19:14
bacthe count is going down but it hasn't found any valid bundles yet19:14
bacmeanwhile on staging there are 11 bundles that are valid19:14
sinzuiI saw19:15
rick_h_ok, well keep an eye on the queue and see if we can get down to 019:15
bacthe entire set is about 14 baskets so something more than 14 bundles19:15
rick_h_bac: right, but I'm thinking the queue was already huge from something? The charms queue is a LOT larger than the number of charms in the store as well19:16
rick_h_32K ?19:16
bacrick_h_: i don't have the initial numbers.  32k is nuts.19:16
rick_h_bac: right, hmm the countvs just jumped up again19:17
bacwe should ask to have the queue process paused19:17
rick_h_yea19:17
bacthere are 2410 charms.  queue has been run ~16 times since launch.  16 * 2410 = 3856019:18
rick_h_bac: heh, well good to know we got through 500ish19:18
bacwell, ok, maybe it was only run 15 times19:19
sinzuibac, rick_h_ I think this can help bring some sanity to the queue numbers http://pastebin.ubuntu.com/6372276/19:21
sinzuirick_h_, bac: I timed ingest to take between 13 and 17 minutes last august. I think we want to watch the numbers to learn what the current time is19:22
rick_h_sinzui: rgr, thanks. Will see if we can get a vanguard to take it up 19:22
rick_h_sinzui: bac jjo is chatting now19:24
sinzuirick_h_, bac: Note that http://manage.jujucharms.com/tools/store-missing says it has not completed a single charm ingest for 4 hours19:28
bacinteresting19:28
rick_h_sinzui: yea, I think this might lead to getting logs and finding an issue19:28
sinzuiDid we do something mad like change ingest to get every history revision?19:29
bacsinzui: queue finds the last 10 revisions19:29
rick_h_sinzui: well there's the backfill that does 10 revisions back19:29
rick_h_so not every, but 1019:29
bacrick_h_: can jjo determine if ingest is running?19:30
rick_h_bac: not asked. he's EOD'ing in 30min so get anything we need now19:30
bacit may make sense to turn off the queue cron and run it once manually19:31
bacrick_h_: he seems to be responding to your queries19:31
rick_h_asking for the logs now19:31
sinzuirick_h_, We discusses this last July and August. back history has to be done in a separate proc because we need to guarantee responsive ingestion. I think 10 rev is less that 150 minutes but certainly way to long the the 15 minute balance we setup19:33
bacrick_h_, benji: is this deploy to production the first with 10 revision history turned on?19:36
rick_h_bac: I didn't think so. :/ it's been a long while19:36
bacrick_h_: i don't know the last rollout to prod19:36
rick_h_bac: ok, so charm queue is 019:36
bacso ingest is running19:37
sinzuibac, rick_h_ http://manage.jujucharms.com/tools/store-missing19:37
sinzui^ happy19:37
Makyogary_poster, ping19:38
bacsinzui, rick_h_: is there anything about the proofing on production that would hang?  egress firewall rules?19:38
sinzuilet watch heartbeat for 15 minutes19:38
bacwait, we're talking to ourselves so it can't be an issue19:38
rick_h_bac: no, proofing doesn't hit the net. Just the db19:38
gary_posterMakyo, sorry! was on phone with Jeff and lost track o' time.  coming19:39
Makyogary_poster, ah, okay!19:39
bacrick_h_: well, ingest hits a cw url which hits the db19:39
bacrick_h_: i'm just trying to think about differences with prod that could be hanging ingest19:39
sinzuibac. I am not current with proof changes. I think the last change made in cw was to include the version of proof in the cache to ensure the right version is always installed19:40
rick_h_bac: ah, understood19:40
* hatch sorry those of you who import your coke from Mexico http://gizmodo.com/mexican-coke-is-ditching-cane-sugar-for-high-fructose-c-145943747719:40
sinzuihatch, you have just ruined my day19:41
* sinzui goes back to steaming his brownies to make them soft again19:41
hatchlol19:41
bacsinzui, rick_h_: baskets just dropped from 16 to 13.  no bundles added though19:43
rick_h_bac: cool. 19:43
sinzuibac, why is basket ingestion so slow less than 1 a minute19:43
bacsinzui: unclear19:44
bachatch: that makes no sense.  1 peso/liter isn't that much.  i think this is just another 'New Coke' moment to force a change to cheaper corn-based coke.  glad i don't drink the stuff anymore19:45
rick_h_sinzui: not sure, you'd think it'd be faster. One file to process really19:45
hatchbac: You're probably right19:45
sinzuirick_h_, bac, given the immediate fix we saw in the missing list. It is clear that the queue must be drained to get updates to the db and es. I think we might need to slow down the supervisord proc enqueue less often19:45
rick_h_bac: sinzui yea, just got the queue refilled, but didn't get through the bundles :/19:46
sinzuirick_h_, I see the bug.19:46
bacrick_h_: but it doesn't make complete sense.  the items are processed in order, right?  so it will eventually get to all of the bundles.  yet we see none being added19:46
bacor is it not ordered?19:47
rick_h_benji: so my jujuclient fix got shot down. He's prefer we band-aid it in the deployer middleware itself and not in the deployer/client. 19:47
benjirick_h_: ooh19:47
bacsinzui: and?19:47
sinzuirick_h_, bac, we only queue charm tips, but the queue size indicates we are ingesting revisions. that is F*CKING crack19:47
rick_h_bac: right, but there were 16? so we're at 27, so we've got through 5 of them so far. 19:47
benjirick_h_: well, I guess we can resurect my branch, since that's what I was doing19:47
rick_h_benji: +1 19:47
sinzuiproduction will is on crack and may die of complications in a few hours19:47
rick_h_benji: he's not a fan of the limited 'approved' set of stuff and doesn't want the band-aid in the client19:48
bacsinzui: but this scheme has been running on staging with no issues19:48
rick_h_benji: and he's all on board for fixing the communication/deployer as a library issues19:48
benjirick_h_: re. library: cool!19:48
bacrick_h_: my point is since deployment hours ago no valid bundles have been ingested.  that makes no sense19:49
rick_h_bac: understood19:49
sinzuibac, I see staging trying to do that. At least it gets little cycles.19:49
bacrick_h_: sinzui: do you want to move to a hangout in 10 minutes?19:49
rick_h_bac: sure19:49
bacfeel free to fire one up now but i can't join for a few19:50
rick_h_benji: let me know what you need to help move this forward then. gary_poster fyi ^^19:50
sinzuibac. Nonetheless. There are only about 130 good charms and 600 bogus charms. The queue looks at those for new revisions. I think charmworld is asking if a revision ever changes, which is a violation of reality19:50
gary_posterrick_h_, was following along as best I culd while on a call.  could you give me a quick hangout so I can make sure I understand, before you have the bigger powow?19:51
rick_h_gary_poster: https://plus.google.com/hangouts/_/7acpjqjf6evpv7503n0uq4psjg?hl=en19:52
sinzuirick_h_, bac, when the charm queue clear we got updates, since we cannot ever clear the bundle queue we don't get updates. I makes sense to me19:53
bacrick_h_: ping when you're readu19:55
bacs/readu/ready/19:56
rick_h_bac: https://plus.google.com/hangouts/_/7acpjqjf6evpv7503n0uq4psjg?hl=en19:56
rick_h_bac: you're right, sorry didn't think it through19:56
hatchMakyo: another ie branch to qa :) sorry https://codereview.appspot.com/22450044/20:03
sinzuibac, rick_h_ there are exactly 764 branches charms and bundles to process in Lp. Production might know about 20 more deleted charm branches. Any queuing greater than that is sign of insanity. I the db is corrupt with bogus charms. It might be fair to say that the queue/ingest is being feed revisions, not charms and bundles. It would be okay to queue revisions if the queue could just ask for changes since a last date, but I am20:03
sinzui sure that code was not written20:03
rick_h_sinzui: we think bundles is hitting egress filtering and causing it to timeout/take long20:07
bacrick_h_: https://rt.admin.canonical.com//Ticket/Display.html?id=65726&results=564596e05af8d34ac09865c9e1dfbc6920:08
rick_h_bah20:08
bacsinzui: i agree with your thoughts on queueing up revisions and we need to fix that.  as rick_h_ said, the immediate problem causing the delay is the bundle proofing is being blocked by an egress firewall and timing out on each proof request.20:19
rick_h_bac: worse than that, it's not running the right version of charmtools!20:20
sinzuirick_h_, charmtools in in the download cache. Are you saying the wrong cache was deployed to production?20:21
rick_h_sinzui: so we tried to change the proof url since it was hard coded to manage.jujucharms.com. The version installed is 1.120:21
rick_h_sorry, 1.1.0 and it should be 1.1.220:22
* rick_h_ is checking download-cache list right now20:22
rick_h_bah, there's something bigger there. the wrong version is installed, why the runs aren't throwing errors/exceptions to the wind I don't know. 20:24
sinzuiAbel added charm-tools to the download cache to ensure the code we deploy has a matching proof. LTS was running ancient charm-tools20:25
rick_h_sinzui: yes, and the download-cache and requirements.txt are updated to 1.1.220:26
rick_h_sinzui: somehow it's not updated on production. I asked for a change to bundles.py and there was no bundles.py on the machine because it was still at 1.1.020:27
rick_h_https://pastebin.canonical.com/100009/ was the result of looking for anything charmtools20:27
=== BradCrittenden is now known as bac
rick_h_we lost our webops and waiting for a new person to torture. approx 2hrs according20:27
rick_h_according to jjo20:27
hatchI'm going to run and grab a bite/run an errand, be back in < 1h20:28
sinzuirick_h_, I think this step in deploy is supposed to guarantee we get the download cache ./build-charmworldgo ${REVNO?}20:34
rick_h_sinzui: ok, will look into that. thanks for the heads up20:35
rick_h_sinzui: is that something in the webops add-on? I can't find that file in the charm/source?20:39
sinzuirick_h_, yep20:40
rick_h_sinzui: ah ok. Then definitely good to know since I can't see it atm20:40
sinzuirick_h_, they make a fat charm: https://wiki.canonical.com/InformationInfrastructure/WebOps/CDO/Charmworld20:40
sinzuiThey pull the revno we request and pull download-cache, then pack them into the charm for upgrade-charm to extract20:41
rick_h_sinzui: rgr, thaniks20:42
sinzuirick_h_, I need to get children from school. I am searching for a branch that might contain the script that makes the fat charm20:43
rick_h_sinzui: ok, thanks. I'm sure we'll be able to move forward once the new vanguard arrives. 20:44
bacrick_h_: proposed change to marco20:55
bacrick_h_: we're in a lull so i'm going to step out for a bit before it rains again.  that ok with you?20:56
hatchbak20:57
benjirick_h_: I have to go to my appointment now, but here is a WIP branch: lp:~benji/+junk/sanitize-constraints-and-other-fun 21:00
benjirick_h_: as best I can tell we just need to fill in the tests and implementation for the sanitize_constraints function and we'll be good to go21:01
benjiI invision the remaining tests to be in their own class and directly excersize the function21:01
benjiafter that we just have to do some test deploys and we should be done21:02
hatchMakyo: not sure what happened it didn't get the tests before https://codereview.appspot.com/21430043/ here is a proposal with the tests21:02
hatchbut thanks for both qa's21:02
rick_h_benji: rgr, I'm working on broken charmworld deploy right now, but will see what I can do21:03
benjirick_h_: thanks21:03
hatchgary_poster: so looks like all of the urgents are done - shall I now switch to the spreadsheet?21:11
gary_posterhatch, lemme take a look at the kanban...while you review this, please? :-D https://codereview.appspot.com/2250004321:11
hatchit's on!21:12
Makyohatch, ah, cool, thanks21:13
gary_posterhatch, hook up the import control?  That's on the spreadsheet *and* in the bundle lane *and* hopefully fast. :-)  After that, yeah, spreadsheet.21:14
hatchgot it!21:15
gary_postercool thank you21:15
hatchgary_poster: in this branch I deleted the importexport.js file that confuses everyone21:47
gary_posterhatch sounds good.  It had confused me too :-)21:48
gary_posterbenji, rick_h_, I need to go in 9.  Can be back later.  Where are we?  https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.vq3atlo18hgrrbfqbii5cgqch4 ?21:52
* gary_poster back in 2 hours or so22:02
huwshimiMorning22:09
hatchmorning huwshimi22:14
huwshimihatch: Welcome back :)22:15
hatchspanks - glad to be back22:15
hatchalbeit only for a couple days haha22:15
huwshimihatch: Are you off next week?22:18
hatchhuwshimi: only Friday and Monday, going snowboarding22:20
huwshimiOh nice!22:20
hatchyeah I'm horribly addicted to it22:21
huwshimihehe22:25
hatchhuwshimi: this morning https://plus.google.com/118445028821328031751/posts/R5RRM1QRxNt22:25
huwshimihatch: Woah!22:27
huwshimihatch: Is that at your house?22:27
hatchright beside22:28
huwshimiwow22:28
hatchyeah it's awesome here in the winter :)22:29
hatchif you can handle the temperatures that is haha22:29
huwshimihatch: I think we had about two minutes of snowflakes amongst the rain this year22:30
hatchhaha - most would prefer that :)22:30

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