Tuesday, 2018-08-21

*** cdent has quit IRC00:02
*** rpioso is now known as rpioso|afk00:13
*** phuongnh has joined #openstack-oslo01:02
*** njohnston has quit IRC01:03
*** njohnston has joined #openstack-oslo02:02
*** gcb_ has joined #openstack-oslo02:07
*** larainema has joined #openstack-oslo03:21
*** hoonetorg has quit IRC04:29
*** hoonetorg has joined #openstack-oslo04:47
*** hoonetorg has quit IRC04:54
*** hoonetorg has joined #openstack-oslo05:11
*** jaosorior has quit IRC05:45
*** jaosorior has joined #openstack-oslo05:47
*** rcernin has quit IRC06:38
*** rcernin has joined #openstack-oslo06:40
*** pcaruana has joined #openstack-oslo06:42
*** rcernin has quit IRC06:51
*** snapiri has quit IRC07:02
*** snapiri has joined #openstack-oslo07:05
*** mattgo has joined #openstack-oslo07:39
*** shardy has joined #openstack-oslo08:00
openstackgerritYikun Jiang (Kero) proposed openstack/oslotest master: Add uuidsentinel test module  https://review.openstack.org/59406808:09
*** e0ne has joined #openstack-oslo08:10
*** cdent has joined #openstack-oslo08:33
*** shardy has quit IRC08:43
*** shardy has joined #openstack-oslo08:44
openstackgerritYikun Jiang (Kero) proposed openstack/oslotest master: Add uuidsentinel test module  https://review.openstack.org/59406808:52
openstackgerritClaudiu Belu proposed openstack/oslo.privsep master: Avoids calling ffi.dlopen(None) on Windows  https://review.openstack.org/59408909:36
openstackgerritClaudiu Belu proposed openstack/oslo.service master: Skips signal handling on Windows  https://review.openstack.org/59389709:51
*** rpioso|afk is now known as rpioso10:19
*** rpioso is now known as rpioso|afk11:09
*** snapiri has quit IRC11:16
*** ansmith has quit IRC11:33
*** e0ne has quit IRC11:36
openstackgerritMerged openstack/oslo.rootwrap master: add lib-forward-testing-python3 test job  https://review.openstack.org/59119211:38
openstackgerritYikun Jiang (Kero) proposed openstack/oslotest master: Add uuidsentinel test module  https://review.openstack.org/59406811:43
*** phuongnh has quit IRC11:50
*** ujjain has quit IRC11:51
*** larainema has quit IRC11:54
*** ujjain has joined #openstack-oslo12:00
*** raildo has joined #openstack-oslo12:17
*** moguimar has quit IRC12:31
*** ssbarnea is now known as ssbarnea|ruck12:35
*** kgiusti has joined #openstack-oslo12:45
*** efried is now known as efried_goatin13:03
*** ansmith has joined #openstack-oslo13:09
*** moguimar has joined #openstack-oslo13:09
*** r-mibu has joined #openstack-oslo13:15
*** efried_goatin is now known as efried13:37
kgiustidhellmann: ping Q: oslo.messaging seems to be inconsistent when it comes to logging: some files import the standard library, others use oslo.log.13:37
kgiustidhellmann: in reference to https://bugs.launchpad.net/oslo.messaging/+bug/1788119 - should oslo.messaging be using oslo.log throughout the code?13:38
openstackLaunchpad bug 1788119 in oslo.messaging "Logger should be got by oslo.log getLogger instead of logging package" [Undecided,New]13:38
kgiustidhellmann: or should we use the python library and remove the dependency on oslo.log?13:39
*** e0ne has joined #openstack-oslo13:40
openstackgerritStephen Finucane proposed openstack/oslo.config master: sphinxext: Print output to logs, not to file  https://review.openstack.org/58352413:51
openstackgerritStephen Finucane proposed openstack/oslo.config master: sphinxext: Handle multi-line group descriptions  https://review.openstack.org/58352513:51
stephenfinAnyone fancy taking a look at those two patches?13:51
stephenfinSome fixes for equivalent functionality in oslo.policy incoming shortly13:51
openstackgerritEric Fried proposed openstack/oslo.utils master: UUID sentinel  https://review.openstack.org/59417913:53
efriedcdent, dhellmann, bnemec: ^ alternative to https://review.openstack.org/59406813:54
openstackgerritStephen Finucane proposed openstack/oslo.policy master: sphinxext: Start parsing 'DocumentedRuleDefault.description' as rST  https://review.openstack.org/59422214:05
dhellmannkgiusti : we have to be careful about circular dependencies between the oslo libraries. I don't know if that applies here, but it's something to keep in mind. That said, oslo.logging does have some capabilities that the stdlib doesn't have, especially the keyword argument handling adapter. That's probably why oslo.log is used some places14:05
dhellmannI'm not sure it matters about being 100% consistent14:06
*** pbourke has quit IRC14:06
lbragstadstephenfin: i can review the oslo.policy patch whenever you get that up for review14:06
dhellmannthough that bug sounds interesting14:06
lbragstadnice catch14:06
stephenfinlbragstad:  https://review.openstack.org/59422214:06
stephenfinIt's trivial enough14:06
dhellmannkgiusti : that bug is most likely caused by calling getLogger with inconsistent names. If you use __name__ you get oslo_messaging.something but in some places we ask explicitly for 'oslo.messsaging.something' (note the _ vs . there)14:07
*** pbourke has joined #openstack-oslo14:07
dhellmannand I think the version of getLogger in oslo.log does that name translation14:08
dhellmannso yeah, I guess we should at least be consistent about that14:08
*** jbadiapa has joined #openstack-oslo14:11
kgiustidhellmann: thanks for the info.  I'll dig a bit further.  I really wanted to be sure there wasn't any plans to remove the oslo.log dependency from o.m. before I tried any fixes.14:23
*** lbragstad has quit IRC14:26
dhellmannkgiusti : I think there's a link between messaging and logging for notifications, but I can't remember which direction that goes14:30
dhellmannlike there's a notification handler that logs, maybe?14:30
kgiustidhellmann: yup - you can configure logging as an alternative to message generation.  I'm not inclined to touch that code... ;)14:32
dhellmannyeah14:37
*** spilla has joined #openstack-oslo14:46
*** rpioso|afk is now known as rpioso14:48
*** lbragstad has joined #openstack-oslo15:02
*** pcaruana has quit IRC15:09
*** sambetts|afk is now known as sambetts15:16
openstackgerritmelissaml proposed openstack/taskflow master: Remove the duplicated word  https://review.openstack.org/59431915:28
*** jbadiapa has quit IRC15:33
*** e0ne has quit IRC15:44
*** r-mibu has quit IRC16:04
*** rpioso is now known as rpioso|afk16:12
openstackgerritClaudiu Belu proposed openstack/oslo.service master: Skips signal handling on Windows  https://review.openstack.org/59389716:34
*** sambetts is now known as sambetts|afk16:52
*** mattgo has quit IRC16:54
*** shardy has quit IRC16:57
*** nicolasbock has joined #openstack-oslo16:59
*** sean-k-mooney has quit IRC17:00
openstackgerrit98k proposed openstack/pycadf master: import zuul job settings from project-config  https://review.openstack.org/59446117:09
openstackgerrit98k proposed openstack/pycadf master: switch documentation job to new PTI  https://review.openstack.org/59446217:09
openstackgerrit98k proposed openstack/pycadf master: add python 3.6 unit test job  https://review.openstack.org/59446317:09
openstackgerrit98k proposed openstack/pycadf master: add lib-forward-testing-python3 test job  https://review.openstack.org/59446417:09
openstackgerrit98k proposed openstack/pycadf master: import zuul job settings from project-config  https://review.openstack.org/59446517:09
openstackgerrit98k proposed openstack/pycadf master: switch documentation job to new PTI  https://review.openstack.org/59446617:09
*** spilla has quit IRC17:35
*** e0ne has joined #openstack-oslo17:42
*** dave-mccowan has quit IRC17:44
*** jbadiapa has joined #openstack-oslo18:09
openstackgerritSam Yaple proposed openstack/oslo.log master: [WIP] Fix fluentd outputter  https://review.openstack.org/59450318:13
*** dave-mccowan has joined #openstack-oslo18:30
*** dave-mccowan has quit IRC18:35
*** NobodyCam has quit IRC18:37
*** jokke_ has quit IRC18:37
*** ttx has quit IRC18:37
*** fungi has quit IRC18:37
*** dstanek has quit IRC18:42
*** hrybacki has quit IRC18:42
*** mnaser has quit IRC18:42
*** dstanek has joined #openstack-oslo18:43
*** fungi has joined #openstack-oslo18:48
*** e0ne has quit IRC18:57
*** cdent has quit IRC19:16
*** rpioso|afk is now known as rpioso19:17
*** bnemec has quit IRC19:20
*** bnemec has joined #openstack-oslo19:20
*** melwitt has quit IRC19:33
*** melwitt has joined #openstack-oslo19:34
*** sambetts|afk has quit IRC19:42
*** sambetts_ has joined #openstack-oslo19:45
*** samueldmq has quit IRC19:46
*** samueldmq has joined #openstack-oslo19:49
openstackgerritMerged openstack/taskflow master: Remove the duplicated word  https://review.openstack.org/59431919:58
*** gmann_ has joined #openstack-oslo20:13
*** jaosorior_ has joined #openstack-oslo20:14
*** jaosorior has quit IRC20:17
*** edmondsw_ has joined #openstack-oslo20:18
*** moguimar has quit IRC20:19
*** edmondsw has quit IRC20:19
*** gmann has quit IRC20:19
*** edmondsw_ is now known as edmondsw20:19
*** gmann_ is now known as gmann20:19
*** kgiusti has left #openstack-oslo20:27
openstackgerritMerged openstack/oslo.log master: Fix lower-constraints job  https://review.openstack.org/58318220:41
*** ansmith has quit IRC20:43
*** raildo has quit IRC20:46
*** harlowja has joined #openstack-oslo21:02
openstackgerritMerged openstack/oslo.log master: Migrate to stestr  https://review.openstack.org/57120521:11
openstackgerritMerged openstack/oslo.log master: Follow the new PTI for document build  https://review.openstack.org/54933821:11
openstackgerritMerged openstack/oslo.log master: import zuul job settings from project-config  https://review.openstack.org/58869821:11
openstackgerritMerged openstack/oslo.log master: rewrite tests to not rely on implementation details of logging module  https://review.openstack.org/59125921:11
openstackgerritMerged openstack/oslo.log master: add python 3.6 unit test job  https://review.openstack.org/58960021:11
openstackgerritMerged openstack/oslo.log master: add lib-forward-testing-python3 test job  https://review.openstack.org/59118621:11
*** ansmith has joined #openstack-oslo21:38
*** rha has quit IRC22:02
*** rha has joined #openstack-oslo22:03
*** rcernin has joined #openstack-oslo22:10

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