*** oomichi__ has quit IRC | 00:02 | |
*** dims_ has joined #openstack-oslo | 00:03 | |
*** dims_ has quit IRC | 00:06 | |
*** dims_ has joined #openstack-oslo | 00:06 | |
*** dims has quit IRC | 00:07 | |
amaurymedeiros | is there a way to Sync code from oslo.utils similar to https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator? | 00:16 |
---|---|---|
amaurymedeiros | i'm trying to use update.sh to update strutils to keystone, but i'm getting module not found, because strutils belongs to oslo.utils | 00:17 |
*** HenryG has quit IRC | 00:18 | |
*** tsekiyama has quit IRC | 00:34 | |
*** mtanino has quit IRC | 00:35 | |
*** arnaud has quit IRC | 00:52 | |
*** arnaud__ has quit IRC | 00:52 | |
*** arnaud has joined #openstack-oslo | 00:53 | |
*** arnaud__ has joined #openstack-oslo | 00:53 | |
*** oomichi has quit IRC | 01:02 | |
*** mriedem_away has left #openstack-oslo | 01:14 | |
*** arnaud__ has quit IRC | 01:24 | |
*** arnaud has quit IRC | 01:24 | |
*** takedakn has quit IRC | 01:38 | |
*** zz_jgrimm is now known as jgrimm | 01:38 | |
*** jecarey has joined #openstack-oslo | 01:48 | |
jgrimm | amaurymedeiros, 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 import | 01:54 |
amrith | amaurymedeiros, 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 |
jgrimm | amaurymedeiros, 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 needed | 02:00 |
*** arnaud has joined #openstack-oslo | 02:05 | |
*** arnaud has quit IRC | 02:07 | |
amaurymedeiros | jgrimm, 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/2 | 02:15 |
amrith | so, 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 | +1 | 02:18 |
*** harlowja is now known as harlowja_away | 02:23 | |
*** dims_ has quit IRC | 02:56 | |
*** dims has joined #openstack-oslo | 02:57 | |
*** dims has quit IRC | 03:01 | |
*** noelbk has quit IRC | 03:12 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 03:20 | |
*** jgrimm is now known as zz_jgrimm | 04:01 | |
*** dougwig has quit IRC | 04:03 | |
*** yamahata has joined #openstack-oslo | 04:10 | |
*** HenryG has joined #openstack-oslo | 04:10 | |
*** arnaud has joined #openstack-oslo | 04:15 | |
*** arnaud has quit IRC | 04:21 | |
*** oomichi has joined #openstack-oslo | 04:24 | |
*** e0ne has joined #openstack-oslo | 04:34 | |
*** e0ne has quit IRC | 04:35 | |
*** e0ne has joined #openstack-oslo | 04:40 | |
*** e0ne has quit IRC | 04:41 | |
*** harlowja_at_home has joined #openstack-oslo | 04:54 | |
*** harlowja_at_home has quit IRC | 04:56 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/oslo.concurrency: Improve lock_path help and documentation https://review.openstack.org/130354 | 04:58 |
*** yamahata has quit IRC | 04:59 | |
*** yamahata has joined #openstack-oslo | 04:59 | |
*** exploreshaifali has joined #openstack-oslo | 05:17 | |
*** exploreshaifali has quit IRC | 06:02 | |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 06:06 | |
*** i159 has joined #openstack-oslo | 06:37 | |
*** harlowja_away has quit IRC | 06:47 | |
*** andreykurilin has quit IRC | 06:47 | |
*** zz_jgrimm has quit IRC | 06:47 | |
*** anteaya has quit IRC | 06:47 | |
*** ruhe has quit IRC | 06:47 | |
*** dhellmann has quit IRC | 06:47 | |
*** yamahata has quit IRC | 06:47 | |
*** oomichi has quit IRC | 06:47 | |
*** morganfainberg has quit IRC | 06:47 | |
*** d0ugal has quit IRC | 06:47 | |
*** i159 has quit IRC | 06:57 | |
*** yamahata has joined #openstack-oslo | 07:00 | |
*** oomichi has joined #openstack-oslo | 07:00 | |
*** morganfainberg has joined #openstack-oslo | 07:00 | |
*** anteaya has joined #openstack-oslo | 07:00 | |
*** d0ugal has joined #openstack-oslo | 07:00 | |
*** ruhe has joined #openstack-oslo | 07:00 | |
*** dhellmann has joined #openstack-oslo | 07:00 | |
*** yamahata has quit IRC | 07:01 | |
*** harlowja_away has joined #openstack-oslo | 07:10 | |
*** andreykurilin has joined #openstack-oslo | 07:10 | |
*** zz_jgrimm has joined #openstack-oslo | 07:10 | |
*** yamahata has joined #openstack-oslo | 07:16 | |
*** i159 has joined #openstack-oslo | 07:16 | |
*** andreykurilin_ has joined #openstack-oslo | 07:22 | |
*** harlowja_away has quit IRC | 07:28 | |
*** andreykurilin has quit IRC | 07:28 | |
*** zz_jgrimm has quit IRC | 07:29 | |
*** i159 has quit IRC | 07:29 | |
*** yamahata has quit IRC | 07:29 | |
*** anteaya has quit IRC | 07:29 | |
*** ruhe has quit IRC | 07:29 | |
*** dhellmann has quit IRC | 07:29 | |
*** oomichi has quit IRC | 07:29 | |
*** morganfainberg has quit IRC | 07:29 | |
*** d0ugal has quit IRC | 07:29 | |
*** d0ugal has joined #openstack-oslo | 07:32 | |
*** morganfainberg has joined #openstack-oslo | 07:32 | |
*** oomichi has joined #openstack-oslo | 07:32 | |
*** harlowja_away has joined #openstack-oslo | 07:32 | |
*** andreykurilin has joined #openstack-oslo | 07:32 | |
*** zz_jgrimm has joined #openstack-oslo | 07:32 | |
*** lbragstad has quit IRC | 07:36 | |
*** lbragstad has joined #openstack-oslo | 07:36 | |
*** oomichi has quit IRC | 07:36 | |
*** morganfainberg has quit IRC | 07:36 | |
*** d0ugal has quit IRC | 07:36 | |
*** i159 has joined #openstack-oslo | 07:37 | |
*** yamahata has joined #openstack-oslo | 07:37 | |
*** anteaya has joined #openstack-oslo | 07:37 | |
*** ruhe has joined #openstack-oslo | 07:37 | |
*** dhellmann has joined #openstack-oslo | 07:37 | |
*** d0ugal has joined #openstack-oslo | 07:39 | |
*** d0ugal is now known as Guest351 | 07:39 | |
*** Guest351 has quit IRC | 07:39 | |
*** oomichi has joined #openstack-oslo | 07:40 | |
*** morganfainberg has joined #openstack-oslo | 07:40 | |
*** dmatthews__ has joined #openstack-oslo | 07:40 | |
*** dmatthews__ is now known as d0ugal | 07:46 | |
*** d0ugal has quit IRC | 07:46 | |
*** d0ugal has joined #openstack-oslo | 07:46 | |
*** i159 has quit IRC | 07:57 | |
*** andreykurilin_ has quit IRC | 08:02 | |
*** SridharG has joined #openstack-oslo | 08:04 | |
*** i159 has joined #openstack-oslo | 08:08 | |
*** andreykurilin_ has joined #openstack-oslo | 08:09 | |
*** takedakn has joined #openstack-oslo | 08:16 | |
*** i159 has quit IRC | 08:18 | |
*** andreykurilin_ has quit IRC | 08:19 | |
*** i159 has joined #openstack-oslo | 08:25 | |
*** viktors|afk is now known as viktors | 08:26 | |
*** oomichi has quit IRC | 08:26 | |
*** ihrachyshka has joined #openstack-oslo | 08:30 | |
*** andreykurilin_ has joined #openstack-oslo | 08:38 | |
*** andreykurilin_ has quit IRC | 08:44 | |
*** e0ne has joined #openstack-oslo | 08:55 | |
*** alexpilotti has joined #openstack-oslo | 09:04 | |
*** exploreshaifali has joined #openstack-oslo | 09:05 | |
openstackgerrit | A change was merged to openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/130327 | 09:05 |
*** takedakn has quit IRC | 09:05 | |
*** takedakn has joined #openstack-oslo | 09:13 | |
*** ihrachyshka has quit IRC | 09:15 | |
*** liusheng has quit IRC | 09:19 | |
*** i159 has quit IRC | 09:28 | |
*** pblaho has joined #openstack-oslo | 09:29 | |
*** takedakn has quit IRC | 09:48 | |
*** takedakn has joined #openstack-oslo | 09:48 | |
*** takedakn has quit IRC | 09:49 | |
openstackgerrit | Radoslav Gerganov proposed a change to openstack/oslo.vmware: Use nova.image.API in the image_transfer module https://review.openstack.org/130726 | 09:51 |
*** pblaho has quit IRC | 09:51 | |
*** sreshetnyak has joined #openstack-oslo | 09:54 | |
*** ihrachyshka has joined #openstack-oslo | 09:54 | |
*** morganfainberg has quit IRC | 09:55 | |
*** morganfainberg has joined #openstack-oslo | 09:56 | |
*** morganfainberg has left #openstack-oslo | 09:56 | |
*** morganfainberg has joined #openstack-oslo | 09:56 | |
*** jecarey has quit IRC | 09:58 | |
*** tcammann has quit IRC | 09:58 | |
*** bnemec has quit IRC | 09:58 | |
*** ekarlso has quit IRC | 09:58 | |
*** rodrigods has quit IRC | 09:58 | |
*** palendae has quit IRC | 09:58 | |
*** jecarey has joined #openstack-oslo | 09:59 | |
*** tcammann has joined #openstack-oslo | 09:59 | |
*** bnemec has joined #openstack-oslo | 09:59 | |
*** rodrigods has joined #openstack-oslo | 09:59 | |
*** ekarlso has joined #openstack-oslo | 09:59 | |
*** palendae has joined #openstack-oslo | 09:59 | |
openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo.db: Determine SQLA String and Text types as different https://review.openstack.org/130729 | 10:04 |
openstackgerrit | Radoslav Gerganov proposed a change to openstack/oslo.vmware: Use nova.image.API in the image_transfer module https://review.openstack.org/130726 | 10:05 |
*** yamahata has quit IRC | 10:10 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Common alembic environment runner. https://review.openstack.org/112859 | 10:33 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Implementation of Alembic as migration engine https://review.openstack.org/99965 | 10:34 |
*** e0ne has quit IRC | 10:47 | |
*** dims has joined #openstack-oslo | 10:47 | |
*** e0ne has joined #openstack-oslo | 10:48 | |
*** exploreshaifali has quit IRC | 11:00 | |
*** pblaho has joined #openstack-oslo | 11:16 | |
*** amrith is now known as _amrith_ | 11:49 | |
*** SridharG has quit IRC | 12:30 | |
*** SridharG has joined #openstack-oslo | 12:32 | |
*** i159 has joined #openstack-oslo | 12:33 | |
*** jecarey has quit IRC | 12:48 | |
*** dims has quit IRC | 12:51 | |
*** dims has joined #openstack-oslo | 12:52 | |
ttx | dhellmann: 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 |
ttx | At this point I'm using a combination of reading the title, author and topic to filter for libraries-I-know :) | 12:54 |
ttx | difficult to tell "Automatically expand eventlet executor greenthread pool" is oslo.messaging for example | 12:54 |
*** exploreshaifali has joined #openstack-oslo | 12:55 | |
*** exploreshaifali has quit IRC | 13:02 | |
*** jeckersb_gone is now known as jeckersb | 13:04 | |
*** kgiusti has joined #openstack-oslo | 13:04 | |
dhellmann | ttx: yeah, I like that idea | 13:15 |
dhellmann | ttx: 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 IRC | 13:18 | |
ttx | I use a regular browser | 13:20 |
ttx | dhellmann: ^ | 13:21 |
*** i159 has quit IRC | 13:21 | |
ttx | got to run, bbl | 13:21 |
dhellmann | ttx: ok, thanks. I was trying to set up my cloud box to make releases, but I can just do it on my laptop | 13:22 |
*** yamahata has joined #openstack-oslo | 13:22 | |
*** _amrith_ is now known as amrith | 13:27 | |
*** gordc has joined #openstack-oslo | 13:30 | |
openstackgerrit | Doug Hellmann proposed a change to openstack/cliff: Add pbr to installation requirements https://review.openstack.org/130779 | 13:30 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.concurrency: Add pbr to installation requirements https://review.openstack.org/130781 | 13:31 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.config: Add pbr to installation requirements https://review.openstack.org/130782 | 13:32 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.db: Add pbr to installation requirements https://review.openstack.org/130783 | 13:32 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Add pbr to installation requirements https://review.openstack.org/130784 | 13:33 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.log: Add pbr to installation requirements https://review.openstack.org/130785 | 13:33 |
*** mriedem has joined #openstack-oslo | 13:34 | |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.messaging: Add pbr to installation requirements https://review.openstack.org/130786 | 13:34 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.middleware: Add pbr to installation requirements https://review.openstack.org/130787 | 13:34 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.serialization: Add pbr to installation requirements https://review.openstack.org/130788 | 13:35 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.utils: Add pbr to installation requirements https://review.openstack.org/130789 | 13:35 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.vmware: Add pbr to installation requirements https://review.openstack.org/130790 | 13:36 |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:36 | |
*** pblaho has quit IRC | 13:36 | |
*** bknudson has joined #openstack-oslo | 13:36 | |
openstackgerrit | Doug Hellmann proposed a change to openstack/taskflow: Add pbr to installation requirements https://review.openstack.org/130791 | 13:37 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslotest: Add pbr to installation requirements https://review.openstack.org/130793 | 13:37 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslosphinx: Add pbr to installation requirements https://review.openstack.org/130794 | 13:38 |
*** mtanino has joined #openstack-oslo | 13:38 | |
*** bknudson has quit IRC | 13:42 | |
*** bknudson has joined #openstack-oslo | 13:49 | |
*** SridharG has quit IRC | 13:50 | |
*** jecarey has joined #openstack-oslo | 13:55 | |
bogdando | please take a look https://bugs.launchpad.net/oslo.log/+bug/1385295 | 14:00 |
bogdando | looks like use_syslog is broken in Juno | 14:00 |
*** ajo has quit IRC | 14:04 | |
*** ajo has joined #openstack-oslo | 14:09 | |
*** pblaho has joined #openstack-oslo | 14:10 | |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: ServiceRestartTest: make it more resilient https://review.openstack.org/129278 | 14:14 |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: threadgroup: don't log GreenletExit https://review.openstack.org/130799 | 14:14 |
ihrachyshka | dhellmann: 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 |
dhellmann | ihrachyshka: ok, at this point it will probably be some time after the summit before I get back to looking at bugs and patches | 14:16 |
*** SridharG has joined #openstack-oslo | 14:24 | |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: threadgroup: don't log GreenletExit https://review.openstack.org/130799 | 14:25 |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: ServiceRestartTest: make it more resilient https://review.openstack.org/129278 | 14:25 |
*** jaypipes is now known as leakypipes | 14:30 | |
ihrachyshka | gordc: hey | 14:33 |
ihrachyshka | gordc: 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 duplicates | 14:34 |
gordc | ihrachyshka: cool cool. thanks for heads up | 14:35 |
ihrachyshka | gordc: so what's the plan? :) | 14:35 |
gordc | ihrachyshka: 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 |
gordc | ihrachyshka: they just need to sync in new middleware from oslo and update references accordingly. | 14:36 |
ihrachyshka | ok, cool, I'll ask her (?) to proceed with your patch | 14:36 |
ihrachyshka | gordc: yeah, I know the story behind shim | 14:36 |
gordc | ihrachyshka: 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 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo-specs: Expand instructions in "alternatives" section https://review.openstack.org/128439 | 14:38 |
openstackgerrit | A change was merged to openstack/oslo-specs: Expand instructions in "alternatives" section https://review.openstack.org/128439 | 14:43 |
*** i159 has joined #openstack-oslo | 14:49 | |
*** ihrachyshka has quit IRC | 14:49 | |
*** i159 has joined #openstack-oslo | 14:50 | |
*** tsekiyama has joined #openstack-oslo | 14:51 | |
*** AAzza_afk has joined #openstack-oslo | 14:52 | |
*** AAzza_afk is now known as AAzza | 14:52 | |
*** dims is now known as dimsum_ | 15:16 | |
*** AAzza has left #openstack-oslo | 15:33 | |
*** zz_jgrimm is now known as jgrimm | 15:38 | |
*** jogo is now known as flashgordon | 15:45 | |
*** ihrachyshka has joined #openstack-oslo | 15:50 | |
*** miqui has quit IRC | 15:59 | |
*** palendae has quit IRC | 15:59 | |
*** miqui has joined #openstack-oslo | 15:59 | |
*** i159 has quit IRC | 16:00 | |
*** noelbk has joined #openstack-oslo | 16:00 | |
*** sreshetnyak has quit IRC | 16:00 | |
*** sreshetnyak has joined #openstack-oslo | 16:00 | |
*** palendae has joined #openstack-oslo | 16:01 | |
*** harlowja_at_home has joined #openstack-oslo | 16:06 | |
bknudson | is this the meeting time or was it rescheduled? | 16:06 |
*** bnemec is now known as beekneemech | 16:06 | |
beekneemech | Not sure. dhellmann^ | 16:07 |
* dhellmann wandered off without his phone and lost track of time | 16:07 | |
beekneemech | bknudson: ^ | 16:08 |
bknudson | you need the watch | 16:08 |
*** exploreshaifali has joined #openstack-oslo | 16:17 | |
openstackgerrit | Ken Giusti proposed a change to openstack/oslo.messaging: Provide better detection of failures during message send https://review.openstack.org/126065 | 16:18 |
openstackgerrit | Ken Giusti proposed a change to openstack/oslo.messaging: Implement message flow control using link credit https://review.openstack.org/127550 | 16:18 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/130088 | 16:19 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.config: Updated from global requirements https://review.openstack.org/130323 | 16:19 |
*** exploreshaifali has quit IRC | 16:26 | |
dimsum_ | dhellmann: beekneemech: LOL | 16:27 |
*** freyes has joined #openstack-oslo | 16:34 | |
openstackgerrit | A change was merged to openstack/stevedore: Add pbr to dependency list https://review.openstack.org/130641 | 16:34 |
openstackgerrit | Joshua Harlow proposed a change to openstack/oslo.db: Fix more invalid encoding and scoping issues https://review.openstack.org/120198 | 16:35 |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo.messaging: Raise the number of RPC threads to run in parallel https://review.openstack.org/130278 | 16:38 |
*** harlowja_at_home has quit IRC | 16:44 | |
beekneemech | dhellmann: BTW, do we want to release oslo.concurrency early next week? | 16:46 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: db: Add long-lived-transactionalized-db-fixtures https://review.openstack.org/117335 | 16:46 |
beekneemech | And so it begins... | 16:46 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: config: Add a spec for a oslo.config dictionary based config opts https://review.openstack.org/130047 | 16:47 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: config: Add a spec for a oslo.config dictionary based config opts https://review.openstack.org/130047 | 16:48 |
dhellmann | beekneemech: if you're ready to release oslo.concurrency, let's do it | 16:48 |
dhellmann | beekneemech: do you want me to do it? | 16:48 |
beekneemech | dhellmann: Yeah, I think we're ready. | 16:48 |
beekneemech | dhellmann: I'm pretty sure I don't have release permissions. | 16:48 |
dhellmann | beekneemech: I can do it today, since nothing uses it | 16:48 |
beekneemech | Ah, true. | 16:49 |
dhellmann | beekneemech: we should fix the permissions thing, but I should write up how to use ttx's release script first | 16:49 |
beekneemech | dhellmann: Yeah, I've just never released anything so I haven't been through the process. | 16:49 |
*** viktors has quit IRC | 16:49 | |
beekneemech | And I hear the release people get perturbed if you screw one up. :-) | 16:50 |
beekneemech | dhellmann: 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 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: middleware: oslo.middleware healthcheck https://review.openstack.org/130453 | 16:51 |
dhellmann | beekneemech: yeah, I guess we have to set the topics by hand | 16:52 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: cache: Convert oslo.cache to use dogpile.cache https://review.openstack.org/124776 | 16:53 |
dhellmann | beekneemech: 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 happen | 16:53 |
beekneemech | dhellmann: 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-oslo | 16:54 | |
dhellmann | yeah | 16:54 |
beekneemech | Unless they happen to remember to change their local topic branch name. | 16:54 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: db: Enable MySQL Connector driver https://review.openstack.org/125044 | 16:55 |
*** ihrachyshka has quit IRC | 16:55 | |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: i18n: Add spec for more-gettext-support https://review.openstack.org/121459 | 16:55 |
beekneemech | Bleh, this is dropping all of the votes on the changes I alter. :-( | 16:56 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: db: Make EngineFacade a Facade https://review.openstack.org/125181 | 16:56 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: messaging: Adding spec for trusted messaging plugin. https://review.openstack.org/128926 | 16:59 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: db: Alembic environment runner specification https://review.openstack.org/112842 | 16:59 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: middleware: Add CORS middleware spec https://review.openstack.org/119485 | 17:00 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: log: Restore remove-context-adapter spec https://review.openstack.org/124847 | 17:01 |
beekneemech | dhellmann: Okay, that should take care of all the lib-specific specs. I left the adoption, incubator, and cross-project specs alone. | 17:02 |
beekneemech | dhellmann: Oh, and you were going to rebase https://review.openstack.org/#/c/113940/ right? | 17:03 |
dhellmann | beekneemech: 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 rebase | 17:05 |
beekneemech | dhellmann: Yeah, I should have time to do it. I'm just waiting for images to build anyway. :-) | 17:06 |
dhellmann | beekneemech: cool, thank you | 17:06 |
dhellmann | beekneemech: oslo.concurrency 0.1.0 released | 17:12 |
beekneemech | dhellmann: \o/ | 17:13 |
dhellmann | beekneemech: I think we need to do oslo.context next, so we can update the logging code to use it | 17:14 |
dhellmann | we still need a spec for that | 17:14 |
beekneemech | dhellmann: Sounds good | 17:15 |
dhellmann | but first I need food :-) | 17:15 |
beekneemech | I'm headed that way as soon as I finish this rebase. :-) | 17:16 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-incubator: add list_opts to all modules with configuration options https://review.openstack.org/113940 | 17:19 |
*** takedakn has joined #openstack-oslo | 17:25 | |
*** takedakn has quit IRC | 17:31 | |
*** takedakn has joined #openstack-oslo | 17:32 | |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo-incubator: add list_opts to all modules with configuration options https://review.openstack.org/113940 | 17:35 |
*** alexpilotti has quit IRC | 17:49 | |
*** alexpilotti has joined #openstack-oslo | 17:50 | |
dimsum_ | added a global req review - https://review.openstack.org/#/c/130862/ oslo.concurrency | 17:51 |
*** alexpilotti has quit IRC | 17:51 | |
dimsum_ | Anyone cutting a review for Nova? | 17:51 |
*** andreykurilin_ has joined #openstack-oslo | 17:51 | |
*** alexpilotti has joined #openstack-oslo | 17:52 | |
*** alexpilotti has quit IRC | 17:53 | |
*** harlowja_away is now known as harlowja | 17:55 | |
*** alexpilotti has joined #openstack-oslo | 17:55 | |
*** harlowja is now known as harlowja_away | 17:56 | |
*** alexpilotti has quit IRC | 17:56 | |
*** harlowja_away is now known as harlowja | 17:58 | |
*** alexpilotti has joined #openstack-oslo | 17:58 | |
*** alexpilotti has quit IRC | 17:59 | |
harlowja | dhellmann so the current thinking is to have pbr back in requirements.txt | 17:59 |
harlowja | ? | 17:59 |
harlowja | final answer this time :-P | 17:59 |
*** alexpilotti has joined #openstack-oslo | 18:00 | |
*** alexpilotti has quit IRC | 18:01 | |
*** alexpilotti has joined #openstack-oslo | 18:02 | |
*** alexpilotti has quit IRC | 18:03 | |
*** exploreshaifali has quit IRC | 18:13 | |
*** AAzza_afk has joined #openstack-oslo | 18:14 | |
*** AAzza_afk is now known as AAzza | 18:14 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/oslo.concurrency: Improve lock_path help and documentation https://review.openstack.org/130354 | 18:28 |
*** AAzza has left #openstack-oslo | 18:30 | |
*** sreshetnyak has quit IRC | 18:32 | |
*** yamahata has quit IRC | 18:32 | |
*** amrith is now known as _amrith_ | 18:35 | |
*** _amrith_ is now known as amrith | 18:36 | |
*** e0ne has quit IRC | 18:44 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/oslo.concurrency: Improve lock_path help and documentation https://review.openstack.org/130354 | 18:44 |
*** exploreshaifali has joined #openstack-oslo | 18:46 | |
*** andreykurilin_ has quit IRC | 18:46 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/oslo.concurrency: Allow the lock delay to be provided https://review.openstack.org/130872 | 18:54 |
openstackgerrit | Joshua Harlow proposed a change to openstack/oslo.concurrency: Allow the lock delay to be provided https://review.openstack.org/130872 | 18:55 |
*** ajo_ has joined #openstack-oslo | 18:58 | |
dhellmann | harlowja: yeah, apparently pip and setuptools do weirdness if you don't and it is installed as a drive-by dependency | 19:01 |
harlowja | hmmm, sad | 19:01 |
dhellmann | harlowja: yeah | 19:02 |
*** exploreshaifali has quit IRC | 19:06 | |
*** jecarey has quit IRC | 19:10 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/pylockfile: Begin moving some of the common code to a shared base https://review.openstack.org/122253 | 19:17 |
*** pblaho has quit IRC | 19:21 | |
*** ajo_ has quit IRC | 19:30 | |
*** ajo_ has joined #openstack-oslo | 19:35 | |
*** andreykurilin_ has joined #openstack-oslo | 20:01 | |
*** ajo_ has quit IRC | 20:06 | |
*** e0ne has joined #openstack-oslo | 20:09 | |
*** amrith is now known as _amrith_ | 20:19 | |
*** e0ne has quit IRC | 20:22 | |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 20:24 | |
*** e0ne has joined #openstack-oslo | 20:26 | |
*** e0ne has quit IRC | 20:28 | |
*** kgiusti has left #openstack-oslo | 20:39 | |
*** ajo_ has joined #openstack-oslo | 20:56 | |
*** jgrimm is now known as zz_jgrimm | 20:57 | |
openstackgerrit | A change was merged to openstack/oslo.log: Updated from global requirements https://review.openstack.org/130326 | 20:59 |
dimsum_ | dhellmann: patching oslo.concurrency w/ nova - runs into this problem | 21:01 |
dimsum_ | DuplicateOptError: duplicate option: default_log_levels | 21:01 |
dimsum_ | http://logs.openstack.org/70/130870/3/check/gate-nova-pep8/8ad1abf/console.html | 21:01 |
beekneemech | dimsum_: That opt must have changed between the version in nova and the one in oslo.concurrency. | 21:04 |
beekneemech | We'll need to figure out which is out of date and update it. | 21:05 |
dimsum_ | nova/nova/openstack/common/log.py | 21:05 |
dimsum_ | oslo.concurrency/oslo/concurrency/openstack/common/log.py | 21:05 |
dimsum_ | both have it | 21:06 |
*** ajo_ has quit IRC | 21:08 | |
*** ajo_ has joined #openstack-oslo | 21:08 | |
beekneemech | dimsum_: Ah, DEFAULT_LOG_LEVELS is different between the two. | 21:09 |
beekneemech | Wow, that could be nasty. | 21:10 |
beekneemech | We 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 release | 21:11 |
dimsum_ | sigh | 21:12 |
beekneemech | Hmm, yeah, maybe oslo.concurrency just shouldn't use oslo logging for now. | 21:13 |
beekneemech | In fact, now that I say that I vaguely recall having that discussion at some point. | 21:13 |
beekneemech | Eventually 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 |
beekneemech | dhellmann: ^Thoughts? | 21:14 |
*** ajo_ has quit IRC | 21:15 | |
dimsum_ | technically it's fileutils that using it not lockutils or processutils | 21:15 |
dimsum_ | beekneemech: more local change is to nuke fileutils from oslo.concurrr by having local copy of ensure_tree | 21:18 |
dimsum_ | and then make a new release | 21:18 |
dhellmann | beekneemech: 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 logging | 21: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 itselg | 21:22 |
dhellmann | dimsum_: grr | 21:23 |
dhellmann | dimsum_: so that's where your suggestion about ensure_tree came from? | 21:23 |
dimsum_ | y | 21:23 |
dhellmann | does making the value of DEFAULT_LOG_LEVELS the same fix it? | 21:24 |
dimsum_ | "DuplicateOptError: duplicate option: default_log_levels" | 21:24 |
dhellmann | because I'd rather actually work on oslo.context and then oslo.log than hack around with oslo.concurrency | 21:24 |
dhellmann | if all of the option settings are the same it should be OK to register the same option twice | 21:24 |
beekneemech | It should fix it, but we're almost certainly going to run into this again. | 21:24 |
dhellmann | certainly | 21:24 |
beekneemech | And potentially going to break the world every time we release oslo.concurrency, if there was a log module update. | 21:25 |
dhellmann | right, let's work our way from the bottom up instead | 21:26 |
dimsum_ | opened a bug https://bugs.launchpad.net/oslo.concurrency/+bug/1385492 | 21:26 |
dimsum_ | dhellmann: which is? | 21:26 |
beekneemech | Wait, is fileutils seriously pulling in log.py solely for one debug message? | 21:27 |
beekneemech | Yes, yes it is. | 21:27 |
dhellmann | beekneemech: if that's the case we can just fix that and sync a new copy into oslo.concurrency | 21:27 |
beekneemech | We should just nuke that. | 21:28 |
beekneemech | dhellmann: Yeah | 21:28 |
dhellmann | dimsum_: oslo.context, then oslo.log, then a new oslo.concurrency | 21:28 |
dimsum_ | cool +1 | 21:28 |
dhellmann | dimsum_, beekneemech : basically, we have to do oslo.context and oslo.log before we start any other libraries | 21:28 |
dhellmann | I added some notes to the oslo.context section of https://etherpad.openstack.org/p/kilo-oslo-library-proposals | 21: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 fileutils | 21:30 |
dhellmann | dimsum_: yep, let's get rid of the oslo logging stuff in fileutils and just use the python log module | 21:30 |
dimsum_ | dhellmann: +1 | 21:30 |
dhellmann | dimsum_, 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 |
beekneemech | dhellmann: Sounds good | 21:32 |
beekneemech | dimsum_: 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 |
beekneemech | dimsum_: Perfect :-) | 21:33 |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 21:36 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo-incubator: Switch fileutils to use python logging https://review.openstack.org/130892 | 21:36 |
dhellmann | dimsum_: +2 | 21:38 |
*** jeckersb is now known as jeckersb_gone | 21:39 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Merge fileutils from oslo-incubator https://review.openstack.org/130893 | 21:41 |
dimsum_ | that should do the trick. cross my fingers | 21:41 |
dimsum_ | dhellmann: beekneemech has karma for pushing out 0.1.1? | 21:44 |
beekneemech | No | 21:44 |
dimsum_ | k, dhellmann is on the hook then :) | 21:44 |
beekneemech | The 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 |
openstackgerrit | A change was merged to openstack/oslo-incubator: Switch fileutils to use python logging https://review.openstack.org/130892 | 21:45 |
beekneemech | dimsum_: I think it's fine, but it wouldn't hurt. | 21:46 |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Merge fileutils from oslo-incubator https://review.openstack.org/130893 | 21:47 |
dimsum_ | there you go | 21:47 |
beekneemech | I wonder how many of those other modules should be removed now too. | 21:47 |
dhellmann | dimsum_: should that be linked to the bug you opened? | 21:47 |
beekneemech | Ooh, right. | 21:47 |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.concurrency: Merge fileutils from oslo-incubator https://review.openstack.org/130893 | 21:48 |
* dhellmann is going to be a stickler for bug/blueprint linkages this cycle | 21:48 | |
dimsum_ | :) | 21:48 |
beekneemech | dhellmann: 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 message | 21:49 |
dimsum_ | back in a bit | 21:49 |
*** mriedem has left #openstack-oslo | 21:49 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 21:58 | |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo.concurrency: Remove unused incubator modules https://review.openstack.org/130894 | 21:58 |
openstackgerrit | Ben Nemec proposed a change to openstack/oslo.concurrency: Clean up lockutils logging https://review.openstack.org/130648 | 22:02 |
*** dims_ has joined #openstack-oslo | 22:03 | |
*** dims__ has joined #openstack-oslo | 22:05 | |
dhellmann | beekneemech, dimsum_ : I updated https://wiki.openstack.org/wiki/Oslo/Dependencies to highlight some of the ones we know we're working on this cycle | 22:05 |
*** dimsum_ has quit IRC | 22:06 | |
*** dims_ has quit IRC | 22:08 | |
* beekneemech stretches the window across both monitors and it's _almost_ wide enough :-) | 22:09 | |
beekneemech | dhellmann: lgtm | 22:09 |
dims__ | :) | 22:11 |
*** andreykurilin_ has quit IRC | 22:13 | |
* dims__ checks on 130894 and 130893 | 22:14 | |
*** gordc has quit IRC | 22:18 | |
*** SridharG has quit IRC | 22:23 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/130088 | 22:32 |
*** bknudson has quit IRC | 22:37 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/130088 | 22:38 |
*** rjrjr__ has joined #openstack-oslo | 22: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_away | 23:00 | |
*** harlowja_away is now known as harlowja | 23:24 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add a history retry object, makes retry histories easier to use https://review.openstack.org/123616 | 23:53 |
*** _amrith_ is now known as amrith | 23:56 | |
openstackgerrit | A change was merged to openstack/oslo.concurrency: Merge fileutils from oslo-incubator https://review.openstack.org/130893 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!