gary_posteranother review, though no rush: I plan to go now :-) : https://codereview.appspot.com/864704301:21
bachi frankban.  i'm trying to reproduce bug 1167344 but can't do it right now12:52
_mup_Bug #1167344: Deployment via the juju-core API fails with a "no options to set" error in some cases <juju-core:New> <juju-gui:Triaged> < https://launchpad.net/bugs/1167344 >12:52
bacwhen i try to add dokuwiki i get a ghost service but then the service never fully deploys and i don't see the error you reported12:52
rogpeppebac: is this with trunk?12:53
bacrogpeppe: i'm using a local branch based on trunk from ~1700UTC yesterday12:54
bactrunk + some debug messages12:55
frankbanrogpeppe, bac: I bet this is fixed in trunk12:55
frankbanbac: I just succeeded in deploying docuwiki, which was one of the problematic charms12:56
rogpeppebac: hmm, i think i fixed it in rev 113912:56
bacfrankban: ok, i'll update and try again12:56
bacrogpeppe: thanks!12:56
rogpeppebac: but i'd be interested to know if you're running a version before that12:56
rogpeppebac: because perhaps i didn't really fix the problem...12:57
bacrogpeppe: actually i was running older than that.  looks like 113012:58
rogpeppebac: hmm, i'm a bit surprised it doesn't fail for you12:58
rogpeppebac: but that bug doesn't have much info on how to reproduce it...12:59
bacrogpeppe: we'll see what happens now13:03
gary_posterrick_h_, fwiw the jenkins failure we just had (in IE) was browser search widget: should support search change events: timeout of 200ms exceeded.  I am guessing that's an intermittent one, unfortunately.  retrying, but if you all have ideas on making that more reliable it would be fab13:03
rick_h_gary_poster: ok, hatch and I were talking about it a little bit the other day. There's one idea to try out on it. I'll file a bug/card for it.13:05
gary_postercool thanks hatch13:17
gary_postercool thanks rick_h_ 13:18
benjigary_poster: I commented on the ID-munging branch not realizing that it had already landed.13:22
benjiI think the things I noted are worth another branch.13:23
gary_posterbenji, thanks for safe id review comments.  This is landed already, so I passed them on to Francesco, who is working in a related part of the codebase in his current branch.  FWIW, my choice of underlines only within the .replace was conscious--in fact, I initially had what you describe.  I decided I wanted a dash to separate the replaced value from the hash value, and to have the division between the two visually c13:24
gary_posterlear by not usuing hyphens elsewhere.  I don't think the decision is problematic (unlike, potentially, your comment about needing to not start with a number--actually in html 5 you can, and in html 4 you can't but you also can't start with a dash, and I forget about the CSS rule but it is in the bug, but anyway it sounds fine to me).  Francesco and I agreed that he would add the character, and I said that he ought t13:24
gary_postero do whatever he wanted with the dash stuff, which IMO is entirely stylistic.13:24
gary_posterbenji, I started writing the above not realizing that you would write about it here before I finished.  Or that I would write a novel. :-)13:24
benjisounds like it's being handled13:25
bacfrankban, rogpeppe: ok, with juju-core trunk i can now deploy dokuwiki.  frankban did the error in bug 1167344 happen frequently?  was there any way to trigger the failure?13:29
_mup_Bug #1167344: Deployment via the juju-core API fails with a "no options to set" error in some cases <juju-core:New> <juju-gui:Triaged> < https://launchpad.net/bugs/1167344 >13:29
frankbanbac: the only way to dupe I know is deploying dokuwiki from the GUI.13:30
bacfrankban: how often did you see the failure?13:31
hazmatinteresting migration comparison story of backbone style to angular .. http://www.localytics.com/blog/2013/angularjs-at-localytics/13:31
rogpeppebac: in trunk you should not get that error because that error no longer exists...13:31
frankbanbac: IIRC, it always failed13:31
bacrogpeppe: ok.13:32
benjiI am getting a connection refused error when connecting to the juju websocket; is this a known issue?13:35
bacgary_poster: do we want to keep bug 1166153 around since we aren't using YUI 3.10 and it may be a bug in their code which is a pre-release version?13:39
_mup_Bug #1166153: The env view slider does not work in YUI >= 3.10 when the thumb is used to zoom in/out  <juju-gui:Triaged> < https://launchpad.net/bugs/1166153 >13:39
gary_posterbac, how bout mark as Incomplete13:40
bacgary_poster: but keep the card in the lane?13:40
gary_posterbac no13:40
gary_posterdelete card13:41
bacgary_poster: +113:41
gary_posterthank you for the gardening13:41
bacugh, if you destroy a service and deploy another juju-core doesn't re-use the existing machine but creates a new one.  rogpeppe did pyjuju work that way?  is there a plan to re-use machines?13:44
gary_posterpyjuju reuses13:45
gary_posterpretty sure13:45
rogpeppebac: reusing is bogus, i'm afraid13:45
rogpeppebac: because a service has a right to assume it's being started on a pristine machine13:45
bacrogpeppe: yuck.13:45
rogpeppebac: well, in a pristine *environment* anyway13:45
rogpeppebac: when we implement containerisation, we'll be able to reuse machines13:46
gary_postercontainerisation will be the default deployment behavior then, is the plan?13:46
rogpeppegary_poster: yes13:46
rogpeppegary_poster: (probably)13:46
gary_posteryeah, assuming it is stable enough etc :-)13:46
rogpeppebac: in the meantime, there's this in the offing: https://codereview.appspot.com/8520043/13:47
gary_posterfrankban https://codereview.appspot.com/8657043/ LGTM13:47
frankbangary_poster: cool, thanks13:48
gary_posterMakyo, could you please send a quick email to hazmat documenting what you think he can show off on the tablet? To re-re-reiterate, "pretty picture only" is completely fine, per re-re-reconfirmation yesterday from Robbie via Deryck13:55
Makyogary_poster, Sure.13:55
gary_posterthank you13:55
rick_h_hatch: once you grab a coffee I wanted to see if you could block out some time to chat url adjusting, dispatch, etc for subapp/app/etc. 13:59
hatchsure thing....10mins/14:00
gary_postervery small review request for small sandbox fix (card is in story A/review): https://codereview.appspot.com/866304314:06
bacgary_poster: i'll do the review but i don't find the card14:07
gary_posterbac thank you. I sometimes have to reload? "Fix sandbox setconfig and speed up test"14:07
gary_posteri stitle on board14:07
benjiI have to resize my root partition, it has reached capacity.  I'll (hopefully) be back soon.14:08
frankbanguihelp: could anyone confirm bug 1167967? thanks, if it's confirmed I will create a card14:20
_mup_Bug #1167967: The GUI hangs when a service is destroyed from the service detail view <juju-gui:Triaged> < https://launchpad.net/bugs/1167967 >14:20
frankban(not related to juju-core)14:20
bacfrankban: yes, i see that problem on uistage14:23
frankbanbac: ok thanks14:23
bacafter destroy it tries to stay on the service view, an error pops up, and the main canvas still shows the service.  after reload the service is gone14:23
frankbanbac: I'll add this info to the bug. card created in High-Priority Maintenance14:25
Makyogary_poster, alright if I land the add relation touch branch?14:26
gary_posterfrankban, y14:26
gary_postersorry Makyo y14:26
Makyogary_poster, thanks14:26
gary_posterbac thanks.  one more review of https://codereview.appspot.com/8663043/, anyone?14:43
gary_posterIt's so small!  so fast!14:46
MakyoI'll take it.14:49
gary_posterty :-)14:49
benjiI now have more (unwanted) experience recovering from grub boot failures.14:55
hatchanyone need any reviews?15:01
teknicohatch, a little birdie (well, not that little actually ;-) ) told me you're a node.js expert and I have a few questions, how about a quick hangout when you have time?15:04
frankbanrogpeppe: I see juju failing to destroy some services, maybe those in an error state, but not sure. Is it a known issue? if not do you want to try to dupe?15:04
rogpeppefrankban: what are the symptoms?15:05
frankbanrogpeppe: the service is still there after a "juju destroy-service" call15:05
rogpeppefrankban: for how long?15:05
rogpeppefrankban: does it remain forever?15:05
frankbanrogpeppe: it's about five minutes now. in the delta, I've seen a "change" entry but not a "remove" entry. everything works well (and fast) for other services.15:06
frankbanrogpeppe: the only difference is that the failing service (buildbot-master) is in an error state. could you try to dupe? just 1) bootstrap an env, 2) deploy buildbot-master, 3)wait till status says 'hook failed: "install" and 4) destroy buildbot-master15:08
rogpeppefrankban: interesting15:08
rogpeppefrankban: will try to dupe15:08
frankbanrogpeppe: btw, the error state could be a coincidence15:08
rogpeppefrankban: maybe...15:08
hatchteknico: lol...damn birds! I'm just eating some breakfast right now :) chat in 5?15:09
teknicohatch, yes, five or so, back soon15:09
gary_posterbcsaller, we need to deliver the charm for IS by tomorrow.  If you need additonal hands on task, please let me know.15:09
gary_posterWe can improve charm next week, but we need functionality tomorrow15:10
bcsallergary_poster: good info, I'll let you know at the meeting one way or the other15:10
gary_postercool thx bcsaller 15:10
* gary_poster takes ubuntu touch tablet to UPS. biab15:11
teknicoback. hatch, when you're ready, no hurry15:23
hatchsure now works15:23
gary_posterjujugui, like yesterday.please update kanban board to reflect reality in next few minutes.  I will review 3 or 4 minutes before call to try to expedite call15:50
hatchgary_poster: can you fill us in on the canonistack stuff? The email didn't include any details :)15:51
gary_posterhatch, oh, right!  I meant to advertise widely.  summary: canonistack is not meant to be reliable.  if you want something reliable, you have their blessing to look elsewhere.  deryck is verifying15:52
gary_posterto make sure they say the same thing twice :-)15:52
gary_posterbut that's what they said15:52
gary_posterso we can switch to ec2 or hp cloud15:52
hatchwell there is a difference between reliable and not even able to start....but ok :) lol15:52
rick_h_yea, thought reliable was 'your instance can go away' not 'no instances for you ever'15:53
hatchbut if we can switch that's also good news15:53
gary_posterrick_h_, hatch, heh, yeah, I said we could work with 95% reliable but not with <10% reliable :-)15:53
hatchjeesh we could even work with 75% reliable :D15:54
gary_posterjujugui call in 2; reviewing kanban15:57
gary_poster3, that is15:57
rick_h_ jovan2 luca__ you guys have a minute? 16:09
jovan2hi rick_h: sure16:10
rick_h_https://plus.google.com/hangouts/_/5fea451cd85e565fc97faa543c4f904f4c39dac0?authuser=0&hl=en jovan2 luca__ 16:10
bachatch: tomorrow's meeting i go to the roof!16:15
hatchhaha is the weather that nice there all the time?16:16
hatchdoes anyone know of any two charms with multiple possible relationships?16:20
hatchjujugui ^16:20
hatchsave me from having to write out a fake charm data16:21
Makyohatch, hadoop to itself.16:22
Makyo8 possible relations.16:22
frankbanhatch: mysql-mediawiki16:22
MakyoYeah, that too, should have a slave and a db.16:22
hatchooo I missed tests for peer relations16:23
hatchthanks Makyo :)16:23
Makyohatch, well, this was one hadoop instance to another, but hey, bonus :)16:23
hatchmysql and mediawiki don't have multiple possible requires=provides interfaces :(16:26
hatchMakyo: can hadoop actually relate to itself? it doesn't share any of the same interface names in it's provides/requires16:27
hatchit has namenode, ganglia, jobtracker vs secondarynamenode, datanode, tasktracker, mapred-namenode16:28
Makyohatch, I see 8 possible relations (well, four, but going both ways) on uistage16:28
hatchso it relates on the interface type not the interface name :/16:29
hatchso now I'm totally confused - why are there multiple ways to specify the same interface?16:35
hatchhadoop for example has secondarynamenode, datanode and mapred-namenode all pointing to dfs16:36
hatchare each one handled by a different hook?16:36
hatchfor example serviceA:foo could relate to serviceB:bar if their interfaces are the same16:42
hatchthe name is irrelevant16:43
gary_posterhatch, bcsaller is the one to talk to.  Everyone else will be guessing at least in part (at least I would be)16:43
bcsallerhatch: the name of what you call the relationship in your charm determines what hook get called. Each relationship has its own hooks.16:44
hatchok so serviceA:bar could relate to serviceB:foo as long as their interfaces are the same16:45
hatcheven though they are named something totally different16:45
bcsallerhatch: yes, its the interface that is important, not what the charm calls it locally. Sorry if that wasn't clear16:45
gary_posterbenji, your current work is, partly or completely, in juju core/go?16:46
benjigary_poster: I think everything I need to do is constrained to the gui16:46
benji(but I have to get juju to run to actually see it work)16:47
hatchctrl+a del16:51
gary_postercool thanks benji.  did you get help from rogpeppe?16:51
benjigary_poster: we're in the middle of diagnosing now; he and mgz seem to have tracked it down, but it doesn't sound like a quick fix16:52
gary_posteruh oh16:52
gary_posterjujugui, heads up: AIUI, Tuesday is the last day to make any changes to Juju Core.  Hopefully we are set *now*, but if we are not, the sooner we discover a bug in Juju Core the better.16:57
rick_h_jovan2: ping, is the font color for the "Recent activity" line in the charm token that orange color used in links?17:25
rick_h_jovan2: it's blocked out as "(Charm activity info?)" and I wondered if that orange was part of noting it, or actual for the text17:26
jovan2rick_h_: no it should not be orange17:26
rick_h_jovan2: the reason I ask is that otherwise the recent activity blends into the description below17:26
rick_h_jovan2: so I was trying to see how to seperate it visually17:26
jovan2rick_h_: greg has done some work on the visual design today and hope to issue it to you tomorrow after review with Alehandra17:27
rick_h_jovan2: ok, I'll leave it as is for now then. http://uploads.mitechie.com/lp/recent_activity_blends.png is what I mean for the record17:28
jovan2rick_h_: yeah I see it's a problem but leave as is for now17:28
rick_h_jovan2: k, thanks17:29
rick_h_doh and he's gone17:48
hatchAssertionError: expected 'No shared interfaces.' to equal 'No shared Interfaces.'18:52
hatchanyone else see a difference there?18:52
rick_h_case matters ftw18:58
hatchI swore I read that line 20 times and didn't see that hah19:01
hatchit could also be that i'm "hate coding" right now because I'm sick of rewriting this relations code :D19:01
hatchgood news is that it gets better every time....well avoiding the fact that it was done to the wrong spec the first two times19:02
rick_h_hatch: got a second for a quick review? https://codereview.appspot.com/8561048/19:08
rick_h_awesome, appreciate it ( jcsackett ^ )19:11
jcsackettrick_h_: oh, thanks.19:13
benjiis anyone else seeing "[14:30:34.429] The connection to wss://ec2-72-44-32-91.compute-1.amazonaws.com:17070/ was interrupted while the page was loading. @ http://localhost:8888/juju-ui/assets/javascripts/reconnecting-websocket.js:80" when connecting to the go environment?19:31
Makyobenji, I got that when I rebootstrapped, and had to go delete the certificates from FF, then re-accept them manually.19:35
benjiunfortunately, I have already gone through that procedure... no wait!  I have rebootstrapped, thanks Makyo19:36
MakyoHmm.  It appears that destroying a service while it's pending removes the service and unit(s), but not the machine(s).19:41
MakyoAnd destroying a service that is stuck in pending or in error fails silently.19:42
MakyoNot having any GUI-core communication issues, though.19:43
gary_postergreat Makyo.  please file those juju core bugs and maybe put them in our tracking?  Also, benji or Makyo, would be cool to see that rebootstrap thing in the blog notes, like Makyo's earlier helpful blog.19:57
gary_posterreview request: small changes, and would particularly like bcsaller to look because I touched the router. https://codereview.appspot.com/868004319:58
bcsallergary_poster: reading it now19:59
gary_posterthank you19:59
MakyoI'll do the blog thing real quick, then bugs.19:59
gary_postercool thanks19:59
bacany one up for a review of destroy_service addition to fakeness? https://codereview.appspot.com/868404320:58
* bac -> dw20:59
* gary_poster will if he has time before dinner21:32
gary_posterMakyo, you still around to review https://codereview.appspot.com/8680043/ by chance?  (hatch, I'd ask you, but I bet you are trying to get your branch wrapped up)22:24
hatchI can do it22:25
MakyoI'm around, too.22:25
hatchok you can do it22:25
gary_poster(bac I decided not to review yours till after you replied to Ben)22:25
gary_posterok thanks, hatch, Makyo :-)22:25
bacgary_poster: ok.  thanks for the review bcsaller22:32

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