Tuesday, 2015-09-01

docaedostill need to catch up on the scroll back and get a handle on the optimization goal here00:00
kfox1111k. I submitted an updated cors patch with the bits I think need to be in it. I don't have a way to test though.00:03
kfox1111do you have a horizon of your own that you can load the plugin into?00:03
*** kzaitsev_mb has quit IRC00:04
kfox1111if not, we can do a bit of testing with curl.00:04
docaedoI don't have a running copy of horizon for testing the plugin00:06
kfox1111then we can test with something like:00:06
kfox1111curl -H "Origin: http://foo.com" -H "Access-Control-Request-Method: GET" -H "Access-Control-Request-Headers: X-App-Catalog-Versions: 1 2 3 4" -X OPTIONS localhost:18001/api/v1/assets -vvv00:06
docaedoso .. intention with the patch is to make the json file as small as possible be gzippin on the server each time it's created?00:06
kfox1111part of it yes.00:07
kfox1111the other part is to enable CORS on it.00:07
kfox1111with the CORS preflight check working.00:07
kfox1111we never used that feature before, so we didn't implement it before.00:07
kfox1111I modified the plugin to send over the horizon/app-catalog-ui versions as part of the CORS call. That way, we can get a good handle on just when its safe to deprecate v1 once we've moved on.00:08
kfox11112 years from now, when we don't see any more horzion liberties, we can say, wahoo, byby v1. :)00:09
kfox1111or, shoot, we still have a million v1 users... :)00:11
docaedothat makes sense00:11
docaedowhat's "CORS preflight check"?00:11
kfox1111long story... short ver, if your handing over some other headers, it has to check with the server if its "ok" to do so. it does that by doing an OPTIONS http call.00:12
kfox1111if the server returns stuff that looks like the request its about to make, the browser goes ahead and allows the call.00:12
docaedogot it00:14
docaedoI'll switch machines later tonight or else first thing tomorrow and test the compressing patch you posted00:16
kfox1111k. and also please double check the OPTIONS thing works.00:16
openstackgerritKevin Fox proposed stackforge/apps-catalog-ui: Initial stab at pbr and version passing.  https://review.openstack.org/21906800:21
kfox1111ok... we're getting to be in pretty good shape for liberty.00:22
kfox1111The main things I'm hoping for by then are tighten-schema, ui-packages and hopefully magic-search-gridview. (And a bumch of patches get merged. :)00:24
*** Berezin has joined #openstack-app-catalog00:45
*** kzaitsev_mb has joined #openstack-app-catalog01:01
*** kzaitsev_mb has quit IRC01:05
*** Berezin has quit IRC01:12
*** kzaitsev_mb has joined #openstack-app-catalog01:32
*** kzaitsev_mb has quit IRC01:48
*** Berezin has joined #openstack-app-catalog06:02
*** Berezin has quit IRC06:44
*** kzaitsev_mb has joined #openstack-app-catalog06:59
*** kzaitsev_mb has quit IRC07:04
*** kzaitsev_mb has joined #openstack-app-catalog08:24
*** kzaitsev_mb has quit IRC08:57
*** kzaitsev_mb has joined #openstack-app-catalog09:00
*** kzaitsev_mb has quit IRC09:04
*** kzaitsev_mb has joined #openstack-app-catalog10:10
*** kzaitsev_mb has quit IRC10:54
*** kzaitsev_mb has joined #openstack-app-catalog11:32
*** kebray has joined #openstack-app-catalog14:48
*** kebray has quit IRC15:04
*** kebray has joined #openstack-app-catalog15:11
*** kebray has quit IRC15:17
*** Alec has joined #openstack-app-catalog15:41
Alec@docaedo hi Christopher, have a question regarding a review for an addition to openstack_catalog/web/static/assets.yaml15:44
*** j_king has left #openstack-app-catalog15:50
kfox1111Alec: hi. whats your question?16:03
Alecyes, was wondering if this review needs another +2 to move forward16:06
kfox1111ah. yeah.16:07
kfox1111so, its looking good. we were debating one thing.16:07
kfox1111its tagged as an app, and has no icon.16:07
Alecwas waiting for the app catalog web page to update with the new glance image16:07
kfox1111we've been making a new ui as part of horizon:16:08
Alecok, what is the semantic of "app" tag and what are the other choice?16:08
kfox1111and it looks a little weird to not have icons for it.16:08
kfox1111the semantic of app is, something a user is expecting to run. like:16:09
AlecI could add an icon but did not because the app catalog does not show any for the glance images16:09
kfox1111user searches for Kloudbuster, hits "install", hits "Launch" and has a usable thingy.16:09
Alecok that is not quite our situation16:09
kfox1111everything else is a component, which is something that you load into openstack, as a resource. maybe a part of an app, or a solum language pack, or something that enhances openstack, but the user doesn't directly interact with.16:10
Alecin our case, this image is launched by an external app (kloudbuster) in potentially lots of VMs and is driven by that app16:10
kfox1111ah. ok. so then its a component.16:11
Alecso normally you don;t launch that image directly (you could but it does not do anything by itself)16:11
kfox1111so just drop the tag and I think we're good to go. :)16:11
Alecok I'll just remove it then and resubmit16:11
Alecstill need an icon?16:11
kfox1111sorry for the delay. we were trying to figure out if openstack had an art subproject that we could get to build icons for openstack projects if no artwork was avaiable.16:11
kfox1111no, we're good without one. thanks. :)16:12
Alecok, the snapshot of app catalog looks nice though ;-)16:12
Alecwhy no icon for glance images that are just "component"?16:13
*** kzaitsev_mb has quit IRC16:13
kfox1111Thanks. :)16:14
kfox1111probably could add icons for all the components too. Just haven't gotten there yet.16:15
kfox1111kzaitsev_ws: is it still broken? https://murano-ci.mirantis.com/logs/15/218515/5/check/gate-murano-dashboard-selenium/3091f48/artifacts/test_check_ability_create_two_dependent_apps.png16:17
kfox1111kzaitsev_ws: oh. nm. murano ci check didn't retrigger.16:21
openstackgerritKevin Fox proposed stackforge/apps-catalog: More CORS support  https://review.openstack.org/21902416:23
openstackgerritKevin Fox proposed stackforge/apps-catalog: Break out the code to update assets.  https://review.openstack.org/21867016:30
kfox1111docaedo: can I get a workflow on that last one? Based on the review on the cors infra patch, it might be good to try to just point it at the update_assets.sh script.16:31
docaedokfox1111: Let's float the idea on #openstack-infra to see if there's going to be even more pushback - maybe I'm mistaken and they won't care at all about having the script that gets triggered live in the app-catalog repo16:35
kfox1111one way or another, I think that review does clean up the code. so I think we should merge it.  that being said,16:36
kfox1111I'm ok floating the idea of calling it from the infra side if you think that woudl be quicker then just tweaking the revew and seing what happens.16:37
kfox1111may be easier if the review was merged first though.16:38
docaedocolor me wrong!16:38
docaedoclarkb says "if its an app catalog utility regardless of config management system I would put it there"16:38
kfox1111cool. :)16:38
kfox1111that makes it easy then. :)16:39
docaedoso I'll give you the workflow on that, and adjust my infra CORS one with yolandas notes, and have it call the script16:39
kfox1111I think I have the newest version of it checked out already. I can tweak it pretty quickly if you'd rather.16:39
docaedothat works for me too, thank you!16:40
docaedogotta step out, back in a bit though16:41
kfox1111yeah, me too. meeting time.16:41
*** kebray has joined #openstack-app-catalog16:42
*** rhagarty_ has quit IRC16:50
*** rhagarty has quit IRC16:50
openstackgerritMerged stackforge/apps-catalog: Break out the code to update assets.  https://review.openstack.org/21867016:51
openstackgerritAlec Hothan proposed stackforge/apps-catalog: Add new glance image for the openstack/kloudbuster application  https://review.openstack.org/21852916:56
*** rhagarty has joined #openstack-app-catalog16:57
*** rhagarty_ has joined #openstack-app-catalog16:58
*** kebray has quit IRC17:01
*** kzaitsev_mb has joined #openstack-app-catalog17:05
kfox1111kzaitsev_mb: interesting results...17:05
kzaitsev_mbkfox1111: yep17:06
kfox1111don't think that's anything I did?17:06
kzaitsev_mbthe patch with compress was merged, but djang1.8 landed today17:06
kzaitsev_mbI thought I foresaw it =)17:06
kzaitsev_mbbut apparently I didn't17:06
kzaitsev_mbnot cam out, but support landed to global/requirements17:06
kfox1111so should I just recheck it, or should I wait?17:07
kzaitsev_mband they moved formtools from django.contrib to a separate modeule, that is not present in global requirements.17:07
kzaitsev_mbunfortunatelly wait untill global requirements commit lands https://review.openstack.org/#/c/219222/ this one17:07
kfox1111oh, yeah... I remember that.. I thougth that happened a while ago though?17:08
kzaitsev_mbcause murano has it's dashboard gate blocked =/17:08
kzaitsev_mbdjango1.8 commit was submitted a week ago or so. but horizon doesn't seem to use the contrib.formtools module itself. So no one noticed =/17:08
kfox1111I've seen that somewhere though... where did I see that... I could have swarn it was horizon...17:09
kfox1111brain.. .not .... working. :)17:09
*** Alec has quit IRC17:37
*** kebray has joined #openstack-app-catalog17:49
*** kebray has quit IRC18:45
*** kebray has joined #openstack-app-catalog19:04
*** kebray has quit IRC19:26
*** kfox1111 has quit IRC19:34
*** kfox1111 has joined #openstack-app-catalog19:34
*** kebray has joined #openstack-app-catalog19:47
*** kfox1111 has quit IRC19:47
*** kfox1111 has joined #openstack-app-catalog19:52
*** kzaitsev_mb has quit IRC19:53
*** kzaitsev_mb has joined #openstack-app-catalog20:01
*** kebray has quit IRC20:03
*** kebray has joined #openstack-app-catalog20:05
*** kzaitsev_mb has quit IRC20:11
*** kzaitsev_mb has joined #openstack-app-catalog20:15
*** kzaitsev_mb has quit IRC20:44
*** kzaitsev_mb has joined #openstack-app-catalog20:54
docaedoI'm  in the cross project meeting ATM but wanted to share here, TC voted happily/easily to make app-catalog an official openstack project, in the big tent, or whatever the phrase is :)21:02
* kfox1111 dances21:05
kfox1111congrats. :)21:06
docaedothanks! couldn't have happened without you and kzaitsev_mb, should really help us with people considering it a "real" thing, and hopefully getting more folks involved21:11
docaedokfox1111: I'm transferring a VM right now so I can test the infra stuff, and make some adjustments to cover what yolanda noted re: mod_rewrite21:12
*** kebray has quit IRC21:17
kfox1111ok. cool.21:19
kfox1111then I should be able to commit all the plugin changes.21:19
*** kebray has joined #openstack-app-catalog21:21
*** kzaitsev_mb has quit IRC21:31
*** kzaitsev_mb has joined #openstack-app-catalog21:32
*** kebray has quit IRC21:35
*** kebray has joined #openstack-app-catalog21:36
kfox1111wow... my day reguarding instance users went to hopeless to hopeful again in the span of about two hours. :/21:53
docaedothat's excellent21:56
kfox1111but stoked about the tent! :)21:57
*** kzaitsev_mb has quit IRC21:58
kfox1111so, do we know when the rename's going to happen?22:15
docaedo9/8 is next gerrit downtime AFAIK, I'll coordinate with infra to make sure the repo is renamed accordingly (but I called the repos app-catalog in the governance change, so that alone might resolve things)22:17
docaedoand moving/renaming repos requires gerrit downtime, so that's when it happens22:18
docaedoand this gets it done before the mass move that's happening I think in October22:18
kfox1111nice. :)22:20
kfox1111will make packaging a bit easier too.22:20
kfox1111I'm having to overide defaults since app-catalog != apps-catalog22:20
kfox1111having them match again will help.22:21
*** kzaitsev_mb has joined #openstack-app-catalog22:29
*** kebray has quit IRC22:42
kfox1111docaedo: can you review the kloudbuster image when you get a chance. I think its good to go.22:45
docaedokfox1111: thanks for the poke - have many things to juggle with getting up to speed on new job, appreciate the reminders. Was just now looking for some examples in other puppet manifests hosted by infra for the mod_rewrite usage, and will see what I can find there22:47
kfox1111np. just trying not to forget about it myself. :)22:48
openstackgerritMerged stackforge/apps-catalog: Add new glance image for the openstack/kloudbuster application  https://review.openstack.org/21852922:48
docaedokfox1111: I think this is what they're looking for: https://review.openstack.org/#/c/218898/23:11
docaedotested on my local infra copy with puppet apply, and it did all the things it was meant to and didn't seem to do anything it wasn't supposed to do23:11
docaedothat includes creating the assets stuff23:12
kfox1111k... did you try the curl?23:12
docaedooh no I didn't what the curl I should try again?23:12
kfox1111sec.... let me find it again...23:13
docaedoalso I just noticed I'm going to fail puppet lint, damnit :) I always get lazy on the little ones, it's going to have to get fixed in #723:13
kfox1111curl -X OPTIONS localhost:18001/api/v1/assets -vvv23:16
kfox1111with localhost:18001 updated.23:16
kfox1111see what the output headers look like.23:16
kfox1111ok. I think that will work.23:18
kfox1111wont know 100% for sure until we point a plugin at it. but it seems to have all the right bits in it.23:19
docaedoyep, nice work man!23:19
kfox1111thanks. you too. :)23:19
kfox1111the internet's full of answers. ;)23:19
kfox1111heaven help mankind when the google evermind becomes conscious. ;)23:20
docaedono doubt23:21
kfox1111failed another lint?23:23
docaedoyeah but it failed the lines I thought we were supposed to add based on yolanda's comment, and the other manifests I looked at that were vaguely similar23:26
docaedoI put:23:27
docaedo  httpd_mod { 'rewrite':23:27
docaedo    notify => Service['httpd'],23:27
docaedo    ensure => present23:27
docaedo  }23:27
*** Berezin has joined #openstack-app-catalog23:28
docaedooh I can swap those, let me see...23:28
kfox1111ensure first maybe?23:28
docaedoyeah I think so23:28
docaedoyep, that resolved the issues, hopefully it still handles whatever is buggy about the mod module23:29
*** Berezin has quit IRC23:47
*** kebray has joined #openstack-app-catalog23:51
*** Berezin has joined #openstack-app-catalog23:52
*** Berezin has quit IRC23:56
*** Berezin1 has joined #openstack-app-catalog23:56

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