/srv/irclogs.ubuntu.com/2014/09/24/#juju-gui.txt

hatch__huwshimi: how goes the battle?01:18
huwshimihatch__: Setting up handlebars...01:19
Makyobleck, that stupid test took forever, know what I'm doing on catchup week.01:20
Makyojujugui if anyone's around, https://github.com/juju/juju-gui/pull/578 is ready for rereview01:20
hatch__Makyo:  the notification tests?01:20
hatch__huwshimi:  setting it up....that should be easy???01:20
hatch__I suppose maybe not heh01:20
hatch__Makyo:  sure01:20
hatch__Makyo:  why the changes to pass the index through?01:21
Makyohatch__, per rick_h_ 's comment01:22
huwshimihatch__: It's not a problem, just a little hassle, having not done it before01:22
hatch__Makyo:  gotcha01:22
hatch__huwshimi: ahh yes01:22
hatch__huwshimi:  you've sure been dipping ways out of your comfort zone lately :) good to see01:22
MakyoNo tests around _waitOnLevel or whatever.  Will have to get some in there.01:22
huwshimihatch__: There's 70 thousand ways to set it up01:22
hatch__lol tis true...tis true01:23
hatch__considering it's just a templating language there are also as many different processors hah01:23
huwshimihatch__: I've gone for a non-YUI way to reduce the dependency01:24
rick_h_hatch__: yea, the idea is to help the user tell stacks of changesets apart a little to help make the notification seem like it's not a true repeat/etc01:27
hatch__rick_h_:  yeah np I just didn't see the comment01:28
hatch__so running Ubuntu on metal on this thing I get 3.5H battery life max....running Ubuntu in Parallels I get 5.5H max01:35
hatch__lol01:35
huwshimirick_h: Reading your blog post and got to the first screenshot of the machine view and now that we've removed the "Machines" title from the top of the list it makes it unclear that what you're looking at is a list of machines.04:44
rick_h_huwshimi: yea, I break it down into smaller bits, I just think the screenshot will get pushed down to 600px or so and you won't really read it at that level04:45
rick_h_so each screenshot after that is just a 'section' 04:45
rick_h_to aid in blog-friendly sizing04:45
huwshimirick_h_: I think this is a problem with the interface, not with your post04:45
rick_h_huwshimi: oh, you mean the machine column poist?04:45
rick_h_bah04:45
rick_h_huwshimi: I think it'll be more clear with the hardware info there 04:46
rick_h_huwshimi: and the nav button is machine04:46
rick_h_huwshimi: so in usage I think it'll be ok, but we'll find out from the users for sure. 04:46
huwshimirick_h_: You can figure it out, but it's now implicit04:47
rick_h_huwshimi: true04:47
rick_h_ok, /me is toast and going to head to bed. See you all for release day tomorrow. 04:54
huwshimirick_h_: Night04:57
=== uru_ is now known as urulama
rick_h_morning 11:31
* rick_h_ heads to take boy to day care and move to coffee shop for emergency injections11:54
* rick_h_ is back12:19
jcsackettMakyo: you around/available to chat?12:27
rick_h_jcsackett: probably a bit early yet. Anything I can help with?12:27
rick_h_jcsackett: I'd expect him in about 1.5hr ish12:27
jcsackettrick_h_: i'm cleaning up tests, looking in test_environment_view.js12:28
jcsackettit has tests for "remove relation" and "confirm" buttons, which are gone with MV (we just have the delete icon on the relation now when you click, and no confirm.12:28
rick_h_jcsackett: ah ok12:29
jcsacketti'm trying to figure out if this needs to be rewritten, or if we have tests for the new function and these can just be deleted.12:29
jcsacketti'm also trying to make sure i'm right about what these are testing. :p12:29
rick_h_jcsackett: gotcha, check out 969dd69e7c52d911b55d98ceda37e97465af9c7e12:30
jcsackettline 1004 or thereabouts "must be able to remove a relation between services" if you want to look.12:30
rick_h_jcsackett: and c9beb0576b03fdd395a4dfd3085f68963d173d0612:30
rick_h_jcsackett: and 5b6a94999f21c1039d86f89bc4cb9f263439602a 12:31
rick_h_jcsackett: and 45f9c937529e87b9adadb257a3d21d9f1b82206612:31
* jcsackett laughs12:31
jcsackettthis is a lot hashes. :p12:31
rick_h_jcsackett: so those have commits that seem to be about removing relation stuff12:31
rick_h_well, it's 4 commits12:31
rick_h_and I'd check and see what tests those commits have in them12:31
rick_h_and it might point you towards the idea that it's well tested elsewhere12:31
rick_h_or it might give you no hope and have you thinking of rewriting what's there12:31
jcsackettrick_h_: what's the best way to look at what changed in a single commit? "git diff sha1" shows me the diff between my branch and that sha, which is not great.12:32
rick_h_jcsackett: 'git log -p $sha'12:32
jcsackettrick_h_: fantastic, thanks.12:33
rick_h_kadams might be interested in those commits as well I guess12:34
jcsacketthuzzah, did need to rewrite, but just remove the parts about confirming, since the parts about removing are still valid.12:39
* jcsackett continues on12:40
rick_h_jcsackett: yay12:40
* jcsackett groans at tests that pass on test-prod locally and fail on jenkins12:55
rick_h_jcsackett: hmm, that's new12:56
jcsackettrick_h_: has to be an isolation thing; the tests legitemately don't have an ecs set up, so that failure should happen, but doesn't happen locally.12:57
jcsackettwhich is fun.12:57
jcsackettshould be an easy fix, at least.12:57
rick_h_jcsackett: heh, /me can't wait for a week of cleanup...12:57
* jcsackett nods12:58
jcsackettrick_h_: oh this is annoying.13:11
rick_h_jcsackett: ugh, notifier tests hitting timing issues on you. Retry :/13:13
rick_h_that one is the #1 code to get ripped out 13:13
rick_h_bac: when you're up and about appreciate some english skills on the blog post please. 13:16
rick_h_bac: since it was written a bit late I don't trust myself :)13:16
bacrick_h_: i'm about13:17
bacrick_h_: looking now13:17
rick_h_bac: sorry, meant that more as a 'when you have time, no rush' than anything13:17
bacrick_h_: you prefer comments or edit in place?13:19
rick_h_bac: feel free to edit in place13:19
bacrt13:19
rick_h_bac: comments if you're not sure or it's a big change you'd suggest. I can go update it. 13:19
rick_h_kadams54: how goes, did the relation stuff make any better sense later on?13:30
kadams54rick_h_: Going well. Should be wrapping up on it soon.13:33
rick_h_kadams54: awesome13:33
* rick_h_ heads back home from the coffee shop13:34
jcsackettjujugui: can i get one more review and QA on https://github.com/juju/juju-gui/pull/58013:39
rick_h_hazmat: getting ping'd by PR and such on machine view stuff and the question came up about comparing to tools available for puppet/chef. It looks like chef is in silent development mode. Is there anything I should make sure to watch as something similar I'd be expected to know/compare it to?13:54
rick_h_hazmat: last stuff I poked around was the basic ui for docker and I guess the AMZ stuff that has the ability to build out the services. 13:54
=== kadams54_ is now known as kadams54
frankbanguihelp: need reviews for https://github.com/juju/juju-gui/pull/581 anyone?14:16
rick_h_frankban: looking14:17
frankbanthanks14:17
rick_h_frankban: have a sec to chat as well? standup room when you get a sec14:17
frankbanrick_h_: sure joining14:17
* frankban bbiab14:23
hatchjujugui call in 1014:50
rick_h_says you :P14:50
hatch8?14:52
hatch:)14:52
rick_h_jujugui call now!15:00
rick_h_wheee15:00
rick_h_rogpeppe: ^15:00
=== arosales_ is now known as arosales
hatchlooks like the save config command no longer sends only the changed fields...but instead sends all of them15:33
hatchnot really an issue but we may want to look into that15:33
hatchwait nm ignore that15:40
jcsackettrick_h_: can you look at my reply on https://github.com/juju/juju-gui/pull/582 when you have a chance? want to make sure we're on the same page.15:45
=== fabrice is now known as fabrice|family
rick_h_jcsackett: looking at the diff there's the green lines of adding in the stuff in taht if statement15:47
rick_h_rmrelation_dialog.hide()15:47
rick_h_...15:47
rick_h_jcsackett: so it appears that only the if itself was removed vs the code it contained?15:47
jcsackettrick_h_: bah, misread side-by-side.15:48
jcsackettrick_h_: you're right. updating.15:48
rick_h_jcsackett: cool, thanks for the double check15:48
jcsackettrick_h_: removing flags gets a bit confusing. :p15:48
rick_h_100%15:48
kadams54guihelp: Looking for reviews and QA on https://github.com/juju/juju-gui/pull/58316:07
kadams54hatch: ^^^ I'm sure you'll have an opinion :-)16:07
hatchkadams54: heh I'll look a little later16:08
kadams54hatch: oh yeah, heads down… get back to work! ;-)16:08
kadams54Going to lunch and then starting in on lots of QAing <- jcsackett, frankban 16:09
rick_h_frankban: Makyo if you guys could peek at ^ that'd be great please. The relation stuff is tricky 16:09
frankbankadams54: looking16:10
Makyorick_h_, kadams54 sure16:10
rick_h_ty all16:10
jcsackettkadams54: thanks.16:13
=== urulama is now known as urulama-afk
Makyojujugui any further thoughts on https://github.com/juju/juju-gui/pull/577 ?  It's trivial, may just ship it16:43
rick_h_Makyo: feel fre16:43
MakyoCool, thanks16:44
frankbankadams54: reviewed16:50
frankbankadams54: I have to go in 5, if my branch looks good to you please feel free to shipit later16:51
kadams54frankban: Thanks! Will do.16:51
* rick_h_ goes to make up some lunch16:53
rogpeppejaasteam: trivial lppublish bug fix: https://github.com/juju/charmstore/pull/11717:04
rogpeppeif anyone is, like me, interested in studying the current charm and bundle "corpus", here's a script that downloads all charms and bundles: http://paste.ubuntu.com/8419426/17:06
hatchphew I think all done this now17:06
hatchwhat a mess17:06
hatchoh well17:07
hatch*stretch*17:08
rogpeppeand here *are* all currently known bundles, in one lump: http://paste.ubuntu.com/8419463/17:11
hatchrick_h_: so far the only known bug this will create moving forward is that once you resolve a conflict the setConfig record is still there even if the value which was changed is no longer changed - but it is set to the appropriate values as a workaround17:12
hatchbut I figure that this is a limited use case anyways....and that isssue is even MORE limited17:12
hatchheh17:12
hatchMarky Mark and the Funky Bunch - Good Vibrations (feat. Loleatta Holloway)17:15
hatchhttps://www.youtube.com/watch?v=-eSN8Cwit_s17:15
urulama-afkso, rogpeppe, how are we going to deal with the first "cloud" charm, as it has orangebox bundle inside?17:17
rogpeppeurulama-afk: it would turn into two charms with the names ~kirkland/bundle/transcode-cluster-cloud and ~kirkland/bundle/transcode-cluster-orange-box17:19
rogpeppes/two charms/two bundles/17:19
urulama-afkrogpeppe: ok, and to my actual question: can we reference one bundle from another?17:19
rogpeppeurulama-afk: no, bundles can only refer to charms17:20
rogpeppeurulama-afk: unless... oh yes, there might be some kind of inheritance thing going o17:20
rogpeppeurulama-afk: but only within a set of bundles within the same file17:21
* rogpeppe tries to remember where the original bundle specification was documented17:21
rogpeppehazmat: any idea?17:21
urulama-afkrogpeppe: so instead of juju deploy ~kirkland/bundle/transcode-cluster, we need juju deploy ~kirkland/bundle/transcode-cluster-orangebox and then ~kirkland/bundle/transcode-cluster-cloud17:22
urulama-afkrogpeppe: is that correct?17:22
rogpeppeurulama-afk: the first one wouldn't be valid anyway17:22
rick_h_hatch: yea, sounds good to me 17:23
hatchjujugui I need two QA's on this branch https://github.com/juju/juju-gui/pull/584 no review yet - I still need to write the tests, but the qa should have quite a bit of exploratory work around it.17:27
jcsacketthatch: can "modifyUnits" in service-overview.js be removed? it's part of the old scale up system, right?17:38
hatchsec otp17:38
jcsackettline 511, or thereabouts when you have a moment.17:38
hatchok looking17:39
hatchjcsackett: yeah looks like it was only part of the old stuff17:41
jcsacketthatch: whoo. that's good news, means i can start resolving some test issues by ripping that out. :)17:41
hatchawesome - also the template and the _modifyUnits stuff17:41
jcsacketthatch: second question for you, b/c i can't remember. is this one of our spurious issues? http://ci.jujugui.org:8080/job/juju-gui/1943/console17:41
hatchI'm pretty sure the spurious ones were only in ie now17:42
jcsackettfantastic. why doesn't this show the failure output in a useful fashion? :p17:42
hatchjcsackett: yeah that looks like a real error17:42
rick_h_jcsackett: don't think so. It has a failure on 'ecs methods - is instantiable'17:42
jcsackettrick_h_: where are you seeing that?17:43
jcsacketti've stared at this so much i just see text without meaning.17:43
hatchin the sauce video17:43
rick_h_jcsackett: clicking on the link to the sauce labs, clicking on the screenshots17:43
jcsackettaaaah17:43
hatchor the screenshots17:43
hatchheh :)17:43
rick_h_start with screenshots and then go to video when they skip what you want to see ime17:43
jcsackettok, i badly need lunch and then i'll return to this.17:45
* hatch puts his HTC One in his front pocket and goes to lunch without any fear of it bending...17:55
rick_h_lol17:55
kadams54lol 17:56
hatch:D17:58
hazmatrick_h_, re competing guis.. see panamax.io for docker from telco centurylnk18:00
rick_h_hazmat: ty18:00
hazmatrick_h_, also ansible tower though its a different focus18:01
hazmatre puppet/chef the chef ui is as minimalist as ever.. no changes really. salt has halite (https://github.com/saltstack/halite) which is also minimal but functionaly.. there's a couple more polished ones for puppet 18:03
=== mup_ is now known as mup
hatchwe really need a better container story...18:46
rick_h_hatch: you up for air or want to pass on meeting today?18:54
hatchummm I have nothing to chat about, could keep on this test fixing unless u have something?18:55
rick_h_hatch: rgr18:55
rick_h_hatch: sounds good, thanks18:55
hatchjujugui speaking of which I still need some qa's on my PR so we can get it landed as soon as I finish these tests18:58
kadams54hatch: QA-ing another card right now; I can start in on yours in about 10 minutes or so.18:59
hatchcool thx18:59
hatchmy qa is time consuming - lots of possible interactions19:00
hatchso wanted to get as much of a head start as possible :)19:00
kadams54hatch: my favorite kind :-)19:00
rick_h_hatch: post YUI javascript session added to brussels and noted you'll have research to present. 19:15
hatchas marky mark would say "word-to-your-motha"19:16
Makyo...19:16
rick_h_ummm, wow19:16
hatchMakyo: did you miss my previous link to the video?19:16
hatchlol19:16
hatchok maybe that was only funny in my head19:16
* hatch goes back to clown college 19:17
MakyoHahaha19:17
hatchok after wiping the tomatoes off my face after that performance I'm going back head down to get this finished :)19:19
MakyoDeploying the mongo bundle to local provider was a bad idea.  My laptop is supremely unhappy.19:29
hatch:) io is probably hammered19:32
MakyoIt just turned off :P19:36
hatchlol nooooo19:36
hatchjujugui we have tests for conflicts in the inspector constraints UI....this doesn't even exist any longer does it? cc jcsackett19:37
jcsacketthatch: there's constraints editing stuff in the scale up UI--if it's not that, then no.19:37
hatchright....but that's not databound19:37
hatchso ok this is all for the old UI19:37
hatchctrl+a del+19:38
hatchit's so nice working directly in Ubuntu19:40
rick_h_hatch: <319:45
hatch:)19:46
rick_h_jcsackett: how you doing? need a hand duping tests on that one branch still or anything?19:50
jcsackettrick_h_: i was able to replicate them, with some pain, locally, and i think i fixed them. watching CI again now.19:52
jcsackettrick_h_: and fortunately working on the last of the mv work as i wait, so things continue along.19:52
jcsackettrick_h_: that stupid branch, man.19:52
rick_h_jcsackett: ok, ping if you need anything19:52
rick_h_Makyo: did you replicate frankban's QA issue at all?19:53
Makyorick_h_, Oh, hm, I didn't try that.  Back up and running, will focus.20:02
Makyorick_h_, it's existing, shouldn't block kadams54 - I'll get on that right now.20:04
rick_h_Makyo: ty!20:08
rick_h_kadams54: can you let me know when you address frankban's comments then and go ahead and land that one please?20:13
kadams54rick_h_: sure. Was planning on doing that after I finished QA on hatch's PR.20:14
rick_h_kadams54: ok cool thanks20:14
hatchugh just waisted an hour on tests which don't matter20:14
* hatch throws a "our test suite is a mess" hissy fit20:15
hatchkadams54: I'm just about ready to push up the tests - any big issues?20:19
kadams54hatch: Don't know.20:19
kadams54hatch: Or: yes.20:20
hatchwait I lied...I have another failing test20:20
hatchthere is an issue?20:20
kadams54hatch: I ran into an uncaught error while just trying to deploy mysql. I'm not sure if it's on your branch or a regression in develop. I'm checking develop now.20:20
hatchhmm I can deploy it here so I hope it's develop heh20:21
kadams54guihelp: well fudge. develop seems to have a pretty serious regression on it. This is what I get when I just try to drag-n-drop a service to the canvas (selecting auto-deploy in the deploy summary for the units):20:23
kadams54"Uncaught TypeError: Cannot read property 'apply' of undefined", environment-change-set.js:24420:23
rick_h_kadams54: I can replicate here, in the function call the env is a list vs a single environment20:24
rick_h_jujugui http://paste.ubuntu.com/8420601/ is the breakage point and the value of env is a list of two items long20:25
kadams54rick_h_: Maybe introduced by the multi-ecs work then? I haven't done any deeper digging (bisecting) so that's purely a guess.20:25
rick_h_yep, here  Y.soon(Y.bind(this._commitNext, this, [env, currentIndex]));20:25
rick_h_that is broken and passing env, index around as env20:26
rick_h__commitNext: function(env, currentIndex) { is the signature20:26
rick_h_so the [] is not needed20:26
rick_h_or is off or something20:26
hatch+129  −390   awww yeah20:27
rick_h_kadams54: ok, keep going on QA, I'm going to do a quick test removing the [] and make sure that fixes it20:28
kadams54rick_h_: will do20:29
hatchjujugui my branch is completely finished now and ready for reviews/qa https://github.com/juju/juju-gui/pull/58420:31
rick_h_hatch: rgr20:35
hatchrick_h_: where is your test :PPP20:39
hatchrick_h_: what would you like me on?20:39
rick_h_hatch: so general QA and reach out if anyone needs a hand like jcsackett with the FF or Makyo with the relation line issue. 20:40
* hatch opens arms "come to me with your problems"20:41
rick_h_hah20:41
hatchI can hop on some more reviews once your fix lands20:42
hatchthe ci box is just burrrrnin20:42
rick_h_we've got 5 branches in the queue to get through CI and then I can work on the release tonight at the coffee shop20:42
kadams54jujugui: FYI, I'm going to have to leave for dinner/family stuff around 5, but should be back on around 6.20:44
rick_h_kadams54: ok, what's left on your plate between now and 5?20:45
hatchI cleared my schedule (because it was SOOOO full) for tonight if we need extra backup20:45
rick_h_hatch: hah, no kiting to celebrate pre-release?20:45
hatchno wind!20:46
kadams54rick_h_: I just addressed frankban's comments on my PR#583. Hopefully the merge build will go through fine and I'll be able to ship that.20:46
rick_h_kadams54: ok cool20:46
rick_h_kadams54: then enjoy dinner and if anything goes boom we'll get it covered. Thanks for the work today!20:46
kadams54rick_h_: Currently QAing Jeff's in a real env (i.e., off his branch and not with latest from develop merged in) and trying to figure why I'm not seeing an orange exclamation mark.20:46
rick_h_kadams54: ok, I'm loading a live env as well atm20:47
kadams54Yeah, that's why I'm not yelling at hatch yet :-)20:47
kadams54Since you didn't run into any problems.20:47
rick_h_kadams54: yea, maybe check the source and it's a build issue where the css has an issue or something20:48
Makyojujugui https://github.com/juju/juju-gui/pull/586 quick relations thing, just moving some code to the right place.20:48
rick_h_Makyo: looking while the source updates on jeff's live env20:49
* Makyo runs to exercise dogs real quick20:49
kadams54*sigh* Just had to explain to my second grader that whacking the iPhone is not going to solve the problems he's experiencing.20:49
rick_h_lol20:49
rick_h_"does it bend?! I mean blend?!"20:50
hatchlol20:50
kadams54He seems determined to prove that even 5Ses are prone to bending…20:51
kadams54Provided you whack them repeatedly when Minecraft freezes up.20:51
hatchgive him an 80's Nintendo controller - those things were bullet proof20:51
rick_h_"I TOLD you this one's dead I need a new one!"20:51
hatchhaha20:51
kadams54Too bad the cartredges weren't ;-)20:51
hatchtruth20:51
hatchbut the crt tv screens were very strong so you could huck that controller at it20:51
kadams54None of the "Wiimote through the brand new flatscreen" problems.20:52
hatchexactly....things just aren't made like they used to be20:52
rick_h_woot! hatch qa ok on a live env here20:54
kadams54hatch: I don't see any related CSS. Does the orange exclamation mark come from _showConflictUI?20:54
hatchyusssss20:54
rick_h_orange point works for me20:54
hatchkadams54: it's always been there20:54
rick_h_in both dev and live lxc env20:54
hatchthat was all the oldschool UI I just hooked up20:54
kadams54Ok20:55
kadams54Holy crap hatch, you weren't kidding when you said CI was on fire.20:56
hatchhaha yup20:57
rick_h_shhhh, she's behaving at the moment20:57
rick_h_go baby go!20:57
hatchlol right20:57
kadams54hatch, rick_h_: Well I gotta pack up. I'll have to circle back on Jeff's branch and the problems I'm having later tonight. I'll also check on my PR and make sure the merge build went through OK.20:58
kadams54It may take until 6 before that thing actually builds :-)20:58
hatchhaha20:58
kadams54See ya'll in a bit.20:58
rick_h_kadams54: all good, Makyo can you do a second opinion  on hatch's branch please?20:58
rick_h_hatch: you looking to be a second on Makyo's branch?20:58
rick_h_hatch: I did QA so just need second review20:59
hatchon it20:59
rick_h_jujugui wife is tied up at work so I have to get the boy. Carry on and I'll be back shortly. 21:01
jrwren_I'm EOD. I'll chat ya'll tomorrow.21:04
hatchcya jrwren_21:04
jcsacketthatch: can you look at the cursed branch? it finally finished its CI run and now has *more* errors.21:26
jcsackettat this point i think i might be breaking it rather than fixing it.21:26
hatchjcsackett: haha ok looking21:27
hatchjcsackett: very odd....21:34
jcsackettNot so much odd as a damn mess. 21:34
hatchjcsackett: can you land it in smaller bits? Or is this all required?21:35
jcsackettI mean, that *is* one small bit. 21:35
hatchyeah I mean there isn't much code changes for all the tests that need changing21:37
hatchare all the test changes necessary with that code change?21:37
hatchjcsackett: well the good news is that I can reproduce the failure locally21:39
jcsackettReally? Cause I can't. That passed. 21:39
jcsackettWhat did you do?21:39
hatchjust ran them in FF21:40
hatchfailed hard first run21:40
hatchthat's where they are failing in CI too21:40
hatchthey pass fine for you?21:40
jcsackettYeah. Or I wouldn't have submitted. 21:41
rick_h_hatch: can you pick it up since you can dupe please?21:41
rick_h_jcsackett: and you can run with the otehrs 21:41
jcsackettWorks for me, if that works for hatch. 21:42
rick_h_I'll bribe him with a mouse 21:42
hatchlol21:42
hatchyou could pay for my geekdesk? :P21:43
hatchyeah I'll take over on this21:43
rick_h_lol, man price of a bribe is going way up21:43
rick_h_jcsackett: can you get hatch the starter commit sha?21:43
hatchhaha - other cheaper options include a PS421:43
rick_h_jcsackett: I think maybe all the work to 'make them pass' might have left things worst than they started out at21:43
hatchDestiny for PS421:43
hatchhmm....that's all for now21:43
hatchI will just start from scratch21:44
hatchjic21:44
jcsacketthatch: i can send you the initial branch. 21:44
hatchnah it's ok I got the commits21:44
jcsacketthatch: basically just grab the app/app.js flag stuff. the endpoints bit can be ignored.21:46
jcsackettand it's the app setting the ecs on by default that started all this fallout.21:46
hatchcool21:47
MakyoWe wearing jenkins out?21:54
rick_h_npm out21:54
rick_h_npm is getting angry at all the downloading21:55
MakyoBoo21:55
rick_h_Makyo: how went the QA of hatch's stuff?21:58
MakyoOh, it went okay, one sec.21:59
rick_h_Makyo: lint error in your branch killing CI22:09
rick_h_Makyo: sorry, I misread it the first time I looked22:09
rick_h_test/test_environment_view.js: line 1032, col 51, Missing semicolon. (W033)22:09
rick_h_Makyo: ^22:09
Makyorick_h_, ack, on it22:10
rick_h_jujugui I'm going to head off to the coffeee shop for CHC tonight. I'll check in from there in 45ish. Makyo if QA goes well please ship hatch's and if anyone sees kadams back ask about his branch. He's pushed another commit of comments, but not sure if it's ready foir landing22:13
MakyoSounds good.  SEe ya later22:13
hatchjcsackett: so here is the problem....as far as what's causing it...I have no idea       `  http://0.0.0.0:8888/login/undefinedapi/3/search/interesting `22:16
jcsacketthatch: should i run my localhost for that, or is taht something else?22:17
jcsackettoh!22:17
jcsacketti missed the undefined.22:17
jcsackettthat's...*really* weird.22:17
hatchit happens when I add the ecs to the env22:17
hatchfor the endpoints22:18
hatch(which needs to be done else it fails even sooner)22:18
hatchjcsackett: even if I skip the ecs and just stub out the deployer bar instantiate method I get the same issue22:25
hatchgoing to take a bit longer22:25
jcsacketthatch: if you're still grinding after i finish off the other flag locations i'll start poking back at it.22:26
hatchjcsackett: ok so for some reason the test is passing...but then comes back and fails in the 'after' because the url was changed to /login22:33
hatchlol22:35
hatchjcsackett: solved that one22:36
hatchthe env had no creds so it would redirect the user away22:36
jcsacketthatch: interesting.22:39
jcsacketti wonder why removing the mv flag would cause that.22:39
hatchwho the heck knows22:40
hatchanother failure down....22:43
hatchlets see if this works22:43
hatchoo it passed all the tests in debug22:43
jcsacketthatch: hopefully you won't have my situation where that means *nothing*. :p22:44
hatchhahaha22:44
jcsackettand if it does, i'm removing my LXC and going to go the vagrant route.22:44
jcsacketter, and if you don't, rather.22:44
hatchwell my fixes are totally different than yours22:45
hatchwell...same idea22:45
hatchdifferent approach22:45
hatchjcsackett: well....it failed spectacularly in the browser22:47
jcsacketthatch: i'm sorry to hear that.22:47
hatchlike...ka....fricken...boom!22:47
hatchgood news22:48
hatchdifferent tests failed22:48
hatchlol22:48
hatchprogress??22:48
hatchoo only 15 failures in the browser now22:53
hatchwhy are they always part of notifications...sonofa22:53
hatchjcsackett: got it23:01
jcsacketthatch: nice.23:01
jcsacketti can review it.23:01
hatchjust running lint then I'll push up23:01
hatchjcsackett: https://github.com/juju/juju-gui/pull/58723:06
* hatch crosses fingers it works in CI23:06
rick_h_poor poor CI23:07
* rick_h_ remembered that he needs to generate the changelog...o...m...g23:07
hatchlol23:07
rick_h_woot! my big picture frames of the CA trip shipped. 23:09
rick_h_Tomorrow is going to be win win win23:09
hatchriiiight on23:09
hatchamazon.com has the playstation pkg I want but they won't ship to Canada :///////23:09
hatchI could understand if amazon.ca had them....but seriously they ship from the same damn warehouse23:11
hatchoh I also have to do that blog post tonight23:12
rick_h_hatch: :P23:12
rick_h_hatch: though you can do friday as well if you need23:13
rick_h_hatch: do it during the day tomorrow23:13
rick_h_hatch: lots of time :)23:13
rick_h_lol https://www.youtube.com/watch?v=IROcoJeVfSI#t=348 well my new motox won't bend at least :)23:15
hatchgooood idea23:16
rick_h_ok, land all the things, anger the CI gods for a few more branches23:18
rick_h_time to get changelogging23:18
hatchI can't believe I'm watching this23:21
rick_h_lol23:22
rick_h_I didn't have sound so you've got more info than me23:22
hatch3 branches to go......go go ci go23:24
jcsackettrick_h_: are we comfortable releasing without all the MV cleanup done?23:33
hatchjcsackett: I don't think there would be much issue23:34
hatchsome code bloat but I don't think there is any real side effects23:34
hatchnot that you were asking ME23:34
hatch:P23:34
rick_h_jcsackett: yes23:35
rick_h_jcsackett: and I think we can plan on another release next week23:35
rick_h_jcsackett: with some cleanup and bug fixes/etc23:35
rick_h_jcsackett: so don't go all night if you're not going to make it23:36
rick_h_well, I take that back. Don't go all night even if you will make it23:36
hatchhaha23:37
jcsackettrick_h_: i'm going to keep hacking at this b/c i'm mad at it, but good to know. :)23:37
rick_h_jcsackett: it's 7:30pm man. Go to dinner followed by bed 23:38
rick_h_if we hit any issues tomorrow I'll need you all to help23:38
rick_h_jcsackett: and getting this done isn't going to change anything release-wise23:38
rick_h_seriously, go rub your wife's feet or something :P23:38
* jcsackett laughs23:38
hatchOR23:38
hatchshe could rub yours!23:38
jcsackettshe's not home yet. when she gets home, i'll kick off.23:38
jcsacketthatch: that's not going to happen. :p23:39
hatchlol well it was worth a try23:39
rick_h_ok commit log isn't all that bad actually23:40
rick_h_since 1.1.1 we've basically worked out $#$#@ off on MV23:40
hatchoh no huw today23:49
rick_h_oh yea, it's his thurs23:50
rick_h_he's finally swap daying london23:50
hatchyeah I got to do that!!23:51
* hatch looks at calendar23:51
hatchugh my branch failed23:55
hatchwth it passed in everything locally23:56
rick_h_hatch: yea, giving it another shot23:56
hatchnah it'll fail23:56
rick_h_hatch: but if you want to see if you can dupe that'd be cool23:56
hatchI know how to fix it23:56
hatchbut I can't dupe23:56
rick_h_party23:56
* rick_h_ runs away23:56
hatchso will just have to be a hope and a dream hah23:56
rick_h_hah, hopes and dreams, we're UI Engineering23:56

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