Thursday, 2018-11-01

*** markvoelker has quit IRC00:01
*** brinzhang has joined #openstack-glance00:14
*** Liang__ has joined #openstack-glance00:27
*** Liang__ has quit IRC00:29
*** Liang__ has joined #openstack-glance00:58
*** Liang__ is now known as LiangFang00:59
*** zul has quit IRC01:04
*** sdinescu has quit IRC01:07
*** sdinescu has joined #openstack-glance01:07
*** lixiaoy1 has joined #openstack-glance01:11
*** lixiaoy11 has joined #openstack-glance01:12
*** lixiaoy1 has quit IRC01:12
*** lixiaoy11 has left #openstack-glance01:16
*** MattMan_ has quit IRC01:49
*** MattMan_ has joined #openstack-glance01:49
*** markvoelker has joined #openstack-glance02:03
*** mhen has quit IRC02:13
*** mhen has joined #openstack-glance02:16
*** markvoelker has quit IRC02:35
*** markvoelker has joined #openstack-glance03:32
*** udesale has joined #openstack-glance03:50
*** markvoelker has quit IRC04:06
*** abhishekk has joined #openstack-glance04:43
*** LiangFang has quit IRC04:53
*** Liang__ has joined #openstack-glance04:54
*** markvoelker has joined #openstack-glance05:02
*** abhishekk has quit IRC05:25
*** abhishekk has joined #openstack-glance05:26
*** abhishekk has quit IRC05:26
*** abhishekk has joined #openstack-glance05:27
*** ratailor has joined #openstack-glance05:35
*** abhishekk has quit IRC05:36
*** markvoelker has quit IRC05:36
*** abhishekk has joined #openstack-glance05:37
*** markvoelker has joined #openstack-glance06:33
*** mosulica has joined #openstack-glance06:37
*** brinzhang has quit IRC07:01
*** brinzhang has joined #openstack-glance07:02
*** markvoelker has quit IRC07:05
*** pcaruana|elisa| has joined #openstack-glance07:40
*** pcaruana|elisa| has quit IRC07:59
*** imacdonn has quit IRC08:00
*** udesale has quit IRC08:02
*** markvoelker has joined #openstack-glance08:03
*** pcaruana has joined #openstack-glance08:05
*** kgspooks is now known as kgz08:10
*** markvoelker has quit IRC08:36
*** udesale has joined #openstack-glance09:16
*** markvoelker has joined #openstack-glance09:33
*** markvoelker has quit IRC10:07
openstackgerritPranali Deore proposed openstack/glance-specs master: Spec Lite: Add glance-cache-manage utility using v2 API  https://review.openstack.org/60933110:11
*** abhishekk has quit IRC10:39
*** Liang__ has quit IRC10:52
*** markvoelker has joined #openstack-glance11:04
*** pcaruana has quit IRC11:05
*** udesale has quit IRC11:24
*** zul has joined #openstack-glance11:25
*** ratailor has quit IRC11:36
*** markvoelker has quit IRC11:36
*** pcaruana has joined #openstack-glance11:52
*** markvoelker has joined #openstack-glance12:31
*** rosmaita has joined #openstack-glance12:31
*** udesale has joined #openstack-glance12:32
*** brinzhang has quit IRC12:40
*** jmlowe has quit IRC12:47
*** zul has quit IRC12:52
*** zul has joined #openstack-glance12:55
*** udesale has quit IRC12:56
*** abhishekk has joined #openstack-glance13:00
*** udesale has joined #openstack-glance13:01
*** mosulica has quit IRC13:03
*** zul has quit IRC13:04
*** zul has joined #openstack-glance13:05
*** jmlowe has joined #openstack-glance13:07
*** belmoreira has joined #openstack-glance13:17
*** Liang__ has joined #openstack-glance13:43
*** Liang__ is now known as LiangFang13:44
smcginnisI now have a meeting conflict with the TC meeting being the same time as the glance meeting.14:00
smcginnisI'll try to multitask.14:00
rosmaitaman, glance gets no respect14:00
rosmaitawho scheduled that?14:00
smcginnisThe Rodney Dangerfield of OpenStack. :)14:00
rosmaitaCourtesy meeting reminder on #openstack-meeting-4: flwang1, jokke_, mclaren, mfedosin, nikhil_k, rosmaita, GB21, bpoulos, harshs, abhishekk, dshakhray, kairat, alex_bash, smcginnis, arcolife, Brin, Roamer`, andybotting14:01
abhishekkoops14:01
*** mosulica has joined #openstack-glance14:26
*** Luzi has joined #openstack-glance14:27
*** mvkr has quit IRC14:40
*** mosulica has quit IRC14:42
*** Luzi has quit IRC14:44
*** mosulica has joined #openstack-glance14:46
*** mosulica has quit IRC14:52
*** mosulica has joined #openstack-glance14:53
*** abhishekk has quit IRC15:00
*** jokke_ has joined #openstack-glance15:01
*** jmlowe has quit IRC15:10
*** mvkr has joined #openstack-glance15:11
*** kukacz has quit IRC15:12
*** itlinux has quit IRC15:13
LiangFangrosmaita: just FYI. I was told by some collegue before that md5 is not safe enough now, people is moving to sha serial.15:18
LiangFangmay sha384, sha51215:19
*** kukacz has joined #openstack-glance15:19
rosmaitaLiangFang: that is correct15:20
rosmaitaThe glance multihash (new in Rocky) uses sha512 by default15:21
rosmaitawe still do md5 for the legacy checksum property15:21
LiangFangso in the future, may more and more library not contain md515:21
LiangFangOK15:21
rosmaitaLiangFang: exactly, we need to figure out how to make the transition15:21
*** LiangFang has quit IRC15:32
*** gyee has joined #openstack-glance15:33
*** cranges has joined #openstack-glance16:00
*** cranges has quit IRC16:01
*** mosulica has quit IRC16:02
*** cranges has joined #openstack-glance16:02
*** belmoreira has quit IRC16:11
*** chhagarw has joined #openstack-glance16:12
*** itlinux has joined #openstack-glance16:14
*** imacdonn has joined #openstack-glance16:15
imacdonnjokke_: ?16:22
crangesMy glance-api service won't start after upgrading to Rocky.  Where should I look?16:25
crangeshttp://paste.openstack.org/raw/733780/16:25
crangesERROR stevedore.extension [-] Could not load 'xxxxxxx: No module named common16:25
dhellmanncranges : this tool can help you debug plugin load errors: https://pypi.org/project/entry_point_inspector/16:27
dhellmannI don't know what namespace glance is loading there, but if you run "epi namespace list" you ought to be able to make a guess based on the name starting with "glance"16:28
dhellmannoh, sorry, it's "group list" not namespace list16:28
crangesThanks for the tip..  I just decided to purge glance and reinstall..  works now17:01
*** belmoreira has joined #openstack-glance17:02
*** udesale has quit IRC17:06
*** cranges has quit IRC17:11
*** cranges has joined #openstack-glance17:12
*** cranges has quit IRC17:13
*** belmoreira has quit IRC17:19
*** belmoreira has joined #openstack-glance17:20
*** pdeore has joined #openstack-glance17:52
*** jdillaman has quit IRC18:15
*** jmlowe has joined #openstack-glance18:22
*** mvkr has quit IRC18:47
*** belmoreira has quit IRC18:48
*** pdeore has quit IRC18:51
*** chhagarw has quit IRC18:51
*** itlinux has quit IRC19:01
*** pcaruana has quit IRC19:05
*** belmoreira has joined #openstack-glance19:32
*** itlinux has joined #openstack-glance19:39
*** mvkr has joined #openstack-glance19:51
*** belmoreira has quit IRC20:12
*** itlinux has quit IRC20:14
*** itlinux has joined #openstack-glance20:15
imacdonnrosmaita: ping20:25
rosmaitaimacdonn: pong20:25
imacdonnhi ... so ....... we need to figure out some way to address https://bugs.launchpad.net/python-glanceclient/+bug/178894220:25
openstackLaunchpad bug 1788942 in Glance Client "url quoting breaks x-image-meta-location" [Undecided,New]20:25
imacdonnIMO, using urlparse.quote() may not have been the right thing to do .. it's really intended for quoting the path part of a URL only, not an entire header .... but in any case, I think there's nothing on the server side to decode the percent-encoding20:27
imacdonnOTOH, I don't know what would be the right thing to do, if there even is one20:27
*** KeithMnemonic has joined #openstack-glance20:28
imacdonnI suppose I could do an internal patch on my Icehouse deployments to handle the decoding ... but this issue may pop up somewhere else20:32
rosmaitaimacdonn: yeah, i am not sure of the right thing either20:33
imacdonnanother option is a quick-fix like https://review.openstack.org/#/c/597730/20:33
imacdonnit (probably) wouldn't handle URLs that have non-ASCII content ... but presumably that never did work, and would probably be rare20:34
imacdonn(I mean the server side wouldn't handle it)20:34
rosmaitaright20:35
rosmaitai wonder how bad it would be to leave all c with ord(c) < 128 as they are, and only %-encode the rest20:36
rosmaitai wonder how nonstandard that would be20:37
imacdonnfrom my attempts at researching this, it doesn't seem that there really is a clear standard20:37
rosmaitabut in the meantime, maybe we should do the quick fix ... the issue has been open for a while, and we suspect that the quick fix won't close it, but at least things will be better20:37
rosmaitaimacdonn: you are right about that! (no clear standard)20:38
imacdonnbasically everyone says that you can't rely on anything non-ASCII being handled properly in header content, so you should figure out some way to encode thing20:38
rosmaitai guess as long as the %-decode would work properly when we leave all c with ord(c) < 128 as-is (and i don't see why it wouldn't), we could take that approach20:40
rosmaitaactually it would not work20:40
imacdonnmy concern with that is that there doesn't appear to be anything on the server side to decode it20:40
rosmaitaord('%') < 12820:40
imacdonnit's probably gotten by because noone is actually trying to put any non-ASCII content through headers20:41
imacdonnand my URL case is relatively "corner"20:41
rosmaitayeah20:41
imacdonnmaybe a better approach is to fail if the content is non-ASCII ??20:42
rosmaitawell, the problem is that we have potential unicode values that we're going to transmit as ASCII20:43
rosmaitabut you are right about there being nothing on the server side to handle this correctly20:43
rosmaitai hate to say it, but i think the way to go is to accept your quick fix for now and then figure out a less brittle solution20:44
rosmaitaapparently waiting for the less brittle solution first is not going to get this fixed in a timely fashion20:44
imacdonnI'm good with that20:44
rosmaitai guess this would only need to be backported to rocky20:45
rosmaitapretty sure it's not in queens20:45
imacdonnsounds right .. I believe I first ran into the problem when I started using the rocky version of glanceclient on my admin server20:47
rosmaitaimacdonn: do me a favor and update the test thats in this commit to handle ':' as part of your patch, and then we should be good to go20:49
rosmaitahttps://github.com/openstack/python-glanceclient/commit/818362147d0622752d051bd78ae839497a602e4420:49
imacdonnok, I'll take a look at that20:51
imacdonnactually that test doesn't work, because it's the token case, which has its own "safe" list .. I'll look for, or create, a better place to test this21:00
openstackgerritiain MacDonnell proposed openstack/python-glanceclient master: Don't quote colon in HTTP headers  https://review.openstack.org/59773021:38
imacdonnrosmaita: ^21:39
rosmaitaimacdonn: thanks21:40
*** threestrands has joined #openstack-glance22:17
*** KeithMnemonic has quit IRC22:55
*** mvkr has quit IRC23:22
*** mvkr has joined #openstack-glance23:23
*** gyee has quit IRC23:58

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