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

rick_h_gary_poster: around?00:44
rick_h_or do I start the email00:44
rick_h_ok, well email replied to 00:50
bacrick_h_: you still around?  i see  heartbeat is happy now.  thanks for getting it up again!01:20
bacrick_h_: i read the backlog on webops but i'm not sure what happened01:20
bacrick_h_: review done01:25
rick_h_bac: thanks01:30
rick_h_bac: yea, there was confusion. It was changing that url in the charmtools + restarting supervisord that runs the ingest process01:31
rick_h_bac: but took a while to find someone and to debug that there's not an issue with the version of charmtools on the server01:31
rick_h_bac: ok, I'm submitting the branch now. So in the morning we need to do another deploy to the rev after this lands (assuming the lander doesn't hate me)01:32
rick_h_bac:  the tricky part is that the production-overrides should be used to build a new config, but config changes don't happen all that often so cross your fingers on that part01:33
bacok, i followed it all but that last part01:33
rick_h_bac: so the production_overrides.ini01:33
rick_h_bac: where the port number was added?01:34
bacrick_h_:  if it lands do i need to do anything but request a deploy?01:34
rick_h_bac: hopefully no. THe cowboy will be erased by upgrading charmtools and the deploy should go smoothly01:34
bacrick_h_: thanks again for getting this figured out.  let's hope tomorrow goes smoother.01:35
rick_h_bac: rgr, we're all good. If we can get the charm stuff into shape we'll be good to land this baby and have a long good weekend :)01:35
bacrick_h_: rt.  good night.01:35
rick_h_thanks for getting that branch going. I'm going to go turn the brain off for the night. See ya tomorrow01:35
gary_posterthank you both rick_h_ and bac01:58
hatchevening04:50
huwshimihatch: Hello04:55
hatchhows it going?04:55
huwshimiGood thanks, yourself?04:55
hatchgood good04:57
hatchtrying to fix my vm04:58
hatchdoesn't look like parallels supports ubuntu 13.10 no matter what04:58
huwshimiOuch05:01
huwshimihatch: Dual boot?05:01
hatchI'll have to look into that on the mac mini05:01
huwshimihatch: Buy a new computer? :P05:01
hatchhaha - I willllll but I have pretty much convinced myself I need the 15" with the graphics card05:02
huwshimiheh05:02
hatchdid you end up picking up a new one?05:02
huwshimiYeah, well, it's in the post05:02
huwshimiShould arrive next week05:03
hatchcool what did you go with?05:03
huwshimihatch: Air, i7, 8GB RAM, 256GB SSD05:03
hatchoh cool that's the other option I was thinking of going with05:04
huwshimiSo couldn't just pick up in store05:04
hatchthat thing should fly :)05:04
huwshimiYeah, hopefully05:04
hatchwell this thing has the 2.3 I5 with the intel 3000 and 16GB of ram05:05
huwshimihatch: In one of the stores the guy was trying to convince me of the retina 13" MBP as for $100 more you get faster CPU plus retina, but it weights 400 grams more.05:05
huwshimihatch: Nice. How much?05:05
hatchno idea I bought it in 201105:05
hatchhaha05:05
hatchit only supports 8GB of ram05:05
hatch:)05:06
huwshimiOh right, your current computer05:06
hatchyeah05:06
huwshimihatch: How much would the new MBP cost?05:07
hatchso this thing -is- fast enough for everything i need it for, but no games05:07
hatchlots.....lol05:07
huwshimiheh05:07
hatchsec05:07
hatch$2600+tax05:07
hatchassuming I don't upgrade the cpu or storage05:08
hatchcompared to the $1500 for the air (equipped like yours)05:08
hatchso the question is....do I need to game for that $1100 more? :D05:08
huwshimihatch: haha05:09
hatchit would cost me that for a gaming rig05:09
hatchso really....05:09
huwshimihatch: Or a console05:09
hatchtrue, but I can't take the console with me05:10
hatchat least not very easily05:10
huwshimihatch: Although the CPU will mean running test faster etc.05:10
hatchthe single core speed isn't that much faster on the pro than the air05:10
hatchit just has 2 more cores05:10
hatchand our tests are single threaded right now05:10
huwshimitrue05:11
hatchso it'll only be like 10% faster05:11
huwshimihatch: But you're running VMs etc. as well05:11
hatchare you trying to convince me to buy the mbp?05:11
hatchyour a bad influence05:11
huwshimihaha05:11
huwshiminope, I mean, I think, uh...05:12
hatchok when you get yours you can be my test monkey05:13
hatchyou can run the tests and maybe some games for me to see if they run :P05:14
huwshimiheh05:14
huwshimihatch: I bought it through a local store which meant a saving of $140 vs the online Apple store, but it means I don't have an online tracker to refresh every five minutes05:15
hatchhaha05:15
hatchsmall price to pay for the $140 discount :)05:15
huwshimiyeah05:15
hatchI'm going to try and hold out until a few weeks before the next sprint so I'll have some time to decide05:16
huwshimiI actually didn't realise you could get them cheaper than the Apple store05:16
hatchI didn't either05:16
huwshimiheh05:16
rick_h_morning12:07
rick_h_howdy frankban, did you tinker with benji's branch at all?12:13
frankbanrick_h_: yes. I made some changes, trying now bootstrap + deploy12:21
=== _mup__ is now known as _mup_
rick_h_frankban: ok cool, in looking at his branch I think the missing thing is that we need to send a dict to the deployer call vs the string from sanitize_constraints12:21
rick_h_normally the deployer uses a string because it passes it to the cli as `juju deploy --constraints=""'12:22
rick_h_frankban: ok, let me know how it goes and what I can do to help. I'll stop messing with his branch for now then. 12:22
rick_h_and grab coffee...12:23
frankbanrick_h_: I added that and moved the sanitize logic from base to views, so that, if a constraint is passed but not supported, we can deny the bundle deployment and send to the GUI the error message12:23
frankbanI also created a new deployer tarball with the fix to the deploy() call, adding the missing parameter12:24
frankbanrick_h_: ^^^12:24
gary_postersounds great frankban12:36
gary_posterthank you12:36
=== BradCrittenden is now known as bac
bacgary_poster: staging has been updated with the changes rick landed last night.  it looks happy.12:38
gary_posterawesome bac.  thanks12:39
bacgary_poster: i shall now proceed with an RT to get production updated and less stupid12:39
gary_posterheh, great12:39
bacgary_poster: you're aware of our decision to temporarily fork charm-tools for expediency until we can remove it entirely?12:39
gary_posteryes, bac.  I'd like to talk more about the "remove it entirely decision" once the heat of the moment has passed and see what other options we can pursue, but yes, I know we have a three-stage story of "hack production" (last night), "hack the branch with a fork" (last night and today), and "make it better" (ASAP, such as next week).12:41
rick_h_frankban: awesome12:41
bacgary_poster: yeah, that's a more rational statement.12:43
gary_postercool12:44
rick_h_rational? are we still at rational? :P12:44
gary_posterwe have next week for that :-)12:44
frankbangary_poster, rick_h_: it seems to work, writing some missing tests and proposing12:52
rick_h_frankban: rgr, will look at the MP when it hits12:52
gary_posterawesome!12:53
frankbanbtw, new macbook arrived this morning, left ignored in its box12:59
bacrick_h_: the new deploy produced https://pastebin.canonical.com/100054/12:59
rick_h_frankban: we all appreciate the sacrifice12:59
baci think it is not a problem but causes the webops to be concerned12:59
rick_h_bac: ?!12:59
rick_h_bac: yea, I'm looking for the hook to see what it's doing12:59
bacjujugui: post deploy charmworld is good: http://manage.jujucharms.com/heartbeat13:01
rick_h_bac: cool, waiting to see if the bundles get processed vs blocked13:01
rick_h_bac: yea, not sure on that migrate thing. I don't understand why it's doing that13:02
bacok13:02
rick_h_bac: well, not sure what prepare-upgrade --init is 13:02
bacrick_h_: i'm making a card about the queueing of multiple revisions per sinzui's comments yesterday13:02
rick_h_ooh, let's see if my heartbeat auto reload worked13:02
rick_h_bac: ok, I htink we're fine. In my testing last night we were around 8ish minutes to process the queue. 13:03
rick_h_so we're well under the 15min window for now13:03
rick_h_yay for auto reloading heartbeat page!13:03
* rick_h_ is facinated by the little things in life13:03
benjicool bac13:04
rick_h_gah, docstrings or bust 13:04
rick_h_bac: all 0 so yay!13:05
bacrick_h_: yes, not waiting to timeout due to egress rules speeds up the processing dramatically.  go us.13:05
rick_h_:)13:05
rick_h_bac: it's almost like...you called that one lol13:06
bacgary_poster: webops wants us to move staging off canonistack so it is more production-like13:06
rick_h_bac: I've got a card for that13:06
gary_posterbac ack on call13:06
bacoh good13:06
rick_h_bac: it'll need to be a dupe since we won't be able to auto upgrade it from CI13:06
rick_h_it'll be more a 'practice deploy' than anything from what I can tell13:07
bacoh, ok13:07
bacdearlordletitwork.jujucharms.com13:07
rick_h_lol13:08
rick_h_bac: oh, that whole output was just to test if the prepare-upgrade flag was available to the migrate script13:32
rick_h_bac: I guess it's an artifact that on the first time you update the charm/code they need to be in sync. this protects them from being out of sync13:33
rick_h_bac: since the charm doesn't want to run migrations prepare-upgrade if that's not a valid command13:33
bacrick_h_: hmm, can it be squelched or a message saying "ignore the following non-error"13:33
rick_h_bac: i bet -h goes to stdout or something. I think the best thing is to just remove the check from the charm13:34
rick_h_we're past a point where it's a useful check13:35
bacrick_h_: ok.  i made a card for it.13:35
rick_h_bac: rgr13:35
benjimorning rick_h_, what's the constraint sanitization status?13:41
rick_h_benji: frankban has a branch he's getting up for review13:45
benjirick_h_: great!13:45
* gary_poster running. back in 1.5 or 1.75 hours. (what a week)14:01
frankbangary_poster, rick_h_, benji: the branch is up for review and qa: https://codereview.appspot.com/2281004314:06
benjifrankban: I'll be glad to do one or both; I'll start on the review.14:06
rick_h_frankban: looking14:08
frankbanbenji, rick_h_: thank you both14:08
benjifrankban: I thought that we wanted to ignore invalid constraints, not generate an error.  Have I misunderstood the requirements or your code?14:11
rick_h_benji: we were ignoring when it was in the jujuclient because we could not get errors out of it to the user14:12
rick_h_benji: are these errors able to get to the user from the new location?14:12
rick_h_frankban: ^14:12
frankbanrick_h_: all the bundle validation errors are immediately reported as a GUI notification14:17
rick_h_frankban: ok cool then. 14:17
frankbanbenji, rick_h_: I am not sure the GUI server is the right place for an extensive bundle validation (I also added an XXX), and we can change it later. however ISTM good to have at least a weak firewall before starting all the deployer/ProcessExecutor machinery14:20
rick_h_frankban: yes, we've talked about trying to create a validation library re-used by all entry points for users. (charmtools for proof, deployer for a manual deploy of a bundle)14:21
rick_h_frankban: so long term I think we'd piggy back off the deployer doing validation 14:21
rick_h_frankban: since it's already doing some very limited validating currently14:21
benjiI'm QAing the branch now.14:22
frankbanrick_h_: such a bundle validation library could help also in quickstart14:23
rick_h_frankban: +114:23
bacrick_h_: yes, please14:24
rick_h_frankban: replied with a couple of comments, I didn't see the versions update in the hard coded places in the utils.py ?14:24
rick_h_frankban: ignore me, I did see it. :)14:27
rick_h_even commented on it. /me sips more coffee14:27
frankbanheh, thanks rick_h_, re deployer version, is it ok if instead I add a comment in the hardcoded versions in utils, also mentioning the source branch for that sdist? 14:33
rick_h_frankban: yea, that's cool. Just something so we remember it's our fork 14:33
hatchthis morning I found out that you can stop g+ from showing posts from a community but still be part of the community14:34
rick_h_frankban: since it's not obvious looking at things14:34
hatchbest feature ever14:34
frankbanrick_h_: great, and good point14:34
rick_h_wooo, it's working!14:53
rick_h_will check a failure here in a second. 14:54
hatchhehe oh Chrome15:01
hatch"C:\fakepath\maarten.yaml"15:01
hatchthat's one pretty crazy fakepath :D15:01
rick_h_gary_poster: so the only QA thing is that the two services were stacked on each other. I thought mysql wasn't deployed. :/15:02
rick_h_frankban: got the error when I had invalid constraints, deploy worked on jcastro's bundle 15:02
frankbangary_poster: for when you are back, I think we are hitting this: http://bugs.python.org/issue169233515:02
frankbanrick_h_: http://i.imgur.com/1skzJ3u.gif15:03
rick_h_lol15:03
rick_h_benji: did you also get the layout issue in qa?15:04
benjirick_h_: I had some non-branch related QA issues; redeploying the charm now15:04
rick_h_benji: ah ok15:04
rick_h_benji: do me a favor and don't touch the layout as it comes up then and see if you get the same stacked service issue. I moved it before the relations where set so not sure if it'd do a redraw or anything later in the process15:05
benjirick_h_: k15:05
rick_h_frankban: ok' LGTM'd with qa ok and notes on the qa process. 15:06
frankbancool15:07
benjirick_h_: the services didn't pile up, but I think the canvas scrolled to make it look like they did15:18
rick_h_benji: hmm, ok. 15:20
rick_h_ok, so time to release the hounds?15:20
frankbangary_poster: confirmed the problem is http://bugs.python.org/issue1692335 . easily fixable in more than one ways, let's have a quick call when you are back15:22
frankbanrick_h_: we can be able to add a quick fix to the charm/deployer error handling before releasing the charm. AFAIK the GUI is ready to be released15:24
rick_h_frankban: k15:25
bacuh-oh, searching on charmworld for '~bac' makes for some unpleasantness15:26
rick_h_lol15:26
rick_h_oops15:26
rick_h_bac: requested the log file from webobs and we'll see what hte traceback is15:27
rick_h_although should be able to dupe locally I guess15:27
bacrick_h_: yeah.  i'll look at it.  we'll probably get hate mail from curtis soon.15:28
bacrick_h_: my favorite part of the traceback:  Encountered " <FUZZY_SLOP> "~bac "15:33
bacwho writes an error message like that?15:34
rick_h_bac: heh, not sure. I haven't gotten that far yet15:34
rick_h_bac: I got https://pastebin.canonical.com/100094/ from webops. I don't see the search error, but seeing ingest errors, but things are working so confused. 15:35
rick_h_bac: if you get a sec can you update your bundle and push and make sure it's getting ingested?15:36
rick_h_heartbeat isn't hanging, but wtf is with the connection error in the logs?15:36
bacconnection refused15:37
rick_h_yea, so it's not working. The url isn't good. 15:37
bacport 2464?15:37
baci thought you had a different port for production15:37
rick_h_yea, that's not legit. That's the wrong port. ok, hitting up webops15:38
rick_h_bac: have that RT from the deploy?15:38
rick_h_bac: just the # for it15:38
bacit's in the card15:39
* bac looks15:39
bac6575015:39
rick_h_bac: ah cool, I'll look then. I'll work with webops. If you can update your bundle as our canary in the coal mine it'd be great15:39
bacoh, sure15:40
benjithe "parse string constraints into a dict in deployer" card is not correct -- right? -- and can be deleted15:40
gary_posterfrankban, great.  makes sense.  want to call?15:43
frankbangary_poster: yes thanks15:44
frankbangary_poster: https://plus.google.com/hangouts/_/7ecpir0k4g087sfq5aj9gfos3o?hl=en15:45
rick_h_ugh fail15:49
Makyojujugui call in 1015:50
benjifor all those people who hunger for fewer clicks when attentding the standup, I present your salvation: http://tinyurl.com/gui-hangout-215:51
rick_h_benji: card was in the branch frankban just merged. So it's good and can go away15:51
benjirick_h_: well, I suppose that's one way to look at it, but we didn't do what the card says needs to be done (as best I can tell)15:52
rick_h_benji: ok, well something did it. maybe I'm mising the point, but strings go in, dicts go out to go. 15:54
bactwitter up by $20 since opening.  this will be a "successful" ipo even though the company left all that money on the table.15:54
hatchman I wish I could start a company that's losing money hand over fist and sell it for billions15:55
bacyeah, that second step is the challeng15:57
bacs/challeng/challenge/15:57
hatchI think the steps are15:57
hatchdevelop product15:57
hatchhype hype hype hype15:57
hatchipo15:57
hatchhype > profits15:57
bactwitter was worse.  develop awful, crashy product.  hype.  crash more.  hype.  hire some decent engineer. hype. profit.15:58
Makyojujugui call in 115:59
hatchlol15:59
hatchjcsackett: hey you around?16:16
MakyoRunning to get prescription. Bringing Air with, there's a coffee shop if it takes a while.16:21
hatchrick_h_: the reason there is no tests in my branch is because it's already tested by other tests16:22
hatchjust fyi16:22
frankbanhatch: a live test in a real env deploying services and ingested bundles could really help. the development charm URL is cs:~juju-gui/precise/juju-gui16:23
hatchfrankban: ok cool - are we doing to be doing a charm release along side the gui release then?16:23
frankbanhatch: so basically is juju deploy + expose the charm above, and "juju set juju-gui juju-gui-source=lp:juju-gui"16:23
frankbanhatch: yes16:24
hatchok cool16:24
frankbanhatch: we always have to do that, because GUI releases are now included in the charm16:24
jcsacketthatch: i am.16:24
rick_h_hatch: hah, ok. Heading that one off at the pass 16:24
hatchthat's how I did the demo at the conf on Tuesday - worked well then haha16:24
frankbanhatch: next demo use quickstart ;-)16:25
hatchjcsackett: so the apache front end has stalled, mostly because I Dont' have enough time16:25
hatchthat's what gary said haha16:25
jcsacketthatch: dig.16:25
frankban:-)16:25
jcsacketthatch: i'm happy to explore that.16:25
hatchjcsackett: tbh I'm more interested in a nginx front16:25
hatchbut I don't know if we have an nginx charm....16:25
hatchwell not a promoted one anyways16:26
rick_h_damn chunk mismatch16:26
jcsacketthatch: yeah, doesn't look like there's a reviewed one of those.16:26
hatchbut we could have both so nothing saying we coudln't do the apache one first then do nginx after16:27
jcsacketthatch: i'm interested in apache b/c we *do* have a reviewed one of those, so that's a better story.16:27
hatchright16:27
hatchI'm going to be pretty swamped with life stuff at least until next weekend16:27
jcsacketthatch: indeed, i totally think we can have both. there's an issue for supporting both.16:27
hatchoh?16:27
jcsackettyeah, this weekend is sort of booked for me, but i have some evenings free.16:27
jcsacketthatch: https://github.com/hatched/ghost-charm/issues/1116:28
hatchohhh 'issue' meaninig 'ticket'16:28
hatchI thought you meant there was a problem with supporting both :)16:28
rick_h_hatch: come on man, you wrote a new module. How can 'there are already tests for this'?16:28
hatchbecause the import code is already tested, all I did was move it out16:29
hatchI could repurpose the tests to point to the module directly16:29
hatchif you'd like16:29
jcsacketthatch: no, just that the desired thing is already captured.16:29
hatchis there a juju story for deploying services to a single machine then splitting them off that machine later?16:30
hatchso we could have a bundle to deploy apache/ghost/mysql to a single machine then give them the ability to break one out if needed16:30
hatchmaybe the story there is simply to deploy another mysql/apache/ghost and then change the relationship16:32
lucagary_poster: do you have a hangout?16:33
gary_posterluca, alejandraobregon what hangout?16:33
gary_poster:-)16:33
gary_posterluca https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.t3m5giuddiv9epub48d9skdaso16:34
hatchjcsackett: do you have a live site up with the ghost charm yet?16:36
* bac lunches16:37
jcsacketthatch: no, but only for lack of time.16:37
hatchfrankban: so I `juju deploy cs:~juj-gui/precise/juju-gui` and it's taking forever to get back to the console...16:38
hatchis this normal?16:38
hatchI guess ec2 could be taking forever to respond16:39
hatchyeah juju status is hanging too16:39
rick_h_hatch: yea, welcome to the real cloud16:39
hatchglad this didn't happen during my demo, that would have sucked haha16:40
rick_h_hatch: it'll hang until bootstraping is done and a machine is brought up/setup16:40
rick_h_hatch: ok, one qa feedback on your branch16:42
hatch"""ERROR cannot log in to admin database: unauthorized mongo access: auth fails"""16:42
hatchthat's an odd one16:42
rick_h_hatch: :(16:42
hatchrick_h_: re the merge files - for some reason the parser does not pick up those deps, I have it on my list to fix that parser but that's been on the list for 6 months :)16:43
rick_h_hatch: rgr16:44
hatchIt's using the loader to generate the deps16:44
hatchso it SHOULD pick those up16:44
hatchbut who knows16:44
rick_h_hatch: all good, the only thing I'd love to see changed pre-land is the titles for tooltips16:45
hatchI've spent all of 20s looking at it16:45
hatchyeah that's a good idea16:45
hatchoh rick_h_ the sollution to that error message was to use `sudo` ...16:46
hatchI think I may have a bug on my system16:46
frankbanrick_h_: what shoudl I write in the tests requirements.pip to make it install the stuff in deps?17:10
rick_h_frankban: it's inthe pip command not in the requirements.txt file17:11
rick_h_err, requirements.pip17:11
rick_h_frankban: it's that snippet I linked from charmworld's makefile17:11
frankbanwe already have --find-links17:11
frankbanrick_h_: so, just the name of the packaged in the requirements17:11
rick_h_frankban: right, but it'll still try to go online if it doesn't see it, or if the version is a > XXX ad such17:11
rick_h_frankban: the name==version17:12
frankbanrick_h_: cool, I'll try17:12
rick_h_frankban: --no-index --no-dependencies are the other two options to force it to use offline-only cache17:12
frankbanrick_h_: I don't think we want offline only cache. we want all the other packages (e.g. mock, selenium) to be retrieved from pypi17:13
hatchnananananananananananananaannanana qa man.....qa man......qa man17:14
rick_h_frankban: sure, but you were talking of splitting those into a different file and could be a different pip install command?17:14
rick_h_frankban: but maybe that's step #2 I guess17:14
rick_h_frankban: I just know that if you don't go full-offline-this-directory-only-dammit mode eventually you'll hit an inconsistent version between one and another17:15
frankbanrick_h_: yeah, what if the dependency is unversioned? who takes precedence?17:16
rick_h_frankban: if it's not versioned then that's when it'll go to the net anyway. pip isn't really great at this 'no network' idea17:17
rick_h_frankban: if I recall correctly that is. 17:18
frankbanrick_h_: hum? ok then, and FYI, we are going to also have a customized jujuclient tarball in the charm 17:18
rick_h_frankban: so we use'd pip freeze to generate an initial list of deps + versions17:18
hatchman our app rocks!17:19
rick_h_hatch: lol17:19
rick_h_yea, it's fun to see how excited martin is getting with this feature. A bit scary how everyone's handing out comingsoon urls to potential customers :/ but cool17:20
hatchyeah - mark did that in his keynote haha17:20
hatchmaybe we should put a warning on comingsoon :)17:21
rick_h_abentley: ping, working on an upgrade to the charm on production and jjo and I are trying to figure out if the lp_credentials change will bite us in any way. I see it's a config value, and a file is written, but not seeing anyone using that file?17:31
abentleyrick_h_: lp_credentials should be in use.  Looking...17:32
rick_h_jcsackett: do you know about that? ^ I see it was your branch mentioning a job that needs it but greping for lp_cred or charmbot in charmworld brings up no hits17:36
hatchgary_poster: https://bugs.launchpad.net/juju-gui/+bug/1249026 https://bugs.launchpad.net/juju-gui/+bug/1249028 and https://bugs.launchpad.net/juju-gui/+bug/124903017:36
_mup_Bug #1249026: On a real environment destroying a pending service throws console error <juju-gui:New> <https://launchpad.net/bugs/1249026>17:36
_mup_Bug #1249028: Hovering over the service with your mouse should turn the cursor to a pointer <juju-gui:New> <https://launchpad.net/bugs/1249028>17:36
_mup_Bug #1249030: After destroying a related service the GUI still shows the relation. <juju-gui:New> <https://launchpad.net/bugs/1249030>17:36
gary_posteron call will see soon17:38
hatchsure np https://bugs.launchpad.net/juju-gui/+bug/124903317:40
_mup_Bug #1249033: Long charm names cause the related charms layout to break in fullscreen <juju-gui:New> <https://launchpad.net/bugs/1249033>17:40
hatchhttps://bugs.launchpad.net/juju-gui/+bug/124903917:47
_mup_Bug #1249039: Exporting from real environment exports juju-gui as well <juju-gui:New> <https://launchpad.net/bugs/1249039>17:47
hatchhttps://bugs.launchpad.net/juju-gui/+bug/124904217:51
_mup_Bug #1249042: Destroying a service doesn't give any indication that the machine is sticking around <juju-gui:New> <https://launchpad.net/bugs/1249042>17:51
rick_h_hatch: that one is a long running juju bug/discussion17:51
hatchrick_h_: the gui bug is that we don't tell the user17:51
hatchso if they are a gui user then they will never know17:51
hatchwe should say 'hey btw this machine is still there'17:52
hatchbut feel free to add input to the ticket17:52
rick_h_*cough* machine view *cough* but yea, understood17:52
hatchI'm just creating them - there may be duplicates17:52
hatch:)17:52
gary_posterok looking now17:56
hatchcool17:59
gary_posterhatch ack thanks.  Please let me know what you think about the following.  #1 sounds nasty but I suspect it is longstanding and not a release blocker.  #2 Arguable (since you can drag the service) and not a blocker.  #3 same as #1.  #4 same as #1 but sounds easy to fix. :-P. #5 uh...working as designed?  :-) not a blocker IMO.  #6 yeah that's on the "wouldn't it be nice to fix this" tasklist17:59
gary_posterSo, so far, either known or non-blockers.  Agree?17:59
jcsackettrick_h_: the code that rolled back uses that file.18:00
rick_h_jcsackett: ah!18:00
hatchyep I wouldn't consider any of these blockers unless of course someone already has an idea on how to fix #1/318:00
jcsackettthe presence of that file however shouldn't cause any problems.18:00
gary_posteryeah18:00
rick_h_jcsackett: ok, so it's safe for me deploy the charm then it sounds like18:00
jcsackettit's been running on staging safely.18:00
jcsackettabsent any code touching it.18:00
rick_h_jcsackett: awesome18:00
rick_h_jcsackett: so the charm was updated on staging just not production?18:00
hatchI do like that I can go `juju destroy-machine 1 2 3 4 5 6 7` :D18:00
jcsackettrick_h_: correct, as far as i knwo.18:01
jcsacketts/knwo/know/18:01
rick_h_jcsackett: cool, thanks18:01
hatchgary_poster: I'm just going to run a real bundle import on a real env then It should be QAOK18:01
gary_posterawesome!  thanks hatch18:01
gary_posterI need a quick lunch (and to decompress :-P) and then will be right back18:02
hatch:)18:02
hatchMakyo: are you around?18:12
MakyoYis18:12
hatchcool ok quick q18:13
hatchI exported a bundle from my real env18:13
hatchdeleted the gui from it18:13
hatchthen re-imported it (after emptying out the old stuff)18:13
hatchboth services have x/y coords18:13
hatchbut they were placed ontop of eachother18:13
hatchexactly ontop of eachother18:13
hatchso you couldn't even tell they were there18:13
hatchhttps://gist.github.com/hatched/735927118:14
hatchhere is the resulting bundle that was imported18:14
hatchis there anything wrong with that bundle?18:14
hatchso my question is....is this a bundle import or export issue :)18:14
MakyoWorks for me?18:16
hatchthis is on a real env...18:16
hatchI can destroy and try again see if I can reproduce18:16
rick_h_hatch: I had this happen with one of jcastro's bundles in a real env as well. wordpress and mysql were stacked and couldn't see both18:16
hatchok so it's definitely a bug thanks18:16
MakyoImport, then.18:16
hatchok cool18:17
hatchwill file18:17
hatchgary_poster: one more https://bugs.launchpad.net/juju-gui/+bug/1249051 also not a blocker18:20
_mup_Bug #1249051: Importing a bundle stacks services regardless of x/y annotations on real env <juju-gui:New> <https://launchpad.net/bugs/1249051>18:20
hatchall-n-all I think this is a pretty darn stable release18:20
gary_posterthat's not ideal, but we can live with it for now18:23
gary_posterthank you very much hatch18:23
hatchno problemo18:25
frankbanguihelp, python revievers: I need one review + QA for a really really nice to have for the release: https://codereview.appspot.com/2300004318:25
gary_posterrick_h_, you happen to be available ^^^? about to have my call with benji18:26
gary_posteror bac?18:26
rick_h_gary_poster: working with webops on charm upgrade atm. I can look in a few if all goes well18:26
gary_posterok thank you rick_h_ 18:26
frankbanrick_h_: thanks18:27
bacfrankban: i can now18:30
bacrick_h_: ^^18:30
rick_h_bac: thanks, mjc upgrade is process and wheee18:30
baci don't know what that means18:31
rick_h_bac: upgrading the charm for prodstack mjc to fix the port/ini issue18:31
rick_h_bac: so thanks for looking at frankban's branch18:31
rick_h_abentley: jcsackett so charm is upgraded. Because that's a new config there's some hackery needed to get the initial value going. 18:33
rick_h_abentley: jcsackett just a heads up for when that code comes back and things need to be updated18:34
hatchstepping out for lunch, bbl18:37
bacfrankban: code looks good.  QA now18:38
frankbanbac: great thanks18:39
jcsackettrick_h_: what hackery? the default value should be an empty string, which rights out to an empty file.18:41
jcsacketts/rights/writes/18:41
rick_h_jcsackett: so the first time it runs the default doesn't get written I guess? If another charm deploy happens between now/then it should be ok I think18:42
rick_h_jcsackett: I was pointed at #124484118:42
_mup_Bug #1244841: support atomic upgrade-charm --config var=val ... <canonical-webops> <config> <upgrade-charm> <juju-core:Triaged> <https://launchpad.net/bugs/1244841>18:42
jcsackettrick_h_: so just no file gets written?18:42
rick_h_jcsackett: right, my understanding is that right now the file doesn't exist18:42
jcsacketthuh.18:43
rick_h_jcsackett: I'll verify when I check the log in a sec18:43
rick_h_jcsackett: but it's a limitation of juju upgrading a charm when the upgrade contains a config value that didn't exist before is my understanding18:43
jcsackettwell, if it's going to do something unexpected, "nothing" is one of the better options. :p18:43
rick_h_:)18:43
jcsackettrick_h_: oh, i see; this would have been hidden on staging b/c we did a juju set right after to load the credentials.18:44
jcsackettand we haven't updated the IS docs b/c right now there's no need to set that option/write that file.18:44
rick_h_jcsackett: hmm ok, well that didn't come up as an option in talking. I'm not following 100% I guess. Anyway, heads up there be dragons I guess. 18:45
jcsackettrick_h_: there be no dragons.18:45
rick_h_ok, then awesome ness18:45
jcsackettto get the file to write out, you need to do `juju set charmworld lp_credentials=$BIG_DAMN_STRING`18:46
jcsackettand then on config changed it'll write it out.18:46
jcsackettbut, like i said, IS documentation isn't updated b/c the failure mode is do nothing. so actually, this is everything running as expected. :-)18:47
rick_h_gary_poster: charm is all set, charmworld all good. What's next for release I should look at?18:59
gary_posterrick_h_, ack 1 sec19:02
gary_posterrick_h_, I've been unable to make changelog because of calls etc.  you could take that over.  docs/process.rst has a recommeneded way to generate for a release.  This is what I have so far:19:03
gary_posterrick_h_, http://pastebin.ubuntu.com/6377908/19:05
rick_h_gary_poster: rgr, looking 19:05
gary_posterrick_h_, thank you.  card is in maintenance but you can drag down to bundle and claim19:06
gary_posterbac, in hangout19:06
jcsackettrick_h_: what rev did you release for production charmworld? cleaning up our kanban.19:15
rick_h_jcsackett: 8419:15
rick_h_jcsackett: oh, charmworld was 45019:15
gary_posterfrankban, you are still here! are you landing "Improve bundle deployment error handling" or should we?19:20
frankbangary_poster: waiting bac's QA, and then I'll land it19:21
gary_posteroh cool.  thank yo ufr19:21
gary_posterthank you frankban, that is ;-)19:21
gary_postertab completion on ufr did not work as expected19:21
frankban:-)19:22
bacfrankban: done.  all good.19:24
frankbanbac: thanks, landing19:25
hatchman our traffic is nuts19:42
frankbandone, have a nice evening, and a nice release!19:44
gary_posterthanks frankban !19:44
gary_posterrick_h_, we are ready for release, and you have the charm in.  you doing it, or shall we look for other volunteers?19:49
* hatch hides19:49
rick_h_gary_poster: I'm working through the gui release notes. 19:49
gary_posterawesome thanks rick_h_ !19:49
rick_h_gary_poster: I've got the changelog, tests, xz and working on how to open it up to qa it19:49
hatch*phew*19:49
gary_posterI see  you hatch! :-)19:49
rick_h_gary_poster: if someone wants to do the charm +1 from me :)19:49
* rick_h_ looks at hatch 19:50
hatchis there release notes? I've never done the charm19:50
gary_posterrick_h_, ack, we do that after the tarball release though19:50
rick_h_gary_poster: rgr, ok. know off the top of your head the tar flags for xz files?19:50
rick_h_ah, -J19:50
gary_posterrick_h_, use tar xvaf19:50
hatchrick_h_:  https://xkcd.com/1168/19:50
gary_posterrick_h_, a means "use the file extension to figure it out, buster"19:51
hatchlol19:51
rick_h_gary_poster: ah, good stuff19:51
rick_h_gary_poster: ok, plodding through notes. Man this is a looong list :)19:51
gary_posterrick_h_, :-) on the bright side a lot of the qa has been done19:51
hatchrick_h_: I found using the trunks commit notes to build the changelog19:51
hatchis the best way19:52
rick_h_hatch: yea, got that part. Wasn't that bad since it was only around 40 commits (thought it'd be bigger/longer)19:52
rick_h_and most summarize as "Welcome to bundles"19:52
hatchif it was on git it would be :) haha19:52
hatchso I'm up for whatever right now...if anyone has any tasks they need completed19:53
rick_h_oh bah, the charm is part of this. doh!19:57
gary_posteryou can pass that off rick_h_ .  :-)19:59
gary_posterhatch, I suggest tackling one of the bugs you found, or working on one of https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AtC9etoysSQldDQxMVdmTDB4dm1XXzA0NFlLSUQ4Mmc#gid=019:59
* rick_h_ looks at how far this list goes19:59
gary_posterpreferring ones near the top :-)19:59
hatchgary_poster: alrighty20:00
rick_h_gary_poster: got it, should go through this at some point and looks like I'm mostly through it so far20:00
gary_posterrick_h_, yeah you are getting near-ish to the end. :-)20:00
gary_posterWe might be able to remove some of the more paranoid bits at some point20:00
rick_h_running charm tests failed due to dep building. Missing apt-pgk/hashes.h file. Looking for missing system dep now20:09
hatchthat's an odd one20:10
hatchI've never seen that before20:10
rick_h_looks like libapt-pkg-dev20:10
rick_h_re-running make lint from a clean charm env20:11
rick_h_hmm, that's in the sysdeps make target. So it's a new step in the process.20:12
rick_h_ok, take 320:13
hatchthird time is the.....charm20:14
hatch...see what I did there...20:14
rick_h_sorry, wasn't looking. What was that again?20:14
hatchlol u suck20:14
hatchso after next week's talk on Go/JS I can get started on learning Python well enough to contribute - any good resources for learning about the ecosystem?20:15
rick_h_go to pycon, it's in CA :P20:16
hatchCalifornia?20:16
hatchohh Canada20:16
hatchMontreal20:16
rick_h_nope, bit more north20:16
hatchwow...it's 8 days long20:16
hatchmake that 920:17
rick_h_well tutorials, conference, and sprints. I go for the conference and sprints myself20:17
rick_h_ok, make test now in progress20:17
rick_h_bah, now it's making fun of my environments.yaml file20:18
hatchso they make you sign up for an account before telling you how much it is.....fail20:18
rick_h_wtf, it's complaining about a differnet environment20:19
hatchoh I was looking at the wrong place20:19
* hatch failed20:19
rick_h_yea, reminds me I need to sign up asap. Guess I'll be doing that tonight20:21
hatchflights to Montreal cost me the same as it does to fly to SFO haha20:21
rick_h_gary_poster: crap, getting functional test failures20:29
rick_h_oh man, it wants FF in my lxc?20:29
rick_h_grrrr20:30
* rick_h_ sees what happens when you try to run firefox in a headless lxc20:30
hatchunicorns happen20:30
gary_posterrick_h_, so you are addressing?  If not, and it is too difficult, we can find other options20:33
rick_h_gary_poster: working on it20:33
gary_posterk20:33
rick_h_gary_poster: installing firefox now, pulling all of X and lovelyness so taking a few20:33
gary_posterheh20:33
gary_posterk20:33
rick_h_will have to restart functional tests, destroying old env now20:33
rick_h_now I know why hatch ducked for cover :P20:38
hatchhaha20:38
hatchI've never actually done a charm release20:38
rick_h_still running functional tests, running on test_branch_source forever20:56
rick_h_or 10minish20:56
rick_h_if forever is a bit vague :)20:56
hatchhmm.....delete 100+ LOC, no test failures......win?20:57
rick_h_heh, that's what we call 'test coverage time'20:57
hatchohh the ones which would have failed were already skipped20:58
hatch*phew*20:58
hatchI was a little scared there haha20:58
rick_h_python /home/rharding/bin/juju-test --timeout=120m -v -e ec2 --upload-tools 20-functional.test20:58
rick_h_120minutes?!20:58
hatchjust incase lol20:58
rick_h_well, 20/120 I guess20:58
rick_h_this is the last step before I can merge/push stuff! grrr go go go ec2 go21:00
gary_posterrick_h_, if you have to go, and you have manually qa'd the charm with the new release, I would be fine with you making the release.  Francesco ran the tests earlier today.  All you've done is change the tarball.21:04
rick_h_gary_poster: working on it. Didn't try to deploy it as a local charm yet since I put it all in a tmp dir and I can't figure out how to get juju to deploy it unless it's in a directory named precise. 21:09
gary_posterrick_h_, make deploy.  21:10
hatchgary_poster: do you want me to mark tasks complete somehow on this spreadsheet?21:10
rick_h_oh duh21:10
gary_posterrick_h_, http://jujugui.wordpress.com/2013/10/15/if-you-want-to-run-a-custom-gui/21:10
gary_posterexcept all you have to do is make deploy in this case :-)21:11
gary_posterhatch, mark as Done in spreadsheet21:11
hatchcool21:11
gary_posterin status column hatch21:11
rick_h_gary_poster: trying, getting an error out of make deploy about username/pass/project-name. 21:11
gary_posterrick_h_, :-( worked for me last week21:12
rick_h_gary_poster: ve to do is make deploy in this case :-)21:12
rick_h_bah21:12
rick_h_bad paste21:12
gary_posterrick_h_, you can push charm to ~juju-gui21:12
rick_h_http://paste.mitechie.com/show/1068/21:12
gary_posterrick_h_, then test21:12
gary_posterrick_h_, and if that is ok then merge to ~juju-gui-deployers21:12
gary_posterdoes that make sense?21:13
hatchwoah lbox just exploded on me...21:13
rick_h_gary_poster: yea, makes some sense. I'd like to not have this make deploy error before I go pushing. 21:13
gary_posterrick_h_, never seen :-(21:14
gary_posterthat traceback I mean21:14
gary_posterrick_h_, I will try locally21:15
rick_h_gary_poster: rgr21:15
gary_posterrick_h_, do you want to push up your charm somewhere or if I test juju-gui trunk is that sufficient?21:16
rick_h_gary_poster: I'll push it up to my home I guess. I'd be curious to start with trunk21:16
hatchhas anyone seen this when lboxing? sphinx error? https://gist.github.com/hatched/ad0695b48078503825f721:16
rick_h_hatch: no, but something about getting version from release? Is that our own code? 21:18
hatchfresh trunk with js and handlebars template removal21:18
gary_posterrick_h_, running make deploy from ~juju-gui trunk21:19
rick_h_bah, can I not push this without it going into the store and ingested? /me tries to recall how +junk worked21:19
gary_posterI think bzr just blew up? np?21:19
gary_posterno21:19
gary_posterjuju status21:19
hatchgary_poster: was that to me?21:20
gary_posterrick_h_, you push it but don't call it trunk and it will not be ingested21:20
gary_posterhatch no sorry21:20
rick_h_gary_poster: ah, ok. 21:20
gary_posterhatch dupe on trunk.  investigating.  I think it has to do with CHANGES file...21:23
hatchgary_poster: sorry landing a fix now21:23
hatchI think21:23
rick_h_gary_poster: pushed it to lp:~rharding/+junk/charmers-trunk as it is now. Tests are at 42min run time. 21:23
hatchit's rick_h_'s fault21:23
hatch:P21:23
gary_posteroh cool hatch thanks.  fix in CHANGES, yeah?21:23
rick_h_bah, did I mess up the changes? i tried to dupe the system21:23
hatchgary_poster: yeah the 'unreleased' was missing the trailing :21:23
hatchat least I THINK that's the issue21:24
gary_posterhatch sounds right thanks21:24
rick_h_ah, crap. thanks hatch. Sorry you hit that21:24
hatchpretty fragile system21:24
hatchrick_h_: ahh no problem :)21:24
hatchat least I could do enough python to debug it haha21:24
hatchrick_h_: If you have any ideas on how we can streamline our release process I'd be interested to hear/implement them21:25
rick_h_hatch: taking notes now on the things I've hit along the way. As for streamlining, just tring to get it to work once before I do that. :)21:26
hatch:)21:26
hatchgary_poster: rick_h_ fyi the missing : was the culprit21:28
rick_h_hatch: haha, with one character I can bring down your whole branch!21:28
* rick_h_ pockets that away21:28
hatchlol - maybe that means that we should require an lbox on the 'CHANGES' changes ;)21:28
hatchbecause that got into trunk because of the step that says 'push to trunk'21:29
gary_posterwe effectively do IIRC21:29
gary_posterit's that the deployment instructions do not run lbox21:29
hatchhttps://codereview.appspot.com/23240043/ if anyone is avail for a quick review/qa21:29
hatchright21:29
gary_posterrick_h_, I have trunk up with make deploy21:29
gary_postertrying21:30
rick_h_gary_poster: :( ok I guess. I can't find any reference to "project-name" that was in my traceback. 21:30
gary_posterrick_h_, ok.  I will kill this one and merge your branch and do it again...21:30
rick_h_gary_poster: k21:31
rick_h_test run almost hitting an hour. Is that normal from previous deploys? 21:31
gary_posterrick_h_, yes, think so21:32
rick_h_ok21:32
gary_postercharms are painful beasts21:32
gary_posterto test21:32
rick_h_yea, understood. /me does a s/test_/wait_ so it feels more natural :)21:32
gary_poster:-)21:32
gary_posterhatch, on it21:35
gary_posterhatch, uh, I'm struggling to figure out what to do to qa. :-P21:40
gary_posterthe app seems to work? :-)21:40
hatchgary_poster: deploy two services with the same name21:40
hatchyou should get a notification21:41
hatchsorry :)21:41
gary_posteroh ok21:41
hatchI found if I removed too much - notifications would stop happening :)21:41
huwshimiMorning21:41
hatchmornin21:41
gary_postermorning21:41
gary_posterhatch QAOK21:41
hatchthanks21:42
gary_posterrick_h_, site is up; trying21:42
gary_posterrick_h_, WFM :-)21:43
gary_posterrick_h_, ship it (~_juju-gui and ~juju-gui-charmers)21:43
gary_posterrick_h_, https://ec2-54-205-201-186.compute-1.amazonaws.com/21:43
rick_h_gary_poster: ok, cool then21:43
huwshimigary_poster: I'm happy to have our calls anytime from now.21:44
gary_posterthank you huwshimi!  maybe in a few minutes, then?  would be good21:44
huwshimigary_poster: It looks like we've both changed daylight savings recently...21:44
huwshimigary_poster: Sure!21:44
gary_posterheh, that's right I forgot21:44
gary_postercool21:45
hatchshall I join or is this just 1:1?21:46
rick_h_ok, both branches of the charm pushed, waiting to check on the new revs hitting21:46
gary_posterawesome thanks rick_h_ .  I have a blog post and email waiting in the wings.21:51
rick_h_gary_poster: rgr21:51
rick_h_gary_poster: http://paste.mitechie.com/show/1069/ are my notes for the deploy steps I hit. Done for today, but will look at tweaking those as follow up21:52
huwshimiAre we releasing now!?21:52
rick_h_huwshimi: yes21:52
huwshimiwoah! Nice.21:52
hatchhuwshimi: this way - if it breaks you can fix it while we are out drinking21:52
hatchmohohahaha21:52
huwshimiBoo21:52
gary_poster:-)21:53
huwshimihatch: It's my Friday night before it's yours :P21:53
gary_poster:-)21:53
hatchdarn....21:53
gary_posterthank you rick_h_ !21:53
hatch+1 internets to rick_h_21:54
rick_h_thanks for hte patients. Waiting for ingest at top of the hour and then running away21:56
gary_posterah cool21:56
gary_posterhuwshimi, hatch, https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.dd77sn7kjl6unba21lutdr0p70 when ready?21:57
huwshimigary_poster: on way21:57
rick_h_ok, ingested https://jujucharms.com/precise/juju-gui-78/22:05
rick_h_night all22:05
gary_postercongrats!  night!22:10
huwshimirick_h_: Night Rick, thanks!22:11
rick_h_woot! and with the new juju fixing lxc deployed a bundle from the new charm in lxc. only issue is that stacked service block problem yay!22:30
rick_h_<3 having lxc powers back22:31
=== gary_poster is now known as gary_poster|away

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