*** kzaitsev_mb has joined #openstack-app-catalog | 00:06 | |
*** kzaitsev_mb has quit IRC | 00:12 | |
*** spzala has joined #openstack-app-catalog | 00:28 | |
*** spzala has quit IRC | 00:33 | |
*** pcaruana has quit IRC | 05:05 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 05:45 | |
*** kzaitsev_mb has quit IRC | 05:53 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 05:57 | |
*** kzaitsev_mb has quit IRC | 06:17 | |
*** pcaruana has joined #openstack-app-catalog | 06:17 | |
*** sskripnick has joined #openstack-app-catalog | 10:22 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 10:45 | |
*** kzaitsev_mb has quit IRC | 11:20 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 11:21 | |
*** sskripnick has quit IRC | 13:39 | |
*** sskripnick has joined #openstack-app-catalog | 13:59 | |
*** spzala has joined #openstack-app-catalog | 14:09 | |
*** rhagarty has joined #openstack-app-catalog | 14:19 | |
*** openstackgerrit has quit IRC | 14:37 | |
*** openstackgerrit has joined #openstack-app-catalog | 14:38 | |
*** pcaruana has quit IRC | 15:07 | |
*** rhagarty_ has joined #openstack-app-catalog | 15:20 | |
*** rhagarty has quit IRC | 15:22 | |
*** kzaitsev_mb has quit IRC | 16:42 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 16:55 | |
*** spzala has quit IRC | 16:56 | |
*** spzala has joined #openstack-app-catalog | 16:57 | |
sskripnick | docaedo: hi | 16:57 |
---|---|---|
sskripnick | docaedo: any chances to get this in? https://review.openstack.org/#/c/370295/ | 16:58 |
*** spzala has quit IRC | 17:02 | |
docaedo | sskripnick: I'll go through it right now | 17:19 |
sskripnick | oh, thanks | 17:21 |
docaedo | kzaitsev_mb: can you review (or merge) 370295? It looks good to me but since it's more than just an asset update | 17:34 |
kzaitsev_mb | sure, looking through it | 17:36 |
openstackgerrit | Merged openstack/app-catalog: Revert "Keep assets_merge.yaml" https://review.openstack.org/363317 | 17:40 |
kzaitsev_mb | sskripnick: docaedo: does it upload the binaries which are stored on the storage.a.o.o? | 17:44 |
kzaitsev_mb | i.e. the blobs | 17:45 |
kzaitsev_mb | or does it upload all the blobs, even the ones, that are not on s.a.o.o? | 17:46 |
docaedo | kzaitsev_mb: 370295 doesn't do that, unless I really misunderstood - I thought that was just putting some glare-specific pieces in place that won't impact anything right now | 17:46 |
docaedo | ah I see what you're asking - I don't think it does that but good question | 17:48 |
docaedo | kzaitsev_mb: oh I understand what you're asking, sorry - and I don't see it doing that. good question/point | 17:48 |
kzaitsev_mb | looks like it uploads everything to glare right now, doesn't it? that's good ok for murano/toska/heat | 17:50 |
kzaitsev_mb | but for glance images image_url = asset["attributes"].pop("url", None) | 17:50 |
kzaitsev_mb | line 224 | 17:51 |
kzaitsev_mb | so it looks like it would download all the images right now. is that what we want? | 17:51 |
docaedo | I think that is what we want, but it's going to take a while I think to do that transfer | 17:52 |
docaedo | ultimately we would want everything out of the current rackspace cloudfiles storage | 17:52 |
docaedo | but question is will that break any murano stuff? Or could we do two steps where 1) is copy everything from cloudfiles | 17:52 |
kzaitsev_mb | docaedo: we can think of some clever rewrite logic =) | 17:53 |
docaedo | and 2) update DNS to point storage.a.o.o to itself, and make sure we're mapping properly so murano can still download from that URL | 17:53 |
kzaitsev_mb | so that legacy installations would hit smth/smth/smth that would result in hiting v2 api calls and would proxy that | 17:54 |
kzaitsev_mb | so that is a problem of sorts, but I believe we'll be able to handle that | 17:54 |
kzaitsev_mb | my other nit would be to rename _get_keys to _set_keys, since it's very confusing %) | 17:55 |
kzaitsev_mb | but otherwise the patch looks ok | 17:55 |
docaedo | ah yeah I don't spend enough time thinking about set vs. get, I didn't even think about it | 17:57 |
sskripnick | kzaitsev_mb: docaedo: it uploads all except of glance images | 17:59 |
sskripnick | glance images are "external blobs" | 17:59 |
sskripnick | glare will return 302 redirect to blob's url | 17:59 |
sskripnick | it take less then a minute to import | 18:00 |
kzaitsev_mb | sskripnick: hm | 18:00 |
kzaitsev_mb | that's not what we expected it to be. | 18:01 |
kzaitsev_mb | some of the images should be imported in glare directly | 18:01 |
sskripnick | how do this script know which images to upload? | 18:02 |
kzaitsev_mb | sskripnick: this one only heuristically of course | 18:03 |
kzaitsev_mb | the ones that are on storage.a.o.o are should be imported | 18:03 |
sskripnick | ok | 18:03 |
kzaitsev_mb | because, like docaedo said the ultimate goal is to remove everything from the current rackspace cloudfiles storage | 18:04 |
sskripnick | kzaitsev_mb: btw I can't abandon any patches, because I'm not core =) | 18:04 |
kzaitsev_mb | oh, sure. I'll go through the stale patches | 18:04 |
kzaitsev_mb | going to be afk for a while right now | 18:05 |
*** kzaitsev_mb has quit IRC | 18:09 | |
*** spzala has joined #openstack-app-catalog | 18:59 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 19:45 | |
*** kzaitsev_mb has quit IRC | 20:01 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 20:28 | |
*** spzala has quit IRC | 20:44 | |
*** spzala has joined #openstack-app-catalog | 20:45 | |
*** spzala has quit IRC | 20:49 | |
*** spzala has joined #openstack-app-catalog | 21:52 | |
*** sskripnick has quit IRC | 22:14 | |
*** kzaitsev_mb has quit IRC | 23:27 | |
*** openstackgerrit has quit IRC | 23:42 | |
*** openstackgerrit has joined #openstack-app-catalog | 23:47 | |
*** openstackgerrit has quit IRC | 23:51 | |
*** hogepodge has quit IRC | 23:53 | |
*** kzaitsev_mb has joined #openstack-app-catalog | 23:54 | |
*** openstackgerrit has joined #openstack-app-catalog | 23:55 | |
*** hogepodge has joined #openstack-app-catalog | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!