| *** oomichi has joined #openstack-oslo | 00:06 | |
| *** dstanek_zzz is now known as dstanek | 00:08 | |
| *** yamahata has quit IRC | 00:08 | |
| *** dstanek is now known as dstanek_zzz | 00:18 | |
| *** dstanek_zzz is now known as dstanek | 01:09 | |
| *** dstanek is now known as dstanek_zzz | 01:19 | |
| *** zhiyan_ is now known as zhiyan | 02:00 | |
| *** dstanek_zzz is now known as dstanek | 02:09 | |
| *** zhiyan is now known as zhiyan_ | 02:18 | |
| *** dstanek is now known as dstanek_zzz | 02:19 | |
| *** praneshp_ has joined #openstack-oslo | 02:24 | |
| *** praneshp has quit IRC | 02:26 | |
| *** praneshp_ is now known as praneshp | 02:26 | |
| *** zhiyan_ is now known as zhiyan | 02:27 | |
| *** arnaud__ has joined #openstack-oslo | 02:33 | |
| *** yamahata has joined #openstack-oslo | 02:36 | |
| *** arnaud__ has quit IRC | 02:45 | |
| *** dims has quit IRC | 02:57 | |
| *** dstanek_zzz is now known as dstanek | 03:10 | |
| *** jeffreycoho has quit IRC | 03:20 | |
| *** dstanek is now known as dstanek_zzz | 03:20 | |
| *** czzl has joined #openstack-oslo | 03:28 | |
| zhiyan | dhellmann: hi, around? i have a question on https://bugs.launchpad.net/glance/+bug/1300546 | 03:44 |
|---|---|---|
| zhiyan | for your comments, can you give me more details | 03:45 |
| *** amotoki has joined #openstack-oslo | 03:57 | |
| *** arnaud__ has joined #openstack-oslo | 04:02 | |
| *** morganfainberg has quit IRC | 04:04 | |
| *** morganfainberg has joined #openstack-oslo | 04:04 | |
| *** morganfainberg has quit IRC | 04:05 | |
| *** morganfainberg has joined #openstack-oslo | 04:06 | |
| *** dstanek_zzz is now known as dstanek | 04:11 | |
| *** arnaud__ has quit IRC | 04:15 | |
| *** dstanek is now known as dstanek_zzz | 04:21 | |
| *** czzl has left #openstack-oslo | 04:22 | |
| *** arnaud__ has joined #openstack-oslo | 04:24 | |
| *** dims has joined #openstack-oslo | 04:26 | |
| *** dims has quit IRC | 04:30 | |
| *** arnaud__ has quit IRC | 04:42 | |
| *** rpodolyaka has joined #openstack-oslo | 04:43 | |
| *** rpodolyaka has quit IRC | 04:43 | |
| *** dims has joined #openstack-oslo | 04:46 | |
| *** dims has quit IRC | 04:51 | |
| *** rpodolyaka has joined #openstack-oslo | 04:54 | |
| *** dstanek_zzz is now known as dstanek | 05:12 | |
| *** ildikov has quit IRC | 05:21 | |
| *** dstanek is now known as dstanek_zzz | 05:22 | |
| *** rpodolyaka has quit IRC | 05:29 | |
| *** dims has joined #openstack-oslo | 05:47 | |
| *** ildikov has joined #openstack-oslo | 05:50 | |
| *** dims has quit IRC | 05:51 | |
| *** zhiyan is now known as zhiyan_ | 05:55 | |
| *** zhiyan_ is now known as zhiyan | 05:56 | |
| *** rpodolyaka has joined #openstack-oslo | 06:02 | |
| openstackgerrit | Mark McLoughlin proposed a change to openstack/oslo.config: Add more tests for positional CLI opts https://review.openstack.org/100150 | 06:08 |
| *** rpodolyaka has quit IRC | 06:08 | |
| *** dstanek_zzz is now known as dstanek | 06:12 | |
| *** arnaud__ has joined #openstack-oslo | 06:18 | |
| *** arnaud__ has quit IRC | 06:18 | |
| *** arnaud__ has joined #openstack-oslo | 06:21 | |
| *** dstanek is now known as dstanek_zzz | 06:22 | |
| *** amotoki has quit IRC | 06:26 | |
| *** dims has joined #openstack-oslo | 06:48 | |
| *** dims has quit IRC | 06:52 | |
| *** ihrachyshka has joined #openstack-oslo | 06:57 | |
| *** amotoki has joined #openstack-oslo | 07:06 | |
| *** dstanek_zzz is now known as dstanek | 07:13 | |
| *** dstanek is now known as dstanek_zzz | 07:23 | |
| *** flaper87|afk is now known as flaper87 | 07:24 | |
| *** malor is now known as rpodolyaka | 07:27 | |
| bogdando | hi. https://review.openstack.org/#/c/99017/ is good to go | 07:29 |
| *** tkelsey has joined #openstack-oslo | 07:42 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/oslo.config: Introduce Opts for IP addresses https://review.openstack.org/92396 | 07:43 |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo-incubator: Mark db code as obsolete https://review.openstack.org/100175 | 07:44 |
| *** dims has joined #openstack-oslo | 07:49 | |
| *** dims has quit IRC | 07:53 | |
| *** viktors has joined #openstack-oslo | 07:54 | |
| *** nacim has joined #openstack-oslo | 07:54 | |
| *** i159 has joined #openstack-oslo | 08:02 | |
| *** i159 is now known as I159 | 08:02 | |
| *** markmc has joined #openstack-oslo | 08:05 | |
| openstackgerrit | Zhu Zhu proposed a change to openstack/oslo-incubator: Fix issue of duplicate creation for connections in impl_qpid https://review.openstack.org/100177 | 08:05 |
| I159 | Hi all! Please, review the patch https://review.openstack.org/#/c/93398/, if you were be so pleased. We really need it to be merged. Thanks for care! | 08:06 |
| *** dstanek_zzz is now known as dstanek | 08:14 | |
| *** jaosorior has joined #openstack-oslo | 08:14 | |
| ihrachyshka | dhellmann: hey! thanks for finally +2 json patch for icehouse! I'd like to ask you to do the same with two more patches from the same series that are still waiting for another +2 (markmc already set one). | 08:22 |
| *** dstanek is now known as dstanek_zzz | 08:24 | |
| *** arnaud__ has quit IRC | 08:47 | |
| *** dims has joined #openstack-oslo | 08:49 | |
| *** dims has quit IRC | 08:54 | |
| *** nacim has quit IRC | 08:54 | |
| *** zhiyan is now known as zhiyan_ | 08:56 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo.messaging: Synced from oslo-incubator https://review.openstack.org/95696 | 09:00 |
| *** nacim has joined #openstack-oslo | 09:06 | |
| *** nacim has quit IRC | 09:10 | |
| *** dstanek_zzz is now known as dstanek | 09:15 | |
| *** ihrachyshka has quit IRC | 09:15 | |
| *** nacim has joined #openstack-oslo | 09:23 | |
| *** dstanek is now known as dstanek_zzz | 09:25 | |
| *** praneshp has quit IRC | 09:30 | |
| *** ihrachyshka has joined #openstack-oslo | 09:40 | |
| *** mrda is now known as mrda-away | 09:47 | |
| *** dims has joined #openstack-oslo | 09:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/99084 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.config: Updated from global requirements https://review.openstack.org/99085 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/99086 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Updated from global requirements https://review.openstack.org/100107 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/99087 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslotest: Updated from global requirements https://review.openstack.org/99088 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/99089 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslosphinx: Updated from global requirements https://review.openstack.org/99090 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/99091 | 09:52 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 09:53 |
| *** dims has quit IRC | 09:54 | |
| openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo.db: Fix kwarg passed twice error in EngineFacade.from_config() https://review.openstack.org/100209 | 10:05 |
| *** openstackgerrit has quit IRC | 10:06 | |
| *** openstackgerrit has joined #openstack-oslo | 10:07 | |
| openstackgerrit | Zhu Zhu proposed a change to openstack/oslo-incubator: Fix issue of duplicate creation for connections in impl_qpid https://review.openstack.org/100177 | 10:07 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/oslo.config: Introduce Opts for IP addresses https://review.openstack.org/92396 | 10:08 |
| openstackgerrit | Zhu Zhu proposed a change to openstack/oslo.messaging: Encode/decode the auth_token when pack/unpack message https://review.openstack.org/94881 | 10:09 |
| *** dstanek_zzz is now known as dstanek | 10:16 | |
| *** GheRivero has joined #openstack-oslo | 10:16 | |
| GheRivero | Is it intentional for oslo.db to be a prerelease in pypi? | 10:17 |
| *** dstanek is now known as dstanek_zzz | 10:26 | |
| markmc | GheRivero, likely that it will be in alpha until close to Juno final | 10:31 |
| markmc | GheRivero, https://wiki.openstack.org/wiki/Oslo/VersioningPolicy | 10:32 |
| *** openstackgerrit has quit IRC | 10:35 | |
| GheRivero | but that will avoid it to be installed via tox/requirements | 10:36 |
| *** openstackgerrit has joined #openstack-oslo | 10:36 | |
| markmc | GheRivero, http://lists.openstack.org/pipermail/openstack-dev/2014-June/037218.html | 10:37 |
| GheRivero | ups. I miss that! Thanks markmc | 10:38 |
| markmc | GheRivero, i.e. just do oslo.db>=1.0.0a1 | 10:38 |
| *** dims_ has joined #openstack-oslo | 10:51 | |
| *** dims_ has quit IRC | 10:55 | |
| *** yamahata has quit IRC | 11:10 | |
| *** dstanek_zzz is now known as dstanek | 11:16 | |
| *** dims_ has joined #openstack-oslo | 11:18 | |
| *** dstanek is now known as dstanek_zzz | 11:26 | |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests https://review.openstack.org/93424 | 11:49 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Implementation Alembic as migration engine https://review.openstack.org/99965 | 11:49 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: oslo.db.options module documentation https://review.openstack.org/93436 | 11:50 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: oslo.db.exceptions module documentation https://review.openstack.org/93435 | 11:50 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: oslo.db.api module documentation https://review.openstack.org/93434 | 11:50 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Changes import orders to pass H305 check https://review.openstack.org/99670 | 12:09 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Fixes whitespaces between imports to pass H307 https://review.openstack.org/99671 | 12:09 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Fixes comments to pass E265 check. https://review.openstack.org/99676 | 12:09 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Uses keyword params for i18n string to pass H703 https://review.openstack.org/99674 | 12:09 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Fixes indentations to pass E128 check. https://review.openstack.org/99675 | 12:09 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Updates one line docstring with dot to pass H402 https://review.openstack.org/99672 | 12:09 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Adds empty line to multilines docs to pass H405 https://review.openstack.org/99673 | 12:09 |
| *** pblaho has joined #openstack-oslo | 12:13 | |
| *** ildikov has quit IRC | 12:13 | |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/oslo-incubator: Add URI validation method https://review.openstack.org/99288 | 12:14 |
| *** dstanek_zzz is now known as dstanek | 12:17 | |
| *** morganfainberg has quit IRC | 12:18 | |
| *** morganfainberg has joined #openstack-oslo | 12:19 | |
| *** erecio has joined #openstack-oslo | 12:21 | |
| openstackgerrit | Elena Ezhova proposed a change to openstack/oslo.messaging: Handle unused allowed_remote_exmods in _multi_send https://review.openstack.org/100236 | 12:23 |
| *** yamahata has joined #openstack-oslo | 12:23 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/oslo-specs: graduate-oslo-utils https://review.openstack.org/98431 | 12:29 |
| *** pblaho has quit IRC | 12:37 | |
| *** ildikov has joined #openstack-oslo | 12:39 | |
| *** zhiyan_ is now known as zhiyan | 12:44 | |
| *** pblaho has joined #openstack-oslo | 12:44 | |
| ihrachyshka | markmc: around? | 12:44 |
| markmc | ihrachyshka, hi there | 12:44 |
| ihrachyshka | markmc: re jsonutils fix in icehouse. first, thanks for +2 voting them. the problem is that only one of them was pushed to tree, while two others still wait for approval (no second +2). and those patches are vital for the first one to work properly in all projects (should be synced there in one go). how should we handle that? | 12:49 |
| markmc | heh, small sentences work best on irc | 12:49 |
| * markmc reads | 12:49 | |
| ihrachyshka | markmc: I'm scared by people cherry-picking the merged fix without the two other | 12:49 |
| ihrachyshka | not *scared*, but *concerned* :) | 12:50 |
| ihrachyshka | https://review.openstack.org/#/q/project:openstack/oslo-incubator+branch:stable/icehouse+topic:simplejson-stable-backport,n,z | 12:50 |
| markmc | ping dhellmann again soon, you just pinged a little early for his tz | 12:50 |
| ihrachyshka | markmc: it's hard to tell who's located where :) | 12:52 |
| ihrachyshka | we should attach TZs to nicknames :) | 12:52 |
| * jd__ thanks markmc for that -1 on hacking downgrade | 12:56 | |
| *** ujjain has joined #openstack-oslo | 13:02 | |
| *** ujjain has quit IRC | 13:04 | |
| *** ujjain has joined #openstack-oslo | 13:04 | |
| *** dims_ has quit IRC | 13:10 | |
| *** dims_ has joined #openstack-oslo | 13:10 | |
| *** dims_ has quit IRC | 13:11 | |
| *** dims has joined #openstack-oslo | 13:12 | |
| *** pblaho is now known as pblaho|afk | 13:19 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Fix the notify method of the routing notifier https://review.openstack.org/92403 | 13:26 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Transport reconnection retries for notification https://review.openstack.org/92373 | 13:26 |
| *** pblaho|afk is now known as pblaho | 13:26 | |
| *** jecarey has joined #openstack-oslo | 13:30 | |
| sileht | markmc, I have fixed these two reviews: https://review.openstack.org/92403 https://review.openstack.org/92373 | 13:34 |
| markmc | sileht, cool | 13:34 |
| dhellmann | ihrachyshka: I'm in the US Eastern TZ. Looking at that patch series now. | 13:40 |
| ihrachyshka | dhellmann: thanks, I'll try to memorize that :) | 13:40 |
| dhellmann | ihrachyshka: no worries, I need to read the scrollback in the morning anyway, so just ping with what you need so I can follow up :-) | 13:41 |
| ihrachyshka | roger that captain | 13:42 |
| dhellmann | heh | 13:42 |
| dhellmann | ihrachyshka: I think they're all approved now | 13:43 |
| ihrachyshka | dhellmann: yay, thanks | 13:45 |
| *** mriedem has joined #openstack-oslo | 13:59 | |
| *** ildikov has quit IRC | 14:01 | |
| *** markmcclain has joined #openstack-oslo | 14:04 | |
| *** markmcclain1 has joined #openstack-oslo | 14:06 | |
| *** markmcclain has quit IRC | 14:08 | |
| *** czzl has joined #openstack-oslo | 14:09 | |
| *** dhellmann has quit IRC | 14:12 | |
| *** dhellmann has joined #openstack-oslo | 14:12 | |
| *** zzzeek has joined #openstack-oslo | 14:13 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Fix the notify method of the routing notifier https://review.openstack.org/92403 | 14:21 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Transport reconnection retries for notification https://review.openstack.org/92373 | 14:21 |
| *** czzl has left #openstack-oslo | 14:23 | |
| sileht | sorry markmc, I have missed a pep8 issue, you have to re +2 both patches | 14:24 |
| markmc | sileht, done | 14:26 |
| sileht | markmc, thanks | 14:26 |
| markmc | np | 14:26 |
| openstackgerrit | Tom Cammann proposed a change to openstack/oslo-incubator: Make periodic tasks run on regular spacing interval https://review.openstack.org/99695 | 14:29 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: oslo.db.api module documentation https://review.openstack.org/93434 | 14:29 |
| haypo | sileht: you may add a unit test for priority.lower() for your change "Fix the notify method of the routing notifier" | 14:31 |
| sileht | haypo, it's catched by the rpc2: driver in the unit test test_notify_filtered | 14:34 |
| *** ildikov has joined #openstack-oslo | 14:43 | |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Changes import orders to pass H305 check https://review.openstack.org/99670 | 14:45 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Fixes whitespaces between imports to pass H307 https://review.openstack.org/99671 | 14:45 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Fixes comments to pass E265 check. https://review.openstack.org/99676 | 14:45 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Uses keyword params for i18n string to pass H703 https://review.openstack.org/99674 | 14:45 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Fixes indentations to pass E128 check. https://review.openstack.org/99675 | 14:45 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Updates one line docstring with dot to pass H402 https://review.openstack.org/99672 | 14:45 |
| openstackgerrit | Petr Blaho proposed a change to openstack/oslo.db: Adds empty line to multilines docs to pass H405 https://review.openstack.org/99673 | 14:45 |
| *** ihrachyshka has quit IRC | 15:03 | |
| *** Guest8031 is now known as mgagne | 15:08 | |
| *** mgagne has joined #openstack-oslo | 15:08 | |
| *** sreshetnyak has quit IRC | 15:14 | |
| *** tsekiyama has joined #openstack-oslo | 15:14 | |
| *** markmc has quit IRC | 15:28 | |
| *** viktors is now known as viktors|afk | 15:32 | |
| *** zhiyan is now known as zhiyan_ | 15:52 | |
| *** dstanek is now known as dstanek_zzz | 15:55 | |
| *** I159 has quit IRC | 15:55 | |
| *** dstanek_zzz is now known as dstanek | 15:57 | |
| *** nacim has quit IRC | 16:01 | |
| *** pblaho is now known as pblaho|afk | 16:07 | |
| *** arnaud__ has joined #openstack-oslo | 16:08 | |
| *** jaosorior has quit IRC | 16:12 | |
| *** pblaho|afk is now known as pblaho | 16:12 | |
| *** markmcclain1 has quit IRC | 16:33 | |
| *** praneshp has joined #openstack-oslo | 16:36 | |
| *** pblaho is now known as pblaho|afk | 16:40 | |
| *** pblaho|afk is now known as pblaho | 16:44 | |
| *** pblaho has quit IRC | 16:45 | |
| *** dstanek is now known as dstanek_zzz | 16:53 | |
| *** dstanek_zzz is now known as dstanek | 16:58 | |
| *** markmcclain has joined #openstack-oslo | 17:22 | |
| *** ChanServ changes topic to "#openstack-oslo" | 17:40 | |
| openstackgerrit | George Peristerakis proposed a change to openstack/oslo-incubator: wrapping the mock module with six https://review.openstack.org/87375 | 17:46 |
| *** jaosorior has joined #openstack-oslo | 17:47 | |
| *** dims has quit IRC | 17:53 | |
| *** dims has joined #openstack-oslo | 17:54 | |
| *** harlowja has joined #openstack-oslo | 18:05 | |
| *** harlowja has quit IRC | 18:19 | |
| *** harlowja has joined #openstack-oslo | 18:19 | |
| *** tkelsey has quit IRC | 18:35 | |
| *** mrda-away is now known as mrda | 18:58 | |
| *** erecio has quit IRC | 18:59 | |
| *** erecio has joined #openstack-oslo | 18:59 | |
| *** dims has quit IRC | 19:07 | |
| *** dims has joined #openstack-oslo | 19:18 | |
| *** arnaud__ has quit IRC | 19:22 | |
| *** arnaud__ has joined #openstack-oslo | 19:22 | |
| *** 6JTAACWFP has joined #openstack-oslo | 19:22 | |
| *** dstanek is now known as dstanek_zzz | 19:25 | |
| *** erecio has quit IRC | 19:25 | |
| *** erecio has joined #openstack-oslo | 19:26 | |
| *** mestery has quit IRC | 19:27 | |
| *** tkelsey has joined #openstack-oslo | 19:33 | |
| *** tkelsey has quit IRC | 19:33 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 19:36 |
| *** krotscheck has quit IRC | 19:41 | |
| *** krotscheck has joined #openstack-oslo | 19:42 | |
| *** dstanek_zzz is now known as dstanek | 19:43 | |
| *** praneshp has quit IRC | 19:52 | |
| *** rpodolyaka1 has joined #openstack-oslo | 19:55 | |
| *** rpodolyaka1 has quit IRC | 20:01 | |
| *** rpodolyaka1 has joined #openstack-oslo | 20:10 | |
| *** dhellmann has quit IRC | 20:10 | |
| *** dhellmann has joined #openstack-oslo | 20:11 | |
| *** rpodolyaka1 has quit IRC | 20:13 | |
| *** dims has quit IRC | 20:14 | |
| *** dims has joined #openstack-oslo | 20:15 | |
| *** dstanek is now known as dstanek_zzz | 20:28 | |
| *** praneshp has joined #openstack-oslo | 20:34 | |
| *** markmcclain has quit IRC | 20:35 | |
| *** 6JTAACWFP has left #openstack-oslo | 20:48 | |
| *** dstanek_zzz is now known as dstanek | 20:49 | |
| *** ihrachyshka has joined #openstack-oslo | 20:53 | |
| *** erecio has quit IRC | 20:56 | |
| *** markmcclain has joined #openstack-oslo | 20:58 | |
| *** markmcclain1 has joined #openstack-oslo | 20:59 | |
| *** markmcclain1 has quit IRC | 20:59 | |
| *** markmcclain1 has joined #openstack-oslo | 21:00 | |
| *** markmcclain has quit IRC | 21:03 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Fix doc which should state fetch() usage https://review.openstack.org/100356 | 21:05 |
| *** dhellmann has quit IRC | 21:11 | |
| *** dhellmann has joined #openstack-oslo | 21:12 | |
| *** jaosorior has quit IRC | 21:22 | |
| *** dstanek is now known as dstanek_zzz | 21:41 | |
| *** dstanek_zzz is now known as dstanek | 21:41 | |
| *** mriedem has left #openstack-oslo | 21:57 | |
| *** mriedem has quit IRC | 21:57 | |
| ihrachyshka | dhellmann: hey! a question related to oslo.messaging porting | 22:00 |
| ihrachyshka | dhellmann: in some incubator code that we still use, old notifier and rpc are still used. F.e. log_handler.py, middleware/notifier.py. Meaning, we're probably not safe to remove them. At the same time, it seems we're not safe to run any code that uses old RPC, at least because it attempts to import its options on import and result in duplicate options error. How should we handle that? | 22:02 |
| *** dstanek is now known as dstanek_zzz | 22:02 | |
| *** markmcclain1 has quit IRC | 22:02 | |
| *** dstanek_zzz is now known as dstanek | 22:05 | |
| ihrachyshka | for middleware, it seems this particular 'notifier.py' file is not really imported from anywhere, so maybe it's fine to leave it. But for log_handler, it's not that obvious. Are we safe to leave it? See: https://github.com/openstack/neutron/blob/master/neutron/openstack/common/log_handler.py#L24 | 22:06 |
| ihrachyshka | dhellmann: and on related note, even if we don't use code in middleware/notifier.py, does it mean that we will still keep notifier code in the tree to fulfil middleware dependencies? | 22:07 |
| dhellmann | ihrachyshka: we will only see errors if the options don't match -- are you seeing that? | 22:08 |
| dhellmann | ihrachyshka: the middleware notifier is already in pycadf; the log handler will have to be updated when oslo.log graduates | 22:09 |
| dhellmann | unless it's already in oslo.messaging as it is | 22:09 |
| ihrachyshka | dhellmann: for what I saw before, if oslo.messaging and oslo-rpc are imported, I got duplicate options errors. I don't know whether we may conclude they don't match. | 22:09 |
| ihrachyshka | 'unless it's already in oslo.messaging' log_handler? | 22:10 |
| dhellmann | ihrachyshka: yes, it sounds like they don't, but we shouldn't ever do that | 22:10 |
| dhellmann | ihrachyshka: log_handler.py needs to be added to oslo.messaging (https://wiki.openstack.org/wiki/Oslo/GraduationStatus#log_handler) | 22:11 |
| ihrachyshka | dhellmann: I don't. but now I'm curious whether it's correct to remove rpc and notifier modules from the tree if we still have references to them from other modules (even though maybe those parts of those modules are not used) | 22:11 |
| dhellmann | ihrachyshka: well, we have the whole cycle to work that out | 22:11 |
| ihrachyshka | dhellmann: ok, it's clear on long distance. the question remains how to handle that on short distance (before it's graduated to oslo.messaging). | 22:11 |
| *** zzzeek has quit IRC | 22:12 | |
| dhellmann | ihrachyshka: the other projects haven't reported any issues because of this, yet, so I'm not too worried. | 22:14 |
| ihrachyshka | dhellmann: my reviewers are a bit :) but I'll convince them :) | 22:15 |
| dhellmann | ihrachyshka: I would be very happy to have someone work on moving log_handler to oslo.messaging | 22:15 |
| ihrachyshka | dhellmann: I've put that into personal todo. I can't guarantee I'll take on that now, but that's on my radar now | 22:16 |
| dhellmann | ihrachyshka: thanks, that would be great :-) | 22:17 |
| dhellmann | ihrachyshka: in the mean time, point out to the reviewers that the other projects have not reported issues from any deployers with that log handler | 22:17 |
| ihrachyshka | will do, thanks for clarifications and pointers | 22:18 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Concretely define & check identity transitions https://review.openstack.org/100373 | 22:18 |
| *** jecarey has quit IRC | 22:28 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Concretely define & check identity transitions https://review.openstack.org/100373 | 22:32 |
| *** flaper87 is now known as flaper87|afk | 22:37 | |
| *** mestery has joined #openstack-oslo | 22:42 | |
| *** ihrachyshka has quit IRC | 22:44 | |
| *** dstanek is now known as dstanek_zzz | 22:48 | |
| *** mestery has quit IRC | 22:55 | |
| *** dstanek_zzz is now known as dstanek | 23:09 | |
| *** yamahata has quit IRC | 23:18 | |
| *** dstanek is now known as dstanek_zzz | 23:19 | |
| *** dstanek_zzz is now known as dstanek | 23:22 | |
| *** zzzeek has joined #openstack-oslo | 23:23 | |
| *** dstanek is now known as dstanek_zzz | 23:32 | |
| *** dims has quit IRC | 23:41 | |
| *** dstanek_zzz is now known as dstanek | 23:47 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!