=== Makyo is now known as Makyo|out [12:10] morning frankban and teknico [12:10] bac, good morning [12:11] hi bac [12:12] bac, do you see anything wrong with this diff? ;-) https://bazaar.launchpad.net/~juju-gui/juju-gui/trunk/revision/334 [12:12] teknico: a typo: unrelEased [12:13] i can't spell [12:13] yep :-) [12:13] oops [12:13] I mean, yep to the typo, not to bac's statement :-) [12:14] wow, the release tarball is 25MB, and it expands to 160MB :-) [12:34] hazmat: ping [12:50] bac, pong [12:52] teknico, wow.. that's kinda of silly [12:53] hazmat, well, good compression ratio, I'd say :-) [12:57] node_modules - 127mb [12:57] yui 38m yuidocjs 47m [12:57] silly [12:58] ah.. yuidoc recursively includes yui [13:13] * bac bounced lp2kanban [13:47] bac, frankban, how much time does the "FINAL=1 make dist" command take to complete? is it normal that it hangs in there for dozens of minutes? [13:47] teknico: depending on your upload speed it could take a while [13:47] you can check the project on LP to see that the release has been made [13:48] if so, then you're just waiting while the tarball is uploaded [13:48] bac, I think you told me it took like 10 ir 15 minutes for you last time? [13:48] bac, oh right, it's uploading 25M :-( yep, I'm watching LP staging [13:49] gary_poster: no, last time it took a really long time for me as my internet was being usurped and i didn't know it [13:49] hence those two nice bugs. :) [13:50] :-) [13:51] bac, yep, given the upload bandwidth I have available, it's going to take 13 minutes at full "speed" :-P [13:51] frickin' asymmetric lines [13:51] teknico: you have 1 up? [13:51] bac, what's 1 up? [13:52] teknico: 1mb/s upstream bandwidth [13:52] bac lp2kanban still not working for some reason. (thank you for being lp2kanban doctor btw) [13:52] bac, I have about 300kb/s upstream :-/ [13:53] (and feel free to request that you gain an apprentice) [13:53] gary_poster: first attempt to relaunch resulted in an unnoticed canonistack startup failure. [13:53] oh ok [13:53] gary_poster: a new instance is now alive and coming up [13:53] cool thank you [13:53] this whole cloud fad can't pass fast enough [13:53] oh, forgot the smiley [13:53] * benji imagines a dirty villager yelling "Help! Help! My internet is being usurped!" [13:54] where's round-robin hood when you need him?!? [13:55] heh [14:04] does anyone else want to look at the release on staging, before I make it final? https://staging.launchpad.net/juju-gui/stable/0.2/+download/juju-gui-0.2.tgz [14:07] teknico, we can probably trim the size of the download by getting rid of extraneous node_modules before upload [14:08] hazmat, I'll try, but if compressing a tar archive works as it should, it won't shed much [14:08] the only node_modules we need are runtime app deps afaics (d3, yui), the rest aren't useful for binaries, and can be regen'd for source builds [14:08] the big win being dropping yuidocjs [14:08] teknico, true [14:08] hazmat, also, the yui docs are not going to work that way, though [14:09] teknico, oh the output links to the node module? [14:12] hazmat, apparently the yui doc is in node_modules/yuidocjs/output, and uses the redundant libraries in node_modules/yuidocjs/node_modules [14:12] gary_poster, thanks for you comments! [14:12] I don't think we're pointing to it right now, however [14:13] cool goodspud, thank you :-) [14:14] teknico, but the question is why we need it at all, its not linked from its doc output [14:14] so in the release tarball its not doing anything useful [14:14] teknico, ie make yuidoc -> yuidocs directory.. is not linked to node_modules/yuidoc [14:15] hazmat, right [14:15] hazmat, after nuking the whole node_modules/yuidocjs subtree, the archive size goes down to 19MB, 6MB less [14:15] that's a nice quick win, and 47MB for the extract [14:16] hazmat, yep [14:19] if we can rid of yui in the tarball we can get under 10Mb easily i'd guess. afaics the only thing using it is the node-simulate in build-debug [14:21] hmm.. or not app/ass/js/yui links to node_modules.. i guess for debug using the yui cdn seed would do it though [14:23] hazmat, since I already uploaded to staging, let's do this release as is, and refine the release process afterwards [14:26] teknico, sure just thinking of how to cut the chafe/fat from the land [14:26] future work indeed [14:27] gary_poster, the release checklist says "This is a final release. Consider asking others to verify the package on staging." [14:27] gary_poster, what if there's need for changes? what's the process then? [14:28] teknico, do it again. Staging is irrelevant [14:28] gary_poster, do we delete the release on staging and reuse the same version number? [14:28] In the sense that it is wiped out [14:28] yes teknico if that is possible and easy then that would be what to do [14:30] I feel compelled to add that once a release is made to non-staging, then it should not be replaced. There might be an argument for removing it, however. [14:30] gary_poster, ok, I see how to do that [14:30] benji +1 [14:30] thanks teknico [14:31] benji, totally agreed, only considering doing it while it's not final [14:32] and agreed on the difference between replacing and removing too [14:32] cool; it's really nice that we have the LP staging environment. It makes life so much easier being able to avoid non-reversable things. [15:00] benji: when you can get staging to not timeout... [15:00] well, there's that === Makyo|out is now known as Makyo [15:29] bac bcsaller benji frankban goodspud hazmat Makyo teknico call in 1 [16:18] is adding discussion cards during the weekly call to be considered cheating? :-) [16:36] bcsaller, Makyo I moved our calls to today. 1PM bcsaller, and..I think you are two hours behind me Makyo, in which case it would be 2:30 for you. On your calendar in any case. Do those work for you? [16:36] gary_poster: sounds good [16:36] gary_poster, Yep. Thanks [16:37] thank you both [17:13] guihelp: it seems the latest release is missing read permission to all for build-debug/juju-ui/assets/javascripts/yui/ . As a consequence, nginx returns a 403 forbidden error. [17:14] frankban, oops, really? let me check [17:17] frankban, confirmed :-( there are quite a few directories with that problem in node_modules/, I wonder why [17:18] teknico: if I run make distfile, the resulting tarball seems correct [17:20] frankban, my umask is 0022, I don't get it [17:20] and root's umask is 0022 too [17:21] teknico, could you delete the tarball from the series please? The charm should go back to working then. The next release should have a different release number [17:22] gary_poster, yes, doing it [17:22] thank you [17:23] done on both production and staging [17:25] thanks teknico [17:25] sorry about this [17:35] * gary_poster realizes he has 25 minutes for lunch [17:36] biab [18:10] people may want to review and land this before making a release: https://codereview.appspot.com/7227065 [18:10] if noone gets around to it, I'll have another shot at a release tomorrow [18:26] gary_poster: i am through poking at lp2kanban. please, PLEASE, clean up done:done. :) [22:02] * Makyo walkinates the dogalope.