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

hatchhuwshimi hey you there?01:53
huwshimihatch: Hey01:57
hatchhey check out my msg on your PR01:58
huwshimihatch: Oh, strange... why would they be there?01:59
hatchI have no idea...02:00
hatchmaybe your fork of juju-gui is out of date02:00
huwshimi$ git pull juju develop02:01
huwshimiFrom github.com:juju/juju-gui02:01
huwshimi * branch            develop    -> FETCH_HEAD02:01
huwshimiAlready up-to-date.02:01
hatchthen run git push02:01
hatchto update your fork02:01
hatchI really don't know why those old commits would be there anyways02:04
hatchit should have diff'd them out as far as I know02:04
huwshimiyeah, quite strange.02:06
huwshimibrb, lunch02:07
hatchThanks for fixing that huwshimi  but it looks like it's still pushing all of those commits in, maybe it needs a new PR? 02:13
huwshimihatch: Yeah, I'll delete and start again :)02:21
hatchsorry I wish I knew what the issue was so that we could avoid it in the future hah02:21
hatchhuwshimi I was wondering if you play any games on your new machine?02:30
huwshimihatch: Nah02:31
hatchYou should go buy some and let me know how they run so I can decide on the computer to buy haha02:32
huwshimi:)02:32
huwshimihatch: By the time you're ready the new Airs will be out anyway02:52
hatchlol!02:53
huwshimihatch: https://github.com/juju/juju-gui/pull/1702:58
huwshimihatch: Lots of strangeness02:58
=== gary_poster|away is now known as gary_poster
* rick_h__ goes to doc office. bbl13:22
* gary_poster goes to son's prospective middle school. bbl13:32
benjibac: Spads is asking me if the prod charmworld charm is "jc-charmworld/0"; do you know if that is right?13:53
benjibac: he may mean "service name" instead of "charm"13:54
bacbenji: oh, the charm lives at bzr+ssh://bazaar.launchpad.net/~juju-jitsu/charms/precise/charmworld/trunk/13:55
benjibac: do you know the service name is for the production charmwolrd?13:56
bacno.  on staging it is just 'charmworld'13:56
baccharmworld/013:56
frankbanhazmat: ping re: the deployer MP13:56
bacbenji: if he has access to prod, a juju status should show.  should only be one similar to charmworld13:57
bacbenji: but webops grab everything and deploy locally so it could be there is a renaming13:58
baclet me look at the docs13:58
benjibac: do we run ES as its own charm in prod?13:58
benjis/charm/service/13:59
bacbenji: looking at https://wiki.canonical.com/InformationInfrastructure/WebOps/CDO/Charmworld it looks like jc-charmworld is correct13:59
bacbenji: yes, i would think ES is its own service.  can't see how it'd be otherwise.14:00
bacbenji: the problem with understanding production is that page refers to scripts that we don't have access to, so it's kind of hard to see what's happening.14:01
benjiyeah, really14:01
benjibac: it seems the staging.jujucharms.com environment is in a bad way: 2013-12-12 14:17:53,234 ERROR juju environment not found: is the environment bootstrapped?14:22
benjihave you seen that before?  Should I nuke it from orbit and rebuild from scratch?14:22
bacbenji: but, but, it seems happy http://staging.jujucharms.com/heartbeat14:23
bacbenji: where did that terrifying message come from?14:23
benjibac: happy except that it hasn't ingested since 2013-12-0814:23
benjibac: from Spads in #webops14:23
bacbenji: oh, well14:23
bacbenji: huh.  well it is definitely bootstrapped14:24
benjibac: I guess I need to figure out why my orangify script isn't working so I can poke on the staging env myself14:25
bacbenji: i found some subdirectories in that branch that were created at some point and were messing up the connection.  once i deleted them i could connect.  if 'bzr st' shows anything weird (local, charms, etc) you may try the same.14:27
bacweird = unknown14:27
benjibac: staging.jujucharms.com is running on go-juju, right?14:27
bacyes14:27
bacbenji: you may want to manually copy over the ssh credentials.  the install script is broken in that it doesn't copy if they exist even though new ones may be available14:28
baci've fixed it but haven't yet merged14:28
bacbenji: i'm look at the staging charmworld/0 logs and don't see where that message from spads comes from.  i wonder if his env is not right.  there definitely is a problem with ingesting, and it looks like the job hasn't even tried to run since 9-Dec14:35
bacs/look/looking/14:35
benjibac: I have determined that he was running py-juju14:35
bacoh14:35
bacthat'll do it14:35
benjiheh14:35
bacso, one red herring down14:36
benjibac: I almost have orangify working, when I run "nova list" I get juju-staging-machine-0 through juju-staging-machine-5, is that expected?14:36
baccurtis did fix a problem with mongo the other day14:36
bacbenji: sounds about right14:36
bacbenji: what does juju status do?14:36
benjibac: orangify still fails, this time with "client: fatal: failed to establish ssh session (2)"14:37
benjithat is after printing:14:37
benjiFound a compatible Juju.14:37
benjiOrange credientials loaded14:37
benjiJUJU_HOME set to /home/benji/workspace/staging-tools/staging-tools14:37
benjiState server is at 10.55.32.4814:37
bacbenji: there's the problem14:38
baccredentials is misspelled14:38
benjiheh14:40
bacbenji: prob won't help but try pulling a new version of that branch14:46
* benji tries14:46
bacbenji: so there are lots of problems from 9-Dec and before.  that was when curtis got mongo working again14:46
bacbut it doesn't look like ingest has run since then.  it hasn't generated any errors since that date14:47
benjibac: the main thing I'm interested in is getting ingest to run again; my hypothesis is that it will then make the search results on staging match prod (which are wrong, but at least we'll have an idea of what is going on)14:47
bacright14:47
benjibac: same error after pull14:50
bacbut no typo?14:50
frankbanguihelp: I need two reviews for https://codereview.appspot.com/41350043 (quickstart). Is anyone available? 14:56
sinzuihi bac15:02
bacsinzui: when mongo failed on staging it caused ingest to fail repeatedly and supervisord marked it as FATAL. do you know how to get it going again?15:02
bacsinzui: we're engaging spads in #webops on the issue though he isn't sure15:04
sinzuifor staging?15:04
sinzuibac, he doesn't have access15:04
sinzuibac https://docs.google.com/a/canonical.com/document/d/190xKLPpEPSrlVix4D9eX_aTciiKFo12h0u7MpzqI28I/edit#heading=h.g8pxm6cjaq6915:05
sinzui^ we need to add a section about restarting the service, which I know but cannot type without doing it.15:05
sinzuiper canonistack guidelines, there is a script in /srv that allows webops/devops to restart the services15:06
bacsinzui: can i just stop and restart supervisord?15:08
sinzuibac: I did this http://paste.ubuntu.com/6561801/15:09
sinzuisudo /srv/deploymgr/charmworld restart15:10
bacsinzui: could we have just used the 'start_worker' target in the makefile?15:10
bacwhich i just found15:11
bacsinzui: i'll update the doc unless you want to15:12
sinzuibac: please do update the doc.15:12
bacthanks sinzui15:13
rogpeppebac: sorry, was at lunch15:30
bacrogpeppe: ?15:30
bacrogpeppe: i didn't ping you, perhaps someone else did15:30
bacbut i hope you had a good lunch15:30
rogpeppebac: oh, sorry, i misread a remark of yours15:31
rogpeppebac: it was just below one addressed to me15:31
rogpeppebac: ignore :-)15:31
bacrogpeppe: np15:31
bacrogpeppe: was that a really late lunch or a really long one?  :)15:32
rogpeppebac: late15:32
rogpeppebac: i often forget to have lunch until hunger prompts me :-)15:33
benjibac: the queue size is increasing, so it looks like a queue-clearing is in the future15:46
bacbenji: ok15:47
bacbenji: should be able to do: db['charm-queue'].remove()15:52
bacthough it is kinda scary15:52
benjiheh, yeah15:52
bacand db['basket-queue']15:54
* benji wonders if it is about team call time.15:57
hatchjujugui call in 115:59
hatchbenji are you actually going to make it on time today? :P15:59
benjiI changed my reminder so I can't snooze it.  That seem to help.15:59
gary_posterrick_h__, assuming you are still at dr16:00
hazmathatch, Makyo how'd the rest of the meeting go16:05
hatchhazmat pretty good, hangout after standup to fill you in?16:05
hazmatsure16:06
hazmathatch, sounds good16:06
hatchcool I'll ping when we're done16:06
hatchhazmat https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj0?authuser=1 just finishing up 16:09
bacbenji: so shall we do the cleanup?16:15
benjibac: yeah, we don't have a choice at this point, and since I haven't gotten orangify to work I need to enlist you if you can help16:16
bacbenji: hangout?16:16
benjibac: sure, I'll create one16:16
Makyohazmat, do we have a sprint authorisation number?16:30
rick_h__gary_poster: back now, sorry for the delay. 16:34
gary_posternp rick_h__ .  how are you feeling?  dr give you any meds?16:35
rick_h__gary_poster: yep, shot in the $#@$ and meds to take. Wheeeeeee16:35
gary_posterheh, hope it gets you well fast16:36
rick_h__gary_poster: rgr, working on it. Will be afk today. Sorry for the MIA16:36
gary_posternp at all.  good, go rest16:36
frankbangary_poster: book expensed :-)17:01
gary_posterfrankban, :-) cool17:01
hatchll17:01
hatchbah17:01
hazmatMakyo  sent via msg17:03
Makyohazmat, ty17:05
hatchMakyo did you ever qa my fullscreen removal branch?17:09
Makyohatch, Was going to wait on your verdict re: stuff in trunk fixed in this branch or another, but I can now.17:09
hatchwell....the issues are in trunk17:09
hatchso just trying to fix anything small now17:10
Makyohatch, okay, will in a sec; let me finish code review of frankban's branch17:11
hatchyeah sure, no rush17:11
hatchMakyo so it looks like the issues rick_h__  raised are more serious than a few cleanup lines in this branch unfortunately so I'll have to do them as a follow-up17:22
Makyohatch, np, will QA in a sec with that in mind17:22
hatchjust so you are aware, for whatever reason the browser is being dispatched twice which is causing the issues...heh, more double dispatch....lol17:23
MakyoSiiiigh~17:27
hatchthis issue may have always been there we just never noticed it for whatever reason17:27
hatchI can't really see how any of the changes that have been made would cause this17:27
Makyohatch, lgtm17:56
hatchthanks17:56
bachi benji, did the charms get ingested?17:57
benjibac: yep; I'm now curious about a bundle potentially "stuck" in the bundle queue on prod18:02
benjievery round now staging has one fewer baskets queue than prod and after the queue is drained there is still one basket in the basket queue; I'm wondering if the stuck one is Jorge's18:03
bacbenji: what have the queues on prod looked like?  1164/22 now -- have then been going down?18:03
bacs/then/they18:04
bacs/then/they/18:04
bacgah18:04
benjibac: yep they are occasionally drained to 0/118:04
bacoh, i bet that 1 is the problem!  :)18:04
benjiyep18:04
bacbenji: oh, with orangify you can now say 'source orangify stop'.  pretty darn neat.18:16
bacno more fumbling around looking for that pid file18:16
benjiI saw that, nice addition18:16
bacmy proudest moment of the morning18:17
benji:)18:17
hatchhas anyone flown Delta internationally? 18:52
BradCrittendenhatch: yes19:03
=== BradCrittenden is now known as bac
bacgary_poster: call19:03
baci meant, gary_poster call?19:04
hatchbac and?19:04
gary_posterbac almost19:04
bachatch: and it was fine19:04
bachatch: i think delta is a fine U.S. carrier.  i mean they don't do long hauls like the good asian carriers.19:06
bachatch: i've flown delta on detroit-tokyo and survived.  where you going?19:07
hatchMakyo so those issues that rick_h__  found are actually in release :/20:38
hatchI -think- it has to do with the new token stuff20:38
hatchjust going to bisect to try and roll back to find it20:39
MakyoI thought we knew that b/c it was in trunk? 20:40
hatchwell trunk has a lot of other stuff in it too20:40
hatchcould have been from anything really20:40
MakyoYeah.20:40
MakyoMaybe come up with a test that fails for one of them to make it easier to bisect?20:40
hatchsomehow I broke my repo, trying again, doesn't look like it was the token code though20:46
hatchhypothesis, incorrect20:46
hatch:)20:46
gary_posterolder, hatch?20:53
gary_posterjujugui, looking for small but important review and qa https://github.com/juju/juju-gui/pull/1820:53
Makyogary_poster, on it20:53
gary_posterthanks Makyo 20:53
gary_posterhatch, I mean, the problem is older than the token code?20:54
hatchit appears so, still bisecting20:54
gary_posterk20:55
hatchthe double dispatch is caused by the app routing, then the login comes in as successful and then it routes again20:56
hatchdouble dispatching, but that doesn't appear to be the root cause20:57
hatchjust another bug which we haven't noticed20:57
gary_posterack20:59
hatchhaving to keep killing and restarting the app sure slows down bisection hah20:59
* hatch crosses fingers it wasn't him21:00
hatchgary_poster according to bisect this is the culprit which broke the onboarding stuff....oops we should qa that :)21:03
hatchhttps://github.com/juju/juju-gui/commit/dec7617b378034fd93e41b1c382bb075c8b4978a21:04
hatchnot entirely sure what the issue is21:04
hatchbut at least we know where it is21:04
gary_posterhatch, so that is the token stuff21:04
gary_posterI just sent a note to rick that the git history is flatted rather than collapsed21:05
gary_posterwhich obscures the fact that this is part of the token commit21:05
hatchohhh that's why these old ones are confusing21:05
gary_posteryup21:05
gary_posterI asked him if we can fix that21:05
gary_posterwe'll see21:05
gary_posterflattened21:06
hatchok cool, well hey, we just moved to git, and just used bisect to find an issue.....yay for the conversion :D21:06
gary_poster:-)21:06
gary_posterhatch, would it be helpful for me to look at that with you, or don't bother?21:09
hatchumm I think I'll be ok - I'm going to see if this is also the cause of the other redirection issues that just didn't happen to be noticed until I tried to muck with it recently21:09
MakyoWhoa.21:09
hatchit's very odd though that this commit is the culprit I don't see anything on face value that would cause the onboarding issue21:10
Makyogary_poster, +121:12
gary_posterhatch, currentUrl and onLogin seem most likely culprit in there to me.  It does affect the post-login redirect.21:12
gary_posterMakyo, thanks.  A bit scary, eh?21:12
Makyogary_poster, yeah, thus the 'whoa' :T  Good catch, thanks for the fix!21:12
gary_posterthank you21:13
hatchgary_poster haha nice catch with #18 :)21:17
hatch*phew*21:17
gary_posterhatch, yeah, we kinda need that out soon. :-(21:17
hatchgary_poster fyi - here is the issue https://github.com/juju/juju-gui/commit/dec7617b378034fd93e41b1c382bb075c8b4978a#diff-2de9eda1655d499e934e7465e87ca0bdL95121:21
hatchsuper trivial fix 21:21
hatchor maybe not....I haven't used the token stuff21:22
hatchwana have a quick chat?21:22
gary_posterhatch, how so?  you don't have originalPath anymore21:22
gary_posteryeah sure21:22
rick_h__hatch: the double dispatch I think is from the url replace and re-dispath. It'll hit that twice21:32
rick_h__hatch: by bybassing the viewstate, nothing in subapp is watching/protecting against double dispatch21:34
rick_h__hatch: let me know if a hangout to chat will help, and maybe I'm wrong on this, but I think it's a simple fix.21:34
hatchrick_h__ hey we found the issue, it was actually located in a race condition in the login system that has been there......forever21:51
rick_h__hatch: oh well crap21:52
hatchyeah - it's a pretty odd edge case that has just been slowly exposed as we refactored code over the past month21:52
hatchjshint for chrome: http://mrpotes.github.io/jshint-extension/21:56
* Makyo dogwalks, getting stomped.22:04
huwshimiMorning22:10
rick_h__hatch: ok cool. Never mind then. 22:10
rick_h__glad you got it figured out22:10
gary_postermorning huwshimi.  you have time to talk for 15 min now?  I have family obligations now22:14
gary_posterI mean later22:15
gary_posterbah22:15
huwshimigary_poster: Sure, happy to talk anytime22:15
gary_posterreboot: morning huwshimi.  you have time to talk for 15 min now?  I have family obligations later :-P22:15
huwshimigary_poster: OK!22:15
gary_posterok thanks huwshimi .  https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.87augjtfc2pm1q1ilfuhencae022:16
huwshimigary_poster: Wait, did you need to reboot now, or in 15 minutes?22:16
gary_posterhuwshimi, neither, I am just exhibiting poor communication skillz :-P22:16
huwshimigary_poster: I know, I'm just teasing :)22:16
gary_posterlol22:17
huwshimigary_poster: https://codereview.appspot.com/20120044/diff/1/bin/merge-files#newcode11922:20
* gary_poster runs22:32
gary_posterbye all22:32
=== gary_poster is now known as gary_poster|away

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