Monday, 2016-11-28

*** bpokorny has quit IRC00:00
*** bpokorny has joined #openstack-glance00:02
*** ducttape_ has quit IRC00:13
*** bpokorny has quit IRC00:20
*** bpokorny has joined #openstack-glance00:21
*** bpokorny has quit IRC00:25
*** rcernin has quit IRC00:42
*** catinthe_ has quit IRC01:16
*** catintheroof has joined #openstack-glance01:17
*** catintheroof has quit IRC01:21
*** jamielennox is now known as jamielennox|away01:26
*** ducttape_ has joined #openstack-glance01:53
*** ducttape_ has quit IRC02:06
*** jamielennox|away is now known as jamielennox02:15
*** markvoelker has joined #openstack-glance02:31
*** ducttape_ has joined #openstack-glance04:06
*** ducttape_ has quit IRC04:11
*** links has joined #openstack-glance04:13
*** tshefi has joined #openstack-glance04:34
*** flwang1 has quit IRC04:57
*** pdeore has joined #openstack-glance04:57
*** udesale has joined #openstack-glance04:57
*** pdeore__ has joined #openstack-glance04:58
*** pdeore has quit IRC05:02
*** ratailor has joined #openstack-glance05:04
*** prateek has joined #openstack-glance05:51
*** ezoszed has joined #openstack-glance06:59
*** exploreshaifali has joined #openstack-glance07:06
*** groen692 has joined #openstack-glance07:08
*** e0ne has joined #openstack-glance07:26
*** mfedosin has joined #openstack-glance07:27
*** e0ne has quit IRC07:40
*** rcernin has joined #openstack-glance07:44
*** pcaruana has joined #openstack-glance07:45
*** mnestratov|2 has joined #openstack-glance07:48
*** e0ne has joined #openstack-glance07:48
openstackgerritNguyen Hung Phuong proposed openstack/glance_store: Clean imports in code  https://review.openstack.org/36726508:03
*** ducttape_ has joined #openstack-glance08:08
*** ducttape_ has quit IRC08:13
*** e0ne has quit IRC08:14
*** bkopilov has quit IRC08:30
*** bkopilov has joined #openstack-glance08:32
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-glance09:01
*** e0ne has joined #openstack-glance09:10
*** gabor_antal has joined #openstack-glance10:02
*** gabor_antal_ has quit IRC10:05
*** exploreshaifali has quit IRC10:07
*** ducttape_ has joined #openstack-glance10:09
*** ducttape_ has quit IRC10:14
*** stevemar has quit IRC10:19
*** ccesario has quit IRC10:36
*** ccesario has joined #openstack-glance10:46
*** kragniz has quit IRC10:54
*** kragniz has joined #openstack-glance10:54
*** udesale has quit IRC10:59
*** mfedosin_ has joined #openstack-glance11:06
*** mfedosin__ has joined #openstack-glance11:09
*** mfedosin has quit IRC11:10
openstackgerritCyril Roelandt proposed openstack/glance: Add working functional tests to tox.ini  https://review.openstack.org/39288811:10
openstackgerritCyril Roelandt proposed openstack/glance: Python3: fix test_client_redirects.py  https://review.openstack.org/39288911:10
openstackgerritCyril Roelandt proposed openstack/glance: Python 3: fix glance.tests.functional.db.test_sqlalchemy  https://review.openstack.org/40362011:10
openstackgerritCyril Roelandt proposed openstack/glance: Python3: Fix glance.tests.functional.test_bin_glance_cache_manage  https://review.openstack.org/40362111:10
openstackgerritCyril Roelandt proposed openstack/glance: Python3: Fix glance.tests.functional.test_glance_replicator  https://review.openstack.org/40362211:10
openstackgerritCyril Roelandt proposed openstack/glance: Python3: fix logs/glance.tests.functional.test_healthcheck_middleware  https://review.openstack.org/40362311:10
openstackgerritCyril Roelandt proposed openstack/glance: Python3: fix glance.tests.functional.test_scrubber  https://review.openstack.org/40362411:10
openstackgerritCyril Roelandt proposed openstack/glance: Python 3: fix glance.tests.functional.v1.test_misc  https://review.openstack.org/40362511:10
openstackgerritCyril Roelandt proposed openstack/glance: Python3: fix glance.tests.functional.v2.test_images  https://review.openstack.org/40362611:10
openstackgerritMaxim Nestratov proposed openstack/glance: expand hypervisor_type meta data with Virtuozzo hypervisor  https://review.openstack.org/34162311:12
*** mfedosin_ has quit IRC11:12
openstackgerritMaxim Nestratov proposed openstack/glance: add ploop to supported disk_formats  https://review.openstack.org/34163311:16
*** nicolasbock has joined #openstack-glance11:45
*** ducttape_ has joined #openstack-glance12:10
*** ducttape_ has quit IRC12:15
*** pdeore__ has quit IRC12:17
*** ratailor has quit IRC12:24
*** ducttape_ has joined #openstack-glance12:32
*** ezoszed has quit IRC12:34
*** ezoszed has joined #openstack-glance12:35
*** ezoszed has quit IRC12:35
*** ezoszed has joined #openstack-glance12:35
*** ezoszed has quit IRC12:36
*** ezoszed has joined #openstack-glance12:38
*** prateek has quit IRC12:39
*** mnestratov|2 has quit IRC12:50
*** tshefi has quit IRC13:01
*** exploreshaifali has joined #openstack-glance13:08
*** ducttape_ has quit IRC13:10
*** mnestratov has joined #openstack-glance13:19
*** dims has quit IRC13:33
*** dims has joined #openstack-glance13:44
*** stevemar__ has joined #openstack-glance13:47
*** ducttape_ has joined #openstack-glance13:52
*** ducttape_ has quit IRC13:56
*** prateek has joined #openstack-glance14:00
*** ezoszed1 has joined #openstack-glance14:00
*** ezoszed has quit IRC14:01
*** links has quit IRC14:04
*** exploreshaifali has quit IRC14:12
*** ducttape_ has joined #openstack-glance14:18
*** Lord_balancer is now known as umbSublime14:26
*** rcernin has quit IRC14:27
*** openstackstatus has quit IRC14:28
*** openstackstatus has joined #openstack-glance14:29
*** ChanServ sets mode: +v openstackstatus14:29
*** ckonstanski has joined #openstack-glance14:30
*** rcernin has joined #openstack-glance14:36
*** ccesario has quit IRC14:48
*** ccesario has joined #openstack-glance15:00
openstackgerritMerged openstack/glance: IPv6 fix in Glance for malformed URLs.  https://review.openstack.org/39494615:02
*** ezoszed1 has quit IRC15:13
*** mnestratov has quit IRC15:30
*** ckonstanski has quit IRC15:36
*** dims has quit IRC15:36
*** dims has joined #openstack-glance15:41
*** mtanino has joined #openstack-glance15:42
*** mtanino has quit IRC15:46
*** mtanino has joined #openstack-glance15:50
*** prateek_ has joined #openstack-glance15:50
*** prateek has quit IRC15:53
openstackgerritIan Cordasco proposed openstack/glance: Prepare for oslo.log 3.17.0  https://review.openstack.org/40377515:54
sigmavirusrosmaita: hemanth ^ based on Tony's email15:57
sigmavirusI think I figured out what that test was there for and I think that change will suitably shim the old behaviour into place15:57
sigmavirusIt also documents the upgrade bits for deployers relying on glance-{api,registry} to log to stderr15:57
*** ducttape_ has quit IRC15:59
*** ducttape_ has joined #openstack-glance16:00
rosmaitasigmavirus: looking16:00
*** ducttape_ has quit IRC16:01
*** mvk has quit IRC16:01
*** stevelle has quit IRC16:01
*** david-lyle has quit IRC16:01
*** ameade has quit IRC16:01
*** zigo has quit IRC16:01
*** freerunner has quit IRC16:01
*** keekz has quit IRC16:01
*** zigo has joined #openstack-glance16:01
*** stevelle has joined #openstack-glance16:01
*** keekz has joined #openstack-glance16:02
*** mvk has joined #openstack-glance16:02
*** david-lyle has joined #openstack-glance16:02
*** vryzhenkin has joined #openstack-glance16:02
*** ducttape_ has joined #openstack-glance16:02
*** vryzhenkin is now known as freerunner16:02
*** ameade has joined #openstack-glance16:05
rosmaitasigmavirus: just realized that you made the original change in oslo.log !16:13
*** mvk has quit IRC16:17
sigmavirusrosmaita: yeah because use_stderr being true is terribad16:19
sigmavirusIf you're doing log aggregation you get the logs in /var/log/<service>/<service>-<subservice>.log16:20
sigmavirusand you get them in /var/log/upstart/<command-name>.log16:20
sigmavirus(if you're using something like systemd)16:20
sigmavirusso yes, I did16:20
sigmavirusAnd I'm not ashamed16:20
sigmavirus:P16:20
rosmaitasigmavirus: so were people using log aggregation getting duplicate logs for glance-api?16:21
sigmavirusNot just glance-apip16:21
sigmavirusPretty much, every service16:21
sigmavirusBut it's not just log aggregation, it's pretty much always duplicate logs16:22
sigmavirusUnless you were telling services not to log to files and were relying on stderr logging16:22
sigmavirus(which I doubt very many people were since most of the install guides point people to using log file destinations)16:22
rosmaitaso even if you explicitly set the logfile, you'd still get output to stderr ?16:22
sigmavirusrosmaita: you'd get both16:22
rosmaitaok16:23
sigmavirusIt was just bad16:23
sigmavirusIt was part of why our support team was complaining about log files taking up too much disk space for some services16:23
sigmaviruswe told them to turn off stderr logging when i found this and it helped instantly16:23
rosmaita(i'm suggesting a rewrite of your releasenote, trying to get a good handle of the situation)16:23
sigmavirusrosmaita: well, I'm not even sure if that approach is correct yet16:23
sigmavirusand I'm sure that note could use significant improvement :)16:24
*** rcernin has quit IRC16:27
openstackgerritMaxim Nestratov proposed openstack/glance: add ploop to supported disk_formats  https://review.openstack.org/34163316:33
*** groen692 has quit IRC16:33
*** groen692 has joined #openstack-glance16:34
*** anush has joined #openstack-glance16:57
*** flwang1 has joined #openstack-glance16:57
*** e0ne has quit IRC17:03
*** groen692 has quit IRC17:06
*** pcaruana has quit IRC17:19
*** ducttape_ has quit IRC17:22
*** ducttape_ has joined #openstack-glance17:22
*** ckonstanski has joined #openstack-glance17:34
*** smatzek has joined #openstack-glance17:42
*** prateek_ has quit IRC17:50
stevelleckonstanski: it's going to be a bit tricky to add the right testing to that fix because of the way the code is factored. Hoping my pointer helps point you at how to get a real glance api on the other side of the test17:53
ckonstanskiI saw your comment. Will plug away at it. Gotta be honest: I hacked my local glanceclient because I need to get real work done over here. This upstream fix is now a pet project.17:54
*** catintheroof has joined #openstack-glance18:04
stevelleckonstanski: I figured that is how it would work18:07
*** bpokorny has joined #openstack-glance18:07
stevellewhat we will want to see is one test that just makes a conventional request. The test without your fix should fail. The test with the fix should pass.18:08
stevellethe test shouldn't have py2 vs py3 branching18:08
stevelleThe commit message doesn't have to repeat everything in the bug report, so summarizing that for clarity and brevity would be nice too18:09
ckonstanskiI cannot make py2 fail with the original code. Perhaps the version check will disappear, but don't expect the test to fail under py2. u'foo' == b'foo' in py2. That's the fundamental problem.18:12
*** anush has quit IRC18:23
*** tshefi has joined #openstack-glance18:35
*** chlong has joined #openstack-glance18:42
*** hemanth is now known as hemanth|lunch18:54
*** flwang1 has quit IRC18:56
*** bpokorny has quit IRC19:06
*** bpokorny has joined #openstack-glance19:07
*** bpokorny has quit IRC19:11
rosmaitajokke_: yt?19:12
*** bpokorny has joined #openstack-glance19:21
*** hemanth|lunch is now known as hemanth19:38
*** bpokorny has quit IRC20:02
*** chlong has quit IRC20:02
*** bpokorny has joined #openstack-glance20:02
*** bpokorny has quit IRC20:07
*** chlong has joined #openstack-glance20:18
*** anush has joined #openstack-glance20:31
*** bpokorny has joined #openstack-glance21:00
*** MattMan has quit IRC21:06
*** MattMan has joined #openstack-glance21:06
*** anush has quit IRC21:13
*** chlong has quit IRC21:16
*** smatzek has quit IRC21:28
*** anush has joined #openstack-glance21:29
*** chlong has joined #openstack-glance21:30
*** anush has quit IRC21:37
*** e0ne has joined #openstack-glance22:08
*** chlong has quit IRC22:09
*** mvk has joined #openstack-glance22:25
*** e0ne has quit IRC22:25
*** smatzek has joined #openstack-glance22:26
*** catintheroof has quit IRC22:28
*** zigo has quit IRC22:29
*** zigo has joined #openstack-glance22:36
*** smatzek has quit IRC22:45
rosmaitaglance cores: please look at https://review.openstack.org/#/c/403775/ , it's the last bug with the oslo periodic jobs for glance, would be good to get it merged soon22:54
rosmaitaflaper87: nikhil: hemanth: kragniz: sabari: mfedosin__: kairat: jokke_: flwang: ^^22:56
rosmaitaty22:57
flwangrosmaita: why the patch is still in -1?22:58
flwangi mean working in progress?22:58
stevellebecause sigmavirus wanted to be sure a solution was not rushed. He wanted review23:10
stevellethat is the explanation he gave on the openstack-dev list23:10
*** ducttape_ has quit IRC23:19
*** ducttape_ has joined #openstack-glance23:20
*** ducttape_ has quit IRC23:24
flwangstevelle: ok, thanks for the clarification. the patch looks straightforward, i'm good with it.23:25
rosmaitaflwang: yes, what stevelle said23:27
flwangrosmaita: cool, thanks23:27
rosmaitastevelle: could use your opinion, too23:28
stevellethanks rosmaita, will spend a little time on it this evening. been gone a week so lots to catch up on23:29
rosmaitastevelle: np, don't worry about it23:29
rosmaitajust indicating that i value your opinion, not trying to dump more work on you!23:34
stevelleIt's never a bad thing to get invited to participate :)23:47
*** hemanth is now known as hemanth|away23:55

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