Monday, 2016-01-25

*** EinstCrazy has joined #openstack-oslo00:03
*** EinstCrazy has quit IRC00:08
*** thumpba has joined #openstack-oslo00:15
*** dims_ has joined #openstack-oslo00:19
*** achanda has joined #openstack-oslo00:19
*** thumpba has quit IRC00:20
*** dims has quit IRC00:22
*** rcernin has quit IRC00:30
*** boris-42 has joined #openstack-oslo00:40
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the completer resume() function easier to understand  https://review.openstack.org/27171500:52
*** harlowja_at_home has quit IRC00:56
*** dims_ has quit IRC01:00
*** achanda has quit IRC01:13
*** dims has joined #openstack-oslo01:27
*** dims has quit IRC01:27
*** hogepodge has joined #openstack-oslo01:34
*** hogepodge has quit IRC01:38
*** amotoki has joined #openstack-oslo01:39
*** nihilifer has quit IRC01:46
*** nihilifer has joined #openstack-oslo01:53
*** thumpba has joined #openstack-oslo02:18
*** yamamoto has joined #openstack-oslo02:20
*** thumpba has quit IRC02:23
*** yamamoto has quit IRC02:31
*** EinstCrazy has joined #openstack-oslo02:33
*** EinstCrazy has quit IRC02:40
*** jeckersb is now known as jeckersb_gone02:48
*** thumpba has joined #openstack-oslo02:54
*** zz_dimtruck is now known as dimtruck03:04
*** amotoki has quit IRC03:10
*** dimtruck is now known as zz_dimtruck03:13
*** vilobhmm11 has joined #openstack-oslo03:17
*** amotoki has joined #openstack-oslo03:21
*** yamamoto_ has joined #openstack-oslo03:23
openstackgerritMerged openstack/oslo.versionedobjects: Use oslo_utils for tupleizing versions  https://review.openstack.org/27143503:29
*** amotoki has quit IRC03:30
*** amotoki has joined #openstack-oslo03:35
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.utils: Add method check_string_length  https://review.openstack.org/26733103:36
*** zz_dimtruck is now known as dimtruck03:36
*** dimtruck is now known as zz_dimtruck03:48
*** amotoki has quit IRC04:01
*** pballand has quit IRC04:15
*** amotoki has joined #openstack-oslo04:18
*** zz_dimtruck is now known as dimtruck04:52
*** vilobhmm11 has quit IRC04:57
*** vilobhmm11 has joined #openstack-oslo05:10
*** dimtruck is now known as zz_dimtruck05:11
openstackgerritBoris Pavlovic proposed openstack/osprofiler: [DO NOT MERGE] Testing  https://review.openstack.org/26990805:21
*** EinstCrazy has joined #openstack-oslo05:23
*** rcernin has joined #openstack-oslo05:54
*** achanda has joined #openstack-oslo05:56
*** EinstCrazy has quit IRC05:59
*** thumpba has quit IRC05:59
*** EinstCrazy has joined #openstack-oslo06:03
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.utils: Add method check_string_length  https://review.openstack.org/26733106:05
*** vilobhmm11 has quit IRC06:14
openstackgerritjaveme proposed openstack/oslo.messaging: Fixed a couple of pep8 errors/warnings  https://review.openstack.org/27125306:16
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.messaging: test: Don't cap message's reply time  https://review.openstack.org/26462106:22
*** jamielennox is now known as jamielennox|away06:31
*** jamielennox|away is now known as jamielennox06:33
*** EinstCrazy has quit IRC06:47
*** vilobhmm11 has joined #openstack-oslo06:49
openstackgerritMerged openstack-dev/pbr: Support entry point patching on setuptools < 12  https://review.openstack.org/26999906:50
openstackgerritjaveme proposed openstack/oslo.messaging: assertEquals is deprecated, use assertEqual  https://review.openstack.org/27190906:56
*** davidlenwell has quit IRC07:00
*** davidlenwell has joined #openstack-oslo07:02
*** gcb has quit IRC07:03
*** EinstCrazy has joined #openstack-oslo07:05
*** amotoki_ has joined #openstack-oslo07:09
*** amotoki has quit IRC07:13
*** gcb has joined #openstack-oslo07:14
*** deray has joined #openstack-oslo07:15
*** salv-orlando has joined #openstack-oslo07:18
*** davidlenwell has quit IRC07:24
*** achanda has quit IRC07:24
*** davidlenwell has joined #openstack-oslo07:26
*** davidlenwell has quit IRC07:29
*** davidlenwell has joined #openstack-oslo07:33
*** andymaier has joined #openstack-oslo07:34
*** achanda has joined #openstack-oslo07:35
*** davidlenwell has quit IRC07:41
*** ihrachys has joined #openstack-oslo07:41
*** EinstCrazy has quit IRC07:43
*** davidlenwell has joined #openstack-oslo07:45
*** nkrinner has joined #openstack-oslo07:48
*** EinstCrazy has joined #openstack-oslo07:50
*** achanda has quit IRC07:51
openstackgerritMerged openstack/oslo.messaging: replace string format arguments with function parameters  https://review.openstack.org/27127007:54
*** achanda has joined #openstack-oslo07:54
openstackgerritjaveme proposed openstack/oslo.messaging: Fixed a couple of pep8 errors/warnings  https://review.openstack.org/27125307:56
*** amotoki_ is now known as amotoki07:56
*** ihrachys has quit IRC07:57
*** achanda has quit IRC07:58
*** elarson has quit IRC07:59
*** pblaho has joined #openstack-oslo08:00
openstackgerritJamie Lennox proposed openstack/oslo.context: Add roles to context  https://review.openstack.org/27192808:04
openstackgerritJamie Lennox proposed openstack/oslo.context: Standardize an oslo.policy credentials dictionary  https://review.openstack.org/27192908:04
*** elarson has joined #openstack-oslo08:05
*** EinstCrazy has quit IRC08:05
*** EinstCrazy has joined #openstack-oslo08:06
*** markus_z has joined #openstack-oslo08:10
*** binarin has quit IRC08:11
*** EinstCrazy has quit IRC08:11
*** amotoki has quit IRC08:21
*** ihrachys has joined #openstack-oslo08:28
*** EinstCrazy has joined #openstack-oslo08:30
*** shardy has joined #openstack-oslo08:30
*** thumpba has joined #openstack-oslo08:31
openstackgerritIWAMOTO Toshihiro proposed openstack/oslo.service: Add profile_file option for profiling  https://review.openstack.org/27193908:36
*** thumpba has quit IRC08:36
*** amotoki has joined #openstack-oslo08:40
*** amotoki has quit IRC08:49
*** amotoki has joined #openstack-oslo08:52
*** amotoki has quit IRC08:55
*** achanda has joined #openstack-oslo08:58
*** jecarey_ has joined #openstack-oslo08:59
*** jecarey has quit IRC09:01
*** salv-orlando has quit IRC09:04
*** achanda has quit IRC09:05
*** amotoki has joined #openstack-oslo09:08
*** jecarey_ has quit IRC09:10
*** binarin has joined #openstack-oslo09:11
*** bapalm has quit IRC09:12
*** bnemec has quit IRC09:12
*** binarin has quit IRC09:15
*** binarin has joined #openstack-oslo09:15
*** nkrinner has quit IRC09:16
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.messaging: test: Don't cap message's reply time  https://review.openstack.org/26462109:17
*** amotoki has quit IRC09:18
*** nkrinner has joined #openstack-oslo09:18
*** EinstCra_ has joined #openstack-oslo09:19
*** EinstCrazy has quit IRC09:21
*** mhickey has joined #openstack-oslo09:22
*** yassine__ has joined #openstack-oslo09:26
*** yassine__ has quit IRC09:26
*** yassine__ has joined #openstack-oslo09:27
*** zakora has joined #openstack-oslo09:27
*** vilobhmm11 has quit IRC09:29
*** amotoki has joined #openstack-oslo09:30
openstackgerritjaveme proposed openstack/oslo.messaging: Fixed some warnings about imports  https://review.openstack.org/27197309:37
*** achanda has joined #openstack-oslo10:02
*** salv-orlando has joined #openstack-oslo10:05
openstackgerritjaveme proposed openstack/oslo.messaging: Fixed some warnings about imports and variable  https://review.openstack.org/27197310:07
*** achanda has quit IRC10:07
*** EinstCra_ has quit IRC10:08
*** salv-orlando has quit IRC10:12
*** EinstCrazy has joined #openstack-oslo10:18
*** salv-orlando has joined #openstack-oslo10:22
*** SlickN1k has joined #openstack-oslo10:28
*** ihrachys has quit IRC10:29
*** vipul has joined #openstack-oslo10:29
*** SlickNik has quit IRC10:29
*** SlickN1k is now known as SlickNik10:29
*** vipuls has quit IRC10:30
*** thumpba has joined #openstack-oslo10:33
*** SlickNik has quit IRC10:34
*** vipul has quit IRC10:34
*** SlickNik has joined #openstack-oslo10:34
*** vipul has joined #openstack-oslo10:35
*** SlickN1k has joined #openstack-oslo10:37
*** vipuls has joined #openstack-oslo10:37
*** thumpba has quit IRC10:39
*** vipul has quit IRC10:39
*** SlickNik has quit IRC10:39
*** SlickN1k is now known as SlickNik10:39
*** vipuls has quit IRC10:51
openstackgerritjaveme proposed openstack/oslo.messaging: Fixed some warnings about imports and variable  https://review.openstack.org/27197310:51
*** vipul has joined #openstack-oslo10:51
*** SlickN1k has joined #openstack-oslo10:53
*** SlickNik has quit IRC10:54
*** SlickN1k is now known as SlickNik10:54
*** bapalm has joined #openstack-oslo10:57
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: Check devstack gate zmq  https://review.openstack.org/22977911:08
*** yamamoto_ has quit IRC11:12
*** EinstCrazy has quit IRC11:22
*** EinstCrazy has joined #openstack-oslo11:23
*** ihrachys has joined #openstack-oslo11:30
*** ndipanov has joined #openstack-oslo11:33
*** ozamiatin has joined #openstack-oslo11:38
*** dims has joined #openstack-oslo11:57
*** ndipanov has quit IRC11:57
*** yamamoto has joined #openstack-oslo11:58
*** sputnik13 has joined #openstack-oslo11:58
*** yamamoto_ has joined #openstack-oslo11:59
haypohi. my change https://review.openstack.org/#/c/250731/ was approved, but jenkins failed as usually. gerrit says "Needs Verified" but i see "Verified: +1 Jenkins"12:01
haypodims sent a "recheck" this morning which was followed by jenkins's result "Build succeeded"12:02
*** pbourke has quit IRC12:02
haypoi voted +2a to try to "force" jenkins to move to the next step12:02
*** pbourke has joined #openstack-oslo12:02
haypobut i don't know if it's necessary12:02
haypoi'm expecting the message "start gate job"12:02
dimshaypo : does it have a Depends-On to something else?12:03
*** yamamoto has quit IRC12:03
haypodims: yes, it depends on 4 other changes12:03
dimshaypo : so those have to merge first12:03
haypodims: 6 of them are merged, 1 of them is abandonned12:03
dimshaypo : remove the abandoned one and +2A it please12:04
haypodims: i only added Depends-On to ensure that my oslo.context is not merged by mistake, before my work to prepare other projects for the new API is merged12:04
haypodims: are you ok if i remove all Depends-On? maybe mention them in the commit message, but not use the strict tag?12:05
dimsi know. i meant remove the Depends-On on the abandoned review from the commit message of yours12:05
*** achanda has joined #openstack-oslo12:05
dimssure12:05
dimsye12:05
haypodims: ok, i will do that12:05
dimsthanks haypo12:05
haypodims: why? *you* are helping me, not the opposite :-D12:06
dimshaha :) coffee has not kicked in yet12:06
haypodims: lol12:07
haypodims: are you working week-ends too?12:07
haypodims: i saw that you sent an email about eventlet 0.18 regression ;)12:08
*** achanda has quit IRC12:10
openstackgerritVictor Stinner proposed openstack/oslo.context: Fix request_id type on Python 3: use text (Unicode)  https://review.openstack.org/25073112:10
haypodims: ^^ ok, i removed Depends-On and reapproved my own change. the patch itself didn't change, i only replaced Depends-On in the commit message with a list of changes, and rebsed my change12:14
haypodims: let's see if Jenkins prefer that12:14
haypoi hope that it's the last time that i have to break an API in openstack :-/ it's annoying12:14
*** yamamoto_ has quit IRC12:17
*** dims_ has joined #openstack-oslo12:22
*** dims has quit IRC12:23
*** yamamoto has joined #openstack-oslo12:26
*** hogepodge has joined #openstack-oslo12:27
*** gordc has joined #openstack-oslo12:30
*** hogepodge has quit IRC12:32
*** ericksonsantos has quit IRC12:49
*** thumpba has joined #openstack-oslo12:50
*** lucasagomes is now known as lucas-hungry12:52
*** gcb has quit IRC12:53
*** thumpba has quit IRC12:55
*** salv-orlando has quit IRC13:02
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Fix publisher proxy  https://review.openstack.org/27208513:03
*** yamamoto has quit IRC13:03
*** yamamoto has joined #openstack-oslo13:04
*** yamamoto has quit IRC13:08
*** amrith is now known as _amrith_13:15
*** yamamoto has joined #openstack-oslo13:17
*** amotoki has quit IRC13:19
*** dims_ has quit IRC13:28
*** edmondsw has joined #openstack-oslo13:32
*** markus_z has quit IRC13:32
*** dims has joined #openstack-oslo13:33
*** pradk has joined #openstack-oslo13:39
*** pradk has quit IRC13:39
*** dstanek has quit IRC13:41
*** fesp has joined #openstack-oslo13:42
*** dstanek has joined #openstack-oslo13:42
HenryGWhere is a good place to ask/discuss about debtcollector?13:55
haypoHenryG: harlowja knows it well :) he will be available on this channel, but later ;)13:56
HenryGhaypo: thanks13:57
*** salv-orlando has joined #openstack-oslo13:57
haypoHenryG: in the meanwhile, just ask, don't ask to ask :)13:57
*** fesp has quit IRC13:58
HenryGhaypo: yes I know, but debtcollector is not an oslo module so I first wanted to be sure I didn't go barreling into the wrong channel13:59
HenryGanyways, my question is about this: https://review.openstack.org/#/c/242393/11/neutron/plugins/ml2/models.py13:59
HenryGIs that not what moved_class is for?14:00
*** lucas-hungry is now known as lucasagomes14:00
haypojd__: ^^ any idea?14:01
*** yamamoto has quit IRC14:02
HenryGI may be mis-interpreting gongysh's response though. Maybe he means the move of the class is not a deprecation case. I will ask.14:02
haypoHenryG: (i don't know debtcollector)14:03
*** nkrinner has quit IRC14:04
jd__I think gongysh is missing the point that this is a class that _external_ plugins may rely on14:04
jd__this is why he replied that14:04
*** rlrossit has joined #openstack-oslo14:04
jd__at least it's my understanding :)14:04
HenryGjd__: thanks, I just responded to him with that question14:04
*** nkrinner has joined #openstack-oslo14:05
*** achanda has joined #openstack-oslo14:07
*** thumpba has joined #openstack-oslo14:10
*** achanda has quit IRC14:13
*** kgiusti has joined #openstack-oslo14:18
*** jecarey has joined #openstack-oslo14:24
openstackgerritMerged openstack/oslo.context: Fix request_id type on Python 3: use text (Unicode)  https://review.openstack.org/25073114:28
*** hogepodge has joined #openstack-oslo14:28
*** mdavidson has joined #openstack-oslo14:29
haypodims: ok. ^^ my oslo.context change was merged into master. please increase the major number for the next release, since it breaks the API on Python 314:29
dimshaypo : can you please file a release request? openstack/releases14:29
haypodims: just for oslo.context ?14:30
dimshaypo : yep14:30
haypodims: ok. i can, but not right now14:30
dimshaypo : yep, when you get a chance14:30
*** rlrossit_ has joined #openstack-oslo14:31
*** hogepodge has quit IRC14:33
*** nikhil_k has quit IRC14:33
*** nikhil has joined #openstack-oslo14:34
*** hogepodge has joined #openstack-oslo14:34
*** rlrossit has quit IRC14:34
*** pratikmallya has joined #openstack-oslo14:36
*** jeckersb_gone is now known as jeckersb14:36
stevemardims: poke14:47
stevemarnoticed that the sorting of config options has changed in the latest oslo.config, https://bugs.launchpad.net/oslo.config/+bug/1536899 anyone know how to fix that?14:47
openstackLaunchpad bug 1536899 in oslo.config "config generator sorting has changed" [Undecided,New]14:47
dimsstevemar : saw the bug, haven't dug in yet. travelling this afternoon, will be partially online till thu morning14:49
*** _amrith_ is now known as amrith14:55
*** deray has quit IRC15:01
*** pradk has joined #openstack-oslo15:02
*** gcb has joined #openstack-oslo15:06
stevemardims: cool15:07
stevemardims: any tips on where to look to solve the issue :)15:07
*** achanda has joined #openstack-oslo15:10
*** achanda has quit IRC15:15
*** mtanino has joined #openstack-oslo15:15
dhellmannstevemar : rbradfor is looking at config-related stuff, he might have insight15:19
stevemardhellmann: danke, i also raised an issue with reno over the weekend15:20
*** sigmavirus24_awa is now known as sigmavirus2415:20
dhellmannstevemar : lp is terrible and doesn't show me dates in the bug list view. which one?15:21
stevemardhellmann: https://bugs.launchpad.net/reno/+bug/153745115:22
openstackLaunchpad bug 1537451 in reno "notes missing/overridden in latest version" [Undecided,New]15:22
stevemardhellmann: sort by number :)15:22
dhellmannok, I'll take a look15:22
dhellmannstevemar : show off15:22
stevemardhellmann: pretty sure it affects all projects - haha15:22
dhellmannstevemar : ack, I'll put it on my list for the week15:22
*** achanda has joined #openstack-oslo15:25
*** achanda has quit IRC15:27
*** devananda has quit IRC15:28
*** dtardivel has joined #openstack-oslo15:30
*** zz_dimtruck is now known as dimtruck15:30
*** achanda has joined #openstack-oslo15:31
*** devananda has joined #openstack-oslo15:31
*** jeckersb is now known as jeckersb_gone15:32
*** jeckersb_gone is now known as jeckersb15:38
*** achanda has quit IRC15:39
*** harlowja_at_home has joined #openstack-oslo15:46
rbradforstevemar, I saw this bug on friday15:50
*** regXboi has joined #openstack-oslo15:51
rbradforstevemar, I observed that oslo-config-generator and docs via the show-options: directive for the same project produced different ordered results15:51
stevemarrbradfor: yeah, it's not quite alphabetical, but it's definitely a change15:52
rbradforstevemar, I'll make it a priority to look into reproducing today.15:53
*** nkrinner has quit IRC16:01
*** yportnova has joined #openstack-oslo16:05
*** salv-orl_ has joined #openstack-oslo16:05
*** salv-orlando has quit IRC16:08
*** pratikma_ has joined #openstack-oslo16:10
*** ozamiatin has quit IRC16:11
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Adds document and configuration guide  https://review.openstack.org/27215216:11
*** andymaier has quit IRC16:12
*** pratikmallya has quit IRC16:13
*** pballand has joined #openstack-oslo16:16
*** ndipanov has joined #openstack-oslo16:22
*** kertis has joined #openstack-oslo16:31
*** rcernin has quit IRC16:31
*** ozamiatin has joined #openstack-oslo16:34
openstackgerritGreg Hill proposed openstack/taskflow: Allow for revert to have a different argument list from execute  https://review.openstack.org/27085316:39
*** ndipanov has quit IRC16:43
*** bnemec has joined #openstack-oslo16:45
*** dukhlov has quit IRC16:45
*** gcb has quit IRC16:46
*** zakora has quit IRC16:48
*** mhickey has quit IRC16:48
*** pratikma_ has quit IRC16:54
*** pratikmallya has joined #openstack-oslo16:54
*** harlowja_at_home has quit IRC16:54
*** ihrachys has quit IRC17:07
dimsharlowja : thanks for taking care of the weekly meeting17:07
dimsdhellmann : harlowja : i will be off and on till thu am this week17:07
*** kgiusti has quit IRC17:20
*** mtanino has quit IRC17:20
*** ajo has quit IRC17:20
*** ajo has joined #openstack-oslo17:33
*** tedross has joined #openstack-oslo17:33
*** tedross has left #openstack-oslo17:34
dhellmanndims : ack, thanks for the heads up17:38
*** kgiusti has joined #openstack-oslo17:39
*** mtanino has joined #openstack-oslo17:44
*** EinstCrazy has quit IRC17:46
*** mtanino_ has joined #openstack-oslo17:51
*** mtanino__ has joined #openstack-oslo17:52
*** lucasagomes is now known as lucas-dinner18:01
openstackgerritRonald Bradford proposed openstack/oslo.context: Add additional unit tests  https://review.openstack.org/27108118:02
*** browne has joined #openstack-oslo18:06
harlowjadims np18:11
rbradforstevemar, I can confirm the problem with change of order of options, working on fix now.18:14
*** yamahata has joined #openstack-oslo18:25
*** pratikmallya has quit IRC18:26
stevemarrbradfor: wunderbar!18:28
*** salv-orl_ has quit IRC18:35
*** vilobhmm11 has joined #openstack-oslo18:45
*** EinstCrazy has joined #openstack-oslo18:46
*** ihrachys has joined #openstack-oslo18:49
*** pcaruana has joined #openstack-oslo18:51
openstackgerritJoshua Harlow proposed openstack/taskflow: Make the completer resume() function easier to understand  https://review.openstack.org/27171518:51
*** EinstCrazy has quit IRC18:52
*** ihrachys has quit IRC18:59
*** shardy is now known as shardy_afk19:00
harlowjadims https://review.openstack.org/#/c/272227/ if u get some time, apparently ironic folks waiting for that19:01
*** openstackgerrit has quit IRC19:02
*** openstackgerrit has joined #openstack-oslo19:02
openstackgerritJoshua Harlow proposed openstack/taskflow: Disable oslotest LOG capturing  https://review.openstack.org/27222819:03
harlowjadims do u know if ^ should start making the logs that happen on failure actually get output19:04
harlowjaie http://logs.openstack.org/53/270853/3/check/gate-taskflow-python27/bb1caa5/console.html failed, but no logs where output anywhere :(19:04
dimsharlowja : worth a try19:07
harlowja:)19:07
harlowjak19:07
harlowjalet's see, ha19:07
harlowjawhere art though logs19:08
dimsharlowja : travel day today, hopefully dhellmann can help with that one19:08
harlowjak19:08
harlowjamagic logs go to magic land19:08
harlowjalol19:08
dims:)19:10
*** dims has quit IRC19:28
*** pcaruana has quit IRC19:32
*** salv-orlando has joined #openstack-oslo19:35
*** pcaruana has joined #openstack-oslo19:36
*** pcaruana has quit IRC19:40
*** amrith is now known as _amrith_19:42
*** mhickey has joined #openstack-oslo19:42
*** _amrith_ is now known as amrith19:42
openstackgerritJoshua Harlow proposed openstack/osprofiler: Move osprofiler tests into osprofiler  https://review.openstack.org/27158219:42
*** pcaruana has joined #openstack-oslo19:43
*** rcernin has joined #openstack-oslo19:44
*** salv-orlando has quit IRC19:46
*** EinstCrazy has joined #openstack-oslo19:49
*** yassine__ has quit IRC19:53
*** EinstCrazy has quit IRC19:54
*** pcaruana has quit IRC19:56
*** rlrossit has joined #openstack-oslo20:01
*** rlrossit_ has quit IRC20:04
*** yamahata has quit IRC20:05
*** ihrachys has joined #openstack-oslo20:12
openstackgerritJoshua Harlow proposed openstack/osprofiler: Move osprofiler tests into osprofiler  https://review.openstack.org/27158220:13
*** yamahata has joined #openstack-oslo20:22
openstackgerritJoshua Harlow proposed openstack/osprofiler: Use oslo.utils reflection and avoid refinding decorated name  https://review.openstack.org/27225520:23
openstackgerritJoshua Harlow proposed openstack/taskflow: Add task 'critical' attribute  https://review.openstack.org/27111620:32
*** salv-orlando has joined #openstack-oslo20:32
openstackgerritBoris Pavlovic proposed openstack/osprofiler: [DO NOT MERGE] Testing  https://review.openstack.org/26990820:34
openstackgerritJoshua Harlow proposed openstack/osprofiler: Use oslo.utils reflection and avoid refinding decorated name  https://review.openstack.org/27225520:36
boris-42dims harlowja do we really need to put tests/ under source dir https://review.openstack.org/#/c/271582/ ?20:37
harlowjaboris-42 its the common pattern20:38
harlowjait also includes them in packaging20:38
harlowjaso why not follow common pattern?20:38
boris-42harlowja: but why do you need unit tests in packag?20:38
harlowjai'm not a packager20:38
harlowjalol20:38
boris-42harlowja: I really don't like to mix tests & code20:38
harlowjaso maybe someone else can state why its the good/best thing, afaik its the common pattern that i've seen20:39
boris-42harlowja: so I really don't see good reason to put it under source dir except that everybody is doing that for some reason20:40
boris-42=)20:40
boris-42harlowja: btw I found the problem with osprofiler job20:40
harlowjaand/or such thing being that everyone is doing it likely has a better reason than what i can currently think20:40
boris-42https://review.openstack.org/#/c/272259/20:40
harlowjai'm betting ^ is the truth, not just cause20:40
harlowjacool20:40
harlowjabtw, the whole using of info in that decorator, is sorta busted20:41
harlowjahttps://review.openstack.org/#/c/272255/2/osprofiler/profiler.py20:41
harlowjathat info.copy is really needed, lol20:41
harlowjaotherwise everyone shares the same info and the info['function'] changes all the time20:41
harlowjawhich is ummm, bad20:41
openstackgerritJoshua Harlow proposed openstack/osprofiler: Use oslo.utils reflection and avoid refinding decorated name  https://review.openstack.org/27225520:42
boris-42harlowja: hm don't see the issue in old case20:42
harlowjalet me show u, lol20:43
boris-42harlowja: ya sample lol20:44
*** thumpba has quit IRC20:46
harlowjahttps://gist.github.com/harlowja/0ea75d44eb1b9c0ed7d920:46
harlowjathat info dictionary there will keep on changing, which seems ummm, not what u want20:47
harlowjahttps://gist.github.com/harlowja/09edc0dd4fd1b9309e7b20:47
harlowja^ output20:47
boris-42harlowja: that is a bit wrong usage nO? )20:47
harlowjais it?20:47
harlowjawhy?20:48
harlowjai just dumb library user20:48
boris-42harlowja: ok le'ts merge it20:48
harlowjai didn't notice that info that i gave osprofiler was going to be altered, lol20:48
boris-42harlowja: just for you20:48
boris-42=)20:48
harlowjalol20:48
harlowja#success boris-42 fighting for the dumb users (me)20:49
openstackstatusharlowja: Added success to Success page20:49
harlowjalol20:49
harlowjathere boris-42 now u on https://wiki.openstack.org/wiki/Successes20:50
harlowjau can now get that bonus, lol20:50
harlowjaalso boris-42  https://review.openstack.org/#/c/271579/20:50
boris-42harlowja: do we really need oslo.utils?20:51
harlowjawhy not20:51
boris-42harlowja: https://github.com/openstack/oslo.utils/blob/master/requirements.txt it has really big amount of requriments20:51
boris-42harlowja: like really really relly big20:51
harlowjalol20:51
harlowjaup to u, does it matter?20:51
harlowjaboris-42 https://review.openstack.org/#/c/271579/ is also sorta umm bad, lol20:51
harlowjatrace class decorator will likely apply 2+ times afaik20:52
harlowjagotta protect people from doing dumb things, ha20:52
*** devananda has quit IRC20:53
boris-42harlowja: btw we should get this in https://review.openstack.org/#/c/270898/420:54
boris-42harlowja: after that I will refactor next Dina's patch20:54
harlowjaokie20:55
harlowjadone20:55
*** amotoki has joined #openstack-oslo20:56
*** yamamoto has joined #openstack-oslo20:57
*** dtardivel has quit IRC20:57
harlowjaboris-42 i volunteer u to extra oslo.utils reflection -> https://pypi.python.org/pypi/reflection :-P21:00
harlowjathen many users of the reflection code (and/or that have copied it) can not have to pull in all the oslo.utils stuff21:00
harlowja*volunteer u to make that, lol21:00
openstackgerritJoshua Harlow proposed openstack/osprofiler: Avoid multiple tracing when applying meta or class decorator  https://review.openstack.org/27157921:01
openstackgerritEric Harney proposed openstack/oslo.db: Add exc_filter for invalid Unicode string  https://review.openstack.org/26592121:01
openstackgerritGreg Hill proposed openstack/taskflow: Retrieve the store from flowdetails as well, if it exists  https://review.openstack.org/23994521:04
*** EinstCrazy has joined #openstack-oslo21:09
*** amrith is now known as _amrith_21:12
*** EinstCrazy has quit IRC21:14
*** davidlenwell has quit IRC21:16
*** davidlenwell has joined #openstack-oslo21:18
*** jeckersb is now known as jeckersb_gone21:20
*** kgiusti has left #openstack-oslo21:22
*** dims has joined #openstack-oslo21:27
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow for specifying green threaded to parallel engine  https://review.openstack.org/27228521:36
openstackgerritMerged openstack/osprofiler: Use oslo.utils reflection and avoid refinding decorated name  https://review.openstack.org/27225521:40
openstackgerritRonald Bradford proposed openstack/oslo.config: Retain correct order of defined config options  https://review.openstack.org/27228921:43
boris-42harlowja: so to create one more lib?)21:45
boris-42#success osprofiler functional job passed https://review.openstack.org/#/c/269908/ !21:45
openstackstatusboris-42: Added success to Success page21:45
boris-42harlowja: ^21:45
*** davidlenwell has quit IRC21:55
*** davidlenwell has joined #openstack-oslo21:56
*** regXboi has quit IRC22:01
*** salv-orl_ has joined #openstack-oslo22:06
*** jeckersb_gone is now known as jeckersb22:08
*** salv-orlando has quit IRC22:09
*** mhickey has quit IRC22:10
*** yamamoto has quit IRC22:10
*** shardy_afk has quit IRC22:11
*** dims has quit IRC22:16
*** dims has joined #openstack-oslo22:17
harlowjalol22:17
harlowjanice22:17
harlowjaboris-42 u can create that lib, lol22:17
boris-42harlowja: mreeeeelibs22:18
harlowja:-P22:18
boris-42mreeereposss22:18
harlowja1 billion libs22:18
harlowjalol22:18
boris-42lib per line22:18
boris-42new approach to code22:18
harlowja+222:18
boris-42and 1 service per 5 lines22:18
harlowjaultra-scale22:18
harlowjalol22:18
harlowjasounds like webscale to me22:19
harlowja#success boris-42 is thinking webscale, oh yaaaa22:19
openstackstatusharlowja: Added success to Success page22:19
harlowjalol22:19
boris-42harlowja: =)22:21
boris-42harlowja: yeah baby! https://upload.wikimedia.org/wikipedia/en/4/42/Mike-Myers-Austin-Powers-1-.jpg22:22
harlowjaanyone know if that bot is connected to twitter?22:22
harlowjalol22:22
*** dims has quit IRC22:27
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow for specifying green threaded to parallel engine  https://review.openstack.org/27228522:29
*** rcernin has quit IRC22:31
*** rlrossit has quit IRC22:31
openstackgerritMerged openstack/oslo.context: Add additional unit tests  https://review.openstack.org/27108122:34
*** davidlenwell has quit IRC22:34
*** davidlenwell has joined #openstack-oslo22:36
openstackgerritGreg Hill proposed openstack/taskflow: Allow for revert to have a different argument list from execute  https://review.openstack.org/27085322:43
openstackgerritGreg Hill proposed openstack/taskflow: Refactor Atom/BaseTask/Task/Retry class hierarchy  https://review.openstack.org/26713122:43
openstackgerritRonald Bradford proposed openstack/oslo.config: Retain correct order of defined config options  https://review.openstack.org/27228922:43
harlowjaboris-42 i'm also pretty sure trace_cls is messed up now that i try to get https://review.openstack.org/#/c/271579/ to work22:56
openstackgerritRonald Bradford proposed openstack/oslo.config: Retain correct order of defined config options  https://review.openstack.org/27228922:57
harlowjawill make example22:58
boris-42harlowja: you are doing some mess23:03
boris-42=)23:03
harlowjalol23:03
openstackgerritJoshua Harlow proposed openstack/osprofiler: Track what was traced  https://review.openstack.org/27231423:03
harlowjaboris-42 run ^ with https://gist.github.com/harlowja/851679cc5f513aa54a9423:04
boris-42harlowja: +1 please https://review.openstack.org/#/c/272259/23:04
harlowjabasically trace_cls imho is ummm, def not gonna work if subclasses ever trace themselves23:04
harlowjabecause said tracing messes up the parent classes tracing, lol23:05
harlowjathe usage of settattr in trace_cls is bad23:06
harlowjato much settattr, lol23:06
openstackgerritMerged openstack/taskflow: Use automaton's converters/pydot  https://review.openstack.org/20800623:07
*** sigmavirus24 is now known as sigmavirus24_awa23:07
boris-42harlowja: you "Track what was traced" philosopher!23:08
*** dimtruck is now known as zz_dimtruck23:08
*** amotoki has quit IRC23:12
harlowjalol23:12
harlowjawelll trace_cls isn't easily fixable i thinks23:12
harlowjathe metaclass is prob ok, trace_cls mutates itself and parent classes and stuff23:12
boris-42harlowja: dangerous stuff lol23:14
harlowjaya, crazy shit man23:14
harlowjalol23:14
harlowjawho made that :-P23:14
boris-42harlowja: I made that =)23:14
harlowjadef crzy then23:14
harlowjalol23:14
harlowjaboris-the-crazy23:14
harlowjalol23:14
harlowjamaybe timmieee made it23:15
harlowjalol23:15
*** amotoki has joined #openstack-oslo23:15
*** amotoki has quit IRC23:16
harlowjaya, so boris-42 not sure what u want to do about that, afaik https://review.openstack.org/#/c/269860/ doesn't fix it either :-/23:17
harlowjamaybe just putting docs saying it doesn't work with subclassing23:18
harlowjacan't be traced twice, and/or if u try the second @trace_cls will blow-away the first23:19
harlowjaand/or do other crazy shit23:19
harlowjaanyway, opened https://bugs.launchpad.net/osprofiler/+bug/153794423:25
openstackLaunchpad bug 1537944 in osprofiler "Osprofiler trace_cls has various issues" [Undecided,New]23:25
*** ihrachys has quit IRC23:29
*** EinstCrazy has joined #openstack-oslo23:31
*** EinstCrazy has quit IRC23:36
*** gordc has quit IRC23:36
*** david-lyle has quit IRC23:44
*** jecarey has quit IRC23:45
*** edmondsw has quit IRC23:47
*** EinstCrazy has joined #openstack-oslo23:57
*** bknudson has left #openstack-oslo23:58
*** yamamoto_ has joined #openstack-oslo23:58
openstackgerritJoshua Harlow proposed openstack/taskflow: Avoid process based executor task proxying-back ANY event  https://review.openstack.org/27232623:58

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