Tuesday, 2016-02-23

openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.messaging: Avoid hardcoding the notification topic and specify driver  https://review.openstack.org/28214500:07
*** thumpba has quit IRC00:08
*** takedakn has joined #openstack-oslo00:08
*** takedakn has quit IRC00:08
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.messaging: Get kafka notifications to work with kafka-python 0.9.5  https://review.openstack.org/28207900:10
*** takedakn has joined #openstack-oslo00:15
*** jaypipes has quit IRC00:20
*** salv-orlando has quit IRC00:45
*** takedakn has quit IRC00:49
*** edmondsw has joined #openstack-oslo00:56
*** thumpba has joined #openstack-oslo00:59
*** mtanino has quit IRC01:02
*** EinstCrazy has joined #openstack-oslo01:16
*** jecarey has joined #openstack-oslo01:28
*** jecarey has quit IRC01:33
*** EinstCrazy has quit IRC01:40
*** EinstCrazy has joined #openstack-oslo01:42
*** zz_dimtruck is now known as dimtruck01:43
*** yamamoto has quit IRC01:44
openstackgerritMerged openstack/tooz: Fix calling acquire(blocking=False) twice leads to a deadlock  https://review.openstack.org/27800101:48
*** sputnik13 has quit IRC01:54
*** claudiub|3 has quit IRC01:58
*** edmondsw has quit IRC01:59
*** browne has quit IRC02:05
*** dims has quit IRC02:13
*** salv-orlando has joined #openstack-oslo02:22
*** yamamoto has joined #openstack-oslo02:25
*** salv-orlando has quit IRC02:30
*** achanda has joined #openstack-oslo02:41
openstackgerritMerged openstack/osprofiler: Remove flake8 ignore list in tox.ini  https://review.openstack.org/28228003:17
*** harlowja_at_home has quit IRC03:18
*** yamahata has quit IRC03:34
*** salv-orlando has joined #openstack-oslo03:34
*** browne has joined #openstack-oslo03:39
*** dims has joined #openstack-oslo03:49
*** salv-orlando has quit IRC03:50
*** links has joined #openstack-oslo04:03
*** salv-orlando has joined #openstack-oslo04:26
*** salv-orlando has quit IRC04:32
*** dimtruck is now known as zz_dimtruck04:36
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.utils: Add method check_string_length  https://review.openstack.org/26733105:21
openstackgerritEric Brown proposed openstack/oslo.service: Use choices on the ssl.version option  https://review.openstack.org/28338705:22
openstackgerritEric Brown proposed openstack/oslo.service: Use choices on the ssl.version option  https://review.openstack.org/28338705:24
*** jecarey has joined #openstack-oslo05:29
openstackgerritMerged openstack/oslo.config: add a release note mentioning our use of reno  https://review.openstack.org/28241605:29
*** browne has quit IRC05:30
*** thumpba has quit IRC05:31
*** jecarey has quit IRC05:33
*** deray has joined #openstack-oslo05:36
openstackgerritEric Brown proposed openstack/oslo.service: Use choices on the ssl.version option  https://review.openstack.org/28338705:42
*** dims has quit IRC05:54
openstackgerritChaozhe Chen(ccz) proposed openstack/oslo.log: Clean pep8 ignore  https://review.openstack.org/28339105:55
openstackgerritChaozhe Chen(ccz) proposed openstack/oslo.db: Clean flake8 ignore  https://review.openstack.org/28339205:55
openstackgerritChaozhe Chen(ccz) proposed openstack/oslo.service: Clean flake8 ignore  https://review.openstack.org/28339305:55
openstackgerritEric Brown proposed openstack/oslo.messaging: Use PortOpt on kafka_default_port  https://review.openstack.org/28340106:13
openstackgerritEric Brown proposed openstack/oslo.messaging: Typos of 'recieve' instead of 'receive'  https://review.openstack.org/28340306:16
*** amotoki has joined #openstack-oslo06:19
*** nkrinner has joined #openstack-oslo06:21
openstackgerritJoshua Harlow proposed openstack/taskflow: Remove some of the silly import renames (task_atom, retry_atom)  https://review.openstack.org/28215806:31
openstackgerritEric Brown proposed openstack/oslo.service: Use choices on the ssl.version option  https://review.openstack.org/28338707:21
*** yamahata has joined #openstack-oslo07:21
*** salv-orl_ has joined #openstack-oslo07:41
*** salv-orl_ has quit IRC07:41
*** achanda has quit IRC07:49
*** boris-42 has quit IRC07:54
*** pcaruana has joined #openstack-oslo07:58
*** pcaruana is now known as pcaruana|afk|08:08
*** pcaruana|afk| is now known as pcaruana08:08
*** subscope has joined #openstack-oslo08:32
*** subscope has quit IRC08:36
*** subscope has joined #openstack-oslo08:47
*** dtardivel has joined #openstack-oslo08:58
*** yassine__ has joined #openstack-oslo09:19
*** subscope has quit IRC09:20
*** achanda has joined #openstack-oslo09:22
*** lucas-dinner is now known as lucasagomes09:27
*** achanda has quit IRC09:27
*** jecarey has joined #openstack-oslo09:29
*** jecarey has quit IRC09:34
openstackgerritAlexis Lee proposed openstack/oslo.utils: Add method check_string_length  https://review.openstack.org/26733109:34
*** subscope has joined #openstack-oslo09:43
*** salv-orlando has joined #openstack-oslo09:43
openstackgerritAlexis Lee proposed openstack/oslo.messaging: Typos of 'recieve' instead of 'receive'  https://review.openstack.org/28340309:53
*** subscope has quit IRC09:57
*** subscope has joined #openstack-oslo10:02
*** StanLagun has quit IRC10:12
*** cdent has joined #openstack-oslo10:14
*** EinstCrazy has quit IRC10:15
openstackgerritMerged openstack/oslo.service: Fix typo in help text  https://review.openstack.org/28329210:15
*** cdent has quit IRC10:38
*** itisha has quit IRC10:39
openstackgerritYuriy Nesenenko proposed openstack/oslo.reports: Use file modification events instead of signal handler  https://review.openstack.org/26097610:43
*** sdague has joined #openstack-oslo10:51
*** yassine__ has quit IRC11:02
*** subscope has quit IRC11:21
*** cdent has joined #openstack-oslo11:23
*** achanda has joined #openstack-oslo11:25
*** achanda has quit IRC11:31
*** jaypipes has joined #openstack-oslo11:32
openstackgerritMerged openstack/oslo.config: add support for mutable options in the config generator  https://review.openstack.org/28243511:39
*** yamamoto has quit IRC11:40
*** salv-orlando has quit IRC11:41
*** EinstCrazy has joined #openstack-oslo11:52
*** subscope has joined #openstack-oslo11:58
*** ihrachys has joined #openstack-oslo11:59
*** salv-orlando has joined #openstack-oslo12:15
*** lucasagomes is now known as lucas-hungry12:16
openstackgerritChaozhe Chen(ccz) proposed openstack/oslo.i18n: Clean flake8 ignore  https://review.openstack.org/28354312:19
*** yassine__ has joined #openstack-oslo12:31
*** yassine__ has quit IRC12:31
*** yassine__ has joined #openstack-oslo12:31
*** gordc has joined #openstack-oslo12:34
*** links has quit IRC12:38
*** yamamoto has joined #openstack-oslo12:39
*** fesp has joined #openstack-oslo12:49
*** fesp has quit IRC12:51
*** fesp has joined #openstack-oslo12:53
*** yamamoto has quit IRC12:58
*** subscope has quit IRC13:05
*** fesp has quit IRC13:05
*** jeckersb is now known as jeckersb_gone13:15
*** jeckersb_gone is now known as jeckersb13:16
*** yamamoto has joined #openstack-oslo13:20
*** subscope has joined #openstack-oslo13:22
*** links has joined #openstack-oslo13:23
*** jaypipes has quit IRC13:24
*** kgiusti has joined #openstack-oslo13:24
*** lucas-hungry is now known as lucasagomes13:26
*** achanda has joined #openstack-oslo13:28
*** subscope has quit IRC13:32
*** achanda has quit IRC13:33
openstackgerritMarc Koderer proposed openstack/stevedore: Add a reference to entry_point_inspector  https://review.openstack.org/27729913:33
*** pradk has joined #openstack-oslo13:38
*** pradk has quit IRC13:38
*** salv-orlando has quit IRC13:41
*** edmondsw has joined #openstack-oslo13:45
*** dkehn has quit IRC13:45
*** subscope has joined #openstack-oslo13:45
*** pradk has joined #openstack-oslo13:50
*** subscope has quit IRC13:55
*** dkehn has joined #openstack-oslo13:57
*** salv-orlando has joined #openstack-oslo14:01
*** rlrossit has joined #openstack-oslo14:02
*** pcaruana has quit IRC14:02
*** subscope has joined #openstack-oslo14:04
sdagueanyone around for oslo.db questions ?14:13
sdaguebecause I've hit an issue with enginefascade blocking some testing we need14:13
openstackgerritDoug Hellmann proposed openstack/oslo-specs: add a policy documenting how we manage release notes  https://review.openstack.org/28327014:16
*** rlrossit has quit IRC14:25
*** rlrossit_ has joined #openstack-oslo14:29
*** links has quit IRC14:30
*** mriedem_away has quit IRC14:30
*** mriedem has joined #openstack-oslo14:30
*** links has joined #openstack-oslo14:30
*** regXboi has joined #openstack-oslo14:30
*** achanda has joined #openstack-oslo14:31
*** rlrossit_ has quit IRC14:32
*** rlrossit has joined #openstack-oslo14:32
*** jaypipes has joined #openstack-oslo14:32
*** achanda has quit IRC14:35
*** mriedem is now known as mrieddem14:36
*** mrieddem is now known as mriedem14:36
*** dims has joined #openstack-oslo14:54
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Move server's logic from executors  https://review.openstack.org/28113315:00
*** deray has quit IRC15:01
*** sigmavirus24_awa is now known as sigmavirus2415:01
*** salv-orl_ has joined #openstack-oslo15:05
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.config: Updated from global requirements  https://review.openstack.org/28322615:05
*** ekarlso- has joined #openstack-oslo15:07
*** links has quit IRC15:07
*** ekarlso has quit IRC15:07
*** salv-orlando has quit IRC15:07
*** subscope has quit IRC15:11
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves poller's stop logic  https://review.openstack.org/28362215:12
*** jecarey has joined #openstack-oslo15:13
*** jecarey has quit IRC15:13
*** jecarey has joined #openstack-oslo15:14
dimssileht : dukhlov_ : can you please bless these? https://review.openstack.org/#/c/282079/ https://review.openstack.org/#/c/282145/15:14
dimsdmitryme : dukhlov_ : can you please check on sileht 's review https://review.openstack.org/#/c/280170/15:16
*** salv-orl_ has quit IRC15:17
*** pcaruana has joined #openstack-oslo15:18
silehtdims, done15:18
dimsthanks sileht15:18
openstackgerritChaozhe Chen(ccz) proposed openstack/oslo.middleware: Clean up removed hacking rule from [flake8] ignore lists  https://review.openstack.org/28362615:19
dukhlov_dims, done too15:19
*** pcaruana has quit IRC15:20
*** pcaruana has joined #openstack-oslo15:21
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.messaging: Get kafka notifications to work with kafka-python 0.9.5  https://review.openstack.org/28207915:21
*** pcaruana|afk| has joined #openstack-oslo15:21
dimssileht : fixed https://review.openstack.org/#/c/282079/ dukhlov_ : sileht spotted a cut-n-paste errror15:21
silehtdims, about requirements, is it up2date ?15:22
dimssileht : yep did that first15:22
*** mtanino has joined #openstack-oslo15:22
dimssileht : dukhlov_ : dmitryme : please look through the oslo.messaging open reviews, let's blast through them and make sure there's nothing left to release. we need to wrap up Mitaka work ASAP15:23
silehtdims, oki cool15:23
dimsthanks sileht15:23
openstackgerritMerged openstack/oslo.config: add unreleased page to release notes build  https://review.openstack.org/28241515:24
dukhlov_dims, sileht: how about this one? https://review.openstack.org/#/c/281133/15:24
*** ozamiatin has joined #openstack-oslo15:25
dimssileht : are you comfortable with that one?15:25
ozamiatindims : hi15:25
dimsozamiatin : "<dims>sileht : dukhlov_ : dmitryme : please look through the oslo.messaging open reviews, let's blast through them and make sure there's nothing left to release. we need to wrap up Mitaka work ASAP"15:26
dimslooks like sileht has a +2 already15:27
dukhlov_dims: yes15:27
*** ksngs78 has joined #openstack-oslo15:28
dimsdukhlov_ : +2A15:28
dimsdukhlov_ : did we stress test with that change?15:28
*** pcaruana|afk| has quit IRC15:29
dukhlov_ dukhlov_: with simulator - yes15:29
openstackgerritRoman Podoliaka proposed openstack/oslo.db: Increase the default max_overflow value  https://review.openstack.org/26918615:29
dukhlov_dism:^^^15:29
dukhlov_dims:^^15:29
dukhlov_it seems I'm tired a bit:)15:30
dims:)15:30
dimsozamiatin : one more for your eyes - https://review.openstack.org/#/c/282654/15:31
*** ksngs78 has quit IRC15:32
dimslxsli_web : rbradfor : dhellmann : any oslo.config stuff pending?15:35
dimsah, https://review.openstack.org/#/c/283192/ is left15:35
*** zz_dimtruck is now known as dimtruck15:35
dhellmanndims : +2a15:36
dimsthanks dhellmann15:36
*** pcaruana has quit IRC15:37
openstackgerritMerged openstack/oslotest: Add some gitignore files  https://review.openstack.org/27819515:40
sdaguedims: is there anyone from oslo.db about? Because I'm kind of blocked on this oslo.db issue. And I've got all the context in my head right now. If I task switch away that's going to get lost.15:43
rbradfordims, nothing from me that needs to make the final release15:44
*** yamamoto has quit IRC15:45
dimssdague : best bet is rpodolyaka or pkholkin_ as snikitin is not around15:46
dimssdague : let me try pinging them on other communication channels15:47
rpodolyakasdague: hey! I just noticed the ML thread, looking15:48
rpodolyakaso does not really use the "global" context manager in oslo.db15:48
rpodolyakabut creates its own ones15:48
rpodolyakaso we'll need to tweak nova db fixture a bit15:48
sdaguerpodolyaka: how do I reset it?15:48
rpodolyakasdague: checking right now15:49
*** pcaruana has joined #openstack-oslo15:49
rpodolyakalooks like you have to re-create the context manager with the current API15:49
sdaguebecause I've been reading oslo.db code all morning, and that's very unclear15:49
sdagueok, what would that call look like15:49
sdaguebecause if I call configure again, it's an error15:49
dimslxsli_web : rbradfor : can you please look through the Oslo Review Inbox? (bit.ly/oslo-reviews) to see if there's stuff we need to get in?15:50
rpodolyakasdague: yep, that's the problem15:50
rpodolyakasdague: so you would need to create new instances of these two - https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L134-L13515:50
rpodolyakamaybe add a reset() function to nova/db/sqlalchemy/api?15:51
sdaguerpodolyaka: ok, I can try that15:52
rpodolyakathat's a good catch, though :(15:53
rpodolyakawhen tests pass, but they are testing not what we want...15:53
sdaguerpodolyaka: so would we do reset before or after configure?15:54
rpodolyakasdague: before. you need to configure the new instance before using it15:54
*** nkrinner has quit IRC15:54
sdaguehmmm... before doesn't work15:54
rpodolyakahmm, let me try that...15:55
sdaguerpodolyaka: yeh, I can't come up with a version that works15:57
sdaguerpodolyaka: if you pull 28336415:58
sdagueand run: tox -e functional -- --concurrency=1 test_bug15:58
rpodolyakaack15:58
sdagueit will reproduce the issue15:58
sdaguetest_aggregate_case_sensitivity should fail15:59
sdaguebut if any test runs before it, it passes15:59
sdagueif you can figure out a way to get the session reset correctly, I'd really appreciate it15:59
sdaguebecause I'm having a hard time figuring out what's happening in the oslo.db layers16:00
rpodolyakanp16:00
rpodolyakawe'll try to come up with a patch for nova db fixture now16:00
dhellmanndims , lxsli_web, rbradfor : there's a subtle bug in the mutate changes in oslo.config: The __clear_cache decorator does not clear the cache until *after* the wrapped function is called. That means inside the mutate call where we invoke the hooks, those hooks see the *old* values on the conf object. I'm working on a fix.16:05
dimsrpodolyaka : thanks!16:05
dimsdhellmann : thanks16:05
mriedemlooks like oslo.config 3.8.0 breaks heat unit tests on stable/liberty16:07
mriedemhttp://logs.openstack.org/periodic-stable/periodic-heat-python27-liberty/93ef519/console.html#_2016-02-23_06_16_05_34716:07
dimsmriedem : there you go! https://review.openstack.org/#/c/283659/16:08
openstackgerritDoug Hellmann proposed openstack/oslo.config: clear the cache before mutating the config files  https://review.openstack.org/28366316:11
openstackgerritDoug Hellmann proposed openstack/oslo.log: Allow reload of 'debug' option  https://review.openstack.org/25482116:12
dhellmanndims, lxsli_web, rbradfor : ^^16:12
*** EinstCrazy has quit IRC16:13
*** cdent has quit IRC16:13
*** subscope has joined #openstack-oslo16:14
rbradfordhellmann, looking16:16
*** harlowja_at_home has joined #openstack-oslo16:22
*** yamahata has quit IRC16:25
*** subscope has quit IRC16:27
openstackgerritMerged openstack/oslo.utils: Add method check_string_length  https://review.openstack.org/26733116:27
rbradfordhellmann, I can see the benefit of clearing the cache before processing mutate_files. I do not see how that fixes the log_root.getEffectiveLevel() in  https://review.openstack.org/25482116:28
dhellmannrbradfor : the value of conf.debug is still False when the hook is invoked, so it doesn't reconfigure the logger16:28
dhellmannwith my change, the cache is cleared so the new value of conf.debug is used in the hook16:29
dhellmannwe *could* say that the hooks need to get their values from the fresh dict, but what we want is for apps to write functions that work during initialization as well as the mutate hook, and it would be harder to do that if sometimes the conf object reported stale values16:30
*** cgalan has joined #openstack-oslo16:31
*** claudiub has joined #openstack-oslo16:32
openstackgerritMerged openstack/oslo.service: Correct some help text  https://review.openstack.org/28329516:32
rbradfordhellmann, ok, so the existing tests work now in verifying conf.debug after because the decorater is doing its job, but i see he hook did not adjust level.  I wonder how these tests works locally before?16:32
*** achanda has joined #openstack-oslo16:33
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Support transport URL  https://review.openstack.org/27258316:34
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves logging  https://review.openstack.org/28367416:36
openstackgerritMerged openstack/oslo.config: Add None-check to find_file  https://review.openstack.org/28319216:36
*** dims has quit IRC16:37
*** salv-orlando has joined #openstack-oslo16:37
*** salv-orlando has quit IRC16:37
openstackgerritChaozhe Chen(ccz) proposed openstack/oslo.middleware: Clean up removed hacking rule from [flake8] ignore lists  https://review.openstack.org/28362616:37
*** salv-orlando has joined #openstack-oslo16:37
*** achanda has quit IRC16:38
rpodolyakasdague: shit. I just realized that tweaking the fixture won't help - https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L463-L470 - this has already been evaluated at the module import  time16:39
rpodolyakaso even if I re-create the context managers in nova db fixture, I will already have nova db api methods looking at the old ones16:40
cgalandhellmann, can you take a look at https://review.openstack.org/#/c/275905/ ?16:40
rpodolyakasdague: so looks like the pattern of using decorators needs an explicit reset() method in oslo.db :(16:40
dhellmannrbradfor : the old tests did not look at the value with the cache set from inside the hook. They either didn't pre-populate the cache, or didn't look at the value at the right time in the sequence of steps to notice the issue.16:41
openstackgerritMerged openstack/taskflow: Sqlalchemy-utils double entry (already in test-requirements.txt)  https://review.openstack.org/28295116:41
dhellmanncgalan : let's talk about that in #openstack-release16:41
rbradfordhellmann, ok16:42
cgalandhellmann: ok16:42
*** subscope has joined #openstack-oslo16:45
*** yamamoto has joined #openstack-oslo16:46
lxsli_webdhellmann: thanks for spotting that! I'm working on my oslo.service change right now, ofc a mock can't be set across a process boundary16:47
dhellmannlxsli_web: I wanted to try to get your oslo.log change working, and when I saw the test failing I knew something was wrong outside of oslo.log.16:48
lxsli_webTODO.pop() :)16:48
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves poller's stop logic  https://review.openstack.org/28362216:49
rbradfordhellmann, when you have a moment, I have post gate failure for https://review.openstack.org/#/c/274188/  I've determined why, I'm just not confident a fix is easier, i.e. revert https://review.openstack.org/#/c/274187/ is better choice16:52
*** yamamoto has quit IRC16:53
dhellmannrbradfor : by "post gate failure" do you mean the jobs listed there on the review or something else?16:53
rbradfordhellmann, the jenkins gate for merging failed.16:54
dhellmannok, the "post" queue is something else, so that's why I ask16:55
rbradfordhellmann, it's rather weird because when I run it localling I do not get the error, checking my dependencies locally, but I've tracked down to why the the error can occur16:55
dhellmannit looks like one string has a color escape sequence and the other doesn't16:56
*** rlrossit has quit IRC16:56
dhellmannI'm not sure why that would be related to the patch you suggested reverting?16:56
*** yamahata has joined #openstack-oslo16:57
rbradfordhellmann, so, I moved this logic http://git.openstack.org/cgit/openstack/oslo.log/tree/oslo_log/formatters.py#n234 from oslo.log to oslo.context16:57
dhellmannrbradfor : ^^16:57
rbradforwhat's happening is in the unit tests, attributes are passed to log.record directly, and when in conjunction with a context object, this is subsequently overwritten the context attributes (which are blank)16:58
*** rlrossit has joined #openstack-oslo16:58
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.config: Updated from global requirements  https://review.openstack.org/28322616:59
dhellmannrbradfor : ah, the *color* attribute16:59
rbradforand the *resource* attribute.17:00
rbradfordhellmann, so, one fix, is the formatter does not set record attributes from context if the record has the attribute17:00
dhellmannyeah17:01
*** rlrossit_ has joined #openstack-oslo17:01
rbradfordhellmann, but that's a change from how it was working17:01
openstackgerritMerged openstack/oslo.messaging: Remove duplicate requirements  https://review.openstack.org/28221317:01
rbradforit's a risk, where reverting my other change, and addressing next cycle is lower risk17:01
openstackgerritOpenStack Proposal Bot proposed openstack/taskflow: Updated from global requirements  https://review.openstack.org/28278817:01
dhellmanndo the new attributes in oslo.context cause issues in oslo.log regardless of your new patch? or are they tied together?17:01
sdaguerpodolyaka: is that something that can be done?17:02
sdaguebecause we've got a class of bugs that I really want to test for that we can't right now17:02
rpodolyakasdague: yeah, sure. I'm just trying to figure the best way to do that17:03
rpodolyakaagreed17:03
rpodolyakathere is a thing Mike suggested in the ML thread17:03
*** rlrossit has quit IRC17:03
rpodolyakawe'll need to reapply it for custom context manager instances17:03
rpodolyakaas nova does not use the global one17:03
lxsli_webdhellmann: do you think it's fairly plausible no one ever really tested that reload_config_files is called by SIGHUP?17:05
lxsli_webit seems like it's called on the parent process' CONF not the child's17:07
rbradfordhellmann, these "extra" attributes have a note in oslo.log as a means to catch an exception for "fancier formatting params". My goal here was to strip away the clutter in our primary log format code17:07
rbradforthe alternative is to not set the record attribute from the context, if it exists in the record first, but I'm sure that will cause another edge case17:08
*** dims has joined #openstack-oslo17:09
dhellmannlxsli_web : that seems likely. I'm not sure we've ever really claimed reloading works17:11
lxsli_weble sigh17:11
dhellmannrbradfor : yeah, I'm not sure simplifying that logging code is going to work until we clean up callers17:11
*** browne has joined #openstack-oslo17:12
sdaguerpodolyaka: well if you can propose some example patch that would be good, because I can't translate what mike was saying into what we need.17:12
lxsli_webdhellmann: I have to go out for a couple of hours, might pop back online later. What do you think re: my comment on https://review.openstack.org/#/c/283663 please?17:12
dhellmannlxsli_web : looking17:13
rpodolyakasdague: ok, on it17:13
sdaguerpodolyaka: thanks17:13
rpodolyakanp17:13
dhellmannlxsli_web: this is why i don't like helper functions in tests. They grow all sorts of complexity that makes it harder to understand the side-effects of changing them.17:13
dhellmannlxsli_web : repetition in tests is a feature17:13
lxsli_webdhellmann: until you assume repetition and discover after much debugging they weren't /quite/ the same after all17:14
dhellmannlxsli_web : variation is also a feature17:14
*** cdent has joined #openstack-oslo17:15
lxsli_webso would you rather keep it how you had it?17:15
*** yassine__ has quit IRC17:15
dhellmannlxsli_web : yeah, I pulled the bits of the helper I needed inline on purpose in this case17:15
harlowja_at_homedims, https://review.openstack.org/#/c/283700/17:16
lxsli_webok +217:16
harlowja_at_homeironic folks wanted that one17:16
harlowja_at_homeand it missed yesterdays release17:16
dhellmannlxsli_web : I look forward to continuing this conversation with you over a beer in austin :-)17:16
dimsharlowja_at_home : ack17:16
harlowja_at_homethx!17:16
openstackgerritEric Brown proposed openstack/oslo.messaging: Typos of 'recieve' instead of 'receive'  https://review.openstack.org/28340317:17
lxsli_webdhellmann: sounds good to me, the US have discovered cider at last \o/17:17
openstackgerritEric Brown proposed openstack/oslo.messaging: Typos of 'recieve' instead of 'receive'  https://review.openstack.org/28340317:18
dimslxsli_web : dhellmann : i'll be buying drinks for all of you :)17:19
lxsli_webdims: likewise!17:23
openstackgerritRonald Bradford proposed openstack/oslo.context: Revert add common oslo.log format parameters  https://review.openstack.org/28370717:26
openstackgerritEric Brown proposed openstack/oslo.service: Use choices on the ssl.version option  https://review.openstack.org/28338717:26
openstackgerritRonald Bradford proposed openstack/oslo.log: Use new logging specific method for context info  https://review.openstack.org/27418817:28
openstackgerritRonald Bradford proposed openstack/oslo.context: Revert add common oslo.log format parameters  https://review.openstack.org/28370717:30
*** subscope has quit IRC17:33
*** achanda has joined #openstack-oslo17:35
harlowja_at_homejd__, u on vacation right?17:37
silehtharlowja_at_home, yes he is17:38
harlowja_at_homek17:38
harlowja_at_homethought so17:38
*** achanda has quit IRC17:40
*** lucasagomes is now known as lucas-dinner18:04
*** achanda has joined #openstack-oslo18:07
rpodolyakasdague: ok, look like it works as expected now: http://paste.openstack.org/show/487924/  ; the sensitivity one fails when run on mysql18:08
*** achanda has quit IRC18:10
*** dukhlov_ has quit IRC18:15
*** harlowja_at_home has quit IRC18:16
*** rpodolyaka is now known as rpodolyaka_afk18:25
dhellmanndims : let me know where to be!18:26
*** sputnik13 has joined #openstack-oslo18:31
dhellmannrbradfor : https://review.openstack.org/#/c/283707 doesn't seem to have the marks of a revert generated through gerrit. Did you build it locally?18:33
*** achanda has joined #openstack-oslo18:37
*** ndipanov has quit IRC18:45
*** achanda has quit IRC18:45
openstackgerritDoug Hellmann proposed openstack/oslo-specs: add a policy documenting how we manage release notes  https://review.openstack.org/28327018:48
rbradfordhellmann, I did build it manually.18:50
rbradfordhellmann, are there docs for how to revert via gerrit?18:50
dhellmannrbradfor : there's a button on merged commits18:50
rbradforreally?18:51
*** achanda has joined #openstack-oslo18:51
dhellmannhmm, there used to be18:51
rbradfordhellmann, I see it18:52
openstackgerritMerged openstack/oslo.messaging: Move server's logic from executors  https://review.openstack.org/28113318:52
rbradforso I should abandon said manual verson and do it via gerrit.18:52
dhellmannrbradfor : sure18:52
openstackgerritRonald Bradford proposed openstack/oslo.context: Revert "Add common oslo.log format parameters"  https://review.openstack.org/28374518:53
openstackgerritRonald Bradford proposed openstack/oslo.log: Use new logging specific method for context info  https://review.openstack.org/27418818:54
rbradfordhellmann, that does seem an easier way.18:55
dhellmannrbradfor : ++18:56
*** subscope has joined #openstack-oslo18:56
rbradfordhellmann, it was my first time, who was to know it was easy.18:57
dhellmannrbradfor: the patchset looks the same, but the commit message highlights that it's a revert18:57
*** subscope has quit IRC19:08
openstackgerritMerged openstack/oslo.messaging: [zmq] Support transport URL  https://review.openstack.org/27258319:09
*** ozamiatin has quit IRC19:12
dhellmanndims, rpodolyaka_afk : we should set up reno for oslo.db before merging https://review.openstack.org/#/c/269186/19:13
dhellmannI'll set up the patches19:14
dimsthanks dhellmann19:14
*** ihrachys has quit IRC19:15
*** mtanino has quit IRC19:17
sdaguerpodolyaka_afk: awesome, thanks for that19:19
openstackgerritDoug Hellmann proposed openstack/oslo.db: add reno for release notes management  https://review.openstack.org/28376019:22
sdaguerpodolyaka_afk: I filed - https://bugs.launchpad.net/oslo.db/+bug/1548960 - for eventually fixing this in oslo.db proper19:32
openstackLaunchpad bug 1548960 in oslo.db "oslo.db should provide a way to reset the engine facade for testing" [Undecided,New]19:32
*** salv-orlando has quit IRC19:44
dhellmannrbradfor : could you look at this patch to add reno to oslo.db please? https://review.openstack.org/#/c/283760/19:52
rbradfordhellmann, sure19:52
*** rlrossit_ has quit IRC19:52
openstackgerritMerged openstack/oslo.context: Add roles to context  https://review.openstack.org/27192819:54
rbradfordhellmann, see comments, I'm not sure of the conf.py impact but I'm guessing it's per project for a reason.19:57
openstackgerritDoug Hellmann proposed openstack/oslo.db: add reno for release notes management  https://review.openstack.org/28376020:01
dhellmannrbradfor : done20:01
dhellmannthat'll teach me to blindly copy config files20:01
openstackgerritDoug Hellmann proposed openstack/oslo.db: add reno for release notes management  https://review.openstack.org/28376020:02
rbradfordhellmann, hate to be a pain, I think you missed one heading.20:03
dhellmannrbradfor : we don't generate latex output20:04
dhellmannbut I'll fix that20:04
rbradforok20:04
openstackgerritDoug Hellmann proposed openstack/oslo.db: add reno for release notes management  https://review.openstack.org/28376020:05
rbradforre blindly copying, you did make at least one change for the package version20:05
dhellmannok, now back to my meeting20:05
dhellmannyeah, I had to change that because the build failed20:05
rbradfordhellmann, ++20:06
rbradfordims, can you +2 again on dhellmann oslo.db reno https://review.openstack.org/28376020:07
openstackgerritMerged openstack/oslo.versionedobjects: Fix messages in exceptions raised due to invalid state transitions  https://review.openstack.org/27429720:10
*** itisha has joined #openstack-oslo20:12
*** rlrossit_ has joined #openstack-oslo20:14
*** subscope has joined #openstack-oslo20:16
openstackgerritMerged openstack/stevedore: Add a reference to entry_point_inspector  https://review.openstack.org/27729920:16
openstackgerritRonald Bradford proposed openstack/oslo.config: clear the cache before mutating the config files  https://review.openstack.org/28366320:19
*** regXboi has quit IRC20:23
openstackgerritMerged openstack/taskflow: Updated from global requirements  https://review.openstack.org/28278820:26
*** flwang has quit IRC20:31
openstackgerritMerged openstack/oslo.messaging: Avoid hardcoding the notification topic and specify driver  https://review.openstack.org/28214520:48
*** ativelkov has quit IRC20:48
*** rakhmerov has quit IRC20:49
*** ativelkov has joined #openstack-oslo20:51
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves poller's stop logic  https://review.openstack.org/28362220:51
*** rakhmerov has joined #openstack-oslo20:52
dimsdmitryme : around? can you please fix this merge conflict? https://review.openstack.org/#/c/281254/ (yulia's review)20:54
*** dims has quit IRC21:01
*** sergmelikyan has quit IRC21:01
*** dims has joined #openstack-oslo21:03
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves poller's stop logic  https://review.openstack.org/28362221:05
*** salv-orlando has joined #openstack-oslo21:05
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves logging  https://review.openstack.org/28367421:07
openstackgerritDmitriy Ukhlov proposed openstack/oslo.messaging: Improves logging  https://review.openstack.org/28367421:07
*** kgiusti has quit IRC21:17
*** boris-42 has joined #openstack-oslo21:19
*** salv-orl_ has joined #openstack-oslo21:27
*** salv-orlando has quit IRC21:30
*** amotoki has quit IRC21:30
openstackgerritMerged openstack/oslo.messaging: Get kafka notifications to work with kafka-python 0.9.5  https://review.openstack.org/28207921:40
*** tedross has joined #openstack-oslo21:50
*** cdent has quit IRC21:50
*** cdent has joined #openstack-oslo21:50
*** tedross has left #openstack-oslo21:55
*** dims has quit IRC22:02
*** edmondsw has quit IRC22:09
*** sigmavirus24 is now known as sigmavirus24_awa22:13
*** claudiub has quit IRC22:18
openstackgerritMerged openstack/oslo.db: add reno for release notes management  https://review.openstack.org/28376022:19
openstackgerritMerged openstack/oslo.config: Updated from global requirements  https://review.openstack.org/28322622:21
*** mdrabe has joined #openstack-oslo22:26
*** harlowja_at_home has joined #openstack-oslo22:27
*** cdent has quit IRC22:30
*** vilobhmm11 has joined #openstack-oslo22:33
openstackgerritEric Brown proposed openstack/oslo.service: Use choices on the ssl.version option  https://review.openstack.org/28338722:38
*** dims has joined #openstack-oslo22:40
*** mriedem has quit IRC22:42
*** subscope has quit IRC22:48
dhellmannrbradfor : can you remove your workflow flag from https://review.openstack.org/#/c/283270/ ? for policies we usually ask the whole team to sign up before approving22:56
*** dstanek has quit IRC22:58
*** mkoderer has quit IRC22:58
*** dulek has quit IRC22:58
*** psanchez has quit IRC22:58
*** cgalan has quit IRC22:58
*** pradk has quit IRC22:58
*** sileht has quit IRC22:58
*** john51 has quit IRC22:58
*** hogepodge has quit IRC22:58
*** yamahata has quit IRC22:58
*** pcaruana has quit IRC22:58
*** mattymo has quit IRC22:58
*** dhellmann has quit IRC22:58
*** SlickNik has quit IRC22:58
*** frickler has quit IRC22:58
*** lxsli_web has quit IRC22:58
*** dims has quit IRC22:58
*** harlowja_at_home has quit IRC22:58
*** vilobhmm11 has quit IRC22:58
*** jd__ has quit IRC22:58
*** dimtruck has quit IRC22:58
*** DuncanT has quit IRC22:58
*** eliqiao_ has quit IRC22:58
*** jamespage has quit IRC22:58
*** vipul- has quit IRC22:58
*** davidlenwell has quit IRC22:58
*** clayton has quit IRC22:58
*** lifeless has quit IRC22:58
*** boris-42 has quit IRC22:58
*** rakhmerov has quit IRC22:58
*** ativelkov has quit IRC22:58
*** itisha has quit IRC22:58
*** gordc has quit IRC22:58
*** dtardivel has quit IRC22:58
*** nikhil has quit IRC22:58
*** jamielennox has quit IRC22:58
*** Kiall has quit IRC22:58
*** amrith has quit IRC22:58
*** dguitarbite has quit IRC22:58
*** sigmavirus24_awa has quit IRC22:58
*** zigo has quit IRC22:58
*** mtreinish has quit IRC22:58
*** binarin has quit IRC22:58
*** nihilifer has quit IRC22:58
*** rpodolyaka_afk has quit IRC22:58
*** eezhova has quit IRC22:58
*** jroll has quit IRC22:58
*** salv-orl_ has quit IRC22:58
*** sputnik13 has quit IRC22:58
*** jecarey has quit IRC22:58
*** dkehn has quit IRC22:58
*** sdague has quit IRC22:58
*** dmitryme has quit IRC22:58
*** SpamapS has quit IRC22:58
*** rbradfor has quit IRC22:58
*** elarson has quit IRC22:58
*** pasquier-s has quit IRC22:58
*** wendar has quit IRC22:58
*** sheeprine has quit IRC22:58
*** dougwig has quit IRC22:58
*** jraim has quit IRC22:58
*** tpeoples has quit IRC22:58
*** EmilienM has quit IRC22:58
*** timsim has quit IRC22:58
*** fungi has quit IRC22:58
*** adam_g has quit IRC22:58
*** andreaf has quit IRC22:58
*** gus has quit IRC22:58
*** cloudnull has quit IRC22:58
*** Anticimex has quit IRC22:58
*** flaper87 has quit IRC22:58
*** jdandrea has quit IRC22:58
*** mmasaki has quit IRC22:58
*** mdavidson has quit IRC22:58
*** Kennan has quit IRC22:58
*** kbyrne has quit IRC22:58
*** ttx has quit IRC22:58
*** d0ugal has quit IRC22:58
*** amaurymedeiros has quit IRC22:58
*** kevinbenton has quit IRC22:58
*** hockeynut has quit IRC22:58
*** SergeyLukjanov has quit IRC22:58
*** mordred has quit IRC22:58
*** pkholkin_ has quit IRC22:58
*** HenryG has quit IRC22:58
*** doffm has quit IRC22:58
*** browne has quit IRC22:58
*** ekarlso- has quit IRC22:58
*** Nakato has quit IRC22:58
*** krotscheck has quit IRC22:58
*** bknudson has quit IRC22:58
*** isq_ has quit IRC22:58
*** gcb has quit IRC22:58
*** pblaho has quit IRC22:58
*** lucas-dinner has quit IRC22:58
*** mikal has quit IRC22:58
*** mkoderer__ has quit IRC22:58
*** BAKfr has quit IRC22:58
*** otherwiseguy has quit IRC22:58
*** szaher has quit IRC22:58
*** DinaBelova has quit IRC22:58
*** pleia2 has quit IRC22:58
*** ansiwen has quit IRC22:58
*** bapalm has quit IRC22:58
*** jeckersb has quit IRC22:58
*** gibi has quit IRC22:58
*** melwitt has quit IRC22:58
*** peterstac has quit IRC22:58
*** Horgix has quit IRC22:58
*** _stowa has quit IRC22:58
*** hemanthm has quit IRC22:58
*** ajo has quit IRC22:58
*** odyssey4me has quit IRC22:58
*** zzzeek has quit IRC22:58
*** jasondotstar has quit IRC22:58
*** redrobot has quit IRC22:58
*** haypo has quit IRC22:58
*** sabari has quit IRC22:58
*** zhiyan has quit IRC22:58
*** bogdando has quit IRC22:58
*** jokke_ has quit IRC22:58
*** xek has quit IRC22:58
*** lbragstad has quit IRC22:58
openstackgerritDoug Hellmann proposed openstack/oslo-specs: fix SSL/TLS warnings in build  https://review.openstack.org/28326922:58
openstackgerritDoug Hellmann proposed openstack/oslo-specs: add a policy documenting how we manage release notes  https://review.openstack.org/28327022:58
*** lbragstad has joined #openstack-oslo23:01
*** xek has joined #openstack-oslo23:01
*** jokke_ has joined #openstack-oslo23:01
*** bogdando has joined #openstack-oslo23:01
*** zhiyan has joined #openstack-oslo23:01
*** sabari has joined #openstack-oslo23:01
*** redrobot has joined #openstack-oslo23:01
*** jasondotstar has joined #openstack-oslo23:01
*** zzzeek has joined #openstack-oslo23:01
*** odyssey4me has joined #openstack-oslo23:01
*** ajo has joined #openstack-oslo23:01
*** haypo has joined #openstack-oslo23:01
*** frickler has joined #openstack-oslo23:01
*** SlickNik has joined #openstack-oslo23:01
*** dhellmann has joined #openstack-oslo23:01
*** mattymo has joined #openstack-oslo23:01
*** pcaruana has joined #openstack-oslo23:01
*** yamahata has joined #openstack-oslo23:01
*** lifeless has joined #openstack-oslo23:01
*** clayton has joined #openstack-oslo23:01
*** davidlenwell has joined #openstack-oslo23:01
*** vipul- has joined #openstack-oslo23:01
*** jamespage has joined #openstack-oslo23:01
*** eliqiao_ has joined #openstack-oslo23:01
*** DuncanT has joined #openstack-oslo23:01
*** dimtruck has joined #openstack-oslo23:01
*** jd__ has joined #openstack-oslo23:01
*** harlowja_at_home has joined #openstack-oslo23:01
*** vilobhmm11 has joined #openstack-oslo23:01
*** dims has joined #openstack-oslo23:01
*** kbyrne has joined #openstack-oslo23:01
*** Kennan has joined #openstack-oslo23:01
*** mdavidson has joined #openstack-oslo23:01
*** mmasaki has joined #openstack-oslo23:01
*** jdandrea has joined #openstack-oslo23:01
*** flaper87 has joined #openstack-oslo23:01
*** Anticimex has joined #openstack-oslo23:01
*** gus has joined #openstack-oslo23:01
*** andreaf has joined #openstack-oslo23:01
*** cloudnull has joined #openstack-oslo23:01
*** adam_g has joined #openstack-oslo23:01
*** fungi has joined #openstack-oslo23:01
*** timsim has joined #openstack-oslo23:01
*** EmilienM has joined #openstack-oslo23:01
*** tpeoples has joined #openstack-oslo23:01
*** jraim has joined #openstack-oslo23:01
*** dougwig has joined #openstack-oslo23:01
*** sheeprine has joined #openstack-oslo23:01
*** wendar has joined #openstack-oslo23:01
*** DinaBelova has joined #openstack-oslo23:01
*** boris-42 has joined #openstack-oslo23:01
*** rakhmerov has joined #openstack-oslo23:01
*** ativelkov has joined #openstack-oslo23:01
*** itisha has joined #openstack-oslo23:01
*** gordc has joined #openstack-oslo23:01
*** dtardivel has joined #openstack-oslo23:01
*** nikhil has joined #openstack-oslo23:01
*** jamielennox has joined #openstack-oslo23:01
*** Kiall has joined #openstack-oslo23:01
*** amrith has joined #openstack-oslo23:01
*** dguitarbite has joined #openstack-oslo23:01
*** sigmavirus24_awa has joined #openstack-oslo23:01
*** zigo has joined #openstack-oslo23:01
*** rpodolyaka_afk has joined #openstack-oslo23:01
*** mtreinish has joined #openstack-oslo23:01
*** binarin has joined #openstack-oslo23:01
*** nihilifer has joined #openstack-oslo23:01
*** eezhova has joined #openstack-oslo23:01
*** jroll has joined #openstack-oslo23:01
*** dulek has joined #openstack-oslo23:01
*** psanchez has joined #openstack-oslo23:01
*** dstanek has joined #openstack-oslo23:01
*** mkoderer has joined #openstack-oslo23:01
*** cgalan has joined #openstack-oslo23:02
*** pradk has joined #openstack-oslo23:02
*** john51 has joined #openstack-oslo23:02
*** sileht has joined #openstack-oslo23:02
*** hogepodge has joined #openstack-oslo23:02
*** john51 has quit IRC23:02
*** browne has joined #openstack-oslo23:03
*** ekarlso- has joined #openstack-oslo23:03
*** Nakato has joined #openstack-oslo23:03
*** krotscheck has joined #openstack-oslo23:03
*** bknudson has joined #openstack-oslo23:03
*** isq_ has joined #openstack-oslo23:03
*** gcb has joined #openstack-oslo23:03
*** pblaho has joined #openstack-oslo23:03
*** lucas-dinner has joined #openstack-oslo23:03
*** mkoderer__ has joined #openstack-oslo23:03
*** mikal has joined #openstack-oslo23:03
*** szaher has joined #openstack-oslo23:03
*** BAKfr has joined #openstack-oslo23:03
*** otherwiseguy has joined #openstack-oslo23:03
*** pleia2 has joined #openstack-oslo23:03
*** ansiwen has joined #openstack-oslo23:03
*** bapalm has joined #openstack-oslo23:03
*** jeckersb has joined #openstack-oslo23:03
*** gibi has joined #openstack-oslo23:03
*** Horgix has joined #openstack-oslo23:03
*** melwitt has joined #openstack-oslo23:03
*** peterstac has joined #openstack-oslo23:03
*** _stowa has joined #openstack-oslo23:03
*** hemanthm has joined #openstack-oslo23:03
*** d0ugal has joined #openstack-oslo23:03
*** amaurymedeiros has joined #openstack-oslo23:03
*** kevinbenton has joined #openstack-oslo23:03
*** hockeynut has joined #openstack-oslo23:03
*** SergeyLukjanov has joined #openstack-oslo23:03
*** mordred has joined #openstack-oslo23:03
*** pkholkin_ has joined #openstack-oslo23:03
*** HenryG has joined #openstack-oslo23:03
*** doffm has joined #openstack-oslo23:03
*** lxsli_web has joined #openstack-oslo23:03
*** vilobhmm11 has quit IRC23:03
*** john51 has joined #openstack-oslo23:03
*** ttx has joined #openstack-oslo23:03
*** john51 has quit IRC23:03
*** sigmavirus24_awa has quit IRC23:04
*** dkehn_ has joined #openstack-oslo23:04
*** vilobhmm11 has joined #openstack-oslo23:05
*** vilobhmm11 has quit IRC23:05
*** vilobhmm11 has joined #openstack-oslo23:06
*** salv-orl_ has joined #openstack-oslo23:06
*** sputnik13 has joined #openstack-oslo23:06
*** jecarey has joined #openstack-oslo23:06
*** dkehn has joined #openstack-oslo23:06
*** sdague has joined #openstack-oslo23:06
*** SpamapS has joined #openstack-oslo23:06
*** dmitryme has joined #openstack-oslo23:06
*** rbradfor has joined #openstack-oslo23:06
*** elarson has joined #openstack-oslo23:06
*** pasquier-s has joined #openstack-oslo23:06
*** dmitryme2 has joined #openstack-oslo23:06
*** sputnik13 has quit IRC23:07
*** dkehn has quit IRC23:07
*** dmitryme has quit IRC23:07
*** jecarey has quit IRC23:07
*** SpamapS has quit IRC23:07
*** dmitryme2 is now known as dmitryme23:07
*** sputnik13 has joined #openstack-oslo23:07
*** john51 has joined #openstack-oslo23:07
*** yamamoto has joined #openstack-oslo23:07
*** sigmavirus24_awa has joined #openstack-oslo23:08
*** exploreshaifali has joined #openstack-oslo23:08
*** exploreshaifali has quit IRC23:08
*** SpamapS has joined #openstack-oslo23:09
*** harlowja_at_home has quit IRC23:09
*** harlowja_at_home has joined #openstack-oslo23:10
openstackgerritMerged openstack/oslo.config: clear the cache before mutating the config files  https://review.openstack.org/28366323:11
*** yamamoto has quit IRC23:12
*** rlrossit_ has quit IRC23:12
*** sdague has quit IRC23:16
*** mdrabe has quit IRC23:21
*** dimtruck is now known as zz_dimtruck23:25
*** vilobhmm11 has quit IRC23:26
*** john51_ has joined #openstack-oslo23:37
*** gordc has quit IRC23:41
*** john51 has quit IRC23:41
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.messaging: Added duration to notify server/client  https://review.openstack.org/28125423:44

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