*** bknudson has joined #openstack-oslo | 00:06 | |
*** bknudson has quit IRC | 00:11 | |
*** dstanek_zzz is now known as dstanek | 00:19 | |
*** bknudson has joined #openstack-oslo | 00:24 | |
*** tsekiyam_ has joined #openstack-oslo | 00:27 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Translate engine runner into a well defined state-machine https://review.openstack.org/106535 | 00:28 |
---|---|---|
*** tsekiyama has quit IRC | 00:30 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Have the dispatch_job function return a future https://review.openstack.org/103712 | 00:32 |
*** zzzeek has quit IRC | 00:42 | |
*** arnaud__ has quit IRC | 00:44 | |
*** tsekiyam_ has quit IRC | 00:52 | |
*** dstanek is now known as dstanek_zzz | 00:54 | |
*** lbragstad has quit IRC | 00:57 | |
*** zzzeek has joined #openstack-oslo | 00:59 | |
*** dstanek_zzz is now known as dstanek | 01:02 | |
*** arnaud__ has joined #openstack-oslo | 01:02 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Allow tox tests with complex OS_TEST_DBAPI_CONNECTION URLs https://review.openstack.org/104428 | 01:19 |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Support OS_TEST_DBAPI_ADMIN_CONNECTION override https://review.openstack.org/104447 | 01:19 |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Don't drop pre-existing database before tests https://review.openstack.org/104430 | 01:33 |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Don't drop pre-existing database before tests https://review.openstack.org/104430 | 01:46 |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.utils: Switch to standard python logging https://review.openstack.org/106745 | 01:46 |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.utils: Move units into oslo.utils https://review.openstack.org/106753 | 01:47 |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Allow a jobs posted book to be none by default https://review.openstack.org/106915 | 01:50 |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Allow a jobs posted book to be none by default https://review.openstack.org/106915 | 01:50 |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.utils: Switch to oslo.i18n and remove any dependency on oslo-incubator https://review.openstack.org/106756 | 01:51 |
*** arnaud__ has quit IRC | 01:53 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.utils: export only try_import in __all__ https://review.openstack.org/106778 | 01:54 |
openstackgerrit | A change was merged to openstack-dev/oslo-cookiecutter: Add default openstack-common.conf https://review.openstack.org/106881 | 01:58 |
openstackgerrit | A change was merged to openstack-dev/cookiecutter: Run py33 tests before py2x https://review.openstack.org/106873 | 01:58 |
*** dstanek is now known as dstanek_zzz | 02:01 | |
*** dstanek_zzz is now known as dstanek | 02:03 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Test that concurrent sqlalchemy transactions don't block https://review.openstack.org/104436 | 02:04 |
*** mriedem has left #openstack-oslo | 02:13 | |
*** mriedem has quit IRC | 02:13 | |
*** dims__ has quit IRC | 02:33 | |
*** yamahata has joined #openstack-oslo | 02:39 | |
*** openstackgerrit has quit IRC | 02:42 | |
*** ildikov has quit IRC | 02:42 | |
*** Alexei_987 has quit IRC | 02:42 | |
*** dstanek has quit IRC | 02:42 | |
*** ildikov has joined #openstack-oslo | 02:42 | |
*** Alexei_987 has joined #openstack-oslo | 02:42 | |
*** openstackgerrit has joined #openstack-oslo | 02:44 | |
*** dstanek has joined #openstack-oslo | 02:44 | |
*** jecarey has joined #openstack-oslo | 02:47 | |
*** harlowja is now known as harlowja_away | 02:54 | |
*** arnaud__ has joined #openstack-oslo | 02:56 | |
*** harlowja_away is now known as harlowja | 02:58 | |
*** SridharG has joined #openstack-oslo | 02:59 | |
*** arnaud__ has quit IRC | 03:04 | |
*** jecarey has quit IRC | 03:12 | |
*** dstanek has quit IRC | 03:12 | |
*** jogo has quit IRC | 03:12 | |
*** hyakuhei has quit IRC | 03:12 | |
*** anteaya has quit IRC | 03:12 | |
*** yamahata__ has quit IRC | 03:12 | |
*** morganfainberg has quit IRC | 03:12 | |
*** openstackgerrit has quit IRC | 03:12 | |
*** ildikov has quit IRC | 03:12 | |
*** HenryG has quit IRC | 03:12 | |
*** dteselkin has quit IRC | 03:12 | |
*** krotscheck has quit IRC | 03:12 | |
*** yamahata has quit IRC | 03:12 | |
*** viktors has quit IRC | 03:12 | |
*** gmurphy has quit IRC | 03:12 | |
*** sreshetnyak has quit IRC | 03:12 | |
*** SergeyLukjanov has quit IRC | 03:12 | |
*** ruhe has quit IRC | 03:12 | |
*** gpocentek has quit IRC | 03:12 | |
*** tsufiev_ has quit IRC | 03:12 | |
*** ekarlso has quit IRC | 03:12 | |
*** tcammann_ has quit IRC | 03:12 | |
*** jokke_ has quit IRC | 03:12 | |
*** GheRivero has quit IRC | 03:12 | |
*** boris-42 has quit IRC | 03:12 | |
*** SridharG has quit IRC | 03:12 | |
*** mtreinish has quit IRC | 03:12 | |
*** therve has quit IRC | 03:12 | |
*** mgagne has quit IRC | 03:12 | |
*** ttx has quit IRC | 03:12 | |
*** russellb has quit IRC | 03:12 | |
*** creiht has quit IRC | 03:12 | |
*** flaper87|afk has quit IRC | 03:12 | |
*** Alexei_987 has quit IRC | 03:12 | |
*** zzzeek has quit IRC | 03:12 | |
*** bknudson has quit IRC | 03:12 | |
*** oomichi has quit IRC | 03:12 | |
*** liusheng has quit IRC | 03:12 | |
*** ajo has quit IRC | 03:12 | |
*** bnemec has quit IRC | 03:12 | |
*** ujjain has quit IRC | 03:12 | |
*** jraim has quit IRC | 03:12 | |
*** andreykurilin has quit IRC | 03:12 | |
*** SridharG has joined #openstack-oslo | 03:20 | |
*** openstackgerrit has joined #openstack-oslo | 03:20 | |
*** Alexei_987 has joined #openstack-oslo | 03:20 | |
*** ildikov has joined #openstack-oslo | 03:20 | |
*** yamahata has joined #openstack-oslo | 03:20 | |
*** zzzeek has joined #openstack-oslo | 03:20 | |
*** bknudson has joined #openstack-oslo | 03:20 | |
*** oomichi has joined #openstack-oslo | 03:20 | |
*** liusheng has joined #openstack-oslo | 03:20 | |
*** ajo has joined #openstack-oslo | 03:20 | |
*** viktors has joined #openstack-oslo | 03:20 | |
*** morganfainberg has joined #openstack-oslo | 03:20 | |
*** boris-42 has joined #openstack-oslo | 03:20 | |
*** HenryG has joined #openstack-oslo | 03:20 | |
*** dteselkin has joined #openstack-oslo | 03:20 | |
*** tsufiev_ has joined #openstack-oslo | 03:20 | |
*** ekarlso has joined #openstack-oslo | 03:20 | |
*** gmurphy has joined #openstack-oslo | 03:20 | |
*** mtreinish has joined #openstack-oslo | 03:20 | |
*** tcammann_ has joined #openstack-oslo | 03:20 | |
*** jokke_ has joined #openstack-oslo | 03:20 | |
*** andreykurilin has joined #openstack-oslo | 03:20 | |
*** jogo has joined #openstack-oslo | 03:20 | |
*** krotscheck has joined #openstack-oslo | 03:20 | |
*** SergeyLukjanov has joined #openstack-oslo | 03:20 | |
*** sreshetnyak has joined #openstack-oslo | 03:20 | |
*** ruhe has joined #openstack-oslo | 03:20 | |
*** gpocentek has joined #openstack-oslo | 03:20 | |
*** GheRivero has joined #openstack-oslo | 03:20 | |
*** ttx has joined #openstack-oslo | 03:20 | |
*** mgagne has joined #openstack-oslo | 03:20 | |
*** therve has joined #openstack-oslo | 03:20 | |
*** flaper87|afk has joined #openstack-oslo | 03:20 | |
*** creiht has joined #openstack-oslo | 03:20 | |
*** russellb has joined #openstack-oslo | 03:20 | |
*** jraim has joined #openstack-oslo | 03:20 | |
*** ujjain has joined #openstack-oslo | 03:20 | |
*** bnemec has joined #openstack-oslo | 03:20 | |
*** hyakuhei has joined #openstack-oslo | 03:20 | |
*** anteaya has joined #openstack-oslo | 03:20 | |
*** yamahata__ has joined #openstack-oslo | 03:20 | |
*** jecarey has joined #openstack-oslo | 03:22 | |
*** dstanek has joined #openstack-oslo | 03:22 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Test that concurrent sqlalchemy transactions don't block https://review.openstack.org/104436 | 03:56 |
*** arnaud__ has joined #openstack-oslo | 04:02 | |
*** arnaud__ has quit IRC | 04:23 | |
*** k4n0 has joined #openstack-oslo | 04:27 | |
*** zzzeek has quit IRC | 04:27 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Make MySQL deadlock regex generic across MySQL drivers https://review.openstack.org/106928 | 04:28 |
*** dims__ has joined #openstack-oslo | 04:31 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Translate engine runner into a well defined state-machine https://review.openstack.org/106535 | 04:33 |
*** dims__ has quit IRC | 04:36 | |
*** praneshp has joined #openstack-oslo | 04:37 | |
*** arnaud__ has joined #openstack-oslo | 04:40 | |
*** praneshp_ has joined #openstack-oslo | 04:43 | |
*** praneshp has quit IRC | 04:44 | |
*** praneshp_ is now known as praneshp | 04:44 | |
*** ildikov has quit IRC | 04:48 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add existing types to generated documentation https://review.openstack.org/106930 | 04:48 |
*** dims__ has joined #openstack-oslo | 04:57 | |
*** dims__ has quit IRC | 05:02 | |
*** ildikov has joined #openstack-oslo | 05:17 | |
*** harlowja is now known as harlowja_away | 05:49 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Translate engine runner into a well defined state-machine https://review.openstack.org/106535 | 05:51 |
*** arnaud has joined #openstack-oslo | 05:56 | |
*** dstanek is now known as dstanek_zzz | 06:00 | |
*** dstanek_zzz is now known as dstanek | 06:01 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Imported Translations from Transifex https://review.openstack.org/106938 | 06:02 |
*** dstanek is now known as dstanek_zzz | 06:11 | |
*** praneshp has quit IRC | 06:27 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add existing types to generated documentation https://review.openstack.org/106930 | 06:47 |
*** oomichi has quit IRC | 06:51 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add existing types to generated documentation https://review.openstack.org/106930 | 06:52 |
*** dims__ has joined #openstack-oslo | 06:59 | |
*** dstanek_zzz is now known as dstanek | 07:02 | |
*** dims__ has quit IRC | 07:04 | |
*** arnaud has quit IRC | 07:06 | |
*** arnaud__ has quit IRC | 07:06 | |
*** dstanek is now known as dstanek_zzz | 07:11 | |
*** oomichi has joined #openstack-oslo | 07:16 | |
*** pblaho has joined #openstack-oslo | 07:18 | |
*** tkelsey_ has joined #openstack-oslo | 07:26 | |
*** oomichi has quit IRC | 07:26 | |
*** mkoderer has joined #openstack-oslo | 07:28 | |
*** ihrachyshka has joined #openstack-oslo | 07:43 | |
*** i159 has joined #openstack-oslo | 07:56 | |
*** dims__ has joined #openstack-oslo | 07:59 | |
*** dstanek_zzz is now known as dstanek | 08:02 | |
*** abhishek___ has joined #openstack-oslo | 08:03 | |
*** nacim has joined #openstack-oslo | 08:04 | |
*** dims__ has quit IRC | 08:04 | |
abhishek___ | hi all, can any one please review this oslo-spec, https://review.openstack.org/#/c/106285/ | 08:04 |
abhishek___ | thank you | 08:04 |
*** dstanek is now known as dstanek_zzz | 08:13 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests https://review.openstack.org/93424 | 08:32 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Utility function for robust schema drop https://review.openstack.org/106419 | 08:33 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Implementation Alembic as migration engine https://review.openstack.org/99965 | 08:33 |
*** mrda is now known as mrda-away | 08:35 | |
abhishek___ | dhellmann:Hi, you around? | 08:39 |
*** Alexei_987 has quit IRC | 08:47 | |
*** andreykurilin has quit IRC | 08:54 | |
*** oomichi has joined #openstack-oslo | 08:59 | |
*** jecarey has quit IRC | 09:01 | |
*** dstanek_zzz is now known as dstanek | 09:01 | |
*** jecarey has joined #openstack-oslo | 09:02 | |
*** dstanek is now known as dstanek_zzz | 09:11 | |
*** markmc has joined #openstack-oslo | 09:18 | |
*** dims__ has joined #openstack-oslo | 09:29 | |
*** dims__ has quit IRC | 09:33 | |
openstackgerrit | A change was merged to openstack/oslo.vmware: Do not log the full session ID https://review.openstack.org/102200 | 09:38 |
*** Alexei_987 has joined #openstack-oslo | 09:39 | |
Alexei_987 | markmc: ping | 09:44 |
*** oomichi has quit IRC | 09:55 | |
*** dstanek_zzz has quit IRC | 09:55 | |
*** k4n0 has quit IRC | 09:55 | |
*** jogo has quit IRC | 09:55 | |
*** hyakuhei has quit IRC | 09:55 | |
*** anteaya has quit IRC | 09:55 | |
*** yamahata__ has quit IRC | 09:55 | |
*** morganfainberg has quit IRC | 09:56 | |
*** abhishek___ has quit IRC | 09:56 | |
*** ihrachyshka has quit IRC | 09:56 | |
*** tkelsey_ has quit IRC | 09:56 | |
*** openstackgerrit has quit IRC | 09:56 | |
*** HenryG has quit IRC | 09:56 | |
*** dteselkin has quit IRC | 09:56 | |
*** krotscheck has quit IRC | 09:56 | |
*** ildikov has quit IRC | 09:56 | |
*** yamahata has quit IRC | 09:56 | |
*** viktors has quit IRC | 09:56 | |
*** gmurphy has quit IRC | 09:56 | |
*** sreshetnyak has quit IRC | 09:56 | |
*** SergeyLukjanov has quit IRC | 09:56 | |
*** ruhe has quit IRC | 09:56 | |
*** gpocentek has quit IRC | 09:56 | |
*** tsufiev_ has quit IRC | 09:56 | |
*** ekarlso has quit IRC | 09:56 | |
*** tcammann_ has quit IRC | 09:56 | |
*** jokke_ has quit IRC | 09:56 | |
*** GheRivero has quit IRC | 09:56 | |
*** boris-42 has quit IRC | 09:56 | |
*** i159 has quit IRC | 09:56 | |
*** SridharG has quit IRC | 09:56 | |
*** mtreinish has quit IRC | 09:56 | |
*** therve has quit IRC | 09:56 | |
*** mgagne has quit IRC | 09:56 | |
*** ttx has quit IRC | 09:56 | |
*** russellb has quit IRC | 09:56 | |
*** creiht has quit IRC | 09:56 | |
*** flaper87|afk has quit IRC | 09:56 | |
*** mkoderer has quit IRC | 09:56 | |
*** bknudson has quit IRC | 09:56 | |
*** liusheng has quit IRC | 09:56 | |
*** ajo has quit IRC | 09:56 | |
*** bnemec has quit IRC | 09:56 | |
*** ujjain has quit IRC | 09:56 | |
*** jraim has quit IRC | 09:56 | |
*** markmc has quit IRC | 09:56 | |
*** noelbk has quit IRC | 09:56 | |
*** Alexei_987 has quit IRC | 09:56 | |
*** pblaho has quit IRC | 09:56 | |
*** amotoki_ has quit IRC | 09:56 | |
*** zhiyan has quit IRC | 09:56 | |
*** mrda-away has quit IRC | 09:56 | |
*** dmitryme has quit IRC | 09:56 | |
*** hartsocks has quit IRC | 09:56 | |
*** wendar has quit IRC | 09:56 | |
*** jroll has quit IRC | 09:56 | |
*** AAzza_afk has quit IRC | 09:56 | |
*** sileht has quit IRC | 09:56 | |
*** toabctl has quit IRC | 09:56 | |
*** dhellmann has quit IRC | 09:56 | |
*** JoshNang has quit IRC | 09:56 | |
*** jecarey has quit IRC | 09:56 | |
*** bogdando has quit IRC | 09:56 | |
*** tsufiev has quit IRC | 09:56 | |
*** rpodolyaka has quit IRC | 09:56 | |
*** jd__ has quit IRC | 09:56 | |
*** SlickNik has quit IRC | 09:56 | |
*** Kiall has quit IRC | 09:56 | |
*** harlowja_away has quit IRC | 09:56 | |
*** dhellmann has joined #openstack-oslo | 10:03 | |
*** JoshNang has joined #openstack-oslo | 10:03 | |
*** toabctl has joined #openstack-oslo | 10:03 | |
*** sileht has joined #openstack-oslo | 10:03 | |
*** AAzza_afk has joined #openstack-oslo | 10:03 | |
*** jroll has joined #openstack-oslo | 10:03 | |
*** wendar has joined #openstack-oslo | 10:03 | |
*** hartsocks has joined #openstack-oslo | 10:03 | |
*** dmitryme has joined #openstack-oslo | 10:03 | |
*** mrda-away has joined #openstack-oslo | 10:03 | |
*** zhiyan has joined #openstack-oslo | 10:03 | |
*** amotoki_ has joined #openstack-oslo | 10:03 | |
*** pblaho has joined #openstack-oslo | 10:03 | |
*** noelbk has joined #openstack-oslo | 10:03 | |
*** markmc has joined #openstack-oslo | 10:03 | |
*** Alexei_987 has joined #openstack-oslo | 10:03 | |
*** oomichi has joined #openstack-oslo | 10:03 | |
*** abhishek___ has joined #openstack-oslo | 10:03 | |
*** i159 has joined #openstack-oslo | 10:03 | |
*** mkoderer has joined #openstack-oslo | 10:03 | |
*** tkelsey_ has joined #openstack-oslo | 10:03 | |
*** ildikov has joined #openstack-oslo | 10:03 | |
*** k4n0 has joined #openstack-oslo | 10:03 | |
*** dstanek has joined #openstack-oslo | 10:03 | |
*** yamahata__ has joined #openstack-oslo | 10:03 | |
*** anteaya has joined #openstack-oslo | 10:03 | |
*** hyakuhei has joined #openstack-oslo | 10:03 | |
*** bnemec has joined #openstack-oslo | 10:03 | |
*** ujjain has joined #openstack-oslo | 10:03 | |
*** jraim has joined #openstack-oslo | 10:03 | |
*** russellb has joined #openstack-oslo | 10:03 | |
*** creiht has joined #openstack-oslo | 10:03 | |
*** flaper87|afk has joined #openstack-oslo | 10:03 | |
*** therve has joined #openstack-oslo | 10:03 | |
*** mgagne has joined #openstack-oslo | 10:03 | |
*** ttx has joined #openstack-oslo | 10:03 | |
*** GheRivero has joined #openstack-oslo | 10:03 | |
*** gpocentek has joined #openstack-oslo | 10:03 | |
*** ruhe has joined #openstack-oslo | 10:03 | |
*** sreshetnyak has joined #openstack-oslo | 10:03 | |
*** SergeyLukjanov has joined #openstack-oslo | 10:03 | |
*** krotscheck has joined #openstack-oslo | 10:03 | |
*** jogo has joined #openstack-oslo | 10:03 | |
*** jokke_ has joined #openstack-oslo | 10:03 | |
*** tcammann_ has joined #openstack-oslo | 10:03 | |
*** mtreinish has joined #openstack-oslo | 10:03 | |
*** gmurphy has joined #openstack-oslo | 10:03 | |
*** ekarlso has joined #openstack-oslo | 10:03 | |
*** tsufiev_ has joined #openstack-oslo | 10:03 | |
*** dteselkin has joined #openstack-oslo | 10:03 | |
*** HenryG has joined #openstack-oslo | 10:03 | |
*** boris-42 has joined #openstack-oslo | 10:03 | |
*** morganfainberg has joined #openstack-oslo | 10:03 | |
*** viktors has joined #openstack-oslo | 10:03 | |
*** ajo has joined #openstack-oslo | 10:03 | |
*** liusheng has joined #openstack-oslo | 10:03 | |
*** bknudson has joined #openstack-oslo | 10:03 | |
*** openstackgerrit has joined #openstack-oslo | 10:03 | |
*** SridharG has joined #openstack-oslo | 10:03 | |
*** Kiall has joined #openstack-oslo | 10:03 | |
*** jd__ has joined #openstack-oslo | 10:03 | |
*** SlickNik has joined #openstack-oslo | 10:03 | |
*** harlowja_away has joined #openstack-oslo | 10:04 | |
*** oomichi has quit IRC | 10:04 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: cfgfilter has graduated, remove it https://review.openstack.org/106278 | 10:05 |
*** jecarey has joined #openstack-oslo | 10:08 | |
*** bogdando has joined #openstack-oslo | 10:08 | |
*** tsufiev has joined #openstack-oslo | 10:08 | |
*** rpodolyaka has joined #openstack-oslo | 10:08 | |
*** dstanek is now known as dstanek_zzz | 10:12 | |
*** ihrachyshka has joined #openstack-oslo | 10:23 | |
*** dims__ has joined #openstack-oslo | 10:29 | |
openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/oslo-incubator: Fix safe_encode func in strutils https://review.openstack.org/106986 | 10:30 |
*** dims__ has quit IRC | 10:34 | |
*** pcm_ has joined #openstack-oslo | 10:39 | |
*** pcm_ has quit IRC | 10:40 | |
*** pcm_ has joined #openstack-oslo | 10:40 | |
*** flaper87|afk is now known as flaper87 | 10:48 | |
*** mkoderer has quit IRC | 10:52 | |
openstackgerrit | Roman Vasilets proposed a change to openstack/oslo.db: I have extract debug logging output to a separate function for better readability https://review.openstack.org/106992 | 10:52 |
openstackgerrit | Nejc Saje proposed a change to openstack/oslo.messaging: Fixes incorrect exchange lock in fake driver https://review.openstack.org/106993 | 10:55 |
*** dstanek_zzz is now known as dstanek | 11:03 | |
*** abhishek___ has quit IRC | 11:07 | |
*** yamahata has joined #openstack-oslo | 11:08 | |
*** dstanek is now known as dstanek_zzz | 11:12 | |
*** tsufiev_ has quit IRC | 11:13 | |
*** dims__ has joined #openstack-oslo | 11:14 | |
*** k4n0 has quit IRC | 11:15 | |
openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/oslo-incubator: Fix safe_encode func in strutils https://review.openstack.org/106986 | 11:16 |
*** openstackgerrit has quit IRC | 11:21 | |
*** flaper87 is now known as flaper87|afk | 11:30 | |
*** flaper87|afk is now known as flaper87 | 11:32 | |
*** alexpilotti has joined #openstack-oslo | 11:51 | |
*** AAzza_afk has quit IRC | 11:54 | |
*** flaper87 is now known as flaper87|afk | 11:55 | |
*** alexpilotti has quit IRC | 11:55 | |
*** yamahata has quit IRC | 11:58 | |
*** yamahata has joined #openstack-oslo | 12:00 | |
dhellmann | dims__: do you think oslo.utils is likely to have a release before j-2 next thurs, or should I move the blueprint to j-3? | 12:00 |
dhellmann | harlowja_away: can you make sure all of your blueprints have links to the relevant patches? I think you've started a lot more of the work than it appears from the tracking pages. | 12:02 |
dims__ | dhellmann, if we can do one round of reviews of existing reviews, we'll know better. i think we have covered all items needed for a release already | 12:02 |
dhellmann | dims__: cool, could you make sure all of the patches needed are linked from the bp? | 12:02 |
dims__ | ah will do | 12:02 |
*** dstanek_zzz is now known as dstanek | 12:04 | |
*** dstanek is now known as dstanek_zzz | 12:13 | |
dims__ | dhellmann, https://blueprints.launchpad.net/oslo/+spec/graduate-oslo-utils is up-to-date. i've added links in the commit messages to the BP as well | 12:14 |
dims__ | brb | 12:14 |
*** openstackgerrit has joined #openstack-oslo | 12:17 | |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add an option to run rootwrap as a daemon https://review.openstack.org/81798 | 12:18 |
openstackgerrit | Nejc Saje proposed a change to openstack/oslo.messaging: Fixes incorrect exchange lock in fake driver https://review.openstack.org/106993 | 12:21 |
dhellmann | dims__: thanks! | 12:23 |
*** tongli has joined #openstack-oslo | 12:25 | |
dhellmann | markmc: the oslo.messaging blueprints that were listed for j-2 appear to be gone from launchpad :-/ there were only a couple, but I didn't keep a separate list of their names, do you know them? | 12:27 |
markmc | dhellmann, huh, weird | 12:28 |
dhellmann | yeah | 12:28 |
markmc | I'm not sure there were any? | 12:28 |
dhellmann | there were 1-2, I thought | 12:29 |
markmc | https://blueprints.launchpad.net/oslo.messaging/juno | 12:29 |
dhellmann | maybe that was j1, though | 12:29 |
dhellmann | I really thought there was at least 1 | 12:29 |
markmc | https://blueprints.launchpad.net/oslo.messaging/+spec/notification-dispatcher-filter would have been plausible, but it has stalled | 12:30 |
markmc | sileht, ^^^ | 12:30 |
dhellmann | I wonder if that's it. If the auto-cleaner took them out of the milestone because they weren't approved | 12:30 |
dhellmann | markmc: while I have you, is https://blueprints.launchpad.net/oslo/+spec/oslo-config-cfgfilter done or are there reviews to finish? not much linked from the bp itself. | 12:32 |
markmc | dhellmann, hmm, auto-linking must not have worked ... one sec | 12:33 |
dhellmann | markmc: and I don't see any open reviews | 12:33 |
markmc | dhellmann, bah, went looking at the wrong one ... sorry | 12:39 |
markmc | dhellmann, added a bunch of stuff to https://blueprints.launchpad.net/oslo/+spec/oslo-config-generator | 12:39 |
* markmc looks at cfgfilter now | 12:39 | |
dhellmann | markmc: the reviews only matter if they're still open, unless you just want to be complete | 12:40 |
markmc | dhellmann, I know, I didn't look at the blueprint you linked before digging up links :) | 12:40 |
dhellmann | markmc: ah, ok :-) | 12:40 |
dhellmann | I thought you were digging up closed reviews for both | 12:40 |
markmc | ok, done now | 12:41 |
dhellmann | markmc: thanks! | 12:42 |
markmc | np :) | 12:42 |
dhellmann | time for breakfast... | 12:42 |
markmc | dhellmann, take a glance at the links at the bottom of https://blueprints.launchpad.net/oslo/+spec/oslo-config-generator | 12:42 |
markmc | if you've got a second | 12:42 |
markmc | generator related stuff I hacked up for nova | 12:43 |
markmc | might make sense in oslo.config | 12:43 |
markmc | still WIP | 12:43 |
dhellmann | as part of the discovery logic for the sample generator? | 12:43 |
markmc | one part for making it easier to implement list_opts() discovery method | 12:44 |
*** dims__ has quit IRC | 12:44 | |
markmc | another part to allow testing the generator and option listing | 12:44 |
*** dims__ has joined #openstack-oslo | 12:45 | |
dhellmann | the testing sounds useful; I'm on the fence about the discovery, although I see the usefulness | 12:45 |
*** mriedem has joined #openstack-oslo | 12:48 | |
*** jecarey_ has joined #openstack-oslo | 12:49 | |
Alexei_987 | markmc: Hi are you available? | 12:52 |
openstackgerrit | Alexei Kornienko proposed a change to openstack/oslo.messaging: Moved main loop to server class https://review.openstack.org/107038 | 12:55 |
*** dstanek_zzz is now known as dstanek | 12:58 | |
*** pblaho has quit IRC | 12:59 | |
*** pblaho has joined #openstack-oslo | 13:03 | |
*** jecarey_ has quit IRC | 13:06 | |
*** jecarey has quit IRC | 13:06 | |
*** jecarey has joined #openstack-oslo | 13:07 | |
*** mkoderer has joined #openstack-oslo | 13:07 | |
*** stannie has joined #openstack-oslo | 13:12 | |
*** dstanek is now known as dstanek_zzz | 13:14 | |
*** dims__ has quit IRC | 13:16 | |
*** zzzeek has joined #openstack-oslo | 13:20 | |
*** dims__ has joined #openstack-oslo | 13:22 | |
*** dstanek_zzz is now known as dstanek | 13:24 | |
*** bknudson has quit IRC | 13:27 | |
openstackgerrit | Alexei Kornienko proposed a change to openstack/oslo.messaging: Moved main loop to server class https://review.openstack.org/107038 | 13:31 |
*** openstackgerrit has quit IRC | 13:31 | |
*** 17SAAF480 has joined #openstack-oslo | 13:32 | |
*** lbragstad has joined #openstack-oslo | 13:33 | |
17SAAF480 | Oleksii Chuprykov proposed a change to openstack/oslo.db: Enable skipped tests from test_models.py https://review.openstack.org/94361 | 13:36 |
*** 17SAAF480 has quit IRC | 13:38 | |
*** openstackgerrit has joined #openstack-oslo | 13:40 | |
*** pblaho has quit IRC | 13:42 | |
*** pblaho has joined #openstack-oslo | 13:43 | |
*** bknudson has joined #openstack-oslo | 13:46 | |
*** lbragstad has quit IRC | 13:47 | |
*** zzzeek has quit IRC | 13:48 | |
*** zzzeek has joined #openstack-oslo | 13:48 | |
*** lbragstad has joined #openstack-oslo | 13:50 | |
*** rpodolyaka has quit IRC | 14:02 | |
*** rpodolyaka has joined #openstack-oslo | 14:02 | |
*** lbragstad has quit IRC | 14:08 | |
openstackgerrit | Oleksii Chuprykov proposed a change to openstack/oslo.db: Enable skipped tests from test_models.py https://review.openstack.org/94361 | 14:09 |
*** lbragstad has joined #openstack-oslo | 14:11 | |
openstackgerrit | A change was merged to openstack/oslo.messaging: Fixes incorrect exchange lock in fake driver https://review.openstack.org/106993 | 14:16 |
*** dims__ has quit IRC | 14:19 | |
openstackgerrit | Oleksii Chuprykov proposed a change to openstack/oslo.db: Move all db exception to exception.py https://review.openstack.org/106782 | 14:22 |
*** tsekiyama has joined #openstack-oslo | 14:25 | |
*** abhishek_k_ has joined #openstack-oslo | 14:27 | |
abhishek_k_ | dhellmann: hi, you around?? | 14:30 |
dhellmann | abhishek_k_: what's up? | 14:31 |
abhishek_k_ | hi, could you please have look at this oslo-spec, https://review.openstack.org/#/c/106285/ | 14:31 |
abhishek_k_ | dhellmann: thank you | 14:32 |
dhellmann | abhishek_k_: I'll put it on my review list | 14:33 |
jogo | dhellmann: I have a question too | 14:33 |
jogo | dhellmann: re: https://review.openstack.org/#/c/94001/ Ability to customize default_log_levels for each project | 14:33 |
abhishek_k_ | dhellmann:thank you | 14:34 |
jogo | I want to use it in nova. what is the recommended place to call set_defaults | 14:34 |
dhellmann | jogo: usually before you parse the options | 14:34 |
dhellmann | jogo: but I think it works ok if you at least call it before the options are used (so before you configure logging) | 14:35 |
openstackgerrit | Tom Cammann proposed a change to openstack/oslo.config: Check config default value is correct type https://review.openstack.org/105450 | 14:35 |
jogo | dhellmann: ah I think I see where to do it then, thanks. | 14:36 |
markmc | dhellmann, it'd be in nova.config.parse_args() - there are some other similar set_defaults() calls there | 14:38 |
dhellmann | jogo: ^^ | 14:38 |
dhellmann | markmc: thanks | 14:38 |
markmc | bah, sorry - wrong nick | 14:38 |
dhellmann | markmc: np | 14:38 |
jogo | markmc: right, thats where I am looking | 14:38 |
jogo | although I don't want to be forced to set ogging_context_format_string | 14:39 |
dhellmann | jogo: we should update set_defaults to allow that to be None to ignore changing the default | 14:42 |
jogo | dhellmann: I'll push a patch up shortly | 14:43 |
dhellmann | jogo: ok, ping me here when you do, I'm doing reviews today | 14:43 |
*** dims__ has joined #openstack-oslo | 14:53 | |
jogo | dhellmann: hmm there is a test to set the default contexty format string to none | 14:59 |
jogo | so that is a valid use case? | 15:00 |
*** dims__ has quit IRC | 15:04 | |
dhellmann | jogo: I can't imagine why that would be useful | 15:04 |
*** pblaho has quit IRC | 15:04 | |
dhellmann | jogo: unless setting it to none disables something in the context formatter | 15:04 |
jogo | yeah, I will make sure to keep supporting that use case | 15:04 |
jogo | just to be safe | 15:04 |
dhellmann | jogo: it looks like if that string is None we might see exceptions trying to log | 15:05 |
dhellmann | dims: do you know about setting logging_context_format_string to None? ^^ | 15:06 |
zhiyan | viktors: hi | 15:10 |
viktors | zhiyan: hello! | 15:10 |
zhiyan | i'm re-view https://review.openstack.org/#/c/106818/3/glance/db/sqlalchemy/api.py , can you tell me why we need a lock there? | 15:11 |
zhiyan | viktors: ^ | 15:11 |
jogo | dhellmann: looks like dims added it, so yeah. will push a patch up that will break that shortly | 15:12 |
viktors | zhiyan: this prevents us for race condition - consider the case when we are trying to call a _create_facade_lazily() method when the facade hasn't been initialized yet 2 times simultaneously - 2 facade instances will be created in this case. | 15:13 |
viktors | zhiyan: but it should probably be harmless :) | 15:14 |
zhiyan | viktors: thanks | 15:14 |
*** yamahata has quit IRC | 15:15 | |
zhiyan | viktors: i see, the common purpose is straightforward | 15:15 |
zhiyan | viktors: i'm just thinking if it can be really triggered, but yes | 15:16 |
openstackgerrit | Joe Gordon proposed a change to openstack/oslo-incubator: Make logging_context_format_string optional in log.set_defaults https://review.openstack.org/107082 | 15:16 |
jogo | dhellmann: ^. and its 5pm here in Germany so o/ | 15:16 |
jogo | not sure if the assumption about breaking things is correct yet though | 15:16 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add an option to run rootwrap as a daemon https://review.openstack.org/81798 | 15:16 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Refactor common parts from cmd to wrapper https://review.openstack.org/107083 | 15:16 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add basic benchmark https://review.openstack.org/107084 | 15:16 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add daemon mode to benchmark https://review.openstack.org/107085 | 15:16 |
dhellmann | jogo: added to my review list; enjoy your evening :-) | 15:17 |
viktors | zhiyan: yes, I agree, that it's maybe paranoia at most :) | 15:17 |
viktors | zhiyan: by the way, as for that patch - it there any tool in Glance to update config sample? | 15:18 |
zhiyan | viktors: and if we want to make it really work, we need a double-checked locking mechanism | 15:18 |
zhiyan | imho | 15:18 |
zhiyan | viktors: i have one change there, under review, will be merged soon | 15:19 |
zhiyan | viktors: and by then, yes, we need to get db configs from olso.db directly instead of glance codebase. i will do that. | 15:20 |
viktors | zhiyan: I'm not sure, that it's necessary to use double-checked locking mechanism, but feel free to comment patch :) | 15:21 |
*** openstackgerrit has quit IRC | 15:21 | |
viktors | zhiyan: as for configs - I looked fro something like ``generate-config.sh`` toll, but got no success | 15:22 |
*** abhishek_k_ has quit IRC | 15:22 | |
*** nacim has quit IRC | 15:23 | |
zhiyan | viktors: ify, for sh way: https://review.openstack.org/#/c/83327/ , but i prefer to use new style (and support separated config files) https://review.openstack.org/#/c/83327/ | 15:23 |
*** abhishek_k has joined #openstack-oslo | 15:24 | |
zhiyan | oh, https://review.openstack.org/#/c/104476/ | 15:25 |
viktors | zhiyan: the first approach looks more familiar to me =) | 15:31 |
*** ihrachyshka has quit IRC | 15:37 | |
*** mkoderer has quit IRC | 15:42 | |
*** i159 has quit IRC | 15:43 | |
*** viktors is now known as viktors|afk | 15:46 | |
*** tongli has quit IRC | 15:58 | |
*** tongli has joined #openstack-oslo | 15:58 | |
zhiyan | viktors|afk: just commented MP #106818 | 16:00 |
*** lbragstad has quit IRC | 16:02 | |
*** ildikov has quit IRC | 16:05 | |
*** lbragstad has joined #openstack-oslo | 16:06 | |
*** lbragstad has quit IRC | 16:19 | |
*** dims__ has joined #openstack-oslo | 16:21 | |
*** Alexei_987 has quit IRC | 16:30 | |
*** pblaho has joined #openstack-oslo | 16:31 | |
*** pblaho has quit IRC | 16:37 | |
*** jaosorior has joined #openstack-oslo | 16:39 | |
*** pcm__ has joined #openstack-oslo | 16:40 | |
*** pcm__ has quit IRC | 16:40 | |
*** pcm__ has joined #openstack-oslo | 16:40 | |
*** pcm_ has quit IRC | 16:43 | |
dims__ | dhellmann, the suggestion in https://review.openstack.org/#/c/106531/12/nova/api/openstack/__init__.py,unified breaks tests. Can you please help with it? | 16:47 |
dims__ | i've marked it as WIP till we figure this out | 16:48 |
*** ildikov has joined #openstack-oslo | 16:50 | |
*** krotscheck has quit IRC | 16:54 | |
*** krotscheck has joined #openstack-oslo | 16:54 | |
dhellmann | dims__: sure, let me set up that branch and see what fails | 16:55 |
*** krotscheck has quit IRC | 17:00 | |
*** krotscheck has joined #openstack-oslo | 17:00 | |
dims__ | thanks dhellmann!! | 17:03 |
dhellmann | dims__: having some trouble with dependencies, I think my dev box is borked, so it may take a little while | 17:04 |
*** krotscheck has quit IRC | 17:05 | |
*** krotscheck has joined #openstack-oslo | 17:06 | |
*** harlowja_away is now known as harlowja | 17:08 | |
*** tmcpeak has joined #openstack-oslo | 17:08 | |
*** lbragstad has joined #openstack-oslo | 17:10 | |
*** lbragstad has quit IRC | 17:10 | |
*** lbragstad has joined #openstack-oslo | 17:11 | |
*** lbragstad has quit IRC | 17:13 | |
*** lbragstad has joined #openstack-oslo | 17:14 | |
*** ihrachyshka has joined #openstack-oslo | 17:18 | |
*** Alexei_987 has joined #openstack-oslo | 17:19 | |
*** tkelsey_ has quit IRC | 17:32 | |
dhellmann | bnemec: where's the oslo.serialization repo on github? | 17:39 |
* bnemec facepalms | 17:40 | |
bnemec | dhellmann: https://github.com/cybertron/oslo.serialization | 17:40 |
bnemec | I'll reply to that e-mail with the link. | 17:41 |
dhellmann | bnemec: yeah, the guy with bnemec on github didn't really look like you :-) | 17:47 |
bnemec | Ooh, that reminds me. I need to go request that freenode give me my nick. | 17:48 |
bnemec | The guy who owns it now hasn't used it in well over a year. | 17:48 |
dhellmann | bnemec: LGTM, with one comment for a change to test-requirements.txt later (I don't know if github emails you those or not, so I linked it in a reply on the ML) | 17:54 |
bnemec | dhellmann: Hmm, I wonder why I thought that was necessary. I know I had an error at one point about mock not being found. I'll add a note to the bp to fix that later. | 17:58 |
dhellmann | bnemec: yeah, it's not a big deal | 18:05 |
harlowja | dhellmann what tracking page are u referring to (for linking patches -> blueprints), is it something else than https://blueprints.launchpad.net/oslo? | 18:08 |
harlowja | ^ in reference to a previous message btw | 18:08 |
dhellmann | harlowja: normally an automated thingy updates each individual blueprint with the relevant changes, but that only works if the bp is mentioned in the commit message or the gerrit topic (and possibly only on the first version of a patch) | 18:09 |
dhellmann | harlowja: it would help me help you keep track of when a bp is done if all of the reviews were linked in the whiteboard, so I could look at them and know when they've all merged (or in the case of a new bp, when the work has started vs. not) | 18:10 |
harlowja | sure, some of the current reviews i've got going are likely not connected, although i suppose i should connect some of them to something (minus the ones that are mainly just small changes, cleanup) | 18:11 |
harlowja | retargeting some of the commit messages with blueprints (for ones i can find) | 18:15 |
-openstackstatus- NOTICE: python2.6 jobs are failing due to bug 1342262 "virtualenv>=1.9.1 not found" A fix is out but there are still nodes built on the old stale images | 18:22 | |
*** ChanServ changes topic to "python2.6 jobs are failing due to bug 1342262 "virtualenv>=1.9.1 not found" A fix is out but there are still nodes built on the old stale images" | 18:22 | |
*** ihrachyshka has quit IRC | 18:24 | |
dhellmann | harlowja: ok, this is mostly cleanup so don't add artificial relationships. if any bps have code in flight, make sure there's a link, otherwise it's fine | 18:33 |
harlowja | dhellmann will do :) | 18:33 |
dhellmann | harlowja: thanks | 18:34 |
harlowja | np | 18:34 |
*** SridharG has quit IRC | 19:02 | |
*** lbragstad has quit IRC | 19:08 | |
*** alexpilotti has joined #openstack-oslo | 19:12 | |
*** pblaho has joined #openstack-oslo | 19:16 | |
*** alexpilotti has quit IRC | 19:20 | |
tmcpeak | jogo: got time for a hacking question? | 19:38 |
*** pblaho has quit IRC | 19:39 | |
*** openstackgerrit has joined #openstack-oslo | 19:44 | |
*** abhishek_k has quit IRC | 19:47 | |
*** jaosorior has quit IRC | 19:52 | |
dhellmann | dims__: I get some test failures for libvirt tests, but those fail on master, too, so I think that's just my box being broken. Do you have a log of the failed tests from the i18n changes? | 19:55 |
*** markmcclain has joined #openstack-oslo | 20:00 | |
dims__ | dhellmann, let me recreate it, can you paste me the __init__.py that you would like me to try? | 20:00 |
dhellmann | dims__: oh, wait, I ran the tests with your code but not mine, let me update the code as suggested and see what happens | 20:02 |
dhellmann | dims__: sorry, long day :-/ | 20:02 |
dims__ | :) np | 20:02 |
*** markmcclain1 has joined #openstack-oslo | 20:04 | |
*** markmcclain has quit IRC | 20:04 | |
dims__ | dhellmann, shall i just add "_" to all methods except try_import? (https://review.openstack.org/#/c/106778/9/oslo/utils/importutils.py,unified) | 20:13 |
tmcpeak | anybody here involved with the hacking project? | 20:14 |
tmcpeak | I may have found a bug | 20:14 |
dims__ | tmcpeak, what do you see? | 20:14 |
tmcpeak | dims_: I run a test using "physical_line" | 20:15 |
tmcpeak | and then when I run yield(0, message) | 20:15 |
tmcpeak | it says | 20:15 |
tmcpeak | ne 1256, in check_physical | 20:15 |
tmcpeak | offset, text = result | 20:15 |
tmcpeak | ValueError: need more than 0 values to unpack | 20:15 |
tmcpeak | dims_: same works fine with "logical_line" instead | 20:16 |
dims__ | tmcpeak, trying to wriate a new hacking rule? | 20:20 |
tmcpeak | dims_: yes | 20:20 |
tmcpeak | dims_: I need to use physical_line because the stripping out strings with xxx is messing me up | 20:21 |
*** arnaud__ has joined #openstack-oslo | 20:26 | |
tmcpeak | dims_: it actually probably has nothing to do with hacking | 20:26 |
tmcpeak | it's probably a pep8 bug | 20:26 |
dims__ | tmcpeak did you try? yield 0, "message" | 20:27 |
tmcpeak | the call is yield(0, "message") | 20:28 |
dims__ | you are right i see both usages in our hacking module | 20:29 |
tmcpeak | dims_: yeah either way | 20:30 |
tmcpeak | dims_: for some reason running yield as normal in this function with physical_line is causing an error | 20:31 |
*** markmcclain1 has quit IRC | 20:33 | |
dims__ | tmcpeak, did you try using the tokens? | 20:33 |
*** markmcclain has joined #openstack-oslo | 20:33 | |
tmcpeak | dims_: no, how does that work? | 20:33 |
dims__ | see hacking_no_locals | 20:33 |
*** ihrachyshka has joined #openstack-oslo | 20:34 | |
dims__ | it has a yield too | 20:34 |
tmcpeak | dims_: cool, thanks! | 20:35 |
tmcpeak | dims_: yeah, including logical_line in the function fixes this issue for some reason :D | 20:36 |
dims__ | whew nice | 20:37 |
*** praneshp has joined #openstack-oslo | 20:47 | |
*** tongli has quit IRC | 20:57 | |
*** markmcclain has quit IRC | 21:06 | |
*** russellb has quit IRC | 21:07 | |
*** creiht has quit IRC | 21:07 | |
*** flaper87|afk has quit IRC | 21:07 | |
*** creiht has joined #openstack-oslo | 21:07 | |
*** russellb has joined #openstack-oslo | 21:07 | |
*** flaper87|afk has joined #openstack-oslo | 21:07 | |
*** flaper87|afk is now known as flaper87 | 21:07 | |
*** lbragstad has joined #openstack-oslo | 21:09 | |
*** pcm__ has quit IRC | 21:22 | |
*** lbragstad has quit IRC | 21:26 | |
*** arnaud__ has quit IRC | 21:34 | |
*** markmc has quit IRC | 21:34 | |
*** arnaud__ has joined #openstack-oslo | 21:39 | |
*** mrda-away is now known as mrda | 21:46 | |
dhellmann | dims__: yeah, a _ prefix would mark them as private | 21:47 |
*** dims__ has quit IRC | 21:48 | |
dhellmann | dims__: I'll look at those nova tests in ht morning; I'm signing off for the evening | 21:48 |
*** arnaud__ has quit IRC | 21:52 | |
*** arnaud__ has joined #openstack-oslo | 22:09 | |
*** lbragstad has joined #openstack-oslo | 22:11 | |
*** stannie has quit IRC | 22:12 | |
*** lbragstad has quit IRC | 22:13 | |
*** lbragstad has joined #openstack-oslo | 22:16 | |
*** lbragstad has quit IRC | 22:24 | |
noelbk | Alexei_987: I found my other problem: exchanges were created without queues attached, so messages were just dropped | 22:26 |
noelbk | https://bugs.launchpad.net/oslo.messaging/+bug/1338732 | 22:26 |
noelbk | I'm testing a patched oslo.messaging, then adding tests. How do I run the unit tests? | 22:27 |
Alexei_987 | noelbk: tox -e py27 | 22:27 |
Alexei_987 | noelbk: "The fix is to declare the queue and exchange in the producer as well as the consumer." Not sure that this is a good fix | 22:28 |
Alexei_987 | noelbk: IMHO we should have responsibilities strictly separated | 22:28 |
Alexei_987 | noelbk: and it's a consumer problem that we are losing this message | 22:29 |
*** dstanek is now known as dstanek_zzz | 22:29 | |
noelbk | Alexei_987: in a world with faults, you can never guarantee when rabbit may die between sending and receiving a message. In this case, the consumer is waiting to reconnect when the producer wants to publish a reply | 22:30 |
noelbk | the producer can't assume the consumer has set up the queues for it | 22:31 |
Alexei_987 | noelbk: why you cannot assume that? | 22:32 |
Alexei_987 | noelbk: if you have an incoming rpc call you can assume that someone is waiting for reply | 22:32 |
*** yamahata has joined #openstack-oslo | 22:35 | |
noelbk | 1. caller posts call, 2. receiver gets rpc, 3. rabbit dies and restarts, queues are now gone. 4. receiver posts back to rabbi, but queues are gone, so drops message, 4. receiver reconnects, no message to receive | 22:38 |
noelbk | Alexei_987: ^ | 22:38 |
Alexei_987 | noelbk: yeah but it's still a consumer problem imho | 22:39 |
Alexei_987 | noelbk: for durable query it won't be a problem | 22:39 |
Alexei_987 | noelbk: so making query durable is imho a better fix than splitting this responsibility across producer and consumer | 22:40 |
*** novas0x2a|laptop has joined #openstack-oslo | 22:40 | |
Alexei_987 | noelbk: and you can delete it when client object is destroyed | 22:41 |
noelbk | Alexei_987: The durable thing may solve it sometime... depends on how the queues are backed up. Rabbit may start on another node with the durable stuff gone | 22:41 |
Alexei_987 | you have ha mode for queries | 22:41 |
Alexei_987 | so in this case it's rabbits problem | 22:41 |
noelbk | The current code may fail with durable queues too, since the producer declares a new exchange but doesn't explicitly bind it to any queue | 22:42 |
Alexei_987 | noelbk: that's what I'm talking about we should clearly define responsibilities for each peace of code | 22:42 |
noelbk | I'm not sure how rabbit persists durable exchanges...? | 22:42 |
Alexei_987 | noelbk: producer should not declare exchanges it's not responsible for | 22:42 |
Alexei_987 | noelbk: should be the same way as with queues | 22:43 |
noelbk | I don't see how the consumer can solve the problem. Remember, it waits for 1 sec before reconnecting, leaving plenty of time when rabbit is up but the exchange/queue does not exist to save the producer's response | 22:45 |
*** dims__ has joined #openstack-oslo | 22:45 | |
noelbk | Bot sides have to assume the world gets reset and reconstruct it | 22:45 |
noelbk | And in rabbit, declaring queues and exhcanges is idempotent, no? (as long as the params are the same) | 22:46 |
*** ihrachyshka has quit IRC | 22:47 | |
noelbk | I'd love to get rid of (a) rabbit, (b) the 1 sec delay on reconnect, (c) the bugs in rabbit ha, but until then, I'm not seeing a beeter solution to the current problem | 22:47 |
*** dims__ has quit IRC | 22:49 | |
*** dstanek_zzz is now known as dstanek | 22:50 | |
*** ihrachyshka has joined #openstack-oslo | 22:57 | |
*** tsekiyam_ has joined #openstack-oslo | 23:05 | |
*** tsekiyama has quit IRC | 23:08 | |
*** tsekiyam_ has quit IRC | 23:09 | |
*** harlowja is now known as harlowja_away | 23:16 | |
*** ihrachyshka has quit IRC | 23:23 | |
*** bknudson has quit IRC | 23:24 | |
*** tmcpeak has left #openstack-oslo | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!