Wednesday, 2018-11-21

*** LiangFang has quit IRC00:00
*** brinzhang has joined #openstack-glance01:22
*** brinzhang has quit IRC01:33
*** brinzhang has joined #openstack-glance01:33
*** Sravan_ has quit IRC01:55
*** Sravan has joined #openstack-glance02:03
*** Sravan has quit IRC02:09
*** MattMan_ has quit IRC02:34
*** MattMan_ has joined #openstack-glance02:34
*** mhen has quit IRC02:50
*** mhen has joined #openstack-glance02:52
*** jmlowe has quit IRC02:59
*** jmlowe has joined #openstack-glance03:00
*** udesale has joined #openstack-glance04:17
*** bhagyashris has joined #openstack-glance04:18
*** Sravan has joined #openstack-glance04:24
*** itlinux has joined #openstack-glance04:33
*** pooja_jadhav has joined #openstack-glance04:49
*** ratailor has joined #openstack-glance05:00
*** bhagyashris has quit IRC05:01
*** bhagyashris has joined #openstack-glance06:08
*** pooja_jadhav has quit IRC06:31
*** Sravan has quit IRC06:42
*** bhagyashris has quit IRC06:48
*** mosulica has joined #openstack-glance06:55
*** bhagyashris has joined #openstack-glance07:09
*** rcernin has quit IRC07:26
*** mosulica has quit IRC08:28
*** mosulica has joined #openstack-glance08:30
*** ratailor has quit IRC09:12
*** ratailor has joined #openstack-glance09:13
*** pcaruana has joined #openstack-glance09:48
*** bhagyashris has quit IRC09:51
*** priteau has joined #openstack-glance10:37
*** mvkr has quit IRC11:03
*** Sravan has joined #openstack-glance11:25
*** udesale has quit IRC11:28
*** Sravan has quit IRC11:30
*** mvkr has joined #openstack-glance11:43
*** mugsie has joined #openstack-glance11:48
*** brinzhang has quit IRC12:15
*** brinzhang has joined #openstack-glance12:15
*** brinzhang has quit IRC12:16
*** brinzhang has joined #openstack-glance12:17
*** pvradu has joined #openstack-glance12:31
*** pvradu has quit IRC12:32
*** pvradu has joined #openstack-glance12:33
*** brinzhang has quit IRC12:38
*** MattMan_ has quit IRC12:57
*** MattMan_ has joined #openstack-glance12:57
*** zul has joined #openstack-glance13:13
*** ratailor has quit IRC13:49
*** pvradu_ has joined #openstack-glance14:03
*** pvradu has quit IRC14:07
*** pvradu_ has quit IRC14:07
*** pvradu has joined #openstack-glance14:08
openstackgerritayoung proposed openstack/glance master: Move base policies into code  https://review.openstack.org/50136014:16
*** udesale has joined #openstack-glance14:39
*** ratailor has joined #openstack-glance15:48
*** udesale has quit IRC15:58
*** ratailor has quit IRC16:24
*** pcaruana has quit IRC16:50
*** pvradu has quit IRC17:08
*** mosulica has quit IRC17:20
*** Sravan has joined #openstack-glance17:27
*** Sravan has quit IRC17:28
*** Sravan has joined #openstack-glance17:28
*** ayoung has joined #openstack-glance18:14
ayoungjokke_, when creating a glance image for a large file (1.5 GiB) over a slow connection, does it make sense to upload the image as a file to Swift, and then use an URL for the image instead?  Is there any speed benefit?  Does glance know to not double-book for quota, etc?18:25
openstackgerritBrian Rosmaita proposed openstack/python-glanceclient master: Add image-list filter for multihash  https://review.openstack.org/59452018:26
imacdonnrosmaita: how familiar are you with the CONF hackery in glance_store's unit tests ?18:37
imacdonnmeaning things like: # NOTE(flaper87): temporary until we can move to a fully-local lib. (Swift store's fault)18:38
rosmaitaimacdonn: i'm not sure who the best person is about that anymore ... there's some really weird jumping through hoops you have to do to get the stores initialized correctly18:39
rosmaitajokke_ probably knows the most about it at this point18:39
imacdonnrosmaita: I don't know if I need that or not ... I was trying to create tests for a new method I added to multi_backend, but I'm confused about how to make config available to it18:40
imacdonnWhen my unit test calls store.method(), it fails with: oslo_config.cfg.NoSuchOptError: no such option enabled_backends in group [DEFAULT]18:41
rosmaitaare there any tests that deal with default_stores (the old config option, pretty sure that was in DEFAULT)18:43
imacdonndefault_store was in [glance_store]18:44
rosmaitamaybe check abhishek's multistores patch, he must have added some tests that use enabled_backends18:45
imacdonnI haven't found any that I can make sense of18:47
imacdonnside-note: I haven't found anything that seems to actually test having multiple stores of different types18:54
imacdonnthey all seem to be focused on one particular type (swift, rbd, sheepdog, etc)18:54
rosmaitaimacdonn: i thought there were some, but maybe they're on the glance side18:56
imacdonnI'm sure there must be some really good reason for glance_store to be a separate project ...................... ;)18:56
rosmaitaimacdonn: guess you'll have to do what abhishek did in the setUp() function in test_multistore_filesystem.py18:57
rosmaitaimacdonn: don't ask!18:57
imacdonnI did that - I copied test_multistore_filesystem.py to test_multistore.py and created my test, keeping the setUp(), but store doesn't seem to see the config18:58
rosmaitai think when flavio split it off, the idea was that various services (like an image transfer service) could use glance_store to do stuff directly with the backends18:58
imacdonnyeah ... I'm sure it seemed like a good idea at the time ;)18:58
rosmaitamaybe shoot abhishek an email asking him how he got it to work18:59
imacdonnI don't think he did ... I don't think he made any tests that need config to work in this way18:59
rosmaitai don't have anything useful to suggest19:02
imacdonnk, thanks19:02
rosmaitai don't know if this is related or not19:02
rosmaitayou may have to import the option into oslo.config19:02
imacdonnunit tests aside, do you have any opinion on the approach in https://review.openstack.org/618932 and https://review.openstack.org/618933 ?19:03
rosmaitaiiuc, 618933 will lazy migrate old locations?19:06
imacdonndon't think so ... this is to handle the bug you filed, where you add a location to an image without specifying a backend in the metadata19:07
rosmaitaok19:08
rosmaitayeah, i don't know what i think about that ... it makes sense, but i wonder whether we should just insist on the user explicitly saying which backend19:12
rosmaitaerno and abhishek have more of the multistore vision than i do, i'll defer to them19:13
imacdonnthe sucky thing about that is that it breaks compatibility ... something that consumes the API that worked in Rocky will not work in Stein19:14
imacdonnespecially if you only have one backend!19:14
rosmaitayeah, that's a good point19:15
imacdonnabhishek seemed OK with my proposal. Have not heard anything from jokke_19:19
imacdonn... and I think I may have figured out a solution for my unit test config issue19:20
rosmaitagood job!19:20
rosmaitai think the code looks fine, and it's an intelligent way to pick a "missing" backend19:22
imacdonnthanks19:24
*** mvkr has quit IRC19:40
openstackgerritiain MacDonnell proposed openstack/glance_store master: Add find_backend_for_url()  https://review.openstack.org/61893220:22
*** mvkr has joined #openstack-glance20:35
*** rcernin has joined #openstack-glance22:09
*** priteau has quit IRC22:16
*** imacdonn has quit IRC22:42
*** imacdonn has joined #openstack-glance22:42
*** rosmaita has left #openstack-glance23:02
*** rm_work has quit IRC23:33
*** rm_work has joined #openstack-glance23:33
openstackgerritiain MacDonnell proposed openstack/glance master: Add backend to location metadata  https://review.openstack.org/61893323:35
*** lbragstad is now known as lbragstad_turkey23:46
*** Sravan has quit IRC23:49
*** zul has quit IRC23:51
*** lbragstad_turkey has quit IRC23:56

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