Tuesday, 2014-05-27

*** HenryG has quit IRC00:11
*** zhiyan_ is now known as zhiyan00:38
*** lbragstad has joined #openstack-oslo01:06
*** yamahata has joined #openstack-oslo01:37
*** zhiyan is now known as zhiyan_02:29
*** HenryG has joined #openstack-oslo02:37
*** zhiyan_ is now known as zhiyan02:46
*** zhiyan is now known as zhiyan_02:52
*** zhiyan_ is now known as zhiyan02:52
*** dims has quit IRC03:16
*** zhiyan is now known as zhiyan_04:14
*** zhiyan_ is now known as zhiyan04:21
*** praneshp has joined #openstack-oslo04:50
*** zhiyan is now known as zhiyan_05:40
*** zhiyan_ is now known as zhiyan05:41
rpodolyakamorning all06:07
*** AAzza has joined #openstack-oslo06:26
*** I159 has joined #openstack-oslo06:52
*** boris-42 has quit IRC07:03
*** boris-42 has joined #openstack-oslo07:11
*** markmcclain has joined #openstack-oslo07:23
*** flaper87|afk is now known as flaper8707:26
*** praneshp has quit IRC07:28
*** markmcclain has quit IRC07:35
*** ihrachyshka has joined #openstack-oslo07:38
*** amotoki has joined #openstack-oslo07:49
*** nacim has joined #openstack-oslo08:02
*** viktors|afk has quit IRC08:09
*** viktors has joined #openstack-oslo08:15
openstackgerritJakub Libosvar proposed a change to openstack/oslo-incubator: Remove ValueError when accessing sys.modules  https://review.openstack.org/8502808:16
openstackgerritMark McLoughlin proposed a change to openstack/oslo-incubator: cfgfilter: fix typo in register_cli_opts()  https://review.openstack.org/9567308:22
openstackgerritMark McLoughlin proposed a change to openstack/oslo-incubator: cfgfilter: switch to assertEqual(expected, actual)  https://review.openstack.org/9567408:22
openstackgerritMark McLoughlin proposed a change to openstack/oslo-incubator: cfgfilter: use some testtools helpers  https://review.openstack.org/9567508:22
openstackgerritMark McLoughlin proposed a change to openstack/oslo-incubator: cfgfilter: finally make this thing useful!  https://review.openstack.org/9567608:22
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Fix the test using in-file SQLite database  https://review.openstack.org/9342308:31
*** mrda is now known as mrda_away08:45
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo.messaging: Synced jsonutils from oslo-incubator  https://review.openstack.org/9569608:53
openstackgerritIhar Hrachyshka proposed a change to openstack/taskflow: Synced jsonutils from oslo-incubator  https://review.openstack.org/9569908:53
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Fix changing the type of column deleted  https://review.openstack.org/9341409:04
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests  https://review.openstack.org/9342409:05
openstackgerritVictor Sergeyev proposed a change to openstack/oslo.db: Updated from global requirements  https://review.openstack.org/9294709:08
openstackgerritVictor Sergeyev proposed a change to openstack/oslo.db: Add import_exceptions to tox.ini  https://review.openstack.org/9570209:08
*** ihrachyshka has quit IRC09:18
*** amotoki has quit IRC09:22
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Fix the test using in-file SQLite database  https://review.openstack.org/9342309:24
*** zhiyan is now known as zhiyan_09:39
*** nacim has quit IRC09:47
*** ihrachyshka has joined #openstack-oslo09:56
*** nacim has joined #openstack-oslo10:01
*** nacim has quit IRC10:03
*** nacim has joined #openstack-oslo10:04
*** markmc has joined #openstack-oslo10:16
openstackgerritIhar Hrachyshka proposed a change to openstack/taskflow: Synced jsonutils from oslo-incubator  https://review.openstack.org/9569910:23
openstackgerritIhar Hrachyshka proposed a change to openstack/taskflow: Synced jsonutils from oslo-incubator  https://review.openstack.org/9569910:23
*** nacim_ has joined #openstack-oslo10:31
*** nacim has quit IRC10:33
viktorsmarkmc: hi!11:06
markmcviktors, hi there11:06
viktorsmarkmc:  can you please look at patch in oslo.db https://review.openstack.org/#/c/94552/ (Fix usage of oslo.config) This should fix your notes in  https://etherpad.openstack.org/p/juno-oslo-release-plan11:07
*** zhiyan_ is now known as zhiyan11:17
markmcviktors, cool, will try to look soon11:21
viktorsmarkmc: thanks!11:21
*** dims has joined #openstack-oslo11:28
*** yamahata has quit IRC11:36
*** flaper87 is now known as flaper87|afk12:11
openstackgerritDoug Hellmann proposed a change to openstack/oslotest: Sync new sphinx requirement spec  https://review.openstack.org/9574212:16
openstackgerritDoug Hellmann proposed a change to openstack/oslotest: fix .gitreview after rename  https://review.openstack.org/9534312:17
*** dims has quit IRC12:20
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo.messaging: Synced jsonutils from oslo-incubator  https://review.openstack.org/9569612:20
*** yamahata has joined #openstack-oslo12:42
*** dims has joined #openstack-oslo12:45
*** yamahata has quit IRC12:45
*** yamahata has joined #openstack-oslo12:46
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Fix usage of oslo.config  https://review.openstack.org/9455212:49
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Fix usage of oslo.config  https://review.openstack.org/9455212:58
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Don't delete dict keys while iterating  https://review.openstack.org/9459213:00
openstackgerritRoman Podoliaka proposed a change to openstack/oslo.db: Handle slave database connection in EngineFacade  https://review.openstack.org/9459313:00
ihrachyshkadhellmann: hi. I have a question re H302 in gettextutils module13:13
*** mriedem has joined #openstack-oslo13:24
*** jecarey has quit IRC13:25
openstackgerritAndrey Kurilin proposed a change to openstack/oslo-incubator: Add eventlet.tpool.Proxy for DB API calls  https://review.openstack.org/9517013:32
openstackgerritIhar Hrachyshka proposed a change to openstack/oslotest: Cleanup mock patches on BaseTestCase tearDown()  https://review.openstack.org/9558113:33
*** flaper87|afk is now known as flaper8713:44
openstackgerritMatthew Treinish proposed a change to openstack/oslo-incubator: Make script to run in error msg configurable  https://review.openstack.org/9577213:44
dhellmannihrachyshka: ask away13:45
ihrachyshkadhellmann: I get H302 due to gettextutils._ import from strutils instead of accessing it via module13:45
dhellmannihrachyshka: is the error in gettextutils or in strutils?13:46
ihrachyshkadhellmann: we may a) rework strutils not to do it b) add H302 to ignore list c) add openstack/common to ignore path13:46
ihrachyshkahttps://jenkins01.openstack.org/job/gate-oslo.messaging-pep8/818/console13:46
ihrachyshka2014-05-27 12:24:44.240 | ./oslo/messaging/openstack/common/strutils.py:27:1: H302  import only modules.'from oslo.messaging.openstack.common.gettextutils import _' does not import a module13:46
ihrachyshka2014-05-27 12:24:44.240 | from oslo.messaging.openstack.common.gettextutils import _13:46
dhellmannihrachyshka: you can update the import_exceptions in the hacking section of tox.ini (look at the file in the incubator for an example)13:47
ihrachyshkait seems most of projects ignore it this way or another13:47
ihrachyshkaI just experience it for oslo.messaging13:47
dhellmannihrachyshka: right, update the tox.ini in oslo.messaging with an import exception for gettextutils13:47
ihrachyshkadhellmann: aha, great13:48
dhellmannthat will ignore any imports like that for gettextutils anywhere in oslo.messaging13:48
ihrachyshkathanks, that's the forth option to fix it :)13:48
openstackgerritMarc Koderer proposed a change to openstack/oslotest: Add known issue about time.time mocking  https://review.openstack.org/9541113:51
openstackgerritMarc Koderer proposed a change to openstack/oslotest: Add unit test for olsotest base class  https://review.openstack.org/9551613:51
*** zhiyan is now known as zhiyan_13:57
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo.messaging: Synced jsonutils from oslo-incubator  https://review.openstack.org/9569613:58
*** beekneemech is now known as bnemec14:01
openstackgerritIhar Hrachyshka proposed a change to openstack/oslotest: Cleanup mock patches on BaseTestCase tearDown()  https://review.openstack.org/9558114:05
*** flaper87 is now known as flaper87|afk14:12
rpodolyakathere's been a few patches removing -*- coding: utf-8 -*- markers from 100% ascii-encoded source files. http://docs.openstack.org/developer/hacking/ doesn't have a section on this. Perhaps, we should come up with an opinion on this and write it down?14:23
hayporpodolyaka: i saw a patch recently removing such useless headers14:25
hayporpodolyaka: there are useless in most cases14:25
haypoby the way, # coding: utf-8 is enough14:26
AAzzahaypo: yes it is enough)14:26
haypo"-*-" is ignored and uselss :)14:26
*** ihrachyshka has quit IRC14:26
AAzzahaypo: +1)14:26
AAzzahaypo: sorry for interruption :-[14:27
dhellmannhaypo: isn't the -*- part used by emacs?14:27
rpodolyakahaypo: yeah, I'm more like, do we care?14:27
*** jecarey has joined #openstack-oslo14:28
*** AAzza has left #openstack-oslo14:28
*** sileht has quit IRC14:28
dhellmannrpodolyaka: it seems better to leave the encoding in place14:28
*** sileht has joined #openstack-oslo14:30
rpodolyakadhellmann: yeah, I'd leave it too14:30
* rpodolyaka wishes python 2.x assumed utf-8 by default14:31
*** sileht has quit IRC14:37
*** sileht has joined #openstack-oslo14:39
*** gcb has joined #openstack-oslo14:49
*** sileht has quit IRC14:49
*** sileht has joined #openstack-oslo14:51
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests  https://review.openstack.org/9342414:53
*** gcb has quit IRC14:58
openstackgerritMark McLoughlin proposed a change to openstack/oslo-incubator: cfgfilter: finally make this thing useful!  https://review.openstack.org/9567614:59
openstackgerritMark McLoughlin proposed a change to openstack/oslo-incubator: cfgfilter: fixup use of testmods  https://review.openstack.org/9578814:59
*** morganfainberg_Z is now known as morganfainberg15:04
*** zhiyan_ is now known as zhiyan15:05
*** stannie has joined #openstack-oslo15:09
*** zhiyan is now known as zhiyan_15:14
*** viktors is now known as viktors|afc15:18
*** zhiyan_ is now known as zhiyan15:39
*** zhiyan is now known as zhiyan_15:51
*** shakamunyi has joined #openstack-oslo15:56
*** markmc has quit IRC16:16
*** amotoki has joined #openstack-oslo16:17
*** ihrachyshka has joined #openstack-oslo16:19
*** ihrachyshka has quit IRC16:25
*** ihrachyshka has joined #openstack-oslo16:25
*** nacim_ has quit IRC16:42
*** zhiyan_ is now known as zhiyan16:42
*** ihrachyshka has quit IRC16:46
*** ihrachyshka has joined #openstack-oslo16:48
*** zhiyan is now known as zhiyan_16:51
*** harlowja_away is now known as harlowja_16:55
*** yamahata has quit IRC16:55
*** praneshp has joined #openstack-oslo16:58
*** dims has quit IRC17:08
*** dims has joined #openstack-oslo17:13
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests  https://review.openstack.org/9342417:21
*** I159 has quit IRC17:22
*** amotoki has quit IRC17:23
*** praneshp has quit IRC17:26
*** praneshp has joined #openstack-oslo17:31
*** zhiyan_ is now known as zhiyan17:43
*** zhiyan is now known as zhiyan_17:52
openstackgerritMatthew Treinish proposed a change to openstack/oslo.config: Add a doc sample for how to use the required field  https://review.openstack.org/9584617:59
*** mtreinish has joined #openstack-oslo18:10
*** zhiyan_ is now known as zhiyan18:44
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Adds a single threaded flow conductor  https://review.openstack.org/9393018:46
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Adds a single threaded flow conductor  https://review.openstack.org/9393018:47
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Adds a single threaded flow conductor  https://review.openstack.org/9393018:48
*** zhiyan is now known as zhiyan_18:55
openstackgerritSean Dague proposed a change to openstack/oslo-incubator: always log a traceback in the sys.excepthook  https://review.openstack.org/9586018:55
*** arnaud__ has joined #openstack-oslo18:59
*** arnaud__ has quit IRC18:59
*** gordc has joined #openstack-oslo19:01
*** rpodolyaka1 has joined #openstack-oslo19:04
*** ihrachyshka has quit IRC19:06
*** rpodolyaka1 has quit IRC19:12
openstackgerritMartin Geisler proposed a change to openstack/oslo-incubator: Use correct "coding" file variable to set file encoding  https://review.openstack.org/9586519:14
openstackgerritMatthew Treinish proposed a change to openstack/oslo.config: Add a doc sample for how to use the required field  https://review.openstack.org/9584619:16
openstackgerritMartin Geisler proposed a change to openstack/oslo-incubator: Use correct file variable to set encoding  https://review.openstack.org/9586519:17
openstackgerritMartin Geisler proposed a change to openstack/pycadf: Use Emacs-friendly file variable to set file encoding  https://review.openstack.org/9586919:27
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: add spec for remove-context-adapter  https://review.openstack.org/9587019:31
*** markmc has joined #openstack-oslo19:33
*** zhiyan_ is now known as zhiyan19:46
*** dims has quit IRC19:54
*** zhiyan is now known as zhiyan_19:55
*** markmcclain has joined #openstack-oslo20:00
*** boris-42 has quit IRC20:21
*** boris-42 has joined #openstack-oslo20:22
*** dims has joined #openstack-oslo20:26
openstackgerritDirk Mueller proposed a change to openstack/oslo.db: Robust detection for postgresql connection errors  https://review.openstack.org/9301020:29
*** zhiyan_ is now known as zhiyan20:47
*** jecarey has quit IRC20:48
*** zhiyan is now known as zhiyan_20:56
*** jecarey has joined #openstack-oslo21:02
*** markmc has quit IRC21:11
*** dims has quit IRC21:13
openstackgerritGreg Hill proposed a change to openstack/taskflow: add the ability to inject arguments into tasks at task creation  https://review.openstack.org/9502921:17
bnemecdhellmann: Interesting discussion on https://review.openstack.org/#/c/93010/1/oslo/db/sqlalchemy/session.py21:17
bnemecoslo.db is in kind of a weird gray area right now because it's graduated but hasn't released yet.21:17
dhellmannbnemec: yeah, I need victors or rpodolyaka to prepare a list of the changes that are blocking an initial release21:21
*** markmcclain has quit IRC21:21
bnemecdhellmann: Just for context, I had -2'd https://review.openstack.org/#/c/86930/ because I thought oslo db was being treated as stable, but reading the graduation guidelines closer I see that's not technically true.21:26
bnemecIt just says we need to maintain features and bug fixes in both, but the order where they go in isn't specified.21:26
dhellmannbnemec: we do want changes applied to the library before the incubator once graduation starts21:26
*** arnaud__ has joined #openstack-oslo21:26
dhellmannit's just that we're more lenient about the nature of those changes until the library is actually released21:27
bnemecdhellmann: Okay, that makes sense.  I'll comment on the review.21:27
bnemecdhellmann: Thanks21:27
dhellmannnp21:27
*** mrda_away is now known as mrda21:28
dhellmannbnemec: we have several specs up to be reviews, too, if you have time -- I would much rather focus on those than on incubator patches this week21:29
dhellmannI see you've +2 a few already21:29
openstackgerritFeodor Tersin proposed a change to openstack/cliff: Make show option compatible with Python 2.6.  https://review.openstack.org/9591621:32
-openstackstatus- NOTICE: Zuul is offline due to an operational issue; ETA 2200 UTC.21:33
*** ChanServ changes topic to "Zuul is offline due to an operational issue; ETA 2200 UTC."21:33
*** stannie has quit IRC21:43
bnemecdhellmann: Yeah, I'll try to get through as many specs as I can this week.  I just wanted to bring up the db change because I wasn't sure exactly what our policy on that was.21:44
* bnemec needs a review week21:44
dhellmannbnemec: yeah, it's only loosely defined at this point21:44
dhellmanna review week sounds exhausting :-)21:44
bnemecAnd I still wouldn't get through everything I should review. :-)21:45
*** dims has joined #openstack-oslo21:45
*** mriedem has quit IRC21:46
*** zhiyan_ is now known as zhiyan21:48
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: add spec for graduate-oslo-log  https://review.openstack.org/9592921:52
openstackgerritFeodor Tersin proposed a change to openstack/cliff: Make show option compatible with Python 2.6.  https://review.openstack.org/9591621:53
*** harlowja_ is now known as harlowja_away21:57
*** zhiyan is now known as zhiyan_21:57
*** harlowja_away is now known as harlowja_21:57
*** shakamunyi has quit IRC22:03
*** yamahata has joined #openstack-oslo22:07
*** shakamunyi has joined #openstack-oslo22:10
*** ChanServ changes topic to "#openstack-oslo"22:10
-openstackstatus- NOTICE: Zuul is started and processing changes that were in the queue when it was stopped. Changes uploaded or approved since then will need to be re-approved or rechecked.22:10
*** arnaud__ has quit IRC22:31
*** arnaud has joined #openstack-oslo22:47
*** zhiyan_ is now known as zhiyan22:48
*** gordc has quit IRC22:52
*** zhiyan is now known as zhiyan_22:58
*** yamahata has quit IRC23:16
*** arnaud has quit IRC23:55

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