Monday, 2014-06-23

« Sunday, 2014-06-22 Index Tuesday, 2014-06-24 »
*** yamahata has quit IRC00:03
*** morganfainberg_L has quit IRC00:39
*** arnaud has quit IRC00:51
*** amotoki has joined #openstack-oslo00:59
*** flashgordon is now known as jogo01:01
*** morganfainberg_L has joined #openstack-oslo01:14
*** yamahata has joined #openstack-oslo01:17
*** mestery has joined #openstack-oslo01:28
openstackgerritKevin McCarthy proposed a change to openstack-dev/pbr: Restore Monkeypatched Distribution Instance  https://review.openstack.org/10178501:32
*** yamahata has quit IRC01:58
*** yamahata has joined #openstack-oslo01:58
*** haypo has quit IRC02:15
*** HenryG has joined #openstack-oslo02:21
*** haypo has joined #openstack-oslo02:24
*** haypo has joined #openstack-oslo02:24
*** zzzeek has joined #openstack-oslo02:36
*** arnaud has joined #openstack-oslo02:48
*** lbragstad has quit IRC02:54
*** lbragstad has joined #openstack-oslo02:55
*** zhiyan_ is now known as zhiyan03:04
*** arnaud has quit IRC03:06
*** arnaud has joined #openstack-oslo03:11
*** zzzeek has quit IRC03:34
*** arnaud has quit IRC03:35
*** zzzeek has joined #openstack-oslo03:51
*** arnaud has joined #openstack-oslo04:02
*** zzzeek has quit IRC04:17
*** praneshp_ has joined #openstack-oslo04:23
*** praneshp has quit IRC04:24
*** praneshp_ is now known as praneshp04:24
*** arnaud has quit IRC04:24
*** arnaud has joined #openstack-oslo04:26
*** SridharG has joined #openstack-oslo04:26
*** ildikov has quit IRC05:10
*** morganfainberg_L has quit IRC05:24
*** ajc_ has joined #openstack-oslo05:29
*** ildikov has joined #openstack-oslo05:35
*** arnaud has quit IRC06:04
*** arnaud has joined #openstack-oslo06:08
*** pblaho has joined #openstack-oslo06:16
*** openstackstatus has quit IRC06:27
*** SridharG has quit IRC06:30
*** pblaho is now known as pblaho|afk06:32
*** SridharG has joined #openstack-oslo06:36
*** rdopieralski has joined #openstack-oslo06:52
*** pblaho|afk is now known as pblaho06:53
*** viktors|afk is now known as viktors06:59
*** praneshp has quit IRC07:08
*** praneshp has joined #openstack-oslo07:09
openstackgerritMehdi Abaakouk proposed a change to openstack/oslo.messaging: Ensures listener queues exist in fake driver  https://review.openstack.org/10088907:10
*** praneshp has quit IRC07:21
viktorsbnemec: around?07:28
*** ihrachyshka has joined #openstack-oslo07:29
*** sileht has quit IRC07:35
*** nacim has joined #openstack-oslo07:52
openstackgerritgaryk proposed a change to openstack/oslo.vmware: Tanslations: make use of _LE and _LW  https://review.openstack.org/10182107:57
*** i159 has joined #openstack-oslo08:01
*** ildikov has quit IRC08:03
*** sileht has joined #openstack-oslo08:04
*** mrda is now known as mrda-away08:12
*** arnaud has quit IRC08:19
*** oomichi has quit IRC08:19
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests  https://review.openstack.org/9342408:34
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Implementation Alembic as migration engine  https://review.openstack.org/9996508:34
*** sileht has quit IRC08:38
*** sileht has joined #openstack-oslo08:41
*** markmc has joined #openstack-oslo08:53
*** SridharG has quit IRC08:53
*** ildikov_ has joined #openstack-oslo08:55
*** tkelsey has joined #openstack-oslo08:58
*** tkelsey_ has joined #openstack-oslo08:59
*** ihrachyshka is now known as ihrachyshka|food08:59
*** ujjain has quit IRC09:01
*** ildikov_ has quit IRC09:04
*** ildikov has joined #openstack-oslo09:08
*** ihrachyshka|food has quit IRC09:10
openstackgerritYuriy Taraday proposed a change to openstack/oslo-specs: Add rootwrap-daemon-mode blueprint  https://review.openstack.org/9461309:23
openstackgerritYuriy Taraday proposed a change to openstack/oslo-specs: Add rootwrap-daemon-mode blueprint  https://review.openstack.org/9461309:28
silehtCan you take a look to https://review.openstack.org/#/c/100889/, I have rebased it, and fix the markmc comment09:30
silehtjd__, ^09:30
*** jaosorior has joined #openstack-oslo09:31
*** pblaho is now known as pblaho|afk09:45
*** pblaho|afk is now known as pblaho09:48
*** SridharG has joined #openstack-oslo09:53
*** ihrachyshka has joined #openstack-oslo09:55
*** pblaho is now known as pblaho|afk10:09
*** pblaho|afk is now known as pblaho10:13
*** pblaho is now known as pblaho|afk10:28
*** zhiyan is now known as zhiyan_10:32
jd__"the markmc comment", that sounds like a good title for a movie10:32
*** ildikov has quit IRC11:00
*** yamahata has quit IRC11:13
*** lbragstad has quit IRC11:40
*** pblaho|afk is now known as pblaho11:41
*** pblaho has quit IRC11:50
*** ildikov has joined #openstack-oslo11:54
*** SridharG has quit IRC11:57
openstackgerritChangBo Guo(gcb) proposed a change to openstack/oslo.db: Avoid usage of mutables as default args  https://review.openstack.org/10189312:24
*** erecio has joined #openstack-oslo12:24
openstackgerritChangBo Guo(gcb) proposed a change to openstack/oslo.rootwrap: Avoid usage of mutables as default args  https://review.openstack.org/10189812:30
*** gordc has joined #openstack-oslo12:40
*** ajc_ has quit IRC12:40
*** yamahata has joined #openstack-oslo12:45
*** bknudson has joined #openstack-oslo13:02
openstackgerritAbhishek Kekane proposed a change to openstack/oslo-incubator: Add default log level for websocket  https://review.openstack.org/10190813:10
jaosorioranybody knows what the status of oslo.version is?13:11
jaosoriorIs it recommended for usage?13:11
*** mriedem has joined #openstack-oslo13:13
*** pblaho has joined #openstack-oslo13:14
*** jecarey has quit IRC13:16
*** lbragstad has joined #openstack-oslo13:18
*** tkelsey has quit IRC13:22
*** tkelsey_ has quit IRC13:22
*** markmc has quit IRC13:23
*** SridharG has joined #openstack-oslo13:29
*** dims has joined #openstack-oslo13:31
viktorsbnemec: hi13:33
*** zzzeek has joined #openstack-oslo13:37
bnemecviktors: Hey13:40
*** ildikov has quit IRC13:40
viktorsbnemec: I wanted to ask you review a single patch, but I found some issues there for now13:43
viktorsbnemec: so it's just "hello!"13:44
viktors:)13:44
bnemecviktors: Sounds good.  And hello! :-)13:45
*** ildikov_ has joined #openstack-oslo13:49
*** rustlebee is now known as russellb13:50
*** gordc has quit IRC13:50
thervedims, Hey, around for https://review.openstack.org/100457 ?13:53
dimstherve, looking13:56
dimsah i just left a comment in that13:57
therveYes :)13:57
thervedims, It's not the other way because of testing issues13:57
therveThe first patch did that and failed to run13:57
thervePresumably it will be a very short lived fix anyway13:57
dimstherve, i see. patch seemed counter to the request in the bug report - "The publish_errors option will try to use oslo notifier, but we should use oslo.messaging instead if available."13:58
* dhellmann starts releasing libraries...13:58
thervedims, They can't live next to each other anyway, so I don't think that matters13:58
dimstherve, commit message needs to be fixed at least to say that we try openstack.common.log_handler.PublishErrorsHandler first13:59
dimstherve, i got it now. let's fix the commit message and i'll +2 it13:59
therveI think it says that, although maye not that clearly?13:59
therveI changed it already fwiw :)13:59
jd__unleash the libs, dhellmann!13:59
dhellmannjd__: we need some t-shirts for the oslo team with "Unleash the libs!" for the summit14:01
jd__hehe14:02
ihrachyshkatherve: they can live together, you just don't import them at the same time :)14:05
therveihrachyshka, That's a very limited definitino of live :)14:05
dimstherve, would this work? "Try PublishErrorsHandler in oslo notifier first. If it does not exist fall back14:05
dimsto the one in oslo.messaging." what we have in the commit message is a bit unclear14:05
*** yamahata has quit IRC14:05
thervedims, Well we already try the notifier first...14:05
*** yamahata has joined #openstack-oslo14:06
dimstherve, the words that trips me over is "instead of the one". if you don't want to touch the commit message, that's fine. i'll +2 as-is14:07
dimsjust let me know14:07
therveNo I'd be happy to fix it14:08
thervedims, "If log_handler is not available, fallback to oslo.messaging PublishErrorsHandler14:10
therveto not require oslo notifier." ?14:10
dimssounds good14:10
openstackgerritThomas Herve proposed a change to openstack/oslo-incubator: Use oslo.messaging to publish log errors  https://review.openstack.org/10045714:11
dimsthanks therve14:12
thervedims, Thank you!14:12
*** morganfainberg_L has joined #openstack-oslo14:18
bknudsondhellmann: could you add --no-merges when listing commits in the release?14:18
dhellmannbknudson: yeah, I'll do that next time14:18
dhellmannbknudson: pbr does that, but we have a bug in pbr where it looks at dates of the original commit, but not the merge, so it shows some changes being in a version before they were actually released, so I was trying to be cautious this time around and get *everything*14:22
*** morganfainberg_L is now known as mdrnstm14:23
*** mdrnstm has quit IRC14:23
*** mdrnstm has joined #openstack-oslo14:23
*** tsekiyama has joined #openstack-oslo14:34
*** morganfainberg has quit IRC14:45
*** morganfainberg_Z has joined #openstack-oslo14:45
*** mdrnstm is now known as morganfainberg14:45
*** jecarey has joined #openstack-oslo14:51
*** markmc has joined #openstack-oslo15:03
*** ihrachyshka has quit IRC15:04
openstackgerritDoug Hellmann proposed a change to openstack/oslotest: Add API documentation  https://review.openstack.org/9886115:13
openstackgerritDoug Hellmann proposed a change to openstack/oslotest: Add API documentation  https://review.openstack.org/9886115:18
openstackgerritDoug Hellmann proposed a change to openstack/oslotest: Add API documentation  https://review.openstack.org/9886115:19
*** nacim has quit IRC15:21
*** ildikov_ has quit IRC15:32
*** rdopieralski has quit IRC15:37
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests  https://review.openstack.org/9342415:38
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Implementation Alembic as migration engine  https://review.openstack.org/9996515:38
*** pblaho is now known as pblaho|afk15:47
*** ildikov has joined #openstack-oslo15:53
openstackgerritIlya Pekelny proposed a change to openstack/oslo.db: Add a base test case for DB schema comparison  https://review.openstack.org/9339815:56
*** pblaho|afk is now known as pblaho15:59
*** viktors is now known as viktors|afk16:01
*** jaosorior has quit IRC16:12
*** bknudson has quit IRC16:14
*** praneshp has joined #openstack-oslo16:15
*** dstanek_zzz is now known as dstanek16:24
mriedemi'd like to move cpu_count() from nova into oslo-incubator, but not sure the best place to put it16:24
mriedemhttps://review.openstack.org/#/c/69266/1/nova/utils.py16:24
mriedemi plan on using that in cinder/glance for the same reason as nova created it16:25
mriedemthinking service.py?16:26
*** arnaud__ has joined #openstack-oslo16:29
mriedembut glance doesn't copy the service module from oslo, hmm16:31
dhellmannmriedem: seems a little small for a stand-alone module, but I could see it ending up in oslo.concurrency eventually https://review.openstack.org/#/c/97296/3/specs/juno/graduate-oslo-concurrency.rst16:35
dhellmannservice won't go there, so I'm not sure service is the best place for the function16:35
mriedemdhellmann: cool, i was thinking processutils initially, but that's more CLI stuff rather than threads/processes, so didn't seem to fit as much16:36
dhellmannyeah16:37
mriedemwas also thinking lockutils16:37
mriedemalthough cpu_count is not related to locking16:37
dhellmannmriedem: are there any other related functions in nova that could be combined into a cpuutils module?16:37
mriedemdhellmann: i can look16:38
dhellmannmriedem: the other question I have is, when is that function not implemented in multiprocessing? because maybe that's no longer the case?16:39
mriedemdhellmann: the python docs say it may raise that16:40
*** arnaud__ has quit IRC16:40
mriedemlooks like python 3.4 has it in the os module https://docs.python.org/3/library/os.html#os.cpu_count16:43
dhellmannmriedem: the implementation is using NotImplementedError as the exception any time it sees an error trying to determine the number16:43
dhellmannmriedem: I thought maybe it was an operating system support issue, but it looks like all errors are converted to that exception16:44
* bnemec chuckles while triaging https://bugs.launchpad.net/oslo/+bug/133333716:45
dhellmannmriedem: maybe osutils is the right name? or sysutils?16:46
dhellmannbnemec: heh16:46
mriedemdhellmann: i'm open to either, osutils would be more specific16:47
mriedemdhellmann: also thinking about what oslo would return for NotImplementedError - nova is returning 1 because cpu_count() is being used to set the number of workers, which used to default to 116:47
dhellmannmriedem: I'm not sure the number of CPUs is an operating system choice :-)16:47
mriedeman oslo consumer might use that for something else, so maybe returning None would be more appropriate to signal 'i don't know'16:48
mriedemdhellmann: good point :)16:48
mriedemderp16:48
mriedemi'm fine with sysutils if you are16:48
dhellmannmriedem: it seems like we want a get_number_of_workers() that uses cpu count but defaults to 1, instead of calling the function cpu_count16:48
*** dstanek is now known as dstanek_zzz16:48
bnemecIsn't returning None still going to require wrapper logic in the consumers?16:48
mriedembnemec: yeah, which sucks16:49
mriedemyou could pass in something...16:49
dhellmannand that appears to be the whole point of this function16:49
mriedem'use this if you can't figure it out'16:49
mriedemright16:49
dhellmannI think it's just poorly named16:49
dhellmannif the intent is to get a useful number of workers, then the name should indicate that somehow16:49
bnemecA configurable default could work.16:50
dhellmannget_parallelization_factor() or get_worker_count() or something16:50
bnemecdef cpu_count(default=1):16:50
mriedemi'm also ok with just renaming when moving to oslo and passing a default kwarg16:50
dhellmannbnemec: when would anything ever pass a different value?16:50
bnemecThen if someone wants different behavior they can make that happen.16:50
bnemecdhellmann: I dunno, 1 seems like a reasonable default to me, but if someone wanted to do something different they could default to None and do the wrapper logic.16:52
*** dstanek_zzz is now known as dstanek16:52
mriedemin nova if the option isn't explictly set it just uses cpu_count(), else 1 if NotImplementedError16:53
mriedemglance today defaults to 116:53
dhellmannbnemec: that's true, but I'd rather we create APIs that are well-defined and what you're describing seems like a different function than this one.16:53
mriedemi'd do the same in glance probably, check if the option is set and if not, call cpu_count() else default to 116:53
dhellmannmriedem: so this function is only used to set the default for the number of workers?16:53
mriedemdhellmann: set the number of workers if not explicitly specified in the config16:54
dhellmannmriedem: right16:54
dhellmannand where is that configuration option defined?16:54
mriedemthe only other integrated projects i'm aware of that have a workers value like this is cinder and ceilometer, and they also default to 116:54
mriedemin each project, not oslo16:54
mriedemnova has multiple workers, glance has one for the api worker, cinder has one for the volume api worker16:55
mriedemi think ceilometer has one16:55
bnemecdhellmann: The only thing is that a default of 1 makes it impossible to tell whether that's the actual cpu count or the "couldn't get the cpu count" value.  None requires the wrapper logic which defeats the purpose of the function.16:55
mriedemlooks like trove has an api worker and conductor worker16:55
dhellmannbnemec: right, so the question is, is this function really about getting an accurate CPU count, or is it about getting the default for the number of workers?16:56
bnemecBut I don't have a specific case in mind where someone might want to do something different in the "can't get cpu count" case, so maybe it's not worth worrying about.16:56
dhellmannyeah16:57
bnemecdhellmann: Well, number of workers is pretty service-specific so I feel like that would belong in service.py again.16:58
mriedemso call it get_worker_count() and return 1 if NotImplementedError? is default=1 kwarg even useful?16:58
bnemecIf it's in oslo.concurrency then I think cpu count is the appropriate thing.16:58
dhellmannbnemec: that makes sense, so I think we should put it in with the service code16:59
dhellmannmriedem: I don't think the default argument is useful, but I'm willing to be proven wrong :-)16:59
bnemecThe default argument could always be added later too.17:00
mriedemme neither really since everyone defaults to 117:00
mriedemsure17:00
mriedemi need to dig into trove's usage for api since they default to None17:00
bnemecno args -> 1 defaulted arg isn't going to be an api-breaking change.17:00
mriedemnot sure wtf that means17:00
mriedemgonna eat lunch and then push something up, need to remember to give credit to russellb for the method and tests17:01
openstackgerritChris Krelle proposed a change to openstack/oslo-incubator: Add option to generate_sample.sh to use tox.ini excludes  https://review.openstack.org/7876517:02
*** i159 has quit IRC17:08
*** harlowja_away is now known as harlowja17:08
openstackgerritDoug Hellmann proposed a change to openstack/oslo.i18n: Improve initial documentation  https://review.openstack.org/9696117:17
*** arnaud__ has joined #openstack-oslo17:36
*** ildikov has quit IRC17:42
*** harlowja has quit IRC18:00
mriedembnemec: looks like the difference between 1 workers and None is the difference between a ProcessLauncher and ServiceLauncher in service.py18:02
mriedemonly seeing that because trove_api_workers defaults to None18:02
mriedemso it'd get a ServiceLauncher18:02
openstackgerritpritesh proposed a change to openstack-dev/hacking: Add hacking check for vim configuration in source files  https://review.openstack.org/10196918:06
*** bknudson has joined #openstack-oslo18:06
*** ihrachyshka has joined #openstack-oslo18:08
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H904  https://review.openstack.org/10170118:16
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H803, it's mostly an old flamewar  https://review.openstack.org/10149818:16
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H305 because it's python version specific  https://review.openstack.org/10149918:16
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H402  https://review.openstack.org/10149718:16
*** markmcclain has joined #openstack-oslo18:20
*** ihrachyshka has quit IRC18:21
bnemecmriedem: Oh, because ServiceLauncher doesn't support multiple workers?18:28
bnemecAt least if I'm reading the code right.18:28
mriedembnemec: that's what it looks like18:29
mriedembnemec: i'm asking if the trove guys made a conscious decision there though18:29
mriedemworkers=1 means ProcessLauncher with a child service18:29
mriedemwokers=None means ServiceLauncher by itself18:29
mriedemso any big gains? do you avoid any kind of overhead from ProcessLauncher?18:30
bnemecmriedem: ServiceLauncher supports the restart on sighup functionality, but that's about the extent of my knowledge on these things.18:31
bnemecOh, wait, ProcessLauncher does too.18:32
bnemecSo I have no idea why you would use one over the other. :-)18:32
mriedemi'm in no rush to change this code in trove, but was just thinking ahead18:34
mriedemi've leave the default get_worker_count() to return 1 for now18:35
mriedemand we can override later if needed as previously discussed18:35
*** harlowja has joined #openstack-oslo18:38
*** harlowja has quit IRC18:38
*** harlowja has joined #openstack-oslo18:39
*** harlowja has quit IRC18:39
bnemecmriedem: Yeah, that works for me.18:39
bnemecdhellmann: Any objections? ^18:39
*** ildikov has joined #openstack-oslo18:42
dhellmannbnemec, mriedem : that makes sense18:46
SlickNikmriedem: I don't think that there's was any specific method to choosing those defaults in trove.18:50
SlickNikmriedem: I'd be totally fine with changing that if needed — to align with how other OS projects are doing / going to do it18:51
*** harlowja has joined #openstack-oslo18:51
*** harlowja has quit IRC18:51
mriedemSlickNik: they are the same, None and 118:51
mriedemjust sorted that out myself18:51
mriedemapparently my eyes weren't working before food18:52
SlickNikah, just saw your link from the other window. :)18:52
SlickNikThanks for the clarification18:52
*** harlowja has joined #openstack-oslo18:52
*** harlowja has quit IRC18:52
*** harlowja has joined #openstack-oslo18:53
*** mrda-away is now known as mrda18:56
*** dims_ has joined #openstack-oslo18:57
*** dims has quit IRC18:58
openstackgerritMatt Riedemann proposed a change to openstack/oslo-incubator: Move nova.utils.cpu_count() to service.py module  https://review.openstack.org/10197818:58
mriedembnemec: dhellmann: ^18:59
dhellmannmriedem: thanks, I'll put that in my review queue18:59
mriedemthanks18:59
*** krotscheck has quit IRC19:10
*** hyakuhei1_ has quit IRC19:10
*** krotscheck has joined #openstack-oslo19:27
*** hyakuhei__ has joined #openstack-oslo19:27
openstackgerritMatt Riedemann proposed a change to openstack/oslo-incubator: Move nova.utils.cpu_count() to service.py module  https://review.openstack.org/10197819:35
*** erecio has quit IRC19:37
*** erecio has joined #openstack-oslo19:38
zzzeekdhellmann: do i understand correctly that a test suite, like the one in nova, currently can run tests against SQLite as well as one additional DB (PG or MySQL for example), at most?  e.g. sqlite always loads up and then whatever is set under nova.openstack.common.db.options.connection ?19:50
openstackgerritMatt Riedemann proposed a change to openstack/oslo-incubator: Move nova.utils.cpu_count() to service.py module  https://review.openstack.org/10197819:51
openstackgerritBen Nemec proposed a change to openstack/oslo-incubator: Remove unnecessary setUp functions  https://review.openstack.org/10199019:54
*** ihrachyshka has joined #openstack-oslo19:58
openstackgerritBen Nemec proposed a change to openstack/oslo-incubator: Remove unnecessary setUp function  https://review.openstack.org/10199019:59
mriedembnemec: ha20:03
mriedemi started something20:03
bnemecmriedem: Heh, I was trying to figure out how the dep could be outdated when I pushed it five seconds earlier. :-)20:04
*** markmcclain has quit IRC20:06
*** bknudson has quit IRC20:10
*** mriedem has quit IRC20:10
*** jecarey has quit IRC20:10
*** jecarey has joined #openstack-oslo20:10
*** bknudson has joined #openstack-oslo20:11
*** mriedem has joined #openstack-oslo20:11
*** markmcclain has joined #openstack-oslo20:13
dhellmannzzzeek: that sounds like it might be right, but I don't know enough about how those tests work to say for certain20:17
*** jecarey has quit IRC20:36
*** pblaho has quit IRC20:44
*** jecarey has joined #openstack-oslo20:51
openstackgerritMax Lobur proposed a change to openstack/oslo.db: Add __contains__ to ModelBase to fully behave like a dict  https://review.openstack.org/10200820:53
*** dims__ has joined #openstack-oslo20:54
*** dims__ has quit IRC20:55
*** dims__ has joined #openstack-oslo20:56
*** dims_ has quit IRC20:57
*** ajo|mac has joined #openstack-oslo20:59
*** markmcclain has quit IRC20:59
*** erecio has quit IRC21:00
*** markmcclain has joined #openstack-oslo21:01
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H904  https://review.openstack.org/10170121:02
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H803, it's mostly an old flamewar  https://review.openstack.org/10149821:02
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H305 because it's python version specific  https://review.openstack.org/10149921:02
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H402  https://review.openstack.org/10149721:02
openstackgerritSean Dague proposed a change to openstack-dev/hacking: add hacking preamble  https://review.openstack.org/10201421:02
*** SridharG has quit IRC21:03
*** ihrachyshka has quit IRC21:39
*** ihrachyshka has joined #openstack-oslo21:39
openstackgerritgordon chung proposed a change to openstack/oslo-incubator: mark middlware module as graduating  https://review.openstack.org/10202221:41
*** lbragstad has quit IRC21:53
*** markmc has quit IRC21:54
*** mriedem has left #openstack-oslo22:01
openstackgerritDoug Hellmann proposed a change to openstack/oslo.i18n: Add links to policy discussions  https://review.openstack.org/10202822:03
*** dstanek is now known as dstanek_zzz22:07
*** markmcclain has quit IRC22:08
*** bknudson has quit IRC22:10
*** dstanek_zzz is now known as dstanek22:16
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H904  https://review.openstack.org/10170122:18
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H803, it's mostly an old flamewar  https://review.openstack.org/10149822:18
openstackgerritSean Dague proposed a change to openstack-dev/hacking: add hacking preamble  https://review.openstack.org/10201422:18
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H305 because it's python version specific  https://review.openstack.org/10149922:18
openstackgerritSean Dague proposed a change to openstack-dev/hacking: remove H402  https://review.openstack.org/10149722:18
bnemecSigh.  I just noticed a minor typo in the preamble change after he just pushed a new version to address my previous nit. :-(22:23
*** ajo|mac has quit IRC22:46
openstackgerritpritesh proposed a change to openstack-dev/hacking: Add hacking check for vim configuration in source files  https://review.openstack.org/10196922:48
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Improve WBE testing coverage  https://review.openstack.org/10171722:48
*** ihrachyshka has quit IRC22:48
*** dstanek is now known as dstanek_zzz22:50
*** ajo|mac has joined #openstack-oslo22:53
openstackgerritMichael H Wilson proposed a change to openstack/oslo.messaging: Add first unit test for 0MQ  https://review.openstack.org/9390522:56
openstackgerritMichael H Wilson proposed a change to openstack/oslo.messaging: WIP Transport URL test case  https://review.openstack.org/10204722:56
*** ajo|mac has quit IRC23:06
*** ajo|mac has joined #openstack-oslo23:07
*** markmcclain has joined #openstack-oslo23:12
*** morganfainberg_L has joined #openstack-oslo23:12
*** morganfainberg has quit IRC23:12
*** morganfainberg_Z is now known as morganfainberg23:12
*** markmcclain1 has joined #openstack-oslo23:13
harlowjadhellmann u get anytime to look over https://review.openstack.org/#/c/100048/ yet?23:13
*** markmcclain has quit IRC23:17
*** tsekiyam_ has joined #openstack-oslo23:17
*** tsekiyama has quit IRC23:21
*** tsekiyam_ has quit IRC23:21
*** morganfainberg_L is now known as mdrnstm23:21
*** mdrnstm has quit IRC23:22
*** mdrnstm has joined #openstack-oslo23:22
*** morganfainberg has quit IRC23:22
*** mdrnstm is now known as morganfainberg23:22
*** morganfainberg_Z has joined #openstack-oslo23:22
*** ajo|mac has quit IRC23:35
*** ajo|mac_ has joined #openstack-oslo23:35
*** oomichi has joined #openstack-oslo23:42
*** arnaud__ has quit IRC23:50
« Sunday, 2014-06-22 Index Tuesday, 2014-06-24 »

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