Monday, 2015-08-31

*** kzaitsev_mb has quit IRC00:34
*** kebray has joined #openstack-app-catalog02:33
*** kebray has quit IRC02:33
*** kebray has joined #openstack-app-catalog02:34
*** kebray_ has joined #openstack-app-catalog02:53
*** kebray has quit IRC02:56
kfox1111docaedo: for the kloudbustser asset, did you get the image?03:08
docaedoah I downloaded but I don't think I uploaded (started DL then walked away and forgot). I'll verify hash and upload03:24
docaedobtw the infra JSON patch merged :)03:24
docaedouploading now, eta 2 minutes, will review as soon as that asset it up there.03:33
*** kebray_ has quit IRC03:37
*** kebray has joined #openstack-app-catalog03:37
docaedokfox1111: thanks a ton for reminding me about that kloudbuster asset!03:42
*** ig0r__ has joined #openstack-app-catalog04:59
*** ig0r_ has quit IRC04:59
*** kebray has quit IRC05:17
*** kzaitsev_mb has joined #openstack-app-catalog08:49
*** kzaitsev_mb has quit IRC09:17
*** ig0r_ has joined #openstack-app-catalog09:37
*** ig0r__ has quit IRC09:39
*** kzaitsev_mb has joined #openstack-app-catalog10:27
kzaitsev_mbkfox1111: this might fix the issue11:01
*** kzaitsev_mb has quit IRC11:44
*** openstackgerrit has quit IRC11:46
*** openstackgerrit has joined #openstack-app-catalog11:47
*** kzaitsev_mb has joined #openstack-app-catalog11:56
kfox1111docaedo: np. thanks for posting the asset.13:33
kfox1111hmmm.. no icon for kloudbuster...13:39
kfox1111docaedo: Is there an openstack art project? Some folks that could make some icons for openstack projects that get submitted to the app catalog?13:44
openstackgerritKevin Fox proposed stackforge/apps-catalog: Cleanup text a little
kfox1111Can I get a workflow on that one quick?13:49
docaedokfox1111: yep14:42
openstackgerritMerged stackforge/apps-catalog: Cleanup text a little
*** kebray has joined #openstack-app-catalog14:51
kfox1111hmm... getting a 404 on
kzaitsev_mbkfox1111: parameters patch has landed15:02
kfox1111how long does it take to update?15:02
kfox1111kzaitsev_mb: yeah, thanks. :)15:02
kfox1111also saw your other patch. I think that might fix it.15:02
kfox1111docaedo: I think all we're waiting for then is the infra patch for CORS?15:03
kfox1111gota head to work. bbiab.15:03
kzaitsev_mbyep, I've been playing with AUTO_DISCOVER options added in L horizon and had similar problems, so this might actually be the cause15:03
docaedotakes around 15 minutes AFAIK between puppet runs15:05
docaedono more 404 on the assets, looks like it worked!15:11
kzaitsev_mbmight be a good idea to return Content-Type somehow, I guess =) just a suggestion )15:13
kzaitsev_mbthis would probably require configuring web-server though.15:13
kfox1111we need to reconfigure the webserver anyway to support CORS. would be a good time.15:26
docaedoCORS update:
kfox1111thx. looking...15:34
kfox1111docaedo: thanks. comments in the review.15:39
kfox1111might switch from <Directory> to <Location> to implement it too.15:40
docaedoah that's true15:43
docaedoI can update that to use location instead of directory - should we also drop /static at the same time? Do we have any reason to keep that?15:48
docaedo(and thanks for that, I completely forgot about location config!)15:49
kfox1111probably safe to drop /static... would break existing plugin users, but I think that number is 0 at the moment. :)15:50
kfox1111(np. :)15:51
docaedo    <Location <%= @docroot %>/api/v1/assets>15:52
docaedo        Header set Access-Control-Allow-Origin "*"15:52
docaedo        Header set Content-type "application/json"15:52
docaedo    </Location>15:52
docaedo    <Location <%= @docroot %>/api/v1/assets>15:52
docaedo        Header set Access-Control-Allow-Origin "*"15:52
docaedo        Header set Content-type "application/json"15:52
docaedo    </Location>15:52
docaedoerp .. did not mean to paste twice :)15:52
kfox1111hmm.. I think drop the docroot.15:52
kfox1111just <Location /api/v1/assets>15:52
docaedoah yep! thanks15:52
kfox1111perfect. thanks. :)15:55
docaedonp.. stepping out for a bit but back soon15:56
*** kebray has quit IRC16:08
*** rhagarty_ has quit IRC17:09
*** rhagarty has quit IRC17:09
*** rhagarty has joined #openstack-app-catalog17:14
*** rhagarty_ has joined #openstack-app-catalog17:14
*** ig0r__ has joined #openstack-app-catalog17:43
*** ig0r_ has quit IRC17:46
*** kzaitsev_mb has quit IRC17:51
kfox1111docaedo: did you see my icon question? Kloudbuster would be the only non murano app without an icon at this point if it gets added.18:15
docaedodid not see the icon question, no sorry18:21
docaedoI don't really have a strong opinion there, could give a response via review to note adding an icon would be good? ugh that reminds me I want to try to fix the instructions on the wiki today (though I'm not sure what guidance to provide regarding the size/placement options, so would need some help from you there I think)18:23
kfox1111k. was curious if you knew if there's an openstack artwork group or something?18:23
kfox1111icon should be 128x128. top = 0 and left = 0 if thats the case.18:24
kfox1111and height = 128.18:24
kfox1111if not, you can tweak height and top/left until it looks big and centered.18:24
docaedogot it18:24
docaedodon't know if there's a group that does artwork for openstack, though I don't think there is - like, do you mean something the foundation runs?18:25
kfox1111something like openstack-api but for artists.18:25
kfox1111hmm... I was going to throw the horizon/app-catalog-ui version in a header to the asset request, but that would require a preflight request, which may need a slightly different CORS setup... :/18:26
*** ig0r__ has quit IRC18:48
*** kzaitsev_mb has joined #openstack-app-catalog19:00
*** kebray has joined #openstack-app-catalog20:44
*** kebray has quit IRC20:44
*** kebray has joined #openstack-app-catalog20:45
*** kebray has quit IRC20:59
openstackgerritKevin Fox proposed stackforge/apps-catalog: More CORS support
openstackgerritArthur Berezin proposed stackforge/apps-catalog: Adding Couldify TOSCA Orchestrator to the App Catalog.
*** Berezin has joined #openstack-app-catalog21:34
Berezinkzaitsev_ws:  kzaitsev_mb thanks for the help, I've commited a new patch, after the restructure of the files.21:36
Berezinkzaitsev_ws: kzaitsev_mb
kzaitsev_mbBerezin: thanks =) We'll look at it =) The patch looked good the way it was before merge, so I guess it would be ok now too )21:46
kzaitsev_mbBerezin: could you maybe add a bit more info to description — at least the OS of the image21:52
*** Berezin has quit IRC21:57
kfox1111docaedo: alive?22:18
docaedokfox1111: for some definition of alive ;)  Went for an eye exam and had my pupils dilated, never done that before. holy crap does that wreck your vision for a while!22:33
docaedoI'm just now starting to be able to make out letters on the screen, figure in 20-30 minutes I'll be back to norm22:34
kfox1111heh. yeah. :)22:34
kfox1111I've got another set of changes for the cors stuff. :/22:34
kfox1111Something like this:
kfox1111we may want to set a cache header too.22:35
kzaitsev_mbkfox1111: setting cache header is a good idea indeed22:37
kzaitsev_mbhm. is it possible to utilise if-modified-since with this config?22:38
kzaitsev_mbwould be terriffic, considering the size of the file.22:38
kfox1111could. but you still get requests all the way to the server. if we cache the catalog for an hour or a few, then we get no hits at all if its cached.22:39
kzaitsev_mbalso gzipping it should reduce load time, since it should gzip really good.22:39
kfox1111yeah.... I've been wanting an apache module that let you pregzip the file and would then hand it out that way. Haven't seen such a beast yet. :/22:39
kzaitsev_mbI've always been an nginx guy for that matter =)22:40
kfox1111google has a very interesting version of gzip. it brute force compresses it, trying to make it as ideally compressed as possible.22:40
kfox1111takes a long time to compress, but makes the smallest possible gzip. :)22:40
kzaitsev_mb not this?22:41
kfox1111ah... the serving pre-compressed content thing. yeah.22:42
kzaitsev_mbkfox1111: I'm more concerned, that a browser might cache the json forever22:42
kfox1111so... docaedo... you remember when I wanted to build the assets json file out of our own scripts?22:42
kzaitsev_mbthen we would not be able to update anything22:42
kfox1111cause now I want to change the infra code to pregzip it. ;)22:42
kfox1111kzaitsev_mb: have you run into that problem?22:43
kzaitsev_mbkfox1111: why? deflate should be good enough for the task, I guess22:43
kzaitsev_mbkfox1111: with some js/css files, sometimes.22:44
kzaitsev_mbI'm not sure on the state of browser caching right now.22:44
* kfox1111 shurgs. no request scales better then some requests? :)22:44
kfox1111guess we can just do the gzip thing for now. we can always enable more caching later.22:46
kzaitsev_mbto fight that most projects use some hashing techique. we could too.22:46
kfox1111if we don't hit v2 before then. :)22:46
kzaitsev_mbI've used some on pure apache at some point in time )22:47
kfox1111I'm not sure I know of a way to do hashing without doing a round trip anwya.22:47
kzaitsev_mbI mean md5sum the file and add it's hash to it's name like assets.12345qwerty and remove it with mod rewrite22:47
kzaitsev_mbduring file creation =)22:48
kfox1111yeah, but since the plugin hits, how would it know the sum to hit?22:48
kzaitsev_mbit woudn't need to22:48
kzaitsev_mbor it would22:48
kfox1111but then it wouldn't save you a round trip, which is the reason to cache it.22:49
kzaitsev_mbyep, you're right =)22:49
kzaitsev_mbyeah, the requester needs to know that22:49
kzaitsev_mbwell as of now this is all not really that significant )22:50
kfox1111what distro's the webserver running?22:51
kfox1111do we have this available? :)22:52
kzaitsev_mbalthough deflate looks like a bit more robust idea to me. it would set the header itself22:52
kzaitsev_mbwell, if you insist — do not forget to set "Content-Encoding: gzip"22:53
*** Berezin has joined #openstack-app-catalog22:53
kfox1111Berezin: wb22:53
kfox1111kzaitsev_mb: I'm liking that serving pre-compressed content example. I think it should be pretty easy to adapt.22:55
kzaitsev_mband "Vary: Accept-Encoding"22:57
kzaitsev_mband not set the previous one =)22:58
kzaitsev_mbI'm confused22:58
kzaitsev_mbshould probably get some sleep =)22:58
*** kzaitsev_mb has quit IRC23:19
*** kzaitsev_mb has joined #openstack-app-catalog23:32
*** Berezin has quit IRC23:35
*** Berezin has joined #openstack-app-catalog23:45
kfox1111docaedo: still here?23:50
*** Berezin has quit IRC23:57
kfox1111docaedo: do you have a way to test the puppet stuff?23:58
docaedokfox1111: Hi, I'm here/back for a little bit23:59
docaedodo have a way to test the puppet stuff23:59

Generated by 2.14.0 by Marius Gedminas - find it at!