*** salv-orlando has joined #openstack-oslo | 00:00 | |
*** salv-orl_ has quit IRC | 00:03 | |
*** sdake has quit IRC | 00:07 | |
*** sdake has joined #openstack-oslo | 00:13 | |
harlowja | i'm around | 00:13 |
---|---|---|
*** zz_dimtruck is now known as dimtruck | 00:19 | |
*** mixos has quit IRC | 00:19 | |
dims | harlowja : let's do all the deprecations removals etc and cut a 2.x | 00:20 |
*** EinstCrazy has quit IRC | 00:25 | |
*** dims has quit IRC | 00:42 | |
*** salv-orlando has quit IRC | 00:51 | |
*** EinstCrazy has joined #openstack-oslo | 01:01 | |
*** sdake has quit IRC | 01:10 | |
*** dims has joined #openstack-oslo | 01:16 | |
openstackgerrit | Merged openstack/taskflow: Revert "Remove failure version number" https://review.openstack.org/260164 | 01:19 |
*** sputnik1_ has quit IRC | 01:30 | |
*** sputnik13 has joined #openstack-oslo | 01:35 | |
*** dims has quit IRC | 01:40 | |
*** dimtruck is now known as zz_dimtruck | 01:43 | |
*** mixos has joined #openstack-oslo | 01:48 | |
*** zz_dimtruck is now known as dimtruck | 01:50 | |
*** dims has joined #openstack-oslo | 01:50 | |
*** mriedem_away has quit IRC | 01:53 | |
*** mriedem has joined #openstack-oslo | 01:58 | |
mriedem | this is an annoying new bug in oslo.messaging 4.0.0 https://bugs.launchpad.net/oslo.messaging/+bug/1528415 | 01:59 |
openstack | Launchpad bug 1528415 in oslo.messaging ""ERROR [oslo_messaging._drivers.zmq_driver.zmq_async] ZeroMQ not found!" shows up in other projects and isn't necessary" [Medium,Confirmed] | 01:59 |
harlowja | bogdando ^ | 02:05 |
harlowja | isn't that basically the API that mistral and others care about ... | 02:06 |
harlowja | post things to work on, let something work on them, or die trying to do that | 02:06 |
*** dims has quit IRC | 02:06 | |
*** Juexun has joined #openstack-oslo | 02:08 | |
*** sdake has joined #openstack-oslo | 02:17 | |
*** jeckersb is now known as jeckersb_gone | 02:28 | |
*** dims has joined #openstack-oslo | 02:29 | |
*** bardia has quit IRC | 02:32 | |
harlowja | sooooooo dims | 02:33 |
harlowja | how much u want to know ;) | 02:33 |
dims | harlowja : beat up right now. will you be around tomorrow? | 02:34 |
harlowja | clearance code please | 02:34 |
harlowja | lol | 02:34 |
dims | lol | 02:34 |
* harlowja is just saying that the job board api looks a lot like what is getting thought about for oslo.messaging | 02:34 | |
*** sdake_ has joined #openstack-oslo | 02:34 | |
*** sdake has quit IRC | 02:34 | |
harlowja | 260246 could be that addition to tooz | 02:34 |
harlowja | possibly | 02:34 |
harlowja | get people to think about it at least | 02:35 |
harlowja | whether it merges or not, meh, whatever :-P | 02:35 |
harlowja | its the season of giving, ha | 02:35 |
*** harlowja_at_home has joined #openstack-oslo | 02:39 | |
dims | thanks harlowja : i'll take a look at that | 02:39 |
harlowja | :-P | 02:39 |
harlowja | sputnik1_ i think had some thoughts on it to | 02:39 |
harlowja | if he's around | 02:39 |
harlowja | dims cools | 02:39 |
harlowja | dims https://review.openstack.org/#/c/260164/ | 02:40 |
harlowja | if u get bored, ha | 02:40 |
*** zzzeek has quit IRC | 02:44 | |
*** gcb has joined #openstack-oslo | 02:44 | |
*** dims has quit IRC | 02:45 | |
*** zzzeek has joined #openstack-oslo | 02:45 | |
*** dims has joined #openstack-oslo | 02:47 | |
*** gcb has quit IRC | 02:52 | |
*** salv-orlando has joined #openstack-oslo | 02:55 | |
*** gcb has joined #openstack-oslo | 03:05 | |
*** links has joined #openstack-oslo | 03:06 | |
*** jecarey has joined #openstack-oslo | 03:08 | |
*** dims has quit IRC | 03:08 | |
*** harlowja_at_home has quit IRC | 03:09 | |
*** harlowja_at_home has joined #openstack-oslo | 03:09 | |
*** Juexun_ has joined #openstack-oslo | 03:12 | |
*** Juexun has quit IRC | 03:12 | |
*** yamahata has quit IRC | 03:14 | |
*** harlowja_at_home has quit IRC | 03:14 | |
*** mriedem has quit IRC | 03:19 | |
*** Juexun_ has quit IRC | 03:19 | |
*** vilobhmm111 has quit IRC | 03:21 | |
harlowja | dims soooooooo | 03:22 |
harlowja | thats trickay | 03:22 |
harlowja | lol | 03:22 |
harlowja | https://etherpad.openstack.org/p/mitaka-taskflow-2.0 listed all the dates of deprecations.... blah blah | 03:23 |
harlowja | and i think only allowed to remove things older than 1.5 years (3 cycles?) | 03:23 |
harlowja | 'What got removed:' section in ^ | 03:24 |
*** salv-orlando has quit IRC | 03:35 | |
*** harlowja_at_home has joined #openstack-oslo | 03:39 | |
*** deray has joined #openstack-oslo | 03:54 | |
*** jeckersb_gone is now known as jeckersb | 03:56 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Trap and expose exception any 'args' https://review.openstack.org/237279 | 03:58 |
*** jecarey has quit IRC | 03:59 | |
*** jecarey has joined #openstack-oslo | 04:01 | |
*** zqfan has joined #openstack-oslo | 04:05 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Track dirtiness of models and use it during saving https://review.openstack.org/241441 | 04:19 |
*** nikhil has joined #openstack-oslo | 04:33 | |
*** vilobhmm11 has joined #openstack-oslo | 04:36 | |
*** nikhil_k has quit IRC | 04:36 | |
*** deray has quit IRC | 04:41 | |
*** deray has joined #openstack-oslo | 04:46 | |
openstackgerrit | Merged openstack/oslo.utils: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/259950 | 04:56 |
*** jamespage has quit IRC | 05:00 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/oslo.messaging: zmq: Don't log error when can't import zmq module https://review.openstack.org/260322 | 05:22 |
*** sdake_ has quit IRC | 05:23 | |
*** salv-orlando has joined #openstack-oslo | 05:41 | |
*** harlowja_at_home has quit IRC | 05:42 | |
*** mixos has quit IRC | 05:45 | |
*** yamahata has joined #openstack-oslo | 05:59 | |
*** sputnik13 has quit IRC | 06:12 | |
*** sputnik13 has joined #openstack-oslo | 06:14 | |
*** salv-orlando has quit IRC | 06:16 | |
*** sputnik13 has quit IRC | 06:16 | |
*** sputnik13 has joined #openstack-oslo | 06:18 | |
*** sputnik13 has quit IRC | 06:20 | |
*** sputnik13 has joined #openstack-oslo | 06:22 | |
*** vilobhmm11 has quit IRC | 06:28 | |
openstackgerrit | Merged openstack/oslo.messaging: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/259946 | 06:32 |
*** pasquier-s has quit IRC | 06:34 | |
*** pasquier-s has joined #openstack-oslo | 06:34 | |
*** dimtruck is now known as zz_dimtruck | 06:40 | |
*** sdake has joined #openstack-oslo | 06:58 | |
*** sdake has quit IRC | 07:15 | |
*** shardy has joined #openstack-oslo | 07:25 | |
*** vilobhmm11 has joined #openstack-oslo | 08:05 | |
*** mkovacik_ has joined #openstack-oslo | 08:08 | |
*** e0ne has joined #openstack-oslo | 08:11 | |
*** binarin has quit IRC | 08:24 | |
*** e0ne has quit IRC | 08:28 | |
*** e0ne has joined #openstack-oslo | 08:33 | |
*** yamamoto has joined #openstack-oslo | 08:33 | |
*** yamamoto has quit IRC | 08:36 | |
*** yamamoto has joined #openstack-oslo | 08:36 | |
openstackgerrit | Marian Horban proposed openstack/oslo.service: Refactoring of tests/eventlet_service.py https://review.openstack.org/260384 | 08:38 |
*** salv-orlando has joined #openstack-oslo | 08:40 | |
*** yamamoto has quit IRC | 08:41 | |
openstackgerrit | Marian Horban proposed openstack/oslo.service: Refactoring of tests/eventlet_service.py https://review.openstack.org/260384 | 08:43 |
openstackgerrit | Marian Horban proposed openstack/oslo.service: Graceful shutdown added to ServiceLauncher https://review.openstack.org/260386 | 08:43 |
bogdando | harlowja, should we rework and move the spec for the tooz? | 08:44 |
bogdando | or fork it perhaps | 08:45 |
*** markus_z has joined #openstack-oslo | 08:47 | |
*** andreaf has quit IRC | 08:48 | |
*** andreaf has joined #openstack-oslo | 08:49 | |
*** salv-orl_ has joined #openstack-oslo | 08:51 | |
*** salv-orl_ has quit IRC | 08:51 | |
*** salv-orlando has quit IRC | 08:51 | |
*** salv-orlando has joined #openstack-oslo | 08:52 | |
*** salv-orl_ has joined #openstack-oslo | 08:55 | |
*** salv-orlando has quit IRC | 08:58 | |
openstackgerrit | Chaozhe Chen proposed openstack/oslo.versionedobjects: Remove meaningless default value in __get__() https://review.openstack.org/260394 | 08:58 |
openstackgerrit | Merged openstack/oslo.vmware: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/259952 | 09:02 |
*** jamespage has joined #openstack-oslo | 09:08 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: zmq: Don't log error when can't import zmq module https://review.openstack.org/260322 | 09:09 |
*** yassine__ has joined #openstack-oslo | 09:11 | |
*** e0ne has quit IRC | 09:16 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: zmq: Don't log error when can't import zmq module https://review.openstack.org/260322 | 09:16 |
*** yottatsa has joined #openstack-oslo | 09:19 | |
*** vilobhmm11 has quit IRC | 09:20 | |
*** e0ne has joined #openstack-oslo | 09:20 | |
openstackgerrit | javeme proposed openstack/oslo.log: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/260407 | 09:22 |
*** e0ne has quit IRC | 09:24 | |
openstackgerrit | javeme proposed openstack/oslo.policy: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/260409 | 09:26 |
*** deray has quit IRC | 09:36 | |
openstackgerrit | javeme proposed openstack/oslo.service: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/260412 | 09:37 |
*** yamamoto has joined #openstack-oslo | 09:42 | |
*** EinstCrazy has quit IRC | 09:44 | |
*** yottatsa has quit IRC | 09:44 | |
*** yamamoto has quit IRC | 09:48 | |
*** yottatsa has joined #openstack-oslo | 09:48 | |
*** yamamoto has joined #openstack-oslo | 09:58 | |
*** yottatsa has quit IRC | 10:02 | |
*** cdent has joined #openstack-oslo | 10:07 | |
*** yamamoto has quit IRC | 10:10 | |
*** yamamoto has joined #openstack-oslo | 10:23 | |
*** yamamoto has quit IRC | 10:24 | |
*** yamahata has quit IRC | 10:25 | |
*** e0ne has joined #openstack-oslo | 10:26 | |
openstackgerrit | Elena Ezhova proposed openstack/oslo.service: Run sslutils and wsgi tests for python3 https://review.openstack.org/260432 | 10:27 |
*** mhickey has joined #openstack-oslo | 10:37 | |
*** yottatsa has joined #openstack-oslo | 10:39 | |
*** EinstCrazy has joined #openstack-oslo | 10:54 | |
*** yottatsa has quit IRC | 11:02 | |
*** yamamoto has joined #openstack-oslo | 11:02 | |
*** yamamoto has quit IRC | 11:02 | |
*** yottatsa has joined #openstack-oslo | 11:04 | |
*** gcb has quit IRC | 11:08 | |
*** gcb has joined #openstack-oslo | 11:12 | |
*** shardy has quit IRC | 11:14 | |
*** lucas-dinner is now known as lucasagomes | 11:15 | |
*** deray has joined #openstack-oslo | 11:20 | |
*** gcb has quit IRC | 11:23 | |
*** yottatsa has quit IRC | 11:35 | |
*** yottatsa has joined #openstack-oslo | 11:38 | |
*** salv-orl_ has quit IRC | 11:44 | |
*** lxsli_festive has quit IRC | 11:48 | |
*** lxsli_festive has joined #openstack-oslo | 11:52 | |
openstackgerrit | Zhihai Song proposed openstack/oslo.config: Replace assertEqual with assertTrue and assertFalse https://review.openstack.org/260461 | 12:07 |
*** jaypipes has joined #openstack-oslo | 12:09 | |
openstackgerrit | javeme proposed openstack/oslo.service: fix a couple of assert issues https://review.openstack.org/260412 | 12:09 |
*** yottatsa has quit IRC | 12:17 | |
*** yamamoto has joined #openstack-oslo | 12:19 | |
openstackgerrit | Merged openstack/oslo.service: Updated from global requirements https://review.openstack.org/258106 | 12:19 |
*** gcb has joined #openstack-oslo | 12:22 | |
*** yamamoto has quit IRC | 12:23 | |
*** yamamoto has joined #openstack-oslo | 12:25 | |
*** gordc has joined #openstack-oslo | 12:30 | |
*** EinstCrazy has quit IRC | 12:35 | |
*** ndipanov has joined #openstack-oslo | 12:36 | |
*** EinstCrazy has joined #openstack-oslo | 12:37 | |
openstackgerrit | janonymous proposed openstack/automaton: Put py34 first in the env order of tox https://review.openstack.org/260475 | 12:40 |
*** yamamoto has quit IRC | 12:40 | |
*** pblaho has joined #openstack-oslo | 12:43 | |
*** salv-orlando has joined #openstack-oslo | 12:45 | |
*** yamamoto has joined #openstack-oslo | 12:50 | |
*** sdake has joined #openstack-oslo | 12:54 | |
openstackgerrit | janonymous proposed openstack/oslo.db: Put py34 first in the env order of tox https://review.openstack.org/260489 | 12:55 |
openstackgerrit | janonymous proposed openstack/oslo.vmware: Put py34 first in the env order of tox https://review.openstack.org/260491 | 12:56 |
*** gcb has quit IRC | 12:57 | |
*** yottatsa has joined #openstack-oslo | 13:01 | |
*** dims has joined #openstack-oslo | 13:01 | |
openstackgerrit | Julien Danjou proposed openstack/tooz: etcd: driver with lock support https://review.openstack.org/246879 | 13:01 |
openstackgerrit | janonymous proposed openstack/pycadf: Put py34 first in the env order of tox https://review.openstack.org/260516 | 13:15 |
*** ihrachys has joined #openstack-oslo | 13:17 | |
*** salv-orlando has quit IRC | 13:18 | |
*** salv-orlando has joined #openstack-oslo | 13:18 | |
*** sdake has quit IRC | 13:19 | |
*** amrith is now known as _amrith_ | 13:23 | |
openstackgerrit | Marian Horban proposed openstack/oslo.service: Removed double stopping of service on SIGHUP https://review.openstack.org/258499 | 13:32 |
*** shardy has joined #openstack-oslo | 13:34 | |
*** _amrith_ is now known as amrith | 13:41 | |
*** yamamoto has quit IRC | 13:47 | |
*** yamamoto has joined #openstack-oslo | 13:48 | |
*** yottatsa has quit IRC | 13:57 | |
*** links has quit IRC | 13:59 | |
*** Qwazerty has joined #openstack-oslo | 14:01 | |
*** Qwazerty has left #openstack-oslo | 14:02 | |
*** amrith is now known as _amrith_ | 14:04 | |
mhickey | dims: Hi. Do you have time for a question? | 14:09 |
*** sdake has joined #openstack-oslo | 14:09 | |
*** yottatsa has joined #openstack-oslo | 14:10 | |
*** jeckersb is now known as jeckersb_gone | 14:14 | |
*** zakora has joined #openstack-oslo | 14:16 | |
*** yamamoto has quit IRC | 14:22 | |
*** jeckersb_gone is now known as jeckersb | 14:23 | |
*** markus_z has quit IRC | 14:31 | |
*** mriedem has joined #openstack-oslo | 14:35 | |
*** ihrachys has quit IRC | 14:36 | |
*** links has joined #openstack-oslo | 14:54 | |
dims | ozamiatin : let's get this in to make mriedem happy :) https://review.openstack.org/#/c/260322/ | 14:54 |
dims | mhickey : hi, what's up? | 14:55 |
ozamiatin | dims: agree:) | 14:55 |
mriedem | thanks | 14:55 |
mhickey | dims: Hey. Do you have knowledge on o.vo? | 14:55 |
mriedem | mhickey: dansmith is the main guy and he's out for the year | 14:56 |
dims | mhickey : go ahead and ask :) i'll muddle through it | 14:56 |
mhickey | dims: patch https://review.openstack.org/#/c/258026/ | 14:56 |
dims | y | 14:57 |
mhickey | dims: reying to figure out why class FakeResource needs to be regsitered as an object. Error if set as register_if(False): http://paste.openstack.org/show/482324/ | 14:57 |
mhickey | dims: seems that the rpc pull/push calls obj_class_from_name() which checks the registry | 14:59 |
*** lucasagomes is now known as lucas-hungry | 14:59 | |
dims | mhickey : got a traceback that shows the error? | 15:00 |
dims | mhickey : worst case i can't spot anything quick, we'll have to wait for dansmith | 15:00 |
mhickey | dims: can you see the paste url? | 15:00 |
*** deray has quit IRC | 15:01 | |
dims | oops. looking now | 15:01 |
mhickey | dims: thats ok. I can put dan on the patch. it is not a rush. | 15:01 |
mriedem | mhickey: i had a similar series for cinder https://review.openstack.org/#/q/status:merged+project:openstack/cinder+branch:master+topic:bug/1514926 | 15:02 |
mriedem | might help explain/show how i did it there | 15:02 |
mriedem | since they were registering test objects that didn't need to be registered | 15:02 |
mhickey | mriedem: thanks for this. i will give it a look. | 15:03 |
*** _amrith_ is now known as amrith | 15:04 | |
mhickey | mriedem: I am using the _if(False) but unfortunately the test is looking for the object to be registered when using it. | 15:06 |
*** takedakn has joined #openstack-oslo | 15:07 | |
*** ndipanov has quit IRC | 15:08 | |
mriedem | there is a way to register objects just for a test run i think | 15:09 |
mhickey | mriedem: ok, sounds interesting | 15:10 |
*** sdake has quit IRC | 15:10 | |
mriedem | mhickey: are the neutron tests resetting the registery for each run? https://review.openstack.org/#/c/243826/4/cinder/test.py | 15:12 |
*** takedakn has quit IRC | 15:15 | |
dims | mhickey : try with using "with mock.patch.object(FakeResource, 'obj_class_from_name'):" for your self.rpc.pull - i see that pattern in o.vo tests/test_objects.py (test_obj_primitive_field_namespace) | 15:18 |
*** zigo_ is now known as zigo | 15:19 | |
*** yottatsa has quit IRC | 15:20 | |
mhickey | mriedem: no | 15:21 |
*** thumpba has joined #openstack-oslo | 15:21 | |
*** yottatsa has joined #openstack-oslo | 15:22 | |
mhickey | dims: let me try .. | 15:24 |
mhickey | dims: same error | 15:26 |
*** zz_dimtruck is now known as dimtruck | 15:26 | |
*** jecarey has quit IRC | 15:27 | |
mhickey | mriedem, dims: thanks for the feedback. i think i will wait till dan returns. I am on holidays shortly myself! :) | 15:29 |
mhickey | mriedem, dims: Happy holidays! :) | 15:32 |
mriedem | good idea, don't try tackling this before a vacation, you'll ruin your time off | 15:32 |
mhickey | sure, :) | 15:32 |
dims | mhickey : mriedem : yep, happy holidays! | 15:32 |
*** yottatsa has quit IRC | 15:37 | |
*** yottatsa has joined #openstack-oslo | 15:41 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: (WIP) [zmq] Heartbeat implementation https://review.openstack.org/257346 | 15:44 |
*** kgiusti has joined #openstack-oslo | 15:44 | |
*** jecarey has joined #openstack-oslo | 15:48 | |
*** yamamoto has joined #openstack-oslo | 15:49 | |
*** jecarey has quit IRC | 15:50 | |
*** jecarey has joined #openstack-oslo | 15:51 | |
*** yottatsa has quit IRC | 15:54 | |
openstackgerrit | Merged openstack/oslo.middleware: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/259948 | 16:03 |
*** lucas-hungry is now known as lucasagomes | 16:04 | |
*** mhickey has quit IRC | 16:06 | |
mriedem | dims: what is changbo's irc nick? | 16:06 |
mriedem | gcb i guess | 16:07 |
dims | mriedem : yep | 16:08 |
*** yamamoto has quit IRC | 16:19 | |
*** pblaho has quit IRC | 16:20 | |
*** harlowja_at_home has joined #openstack-oslo | 16:23 | |
openstackgerrit | janonymous proposed openstack/automaton: Put py34 first in the env order of tox ,remove py33 from envlist and python2 from trove classifier https://review.openstack.org/260475 | 16:24 |
*** shardy has quit IRC | 16:24 | |
openstackgerrit | Ronald Bradford proposed openstack/oslo.config: Cleanly handle an undefined attribute reference https://review.openstack.org/260225 | 16:27 |
openstackgerrit | janonymous proposed openstack/automaton: Put py34 first in the env order of tox ,remove py33 from envlist and python2 from trove classifier https://review.openstack.org/260475 | 16:27 |
*** zakora has quit IRC | 16:33 | |
mhorban | Hi guys, could you please look at https://review.openstack.org/#/c/260386/ . Many of services are started by ServiceLauncher or ProcessLauncher depends on configuration.... But ServiceLauncher doesn't support whole list of signals that ProcessLauncher supports. So handlers of signals will be different for nova depends on config option 'workers'. This patch fixes it. | 16:41 |
openstackgerrit | Chaozhe Chen proposed openstack/oslo.versionedobjects: Remove meaningless default value in __get__() https://review.openstack.org/260394 | 16:41 |
*** dims has quit IRC | 16:42 | |
openstackgerrit | ayoung proposed openstack/oslo.policy: Convert Exceptions to failures. https://review.openstack.org/165908 | 16:43 |
*** yamamoto has joined #openstack-oslo | 16:44 | |
*** thumpba_ has joined #openstack-oslo | 16:46 | |
openstackgerrit | Joshua Harlow proposed openstack/debtcollector: Add 'removed_class' class decorator https://review.openstack.org/260117 | 16:46 |
*** thumpba has quit IRC | 16:49 | |
rbradfor | lxsli_festive, I have a few questions on status of your mutable config work. | 16:49 |
*** yamamoto has quit IRC | 16:56 | |
*** yamamoto has joined #openstack-oslo | 16:56 | |
*** yamamoto has quit IRC | 16:58 | |
gordc | harlowja_at_home: quick q, when using debtcollector (specifically @removals.remove) do we need to have this line as well: warnings.simplefilter('always') | 17:07 |
harlowja_at_home | gordc, not needed, if u can use https://docs.python.org/3.1/using/cmdline.html#cmdoption-trace-W thats prefered | 17:08 |
harlowja_at_home | ^ that -W option allows for altering what is showed via cmdline | 17:08 |
harlowja_at_home | for example -Wonce | 17:08 |
harlowja_at_home | all gordc that 'always' does is make it always show things (no matter how many times they are repeated) | 17:09 |
harlowja_at_home | https://docs.python.org/3.1/library/warnings.html#the-warnings-filter | 17:09 |
harlowja_at_home | 'once' is probably better | 17:09 |
gordc | harlowja_at_home: hmm. i guess main reason i ask is i can't figure out why it's throwing an error rather than a warning | 17:09 |
gordc | http://logs.openstack.org/39/256039/2/check/gate-keystonemiddleware-python27/754bb37/testr_results.html.gz | 17:09 |
harlowja_at_home | gordc, ah, thats cause keystone turned those things into errors | 17:09 |
harlowja_at_home | from what i remember they changed warnings to blowup instead of warn | 17:10 |
gordc | oh. fun | 17:10 |
harlowja_at_home | ya | 17:10 |
harlowja_at_home | gordc, https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/tests/unit/utils.py#L29 | 17:10 |
harlowja_at_home | right there i think ;) | 17:10 |
gordc | ah got it. | 17:11 |
harlowja_at_home | thats one way to ensure u don't have deprecation warnings :-P | 17:11 |
gordc | i thought it was because i didn't add the warnings.simplefilter | 17:11 |
*** dims has joined #openstack-oslo | 17:12 | |
*** yamahata has joined #openstack-oslo | 17:12 | |
openstackgerrit | Joshua Harlow proposed openstack/tooz: Add programatic introspection of drivers characteristic(s) https://review.openstack.org/240681 | 17:12 |
harlowja_at_home | gordc, nah, i'm pretty sure thats just cause they are doing that :) | 17:12 |
*** boris-42 has quit IRC | 17:13 | |
gordc | harlowja_at_home: cool cool. thanks for help | 17:14 |
harlowja_at_home | np | 17:14 |
*** harlowja_at_home has quit IRC | 17:18 | |
*** yottatsa has joined #openstack-oslo | 17:19 | |
*** jecarey has quit IRC | 17:21 | |
*** bauzas has joined #openstack-oslo | 17:21 | |
bauzas | heya | 17:21 |
bauzas | I have a change that is trampled in the nova gate because http://paste.openstack.org/show/482530/ | 17:21 |
bauzas | can someone tell me if fixing https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/_drivers/impl_fake.py#L176-L185 is a terrible idea for making sure that datetimes still work over the wire ? | 17:22 |
bauzas | tbc, everything works fine but the functional test because of that fake driver | 17:23 |
*** devananda has quit IRC | 17:30 | |
bauzas | dims: any idea who could help me for the above ? | 17:31 |
dims | ozamiatin : any ideas? ^^ | 17:32 |
dims | bauzas : am on a call, will pay attention in a little bit | 17:32 |
bauzas | dims: np, cool | 17:37 |
*** devananda has joined #openstack-oslo | 17:40 | |
*** e0ne has quit IRC | 17:47 | |
openstackgerrit | Merged openstack/taskflow: Start rename of BLATHER -> TRACE https://review.openstack.org/258752 | 17:48 |
*** yottatsa has quit IRC | 17:49 | |
*** cburgess has quit IRC | 17:50 | |
*** cburgess has joined #openstack-oslo | 17:52 | |
stevemar | gordc: thoughts on what to do about it? cap sounds like the easiest | 17:57 |
dims | stevemar : we broke something in yesterday's releases? | 18:00 |
*** openstackgerrit has quit IRC | 18:02 | |
*** openstackgerrit has joined #openstack-oslo | 18:02 | |
*** yottatsa has joined #openstack-oslo | 18:04 | |
*** yottatsa has quit IRC | 18:06 | |
dims | bauzas : we did not break it with yesterday's release did we? | 18:06 |
dims | bauzas : that thing has not changed for a while | 18:06 |
*** cdent has quit IRC | 18:08 | |
*** yottatsa has joined #openstack-oslo | 18:14 | |
*** links has quit IRC | 18:15 | |
*** yottatsa has quit IRC | 18:16 | |
stevemar | dims: yeah, it's been broken for a while | 18:16 |
stevemar | talking about it in -meeting | 18:16 |
openstackgerrit | Xia Linjuan proposed openstack/oslo.messaging: Clarify the notification_topics configuration https://review.openstack.org/260657 | 18:18 |
*** dimtruck is now known as zz_dimtruck | 18:31 | |
gordc | stevemar: cap pycadf in liberty i guess. i don't understand why your tests turn warnings into errors.lol | 18:32 |
*** harlowja has quit IRC | 18:36 | |
dougwig | dhellmann: hi oslo. as pointed out in a review adding locale stuff, we have a repo neutron-lib, with a module neutron_lib, but the local files go into neutron-lib/, and are the only files in that dir, while other modules have locale with their python code, courtesy of no hyphen. how did that come about? | 18:36 |
*** harlowja has joined #openstack-oslo | 18:36 | |
*** e0ne has joined #openstack-oslo | 18:37 | |
*** vilobhmm11 has joined #openstack-oslo | 18:37 | |
*** jecarey has joined #openstack-oslo | 18:39 | |
dhellmann | dougwig : aejager may know this stuff better than I do, but the locale dir needs to be named for the dist name rather than the python package name, so that's probably why | 18:41 |
dhellmann | dougwig : link to the review? | 18:41 |
*** jecarey has quit IRC | 18:41 | |
*** jecarey has joined #openstack-oslo | 18:41 | |
dougwig | dhellmann: https://review.openstack.org/#/c/257775/ | 18:42 |
*** jecarey has quit IRC | 18:46 | |
dhellmann | dougwig : looking | 18:48 |
*** salv-orlando has quit IRC | 18:48 | |
dougwig | dhellmann: ty | 18:48 |
dhellmann | dougwig : using oslo.messaging as an example, the locale dir should be under a dir named for $(python setup.py --name) which is the dist name | 18:52 |
dhellmann | dougwig : so I think garyk is right, but for the wrong reason | 18:52 |
dhellmann | dougwig : I've asked AJaeger to join us, since he knows this stuff better than I do | 18:53 |
dhellmann | though it might be too late in the day for him | 18:53 |
*** AJaeger has joined #openstack-oslo | 18:54 | |
AJaeger | Hi, dhellmann asked me to join for a translation question. Could somebody recap, please? | 18:54 |
dougwig | dhellmann: he's reviewing in infra like a madman, as usual, so i think he's around. :) | 18:54 |
dhellmann | hi, AJaeger, thanks | 18:54 |
AJaeger | dougwig: me? | 18:54 |
dhellmann | yep | 18:54 |
dhellmann | :-) | 18:54 |
dougwig | AJaeger: 11:36 AM <dougwig> dhellmann: hi oslo. as pointed out in a review adding locale stuff, we have a repo neutron-lib, with a module neutron_lib, but the local files go into neutron-lib/, and are the only files in that dir, while other modules have locale with their python code, courtesy of no hyphen. how did that come about? | 18:54 |
dhellmann | AJaeger: dougwig had a question about https://review.openstack.org/#/c/257775/ ^^ | 18:54 |
dougwig | AJaeger: 11:42 AM <dougwig> dhellmann: https://review.openstack.org/#/c/257775/ | 18:54 |
dhellmann | I replied: dougwig : using oslo.messaging as an example, the locale dir should be under a dir named for $(python setup.py --name) which is the dist name | 18:54 |
AJaeger | dougwig: our infra scripts are written in such a way that they expect the locale file be placed in | 18:55 |
AJaeger | $repo/locale/$repo.pot | 18:55 |
dhellmann | ah, fun, so I'm wrong about the reason | 18:55 |
dougwig | AJaeger: ok, so putting both of your answers together, we expect the dist name to always equal the *repo* name, not the module name? | 18:55 |
AJaeger | dhellmann: you can check project-config, it's in jenkins/jobs/*translation* | 18:55 |
AJaeger | dougwig: repo name it is indeed | 18:56 |
dhellmann | dougwig : the dist and repo name do not need to match | 18:56 |
dougwig | gotcha, ok. | 18:56 |
dhellmann | dougwig : and I was completely wrong about the reasoning, it has nothing to do with dist name | 18:56 |
dougwig | annoying for tab completion in my source dir, but i understand now. thank you. :) | 18:56 |
AJaeger | dougwig: the scripts are complicated enough and stevenk and myself prefer to not make them more complicated with some way to change the repos | 18:56 |
dougwig | dhellmann: ack, thanks. | 18:56 |
AJaeger | but if either of you finds an elegant way to change the infra scripts, I happily review | 18:57 |
dougwig | AJaeger: i was looking to understand, not change. | 18:57 |
AJaeger | and I had hoped somebody wanted to help ;) | 18:57 |
dhellmann | yeah, I'm happy with how things are I just wanted to make sure dougwig had the right info | 18:57 |
dhellmann | aw | 18:57 |
*** ihrachys has joined #openstack-oslo | 18:58 | |
AJaeger | dougwig: you should name the repos properly, so repo name = module name ;) | 18:59 |
dougwig | AJaeger: isn't the norm in python for the repo to be a-b and the module to be a_b ? | 19:00 |
*** zz_dimtruck is now known as dimtruck | 19:00 | |
AJaeger | when translations where done, we only had repos without "-" and "_" ;) | 19:01 |
AJaeger | And then olso changed it's module name from "." to "_" | 19:01 |
AJaeger | and suddenly repos appeared using "-" and wanting translations... | 19:01 |
AJaeger | if we know change "-" to "_" we first need to update all repos... | 19:02 |
AJaeger | s/know/now/ | 19:02 |
*** ianw has quit IRC | 19:06 | |
*** ianw has joined #openstack-oslo | 19:07 | |
*** dimtruck is now known as zz_dimtruck | 19:07 | |
*** vilobhmm11 has quit IRC | 19:09 | |
*** vilobhmm11 has joined #openstack-oslo | 19:09 | |
*** ihrachys has quit IRC | 19:18 | |
*** ozamiatin_ has joined #openstack-oslo | 19:22 | |
*** zqfan has quit IRC | 19:31 | |
*** yassine__ has quit IRC | 19:36 | |
openstackgerrit | Merged openstack/oslo.messaging: zmq: Don't log error when can't import zmq module https://review.openstack.org/260322 | 19:37 |
*** dims has quit IRC | 19:45 | |
*** salv-orlando has joined #openstack-oslo | 19:47 | |
*** lucasagomes is now known as lucas-pto | 19:55 | |
vilobhmm11 | lifeless : ping | 19:56 |
vilobhmm11 | wanted to talk about the consul review if you have few minutes | 19:56 |
*** ihrachys has joined #openstack-oslo | 20:01 | |
*** ihrachys has quit IRC | 20:02 | |
*** ozamiatin_ has quit IRC | 20:03 | |
*** jecarey has joined #openstack-oslo | 20:03 | |
*** dims has joined #openstack-oslo | 20:04 | |
*** AJaeger has left #openstack-oslo | 20:08 | |
*** dims_ has joined #openstack-oslo | 20:11 | |
*** dims has quit IRC | 20:14 | |
*** boris-42 has joined #openstack-oslo | 20:21 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Track dirtiness of models and use it during saving https://review.openstack.org/241441 | 20:24 |
*** e0ne has quit IRC | 20:28 | |
openstackgerrit | Merged openstack/oslo.policy: assertIsNone(val) instead of assertEqual(None,val) https://review.openstack.org/260409 | 20:30 |
*** e0ne has joined #openstack-oslo | 20:32 | |
openstackgerrit | Merged openstack/tooz: Remove tested under 2.6 from docs https://review.openstack.org/259724 | 20:32 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Fix currently broken and inactive mysql tests https://review.openstack.org/260707 | 20:36 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Fix currently broken and inactive mysql tests https://review.openstack.org/260707 | 20:39 |
lifeless | vilobhmm11: hi | 20:41 |
openstackgerrit | Vilobh Meshram proposed openstack/tooz: Add Consul Driver https://review.openstack.org/245362 | 20:43 |
openstackgerrit | Merged openstack/oslo.db: Updated from global requirements https://review.openstack.org/260260 | 20:44 |
vilobhmm11 | lifeless : I didn;t quite understand the last part of your comment here https://review.openstack.org/#/c/245362/18/tooz/drivers/consul.py line #85 | 20:47 |
vilobhmm11 | so wanted to discuss just that I get what you are trying to point out here | 20:47 |
*** salv-orlando has quit IRC | 20:49 | |
lifeless | vilobhmm11: ok | 20:49 |
lifeless | vilobhmm11: so, doing the owner = at the start of the while loop will be ok | 20:49 |
lifeless | vilobhmm11: my other comment was observing that if the put is taking the lock, it shouldn't matter who the owner is | 20:50 |
lifeless | vilobhmm11: the current code does these operations - read, lock, verify-or-loop | 20:51 |
lifeless | vilobhmm11: but there is no need for the read | 20:51 |
lifeless | vilobhmm11: *except* in the case where we already own the lock, right ? | 20:52 |
*** kgiusti has left #openstack-oslo | 20:52 | |
vilobhmm11 | lifeless : if put is taking the lock and if the process/service crashes then won't it be good to know who the owner was before | 20:54 |
lifeless | vilobhmm11: I don't understand | 20:54 |
vilobhmm11 | by read you mean read who's the owner right ? | 20:55 |
lifeless | vilobhmm11: which process, which service? There must be three involved right - consul, and two processes using the thing, lets call them c (for client) c1 and c2 | 20:55 |
vilobhmm11 | yes | 20:55 |
lifeless | vilobhmm11: we read the owner, but we only use that to re-acquire a lock if we already own it. | 20:55 |
lifeless | what benefit is there reading the owner other than that? | 20:56 |
vilobhmm11 | lifeless : it can be used to block if the owener is != the session id of the one who owns the lock | 20:57 |
vilobhmm11 | lifeless : so you mean we need not keep track of the owner information ? | 20:57 |
lifeless | in the case that we acquire it first try | 20:58 |
lifeless | I'm saying ,what if we did this: | 20:58 |
lifeless | put | 20:58 |
lifeless | if acuired: save that and return | 20:58 |
lifeless | read owner | 20:58 |
lifeless | if self: save that and return | 20:58 |
lifeless | while True: | 20:59 |
lifeless | put | 20:59 |
lifeless | if acquired: save and return | 20:59 |
vilobhmm11 | thinking... | 20:59 |
harlowja | yup looks like vilobhmm11 is thinking | 21:00 |
lifeless | or something like that - I'm sure it can be tweaked, but basically - less gets, since we only ever need it once: to re-acquire a lock we previously held | 21:00 |
vilobhmm11 | harlowja : :P | 21:00 |
lifeless | and btw - I don't know if that re-acquire can ever work | 21:00 |
lifeless | since aren't session id's transient | 21:00 |
lifeless | so if c1 crashes, when it starts again it has a new session id and cannot re-acquire the lock | 21:01 |
lifeless | but - I am new to consul, so may be totally wrong :) | 21:01 |
vilobhmm11 | session id's are transient but if they have already been stored in the kv store and then the process crashes then they can be the source of truth for who owned the lock | 21:03 |
lifeless | I don't understand | 21:03 |
lifeless | if I have a random session id | 21:03 |
lifeless | how do I map that back to a real world thing | 21:04 |
lifeless | or put it another way | 21:04 |
lifeless | how can this code ever be True: | 21:04 |
lifeless | if owner == self._session_id | 21:04 |
vilobhmm11 | I take that back…every time a process starts it will have to regenerate its session | 21:04 |
vilobhmm11 | and agree that its transient and will be alive till the lifetime the client is up and running | 21:05 |
*** edmondsw has joined #openstack-oslo | 21:06 | |
*** edmondsw has quit IRC | 21:06 | |
*** amrith is now known as _amrith_ | 21:16 | |
*** davidlenwell has quit IRC | 21:24 | |
*** adam_g has quit IRC | 21:27 | |
lifeless | vilobhmm11: so it sounds like the owner recovery thing won't work and can be deleted ? | 21:27 |
vilobhmm11 | lifeless: makes sense..will edit that..please add coment on patch #19 if you feel anything else needs to be addressed | 21:28 |
lifeless | vilobhmm11: I'll wait for the new code, since its simpler, I'll give it a full review then | 21:30 |
*** adam_g has joined #openstack-oslo | 21:33 | |
*** adam_g has quit IRC | 21:34 | |
*** adam_g has joined #openstack-oslo | 21:34 | |
*** davidlenwell has joined #openstack-oslo | 21:36 | |
*** e0ne has quit IRC | 21:40 | |
*** davidlenwell has quit IRC | 21:43 | |
*** e0ne has joined #openstack-oslo | 21:45 | |
*** adam_g has quit IRC | 21:52 | |
openstackgerrit | Julien Danjou proposed openstack/tooz: etcd: driver with lock support https://review.openstack.org/246879 | 21:54 |
*** davidlenwell has joined #openstack-oslo | 21:55 | |
*** adam_g has joined #openstack-oslo | 21:56 | |
*** adam_g has quit IRC | 21:56 | |
*** adam_g has joined #openstack-oslo | 21:56 | |
*** e0ne has quit IRC | 21:59 | |
*** e0ne has joined #openstack-oslo | 21:59 | |
*** davidlenwell has quit IRC | 22:02 | |
*** adam_g has quit IRC | 22:06 | |
*** adam_g has joined #openstack-oslo | 22:10 | |
*** adam_g has quit IRC | 22:11 | |
*** adam_g has joined #openstack-oslo | 22:11 | |
*** davidlenwell has joined #openstack-oslo | 22:12 | |
*** e0ne has quit IRC | 22:16 | |
*** david-lyle has joined #openstack-oslo | 22:17 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Track dirtiness of models and use it during saving https://review.openstack.org/241441 | 22:19 |
*** pratikmallya has joined #openstack-oslo | 22:22 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Track dirtiness of models and use it during saving https://review.openstack.org/241441 | 22:22 |
*** ndipanov has joined #openstack-oslo | 22:24 | |
*** yamamoto has joined #openstack-oslo | 22:24 | |
*** yamamoto has quit IRC | 22:25 | |
*** mriedem has quit IRC | 22:32 | |
*** jecarey has quit IRC | 22:40 | |
*** jecarey has joined #openstack-oslo | 22:42 | |
openstackgerrit | Merged openstack/taskflow: Fix currently broken and inactive mysql tests https://review.openstack.org/260707 | 22:44 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use the retrying lib. to do basic sqlalchemy engine validation https://review.openstack.org/260743 | 23:15 |
*** dims has joined #openstack-oslo | 23:18 | |
*** dims_ has quit IRC | 23:19 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use the retrying lib. to do basic sqlalchemy engine validation https://review.openstack.org/260743 | 23:21 |
*** harlowja has quit IRC | 23:25 | |
*** harlowja has joined #openstack-oslo | 23:25 | |
*** sputnik1_ has joined #openstack-oslo | 23:35 | |
*** salv-orlando has joined #openstack-oslo | 23:47 | |
*** ChuckC has joined #openstack-oslo | 23:49 | |
openstackgerrit | Joshua Harlow proposed openstack/debtcollector: Add 'removed_class' class decorator https://review.openstack.org/260117 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!