/srv/irclogs.ubuntu.com/2014/04/21/#juju-gui.txt

rick_h_morning party people11:35
rick_h_well, non-holiday'ing people11:35
rick_h_jujugui off to take wife to work (her car is kaput) back in a few minutes12:41
bacack12:41
* rick_h_ is back13:00
jcsackettmorning all.13:01
bachi jcsackett13:01
bacwelcome13:01
rick_h_woo! and today the team is back to full strength for the first time since Feb. Well, aside from everyone being out this week13:01
bacrick_h_: don't you have a fleet of fancy cars?  "kaput" seems to happen a lot.13:02
bacor does michigan do that to them?13:02
rick_h_bac: oh really? the wife's subaru is leaking something. So we're just not driving it until we take it in tomorrow13:02
jcsackettrick_h_: i have a subaru. leaking something is its perpetual state.13:02
rick_h_they don't really go kaput, but we do take them in for regular maintanece, new tires, winter/summer tire swap13:02
bacah, gotcha.13:03
rick_h_so I guess it might seem that way, but actually we're pretty trouble free13:03
rick_h_oh her battery died this winter, but that's just fun13:03
bacour new-to-us car was kaputting but a new battery this weekend should do the trick13:03
bacmost expensive cost-to-volume car battery i've ever bought13:03
jcsackettrick_h_: you free to chat this AM? about the card i pinged about friday or other things?13:04
rick_h_jcsackett: yep, in a few min13:05
jcsackettcool.13:05
bachey jcsackett could you look at this charmworld exodus branch when you get a chance? https://codereview.appspot.com/89770044/13:28
jcsackettbac: looking now.13:29
rick_h_jcsackett: k, free link me up to a hangout when you're ready13:30
jcsackettrick_h_: ok.13:30
jcsackettbac: so, i'll actually review right after chatting with rick_h_ 13:31
bacok13:31
jcsackettrick_h_: link is https://plus.google.com/hangouts/_/76cpjte2d4j5v28msijuvbb2tg?hl=en13:32
rick_h_jcsackett: "this party is over"13:32
jcsacketti freaking hate google.13:32
jcsackettrick_h_: you're on plus now?13:32
rick_h_jcsackett: https://plus.google.com/hangouts/_/7ecpi051hcetihavdvas78pick?hl=en13:33
jcsackettbac: ok, looking at your branch now.13:53
jcsackettbac: did you sort out logging weirdness?13:53
bacno, not yet13:54
baci improved it in another branch but we still just pretend to have hiearchical logs13:54
jcsackettbac: ok. i saw use of the charm.NAME system and dared to hope you had figured it out. :p13:55
bacjcsackett: no, i just perpetuating the fraud13:55
bacs/i j/i am j/13:56
jcsackettbac: comments on the diff.14:09
rick_h_jcsackett: some fyi https://github.com/juju/juju-gui/blob/develop/HACKING.rst#typical-github-workflow 14:10
rick_h_jcsackett: and the section under there for helpful aliases/etc14:10
jcsackettrick_h_: thanks--i read the workflow bit earlier, i'll check out the aliases.14:11
rick_h_jcsackett: cool14:11
jcsackettoh my lord. i've somehow made xchat notify me on all messages in any channel. this cannot stand.14:39
bacjcsackett: changes made to https://codereview.appspot.com/89770044/14:49
Makyojujugui call in 1014:50
rick_h_hey, one minute to go still 14:50
MakyoI'm going by my phone alarm, so t-mo says it's time!14:52
jcsackettbac: looking.14:54
jcsackettbac: LGTM. Thanks!14:55
Makyojujugui call in 214:58
MakyoToday's house: http://www.zillow.com/homedetails/6216-Becker-Ln-Loveland-CO-80538/13881778_zpid/ Excited, but worried about the lack of bathroom pics.15:11
jcsackettnice.15:12
rick_h_yummy wood floors15:12
rick_h_ok, the moon pic is a bit of a sales job15:13
MakyoTotally, haha.15:15
MakyoDitto the elk.  That's across the road15:15
jcsackettthat kitchen, though. that's awesome.15:16
rick_h_kadams54: is this rebased on develop?15:16
MakyoRight? Granite countertops with glass-tile backsplash.15:16
kadams54rick_h_: Yes, rebased on develop as of Friday morning.15:17
kadams54Give me a moment and I'll make sure it's got the absolute latest.15:17
rick_h_kadams54: all good, monday readings I confused myself15:17
jcsackettMakyo: if you end up w/ that kitchen, ima be jealous.15:17
MakyoI'm already excited. Only problem is going to be brewing, that'll have to be all outside, since that stovetop won't work for it.15:18
kadams54rick_h_: Confirmed: my deployed-unit-tokens is rebased on the latest from develop.15:18
rick_h_kadams54: k, feedback done15:19
rick_h_jujugui quick mechanical review please https://github.com/juju/juju-gui/pull/24715:39
bacrick_h_: ok15:39
rick_h_thanks bac15:40
bacrick_h_: done15:44
bacrick_h_: that's a boar15:44
rick_h_thanks bac, it's too monday to trust my eyeballs15:45
bacas in boaring15:45
rick_h_oh lol, just seeing the comment15:45
hatchrick_h_ your flags branch is missing all of the css flag changes15:54
* rick_h_ goes to look15:57
rick_h_:qa16:00
rick_h_bah16:00
hatchsee what I mean?16:01
rick_h_hatch: yea, there was one that was state. Updated 16:02
rick_h_the others are all mv and that's the same16:02
rick_h_and no ecs related ones I see16:02
hatchoh right - I spent some time merging the li and state ones before16:02
rick_h_thanks for the catch16:02
hatchso does your branch remove ecs entirely?16:03
rick_h_hatch: yes, it's all mv16:03
rick_h_hatch: they need each other so making it all part of that work. qa and such should all be done together16:04
hatchyeah ok cool, just wondering so when I load it up in the future I know what flags to use16:04
rick_h_yep, il/mv and by thurs only mv is the plan16:04
hatchwas `state` too long to type? lol16:07
Makyo*weekly iteration of  "I hate promises."*16:07
hatchhaha Makyo we still have promises? 16:07
MakyoYep.16:07
hatchlemme guess.....syntax error without any information as to where the error is occuring? 16:08
hatchoccurring 16:08
MakyoClose.  Env trying to access non-existent property with no indication on where that call to the env is being made.16:09
rick_h_hatch: no, il was what I told everyone to use. It was what design is testing as, what they've shared with Mark S during demos, etc16:09
rick_h_hatch: so state was just not what was agreed among a bunch of people and made up and made state stuff hidden during qa/testing of other branches of code16:10
kadams54guihelp: quick YUI question: Node.append() seems to only take a single element (string, Node, or HTMLElement); what if you have an array of Nodes (Node[]) that you want to append?16:10
hatchMakyo ugh that's even worse....YUI will -not- be adding in anything for proper stack traces either unfortunately - they are keeping it exactly to A+ :(16:11
rick_h_I thought append would take a nodelist as well as a node?16:11
hatchkadams54 it'll take a string16:11
hatchif it's a nodelist then it'll loop over it16:11
hatchnot very performant16:11
kadams54Not a NodeList16:11
rick_h_yea, the point is to make it fast16:11
kadams54Though I suppose it could be16:11
hatchkadams54 can you create a string and append the string?16:12
kadams54Yeah, that was going to be my ugly hack fallback16:12
rick_h_kadams54: what about making the <ul> a var form Y.Node.create and then in each loop iteration add to that node16:12
kadams54nodes.join('')16:12
rick_h_kadams54: and at the end put that UL into the dom16:12
rick_h_kadams54: so you're just appending one node to a container in the dom, and building the whole UL programatically outside of the DOM16:12
kadams54Ah yeah, that would work16:13
hatchjust an FYI datatable does everything as string templates unless it's forced not too which gives it like a 75% boost in performance16:13
kadams54*sob*16:15
rick_h_kadams54: ?16:15
kadams54It just seems wrong to have this tree API (DOM) but instead fallback to string manipulations16:16
rick_h_ah, gotcha16:16
kadams54It makes me sad16:16
* rick_h_ was worried you hit another issue 16:16
rick_h_meh, react shows you don't have to go to pure strings for performance. 16:17
* rick_h_ wants YUI widgets on top of a react backend badly16:18
hatchI can't really comment, I don't know the context of what you're doing16:18
rick_h_hatch: generating tokens (Views) to put into a div in a loop16:19
hatchreact works a little differently, if it's calculated diff required it to generate a ton of elements in the DOM its performance would likely also suffer16:19
hatchohh, is this regarding generating containers for the token views?16:20
rick_h_right, but the point is that the calculations happen outside of in the DOM tree, but in a pure memory tree and thus is fast16:20
rick_h_hatch: rgr16:20
rick_h_hatch: well, kind of, but that area of the code yet16:20
hatchoh....well I gave Huw the information to make his branch work properly16:20
hatchto the 'convention' that is16:20
hatchhttps://github.com/juju/juju-gui/pull/233/files#diff-947e688a9f9e23383a48af542be63bbfR14216:21
hatch^ kadams54 16:21
hatchand then you pass the 'parent' node into the render method as a param16:21
rick_h_hatch: but kadams54 and I disagree withthe container being a LI16:22
rick_h_so you test that with a container that's not a normal div? That seems broken16:22
hatchI'm not sure I understand?16:22
hatcheach token isn't a li>16:22
rick_h_it is in the use case of the panel, but if that token were reused anywhere else? Or just want to create atest of the token. Why an IL is a specific detail in one use case and we disagee it should be a View's container template. 16:23
hatchhttps://gist.github.com/hatched/533588638633a831a29116:25
kadams54The goal is to keep the Token object as stupid as possible16:25
jcsackettjujugui, i am routinely having the gui hang at the loading screen. i've got the gui running in an LXC, and accessing via browser on my main machine. any gotchas y'all can think of?16:25
hatchjcsackett open the network tab16:25
rick_h_jcsackett: check the console?16:25
hatchis it hanging on loading YUI16:25
hatch?16:25
rick_h_jcsackett: if it's handing at loading it should be dumping something out to you16:25
rick_h_hanging that is16:26
kadams54So making the token handle a "containerTemplate" seems like the wrong place for that logic.16:26
kadams54I'd rather the code using the token (MachineViewPanelView in this case) handle that.16:26
jcsacketti have no errors in console, and everything is actually pulled down. just sitting at the spinner.16:26
hatchjcsackett odd.....usually it hangs on loading YUI 16:27
hatchjcsackett maybe `make clean-all && make devel` 16:27
hatchkadams54 I'm skeptical of making it 'too dumb' imho16:28
hatchnow that means that it requires extra setup for testing and whatnot 16:28
hatchespecially if there isn't a use case that you can think of where it's container would be different16:28
rick_h_hatch: disagree, it just needs to be rendered. Put it in a container div. 16:28
rick_h_what testing bootstrapping does it add to?16:29
kadams54Yeah, seems like dumber object == less testing bootstrap16:29
hatchthen why supply a container at all?16:29
hatchjust let it create it's own div and be done with it16:29
hatchI was under the impression that the <li> was an important element to the view16:30
jcsacketthrm. make clean-all etc seems to have fixed it. i am suspicious of my setup now.16:30
jcsackettis lxc still a way people work on this? i note vagrant is now documented.16:30
hatchjcsackett it's ok, it happens16:30
hatchjcsackett a bunch of use use vagrant16:30
rick_h_to the panel and only for css purposes.16:30
rick_h_jcsackett: yes, I work in lxc16:30
jcsackettcool.16:31
hatchjcsackett the makefile is kind of like a magic script16:31
hatchyou can't read it16:32
hatchyou just hope it works16:32
hatch:P16:32
hatchlol16:32
rick_h_because merge-files is so much better :P16:32
rick_h_magic that sucks to no end when it breaks. 16:32
hatchoh yeah but I've got that figured out now16:32
* rick_h_ hates looking up how to debugger into a nodejs script every time that damn file has to be touched16:32
hatchI know why merge-files breaks16:32
hatchlol16:32
kadams54OK, heading out for my dr appt16:33
hatchrick_h_ maybe one of the next week tasks can be to rewrite our makefile in grunt :)16:33
rick_h_hatch: heh, I just got 10 people to buy the makefile book after a talk on grunt at pycon because I ranted on how it's just reinventing the wheel poorly16:34
hatchclearly they haven't seen our makefile....16:34
hatch:P16:34
rick_h_oh if only other people didn't write things we don't ourselves agree with or follow...oh wait. :P16:35
hatchgrunt, gulp, etc etc etc are 'task runners' 16:35
rick_h_funny that, Make is a build tool. I don't need tasks run, I need things built16:35
rick_h_if I want to go ssh into a box and run a command I'll use juju actions (coming to an environment near you)16:35
hatchright, but it's impossible to follow at a glance16:35
rick_h_hatch: so we have a complicated one, sorry. https://github.com/bookieio/breadability/blob/master/Makefile start there :)16:36
hatchthat's the exact issue....as the complexity increases it becomes less and less manageable 16:37
hatchit's like writing a js app in a single jquery file16:37
hatch:)16:37
rick_h_anyway, refactoring of the makefile is welcome. grunt/etc aren't magic bullets of complexity management either and it's another tool/etc we don't need. Everyone (almost) in the company speaks makefile and it's the standard and we've not hit anything it's not capable of doing. 16:38
rick_h_education + refactoring > buying into another tool of doom 16:38
hatchI'm not sure it can be refactored to be legible....hope I can be proved wrong :)16:38
hatchoh....btw my containerTemplate proposal would still allow you to render the view in tests as a div and in the app as a li16:39
rick_h_hatch: right, by passing the arg in to override16:39
rick_h_hatch: understand16:39
hatchmaybe what you're looking for then is to have the view generate a string which then gets appended outside of the view16:40
hatchso the view is kind of like a template generator16:40
hatchoh that won't work16:40
hatchwell, it would16:40
rick_h_hatch: yea, I've done it both ways before. Where render returned the html, and where you specify the 'renderTo', and it just renders itself16:40
hatchbut probably wouldn't be worth the extra work16:41
hatchyeah I've never liked that approach, it's confusing16:41
hatchbetter to stick to conventions whenever possible16:41
rick_h_exactly, let's get this landed and move forward. We can chat about it at vegas if required. 16:41
rick_h_but I think the root of it is that kadams54 and I get an icky feeling when a View is anything but a div. 16:41
hatchpoor poor back end devs16:42
hatch:P16:42
rick_h_ok, time to get some real work done. 16:42
hatchwe should maybe re-evaluate huws branch then16:43
hatchto align the two16:43
rick_h_hatch: +116:43
hatchMakyo as a general rule do you use pointers in Go or assign return values to variables instead? I prefer the latter17:04
Makyohatch, the latter, I believe, and you can return multiple values, so it's usually retVal, err = fn(params)17:06
hatchMakyo yeah that's what I prefer too but I've seen the other way in others code and I can't see why they would do it that way17:11
Makyohatch, just different backgrounds, probably. That's more C/C++ish17:11
MakyoTook a lot of getting used to working with Qt/.17:11
hatchyeah that's what I was thinking17:11
hatchI remember doing that back in the C++ days17:12
rick_h_jujugui quick review + QA please. One line fix + bunch of lines of test. https://github.com/juju/juju-gui/pull/248 17:13
rick_h_hatch: what's viewNavigate now?17:38
hatchchangeState17:38
rick_h_hatch: ty17:38
jcsackettah, changeState. my good friend.17:39
hatchnp, yeah, everywhere that fires viewNavigate atm should fire a changeState17:41
hatchthe handler is also in the same place in browser.js17:41
jcsacketthatch, do you have a moment to chat, or are you actually taking today off but hanging in the channel?17:45
hatchjcsackett I'm working on some Go stuff but I can context switch for you :)17:46
hatchfire me a hangout link17:46
rick_h_me too, want to bug hatch as well :)17:47
* hatch knew he shoud have started his house work already17:47
jcsackettso...the section of plus that lets me fire up a hangout isn't loading.17:47
jcsackettrick_h_, you want to try?17:47
rick_h_jcsackett: yep, inviting17:48
rick_h_hatch: do we have anything that tests the dispatching in browser.js for the new state stuff?18:26
hatchrick_h_ I think the charmbrowser and inspector methods are unit tested but the state instantiation isn't 18:28
rick_h_hatch: yea, I've added the dispatch stuff to the machine method in the browser.js but not seeing a good place to test it. Will tack it into the existing browser_app tests for the moment.18:29
hatchit's probably stable enough now that they could be added18:29
hatchthere is a dispatcher describe in the browser.js test file18:30
rick_h_ah, describe('state dispatchers', looks like what I want18:31
hatchtests so need refactoring heh18:32
rick_h_yea18:32
hatchya know, I'm not convinced the go ecosystem is ready to be a webserver18:37
hatchI mean, sure it works and all that, but node/python have so many more libs18:37
rick_h_jujugui looking for two reviews now please https://github.com/juju/juju-gui/pull/249 and https://github.com/juju/juju-gui/pull/24818:44
rick_h_nothing huge, QA required18:45
rick_h_kadams54: make sure to rebase, I had the one landing that changed the flag names this morning18:45
rick_h_kadams54: was going to ping but you were afk for a chunk there18:45
kadams54Yeah, will do18:45
rick_h_or af-irc18:45
rick_h_kadams54: if you get a second before your next branch a couple of reviews ^ appreciated18:49
kadams54Sure18:49
kadams54I'll dig into both18:49
rick_h_ty much, hopefully small/quick18:49
kadams54rick_h_: both look good and ready to merge.18:56
rick_h_kadams54: ty much sir18:57
rick_h_woot! landing 3 branches in one day. Take that management school! 18:59
* rick_h_ stops taking another card because it's meeting time coming, doh!18:59
Makyojujugui time to split my day. Will be back and have this card proposed later tonight.19:12
MakyoPutting laptop away because contractor's here for a bit longer.  Email or phone is best.19:12
rick_h_Makyo: rgr, I'll be around post 5:30 your time if you need anything (think you're 3hr behind)19:12
MakyoIt's 1:13 now.19:13
rick_h_Makyo: ok, 6:30 your time then19:13
MakyoWill be back 5:30 my time if commute goes well.19:13
MakyoCheers, thanks rick_h_ 19:13
BradCrittendenhey rick_h_ i'm looking at bug 1309678 -- i'm not sure why juju-gui even cares about the control-bucket.  you have any ideas?19:33
_mup_Bug #1309678: a value is required for the control bucket field <juju-quickstart:In Progress by bac> <https://launchpad.net/bugs/1309678>19:33
=== BradCrittenden is now known as bac
bacrick_h_: it could be that previously juju would fail if it weren't there but now it generates one19:33
bacrick_h_: oops s/juju-gui/juju-quickstart/ above19:34
jcastrodoes anyone have the gui working in trusty/power?19:43
jcastrohttp://pastebin.ubuntu.com/7301778/19:43
jcastroI get this still19:43
jcastrorick_h_, ping19:51
rick_h_jcastro: this is the trusty charm?19:58
rick_h_jcastro: can you verify that those packages are packaged for power then? Do we have a bad dep?19:58
jcastrono19:59
jcastrobut it worked for matt on another machine19:59
jcastroodd, investigating19:59
rick_h_jcastro: is there someone that has access to a power machien to test the apt-get command? apt-get install libapt-pkg-dev python-apt python-launchpadlib python-tempita python-yaml19:59
jcastrosec, I am pulling the charm pristine20:01
rick_h_jcastro: k20:01
jcastrosorry it's a long pull20:02
rick_h_jcastro: np, standing by20:02
jcastrorick_h_, is there a trick for a lightweight checkout?20:03
rick_h_jcastro: looking if we have a release of the charm sans history20:04
jcastrorick_h_, can I clone from github directly? 20:05
jcastrois that fine?20:05
rick_h_jcastro: the charm isn't on github, only the gui itself20:06
rick_h_jcastro: the charm includes the gui release tarball in it20:06
rick_h_jcastro: so it works 100% offline20:06
jcastroack20:06
jcastrowhere's the charm20:06
jcastrolp:juju-gui?20:06
jcastroI need to grab it locally and then tarball it to the machine because it's timing out20:06
rick_h_https://code.launchpad.net/~juju-gui-charmers/charms/trusty/juju-gui/trunk20:07
jcsackettrick_h_, i've started creating a test suite for service-inspector b/c it doesn't have one at all. is this actually worth while, or do we assume service inspector is safely tested implicitly?20:10
jcsackettasking b/c i realize i'm delaying just landing a one line fix for not much added value.20:11
jcsacketts/landing/reviewing and landing/20:11
rick_h_jcsackett: huh? it should have some tests. Maybe looking in the wrong place?20:11
rick_h_jcsackett: it's a viewlet and part of the viewlet-mananger20:11
rick_h_jcsackett: for your issue of hte close button I'd not worry about it atm. As we've said the goal is to remove that all together by thurs20:12
rick_h_if you can add it cheaply awesome, otherwise let's deprecate it asap20:12
jcsackettrick_h_, it's increasingly obvious that it's not super cheap. i'll get the fix up for review.20:12
rick_h_jcsackett: rgr20:12
jcsackettrick_h_, also, the viewlet manager test stuff has no actual mention of the service-inspector, as it stands.20:13
rick_h_jcsackett: k, I know there's some test gaps (we've got a sprint goal to get a code coverage tool into place) but I know there are some tests. 20:14
jcsackettrick_h_, like i said, it may well be tested implicitly. and there are lots of tests, so i'll be the first to admit i could've missed them, but ack shows no use of any of the module names, objects, etc anywhere in tests.20:15
jcsackettanyway, something for another day.20:15
rick_h_jcsackett: k, yea can look around later. 20:15
rick_h_jcastro: any luck? Want me to build a tarball?20:15
rick_h_or zip I guess20:15
jcastroalmost ready20:15
rick_h_jcastro: k20:15
jcastrorick_h_, deploying now20:16
* rick_h_ crosses fingers I guess20:17
jcastro  agent-state-info: 'hook failed: "install"' again20:17
rick_h_k, so question is if it's the same point and if so what's the actual failure in that apt-get command20:17
jcastroyeah20:17
jcastroso the packages are all there on ppc20:17
jcastrorick_h_, I think I found the issue20:21
rick_h_jcastro: what can we do to help?20:21
jcastroI think you need to do an apt-get update first20:22
jcastrowhen I ran it it 404'ed all the debs20:22
rick_h_jcastro: ah, ok20:22
* rick_h_ checks out the code for the charm20:22
rick_h_doesn't juju do that when you bring up a new machine?20:22
jcastrono idea20:23
jcastrohey how do I do an apt-get update in python?20:23
jcastroso I can just fix it for the demo?20:23
rick_h_sec, looking at the code20:23
rick_h_jcastro: I think you can just add 20:25
rick_h_run('apt-get', 'update') 20:25
rick_h_before the install in line 36ish20:25
jcastrorick_h_, I can confirm that fixes the issue!20:31
rick_h_jcastro: rockit, I've added a card and we'll try to get that updated. 20:32
rick_h_jcastro: you unblocked for the demo for now then?20:32
jcastroyes20:32
jcastrowe need this fixed by EOW btw20:32
rick_h_jcastro: ok cool, thanks for calling to get me on it20:32
rick_h_jcastro: k, will do. 20:32
jcsackettjujugui: super short pr, can i get a review?20:32
jcsacketthttps://github.com/juju/juju-gui/pull/25020:32
rick_h_jcsackett: will do20:33
jcsackettthanks rick_h_ 20:33
jcsackettrick_h_, just discovered on a make check i have a test failure. not sure why it didn't show on test-debug, but it answers my earlier testing question.20:37
jcsackettupdating and pushing in a moment.20:37
rick_h_jcsackett: k20:38
jcsacketthrm. i'm going to need find a way to get juju-gui email to go to my work email, not my personal.20:41
rick_h_there's a trick to it20:41
rick_h_can look it up later20:41
jcsackettthat would be awesome.20:42
jcsackettIs there a way to squash/cleanup commits once you've got PR up?20:58
rick_h_git rebase -i20:58
rick_h_clean up20:58
rick_h_git push origin XXX:XXX --force20:58
jcsackettAh. And that doesn't monkey up the PR? Or we just don't care?20:59
rick_h_we don't care20:59
jcsackettCool. 20:59
jcsackettThanks.20:59
rick_h_jcsackett: https://github.com/settings/emails and add the other address21:12
rick_h_jcsackett: so I added my canonical address there as an additional and it's picked up from my commits in github21:13
rick_h_ok, EOD for now all check you later!21:14
bacjujugui: quickstart review at https://codereview.appspot.com/90060043 , though it may make more sense to let frankban look tomorrow21:16
rick_h_jcsackett: once you're cool with your branch you have to enter :shipit: into a comment21:38
rick_h_jcsackett: and the lander will pick it up21:38
jcsackettrick_h_ saw that.21:38
jcsackettjust waiting for the CI to finish.21:39
rick_h_jcsackett: cool21:39
rick_h_jcsackett: the windows.flag.state should all be windows.flag.il now as well21:40
rick_h_jcsackett: my branch earilier changed them all21:40
jcsackettrick_h_, well, guess i'll have another squashed commit and another wait for CI. :p21:40
rick_h_:)21:40
rick_h_if you're confident about it you can :shipit: before the test run goes21:40
jcsackettah well. at least i got dev stuff sorted today and a minor bug addressed. i'll call it a win.21:40
rick_h_landing will run all the same tests, it's just generally good to wait so it doesn't hold up anyone else but you're good21:41
rick_h_all good :)21:41
jcsacketteh, it's not blocking me and i'm unlikely to do more work today, so we'll see everything go green this time.21:41
jcsacketti may not be as patient in the future. :P21:41
rick_h_k21:41
jcsackettwell, more coding. i have lots of emails to sort out--still on all the qa team lists.21:41
rick_h_wheeee21:42
marcoceppirick_h_: you around?22:11
marcoceppior anyone, how do I view the import errors on charmworld?22:11
rick_h_marcoceppi: I'm cooking dinner but around. 22:11
rick_h_marcoceppi: we request a log from webops and look at it :/22:11
marcoceppirick_h_: so, vsm is in charmstore but not in charmworld - promulgated on Friday22:12
rick_h_marcoceppi: what are you looking at22:12
marcoceppihttps://manage.jujucharms.com/api/3/charm/precise/vsm22:12
marcoceppihttps://store.juju.ubuntu.com/charm-info?charms=cs:precise/vsm22:12
rick_h_marcoceppi: https://store.juju.ubuntu.com/charm-info?charms=cs:precise/vsm ?22:12
rick_h_ok, looking22:12
rick_h_marcoceppi: k, file a bug please and we'll try to get the logs for it. 22:14
rick_h_marcoceppi: it proof's but yes it's not in the mjc at all. 22:14
marcoceppirick_h_: right, I'll open a bug22:15
rick_h_marcoceppi: ty22:15
lazyPowerhttps://bugs.launchpad.net/charmworld/+bug/1310825 - bug deployed22:17
_mup_Bug #1310825: VSM charm not listed in charmworld <charmworld:New> <https://launchpad.net/bugs/1310825>22:17
rick_h_cool, requested logs from webops, will have more info by tomorrow morning hopefully22:18
lazyPowerthanks a million rick22:25
rick_h_strange it won't ingest. I wonder if there's some strange bzr history thing going on22:28
rick_h_marcoceppi: lazyPower logs don't show anything...seems to have fallen into a black hole. I see errors for a couple of vsm branches, but nadda for the promulgated charm22:43
rick_h_marcoceppi: lazyPower any chance of doing something small to get a second revision out of it?22:43
marcoceppi\o/ rick_h_ all hail the glory of the black hole22:43
marcoceppirick_h_: lazyPower pushed a change to the branch recently22:44
rick_h_marcoceppi: ok, the store only shows rev 1 22:44
rick_h_marcoceppi: will see if it picks up a rev 2 and if we get any info22:44
marcoceppirick_h_: ack, thanks again for taking a look22:44
* rick_h_ hates tihs kind of invisible issue22:45
rick_h_the only feedback is on other versions Hook not found ~springfield-team/charms/precise/vsm/trunk common.sh22:45

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