Sunday, 2015-08-30

kfox1111docaedo: any further progress on the json infra patch?03:20
kfox1111We've got a coulpe of outstanding additions to the catalog that probably need to wait until the patch goes through.03:21
kfox1111I also have 3 patches to the app catalog waiting.03:21
kfox1111 also want to make another pypi release to do more work on rpms/debs which is blocked too.03:22
kfox1111docaedo: gota head out... thanks for looking into it.03:42
*** kfox1111 is now known as kfox1111_away03:45
*** ig0r_ has quit IRC04:59
*** ig0r_ has joined #openstack-app-catalog05:01
*** kzaitsev_mb has joined #openstack-app-catalog09:18
*** kzaitsev_mb has quit IRC09:24
*** kzaitsev_mb has joined #openstack-app-catalog09:26
*** ig0r__ has joined #openstack-app-catalog09:38
*** ig0r_ has quit IRC09:39
*** kzaitsev_mb has quit IRC09:47
*** kzaitsev_mb has joined #openstack-app-catalog10:44
*** ig0r_ has joined #openstack-app-catalog12:04
*** ig0r__ has quit IRC12:05
*** kzaitsev_mb has quit IRC13:06
*** kzaitsev_mb has joined #openstack-app-catalog14:27
*** kzaitsev_mb has quit IRC14:32
kfox1111_awaydocaedo: you around?16:22
*** kfox1111_away is now known as kfox111116:22
docaedokfox1111: I am16:22
kfox1111plan b...16:22
kfox1111I'm updating the app catalog to have a tool to refresh the assets json file if changed, and create it if it doesnt exist.16:23
kfox1111if we could change the infra patch to just exec it, that might be easier to approve, and gives us the ability to change things more easily in the future.16:23
kfox1111also, we need to update the CORS patch to work in /api/v1/assets16:23
docaedooh yeah I forgot about the cors patch, I will update that and make it depend on the infra json patch16:24
docaedobut I can get it pushed through either today or tomorrow (the one I put it)16:24
docaedohow would the tool you're talking about work?  What would trigger it?16:25
openstackgerritKevin Fox proposed stackforge/apps-catalog: Break out the code to update assets.
kfox1111just exec tools/ No triggers or anything. just always call it.16:25
kfox1111it will update things only if they changed.16:25
docaedo"just always call it" - you mean every puppet run?16:26
kfox1111then we can take care of the logic on our end.16:27
kfox1111if we add or move a file, then we just update the shell script in our repo rather then having to tweak infra.16:28
docaedoI can almost guarantee that won't fly, infra wants all things that happen on their servers to be controlled by their servers. landing a patch like this (that would then let us do literally anything we want later by changing the shell script would get a lot of pushback, because they would say "why can't you do this in puppet, on infra?"16:31
kfox1111I'd push back and say its a build script.16:32
docaedowhat is? the shell script is 218670?16:33
kfox1111most projects are allowed to control how their stuff is built.16:33
kfox1111yea. 21867016:33
docaedoyes true, but projects that are hosted on infra are controlled through puppet scripts in infra repo16:34
docaedothing is, infra would have zero control over what that script changes to later16:35
* kfox1111 shrugs.16:35
docaedoso right now, it's a build script16:35
kfox1111they're being slow to fix things. so I'm suggesting a faster way in the futre. if htey want to commit to faster reviews, then we can keep doing it the other way.16:36
docaedosome day later, I go crazy and turn it into a back door that let's me log in to that server and start running portscans, or mining bitcoins :)16:36
kfox1111that could be true if we were shipping a that was a cgi,16:36
kfox1111and we updated that.16:36
kfox1111at the end of the day, if its running code from another sourcd, they have to trust somewhat, and remediate if the actor goes bad.16:37
docaedoyes, true, there are other cases where they trust external source and risk someone doing something bad, but in this case trying to switch to this method would guarantee a whole lot more scrutiny, so would not further the cause16:38
docaedoinstead, I'll keep the conversation going on the infra channel today, I promise, and will rattle all the cages I can16:39
docaedoat this point we will get a +2 from core as soon as nibs responds (should be today)16:40
docaedoin the mean time I'll look at the other stuff that's outstanding today and give notes or plusses16:41
kfox1111k. I still won't withdraw the update patch though. it makes it so that it doesn't rebuild assets if it hasn't changed, so its still a useful thing even if infra doesn't call it.16:42
kfox1111if you can start the cors one soon though, that would be great. I'm expecting that one wont be very controvertial.16:43
kfox1111with the way reviews have been going lately, I'm not so postive about the assets one getting resolved today. :/16:44
docaedothe way the infra puppet one works right now, the "build json file" is subscribed to the repo check job, so the only time it runs is if the repo is updated, so I think it's pretty close to only rebuilding assets when they change16:49
docaedowell, not exactly that I guess - rebuilds the assets file upon commit anyway16:50
kfox1111hmm.. k.16:51
kfox1111not sure what nib's concern was then.16:54
kfox1111still think, the moment we want to enhance things a bit, we're going to be right back here again.16:56
kfox1111like, when we add initial stars support, we're going to probably want to merge stars stored in a db back into the generated assets file.16:57
kfox1111so the puppet job cant just be what it is, and the update code should probably live in our repo.16:57
docaedonibs concern was misplaced, he realized it after we chatted on IRC but then did not get back to remove the -1 unfortunately16:58
kfox1111ah. ok.16:58
docaedobut I think adding stars/rating etc. is going to be a much bigger thing that I want to put up in concert with a lot of other new stuff16:59
docaedoso .. no matter what, it's going to need/get lets of review and looking at before it goes in16:59
docaedothis week I will put together that etherpad page I promised (I'll shoot for doing that tomorrow)17:00
docaedowhere we can outline all the functions/features we want the site to have17:00
docaedoand in the same place we can start noting what frameworks would solve this, and balance something like django against writing from scratch (or else something in between, lighter/smaller than django, but still giving some common methods for the things we want the site to do)17:01
kfox1111The more I play with angular, the more I'm for it.17:02
kfox1111I'm kind of hoping we can share most of the code between the website and the plugin.17:02
docaedobig reason is that I fear if we start doing it in really small steps (like DB backend for holding stars for ratings), we will end up stacking up tons of little bits on top of some simple initial decisions, and likely end up with a horror show that we have to start from fresh with anyway17:02
docaedosharing code between plugin and website for the presentation layer would be AOK with me17:03
kfox1111yeah, could be. though often you kind of have to assume your going to throw the first one away anyway (mythical man month), so that might not be too bad.17:04
kfox1111I'm kind of thinking maybe we just do pecan for the rest api like most of the other services use,17:10
kfox1111and keep doing the website staatic on the server side, and use angular to do the dynamic bits.17:10
docaedonot a bad idea, and ++++ on sticking with pecan as that seems to be the favorite17:14
*** kzaitsev_mb has joined #openstack-app-catalog17:30
kfox1111kzaitsev_mb: I think the murano job that's failing is not due to the patch?17:34
kzaitsev_mbkfox1111: CI guys were meddling with the servers last week — adding 2d job17:36
kzaitsev_mblooks like that might be related17:36
kfox1111ah. ok.17:36
kfox1111any more movement on the other patch? it seems to have quited down. :/17:37
kzaitsev_mbon the other hand my test empty patch was fine17:37
kfox1111still think it might make it in in time?17:37
kzaitsev_mbstill think so, yes17:37
kfox1111I think its something slightly more subtile...17:37
kfox1111I'm adding a javascript file, so the cache needs rebuilding.17:37
kfox1111and its not automatically rebuilding the cache.17:37
kfox1111so so long as you don't add a javascript file, it works.17:37
kfox1111I'm guessing if we disabled the inclusion of the javascript file, it would pass.17:41
*** kzaitsev_mb has quit IRC19:34
*** kzaitsev_mb has joined #openstack-app-catalog19:38
*** kzaitsev_mb has quit IRC19:48
*** kzaitsev_mb has joined #openstack-app-catalog20:04
*** kzaitsev_mb has quit IRC20:23
*** kzaitsev_mb has joined #openstack-app-catalog20:40
*** kzaitsev_mb has quit IRC20:55
*** kzaitsev_mb has joined #openstack-app-catalog21:22
*** kzaitsev_mb has quit IRC22:38
*** kzaitsev_mb has joined #openstack-app-catalog22:44
*** kzaitsev_mb has quit IRC22:50
*** kzaitsev_mb has joined #openstack-app-catalog22:56
*** kzaitsev_mb has quit IRC23:22
*** kzaitsev_mb has joined #openstack-app-catalog23:54

Generated by 2.14.0 by Marius Gedminas - find it at!