Friday, 2014-10-24

*** oomichi__ has quit IRC00:02
*** dims_ has joined #openstack-oslo00:03
*** dims_ has quit IRC00:06
*** dims_ has joined #openstack-oslo00:06
*** dims has quit IRC00:07
amaurymedeirosis there a way to Sync code from oslo.utils similar to https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator?00:16
amaurymedeirosi'm trying to use update.sh to update strutils to keystone, but i'm getting module not found, because strutils belongs to oslo.utils00:17
*** HenryG has quit IRC00:18
*** tsekiyama has quit IRC00:34
*** mtanino has quit IRC00:35
*** arnaud has quit IRC00:52
*** arnaud__ has quit IRC00:52
*** arnaud has joined #openstack-oslo00:53
*** arnaud__ has joined #openstack-oslo00:53
*** oomichi has quit IRC01:02
*** mriedem_away has left #openstack-oslo01:14
*** arnaud__ has quit IRC01:24
*** arnaud has quit IRC01:24
*** takedakn has quit IRC01:38
*** zz_jgrimm is now known as jgrimm01:38
*** jecarey has joined #openstack-oslo01:48
jgrimmamaurymedeiros, you don't sync (IOW 'copy') code from oslo.utils.... its in the oslo.utils library.. looks like keystone already has oslo.utils so just need to import01:54
amrithamaurymedeiros, strutils used to be in oslo-incubator (where you had to copy code). Now that it graduated, you add oslo.utils to requirements.txt. keystone already has this. You shouldn't have to copy files any longer.01:58
jgrimmamaurymedeiros, and i don't see any actual current users of strutils in keystone.. sure you need it at all? maybe its just a delete cleanup needed02:00
*** arnaud has joined #openstack-oslo02:05
*** arnaud has quit IRC02:07
amaurymedeirosjgrimm, amrith, thanks for the clarifications... i was asking because of a bug reported. but i guess this commit is trying to fix it https://review.openstack.org/#/c/130672/202:15
amrithso, this commit isn't just syncing the modules, it is getting rid of the ones that aren't used any more and syncing the ones that are still in use.02:17
jgrimm+102:18
*** harlowja is now known as harlowja_away02:23
*** dims_ has quit IRC02:56
*** dims has joined #openstack-oslo02:57
*** dims has quit IRC03:01
*** noelbk has quit IRC03:12
*** sigmavirus24 is now known as sigmavirus24_awa03:20
*** jgrimm is now known as zz_jgrimm04:01
*** dougwig has quit IRC04:03
*** yamahata has joined #openstack-oslo04:10
*** HenryG has joined #openstack-oslo04:10
*** arnaud has joined #openstack-oslo04:15
*** arnaud has quit IRC04:21
*** oomichi has joined #openstack-oslo04:24
*** e0ne has joined #openstack-oslo04:34
*** e0ne has quit IRC04:35
*** e0ne has joined #openstack-oslo04:40
*** e0ne has quit IRC04:41
*** harlowja_at_home has joined #openstack-oslo04:54
*** harlowja_at_home has quit IRC04:56
openstackgerritAndreas Jaeger proposed a change to openstack/oslo.concurrency: Improve lock_path help and documentation  https://review.openstack.org/13035404:58
*** yamahata has quit IRC04:59
*** yamahata has joined #openstack-oslo04:59
*** exploreshaifali has joined #openstack-oslo05:17
*** exploreshaifali has quit IRC06:02
*** r1chardj0n3s is now known as r1chardj0n3s_afk06:06
*** i159 has joined #openstack-oslo06:37
*** harlowja_away has quit IRC06:47
*** andreykurilin has quit IRC06:47
*** zz_jgrimm has quit IRC06:47
*** anteaya has quit IRC06:47
*** ruhe has quit IRC06:47
*** dhellmann has quit IRC06:47
*** yamahata has quit IRC06:47
*** oomichi has quit IRC06:47
*** morganfainberg has quit IRC06:47
*** d0ugal has quit IRC06:47
*** i159 has quit IRC06:57
*** yamahata has joined #openstack-oslo07:00
*** oomichi has joined #openstack-oslo07:00
*** morganfainberg has joined #openstack-oslo07:00
*** anteaya has joined #openstack-oslo07:00
*** d0ugal has joined #openstack-oslo07:00
*** ruhe has joined #openstack-oslo07:00
*** dhellmann has joined #openstack-oslo07:00
*** yamahata has quit IRC07:01
*** harlowja_away has joined #openstack-oslo07:10
*** andreykurilin has joined #openstack-oslo07:10
*** zz_jgrimm has joined #openstack-oslo07:10
*** yamahata has joined #openstack-oslo07:16
*** i159 has joined #openstack-oslo07:16
*** andreykurilin_ has joined #openstack-oslo07:22
*** harlowja_away has quit IRC07:28
*** andreykurilin has quit IRC07:28
*** zz_jgrimm has quit IRC07:29
*** i159 has quit IRC07:29
*** yamahata has quit IRC07:29
*** anteaya has quit IRC07:29
*** ruhe has quit IRC07:29
*** dhellmann has quit IRC07:29
*** oomichi has quit IRC07:29
*** morganfainberg has quit IRC07:29
*** d0ugal has quit IRC07:29
*** d0ugal has joined #openstack-oslo07:32
*** morganfainberg has joined #openstack-oslo07:32
*** oomichi has joined #openstack-oslo07:32
*** harlowja_away has joined #openstack-oslo07:32
*** andreykurilin has joined #openstack-oslo07:32
*** zz_jgrimm has joined #openstack-oslo07:32
*** lbragstad has quit IRC07:36
*** lbragstad has joined #openstack-oslo07:36
*** oomichi has quit IRC07:36
*** morganfainberg has quit IRC07:36
*** d0ugal has quit IRC07:36
*** i159 has joined #openstack-oslo07:37
*** yamahata has joined #openstack-oslo07:37
*** anteaya has joined #openstack-oslo07:37
*** ruhe has joined #openstack-oslo07:37
*** dhellmann has joined #openstack-oslo07:37
*** d0ugal has joined #openstack-oslo07:39
*** d0ugal is now known as Guest35107:39
*** Guest351 has quit IRC07:39
*** oomichi has joined #openstack-oslo07:40
*** morganfainberg has joined #openstack-oslo07:40
*** dmatthews__ has joined #openstack-oslo07:40
*** dmatthews__ is now known as d0ugal07:46
*** d0ugal has quit IRC07:46
*** d0ugal has joined #openstack-oslo07:46
*** i159 has quit IRC07:57
*** andreykurilin_ has quit IRC08:02
*** SridharG has joined #openstack-oslo08:04
*** i159 has joined #openstack-oslo08:08
*** andreykurilin_ has joined #openstack-oslo08:09
*** takedakn has joined #openstack-oslo08:16
*** i159 has quit IRC08:18
*** andreykurilin_ has quit IRC08:19
*** i159 has joined #openstack-oslo08:25
*** viktors|afk is now known as viktors08:26
*** oomichi has quit IRC08:26
*** ihrachyshka has joined #openstack-oslo08:30
*** andreykurilin_ has joined #openstack-oslo08:38
*** andreykurilin_ has quit IRC08:44
*** e0ne has joined #openstack-oslo08:55
*** alexpilotti has joined #openstack-oslo09:04
*** exploreshaifali has joined #openstack-oslo09:05
openstackgerritA change was merged to openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/13032709:05
*** takedakn has quit IRC09:05
*** takedakn has joined #openstack-oslo09:13
*** ihrachyshka has quit IRC09:15
*** liusheng has quit IRC09:19
*** i159 has quit IRC09:28
*** pblaho has joined #openstack-oslo09:29
*** takedakn has quit IRC09:48
*** takedakn has joined #openstack-oslo09:48
*** takedakn has quit IRC09:49
openstackgerritRadoslav Gerganov proposed a change to openstack/oslo.vmware: Use nova.image.API in the image_transfer module  https://review.openstack.org/13072609:51
*** pblaho has quit IRC09:51
*** sreshetnyak has joined #openstack-oslo09:54
*** ihrachyshka has joined #openstack-oslo09:54
*** morganfainberg has quit IRC09:55
*** morganfainberg has joined #openstack-oslo09:56
*** morganfainberg has left #openstack-oslo09:56
*** morganfainberg has joined #openstack-oslo09:56
*** jecarey has quit IRC09:58
*** tcammann has quit IRC09:58
*** bnemec has quit IRC09:58
*** ekarlso has quit IRC09:58
*** rodrigods has quit IRC09:58
*** palendae has quit IRC09:58
*** jecarey has joined #openstack-oslo09:59
*** tcammann has joined #openstack-oslo09:59
*** bnemec has joined #openstack-oslo09:59
*** rodrigods has joined #openstack-oslo09:59
*** ekarlso has joined #openstack-oslo09:59
*** palendae has joined #openstack-oslo09:59
openstackgerritVictor Sergeyev proposed a change to openstack/oslo.db: Determine SQLA String and Text types as different  https://review.openstack.org/13072910:04
openstackgerritRadoslav Gerganov proposed a change to openstack/oslo.vmware: Use nova.image.API in the image_transfer module  https://review.openstack.org/13072610:05
*** yamahata has quit IRC10:10
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Common alembic environment runner.  https://review.openstack.org/11285910:33
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Implementation of Alembic as migration engine  https://review.openstack.org/9996510:34
*** e0ne has quit IRC10:47
*** dims has joined #openstack-oslo10:47
*** e0ne has joined #openstack-oslo10:48
*** exploreshaifali has quit IRC11:00
*** pblaho has joined #openstack-oslo11:16
*** amrith is now known as _amrith_11:49
*** SridharG has quit IRC12:30
*** SridharG has joined #openstack-oslo12:32
*** i159 has joined #openstack-oslo12:33
*** jecarey has quit IRC12:48
*** dims has quit IRC12:51
*** dims has joined #openstack-oslo12:52
ttxdhellmann: small suggestion: if we are to keep a unique oslo-specs repo for all of Oslo, we should mandate that commit messages start with the name of the affected library (when affecting a lib)12:53
ttxAt this point I'm using a combination of reading the title, author and topic to filter for libraries-I-know :)12:54
ttxdifficult to tell "Automatically expand eventlet executor greenthread pool" is oslo.messaging for example12:54
*** exploreshaifali has joined #openstack-oslo12:55
*** exploreshaifali has quit IRC13:02
*** jeckersb_gone is now known as jeckersb13:04
*** kgiusti has joined #openstack-oslo13:04
dhellmannttx: yeah, I like that idea13:15
dhellmannttx: I had some trouble doing the oauth dance with the release scripts with a console-based browser. Have you made that work, or do you use a regular browser?13:17
*** bknudson has quit IRC13:18
ttxI use a regular browser13:20
ttxdhellmann: ^13:21
*** i159 has quit IRC13:21
ttxgot to run, bbl13:21
dhellmannttx: ok, thanks. I was trying to set up my cloud box to make releases, but I can just do it on my laptop13:22
*** yamahata has joined #openstack-oslo13:22
*** _amrith_ is now known as amrith13:27
*** gordc has joined #openstack-oslo13:30
openstackgerritDoug Hellmann proposed a change to openstack/cliff: Add pbr to installation requirements  https://review.openstack.org/13077913:30
openstackgerritDoug Hellmann proposed a change to openstack/oslo.concurrency: Add pbr to installation requirements  https://review.openstack.org/13078113:31
openstackgerritDoug Hellmann proposed a change to openstack/oslo.config: Add pbr to installation requirements  https://review.openstack.org/13078213:32
openstackgerritDoug Hellmann proposed a change to openstack/oslo.db: Add pbr to installation requirements  https://review.openstack.org/13078313:32
openstackgerritDoug Hellmann proposed a change to openstack/oslo.i18n: Add pbr to installation requirements  https://review.openstack.org/13078413:33
openstackgerritDoug Hellmann proposed a change to openstack/oslo.log: Add pbr to installation requirements  https://review.openstack.org/13078513:33
*** mriedem has joined #openstack-oslo13:34
openstackgerritDoug Hellmann proposed a change to openstack/oslo.messaging: Add pbr to installation requirements  https://review.openstack.org/13078613:34
openstackgerritDoug Hellmann proposed a change to openstack/oslo.middleware: Add pbr to installation requirements  https://review.openstack.org/13078713:34
openstackgerritDoug Hellmann proposed a change to openstack/oslo.serialization: Add pbr to installation requirements  https://review.openstack.org/13078813:35
openstackgerritDoug Hellmann proposed a change to openstack/oslo.utils: Add pbr to installation requirements  https://review.openstack.org/13078913:35
openstackgerritDoug Hellmann proposed a change to openstack/oslo.vmware: Add pbr to installation requirements  https://review.openstack.org/13079013:36
*** sigmavirus24_awa is now known as sigmavirus2413:36
*** pblaho has quit IRC13:36
*** bknudson has joined #openstack-oslo13:36
openstackgerritDoug Hellmann proposed a change to openstack/taskflow: Add pbr to installation requirements  https://review.openstack.org/13079113:37
openstackgerritDoug Hellmann proposed a change to openstack/oslotest: Add pbr to installation requirements  https://review.openstack.org/13079313:37
openstackgerritDoug Hellmann proposed a change to openstack/oslosphinx: Add pbr to installation requirements  https://review.openstack.org/13079413:38
*** mtanino has joined #openstack-oslo13:38
*** bknudson has quit IRC13:42
*** bknudson has joined #openstack-oslo13:49
*** SridharG has quit IRC13:50
*** jecarey has joined #openstack-oslo13:55
bogdandoplease take a look https://bugs.launchpad.net/oslo.log/+bug/138529514:00
bogdandolooks like use_syslog is broken in Juno14:00
*** ajo has quit IRC14:04
*** ajo has joined #openstack-oslo14:09
*** pblaho has joined #openstack-oslo14:10
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo-incubator: ServiceRestartTest: make it more resilient  https://review.openstack.org/12927814:14
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo-incubator: threadgroup: don't log GreenletExit  https://review.openstack.org/13079914:14
ihrachyshkadhellmann: hey. spent a bit more time on that GreenletExit failure in test_service, and realized that GreenletExit is correct behaviour, it's just a bug in service module that we log it sometimes. Sent a patch for that: ^^ As for the failure itself, it's a problem in the unit test only, also fixed as a separate patch: ^^14:14
dhellmannihrachyshka: ok, at this point it will probably be some time after the summit before I get back to looking at bugs and patches14:16
*** SridharG has joined #openstack-oslo14:24
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo-incubator: threadgroup: don't log GreenletExit  https://review.openstack.org/13079914:25
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo-incubator: ServiceRestartTest: make it more resilient  https://review.openstack.org/12927814:25
*** jaypipes is now known as leakypipes14:30
ihrachyshkagordc: hey14:33
ihrachyshkagordc: do you have any plans to proceed with middleware work for neutron? https://review.openstack.org/#/c/127036/ there is a parallel patch for that (with the same issues as revealed in your review), I wonder how we proceed with duplicates14:34
gordcihrachyshka: cool cool. thanks for heads up14:35
ihrachyshkagordc: so what's the plan? :)14:35
gordcihrachyshka: i was going to continue working on it but got sidetracked so if the new committer wants to take over i'm ok.14:36
gordcihrachyshka: they just need to sync in new middleware from oslo and update references accordingly.14:36
ihrachyshkaok, cool, I'll ask her (?) to proceed with your patch14:36
ihrachyshkagordc: yeah, I know the story behind shim14:36
gordcihrachyshka: cool cool. thanks for asking. i'd continue but i couldn't quickly do a sync where i had to track down all the patches i was pulling in.14:37
openstackgerritDoug Hellmann proposed a change to openstack/oslo-specs: Expand instructions in "alternatives" section  https://review.openstack.org/12843914:38
openstackgerritA change was merged to openstack/oslo-specs: Expand instructions in "alternatives" section  https://review.openstack.org/12843914:43
*** i159 has joined #openstack-oslo14:49
*** ihrachyshka has quit IRC14:49
*** i159 has joined #openstack-oslo14:50
*** tsekiyama has joined #openstack-oslo14:51
*** AAzza_afk has joined #openstack-oslo14:52
*** AAzza_afk is now known as AAzza14:52
*** dims is now known as dimsum_15:16
*** AAzza has left #openstack-oslo15:33
*** zz_jgrimm is now known as jgrimm15:38
*** jogo is now known as flashgordon15:45
*** ihrachyshka has joined #openstack-oslo15:50
*** miqui has quit IRC15:59
*** palendae has quit IRC15:59
*** miqui has joined #openstack-oslo15:59
*** i159 has quit IRC16:00
*** noelbk has joined #openstack-oslo16:00
*** sreshetnyak has quit IRC16:00
*** sreshetnyak has joined #openstack-oslo16:00
*** palendae has joined #openstack-oslo16:01
*** harlowja_at_home has joined #openstack-oslo16:06
bknudsonis this the meeting time or was it rescheduled?16:06
*** bnemec is now known as beekneemech16:06
beekneemechNot sure. dhellmann^16:07
* dhellmann wandered off without his phone and lost track of time16:07
beekneemechbknudson: ^16:08
bknudsonyou need the watch16:08
*** exploreshaifali has joined #openstack-oslo16:17
openstackgerritKen Giusti proposed a change to openstack/oslo.messaging: Provide better detection of failures during message send  https://review.openstack.org/12606516:18
openstackgerritKen Giusti proposed a change to openstack/oslo.messaging: Implement message flow control using link credit  https://review.openstack.org/12755016:18
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/13008816:19
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo.config: Updated from global requirements  https://review.openstack.org/13032316:19
*** exploreshaifali has quit IRC16:26
dimsum_dhellmann: beekneemech: LOL16:27
*** freyes has joined #openstack-oslo16:34
openstackgerritA change was merged to openstack/stevedore: Add pbr to dependency list  https://review.openstack.org/13064116:34
openstackgerritJoshua Harlow proposed a change to openstack/oslo.db: Fix more invalid encoding and scoping issues  https://review.openstack.org/12019816:35
openstackgerritIhar Hrachyshka proposed a change to openstack/oslo.messaging: Raise the number of RPC threads to run in parallel  https://review.openstack.org/13027816:38
*** harlowja_at_home has quit IRC16:44
beekneemechdhellmann: BTW, do we want to release oslo.concurrency early next week?16:46
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: db: Add long-lived-transactionalized-db-fixtures  https://review.openstack.org/11733516:46
beekneemechAnd so it begins...16:46
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: config: Add a spec for a oslo.config dictionary based config opts  https://review.openstack.org/13004716:47
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: config: Add a spec for a oslo.config dictionary based config opts  https://review.openstack.org/13004716:48
dhellmannbeekneemech: if you're ready to release oslo.concurrency, let's do it16:48
dhellmannbeekneemech: do you want me to do it?16:48
beekneemechdhellmann: Yeah, I think we're ready.16:48
beekneemechdhellmann: I'm pretty sure I don't have release permissions.16:48
dhellmannbeekneemech: I can do it today, since nothing uses it16:48
beekneemechAh, true.16:49
dhellmannbeekneemech: we should fix the permissions thing, but I should write up how to use ttx's release script first16:49
beekneemechdhellmann: Yeah, I've just never released anything so I haven't been through the process.16:49
*** viktors has quit IRC16:49
beekneemechAnd I hear the release people get perturbed if you screw one up. :-)16:50
beekneemechdhellmann: So bp links in the commit message don't work, so how do we link specs to bps?  Does the topic have to be set manually?16:51
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: middleware: oslo.middleware healthcheck  https://review.openstack.org/13045316:51
dhellmannbeekneemech: yeah, I guess we have to set the topics by hand16:52
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: cache: Convert oslo.cache to use dogpile.cache  https://review.openstack.org/12477616:53
dhellmannbeekneemech: the problem with the linking is our gerrit doesn't search the "oslo" project group, so we need to figure out how to fix that in infra. Filing a bug would be a good first step, maybe one of the infra team can help us. ttx and I talked about allowing a more verbose syntax like bp/group/foo-bar and if that's used assuming the "group" is where the search should happen16:53
beekneemechdhellmann: Yeah, I think we need a real fix because setting the topic once isn't going to do any good.  As soon as someone pushes another patch set it will get broken again.16:54
*** exploreshaifali has joined #openstack-oslo16:54
dhellmannyeah16:54
beekneemechUnless they happen to remember to change their local topic branch name.16:54
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: db: Enable MySQL Connector driver  https://review.openstack.org/12504416:55
*** ihrachyshka has quit IRC16:55
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: i18n: Add spec for more-gettext-support  https://review.openstack.org/12145916:55
beekneemechBleh, this is dropping all of the votes on the changes I alter. :-(16:56
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: db: Make EngineFacade a Facade  https://review.openstack.org/12518116:56
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: messaging: Adding spec for trusted messaging plugin.  https://review.openstack.org/12892616:59
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: db: Alembic environment runner specification  https://review.openstack.org/11284216:59
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: middleware: Add CORS middleware spec  https://review.openstack.org/11948517:00
openstackgerritBen Nemec proposed a change to openstack/oslo-specs: log: Restore remove-context-adapter spec  https://review.openstack.org/12484717:01
beekneemechdhellmann: Okay, that should take care of all the lib-specific specs.  I left the adoption, incubator, and cross-project specs alone.17:02
beekneemechdhellmann: Oh, and you were going to rebase https://review.openstack.org/#/c/113940/ right?17:03
dhellmannbeekneemech: yeah, unless you have a few minutes to do it? we need to add a list_opts() function for the log code, since it will still be present after the rebase17:05
beekneemechdhellmann: Yeah, I should have time to do it.  I'm just waiting for images to build anyway. :-)17:06
dhellmannbeekneemech: cool, thank you17:06
dhellmannbeekneemech: oslo.concurrency 0.1.0 released17:12
beekneemechdhellmann: \o/17:13
dhellmannbeekneemech: I think we need to do oslo.context next, so we can update the logging code to use it17:14
dhellmannwe still need a spec for that17:14
beekneemechdhellmann: Sounds good17:15
dhellmannbut first I need food :-)17:15
beekneemechI'm headed that way as soon as I finish this rebase. :-)17:16
openstackgerritBen Nemec proposed a change to openstack/oslo-incubator: add list_opts to all modules with configuration options  https://review.openstack.org/11394017:19
*** takedakn has joined #openstack-oslo17:25
*** takedakn has quit IRC17:31
*** takedakn has joined #openstack-oslo17:32
openstackgerritBen Nemec proposed a change to openstack/oslo-incubator: add list_opts to all modules with configuration options  https://review.openstack.org/11394017:35
*** alexpilotti has quit IRC17:49
*** alexpilotti has joined #openstack-oslo17:50
dimsum_added a global req review - https://review.openstack.org/#/c/130862/  oslo.concurrency17:51
*** alexpilotti has quit IRC17:51
dimsum_Anyone cutting a review for Nova?17:51
*** andreykurilin_ has joined #openstack-oslo17:51
*** alexpilotti has joined #openstack-oslo17:52
*** alexpilotti has quit IRC17:53
*** harlowja_away is now known as harlowja17:55
*** alexpilotti has joined #openstack-oslo17:55
*** harlowja is now known as harlowja_away17:56
*** alexpilotti has quit IRC17:56
*** harlowja_away is now known as harlowja17:58
*** alexpilotti has joined #openstack-oslo17:58
*** alexpilotti has quit IRC17:59
harlowjadhellmann so the current thinking is to have pbr back in requirements.txt17:59
harlowja?17:59
harlowjafinal answer this time :-P17:59
*** alexpilotti has joined #openstack-oslo18:00
*** alexpilotti has quit IRC18:01
*** alexpilotti has joined #openstack-oslo18:02
*** alexpilotti has quit IRC18:03
*** exploreshaifali has quit IRC18:13
*** AAzza_afk has joined #openstack-oslo18:14
*** AAzza_afk is now known as AAzza18:14
openstackgerritAndreas Jaeger proposed a change to openstack/oslo.concurrency: Improve lock_path help and documentation  https://review.openstack.org/13035418:28
*** AAzza has left #openstack-oslo18:30
*** sreshetnyak has quit IRC18:32
*** yamahata has quit IRC18:32
*** amrith is now known as _amrith_18:35
*** _amrith_ is now known as amrith18:36
*** e0ne has quit IRC18:44
openstackgerritAndreas Jaeger proposed a change to openstack/oslo.concurrency: Improve lock_path help and documentation  https://review.openstack.org/13035418:44
*** exploreshaifali has joined #openstack-oslo18:46
*** andreykurilin_ has quit IRC18:46
openstackgerritJoshua Harlow proposed a change to openstack/oslo.concurrency: Allow the lock delay to be provided  https://review.openstack.org/13087218:54
openstackgerritJoshua Harlow proposed a change to openstack/oslo.concurrency: Allow the lock delay to be provided  https://review.openstack.org/13087218:55
*** ajo_ has joined #openstack-oslo18:58
dhellmannharlowja: yeah, apparently pip and setuptools do weirdness if you don't and it is installed as a drive-by dependency19:01
harlowjahmmm, sad19:01
dhellmannharlowja: yeah19:02
*** exploreshaifali has quit IRC19:06
*** jecarey has quit IRC19:10
openstackgerritJoshua Harlow proposed a change to openstack/pylockfile: Begin moving some of the common code to a shared base  https://review.openstack.org/12225319:17
*** pblaho has quit IRC19:21
*** ajo_ has quit IRC19:30
*** ajo_ has joined #openstack-oslo19:35
*** andreykurilin_ has joined #openstack-oslo20:01
*** ajo_ has quit IRC20:06
*** e0ne has joined #openstack-oslo20:09
*** amrith is now known as _amrith_20:19
*** e0ne has quit IRC20:22
*** r1chardj0n3s_afk is now known as r1chardj0n3s20:24
*** e0ne has joined #openstack-oslo20:26
*** e0ne has quit IRC20:28
*** kgiusti has left #openstack-oslo20:39
*** ajo_ has joined #openstack-oslo20:56
*** jgrimm is now known as zz_jgrimm20:57
openstackgerritA change was merged to openstack/oslo.log: Updated from global requirements  https://review.openstack.org/13032620:59
dimsum_dhellmann: patching oslo.concurrency w/ nova - runs into this problem21:01
dimsum_DuplicateOptError: duplicate option: default_log_levels21:01
dimsum_http://logs.openstack.org/70/130870/3/check/gate-nova-pep8/8ad1abf/console.html21:01
beekneemechdimsum_: That opt must have changed between the version in nova and the one in oslo.concurrency.21:04
beekneemechWe'll need to figure out which is out of date and update it.21:05
dimsum_nova/nova/openstack/common/log.py21:05
dimsum_oslo.concurrency/oslo/concurrency/openstack/common/log.py21:05
dimsum_both have it21:06
*** ajo_ has quit IRC21:08
*** ajo_ has joined #openstack-oslo21:08
beekneemechdimsum_: Ah, DEFAULT_LOG_LEVELS is different between the two.21:09
beekneemechWow, that could be nasty.21:10
beekneemechWe need oslo.log real bad. :-/21:10
dimsum_beekneemech: only other alternative is we remove openstack.common.log from fileutils, we can then remove oslo.concurrency copy of log.py and then do a new release21:11
dimsum_sigh21:12
beekneemechHmm, yeah, maybe oslo.concurrency just shouldn't use oslo logging for now.21:13
beekneemechIn fact, now that I say that I vaguely recall having that discussion at some point.21:13
beekneemechEventually it will get all the pretty context information from the context formatter so we won't have to use oslo logging to get the benefits anyway.21:14
beekneemechdhellmann: ^Thoughts?21:14
*** ajo_ has quit IRC21:15
dimsum_technically it's fileutils that using it not lockutils or processutils21:15
dimsum_beekneemech: more local change is to nuke fileutils from oslo.concurrr by having local copy of ensure_tree21:18
dimsum_and then make a new release21:18
dhellmannbeekneemech: yeah, change oslo.concurrency to just use python's logging module. That's part of the graduation steps for all our libs, since oslo.log is just going to be for configuring logging21:21
* dhellmann dropped the ball on oslo.log again this week for summit prep :-/21:21
dimsum_beekneemech: dhellmann: won't work as the oslo-inc logging is needed by fileutils module not the code in oslo-concurr itselg21:22
dhellmanndimsum_: grr21:23
dhellmanndimsum_: so that's where your suggestion about ensure_tree came from?21:23
dimsum_y21:23
dhellmanndoes making the value of DEFAULT_LOG_LEVELS the same fix it?21:24
dimsum_"DuplicateOptError: duplicate option: default_log_levels"21:24
dhellmannbecause I'd rather actually work on oslo.context and then oslo.log than hack around with oslo.concurrency21:24
dhellmannif all of the option settings are the same it should be OK to register the same option twice21:24
beekneemechIt should fix it, but we're almost certainly going to run into this again.21:24
dhellmanncertainly21:24
beekneemechAnd potentially going to break the world every time we release oslo.concurrency, if there was a log module update.21:25
dhellmannright, let's work our way from the bottom up instead21:26
dimsum_opened a bug https://bugs.launchpad.net/oslo.concurrency/+bug/138549221:26
dimsum_dhellmann: which is?21:26
beekneemechWait, is fileutils seriously pulling in log.py solely for one debug message?21:27
beekneemechYes, yes it is.21:27
dhellmannbeekneemech: if that's the case we can just fix that and sync a new copy into oslo.concurrency21:27
beekneemechWe should just nuke that.21:28
beekneemechdhellmann: Yeah21:28
dhellmanndimsum_: oslo.context, then oslo.log, then a new oslo.concurrency21:28
dimsum_cool +121:28
dhellmanndimsum_, beekneemech : basically, we have to do oslo.context and oslo.log before we start any other libraries21:28
dhellmannI added some notes to the oslo.context section of https://etherpad.openstack.org/p/kilo-oslo-library-proposals21:29
dimsum_dhellmann: +1 to that, in the short term, can we do 0.1.1 with beekneemech's suggestion?21:29
dimsum_nuke LOG.debug in fileutils21:30
dhellmanndimsum_: yep, let's get rid of the oslo logging stuff in fileutils and just use the python log module21:30
dimsum_dhellmann: +121:30
dhellmanndimsum_, beekneemech : if you get the patch for that together we can probably cut a new release of oslo.concurrency monday. I'm out starting wednesday to take a few days off before the summit.21:31
beekneemechdhellmann: Sounds good21:32
beekneemechdimsum_: I'll go ahead and do the incubator change (unless you already have :-)21:32
dimsum_about to hit git review :) so you can both +2 it :)21:33
beekneemechdimsum_: Perfect :-)21:33
*** r1chardj0n3s is now known as r1chardj0n3s_afk21:36
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo-incubator: Switch fileutils to use python logging  https://review.openstack.org/13089221:36
dhellmanndimsum_: +221:38
*** jeckersb is now known as jeckersb_gone21:39
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Merge fileutils from oslo-incubator  https://review.openstack.org/13089321:41
dimsum_that should do the trick. cross my fingers21:41
dimsum_dhellmann: beekneemech has karma for pushing out 0.1.1?21:44
beekneemechNo21:44
dimsum_k, dhellmann is on the hook then :)21:44
beekneemechThe log.py options won't get registered as long as it isn't imported, right?21:45
dimsum_better to yank the module itself?21:45
openstackgerritA change was merged to openstack/oslo-incubator: Switch fileutils to use python logging  https://review.openstack.org/13089221:45
beekneemechdimsum_: I think it's fine, but it wouldn't hurt.21:46
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Merge fileutils from oslo-incubator  https://review.openstack.org/13089321:47
dimsum_there you go21:47
beekneemechI wonder how many of those other modules should be removed now too.21:47
dhellmanndimsum_: should that be linked to the bug you opened?21:47
beekneemechOoh, right.21:47
openstackgerritDavanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Merge fileutils from oslo-incubator  https://review.openstack.org/13089321:48
* dhellmann is going to be a stickler for bug/blueprint linkages this cycle21:48
dimsum_:)21:48
beekneemechdhellmann: Quite right too.  You're the one who gets stuck dealing with unlinked bugs most of the time. :-)21:48
dimsum_and wriing things down too :)21:48
dimsum_updated the commit message21:49
dimsum_back in a bit21:49
*** mriedem has left #openstack-oslo21:49
*** sigmavirus24 is now known as sigmavirus24_awa21:58
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Remove unused incubator modules  https://review.openstack.org/13089421:58
openstackgerritBen Nemec proposed a change to openstack/oslo.concurrency: Clean up lockutils logging  https://review.openstack.org/13064822:02
*** dims_ has joined #openstack-oslo22:03
*** dims__ has joined #openstack-oslo22:05
dhellmannbeekneemech, dimsum_ : I updated https://wiki.openstack.org/wiki/Oslo/Dependencies to highlight some of the ones we know we're working on this cycle22:05
*** dimsum_ has quit IRC22:06
*** dims_ has quit IRC22:08
* beekneemech stretches the window across both monitors and it's _almost_ wide enough :-)22:09
beekneemechdhellmann: lgtm22:09
dims__:)22:11
*** andreykurilin_ has quit IRC22:13
* dims__ checks on 130894 and 13089322:14
*** gordc has quit IRC22:18
*** SridharG has quit IRC22:23
openstackgerritOpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/13008822:32
*** bknudson has quit IRC22:37
openstackgerritA change was merged to openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/13008822:38
*** rjrjr__ has joined #openstack-oslo22:51
rjrjr__how do i get the section names (group names) from a ini file using oslo.config?22:51
rjrjr__is the Base class in the iniparser.py file considered publicly exposed.  (i.e. if I extend that class to get the section names, do i have to worry about that class changing in the future?)22:52
*** harlowja is now known as harlowja_away23:00
*** harlowja_away is now known as harlowja23:24
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Add a history retry object, makes retry histories easier to use  https://review.openstack.org/12361623:53
*** _amrith_ is now known as amrith23:56
openstackgerritA change was merged to openstack/oslo.concurrency: Merge fileutils from oslo-incubator  https://review.openstack.org/13089323:59

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