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

=== _mup__ is now known as _mup_
=== rogpeppe3 is now known as rogpeppe
bachi luca__, i've got a branch that implements the feedback box.  do you have a moment to have a look?12:20
gary_posterI increased limit to story  1 to 612:23
gary_posterfrankban, thank you for fixing the test problem introduced by the new charm cookie functionality.  AFAICT the most recent failure is different: the appflower search didn't ever actually execute on FF...when it had just worked before.  :-/  are you looking at that as well?  would you like to pass the new error to someone else?12:38
bacgary_poster: the feedback form translates google-supplied content to spanish for me.  The last checkbox of the first question is 'otros' and the button is labeled 'Enviar'.12:39
rick_hgary_poster: is it looking at the live data for that?12:40
rick_hgary_poster: because appflower isn't in the 'interesting' data atm and that will change over time12:40
gary_posterbac, huh.  PR?12:40
bacgary_poster: yeah, but i presume it would do it for any non-english region12:40
gary_posterrick_h, live data, but we do an explicit search (we type it in and press enter)12:41
frankbangary_poster: morning, yes the new error is different, and unexpected. Is there a 30m timeout for jenkins tests?12:41
baci'm always battling google to speak english to me12:41
rick_hgary_poster: ok, charms can be deleted. If you're doing an explicit search I'd suggest using something we use/want to keep around like mongodb or apache or something12:41
gary_posterbac, uh.  ok.  any suggested actions?12:41
gary_posterfrankban, yes there is12:41
bacgary_poster: not at this time12:42
frankbangary_poster: perhaps now the suite takes more and we could just increase it?12:42
gary_posterbac, ok thanks.  I already asked luca to investigate whether we had a surveymonkey account we could use, because the lawyer told me that Google's TOS means that they can use our data too, or at least it is unclear.12:43
bacteknico: does google render any of this in italian for you?  https://docs.google.com/a/canonical.com/forms/d/1Me4macMBX6KydCYyCC6YEjKnpCKakQCYlGlGxIB6Z0w/viewform?pli=112:43
gary_posterrick_h, noted, good pointm thanks.12:43
gary_posterfrankban, ok I increased it to 45.  That looks like a different problem though, yeah?  when I watch the video the search never executes.  it does appear to be intermittent though :-/ : the re-run succeeded.12:45
frankbangary_poster: last jenkins run succeeded in 27 minutes. the previous one failed after 29m. if jenkins quit the driver on timeout a stale element error could be related12:46
gary_posterfrankban, ah ok.  thanks, good possibility.12:46
gary_posterhey world, these branches need reviews:12:47
gary_posterhttps://codereview.appspot.com/10401043/12:47
gary_posterhttps://codereview.appspot.com/10397045/12:47
gary_posterhey luca__ I will triage your bugs from Monday as high.  Huw has a question for you on https://bugs.launchpad.net/juju-gui/+bug/1191868, though12:48
_mup_Bug #1191868: There is no dark grey hover over on alerts <charmbrowser> <juju-gui:New> <https://launchpad.net/bugs/1191868>12:48
gary_postermatsubara, when you get back to tarmac, please increase the tarmac jenkins job timeout from 30 minutes to 45 minutes (assuming you have set one).  How is tarmac going?  I suspect you have a lot of projects going at once12:51
matsubaragary_poster, yes, lots of things to do. I think I should have something to test later on today (in any case I'll do my tests in the jujugui-trunk-merger job to not mess with your workflow)12:55
gary_posterthanks matsubara !12:55
matsubaragary_poster, did you change the job to run under 45 on jenkins?12:55
gary_postermatsubara, I changed the job we use but not the tarmac job (didn't look for that one and don't have bookmark :-P )12:55
matsubaragary_poster, thanks. the other one is disabled so should be ok12:56
gary_postercool thanks matsubara 12:56
matsubaranp12:57
luca__gary_poster: I'll take a look now13:00
gary_posterluca__, thank you.  also, I'm not clear on how drastic the changes will be to the masthead from the current round of redesign.  should I ask for your three bugs to be fixed, because subsequent changes will also be relatively small; or should I wait until the full masthead changes are settled?13:03
teknicobac: in Chromium everything is in English. In Firefox the last checkbox of the first question is named 'Altro', the submit button is labeled 'Invia', and there's more Italian text below the button.13:04
BradCrittendenteknico: that's what i see in chromium (but in spanish).  in safari all is english13:05
=== BradCrittenden is now known as bac
luca__gary_poster: considering that we've all decided to implement the new design I think they should be classed as low priority. Alerts will no longer live in the masthead, for example. However, If Huw has time then it might be good to fix just in case we can't implement the new stuff.13:05
gary_posterluca__, ok.  I'll put them at low priority for now.  I'm optimistic. :-)13:05
gary_poster(that we can implement the new stuff)13:05
gary_posterthanks13:06
rick_horangesquad: appliance guy supposed to arrive in 20min so might be a few min late to standup while i point him at our dryer. FYI13:17
marcoceppiHey guys, do you need to do anything special to deploy juju-gui on 0.7?13:22
gary_postermarcoceppi, pyJuju 0.7?13:23
marcoceppigary_poster: yup13:23
marcoceppisomeone's getting the "connecting to environment" circle screen on 0.7 in #juju13:23
marcoceppiiirc, last time i deployed against pyjuju it just worked(tm)13:24
gary_posternot to our knowledge marcoceppi .  our primary tests are still with that.  I'll head over to #juju13:24
marcoceppigary_poster: thanks!13:24
abentleyjcsackett: ping for standup13:33
jcsackettabentley: ack, was searching for headset.13:33
gary_posterjujugui, is there anyone who is willing to  try to help a smart user on #juju who is trying to diagnose a problem like https://bugs.launchpad.net/juju-gui/+bug/1180095 ?  right now he wants to look at haproxy logs, and I don't kbnow where they are, and don't have time to dig in.13:43
_mup_Bug #1180095: GUI charm may have difficulties working with Juju on MAAS <juju-gui:Invalid> <https://launchpad.net/bugs/1180095>13:43
gary_posterhe is mixired13:43
gary_postermaxired13:44
=== BradCrittenden is now known as bac
bacgary_poster: do you want to land the feedback pointing to google form or wait for resolution on the alternative?13:46
gary_posterbac land it, unless you disagree13:46
bacgary_poster: ok, i wasn't sure how creeped out you were about G's TOS13:47
gary_posterbac, not enough to worry about it ATM.  I think we should start collecting if we can.13:49
bacrt13:49
benjirick_h: I could use some YUI help if you have a minute13:54
rick_hbenji: definitely13:59
rick_hbenji: guichat?13:59
benjirick_h: +113:59
rick_hbenji: taken14:00
abentleyjcsackett: It looks like The Verge lets you share articles on G+.  The first click is a +1, and then when you hover, you get a box to share the article.  http://www.theverge.com/2013/6/19/4443588/pindrop-acoustic-analysis-fights-phone-fraud-social-engineering14:03
jcsackettabentley: thanks!14:04
abentleynp14:05
jcsackettabentley: so, it looks like what that's doing is +1 the g+ post verge makes for the article.14:09
abentleyjcsackett: Oh, so not so applicable.14:10
abentleyjcsackett: Sorry.14:10
jcsackettabentley: yeah.14:10
jcsackettabentley: no, still good to see. it's possible that's a road we'll want to go down.14:10
jcsacketti mean, it's not one *i* want to go down... :-P14:11
abentleyjcsackett: I'm envisioning a bot that posts a new story every time a charm is updated...14:11
jcsackettabentley: we can just bolt that into ingest, right? ;-P14:12
gary_poster:-)14:12
* abentley really doesn't get why G+ won't release a write API.14:12
jcastrobecause people will spam it14:13
jcastroit'll end up worthless like twitter14:13
rick_hjcastro: abentley might know a good way to reset the MP to a non-mimms state. 14:21
rick_habentley: jcastro was just asking about the bug about the missing proposals ^14:21
rick_habentley: I showed him the bug with your comment. Maybe you can help him un-fubar the missing ones?14:21
abentleyjcastro: Just request ~charmers as a reviewer.14:21
maarten__Hi. I installed ppa:juju/devel and the arrows between the services keep on changing positions...14:22
gary_postermaarten__, hi.  I'm not clear on what you are reporting, I'm afraid.  You installed pyJuju from the PPA, and you installed the GUI charm, I'm guessing, and then what services/arrows are you referring to?14:29
maarten__I installed gojuju and when I use the juju gui, the position of the arrows and the services changes and they are not connected.14:30
gary_postermaarten__, thank you for the report.  Is this something you can screenshot?14:32
maarten__gary_poster,just send you the screenshot14:38
gary_postermaarten__, thanks for screenshot.  we have not seen this before.  automatic service position changes are supposed to happen, but only if someone else connected to the same environment moves the service (positioning is shared).  In that case, of course, relations should maintain positions with their associated services.  Does anyone else have access to the GUI who might be moving service positions?14:45
gary_postermaarten__, sent you a reply and filed bug 1192596.  Moving services is expected in a shared environment, & a big surprise otherwise.15:00
_mup_Bug #1192596: Relation endpoints can get misaligned (after annotation update changes service positions?) <juju-gui:Triaged> <https://launchpad.net/bugs/1192596>15:00
gary_posterMakyo, if you have availability could you add this bug to maintenance in kanban, and if you have further availability could yo uinvestigate?15:01
Makyogary_poster, sure.15:01
gary_posterthank you Makyo 15:02
gary_posterjujgui, after daily call I would like to talk to people about some "bundle" design decisions.  If you are at all interested, please take a look at the definitions and the Q & A  of https://docs.google.com/a/canonical.com/document/d/1gDspsjnYf1NqDvgyTqRCxtek5JWULMwSfTLwiZn12Nk/edit#15:04
gary_posterbefore the meeting, as a prep15:04
gary_postercore question: am I right that using/incproprating the juju-deployer is the right decision?15:05
gary_posterincorporating15:05
gary_posterrick_h, hatch, if you want to attend shared web meeting https://plus.google.com/hangouts/_/cd640721e4194224f0768c6ca181bb8ba92e7b1a15:08
bacluca__: i'd really like you to look at my implementation of the feedback box.  the review is at https://codereview.appspot.com/1040704415:13
MakyoWho is doing "find all tests that change URL and stop them from doing so" in maintenance?15:14
Makyojujugui ^^^15:14
teknicoMakyo: oops, it's me15:14
rick_hMakyo: teknico had a branch for that15:14
Makyoteknico, I'll put your face on it.15:14
Makyoteknico, thanks, too.  That's annoying :)15:14
teknicotoo late :-)15:14
teknico(meaning, I just did it)15:15
teknicosorry about that15:15
benjirick_h: I can't figure out how to get the charm tokens to be draggable out of the side bar.15:15
rick_hbenji: otp, back in a sec15:24
jcsackettluca__: got a bit to chat on g+?15:30
rick_hjcsackett: he's otp with us atm15:30
luca__jcsackett: currently in a call :)15:30
abentleyorangesquad: Could you please review https://code.launchpad.net/~abentley/charmworld/ignore-oneiric/+merge/170386 ?15:42
jcsackettabentley: looking15:44
teknicohatch: you around?15:45
hatchsort of :)15:46
hatchwhat's up?15:46
teknicoI have one single test that's not running anymore, and I cannot locate it: lp:~teknico/juju-gui/test-interdep-cleanup15:46
rick_hbenji: so I think you can only drag within the 'container'. I wonder if you make the container the <body> if it might work?15:47
rick_hbenji: or maybe there's some tweak to that we can ask around in #yui or #yui-hatch :)15:48
benjirick_h: nope, I tried that 15:48
teknicohatch: there should be 950 tests but only 949 run in make test-server, and make test-prod does not output the number of tests at the end, as it does when some of them did not run15:48
rick_hbenji: ok /me goes to check css overflow rules on that sidebar div.15:48
benjirick_h: I removed the overflows and it had no effect15:49
rick_hbenji: ok, one step ahead of me then. 15:49
benji:)15:49
hatchteknico it's building15:49
teknicohatch: thank you15:49
rick_hbenji: yea, then that's the only top of my head stuff I can think of. I'd hit up #yui and bug hatch for his previous DnD experience. I've not used it enough to say anything else without getting really into it15:49
Makyojujugui call in 10, kanban now.15:50
* benji reluctantly heads over to #yui.15:50
hatchbenji: do you have 'constrain2node' set?15:51
benjihatch: not knowingly15:52
hatchok you'd know heh15:52
hatchI've never seen it15:53
hatchit should set it to position absolute and then you're gtg15:53
Makyojujugui call in 215:58
rick_hbenji: let me know when the call is up. I have one more idea to check16:00
hatchteknico: I thought that your branch stopped these GET requests?16:00
teknicohatch: which ones?16:01
hatchwell when you run the tests in the console there is a TON of GET requests to get the json data16:01
hatchApplication start up sets the window.flags data:  expected undefined to deeply equal {}16:02
hatch^ teknico16:02
hatchthere is your failure16:02
jcsackettabentley: r=me.16:02
rick_hbenji: the css rule on .bws-content is interfering with it. That position absolute there in the rule for: #subapp-browser #bws-sidebar .bws-content, #subapp-browser .bws-view-data, #subapp-browser #bws-sidebar, #subapp-browser #bws-fullscreen16:05
hatchlol that's a lot of nested id's16:05
rick_hhatch: well it's several paris of ids :P16:06
rick_hpairs16:06
hatchhaha yeah16:06
hatchthey say pair id's are bad because of the css indexing16:06
teknicohatch: I see it now, thanks16:06
rick_hbenji: so if I uncheck that absolute on the .bws-content div, and manually set a .yui3-charmtoken to position: absolute with left/top of 200px it moves and works just peachy16:07
hatchare you dragging a proxy or the actual element?16:07
benjirick_h: cool, I'll try that in a minute16:07
hatchimho you should drag a proxy16:07
hatchbut I suppose that's implementers choice :)16:09
rick_hhatch: yea, but first stab is an element. 16:11
hatchyeah I suppose16:12
rick_hso with that, lunchables need to be fetched16:12
hatchI wonder if the proxy will have the same issue16:12
hatchI don't really know how the proxies work under the hood16:12
jcsackettrick_h: you want/can you jump in on the deployer/bundles meeting in guichat? i have to run shortly.16:16
jcsackettrick_h: nm. 16:36
rick_hjcsackett: k16:40
gary_posterluca__, guichat really quick?16:48
luca__gary_poster: sure16:50
gary_posterthx16:50
benjirick_h: I was (almost) able to reproduce your results following the procedure you gave, but it seems that it relies on having the original css for inital layout and then swichting those styles. (when applied from the begining not only is the sidebar all a jumble, but the dragged item is still constrained to the container)17:11
rick_hbenji: ugh and ugh. 17:13
rick_hbenji: I assumed that messing with that css would fubar up some things and send us chasing for a different way to do it. I thought/hoped that the issue was one of pure css vs JS functionality though17:14
benjiI'll look at it harder after lunch.17:14
rick_hbenji: k, I'm trying to get this branch up for review and maybe can see if I can help some. Let me know17:15
gary_posterteknico, are you finishing up your maintenance branch before you leave?17:17
gary_posterMakyo, it turns out that bug from maarten__ has nothing to do with sharing.  The GUI release is broken in Firefox :-(17:17
teknicogary_poster: much as I'm trying, it looks like I will not :-/17:17
gary_posterteknico, ok, I will mark this other issue as critical so we can break WIP17:18
Makyogary_poster, Will investigate trunk.17:18
gary_posterMakyo, to dupe, simply start uistage on Firefox, add two services, and move them around and try to add relations17:19
gary_posterthansk Makyo.  We should make a 0.7.1 afterwards17:19
Makyogary_poster, check17:19
gary_postermatsubara, is there still a chance that we get you soon for regular weekly qa, or is that no longer an option?17:23
matsubaragary_poster, should be an option, yes. How about a round of ET this Friday?17:23
gary_postermatsubara, sounds great.  If possible, please use Firefox, Chrom[e/ium], and IE 10.17:24
gary_posterthank you!17:24
matsubaraok17:24
teknicogary_poster: I caught the freakin' pest :-) so I'll be able to at least propose the branch shortly17:33
rick_horangesquad along with hatch, Makyo, or others interested: code review request please on related charm display: https://codereview.appspot.com/1041604417:35
teknicoeveryone interested in tests (that is, everyone ;-) ) is welcome to have a look at https://codereview.appspot.com/10383047 Thank you!17:40
abentleyjcsackett: What was scan_repo for / is bug #1111708 still relevant?17:51
_mup_Bug #1111708: scan_repo and reindex need to have their own jobs/scripts <ingest> <tech-debt> <charmworld:Triaged> <https://launchpad.net/bugs/1111708>17:51
gary_posterteknico, :-) great17:55
bacteknico: looking18:03
gary_posterme too18:03
bacteknico: and while you wait, i've got one that needs review.  :)  no squid pro quo, but thought you might be interested18:04
teknicobac: I'm not exactly waiting :-) I'm past my EOD and having dinner18:05
bacah, perhaps you'll have a break between your main and dessert!  :)18:06
bacjk, of course18:06
teknicobac: of course, but I did it anyway :-)18:08
* gary_poster tried using Gmail over web interface for half a day. Not my thing. :-) Back to buggy, clunky thunderbird18:50
abentleyjcsackett: Here's another site that seems to allow sharing a URL on G+: http://news.nationalpost.com/2013/06/19/bob-rae-resigning-as-an-mp/18:57
bacgary_poster: in kanban there is an lkk@admin.canonical.com marked as IS.  is that a real person?19:17
gary_posterbac, fascinating question.19:17
bache and/or she is breaking lp2kanban19:18
gary_posterno idea19:18
bacit should be fixed to continue gracefully ignoring people with no LP id19:18
baci've never seen the admin subdomain19:18
gary_posterbac, I could try to see if we can disable this account...but I suspect they are supposed to be there.  would you like me to investigate, or do you think we should just plan to fix lp2kanban, or...?19:19
baci think it'll be an easy fix19:20
gary_postercool19:20
gary_posterthx19:20
bacs/assert/continue19:20
gary_poster:-)19:20
bacgary_poster: my branch got two reviews but luca indicated he had some changes that he'll describe tomorrow, so the branch is stalled19:58
gary_posterok bac.  thanks for heads up. on call19:59
jcsackettabentley: that is awesome! that totally works. and with that to work from, finally able to find documentation.19:59
abentleyjcsackett: Great!20:00
jcsackettabentley: responding to your much earlier question: scan_repo was used to re do the scan job on things already stored in the DB, iirc. as it's not come up, i don't know that it's something we still need.20:11
abentleyjcsackett: I'm inclined to think we don't need either scan_repo or reindex.20:12
jcsackettabentley: i would be inclined to agree.20:12
bachi Makyo, do you have any info on the card 'juju go sandbox: next methods'?20:32
Makyobac, not immediately.  I suppose I'd suggest expose/unexpose, or maybe destroy?20:33
bacMakyo: so i guess that card should spawn another after picking one of those...20:34
Makyobac, yeah, until go-sandbox has at least everything py-sandbox has. 20:34
bacif we do have a sprint in raleigh at least we have a nice airport now. http://www.cnn.com/2013/06/07/travel/airport-architecture-paul-goldberger/index.html   (photos 9 & 10)21:19
gary_posterhey, look at us with our #9 in the world!21:29
* gary_poster has to run21:29
gary_posterhave a great evening21:29
hatchevening23:04
rick_hparty23:32
rick_hhatch: you see the new pr release from yui. good looking stuff23:33
hatchyeah it's looking awesome23:33
hatchwe REALLY need to upgrade heh23:33
rick_h:)23:33
bcsallerwhy can't we?23:33
hatchit's finally coming into it's own23:33
hatchbcsaller: too hard to change those 4 characters and re-make :P23:34
rick_hlol23:34
rick_hcan we hit CI without it landing in trunk?23:34
hatchnot until tarmac is up and running23:34
rick_hideally we'd just upgrade and push it up to CI and see what it said. If all's well then land it23:34
hatchbuut you could pull down your branch and run from your machine23:34
hatchso yes23:34
hatchsortof23:34
hatchhonestly I think the only reason we haven't done it is because noone has taken the time23:36

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