| === kadams54 is now known as kadams54-away | ||
| === kadams54-away is now known as kadams54 | ||
| === kadams54 is now known as kadams54-away | ||
| === lazyWeekend is now known as lazyPower | ||
| === tvansteenburgh1 is now known as tvansteenburgh | ||
| hatch | of course I just updated ghost in my charm lastnight and then they release a new version haha | 14:24 |
|---|---|---|
| rick_h_ | lol | 14:25 |
| hatch | rick_h_: was it windy there this weekend? https://plus.google.com/117915091655057728378/posts/LoANxm9gF8x?pid=6077064898626396242&oid=117915091655057728378 | 14:37 |
| hatch | :) | 14:37 |
| rick_h_ | hatch: heh, little bit but not too bad. wasn't windy enough to get all these leaves out of my yard https://www.flickr.com/photos/7508761@N03/15075217444/ | 14:38 |
| hatch | haha oh wow - burn em all!!!! | 14:42 |
| rick_h_ | too wet and smoky | 14:43 |
| rick_h_ | we had snow on halloween | 14:43 |
| hatch | fire isn't big enough ;) | 14:43 |
| hatch | ohh haha nice | 14:43 |
| hatch | there is snow on the ground here now but on haloween it was all gone | 14:43 |
| hatch | this stuff also probably won't stay | 14:43 |
| hatch | jujugui call in 7 | 14:53 |
| jcsackett | hatch: think your clock is probably wrong. | 14:53 |
| rick_h_ | hatch: :) sorry we finally moved TZ so in 67 min | 14:53 |
| hatch | oh the meeting moved too? | 14:54 |
| jcsackett | hatch: it appears to be pinned to EST/EDT. :p | 14:54 |
| hatch | well look at you commies and your TZ changes | 14:54 |
| rick_h_ | :) | 14:54 |
| bac | i'm with you hatch | 14:54 |
| bac | meetings should be UTC | 14:55 |
| hatch | ooorah | 14:55 |
| * bac has a snack | 14:55 | |
| jcsackett | there's certainly an argument for that. | 14:55 |
| bac | jaasteam: speaking of timezones, i stayed at UTC-4, so i will begin and end an hour ahead of US/East | 14:55 |
| hatch | I'm also always -6 | 14:56 |
| bac | well, it wasn't a personal decision, but my locale does not switch | 14:56 |
| frankban | bac: other channel | 14:56 |
| kadams54 | hatch: how are things going? | 15:07 |
| hatch | kadams54: good - added services was pushed in favour of fixing this bug I'm on atm | 15:09 |
| hatch | unfortunately there were 0 tests for ambiguous relations :/ so I need to write those | 15:09 |
| kadams54 | rick_h_: any suggestions on what I should be working on? | 15:09 |
| hatch | kadams54: probably either updating the More menu (ask luca) | 15:10 |
| hatch | or updating the icons (also ask Luca) | 15:10 |
| kadams54 | luca__: ^ | 15:11 |
| rick_h_ | kadams54: +1 to hatch's stuff. The icons might not be ready until later in the day so I'd suggest the masthead | 15:12 |
| rick_h_ | kadams54: https://bugs.launchpad.net/bugs/1388856 | 15:12 |
| mup | Bug #1388856: The "more" tab in the masthead needs to be updated <juju-gui:New> <https://launchpad.net/bugs/1388856> | 15:12 |
| kadams54 | OK, will do. | 15:12 |
| luca__ | hatch: kadams54 I filed a but for the masthead and Spencer has just finished the icons for AS | 15:14 |
| kadams54 | Great. | 15:15 |
| hatch | http://cdn.instructables.com/F7Z/J9CH/HJKC61PL/F7ZJ9CHHJKC61PL.LARGE.jpg | 15:15 |
| hatch | rick_h_: new bug wrt ambiguous relations https://bugs.launchpad.net/juju-gui/+bug/1388877 just a styling issue though | 15:29 |
| mup | Bug #1388877: ambiguous relation menu not styled properly <juju-gui:New> <https://launchpad.net/bugs/1388877> | 15:29 |
| rick_h_ | hatch: looking | 15:30 |
| rick_h_ | hatch: ok, adding the bug. | 15:32 |
| hatch | I am not sure we have any designs for it - but I figure anything would be better ;) | 15:32 |
| rick_h_ | hatch: yea | 15:32 |
| kadams54 | guihelp: Looking for reviews and QA on a small change (all HTML, no JS/CSS): https://github.com/juju/juju-gui/pull/634 | 15:41 |
| hatch | kadams54: so the tooltop positioning issue can be fixed with .notifier-box.bundle { margin: -5px 0 0 227px; } (from ant__) | 15:42 |
| kadams54 | Great. I'll get that in next. | 15:42 |
| kadams54 | Looks like it's going to be a slew of small branches for me today :-) | 15:42 |
| hatch | no mo survey? :( | 15:42 |
| rick_h_ | hatch: no | 15:43 |
| hatch | we did get some valuable input from there in the past | 15:43 |
| hatch | I'm guessing nothing has come in a long time? | 15:43 |
| rick_h_ | hatch: to be honest, I'm not sure how to look in there :) | 15:43 |
| hatch | lol!!! | 15:43 |
| hatch | might be worth an email to the g-man :) | 15:43 |
| hatch | gary, not the fbi | 15:43 |
| rick_h_ | I think I might have it in a handoff note. I'll look some day when I have a little bandwidth | 15:43 |
| hatch | haha | 15:44 |
| hatch | :) | 15:44 |
| hatch | kadams54: is it Jujucharms or JujuCharms? Juju Charms? also http://www. or just http:// ? | 15:44 |
| rick_h_ | hatch: kadams54 I vote just jujucharms.com for now. | 15:45 |
| rick_h_ | and it's https | 15:45 |
| * rick_h_ adds that to the pr | 15:46 | |
| hatch | if it's https then the top right button also needs to be updated | 15:46 |
| hatch | it's only http | 15:46 |
| kadams54 | I just followed what the bug specified as far as "Jujucharms". That said, "Juju Charms" makes more sense to me. | 15:46 |
| rick_h_ | jujugui call in 7 kanban please | 15:55 |
| kadams54 | guihelp: Another small QA/review: https://github.com/juju/juju-gui/pull/635 | 16:00 |
| kadams54 | hatch: tooltip offset ^ | 16:00 |
| kadams54 | rick_h_, hatch: So to clarify, the link should be: <a href="https://jujucharms.com/docs" target="_blank">jujucharms.com</a> ? | 16:21 |
| rick_h_ | kadams54: there's two links in that menu, jujucharms and the juju docs | 16:22 |
| rick_h_ | kadams54: the first should be https://jujucharms.com and named "jujucharms.com" I think | 16:22 |
| kadams54 | OK | 16:23 |
| rick_h_ | kadams54: the second should be https://jujucharms.com/docs and called "Juju Docs" | 16:23 |
| kadams54 | Got it | 16:23 |
| kadams54 | rick_h_: OK, changes pushed. One last check before shipping? | 16:25 |
| rick_h_ | kadams54: looking | 16:25 |
| kadams54 | Bah, missed the change of "Read the documentation" to "Juju Docs" | 16:26 |
| rick_h_ | kadams54: :+1: | 16:26 |
| rick_h_ | that's ok | 16:26 |
| rick_h_ | was that from luca? | 16:26 |
| kadams54 | No, that was from you, just now :-) | 16:26 |
| rick_h_ | sorry, I was going off memory and might have steered you wrong :) | 16:26 |
| rick_h_ | kadams54: so ignore that one :) | 16:26 |
| kadams54 | OK :-) | 16:26 |
| kadams54 | Shipping | 16:26 |
| hatch | 4 more! | 16:35 |
| hatch | its no wonder there were no tests for this before it was split up heh | 16:35 |
| === fabrice is now known as fabrice|family | ||
| hatch | Clicking the icon in the launcher bar on Ubuntu so that it shows all the windows tiled is awesome - I use it so that I can watch both the console and tests run at the same time :) | 17:18 |
| hatch | one...last....test | 17:32 |
| hatch | Makyo: u around? | 17:33 |
| Makyo | hatch, sure | 17:33 |
| hatch | Makyo: any idea how the heck I'm supposed to unit test this method? https://gist.github.com/hatched/e58e95a9aa5c6fe234f5 | 17:34 |
| hatch | I of course could just stub the methods out and 'fake it' | 17:34 |
| hatch | not sure if that's the best way though | 17:34 |
| Makyo | That might be the only way, unless you want to do it in test_environment_view and have the whole canvas stubbed out. | 17:35 |
| hatch | yeah..no | 17:35 |
| hatch | :) ok thanks | 17:35 |
| === kadams54 is now known as kadams54-away | ||
| hatch | jujugui I need qa and reviews for https://github.com/juju/juju-gui/pull/636 plz and thanks | 18:01 |
| hatch | rick_h_: want to chat about the card in tracking? Should I hop on the other bug in urgent? Added Services? | 18:02 |
| rick_h_ | hatch: let's start with the release. Getting the charms sync'd up. The production ones have changes that need to be bzr merged into the dev ones and might be a conflict in there | 18:02 |
| rick_h_ | hatch: because I screwed up landing changes from eco | 18:03 |
| hatch | oh jeeze I don't even know if this vm has bzr on it | 18:03 |
| hatch | well...it does now | 18:04 |
| hatch | apt ftw | 18:04 |
| rick_h_ | hatch: :) | 18:06 |
| rick_h_ | ty | 18:06 |
| === tvansteenburgh is now known as tvan|lunch | ||
| hatch | now the question is - do I remember how the heck to use it | 18:06 |
| hatch | actually - the first problem is....where are the charms even located? | 18:06 |
| hatch | it's been 2 years and I still don't understand launchpads organizational structure | 18:07 |
| rick_h_ | hatch: look at docs/process and copy/paste | 18:07 |
| rick_h_ | hatch: with some reading/understanding before hitting <enter> | 18:08 |
| hatch | I'm not sure about the charm changes though | 18:11 |
| hatch | there are the ones under juju-gui then there is others somewhere too right? | 18:11 |
| rick_h_ | hatch: it's in the process doc | 18:12 |
| rick_h_ | hatch: copy/paste bzr branch commands for both charms | 18:12 |
| hatch | ok I'll read it closer :) | 18:12 |
| rick_h_ | hatch: Make a new release of the juju-gui charm by doing the following. | 18:13 |
| rick_h_ | start there | 18:13 |
| hatch | so why does production have changes that devel doesn't? | 18:18 |
| hatch | isn't that...backwards? :) | 18:18 |
| rick_h_ | because I screwed up :) | 18:18 |
| rick_h_ | per the previous comment :) | 18:18 |
| hatch | ohhhhh | 18:18 |
| rick_h_ | the eco guys pulled against production, I accepted and merge it. Then realized I should have retargetd the pulls against our dev branches but too late | 18:19 |
| rick_h_ | I fail | 18:20 |
| hatch | tis ok - I'll figure this setup out | 18:21 |
| hatch | is there a reason why we didn't move the charm to github? | 18:21 |
| hatch | obviously prod needs to be in bzr | 18:21 |
| hatch | but the devel ones | 18:21 |
| rick_h_ | because syncing is a pita and it's new work we don't have bandwidth for | 18:21 |
| rick_h_ | as we're learning with the charms elsewhere | 18:22 |
| hatch | yeah syncing my ghost charm to bzr is a pain | 18:23 |
| hatch | might be able to write a script though | 18:23 |
| rick_h_ | one day | 18:23 |
| hatch | ermahgerd this is still downloading | 18:25 |
| hatch | I think we need to truncate this repo lol | 18:25 |
| rick_h_ | :) | 18:25 |
| rick_h_ | jujugui http://techcrunch.com/2014/11/03/microsoft-now-lets-developers-run-ie-on-android-ios-and-os-x/?ncid=rss even easier :) | 18:56 |
| hatch | oh nice | 19:00 |
| hatch | its...still...downloading | 19:03 |
| hatch | how big IS this repo | 19:03 |
| hatch | it must be over 1GB | 19:04 |
| rick_h_ | couple hundred mb I think last time I looked? | 19:04 |
| rick_h_ | no, that's nuts | 19:04 |
| hatch | 1363010kB | 19:05 |
| rick_h_ | which repo is this? | 19:06 |
| rick_h_ | bzr branch lp:? | 19:06 |
| hatch | bzr branch lp:~juju-gui/charms/trusty/juju-gui/trunk/ develop-trunk | 19:06 |
| rick_h_ | du -hs (rharding@hulk:~/tmp/develop-trunk/) | 19:06 |
| rick_h_ | 209M | 19:06 |
| hatch | when I run it on the dir it only give me 56k | 19:07 |
| hatch | er 52k | 19:07 |
| hatch | I guess I'll just let it keep going?!? | 19:08 |
| === tvan|lunch is now known as tvansteenburgh | ||
| hatch | 1471711kB 618kB/s - Fetching revisions:Inserting stream:Estimate 4137/12102 | 19:08 |
| hatch | it's pulling down almost 1.5GB so far | 19:09 |
| hatch | pulled* | 19:09 |
| * rick_h_ thinks you broke it | 19:09 | |
| hatch | into the ether I guess though | 19:09 |
| hatch | rick_h_: https://gist.github.com/hatched/eb87714557aca49d6c02 | 19:10 |
| hatch | think I should just cancel it then? | 19:11 |
| rick_h_ | hatch: :/ I guess, rm'ing and retrying it here | 19:11 |
| hatch | ok canceling | 19:11 |
| hazmat | major gui bundle bug found | 19:12 |
| hazmat | its stripping charm config | 19:12 |
| rick_h_ | hazmat: what's up? | 19:12 |
| rick_h_ | hazmat: that's not different from the charm default? | 19:12 |
| hatch | rick_h_: I'll take lunch while we wait for the result from your re-download | 19:13 |
| hazmat | rick_h_, i have a charm with a default value, the bundle changes that | 19:13 |
| rick_h_ | hazmat: rgr | 19:13 |
| rick_h_ | hatch rgr that is | 19:13 |
| hazmat | rick_h_, the gui or charm store.. strips that .. and breaks the bundle.. | 19:14 |
| rick_h_ | hatch: you mean on the bundle export that? | 19:14 |
| hatch | hazmat: so when deploying a bundle it doesn't respect the charm config options? Or when exporting? | 19:14 |
| rick_h_ | bah, tab completion fail today | 19:14 |
| hazmat | nothing to do with bundle export this is hand crafted | 19:14 |
| hazmat | hatch, on deploy | 19:14 |
| hazmat | hatch, my specific examples of this is the kubernetes bundle | 19:14 |
| rick_h_ | hazmat: ok, can you get me the bundle file and I'll file a bug and we'll look into it | 19:14 |
| hazmat | rick_h_, i have to fix it more immediately | 19:15 |
| hazmat | rick_h_, by changing the charm defaults | 19:15 |
| rick_h_ | hazmat: understood, trying to see what's up. this is the kubernetes that's loaded in charmworld then? | 19:15 |
| rick_h_ | or when you drag/drop deploy the bundle the gui is stripping it? | 19:15 |
| rick_h_ | hazmat: here to help debug/fix just trying to narrow down what's going on. | 19:15 |
| hazmat | rick_h_, yes kubernetes bundle in store | 19:15 |
| hazmat | rick_h_, using gui to deploy | 19:16 |
| hazmat | rick_h_, gui search, gui deploy | 19:16 |
| rick_h_ | hazmat: ok, lookins | 19:16 |
| rick_h_ | looking | 19:16 |
| hatch | I am unable to reproduce - my bundles here are respected by the GUI | 19:16 |
| hatch | on import | 19:16 |
| rick_h_ | hazmat: which config item on which charm? | 19:17 |
| hazmat | rick_h_, flannel options on container type | 19:17 |
| rick_h_ | hazmat: ok, so they're on the bundle after ingestion. checking what gui sends to the guiserver | 19:18 |
| hazmat | rick_h_, k, thanks | 19:18 |
| rick_h_ | will have to bring up a quick env will be a couple of mins | 19:18 |
| hazmat | k | 19:18 |
| hazmat | rick_h_, fwiw my sample size is one, but my orangebox has already been reclaimed in terms of further testing here | 19:20 |
| rick_h_ | hazmat: ok, np. I'm waiting on lxc to come up with the gui to try it out | 19:21 |
| rick_h_ | hazmat: will let you know what I find. I know on export we limit options that are set to the charm default, but can't think of anything on deploy that would strip them. | 19:22 |
| hazmat | .. yeah seems wierd to me.. also on export i think you mean limit to those that are not set to default | 19:23 |
| rick_h_ | correct, if the values is the default we filter it from output to the exported yaml file | 19:23 |
| rick_h_ | hazmat: so the gui sent yaml that includes \"config\":{\"container_type\":\"docker\",\"docker_origin\":\"distro\"} | 19:28 |
| hazmat | hmm | 19:28 |
| rick_h_ | now the yaml is options, I can't recall if that's really config at the api level | 19:28 |
| * rick_h_ always gets it mixed up | 19:28 | |
| hazmat | rick_h_, it should be sending config as options to deployer.. i agree its confusing.. and legacy compatibility is the only reason its like that at all | 19:29 |
| rick_h_ | what went across the websocket: http://paste.ubuntu.com/8807178/ | 19:29 |
| hazmat | ie. that's the issue | 19:29 |
| hazmat | 'config' instead of 'options' | 19:29 |
| hatch | I just tested a bundle I had here and config options were sent properly | 19:30 |
| hazmat | hatch, you mean verified they were set on deployed services? | 19:30 |
| hatch | yep | 19:30 |
| hatch | hazmat: here is the bundle https://gist.github.com/hatched/42791a6cec26ca6e5493 | 19:31 |
| hazmat | hatch, that's interesting given we've identified the breakage point | 19:31 |
| rick_h_ | hatch: how did you deploy that? | 19:31 |
| hatch | clicked 'import' | 19:32 |
| rick_h_ | hatch: via drag/drop? | 19:32 |
| rick_h_ | hatch: right, this is saying is that when the gui sends the bundle 'model' with the yaml to the back end it's using 'config' instead of 'options' | 19:32 |
| rick_h_ | hatch: so your test case is a bit different and not hitting the bug hazmat is | 19:32 |
| rick_h_ | hatch: e.g. the deploy event is causing us to use the wrong string for the options key in the services block | 19:33 |
| hatch | I guess I'm confused - this bundle was exported from the GUi and then imported correctly | 19:33 |
| hatch | so the issue is? | 19:33 |
| rick_h_ | hatch: when you click in the browser to deploy, or drag/drop from the sidebar, the event is sending the wrong data | 19:33 |
| hatch | ohh | 19:33 |
| rick_h_ | hatch: we turn options into config to have the models match and be reusable....that's then sent to the deployer incorrectly | 19:34 |
| hatch | well that's no good | 19:34 |
| rick_h_ | right :) | 19:34 |
| hatch | they should be using the same codepath | 19:34 |
| rick_h_ | hatch: right but in one case the yaml is uploaded, in the other it's the json-ification of the model I think | 19:35 |
| rick_h_ | will have to chase it down, filing a bug now | 19:35 |
| hatch | got it, now I understand :) | 19:35 |
| hatch | ok now I'm really going for lunch | 19:35 |
| rick_h_ | hatch: assigning the bug your way. Don't worry about hte release. I'll do it tonight. I think that should be a small bug fix if you can get it in before your EOD yay | 19:39 |
| rick_h_ | hazmat: so the work around for the moment is to use a drag/drop or import of the bundle. We're due to get a release out the door today/tomorrow so I'll try to get hatch to fix it up and have a new release out by then with the fix | 19:41 |
| rick_h_ | thanks for reporting the issue | 19:42 |
| hazmat | dustin and i've hit a few more bugs the last few days of using the gui | 19:47 |
| hazmat | deploying some services, try to change the config, can't commit, can't clear.. end just reloading the page to get gui in a sane state | 19:47 |
| rick_h_ | hazmat: ok, happy to get those knocked out if we can reproduce them. Appreciate you beating on the gui in anger. | 19:49 |
| rick_h_ | hazmat: ok, happy to get those knocked out if we can reproduce them. Appreciate you beating on the gui in anger. | 19:50 |
| hazmat | rick_h_, you mean using it;-) | 19:50 |
| hazmat | rick_h_, so... the other question that came up is when personal charms can get their icons displayed? | 19:50 |
| rick_h_ | :) | 19:50 |
| rick_h_ | hazmat: that's part of the new back end. We've turned off the filter there and when we update the gui to the new api it'll add that feature. | 19:51 |
| rick_h_ | hazmat: so in about 3wks should have a release with that as a checkbox option in the gui | 19:51 |
| hazmat | k | 19:52 |
| kadams54 | guihelp: https://github.com/juju/juju-gui/pull/637 is ready for reviews and QA. Also a very small PR, for an icon update. | 19:57 |
| Makyo | On it | 19:57 |
| kadams54 | rick_h_: let me know what I should work on next. Otherwise I can start looking at some of the high priority maintenance bugs. | 19:58 |
| rick_h_ | kadams54: where's added services? | 19:58 |
| rick_h_ | kadams54: once we have the release blockers out I'd like to go back and get that wrapped up. | 19:58 |
| rick_h_ | kadams54: did/can you review/qa hatch's card in review? | 19:59 |
| kadams54 | Yeah, will do. | 19:59 |
| rick_h_ | kadams54: if there's nothing to do to move things forward there then yes, please look at maint. | 19:59 |
| kadams54 | I think I've done all I can on added services ATM, so once I QA I'll start in on maint. | 20:00 |
| rick_h_ | grrr google "Leaves warehouse by November 3." | 20:05 |
| === kadams54 is now known as kadams54-away | ||
| hatch | rick_h_: ok I'll hop on that | 20:23 |
| hatch | how big was the download? | 20:23 |
| hatch | rick_h_: added services is waiting on my last branch before it can land don't think there is anything kadams54-away can do on it until that lands | 20:23 |
| rick_h_ | hatch: 134M | 20:23 |
| rick_h_ | hatch: ok | 20:24 |
| hatch | 134M? wth was it downloading for me then? | 20:25 |
| hatch | lol | 20:25 |
| rick_h_ | hatch: the world | 20:25 |
| rick_h_ | to /dev/null | 20:25 |
| hatch | rofl | 20:25 |
| hatch | it did seem that way | 20:25 |
| hatch | rick_h_: ok starting on hazmat's bug | 20:26 |
| hatch | jujugui I still need two reviews and a qa on #636 | 20:27 |
| hatch | it's blocking release | 20:27 |
| rick_h_ | hatch: Makyo was looking? https://github.com/juju/juju-gui/pull/636 and will try to get another one in a bit | 20:27 |
| rick_h_ | or kadams54-away when he's back | 20:27 |
| hatch | ohh ok I just didn't see any comments | 20:28 |
| hatch | or tags on the card | 20:28 |
| === kadams54-away is now known as kadams54 | ||
| kadams54 | I'm back, looking at 636 | 20:34 |
| kadams54 | hatch: ^ | 20:35 |
| hatch | thanks | 20:36 |
| rick_h_ | kadams54: did you see Makyo's qa note on your icon branch as well? | 20:48 |
| kadams54 | rick_h_: yeah, will take a look after I finish with hatch's branch. | 20:49 |
| Makyo | I didn't know we didn't fade un-relate-able ghosts, though I don't remember us ever having done so. | 20:49 |
| Makyo | I guess since they're already faded slightly | 20:49 |
| rick_h_ | Makyo: it could probably use some polish there. | 20:49 |
| hatch | Makyo: the current canvas reactions are all busted | 20:52 |
| hatch | you would have to merge in my branch for the real ones | 20:52 |
| Makyo | Ah, alright | 20:52 |
| hatch | so I'll be sure to get you to QA it once it gets finished | 20:53 |
| hatch | :P | 20:53 |
| Makyo | \o/ | 20:56 |
| kadams54 | Makyo: spriting issue on https://github.com/juju/juju-gui/pull/637 fixed. | 21:13 |
| Makyo | kadams54, +1, thanks! | 21:14 |
| * hatch punches non native promises right in the neck | 21:29 | |
| hatch | rick_h_: from 2013 :) https://github.com/juju/juju-gui/blob/develop/app/store/env/fakebackend.js#L1824-L1827 | 21:32 |
| hatch | any idea why? | 21:32 |
| hatch | Gary was the last to touch | 21:33 |
| hatch | I know why it needed to be changed to config, but why delete options... | 21:35 |
| * hatch runs the tests and hope it has them | 21:38 | |
| hatch | ;) | 21:38 |
| hatch | hazmat: did you have a running gui instance to test a fix? | 21:40 |
| hatch | (I also am booting up an env) | 21:41 |
| hazmat | hatch, nothing i can experiment with on atm | 21:42 |
| hazmat | changing locations | 21:42 |
| hazmat | bbiab | 21:42 |
| hatch | ok I'll report back if this indeed fixes it | 21:42 |
| rick_h_ | hatch: I can bring back up my local env, shoot me the branch url for the gui charm config | 21:44 |
| hatch | rick_h_: thanks but I'm almost there (the branch is https://github.com/hatched/juju-gui/tree/fix-bundle-1388953 if you are interested) | 21:46 |
| rick_h_ | hatch: ok cool, will still QA | 21:47 |
| hatch | rick_h_: see "Last Saskatchewan Pirate" https://play.google.com/store/music/album?id=Bkeqoycogjsra3xvap2dydii3pi&tid=song-Tcbllehpxcy4vmjkdoipcwz7h4u | 21:49 |
| hatch | :D | 21:49 |
| rick_h_ | :P | 21:49 |
| hatch | Here is the original https://play.google.com/store/music/album/Captain_Tractor_East_of_Edson?id=Bcfl5tonv5a2gxiaxy3mjsje7em | 21:50 |
| hatch | haha | 21:50 |
| hatch | The title is an Alberta in-joke, and one of the key tracks on the album is a cover of the Arrogant Worms' humorous "Last Saskatchewan Pirate." | 21:51 |
| hatch | guess I was wrong even that wasn't the original | 21:51 |
| hatch | ok it appears to have worked | 21:55 |
| rick_h_ | hatch: that didn't change for me, the yaml passed is still config | 21:55 |
| hatch | https://gist.github.com/hatched/1ba6ee63e31b9afb95d2 | 21:56 |
| hatch | is that not correct? | 21:56 |
| rick_h_ | hatch: this is different | 21:57 |
| hatch | I don't get it, I deployed his charm from the charmbrowser and it set the proper config options | 21:57 |
| hatch | I thought that was the bug? | 21:57 |
| rick_h_ | hatch: standup | 21:58 |
| hatch | ok one minute need to grab headset | 21:58 |
| rick_h_ | rgr | 21:58 |
| rick_h_ | hatch: oh, now that looks right. | 21:59 |
| rick_h_ | huwshimi: wow early isn't it? | 21:59 |
| huwshimi | Morning | 21:59 |
| huwshimi | rick_h_: 9am | 21:59 |
| rick_h_ | TZ gotta love em | 21:59 |
| huwshimi | :) | 21:59 |
| huwshimi | rick_h_: I have change to DST recently :) | 21:59 |
| rick_h_ | huwshimi: and we just fell back an hour this weekend | 22:00 |
| huwshimi | rick_h_: Ah :) | 22:00 |
| hatch | got to love different countries choosing to change DST's at different time of year | 22:06 |
| huwshimi | hatch: We don't even change DST at the same time within the same country :) | 22:09 |
| hatch | lol thats horrible | 22:09 |
| hatch | we have some :30 timezones but nothing quite that crazy | 22:09 |
| hatch | :) | 22:10 |
| === kadams54 is now known as kadams54-away | ||
| rick_h_ | hatch: did you still need review/QA? | 22:46 |
| rick_h_ | hatch: is there a branch up huwshimi can look over for you? | 22:46 |
| hatch | umm I'm just finishing up the last test for this fix | 22:46 |
| hatch | then that will need a look | 22:46 |
| hatch | I can't remember about the other heh | 22:46 |
| hatch | lemme look | 22:46 |
| hatch | looks good on that one, will ship | 22:47 |
| rick_h_ | k | 22:47 |
| === kadams54-away is now known as kadams54 | ||
| hatch | jujugui lf a qa and reviews on https://github.com/juju/juju-gui/pull/638 ciritical fix blocking release | 22:58 |
| kadams54 | hatch: Looking | 22:58 |
| hatch | hazmat: ^ this is the fix to your bug | 22:59 |
| hazmat | hatch, k, thanks for the quick turnaround.. | 22:59 |
| hatch | we aim to please | 22:59 |
| hatch | (sometimes we miss) ;P | 23:00 |
| rick_h_ | hatch: review in | 23:01 |
| kadams54 | +1'd | 23:02 |
| hatch | and shipped | 23:02 |
| hatch | thanks | 23:02 |
| hatch | rick_h_: ok so tomorrow I'll finish up the added services branch? | 23:04 |
| hazmat | rick_h_, is there a way to refer to a non frozen bundle revision ? | 23:19 |
| hazmat | ie. current kubernetes bundle points to bundle:~hazmat/kubernetes/7/kubernetes and i'd like to document it in a readme | 23:20 |
| hazmat | but thats going to be stale if i have to ref it with a hardcoded revision | 23:20 |
| hatch | hazmat: https://jujucharms.com/bundle/~hazmat/kubernetes/kubernetes/ works | 23:43 |
| hazmat | hatch, for pointing to quickstart ? | 23:44 |
| hazmat | ie. i can't run quickstart that url afaik | 23:44 |
| hatch | oh I have no idea - I just know that the GUI supports it | 23:44 |
| hatch | oh :/ | 23:44 |
| hazmat | quickstart wants either url to bundle content or bundle: .. but the bundle: url embeds the rev | 23:45 |
| hazmat | trying to understand if bundle url has syntax to not do that | 23:45 |
| hatch | hazmat: https://manage.jujucharms.com/api/3/bundle/~hazmat/kubernetes/kubernetes | 23:46 |
| hatch | so it SHOULD work via quickstart? | 23:46 |
| === kadams54 is now known as kadams54-away | ||
| hazmat | hatch, perfect thanks | 23:47 |
| rick_h_ | hazmat: just remove the 7 | 23:47 |
| hatch | thats what I did yeah - he said that wasn't working | 23:47 |
| rick_h_ | hazmat: all revisions are always optional | 23:47 |
| rick_h_ | :/ oh hmm | 23:47 |
| rick_h_ | while that runs, yes hatch the next goal is to get the AS stuff out the door | 23:49 |
| rick_h_ | hazmat: so I did juju quickstart bundle:~hazmat/kubernetes/kubernetes | 23:51 |
| hazmat | rick_h_, cool, thanks i'll update the readme to do that much nicer url | 23:52 |
| rick_h_ | hazmat: and got http://paste.ubuntu.com/8809936/ | 23:52 |
| rick_h_ | so should be working, if not let me know and we'll find out what's up | 23:52 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
