*** harlowja_at_home has quit IRC | 00:49 | |
*** mixos has joined #openstack-oslo | 00:53 | |
*** dims has quit IRC | 01:03 | |
*** mriedem_away has quit IRC | 01:04 | |
*** mixos has quit IRC | 01:13 | |
*** mixos has joined #openstack-oslo | 01:15 | |
*** EinstCrazy has joined #openstack-oslo | 01:21 | |
*** dims has joined #openstack-oslo | 01:37 | |
*** mixos has quit IRC | 01:39 | |
*** mixos has joined #openstack-oslo | 01:57 | |
openstackgerrit | Merged openstack/oslo.messaging: rabbit: Missing to pass parameter timeout to next https://review.openstack.org/263694 | 01:59 |
---|---|---|
*** dims has quit IRC | 02:07 | |
*** ozamiatin__ has quit IRC | 02:17 | |
*** zz_dimtruck is now known as dimtruck | 02:47 | |
*** salv-orlando has quit IRC | 02:52 | |
*** salv-orlando has joined #openstack-oslo | 02:53 | |
*** dimtruck is now known as zz_dimtruck | 02:57 | |
*** zz_dimtruck is now known as dimtruck | 03:02 | |
*** dimtruck is now known as zz_dimtruck | 03:15 | |
*** zz_dimtruck is now known as dimtruck | 03:38 | |
*** links has joined #openstack-oslo | 03:48 | |
*** vilobhmm111 has joined #openstack-oslo | 03:52 | |
*** vilobhmm11 has quit IRC | 03:55 | |
*** deray has joined #openstack-oslo | 04:07 | |
*** salv-orl_ has joined #openstack-oslo | 04:09 | |
*** salv-orlando has quit IRC | 04:11 | |
*** vilobhmm111 has quit IRC | 04:26 | |
*** davidlenwell_ is now known as davidlenwell | 04:35 | |
*** dimtruck is now known as zz_dimtruck | 04:58 | |
*** EinstCrazy has quit IRC | 04:58 | |
*** david-lyle has quit IRC | 05:00 | |
*** david-lyle has joined #openstack-oslo | 05:00 | |
*** mixos has quit IRC | 05:02 | |
*** yamamoto has joined #openstack-oslo | 05:04 | |
*** yamamoto has quit IRC | 05:04 | |
*** yamamoto has joined #openstack-oslo | 05:05 | |
*** yamamoto has quit IRC | 05:05 | |
*** EinstCrazy has joined #openstack-oslo | 05:07 | |
*** zz_dimtruck is now known as dimtruck | 05:13 | |
*** yamamoto has joined #openstack-oslo | 05:18 | |
*** yamamoto has quit IRC | 05:22 | |
*** yamamoto has joined #openstack-oslo | 05:22 | |
*** yamamoto has quit IRC | 05:25 | |
*** deray has quit IRC | 05:26 | |
*** deray has joined #openstack-oslo | 05:28 | |
*** vilobhmm11 has joined #openstack-oslo | 05:29 | |
*** deray_ has joined #openstack-oslo | 05:39 | |
*** gcb_ has joined #openstack-oslo | 05:41 | |
*** gcb has joined #openstack-oslo | 05:41 | |
*** deray has quit IRC | 05:42 | |
*** gcb_ has quit IRC | 05:50 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.messaging: Python 3 deprecated the logger.warn method in favor of warning https://review.openstack.org/261677 | 06:18 |
*** dimtruck is now known as zz_dimtruck | 06:18 | |
*** yamamoto has joined #openstack-oslo | 06:22 | |
*** nkrinner has joined #openstack-oslo | 06:23 | |
*** salv-orl_ has quit IRC | 06:31 | |
*** salv-orlando has joined #openstack-oslo | 06:31 | |
*** EinstCrazy has quit IRC | 06:38 | |
*** EinstCrazy has joined #openstack-oslo | 06:38 | |
openstackgerrit | Angus Lees proposed openstack/oslo.privsep: Improve `helper_command' config default https://review.openstack.org/265695 | 06:49 |
*** EinstCrazy has quit IRC | 06:56 | |
*** EinstCrazy has joined #openstack-oslo | 06:57 | |
*** EinstCrazy has quit IRC | 06:59 | |
*** EinstCrazy has joined #openstack-oslo | 07:02 | |
*** EinstCrazy has quit IRC | 07:05 | |
*** yamahata has quit IRC | 07:05 | |
*** EinstCrazy has joined #openstack-oslo | 07:06 | |
*** EinstCrazy has quit IRC | 07:07 | |
*** EinstCrazy has joined #openstack-oslo | 07:07 | |
*** EinstCrazy has quit IRC | 07:08 | |
*** EinstCrazy has joined #openstack-oslo | 07:09 | |
*** EinstCrazy has quit IRC | 07:10 | |
*** EinstCrazy has joined #openstack-oslo | 07:11 | |
*** gcb has quit IRC | 07:12 | |
*** EinstCrazy has quit IRC | 07:12 | |
*** EinstCrazy has joined #openstack-oslo | 07:13 | |
*** EinstCrazy has quit IRC | 07:14 | |
*** EinstCrazy has joined #openstack-oslo | 07:15 | |
*** EinstCrazy has quit IRC | 07:17 | |
openstackgerrit | Abhishek Kekane proposed openstack/oslo.utils: Add cliutils module to oslo.utils https://review.openstack.org/261319 | 07:21 |
*** EinstCrazy has joined #openstack-oslo | 07:22 | |
*** vilobhmm11 has quit IRC | 07:24 | |
*** yamamoto has quit IRC | 07:37 | |
*** ihrachys has joined #openstack-oslo | 08:08 | |
*** abhishekk has joined #openstack-oslo | 08:18 | |
*** EinstCrazy has quit IRC | 08:19 | |
*** EinstCrazy has joined #openstack-oslo | 08:19 | |
*** zakora has joined #openstack-oslo | 08:20 | |
*** markus_z has joined #openstack-oslo | 08:20 | |
*** deray_ has quit IRC | 08:28 | |
*** boris-42 has joined #openstack-oslo | 08:29 | |
*** openstackgerrit has quit IRC | 08:32 | |
*** openstackgerrit has joined #openstack-oslo | 08:32 | |
*** deray has joined #openstack-oslo | 08:34 | |
*** shardy has joined #openstack-oslo | 08:35 | |
*** yamamoto has joined #openstack-oslo | 08:37 | |
*** bauwser is now known as bauzas | 08:44 | |
*** yamamoto has quit IRC | 08:44 | |
*** salv-orlando has quit IRC | 08:44 | |
*** salv-orlando has joined #openstack-oslo | 08:45 | |
*** sbelous_ has joined #openstack-oslo | 09:04 | |
*** e0ne has joined #openstack-oslo | 09:05 | |
*** EinstCrazy has quit IRC | 09:09 | |
*** EinstCrazy has joined #openstack-oslo | 09:09 | |
*** EinstCrazy has quit IRC | 09:10 | |
*** EinstCrazy has joined #openstack-oslo | 09:11 | |
*** EinstCrazy has quit IRC | 09:11 | |
*** EinstCrazy has joined #openstack-oslo | 09:11 | |
*** ozamiatin__ has joined #openstack-oslo | 09:12 | |
*** EinstCrazy has quit IRC | 09:13 | |
*** EinstCrazy has joined #openstack-oslo | 09:13 | |
*** EinstCrazy has quit IRC | 09:13 | |
*** EinstCrazy has joined #openstack-oslo | 09:13 | |
*** EinstCrazy has quit IRC | 09:16 | |
*** EinstCrazy has joined #openstack-oslo | 09:16 | |
*** ozamiatin__ has quit IRC | 09:21 | |
*** yassine__ has joined #openstack-oslo | 09:23 | |
*** EinstCrazy has quit IRC | 09:24 | |
*** EinstCra_ has joined #openstack-oslo | 09:24 | |
*** ndipanov has joined #openstack-oslo | 09:25 | |
*** ndipanov has quit IRC | 09:26 | |
*** EinstCrazy has joined #openstack-oslo | 09:30 | |
*** EinstCra_ has quit IRC | 09:30 | |
*** yamamoto has joined #openstack-oslo | 09:36 | |
*** yamamoto has quit IRC | 09:38 | |
*** yamamoto has joined #openstack-oslo | 09:39 | |
*** salv-orlando has quit IRC | 09:40 | |
*** salv-orlando has joined #openstack-oslo | 09:41 | |
*** pkholkin has quit IRC | 09:41 | |
*** ndipanov has joined #openstack-oslo | 09:44 | |
*** pkholkin has joined #openstack-oslo | 09:44 | |
*** yamamoto has quit IRC | 09:57 | |
*** yamamoto has joined #openstack-oslo | 09:57 | |
*** DinaBelova has quit IRC | 10:01 | |
*** DinaBelova has joined #openstack-oslo | 10:01 | |
*** SergeyLukjanov has quit IRC | 10:02 | |
*** SergeyLukjanov2 has joined #openstack-oslo | 10:02 | |
*** SergeyLukjanov2 is now known as SergeyLukjanov | 10:02 | |
*** pblaho has joined #openstack-oslo | 10:03 | |
*** ozamiatin has quit IRC | 10:03 | |
*** yamamoto has quit IRC | 10:10 | |
*** salv-orl_ has joined #openstack-oslo | 10:10 | |
*** yamamoto has joined #openstack-oslo | 10:11 | |
*** EinstCrazy has quit IRC | 10:11 | |
*** salv-orlando has quit IRC | 10:12 | |
*** cdent has joined #openstack-oslo | 10:26 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Added redis sentinel HA implementation to zmq driver https://review.openstack.org/261546 | 10:31 |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: (WIP) [zmq] Heartbeat implementation https://review.openstack.org/257346 | 10:46 |
*** yamamoto has quit IRC | 11:00 | |
*** yamamoto has joined #openstack-oslo | 11:00 | |
*** yamamoto has quit IRC | 11:03 | |
*** salv-orl_ has quit IRC | 11:03 | |
*** yamamoto has joined #openstack-oslo | 11:03 | |
*** salv-orlando has joined #openstack-oslo | 11:04 | |
*** salv-orlando has quit IRC | 11:05 | |
*** salv-orlando has joined #openstack-oslo | 11:05 | |
sileht | /1/0 | 11:06 |
*** abhishekk has quit IRC | 11:06 | |
*** mhickey has joined #openstack-oslo | 11:08 | |
*** ozamiatin has joined #openstack-oslo | 11:11 | |
*** dims_ has joined #openstack-oslo | 11:18 | |
*** lucas-dinner is now known as lucasagomes | 11:23 | |
*** yamamoto has quit IRC | 11:26 | |
*** bauzas has quit IRC | 11:35 | |
*** jaypipes has joined #openstack-oslo | 11:37 | |
*** eezhova has joined #openstack-oslo | 11:37 | |
*** sputnik13 has joined #openstack-oslo | 11:40 | |
*** binarin has quit IRC | 12:01 | |
*** binarin has joined #openstack-oslo | 12:01 | |
*** EinstCrazy has joined #openstack-oslo | 12:07 | |
*** ozamiatin__ has joined #openstack-oslo | 12:10 | |
*** abhishekk has joined #openstack-oslo | 12:12 | |
*** yamamoto has joined #openstack-oslo | 12:26 | |
*** HenryG has quit IRC | 12:27 | |
*** HenryG has joined #openstack-oslo | 12:27 | |
*** gordc has joined #openstack-oslo | 12:27 | |
*** yamamoto has quit IRC | 12:30 | |
*** binarin has quit IRC | 12:35 | |
openstackgerrit | Dmitry Tantsur proposed openstack/futurist: Handle exceptions from executor.submit in PeriodicWorker https://review.openstack.org/265360 | 12:41 |
*** alejandrito has joined #openstack-oslo | 12:45 | |
*** alejandrito has quit IRC | 12:46 | |
*** ozamiatin_ has joined #openstack-oslo | 12:46 | |
*** alejandrito has joined #openstack-oslo | 12:46 | |
*** ozamiatin__ has quit IRC | 12:48 | |
*** salv-orlando has quit IRC | 12:55 | |
openstackgerrit | Dmitry Tantsur proposed openstack/futurist: PeriodicWorker.create to accept arguments for periodic tasks https://review.openstack.org/264247 | 12:56 |
*** sputnik13 has quit IRC | 12:56 | |
*** salv-orlando has joined #openstack-oslo | 12:56 | |
*** yamamoto has joined #openstack-oslo | 13:02 | |
*** takedakn has joined #openstack-oslo | 13:06 | |
*** takedakn has quit IRC | 13:09 | |
*** edmondsw has joined #openstack-oslo | 13:11 | |
*** salv-orlando has quit IRC | 13:11 | |
*** salv-orlando has joined #openstack-oslo | 13:12 | |
*** links has quit IRC | 13:13 | |
*** abhishekk has quit IRC | 13:13 | |
*** MVenesio has joined #openstack-oslo | 13:21 | |
*** obondarev has joined #openstack-oslo | 13:32 | |
*** bauzas has joined #openstack-oslo | 13:33 | |
*** EinstCrazy has quit IRC | 13:40 | |
*** amrith is now known as _amrith_ | 13:40 | |
*** yottatsa has joined #openstack-oslo | 13:41 | |
*** jecarey has quit IRC | 13:41 | |
*** yamamoto has quit IRC | 13:42 | |
*** jeckersb_gone is now known as jeckersb | 14:01 | |
*** yamamoto has joined #openstack-oslo | 14:02 | |
*** rlrossit has joined #openstack-oslo | 14:03 | |
openstackgerrit | Dmitry Tantsur proposed openstack/futurist: PeriodicWorker.create to accept arguments for periodic tasks https://review.openstack.org/264247 | 14:10 |
*** takedakn has joined #openstack-oslo | 14:18 | |
*** takedakn has quit IRC | 14:18 | |
ansiwen | mhorban: i'm here | 14:20 |
mhorban | ansiwen: Hi | 14:20 |
mhorban | ansiwen: with eventlet we will not have problems | 14:20 |
ansiwen | mhorban: the background is: I try to exclude rare cornercases, where a "copytruncate" in the logrotate config would still be useful | 14:21 |
*** lucasagomes is now known as lucas-hungry | 14:23 | |
mhorban | ansiwen: When we use eventlet each child process will contain own eventloop. But It we are using system threads - we will have problem that you have described | 14:23 |
ansiwen | so the eventlet eventloop runs in a seperate thread, that is copied with the fork, do I understand that correctly? | 14:25 |
ansiwen | mhorban: 1 | 14:25 |
ansiwen | mhorban: ^ | 14:25 |
*** jecarey has joined #openstack-oslo | 14:25 | |
mhorban | ansiwen: If program use eventlet.monkey_patch thread module in this case program will not use system thread | 14:28 |
*** jecarey has quit IRC | 14:28 | |
*** jecarey has joined #openstack-oslo | 14:29 | |
openstackgerrit | Martin Hickey proposed openstack/oslo.versionedobjects: Add temporary registry pattern to VersionedObjectRegistry https://review.openstack.org/263800 | 14:32 |
ansiwen | mhorban: but if I do a typical fork/exec code, all threads are destroyed, no? but the filedescriptors are copied... so I wonderwhat is closing them in that case | 14:33 |
*** kgiusti has joined #openstack-oslo | 14:33 | |
mhorban | ansiwen: eventlet emulates threads but it doesn't use system threads. It uses coroutines. Program with green threads can be forked safety | 14:36 |
ansiwen | mhorban: sure, I understand that | 14:37 |
ansiwen | mhorban: i'm more thinking of the possibility, that the eventlet mechanism is destroyed in the forked process, like for example with an exec() call | 14:42 |
*** cprmrf has joined #openstack-oslo | 14:43 | |
*** mriedem has joined #openstack-oslo | 14:45 | |
openstackgerrit | Alexis Lee proposed openstack/oslo.config: Fake attrs on _Namespace https://review.openstack.org/264847 | 14:53 |
openstackgerrit | Alexis Lee proposed openstack/oslo.config: Make _Namespace the value-holder https://review.openstack.org/264787 | 14:53 |
openstackgerrit | Alexis Lee proposed openstack/oslo.config: Add 'mutable' param to Opt https://review.openstack.org/251471 | 14:53 |
openstackgerrit | Alexis Lee proposed openstack/oslo.config: Allow namespace modification https://review.openstack.org/253125 | 14:53 |
mhorban | ansiwen: with exec we will inherit file descriptors...And what? Could you please describe whole scenario of possible problem? | 14:54 |
ansiwen | mhorban: a log-file is opened with FastWatchedFileHandler, then the python process forks and execs another program which does not terminate. now the log file is deleted. it will still not be freed, because the exec'd fork doesn't free it, right? | 14:57 |
*** regXboi has joined #openstack-oslo | 14:57 | |
*** pradk has joined #openstack-oslo | 14:58 | |
mhorban | ansiwen: right | 14:58 |
ansiwen | mhorban: ok, thanks... :-) | 14:58 |
*** pratikmallya has joined #openstack-oslo | 14:59 | |
*** deray has quit IRC | 14:59 | |
mhorban | ansiwen: But I believe that there is no possibility to fix it. Isn't it? | 15:01 |
*** pratikma_ has joined #openstack-oslo | 15:01 | |
ansiwen | mhorban: well, at least by carefully avoiding filedescriptor leaks by closing all filedescriptors before exec() | 15:03 |
ansiwen | mhorban: but I hoped there is some python magic maybe, that does that... | 15:03 |
mhorban | ansiwen: :) | 15:04 |
*** pratikmallya has quit IRC | 15:04 | |
*** _amrith_ is now known as amrith | 15:05 | |
*** cdent has quit IRC | 15:10 | |
*** cprmrf has quit IRC | 15:12 | |
*** binarin has joined #openstack-oslo | 15:13 | |
*** obondarev has quit IRC | 15:14 | |
*** ansiwen has quit IRC | 15:14 | |
*** obondarev has joined #openstack-oslo | 15:15 | |
*** ansiwen has joined #openstack-oslo | 15:16 | |
*** mixos has joined #openstack-oslo | 15:16 | |
*** sreshetnyak has quit IRC | 15:18 | |
*** pkholkin has quit IRC | 15:19 | |
*** pkholkin has joined #openstack-oslo | 15:22 | |
*** mixos has quit IRC | 15:25 | |
*** lucas-hungry is now known as lucasagomes | 15:25 | |
*** sreshetnyak has joined #openstack-oslo | 15:26 | |
*** mixos has joined #openstack-oslo | 15:27 | |
*** nkrinner has quit IRC | 15:27 | |
*** yamahata has joined #openstack-oslo | 15:27 | |
*** pblaho has quit IRC | 15:35 | |
openstackgerrit | Soumiyajit proposed openstack/oslo.log: Added tenant id part of the log Closes-bug:1527420 https://review.openstack.org/265884 | 15:37 |
*** zz_dimtruck is now known as dimtruck | 15:37 | |
*** sdague has joined #openstack-oslo | 15:48 | |
sdague | has victor been around of late? | 15:48 |
*** gcb has joined #openstack-oslo | 15:53 | |
*** mtanino has joined #openstack-oslo | 15:53 | |
*** sbelous_ has quit IRC | 16:00 | |
*** sbelous_ has joined #openstack-oslo | 16:01 | |
sdague | or anyone else with experience getting stuff landed in routes | 16:01 |
*** cdent has joined #openstack-oslo | 16:02 | |
*** salv-orl_ has joined #openstack-oslo | 16:09 | |
*** mriedem has left #openstack-oslo | 16:10 | |
*** mriedem has joined #openstack-oslo | 16:10 | |
*** salv-orlando has quit IRC | 16:12 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 16:16 | |
*** deray has joined #openstack-oslo | 16:25 | |
*** mixos has quit IRC | 16:30 | |
*** gcb has quit IRC | 16:31 | |
openstackgerrit | Yulia Portnova proposed openstack/oslo.messaging: Add duration option to simulator.py https://review.openstack.org/265912 | 16:33 |
*** zakora has quit IRC | 16:33 | |
*** julia has joined #openstack-oslo | 16:36 | |
*** julia is now known as Guest85384 | 16:36 | |
dims_ | Guest85384 : ping | 16:37 |
openstackgerrit | Ian Cordasco proposed openstack/oslo.policy: CLI Policy Check tool https://review.openstack.org/170978 | 16:37 |
Guest85384 | dims_ : Hi, it is Julia | 16:38 |
dims_ | Guest85384 : you need a better IRC client :) | 16:39 |
Guest85384 | yep, need to install one | 16:39 |
dims_ | Guest85384 : julia, can you please update the https://review.openstack.org/#/c/247668/ and https://review.openstack.org/#/c/249215/ when you get a chance? | 16:39 |
Guest85384 | Sure I will update it | 16:41 |
dims_ | Thanks Guest85384 / julia | 16:41 |
Guest85384 | I have run some test with rally plugin and simulator with the compression and without. I have not noticed any difference, only with huge messages ~ 250 k | 16:42 |
*** mixos has joined #openstack-oslo | 16:46 | |
openstackgerrit | Yulia Portnova proposed openstack/oslo.messaging: Add duration option to simulator.py https://review.openstack.org/265912 | 16:48 |
openstackgerrit | Eric Harney proposed openstack/oslo.db: Add exc_filter for invalid Unicode string https://review.openstack.org/265921 | 16:49 |
*** pratikma_ has quit IRC | 16:50 | |
*** haypo has joined #openstack-oslo | 16:50 | |
haypo | dhellmann: https://review.openstack.org/#/c/262752/ "This looks good, but I'd like to get the follow-up patching working so we can merge them both at the same time." | 16:50 |
haypo | dhellmann: hum, which second part? | 16:50 |
haypo | dhellmann: the patch alone fixes the issue, no? | 16:50 |
amrith | dims_, how would I know when the new release of oslo.utils is 'made'? | 16:53 |
*** yamamoto has quit IRC | 16:53 | |
haypo | amrith: https://review.openstack.org/#/c/265673/ ? | 16:53 |
amrith | thx haypo | 16:54 |
*** mhickey has quit IRC | 16:55 | |
*** ihrachys has quit IRC | 16:55 | |
*** dims has joined #openstack-oslo | 16:56 | |
*** dims_ has quit IRC | 16:56 | |
*** yamahata has quit IRC | 16:58 | |
dims | Guest85384 : understood | 16:58 |
*** mixos has quit IRC | 17:01 | |
*** mixos has joined #openstack-oslo | 17:03 | |
dims | dukhlov : any doc for the pika driver yet? | 17:05 |
*** yassine__ has quit IRC | 17:05 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.messaging: fix .gitreview - bad merge from pika branch https://review.openstack.org/265932 | 17:06 |
dims | ozamiatin : dukhlov : ^^ oops! | 17:08 |
*** sigmavirus24 is now known as sigmavirus24_awa | 17:09 | |
dims | ozamiatin : dukhlov : https://review.openstack.org/265934 | 17:11 |
*** mixos has quit IRC | 17:11 | |
*** mixos has joined #openstack-oslo | 17:12 | |
*** yamamoto has joined #openstack-oslo | 17:13 | |
*** yamamoto has quit IRC | 17:13 | |
*** timsim has joined #openstack-oslo | 17:14 | |
sdague | um, what's up with oslo.db logging exceptions now? - http://logs.openstack.org/21/265521/2/check/gate-tempest-dsvm-nova-v20-api-legacy/377698c/logs/screen-n-api.txt.gz?level=ERROR#_2016-01-11_16_48_10_120 | 17:17 |
dims | sdague : https://github.com/openstack/oslo.db/compare/4.1.0...4.2.0 - mriedem ^^ | 17:19 |
mriedem | https://github.com/openstack/oslo.db/commit/838e314a21fef9ca6cf99140b2400e0d3b68b109 | 17:20 |
sdague | mriedem: ok, so that results in every overquota operation putting a stacktrace and an ERROR in nova-api | 17:21 |
timsim | Could someone tell me if this is possible with oslo.messaging? I'd like to have a number of tasks that can queue up in separate queues/exchanges/whatevers, and allow "workers" to go decide which tasks/queues/exchanges to give "priority" to, and work from first. :) | 17:22 |
*** yamahata has joined #openstack-oslo | 17:22 | |
*** markus_z has quit IRC | 17:22 | |
*** Guest85384 has quit IRC | 17:22 | |
mriedem | sdague: i think i see the problem | 17:24 |
mriedem | missing if isinstance(exc, self.db_error): | 17:24 |
mriedem | will fix | 17:24 |
sdague | mriedem: thanks | 17:30 |
dims | mriedem : +1 | 17:31 |
mriedem | got the fix, just writing a test | 17:31 |
mriedem | which is the harder part | 17:31 |
*** amrith is now known as _amrith_ | 17:39 | |
*** _amrith_ is now known as amrith | 17:40 | |
openstackgerrit | Matt Riedemann proposed openstack/oslo.db: Don't log non-db error in retry wrapper https://review.openstack.org/265950 | 17:40 |
mriedem | sdague: dims: ^ | 17:40 |
*** deray has quit IRC | 17:41 | |
sdague | mriedem: so just out of curiosity, why not put it into the _is_exception_expected ? | 17:44 |
sdague | which has slightly sideways logic from that | 17:44 |
sdague | and only exists for that single call | 17:44 |
mriedem | The LOG.exception? | 17:44 |
mriedem | that's where it was before and spammed the logs | 17:44 |
mriedem | https://review.openstack.org/#/c/254964/ | 17:45 |
sdague | mriedem: no, I mean you've added another conditional | 17:45 |
sdague | right after a function that computes a conditional | 17:45 |
sdague | and that function that computes the conditional, is used no where else | 17:45 |
mriedem | so rather than return self.exception_checker(exc) | 17:46 |
openstackgerrit | Ronald Bradford proposed openstack/oslo.log: Improve documentataion of Oslo Log Usage https://review.openstack.org/264977 | 17:47 |
mriedem | check that and if it's False, then log the exception and return False? | 17:47 |
sdague | which in your patch will now be ignored | 17:47 |
sdague | so self.exception_checker is 100% dead code | 17:47 |
sdague | in this path | 17:47 |
mriedem | i'm not really following, _is_exception_expected is also checking the exception_checker function that can be passed in | 17:48 |
mriedem | which just defaults to returning False - treating any non-db retry errors as fatal | 17:49 |
*** pratikmallya has joined #openstack-oslo | 17:50 | |
sdague | mriedem: oh, it throws an exception? | 17:50 |
mriedem | no | 17:51 |
mriedem | the exception_checker just checks special cases i think | 17:51 |
mriedem | idk | 17:51 |
sdague | exception_checker is passed in by the caller it appears | 17:52 |
mriedem | yeah | 17:52 |
sdague | however, you are bypassing it entirely right? | 17:52 |
sdague | exception of type Foo comes in | 17:53 |
sdague | no matter what the return value of exception_checker, you are going to ignore it in that block for your patch | 17:54 |
mriedem | this? if ectxt.reraise and isinstance(e, self.db_error): | 17:56 |
mriedem | the reraise flag is already set | 17:56 |
mriedem | which is what exception_checker is used for | 17:56 |
sdague | right | 17:56 |
mriedem | the only thing that if ectxt.reraise and isinstance(e, self.db_error): skips on Foo is logging it at exception level | 17:56 |
sdague | sure | 17:56 |
mriedem | Foo is OverQuota in this case | 17:57 |
mriedem | or are you saying we should just remove L142-146 altogether? | 17:57 |
sdague | sure, but is ectxt.reraise ever used again? | 17:57 |
mriedem | it's used by the context it's in | 17:58 |
mriedem | with excutils.save_and_reraise_exception() as ectxt: | 17:58 |
sdague | oh, because it's a context mager | 17:58 |
sdague | manager | 17:58 |
mriedem | yar | 17:58 |
mriedem | setting that flag to true means it's fatal, stop retrying and reraise | 17:58 |
sdague | yeh, ok so it seems like _is_exception_expected is overloaded with logging, which it shouldn't be | 17:59 |
*** sputnik13 has joined #openstack-oslo | 17:59 | |
*** e0ne has quit IRC | 18:01 | |
sdague | it's confusing that variable is being used for 2 things. It feels like the logic should just be pulled apart. Should I reraise is one thing, should I log and how is another. | 18:01 |
*** ihrachys has joined #openstack-oslo | 18:04 | |
*** SpamapS has quit IRC | 18:10 | |
*** pradk_ has joined #openstack-oslo | 18:10 | |
*** pradk has quit IRC | 18:11 | |
*** pradk_ is now known as pradk | 18:12 | |
*** yamamoto has joined #openstack-oslo | 18:13 | |
*** yassine__ has joined #openstack-oslo | 18:15 | |
*** sileht has quit IRC | 18:17 | |
dims | mriedem : sdague : consensus? | 18:18 |
mriedem | i'm eating a banana | 18:19 |
*** yamamoto has quit IRC | 18:21 | |
dims | mriedem : which game did you like the most? | 18:21 |
dims | over the weekend | 18:21 |
mriedem | i don't want to talk about it | 18:22 |
dims | too early :) | 18:23 |
mriedem | yes, open wounds | 18:23 |
*** sileht has joined #openstack-oslo | 18:33 | |
*** SpamapS has joined #openstack-oslo | 18:36 | |
*** ozamiatin_ has quit IRC | 18:38 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.concurrency: Remove unnecessary package in setup.cfg https://review.openstack.org/265982 | 18:38 |
harlowja | dims yo, have been having a cold, so decided to rest this morning, still have it, but hopefully it goes away soon, will check up on meeting logs | 18:39 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.concurrency: Remove unnecessary package in setup.cfg https://review.openstack.org/265982 | 18:40 |
dims | harlowja : hope you feel better soon! | 18:40 |
harlowja | def, i think its going away, just never sure :( | 18:40 |
harlowja | dims https://review.openstack.org/#/c/265673/ a taskflow release i think is ok | 18:41 |
harlowja | if u dare | 18:41 |
harlowja | and/or do another patchset | 18:42 |
dims | harlowja : let's do another review | 18:44 |
harlowja | k | 18:44 |
*** e0ne has joined #openstack-oslo | 18:49 | |
*** shardy has quit IRC | 18:50 | |
*** barra204 has joined #openstack-oslo | 18:55 | |
*** shakamunyi has quit IRC | 18:56 | |
*** shardy has joined #openstack-oslo | 18:56 | |
*** vilobhmm11 has joined #openstack-oslo | 19:11 | |
*** vilobhmm11 has quit IRC | 19:11 | |
*** vilobhmm11 has joined #openstack-oslo | 19:11 | |
*** shakamunyi has joined #openstack-oslo | 19:12 | |
*** barra204 has quit IRC | 19:12 | |
*** amrith is now known as _amrith_ | 19:13 | |
*** _amrith_ is now known as amrith | 19:37 | |
*** yassine__ has quit IRC | 19:41 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.middleware: Remove old references to namespace packages https://review.openstack.org/266008 | 19:46 |
dims | lifeless : harlowja : let's get https://review.openstack.org/266008 and https://review.openstack.org/#/c/265982/ in - cleaning up packages as clarkb mentioned on infra channel | 19:47 |
harlowja | cool | 19:47 |
harlowja | damn namespace :-P | 19:47 |
*** psanchez has quit IRC | 19:50 | |
*** lucasagomes is now known as lucas-dinner | 19:53 | |
timsim | reposting from earlier: Could someone tell me if this is possible with oslo.messaging? I'd like to have a number of tasks that can queue up in separate queues/exchanges/whatevers, and allow "workers" to go decide which tasks/queues/exchanges to give "priority" to, and work from first. :) | 20:00 |
openstackgerrit | Joshua Harlow proposed openstack/tooz: Add lock breaking https://review.openstack.org/266015 | 20:04 |
*** ihrachys has quit IRC | 20:12 | |
dhellmann | haypo : a follow-up patch changes the default | 20:17 |
rbradfor | dims, could you elaborate on what you found that broke for you to revert https://review.openstack.org/#/c/265534/ I was a bit concerned about this proposed change, but did not have concrete case | 20:17 |
openstackgerrit | Eric Harney proposed openstack/oslo.db: Add exc_filter for invalid Unicode string https://review.openstack.org/265921 | 20:19 |
*** mhickey has joined #openstack-oslo | 20:19 | |
*** fhermeni has joined #openstack-oslo | 20:32 | |
fhermeni | hello there | 20:32 |
openstackgerrit | Graham Hayes proposed openstack/oslo.versionedobjects: Added a state machine field https://review.openstack.org/266028 | 20:34 |
dims | rbradfor : details are in the bug link https://bugs.launchpad.net/oslo.context/+bug/1532427 | 20:44 |
openstack | Launchpad bug 1532427 in oslo.context ""Add properties for id attributes" breaks Heat" [High,Confirmed] - Assigned to Jamie Lennox (jamielennox) | 20:44 |
dims | rbradfor : heat tests failed | 20:44 |
rbradfor | dims, thanks, looking now | 20:44 |
*** amotoki has joined #openstack-oslo | 20:52 | |
*** shardy has quit IRC | 21:09 | |
dims | kgiusti : flaper87 : have you seen this? http://logs.openstack.org/32/265932/1/check/gate-oslo.messaging-dsvm-functional-amqp1-centos7/f5f3eb9/console.html#_2016-01-11_20_03_33_210 | 21:09 |
* flaper87 clicks | 21:09 | |
kgiusti | dims: yeah - actually I just hit that yum install failure locally. | 21:11 |
*** ihrachys has joined #openstack-oslo | 21:11 | |
kgiusti | dims: devstack thinks yum install failed when it actually succeeds | 21:12 |
kgiusti | dims: I thought the problem was local to my vm. Apparently not. | 21:13 |
dims | kgiusti : "1344 yum install failure" | 21:14 |
dims | a few lines before the link above | 21:14 |
dims | that one? | 21:14 |
kgiusti | dims: yeah, it's caused by a 'pipe failure" - see this commit: https://git.openstack.org/cgit/openstack-dev/devstack/commit/?id=2cb3db338a17bd26fef1f56fc155be5d01a6ef7e | 21:14 |
kgiusti | dims: PIPESTATUS[0] is always 1 on my centos7 box | 21:15 |
openstackgerrit | Joshua Harlow proposed openstack/tooz: Track and use ownership information on acquire/break/release https://review.openstack.org/263987 | 21:15 |
dims | kgiusti : any ideas which packages it has problems with? | 21:17 |
kgiusti | dims: for me it failed trying to install epel-release | 21:18 |
kgiusti | dims: I commented out the check of PIPESTATUS, and I'm happily devstack-ing it up. | 21:18 |
*** sigmavirus24_awa is now known as sigmavirus24 | 21:19 | |
dims | kgiusti : please comment on https://review.openstack.org/#/c/266043/ revert | 21:22 |
*** pratikmallya has quit IRC | 21:23 | |
kgiusti | now *that's* service! | 21:25 |
*** e0ne has quit IRC | 21:25 | |
openstackgerrit | Joshua Harlow proposed openstack/tooz: Add lock breaking https://review.openstack.org/266015 | 21:28 |
dims | harlowja : welcome to the social network thingies | 21:31 |
harlowja | dims jd__ convinced me that facebook wasn't cool enough | 21:31 |
harlowja | and he said i should get on there :-P | 21:31 |
harlowja | guess i'm hip now | 21:32 |
dims | :) | 21:32 |
harlowja | lol | 21:32 |
harlowja | or do they call it something other than 'hip' | 21:32 |
harlowja | and by saying hip i am not hip :-/ | 21:32 |
harlowja | lol | 21:32 |
dims | harlowja : http://www.urbandictionary.com/define.php?term=sick | 21:34 |
dims | just learned this a week or so ago from my young ones | 21:34 |
*** kgiusti has quit IRC | 21:35 | |
*** kgiusti has joined #openstack-oslo | 21:36 | |
*** kgiusti has left #openstack-oslo | 21:36 | |
*** MVenesio has quit IRC | 21:36 | |
*** ihrachys has quit IRC | 21:41 | |
*** ihrachys has joined #openstack-oslo | 21:42 | |
fhermeni | harlowja: hi Joshua | 21:45 |
*** pratikmallya has joined #openstack-oslo | 21:46 | |
*** pratikma_ has joined #openstack-oslo | 21:48 | |
*** pratikmallya has quit IRC | 21:51 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 21:51 | |
*** ndipanov has quit IRC | 21:51 | |
*** dstanek has quit IRC | 21:53 | |
*** dstanek has joined #openstack-oslo | 21:53 | |
*** ihrachys has quit IRC | 21:55 | |
*** ihrachys has joined #openstack-oslo | 21:56 | |
*** davidlenwell has quit IRC | 22:05 | |
-openstackstatus- NOTICE: Gerrit is restarting to resolve java memory issues | 22:06 | |
openstackgerrit | Merged openstack/oslo.middleware: Remove old references to namespace packages https://review.openstack.org/266008 | 22:07 |
*** salv-orlando has joined #openstack-oslo | 22:09 | |
openstackgerrit | Martin Hickey proposed openstack/oslo.versionedobjects: Add temporary registry pattern to VersionedObjectRegistry https://review.openstack.org/263800 | 22:09 |
dims | good time to take a break i guess :) | 22:09 |
*** salv-orl_ has quit IRC | 22:12 | |
fhermeni | harlowja: ping. I want to know if you are still waiting for some internal reviews of the super-scheduler spec ? | 22:12 |
*** ozamiatin_ has joined #openstack-oslo | 22:13 | |
harlowja | dims ya, i'm not that kind of sick :-P | 22:14 |
harlowja | sick yo | 22:14 |
harlowja | lol | 22:14 |
harlowja | fhermeni yo yo | 22:14 |
harlowja | fhermeni haven't done much recently | 22:15 |
harlowja | should get back into that thing | 22:15 |
harlowja | fhermeni any updates u want to do to it? | 22:17 |
fhermeni | harlowja: there is a meeting for Watcher project on wed. I will assist to try to discuss about what we are doing | 22:17 |
harlowja | kk | 22:17 |
harlowja | cools | 22:17 |
fhermeni | harlowja: I commented one of their commit to notify them about our work but no news | 22:17 |
harlowja | ya, the last couple weeks people are all over the place | 22:18 |
*** jeckersb is now known as jeckersb_gone | 22:21 | |
*** pratikma_ has quit IRC | 22:26 | |
*** ihrachys has quit IRC | 22:26 | |
*** ihrachys has joined #openstack-oslo | 22:28 | |
*** regXboi has quit IRC | 22:28 | |
*** davidlenwell has joined #openstack-oslo | 22:29 | |
*** ihrachys has quit IRC | 22:32 | |
*** mhickey has quit IRC | 22:33 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 22:34 | |
*** cdent has quit IRC | 22:40 | |
*** pratikmallya has joined #openstack-oslo | 22:43 | |
*** pratikma_ has joined #openstack-oslo | 22:45 | |
*** pratikmallya has quit IRC | 22:48 | |
harlowja | dhellmann dims flaper87 https://bitbucket.org/astanin/python-tabulate/pull-requests/25/ | 22:53 |
harlowja | that might be an ok way to go | 22:53 |
* harlowja likely more compat stuff could be done, but ^ is the real basics, lol | 22:53 | |
*** mixos has quit IRC | 22:54 | |
flaper87 | harlowja: nice | 22:54 |
harlowja | if someone wants to make super-ultra-compat-layer then they need to start ripping prettytablre apart, lol | 22:54 |
harlowja | especially since https://code.google.com/p/prettytable/source/browse/trunk/prettytable.py#1122 has alot of options (same with __init__) | 22:54 |
harlowja | but said layer is maybe better than full on adopting prettytable | 22:55 |
harlowja | *or seems better | 22:55 |
flaper87 | Although, even if we move to tabulate, I think we should pull prettytable in and then give it back if needed. We have to support stable branches and I don't think this switch will happen, transparently, during Mitaka | 22:57 |
*** mriedem is now known as mriedem_away | 22:57 | |
harlowja | ya, perhaps a multi-front approach is the only way | 22:58 |
harlowja | are people really so dependent on the output of these tables :( | 22:58 |
harlowja | that scares me, lol | 22:58 |
flaper87 | you should be scared. There are folks that do depend on this | 22:59 |
harlowja | seems like clients should have a -m option or something, for machine-readable-output (which could use tsv or csv or something) instead | 22:59 |
flaper87 | :P | 22:59 |
harlowja | def | 22:59 |
flaper87 | FWIW, openstack client hsa this, IIRC. | 22:59 |
flaper87 | or at least cliff does (IIRC, again) | 23:00 |
harlowja | thats good, tabulate also has it, https://bitbucket.org/astanin/python-tabulate/src/c300cdf086d8e33dbc5f1c98049d631ad5ad8e4c/tabulate.py?fileviewer=file-view-default#tabulate.py-305 | 23:00 |
* harlowja doesn't think prettytable has it | 23:00 | |
harlowja | it has msword_friendly though, lol | 23:00 |
harlowja | @ https://code.google.com/p/prettytable/source/browse/trunk/prettytable.py#72 | 23:00 |
flaper87 | TBH, I don't care if it's tabulate or prettytable. I care about us being able to provide support if shit happens and prettytable being abandoned is not good. If we want to switch w/o adopting prettytable then we need someone willing to help maintaining prettytable. I don't have time for that :( | 23:01 |
flaper87 | I do see the point about using tabulate and it makes sense to me | 23:02 |
harlowja | redhat can hire the prettytable author right? | 23:02 |
harlowja | ;) | 23:02 |
flaper87 | LOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL | 23:03 |
harlowja | that fixes the glitch, there u go | 23:05 |
openstackgerrit | Joshua Harlow proposed openstack/tooz: Track and use ownership information on acquire/break/release https://review.openstack.org/263987 | 23:08 |
openstackgerrit | Joshua Harlow proposed openstack/tooz: Track and use ownership information on acquire/break/release https://review.openstack.org/263987 | 23:10 |
*** pratikma_ has quit IRC | 23:14 | |
fhermeni | bye people | 23:14 |
*** fhermeni has quit IRC | 23:14 | |
*** dimtruck is now known as zz_dimtruck | 23:16 | |
harlowja | byeeee | 23:19 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Instead of a multiprocessing queue use sockets via asyncore https://review.openstack.org/244376 | 23:21 |
*** rlrossit has quit IRC | 23:22 | |
*** amotoki has quit IRC | 23:25 | |
dhellmann | harlowja, flaper87 : maybe we can move prettytable to https://jazzband.co instead of oslo, and then some of us can contribute as needed | 23:26 |
harlowja | move all of oslo there u say? | 23:26 |
harlowja | +2 | 23:26 |
dhellmann | harlowja : not *all* of it ;-) | 23:26 |
harlowja | oh | 23:26 |
harlowja | move 99% of oslo there u say | 23:26 |
harlowja | lol | 23:26 |
flaper87 | dhellmann: oh mmh, that sounds like a good idea. | 23:27 |
* harlowja only if someone plays an instrument | 23:27 | |
flaper87 | As long as my name is not the only one in the list, I'm ok with that :D | 23:27 |
harlowja | i'll play the chimes | 23:27 |
dhellmann | flaper87 : put me down, too, as long as someone else takes care of the migration | 23:27 |
flaper87 | obviously, this based on the assumption we'll move to tabulate | 23:27 |
* dhellmann watches everyone say "not it" | 23:28 | |
dims | ouch - i can just listen to you all :) | 23:28 |
flaper87 | ok, I'll do the migration | 23:28 |
*** sigmavirus24 is now known as sigmavirus24_awa | 23:28 | |
dhellmann | dims : some of our "everyone should be able to use these" libs might get more traction outside of openstack if we move them back to github. Just food for thought. | 23:28 |
harlowja | flaper87 so u are signing up to be the owner right | 23:28 |
dims | dhellmann : i meant i don't play any instruments :) | 23:29 |
dims | yep. +1 to the idea | 23:29 |
dhellmann | dims : I can't even clap in rhythm | 23:29 |
dims | lol | 23:29 |
harlowja | aka clapping with style | 23:29 |
flaper87 | harlowja: no no, I'll put your name as owner, dims as vice-owner, jd__ as emergency contact and my number to receive pictures of your faces when shit happens | 23:29 |
harlowja | lol | 23:29 |
harlowja | so your gonna be my adult guardian right | 23:30 |
dims | flaper87 : i haven't even seen the code lol | 23:30 |
harlowja | pretty sure that makes u the owner | 23:30 |
harlowja | i'm just a teenager | 23:30 |
harlowja | lol | 23:30 |
flaper87 | lol | 23:31 |
*** MVenesio has joined #openstack-oslo | 23:37 | |
*** pratikmallya has joined #openstack-oslo | 23:37 | |
*** alejandrito has quit IRC | 23:40 | |
*** MVenesio has quit IRC | 23:42 | |
flaper87 | harlowja: dhellmann dims https://github.com/jazzband/prettytable | 23:43 |
flaper87 | will figure out how this works | 23:43 |
flaper87 | apparently, everyone has access to the repo | 23:43 |
flaper87 | I guess that's fine | 23:43 |
harlowja | merge access to? | 23:44 |
flaper87 | yeah, the default is write access | 23:44 |
harlowja | intersting | 23:44 |
flaper87 | not sure how I feel about this | 23:44 |
harlowja | ya | 23:46 |
harlowja | whats this shellcode doing in there, lol | 23:46 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Move failure resolving into its own class + mapping https://review.openstack.org/266078 | 23:48 |
dhellmann | flaper87, harlowja : we can see how it goes. if it doesn't go well, we can set up a different organization and move it there | 23:48 |
flaper87 | right. FWIW, I'm the only one with access to pypi | 23:49 |
dhellmann | flaper87, harlowja : I wonder if we can get the docs out of the google code wiki somehow? | 23:49 |
flaper87 | the rest is git and we can always revert | 23:49 |
dhellmann | flaper87 : yeah, we can keep that list small | 23:49 |
flaper87 | I think they are already in the github repo. Lemme check | 23:49 |
*** jecarey has quit IRC | 23:49 | |
flaper87 | google code has a tool to move projects to GH and I think it moves docs as well | 23:50 |
dhellmann | oh, cool, so we just need to update the link in the readme | 23:50 |
dhellmann | and then add maintainer info to setup.py | 23:50 |
flaper87 | yup, there's a wiki branch. gotta figure out how to make github use that | 23:50 |
dhellmann | k | 23:50 |
flaper87 | I've updated the maintainer info already, pushing in a bit | 23:50 |
dhellmann | flaper87 : sweet. I'm going off to prep to watch some football. Ping me if you need anything like a code review. | 23:51 |
*** zz_dimtruck is now known as dimtruck | 23:51 | |
flaper87 | ++ | 23:52 |
flaper87 | thanks and enjoy :) | 23:52 |
*** haypo has left #openstack-oslo | 23:52 | |
*** gordc has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!