amrith | harlowja, I can confirm that with oslo.utils 2.6.0 installed in my venv, I hit this infinte loop in trove. Picking just the excutils.py from the change you mentioned and dropping it in .tox/py27/lib/python2.7/site-packages/oslo_utils/ fixes the problem. | 00:00 |
---|---|---|
amrith | dims_, ^^ | 00:00 |
amrith | peterstac ^^ | 00:00 |
harlowja | cool | 00:00 |
harlowja | we have liftoff | 00:00 |
amrith | when can we have the fix | 00:00 |
amrith | are we there yet? | 00:00 |
amrith | are we there yet? | 00:00 |
amrith | ;) | 00:00 |
harlowja | lol | 00:00 |
amrith | thanks a lot dims_ harlowja ... | 00:00 |
harlowja | np | 00:00 |
harlowja | as for timeline, dims_ can pick that, i just work here, ha | 00:01 |
dims_ | harlowja: did you check the original problem why we added the code? | 00:01 |
harlowja | afaik it was the failure of exception ----> str | 00:01 |
harlowja | or failure to do the logging | 00:02 |
amrith | once we get past this one, I need to understand this error that Trove spits out ... "RuntimeWarning: It is highly recommended that when eventlet is used that the ['thread'] modules are monkey patched when using the 'oslo.messaging eventlet executor' (to avoid spurious or unexpected lock-ups and/or hangs) | 00:02 |
amrith | " | 00:02 |
harlowja | but the code that added/changed to handle those failures also captured the time.sleep in the try,except blok | 00:02 |
amrith | I hate self modifying code. | 00:02 |
harlowja | amrith well i can tell u why :-P | 00:02 |
amrith | thanks folks, will be back in a couple of minutes with more beer. anybody want anything. | 00:02 |
harlowja | https://github.com/openstack/oslo.utils/blob/master/oslo_utils/eventletutils.py#L54 | 00:02 |
harlowja | amrith ^ thats why, ha | 00:03 |
harlowja | https://github.com/openstack/oslo.utils/blob/master/oslo_utils/eventletutils.py#L111 | 00:03 |
harlowja | lol | 00:03 |
amrith | ;) | 00:03 |
amrith | lol | 00:03 |
harlowja | not patching thread is imho dangerous | 00:03 |
harlowja | to easy to bork all the things | 00:03 |
amrith | yes, I worked at a place once where someone reported a panic caused by some complicated line of code (in C) which looked like x = ptr->foo | 00:03 |
harlowja | mixing 2 locking systems == bad :( | 00:03 |
amrith | and the brilliang enginner who proposed a fix made it this ... | 00:03 |
amrith | if (ptr) | 00:04 |
harlowja | i'd be interested in knowing why thread was not monkey patched | 00:04 |
amrith | x = ptr->foo | 00:04 |
harlowja | because without monkey patching threads, u can have 2 types of locking systems that don't work together at the same time (pthreads based one, and greenlet/eventlet based one) | 00:04 |
amrith | harlowja, the code in question is something that I have feared and on friday evenings I offer the sacrifice of a lamb near a copy of the printout. it is pure black magic. | 00:04 |
harlowja | lol | 00:04 |
dims_ | :) | 00:05 |
harlowja | more beer amrith , get lots of confidence | 00:05 |
harlowja | lol | 00:05 |
amrith | but armed with more beer I will try and get some code checked in today and then look into that eventlet simulator code we have | 00:05 |
harlowja | ha | 00:05 |
harlowja | def | 00:05 |
dims_ | harlowja: ack on the exception->str | 00:05 |
amrith | if some is good, more is better, no? | 00:05 |
amrith | bbiab | 00:05 |
harlowja | dims_ so that review fixes the try, catch block being overeager and uses oslo timeutils | 00:05 |
dims_ | harlowja: cool thanks | 00:06 |
openstackgerrit | Joshua Harlow proposed openstack/oslo.utils: Just use 'exception_to_unicode' to handle exception to string https://review.openstack.org/235021 | 00:07 |
harlowja | dims_ one could say ^ is a followup to that also | 00:08 |
harlowja | because 'exception_to_unicode' is supposed to handle turning exception object into string safely | 00:08 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/234674 | 00:08 |
harlowja | *which avoids the whole translation to exception problem also | 00:08 |
harlowja | *translation of exception -> string problem | 00:08 |
*** zz_dimtruck is now known as dimtruck | 00:09 | |
harlowja | anyways, merge whichever one, or all of them, or none of them, ha | 00:09 |
*** achanda has quit IRC | 00:21 | |
*** thumpba has quit IRC | 00:22 | |
*** achanda has joined #openstack-oslo | 00:25 | |
*** eliqiao has joined #openstack-oslo | 00:26 | |
*** edmondsw has quit IRC | 00:29 | |
*** thumpba has joined #openstack-oslo | 00:30 | |
*** vilobhmm11 has quit IRC | 00:31 | |
*** thumpba has quit IRC | 00:34 | |
*** eliqiao has quit IRC | 00:43 | |
*** dimtruck is now known as zz_dimtruck | 00:43 | |
*** jecarey has joined #openstack-oslo | 00:45 | |
*** jecarey has quit IRC | 00:49 | |
*** eliqiao has joined #openstack-oslo | 00:50 | |
*** browne has quit IRC | 00:58 | |
*** thumpba has joined #openstack-oslo | 01:03 | |
*** thumpba has quit IRC | 01:12 | |
*** thumpba has joined #openstack-oslo | 01:27 | |
*** thumpba has quit IRC | 01:43 | |
*** links has joined #openstack-oslo | 01:46 | |
*** thumpba has joined #openstack-oslo | 01:50 | |
*** zz_dimtruck is now known as dimtruck | 01:58 | |
*** boris-42 has quit IRC | 02:08 | |
*** stevemar_jays is now known as stevemar_ | 02:15 | |
*** yamahata has quit IRC | 02:21 | |
*** jecarey has joined #openstack-oslo | 02:46 | |
*** jecarey has quit IRC | 02:50 | |
*** browne has joined #openstack-oslo | 02:52 | |
*** harlowja_at_home has joined #openstack-oslo | 02:55 | |
*** boris-42 has joined #openstack-oslo | 02:58 | |
*** jecarey has joined #openstack-oslo | 03:05 | |
openstackgerrit | Merged openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/234674 | 03:09 |
*** dimtruck is now known as zz_dimtruck | 03:11 | |
*** itisha has joined #openstack-oslo | 03:16 | |
*** dims_ has quit IRC | 03:18 | |
*** jecarey has quit IRC | 03:35 | |
*** jecarey has joined #openstack-oslo | 03:36 | |
*** jecarey has quit IRC | 03:40 | |
*** sputnik13 has quit IRC | 04:14 | |
*** dims has joined #openstack-oslo | 04:18 | |
*** harlowja_at_home has quit IRC | 04:22 | |
*** dims has quit IRC | 04:24 | |
openstackgerrit | Merged openstack/taskflow: Use the node built-in 'dfs_iter' instead of recursion https://review.openstack.org/205731 | 04:27 |
*** achanda has quit IRC | 04:32 | |
*** stevema__ has joined #openstack-oslo | 04:33 | |
*** stevemar_ has quit IRC | 04:33 | |
*** jecarey has joined #openstack-oslo | 04:36 | |
*** jecarey has quit IRC | 04:42 | |
*** amotoki has quit IRC | 04:57 | |
*** amotoki_ has joined #openstack-oslo | 04:57 | |
*** dims has joined #openstack-oslo | 05:21 | |
*** achanda has joined #openstack-oslo | 05:25 | |
*** thumpba has quit IRC | 05:26 | |
*** dims has quit IRC | 05:26 | |
*** nkrinner has joined #openstack-oslo | 05:35 | |
*** thumpba has joined #openstack-oslo | 05:36 | |
*** thumpba has quit IRC | 05:36 | |
*** thumpba has joined #openstack-oslo | 05:37 | |
*** thumpba has quit IRC | 05:38 | |
openstackgerrit | XiaBing Yao proposed openstack/oslo.config: tuning the indent of comment https://review.openstack.org/235096 | 05:45 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: batch notification listener https://review.openstack.org/234716 | 05:46 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Ensure node 'remove' and 'disassociate' can not be called when frozen https://review.openstack.org/235097 | 05:47 |
*** zqfan has joined #openstack-oslo | 05:55 | |
*** vilobhmm11 has joined #openstack-oslo | 06:12 | |
*** dims has joined #openstack-oslo | 06:22 | |
*** rohit_ has quit IRC | 06:24 | |
*** dims has quit IRC | 06:27 | |
*** amotoki has joined #openstack-oslo | 06:32 | |
*** amotoki has quit IRC | 06:32 | |
*** abhishekk has joined #openstack-oslo | 06:34 | |
abhishekk | hi, I am using graduate.sh script of oslo-incubator to create new library but it is giving error while removing openstack directory as it contains openstack/common/_i18.py file | 06:35 |
abhishekk | is this a bug or I am missing something? | 06:35 |
*** jecarey has joined #openstack-oslo | 06:39 | |
*** jamielennox is now known as jamielennox|away | 06:39 | |
*** dulek has joined #openstack-oslo | 06:44 | |
*** jecarey has quit IRC | 06:44 | |
*** abhishekk has quit IRC | 06:47 | |
*** abhishekk has joined #openstack-oslo | 06:49 | |
*** browne has quit IRC | 07:15 | |
*** browne has joined #openstack-oslo | 07:18 | |
*** ozamiatin has joined #openstack-oslo | 07:19 | |
*** browne has quit IRC | 07:23 | |
*** achanda has quit IRC | 07:27 | |
*** stevema__ has quit IRC | 07:32 | |
*** stevemar_ has joined #openstack-oslo | 07:33 | |
*** stevemar_ has quit IRC | 07:35 | |
*** achanda has joined #openstack-oslo | 07:36 | |
*** achanda has quit IRC | 07:37 | |
*** browne has joined #openstack-oslo | 07:39 | |
*** achanda has joined #openstack-oslo | 07:39 | |
*** itisha has quit IRC | 07:41 | |
openstackgerrit | Merged openstack/oslo.config: Add missing tests and generator code for IPOpt https://review.openstack.org/231771 | 07:49 |
*** amotoki_ has quit IRC | 07:55 | |
openstackgerrit | Merged openstack/oslo.messaging: Provide the executor 'wait' function a timeout and use it https://review.openstack.org/204883 | 08:02 |
*** achanda has quit IRC | 08:03 | |
openstackgerrit | Eric Brown proposed openstack/oslo.config: Add PortOpt for integer with range 1 to 65535 https://review.openstack.org/231257 | 08:05 |
*** markus_z has joined #openstack-oslo | 08:05 | |
*** amotoki has joined #openstack-oslo | 08:06 | |
*** browne has quit IRC | 08:08 | |
*** yassine has joined #openstack-oslo | 08:08 | |
*** achanda has joined #openstack-oslo | 08:12 | |
*** shardy has joined #openstack-oslo | 08:12 | |
*** dims has joined #openstack-oslo | 08:25 | |
*** pballand has quit IRC | 08:26 | |
*** dims has quit IRC | 08:30 | |
*** jecarey has joined #openstack-oslo | 08:40 | |
*** amotoki has quit IRC | 08:43 | |
*** jecarey has quit IRC | 08:44 | |
*** amotoki has joined #openstack-oslo | 08:53 | |
*** ozamiatin_ has joined #openstack-oslo | 08:56 | |
*** ozamiatin has quit IRC | 08:57 | |
*** cdent has joined #openstack-oslo | 08:59 | |
*** exploreshaifali has joined #openstack-oslo | 08:59 | |
*** amotoki has quit IRC | 09:03 | |
*** pballand has joined #openstack-oslo | 09:03 | |
*** vilobhmm11 has quit IRC | 09:11 | |
*** achanda has quit IRC | 09:15 | |
abhishekk | hi, I am using graduate.sh script of oslo-incubator to create new library but it is giving error while removing openstack directory as it contains openstack/common/_i18.py file | 09:16 |
abhishekk | is this a bug or I am missing something? | 09:16 |
*** e0ne has joined #openstack-oslo | 09:26 | |
*** dims has joined #openstack-oslo | 09:27 | |
*** dims has quit IRC | 09:32 | |
*** achanda has joined #openstack-oslo | 09:34 | |
openstackgerrit | Abhishek Kekane proposed openstack/oslo-specs: graduate-oslo-apiclient https://review.openstack.org/235200 | 09:39 |
*** ihrachys has joined #openstack-oslo | 09:46 | |
*** links has quit IRC | 09:47 | |
*** exploreshaifali has quit IRC | 09:48 | |
*** exploreshaifali has joined #openstack-oslo | 09:51 | |
*** dims has joined #openstack-oslo | 09:51 | |
*** achanda has quit IRC | 09:55 | |
*** yassine has quit IRC | 09:57 | |
*** pkholkin has joined #openstack-oslo | 09:59 | |
openstackgerrit | Victor Stinner proposed openstack/oslo.config: Use versionadded and versionchanged in doc https://review.openstack.org/235232 | 10:21 |
openstackgerrit | Victor Stinner proposed openstack/oslo.config: Use versionadded and versionchanged in doc https://review.openstack.org/235232 | 10:23 |
*** pballand has quit IRC | 10:24 | |
*** stevemar_ has joined #openstack-oslo | 10:34 | |
*** stevemar_ has quit IRC | 10:38 | |
*** jecarey has joined #openstack-oslo | 10:41 | |
*** alexpilotti has joined #openstack-oslo | 10:41 | |
*** jecarey has quit IRC | 10:45 | |
dims | mdbooth: good morning/afternoon | 10:51 |
mdbooth | dims: Hey! | 10:51 |
* mdbooth is cooking with gas this morning | 10:51 | |
dims | what do you use usually? :) | 10:51 |
mdbooth | Seriously, though, testscenarios is a pita | 10:51 |
*** pballand has joined #openstack-oslo | 10:53 | |
openstackgerrit | Merged openstack/oslo.utils: Use a stopwatch in 'forever_retry_uncaught_exceptions' https://review.openstack.org/230619 | 10:56 |
mdbooth | Debugging C is so much easier than debugging python :/ | 11:01 |
dims | mdbooth: can i get a peek at your diff? | 11:01 |
openstackgerrit | Tom Cammann proposed openstack/oslo.config: Allow naming of custom types https://review.openstack.org/221394 | 11:01 |
*** zz_dimtruck is now known as dimtruck | 11:02 | |
mdbooth | dims: It's in an extremely messy state right now because, without a debugger, I've had to fling random crap all over the place | 11:02 |
*** haypo has joined #openstack-oslo | 11:02 | |
mdbooth | But sure | 11:02 |
haypo | hi. i want to enforce documentation of changes, i wrote an email to openstack-dev to explain that | 11:03 |
haypo | https://review.openstack.org/#/c/235232/ is a first step | 11:03 |
haypo | i also voted -1 on oslo.config which don't document changes with versionadded/versionchanged | 11:03 |
mdbooth | dims: Also, I'm currently in the middle of 'try random shit until something produces output' | 11:04 |
mdbooth | dims: With all those caveats: | 11:04 |
mdbooth | http://paste.fedoraproject.org/279441/44907074 | 11:04 |
mdbooth | I did actually have a version which passed my initial target set of tests, btw | 11:05 |
mdbooth | But I was unsatisfied with it | 11:05 |
mdbooth | The fact that we only implement any locking for a non-blocking executor is weird | 11:06 |
mdbooth | We should be able to always lock our local state | 11:07 |
mdbooth | However, that causes the tests to hang without producing any output | 11:07 |
mdbooth | So I don't know what's hanging | 11:07 |
mdbooth | I strongly suspect, though, that it will reveal a broken test | 11:08 |
*** ihrachys has quit IRC | 11:08 | |
mdbooth | I already fixed 1 broken test | 11:08 |
openstackgerrit | Tom Cammann proposed openstack/oslo.config: Raise exception when wrong type is used as default https://review.openstack.org/222078 | 11:10 |
mdbooth | Every time I try to do anything new in Python I find I hate it a little more... | 11:10 |
openstackgerrit | Merged openstack/oslo.utils: Just use 'exception_to_unicode' to handle exception to string https://review.openstack.org/235021 | 11:20 |
openstackgerrit | Tom Cammann proposed openstack/oslo.config: Enable inline comments https://review.openstack.org/230411 | 11:23 |
*** ozamiatin_ has quit IRC | 11:26 | |
*** alexpilotti has quit IRC | 11:33 | |
dims | mdbooth: i am all for a state machine, but what exactly happens right now in the code that is bad? what sequence of calls do not work as expected? | 11:34 |
mdbooth | I think I might have the problem | 11:34 |
mdbooth | I think it's another test bug | 11:34 |
mdbooth | Right now the code hangs with no indication of where or during what test, producing no output | 11:35 |
mdbooth | The underlying cause seems to be calling executor.start() twice on the same blocking executor | 11:35 |
mdbooth | The underlying cause for that seems to be mocking | 11:36 |
mdbooth | Specifically in test_server_wait_method | 11:36 |
mdbooth | Interestingly, the testsuite never times out the test | 11:36 |
mdbooth | So, the value of server._executor is part of the state machine | 11:37 |
mdbooth | The state of the state machine is (_executor, _executor_state) | 11:38 |
mdbooth | well, it's state is | 11:38 |
mdbooth | s/it's/its/ | 11:38 |
mdbooth | However, that test pokes something into _executor | 11:38 |
mdbooth | Which breaks it | 11:38 |
mdbooth | You'd thing that just setting _executor_cls to mock.MagicMock would fix it | 11:39 |
mdbooth | But unfortunately MagicMock interprets its init arguments as including a spec | 11:39 |
* mdbooth is about to subclass MagicMock and have it entirely ignore its init arguments | 11:40 | |
dims | mdbooth: sorry i still don't understand what the problem in current code you are trying to solve | 11:49 |
mdbooth | dims: Never mind, I'll doc it properly when it's done :) | 11:50 |
mdbooth | But in short, the current code holds a lock whilst calling something else which holds a different lock | 11:50 |
mdbooth | Which produces a deadlock | 11:50 |
mdbooth | But it works round it by occasionally timing out | 11:50 |
mdbooth | Better to not hold the lock | 11:51 |
dims | you are talking about self._state_cond? | 11:51 |
mdbooth | The state machine is to allow the caller to call in any order | 11:51 |
mdbooth | Yeah | 11:51 |
mdbooth | Basically, you can't call executor whilst holding _state_cond | 11:51 |
mdbooth | Unless you occasionally time out | 11:51 |
dims | that's initialized only once | 11:51 |
mdbooth | which is nasty and slow | 11:51 |
mdbooth | I'll doc it when I'm done | 11:52 |
dims | mdbooth: i am inclined to ship what we have to unblock folks for now | 11:53 |
mdbooth | It already merged, right? | 11:53 |
mdbooth | That's why I'm in no rush | 11:53 |
*** Qwazerty has quit IRC | 11:53 | |
*** Qwazerty has joined #openstack-oslo | 11:53 | |
dims | i was trying to hold off a release pending your investigation. | 11:53 |
mdbooth | The existing patch also introduces a regression | 11:54 |
dims | mdbooth: i'll let you research in your own pace :) | 11:54 |
mdbooth | which is that wait() might not wait | 11:54 |
dims | mdbooth: under which condition? | 11:54 |
mdbooth | If you call it twice | 11:54 |
mdbooth | The second one won't wait | 11:55 |
mdbooth | Without the current patch, both would wait | 11:55 |
dims | if there are 2 wait() the 2nd wait() won't wait. i think that's perfectly fine as long as it does not cleanup the listener | 11:56 |
mdbooth | Ok, cool | 11:56 |
sileht | dims, yes but the application will think that all pending are done, and this is wrong | 11:56 |
mdbooth | My patch is based on that patch, anyway | 11:56 |
sileht | pending/peding thread | 11:56 |
dims | sileht: assuming these 2 wait() are coming from different threads, at least one thread is still waiting | 11:57 |
mdbooth | Sigh, how did it get to 1pm! | 11:57 |
dims | sileht: i am fine with rewriting it with a state machine, we are already broken, so i'd get what we have out and make it better when mdbooth is ready | 11:58 |
dims | sileht: agree? | 11:58 |
sileht | dims, my fix that was using lock+sleep was less complicated to understand and doesn't introduce issue | 11:59 |
* mdbooth doesn't have enough context to make that call, but I appreciate that adding a bug to fix a worse one might be expedient | 11:59 | |
sileht | lock+sleep can always be converted in threading.condition | 11:59 |
sileht | the state machine already exists this is the boolean _running that have two states | 12:00 |
sileht | we don't really need more | 12:00 |
mdbooth | sileht: That's where I started, but it turns out you do | 12:00 |
mdbooth | There are a bunch of reasons, but one of them is that state wraps when you call wait() | 12:01 |
mdbooth | And there are tests which actually do this | 12:01 |
mdbooth | The state machine is mostly to enforce ordering, though | 12:01 |
mdbooth | which is new functionality | 12:01 |
mdbooth | It'll be a lot simpler to understand when I document it properly :) | 12:02 |
* sileht is always afraid by code that need documentation to be understanded | 12:03 | |
mdbooth | documentation == comments :) | 12:03 |
mdbooth | A nice big comment block | 12:03 |
mdbooth | It'll just explain how to read it | 12:03 |
dims | mdbooth: you should rewrite using automaton :) | 12:03 |
mdbooth | dims: Now I have been looking at that lately for nova :) | 12:03 |
mdbooth | It occurred to me | 12:03 |
mdbooth | However, I don't think automaton is required here | 12:04 |
mdbooth | Also, I don't think it allows you to say is state X after state Y? | 12:04 |
mdbooth | Because of the vagiaries of scheduling, you can't be sure that another thread hasn't moved state on several times while you were sleeping | 12:05 |
*** e0ne has quit IRC | 12:06 | |
mdbooth | dims: I'm convinced somebody said automaton is already in use somewhere | 12:07 |
mdbooth | However, I couldn't find it | 12:07 |
mdbooth | I was looking for any existing usage conventions | 12:07 |
mdbooth | If you know any project which is using it I'd be very interested | 12:07 |
dims | y we can ask harlowja those questions | 12:08 |
abhishekk | hi, I am using graduate.sh script of oslo-incubator to create new library but it is giving error while removing openstack directory as it contains openstack/common/_i18.py file | 12:11 |
abhishekk | is this a bug or I am missing something? | 12:11 |
*** ihrachys has joined #openstack-oslo | 12:14 | |
*** eliqiao has quit IRC | 12:23 | |
*** dimtruck is now known as zz_dimtruck | 12:23 | |
*** amotoki has joined #openstack-oslo | 12:26 | |
mdbooth | Which thread is supposed to execute a controller's event loop? | 12:26 |
*** zz_dimtruck is now known as dimtruck | 12:31 | |
*** kgiusti has joined #openstack-oslo | 12:35 | |
*** pradk has joined #openstack-oslo | 12:35 | |
*** stevemar_ has joined #openstack-oslo | 12:35 | |
*** ozamiatin_ has joined #openstack-oslo | 12:36 | |
*** stevemar_ has quit IRC | 12:38 | |
openstackgerrit | Victor Stinner proposed openstack/oslo.serialization: Use versionadded and versionchanged in doc https://review.openstack.org/235297 | 12:39 |
openstackgerrit | Dmitriy Ukhlov proposed openstack/oslo.messaging: Implements rabbit-pika driver https://review.openstack.org/229985 | 12:51 |
*** dimtruck is now known as zz_dimtruck | 12:53 | |
*** regXboi has joined #openstack-oslo | 12:54 | |
mdbooth | Grr! I didn't see how those tests could possibly be related... | 12:59 |
mdbooth | and they're not | 13:00 |
*** amotoki has quit IRC | 13:00 | |
mdbooth | This is the amqp sasl failure | 13:00 |
mdbooth | dims: Well, the unit tests now pass as much with as without the patch :) | 13:02 |
mdbooth | In that there's 1 failure | 13:03 |
*** rlrossit has joined #openstack-oslo | 13:05 | |
haypo | what is this function import_versioned_module() in oslo.utils? | 13:08 |
haypo | module = 'oslo.v%s' % version | 13:08 |
haypo | if submodule: | 13:08 |
haypo | module = '.'.join((module, submodule)) | 13:08 |
haypo | return import_module(module) | 13:08 |
haypo | what is oslo.v<version> ? | 13:08 |
*** gordc has joined #openstack-oslo | 13:10 | |
*** amotoki has joined #openstack-oslo | 13:12 | |
*** jecarey has joined #openstack-oslo | 13:14 | |
*** jecarey has quit IRC | 13:14 | |
*** jecarey has joined #openstack-oslo | 13:15 | |
openstackgerrit | Alexey Yelistratov proposed openstack/oslo.messaging: Implement retries for CALL https://review.openstack.org/234243 | 13:15 |
*** jecarey has quit IRC | 13:16 | |
*** jecarey has joined #openstack-oslo | 13:16 | |
openstackgerrit | Alexey Yelistratov proposed openstack/oslo.messaging: Implement retries for CALL https://review.openstack.org/234243 | 13:20 |
*** fultonj has joined #openstack-oslo | 13:23 | |
*** jecarey has quit IRC | 13:24 | |
*** rohit_ has joined #openstack-oslo | 13:32 | |
*** mriedem has joined #openstack-oslo | 13:35 | |
ozamiatin_ | dims: hi | 13:35 |
dims | ozamiatin_: what's up? | 13:36 |
ozamiatin_ | dims: could you please try devstack trusty with this localrc http://paste.openstack.org/show/476359/ on your workstation locally? | 13:36 |
dims | mdbooth: let me know when you throw a review up | 13:36 |
mdbooth | dims: About to throw something over the wall because I'm going to be late for an appt :/ | 13:37 |
ozamiatin_ | dims: just deploy with that local.conf and run full tempest | 13:37 |
dims | ozamiatin_: yes of course, what do you see? | 13:37 |
mdbooth | dims: Don't have time to write commit message | 13:37 |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Fix locking and in oslo_messaging.server https://review.openstack.org/235347 | 13:37 |
dims | mdbooth: "[WIP] state machine for server.py" is enough | 13:37 |
ozamiatin_ | dims: It should pass, if not i'd like to know how many tests will fail | 13:37 |
mdbooth | Yeah, can -W | 13:38 |
mdbooth | But not now | 13:38 |
mdbooth | later | 13:38 |
dims | ozamiatin_: ack. how exactly do you run tempest? i can run the same commands | 13:38 |
ozamiatin_ | tox -efull in tempest directory after deploy | 13:38 |
dims | ozamiatin_: ack | 13:39 |
ozamiatin_ | dims: thanks | 13:39 |
dims | ozamiatin_: are you seeing results different than what's on the "gate-oslo.messaging-dsvm-functional-zeromq" job? | 13:39 |
dims | oops wrong job name | 13:40 |
dims | ozamiatin_: "gate-tempest-dsvm-full-zmq" | 13:40 |
*** jecarey has joined #openstack-oslo | 13:40 | |
*** jecarey has quit IRC | 13:40 | |
*** jecarey has joined #openstack-oslo | 13:41 | |
ozamiatin_ | dims: yeah, we have ~130 failing tests in the gate but locally 0 failed ) | 13:41 |
dims | ozamiatin_: did you see the failures in say screen-n-api.txt.gz? | 13:47 |
ozamiatin_ | dims: in the gate? | 13:48 |
dims | y | 13:48 |
ozamiatin_ | dims: good idea, now I see that it takes the old driver probably from the liberty branch or so ... | 13:49 |
ozamiatin_ | dims: http://logs.openstack.org/20/232720/12/check/gate-tempest-dsvm-full-zmq/7fd350e/logs/screen-n-cpu.txt.gz | 13:50 |
ozamiatin_ | dims: new driver doesn't have such messages | 13:50 |
dims | ozamiatin_: guess what? http://logs.openstack.org/58/233258/3/check/gate-tempest-dsvm-full-zmq/02b2482/logs/pip-freeze.txt.gz shows something installed "oslo.messaging==2.5.0" | 13:50 |
ozamiatin_ | dims (( | 13:51 |
dims | http://logs.openstack.org/58/233258/3/check/gate-tempest-dsvm-full-zmq/02b2482/logs/devstacklog.txt.gz#_2015-10-09_23_00_20_781 | 13:52 |
ozamiatin_ | dims: looks good | 13:53 |
*** jecarey has quit IRC | 13:54 | |
*** jecarey has joined #openstack-oslo | 13:54 | |
dims | ozamiatin_: LIBS_FROM_GIT=oslo.messaging is somehow missing | 13:54 |
dims | ozamiatin_: i need to figure out project-config magic | 13:57 |
*** zz_dimtruck is now known as dimtruck | 13:57 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:58 | |
ozamiatin_ | dims: ok, it is magic indeed ) | 13:58 |
openstackgerrit | Roman Podoliaka proposed openstack/oslo.db: Refactor deps to use extras and env markers. https://review.openstack.org/184328 | 13:58 |
ozamiatin_ | dims: todays result: http://paste.openstack.org/show/476367/ | 13:58 |
*** jecarey has quit IRC | 13:59 | |
*** mriedem has left #openstack-oslo | 13:59 | |
*** mriedem has joined #openstack-oslo | 14:00 | |
*** sputnik13 has joined #openstack-oslo | 14:00 | |
*** dimtruck is now known as zz_dimtruck | 14:01 | |
*** zz_dimtruck is now known as dimtruck | 14:02 | |
*** ndipanov has quit IRC | 14:04 | |
*** stevemar_ has joined #openstack-oslo | 14:10 | |
gordc | dims: you have your requirements-core powers yet? you think you can take a look at this to unblock ceilometer gate (and whatever else gates against oslo.messaging) https://review.openstack.org/#/c/235047/ | 14:11 |
*** sputnik13 has quit IRC | 14:12 | |
dims | gordc: does it block liberty too? | 14:12 |
gordc | dims: probably. if liberty is uncapped. | 14:13 |
gordc | basically latest oslo.messaging 2.6.1 is broken (it doesn't shut down listeners correctly) | 14:13 |
dims | gordc: let's do this in relmgr-office channel | 14:13 |
openstackgerrit | Victor Stinner proposed openstack/oslo.utils: Fix usage of "deprecated" markup in docstrings https://review.openstack.org/235375 | 14:13 |
gordc | dims: ack | 14:14 |
openstackgerrit | Victor Stinner proposed openstack/oslo.utils: Write document for each unit of oslo_utils.utils https://review.openstack.org/235378 | 14:14 |
*** exploreshaifali has quit IRC | 14:20 | |
*** DuncanT has quit IRC | 14:21 | |
*** jecarey has joined #openstack-oslo | 14:23 | |
*** DuncanT has joined #openstack-oslo | 14:24 | |
openstackgerrit | Victor Stinner proposed openstack/oslo.utils: Use versionadded and versionchanged in doc https://review.openstack.org/235386 | 14:29 |
openstackgerrit | Victor Stinner proposed openstack/oslo.utils: Use versionadded and versionchanged in doc https://review.openstack.org/235386 | 14:33 |
*** pballand has quit IRC | 14:34 | |
*** dims has quit IRC | 14:41 | |
*** dims has joined #openstack-oslo | 14:42 | |
*** lbragstad has quit IRC | 14:44 | |
*** dims_ has joined #openstack-oslo | 14:45 | |
*** lbragstad has joined #openstack-oslo | 14:46 | |
*** lbragstad has quit IRC | 14:48 | |
*** dims has quit IRC | 14:48 | |
openstackgerrit | Marian Horban proposed openstack/oslo.service: Class ReloadNotifier added https://review.openstack.org/228892 | 14:49 |
*** lbragstad has joined #openstack-oslo | 14:53 | |
*** lbragstad has quit IRC | 14:54 | |
*** lbragstad has joined #openstack-oslo | 14:56 | |
*** jecarey has quit IRC | 14:59 | |
*** jecarey has joined #openstack-oslo | 15:01 | |
*** abhishekk has left #openstack-oslo | 15:02 | |
*** jecarey has quit IRC | 15:03 | |
*** amotoki has quit IRC | 15:03 | |
*** jecarey has joined #openstack-oslo | 15:03 | |
*** edmondsw has joined #openstack-oslo | 15:04 | |
dims_ | ozamiatin_: let's see if https://review.openstack.org/235405 can fix the problem | 15:05 |
ozamiatin_ | dims_: thanks, a lot | 15:05 |
*** geoffarnold has joined #openstack-oslo | 15:06 | |
*** nkrinner has quit IRC | 15:07 | |
*** geoffarn_ has joined #openstack-oslo | 15:07 | |
openstackgerrit | Victor Stinner proposed openstack/oslo.concurrency: Use versionadded and versionchanged in doc https://review.openstack.org/235416 | 15:07 |
*** ozamiatin_ has quit IRC | 15:08 | |
*** geoffarnold has quit IRC | 15:10 | |
*** jecarey has quit IRC | 15:17 | |
*** sputnik13 has joined #openstack-oslo | 15:18 | |
*** browne has joined #openstack-oslo | 15:20 | |
*** e0ne has joined #openstack-oslo | 15:25 | |
*** geoffarn_ has quit IRC | 15:28 | |
*** geoffarnold has joined #openstack-oslo | 15:28 | |
*** openstackgerrit has quit IRC | 15:31 | |
*** openstackgerrit has joined #openstack-oslo | 15:31 | |
*** geoffarnold is now known as geoffarnoldX | 15:34 | |
*** geoffarnoldX is now known as geoffarnold | 15:34 | |
*** geoffarnold is now known as geoffarnoldX | 15:35 | |
*** jecarey_ has joined #openstack-oslo | 15:41 | |
*** gordc has quit IRC | 15:41 | |
*** geoffarnoldX has quit IRC | 15:49 | |
openstackgerrit | Tom Cammann proposed openstack/oslo.config: Raise exception when wrong type is used as default https://review.openstack.org/222078 | 15:49 |
*** geoffarnold has joined #openstack-oslo | 15:50 | |
*** yassine_ has joined #openstack-oslo | 15:51 | |
openstackgerrit | Roman Podoliaka proposed openstack/oslo.db: Refactor deps to use extras and env markers. https://review.openstack.org/184328 | 15:58 |
*** ndipanov has joined #openstack-oslo | 16:03 | |
*** yamahata has joined #openstack-oslo | 16:06 | |
*** ozamiatin has joined #openstack-oslo | 16:09 | |
*** geoffarnold has quit IRC | 16:10 | |
*** geoffarnold has joined #openstack-oslo | 16:11 | |
*** pballand has joined #openstack-oslo | 16:14 | |
*** yassine_ has quit IRC | 16:28 | |
*** e0ne has quit IRC | 16:30 | |
*** fultonj has left #openstack-oslo | 16:30 | |
*** geoffarnold has quit IRC | 16:31 | |
*** geoffarnold has joined #openstack-oslo | 16:32 | |
*** jecarey_ has quit IRC | 16:34 | |
*** stevemar_ has quit IRC | 16:35 | |
*** stevemar_ has joined #openstack-oslo | 16:36 | |
*** thangp has joined #openstack-oslo | 16:40 | |
*** ozamiatin has quit IRC | 16:44 | |
*** ihrachys has quit IRC | 16:45 | |
*** geoffarn_ has joined #openstack-oslo | 16:53 | |
*** geoffarnold has quit IRC | 16:53 | |
*** rohit_ has quit IRC | 16:54 | |
*** markus_z has quit IRC | 16:54 | |
*** amotoki has joined #openstack-oslo | 17:06 | |
*** dimtruck is now known as zz_dimtruck | 17:06 | |
harlowja | dims_ mdbooth ya, i was thinking of that (automaton) butttt let me look at that in more depth first :-P | 17:07 |
*** lbragstad_ has joined #openstack-oslo | 17:11 | |
*** bitblt has joined #openstack-oslo | 17:13 | |
*** geoffarnold has joined #openstack-oslo | 17:14 | |
*** geoffarnold is now known as geoffarnoldX | 17:18 | |
*** geoffarn_ has quit IRC | 17:18 | |
*** achanda has joined #openstack-oslo | 17:18 | |
harlowja | mdbooth dims_ i also think alot of this would probably be more sane if '_executor is reset to None' never happened | 17:19 |
harlowja | def reset(self): doesn't do anything, so i'm not sure why we go through a bunch of complexity to handle '_executor' being none | 17:19 |
harlowja | seems just like we cause ourselves pain by doing that... | 17:20 |
*** vilobhmm11 has joined #openstack-oslo | 17:20 | |
*** cdent has quit IRC | 17:22 | |
*** jecarey has joined #openstack-oslo | 17:33 | |
*** geoffarnoldX has quit IRC | 17:35 | |
*** geoffarnold has joined #openstack-oslo | 17:35 | |
*** jecarey_ has joined #openstack-oslo | 17:37 | |
harlowja | mdbooth added a bunch of comments | 17:37 |
harlowja | let me know what u think | 17:37 |
* harlowja really doesn't like the complexity added due to the ability to start/stop/start/stop... | 17:38 | |
harlowja | i'd rather just not allow that full stop | 17:38 |
*** jecarey has quit IRC | 17:38 | |
openstackgerrit | Clark Boylan proposed openstack-dev/pbr: Cleanup jeepyb and pypi-mirror special casing https://review.openstack.org/235508 | 17:42 |
*** rlrossit has quit IRC | 17:42 | |
*** cdent has joined #openstack-oslo | 17:43 | |
*** rlrossit has joined #openstack-oslo | 17:44 | |
*** mriedem has quit IRC | 17:45 | |
*** salv-orlando has joined #openstack-oslo | 17:47 | |
harlowja | mhorban1 https://pypi.python.org/pypi/notifier | 17:48 |
harlowja | 1.0 out with current code | 17:48 |
*** gordc has joined #openstack-oslo | 17:49 | |
*** jecarey has joined #openstack-oslo | 17:52 | |
*** browne has quit IRC | 17:52 | |
*** browne has joined #openstack-oslo | 17:53 | |
*** browne has quit IRC | 17:54 | |
*** jecarey_ has quit IRC | 17:56 | |
*** geoffarnold has quit IRC | 17:56 | |
*** geoffarnold has joined #openstack-oslo | 17:57 | |
harlowja | mhorban1 notifier.readthedocs.org/ also | 18:00 |
harlowja | should suffice for now | 18:00 |
*** lbragstad has quit IRC | 18:00 | |
*** lbragstad_ is now known as lbragstad | 18:00 | |
*** mriedem has joined #openstack-oslo | 18:01 | |
*** pballand has quit IRC | 18:01 | |
*** david-lyle has quit IRC | 18:01 | |
*** david-lyle has joined #openstack-oslo | 18:02 | |
*** amotoki has quit IRC | 18:03 | |
*** pballand has joined #openstack-oslo | 18:03 | |
*** pballand has quit IRC | 18:08 | |
*** e0ne has joined #openstack-oslo | 18:09 | |
*** exploreshaifali has joined #openstack-oslo | 18:13 | |
*** alejandrito has joined #openstack-oslo | 18:14 | |
*** zz_dimtruck is now known as dimtruck | 18:16 | |
*** bitblt has quit IRC | 18:16 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Enable conversion of the tree nodes into a digraph https://review.openstack.org/209214 | 18:16 |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 18:17 |
*** geoffarnold has quit IRC | 18:17 | |
*** geoffarnold has joined #openstack-oslo | 18:18 | |
*** kgiusti has quit IRC | 18:19 | |
*** cdent has quit IRC | 18:23 | |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 18:34 |
*** pradk has quit IRC | 18:35 | |
*** browne has joined #openstack-oslo | 18:35 | |
*** achanda has quit IRC | 18:36 | |
*** pradk has joined #openstack-oslo | 18:36 | |
*** geoffarnold has quit IRC | 18:39 | |
*** geoffarn_ has joined #openstack-oslo | 18:39 | |
openstackgerrit | Joshua Harlow proposed openstack/oslo.utils: Allow 'forever_retry_uncaught_exceptions' to take in different defaults https://review.openstack.org/235543 | 18:39 |
*** jecarey has quit IRC | 18:40 | |
openstackgerrit | Joshua Harlow proposed openstack/oslo.utils: Allow 'forever_retry_uncaught_exceptions' to take in different defaults https://review.openstack.org/235543 | 18:41 |
openstackgerrit | Joshua Harlow proposed openstack/oslo.utils: Allow 'forever_retry_uncaught_exceptions' to take in different defaults https://review.openstack.org/235543 | 18:42 |
*** dims has joined #openstack-oslo | 18:43 | |
*** achanda has joined #openstack-oslo | 18:44 | |
*** dims_ has quit IRC | 18:45 | |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 18:46 |
*** achanda has quit IRC | 18:46 | |
*** dims has quit IRC | 18:47 | |
kashyap | harlowja: Since you have some context related to this, mind taking a peek at this mechanical change? - https://review.openstack.org/#/c/234286/ | 18:49 |
harlowja | seems fair to me, don't think i can +2 it though | 18:50 |
kashyap | harlowja: Okay, give whatever makes you feel right. I'm not trying to extort a +2 :-) | 18:50 |
harlowja | :) | 18:50 |
kashyap | Who normall reviews requirements repo? I'd imagine it's highly context dependent | 18:51 |
bknudson | kashyap: https://review.openstack.org/#/admin/groups/131,members | 18:53 |
kashyap | bknudson: Thank you for the pointer, guessed as much. | 18:53 |
*** subscope has joined #openstack-oslo | 18:56 | |
openstackgerrit | Joshua Harlow proposed openstack/oslo.utils: Allow 'forever_retry_uncaught_exceptions' to take in different defaults https://review.openstack.org/235543 | 18:57 |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 19:00 |
*** geoffarn_ has quit IRC | 19:00 | |
*** geoffarnold has joined #openstack-oslo | 19:01 | |
*** rlrossit has quit IRC | 19:02 | |
*** jecarey has joined #openstack-oslo | 19:06 | |
*** rlrossit has joined #openstack-oslo | 19:07 | |
*** regXboi has quit IRC | 19:10 | |
*** dims has joined #openstack-oslo | 19:11 | |
openstackgerrit | Merged openstack/oslo.db: Updated from global requirements https://review.openstack.org/234136 | 19:16 |
openstackgerrit | Merged openstack/oslo.versionedobjects: Updated from global requirements https://review.openstack.org/235445 | 19:16 |
dims | lifeless: we are talking about the caps over on #openstack-stable if you are around | 19:20 |
*** geoffarnold has quit IRC | 19:21 | |
*** geoffarnold has joined #openstack-oslo | 19:22 | |
*** e0ne has quit IRC | 19:25 | |
*** jecarey has quit IRC | 19:26 | |
*** kgiusti has joined #openstack-oslo | 19:27 | |
*** jecarey has joined #openstack-oslo | 19:29 | |
*** dims has quit IRC | 19:30 | |
*** salv-orlando has quit IRC | 19:32 | |
*** achanda has joined #openstack-oslo | 19:32 | |
openstackgerrit | Merged openstack/oslo.db: Add debug logging for DB retry attempt. https://review.openstack.org/228480 | 19:38 |
*** geoffarnold is now known as geoffarnoldX | 19:39 | |
*** pballand has joined #openstack-oslo | 19:42 | |
*** geoffarnoldX has quit IRC | 19:42 | |
dansmith | sdague: so, devstack is still using o.vo 0.10.0, which means I can't test that my patches to drop InstanceV1 are working | 19:43 |
dansmith | sdague: what do I need to do to get that bumped? | 19:43 |
*** geoffarnold has joined #openstack-oslo | 19:43 | |
*** alexpilotti has joined #openstack-oslo | 19:43 | |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 19:46 |
*** achanda has quit IRC | 19:58 | |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 20:00 |
*** jecarey has quit IRC | 20:02 | |
*** salv-orlando has joined #openstack-oslo | 20:02 | |
*** gordc has quit IRC | 20:03 | |
*** jecarey has joined #openstack-oslo | 20:04 | |
*** geoffarn_ has joined #openstack-oslo | 20:04 | |
*** jecarey has quit IRC | 20:04 | |
*** jecarey has joined #openstack-oslo | 20:04 | |
*** geoffarnold has quit IRC | 20:05 | |
openstackgerrit | Sriram Madapusi Vasudevan proposed openstack/taskflow: feat: add max_dispatches arg to conductor's run https://review.openstack.org/234858 | 20:06 |
*** boris-42 has quit IRC | 20:18 | |
*** achanda has joined #openstack-oslo | 20:19 | |
openstackgerrit | tonytan4ever proposed openstack/taskflow: Register conductor information on jobboard https://review.openstack.org/234448 | 20:22 |
*** geoffarn_ has quit IRC | 20:25 | |
*** subscope has quit IRC | 20:25 | |
*** geoffarnold has joined #openstack-oslo | 20:26 | |
*** jecarey has quit IRC | 20:27 | |
*** jecarey has joined #openstack-oslo | 20:28 | |
*** jecarey has quit IRC | 20:28 | |
*** achanda has quit IRC | 20:28 | |
*** jecarey has joined #openstack-oslo | 20:28 | |
*** jecarey has quit IRC | 20:30 | |
*** jecarey has joined #openstack-oslo | 20:31 | |
*** Guest71541 has joined #openstack-oslo | 20:32 | |
openstackgerrit | Merged openstack/oslo.middleware: Updated from global requirements https://review.openstack.org/234675 | 20:33 |
*** lifeless has quit IRC | 20:34 | |
*** kgiusti has left #openstack-oslo | 20:35 | |
openstackgerrit | Merged openstack/oslo.service: Updated from global requirements https://review.openstack.org/234676 | 20:42 |
*** Guest71541 is now known as dims_ | 20:46 | |
*** geoffarnold has quit IRC | 20:46 | |
*** geoffarnold has joined #openstack-oslo | 20:47 | |
sdague | dansmith: that's going to be pinned by upper-constraints | 20:53 |
sdague | perhaps it's failing somewhere else which is blocking it | 20:53 |
*** achanda has joined #openstack-oslo | 20:56 | |
*** pballand has quit IRC | 20:57 | |
dansmith | sdague: meaning it should be lifting automatically but isn't because of some test fail somewhere? | 20:59 |
*** pballand has joined #openstack-oslo | 20:59 | |
sdague | dansmith: I believe so, | 20:59 |
sdague | lifeless would know for sure | 20:59 |
dansmith | hmm | 21:00 |
dansmith | I don't even know where to look, but I guess I'll just wait | 21:00 |
dims_ | mriedem: when you were talking to lifeless about oslo.db[mysql] oslo.db[fixtures] etc... | 21:02 |
mriedem | dims_: was i talking to him about that? | 21:02 |
dims_ | mriedem: i can't seem to be able to add oslo.db[fixtures] to nova's test-requirements as things end up failing saying oslo.db is already in requirements.txt | 21:03 |
dims_ | so how exactly do we specify some of the extras in test-requirements? | 21:03 |
mriedem | dims_: i think you just use oslo.db[fixtures] in requirements.txt | 21:03 |
*** rlrossit has left #openstack-oslo | 21:04 | |
openstackgerrit | Brant Knudson proposed openstack/oslo.policy: Use JSON generator https://review.openstack.org/234421 | 21:04 |
mriedem | dims_: if you look at how it's used in oslo.db tox.ini. it installs it's own reqs via .[fixtures] | 21:04 |
dims_ | mriedem: that goes back to the original problem where requirements.txt has things used just during test | 21:04 |
dims_ | mriedem: http://paste.openstack.org/show/476434/ if i add oslo.db[fixtures] to test requirements | 21:04 |
dims_ | mriedem: so we should add oslo.db[fixtures] in tox.ini for specific targets? | 21:05 |
mriedem | hmmm | 21:05 |
mriedem | will that work? | 21:05 |
dims_ | trying | 21:05 |
*** geoffarnold has quit IRC | 21:08 | |
dims_ | mriedem: same error - http://paste.openstack.org/show/476435/ | 21:08 |
dansmith | dims_: do you know the answer to the above question? devstack is still using o.vo 0.10.0 | 21:08 |
mriedem | dansmith: https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L203 | 21:08 |
*** vilobhmm111 has joined #openstack-oslo | 21:08 | |
mriedem | that's why | 21:08 |
*** vilobhmm11 has quit IRC | 21:09 | |
mriedem | crystal clear | 21:09 |
dansmith | mriedem: do I have to submit a change to that? | 21:09 |
dims_ | dansmith: yep | 21:09 |
*** geoffarnold has joined #openstack-oslo | 21:09 | |
mriedem | dansmith: there are supposed to be auto-generated ones | 21:09 |
dansmith | man, if we just get like three more steps in this release process, we'll be all set | 21:09 |
mriedem | well, the planets need to align, | 21:09 |
mriedem | jupiter's red spot has to disappear altogether | 21:09 |
*** thangp has quit IRC | 21:09 | |
mriedem | and you have to sacrifice your first born son | 21:10 |
mriedem | or that cat that bites your ankles | 21:10 |
dansmith | I guess I'm SOL on that one | 21:10 |
dansmith | oh, I can sacrifice my cat | 21:10 |
dims_ | dansmith: mriedem: bot generated one is in progress - https://review.openstack.org/#/c/231327/ | 21:10 |
dansmith | dims_: okay thanks | 21:10 |
dims_ | i had +2A'ed it a little while ago | 21:10 |
dansmith | I will rub my stomach and pat my head while jumping in place on one foot until it lands | 21:11 |
dims_ | https://review.openstack.org/#/c/231327/9/upper-constraints.txt,cm | 21:11 |
dims_ | :) | 21:11 |
mriedem | i'm pretty sure your insurance doesn't cover injuries from that | 21:11 |
dansmith | it definitely doesn't under obamacare.. I just got the corporate lecture on how much crappier things are now | 21:11 |
* dansmith shakes his fist | 21:12 | |
mriedem | thanks obama | 21:12 |
openstackgerrit | Eric Brown proposed openstack/oslo.config: Add PortOpt for integer with range 1 to 65535 https://review.openstack.org/231257 | 21:14 |
*** cburgess_ is now known as cburgess | 21:16 | |
dims_ | dansmith: sdague: when are you arriving in tokyo? | 21:19 |
sdague | sunday noon before the conference | 21:19 |
dansmith | dims_: saturday | 21:20 |
dims_ | sdague: k i'll be landing at 4 pm so will be another 2 hours at least to get to hotel | 21:21 |
dims_ | dansmith: cool | 21:21 |
dims_ | mriedem: back to the oslo.db[fixtures], don't see a way to specify the mandatory and the extras separately | 21:23 |
mriedem | dims_: put out the doug signal | 21:23 |
mriedem | or bknudson might know | 21:24 |
mriedem | b/c he did the stuff with ldap in keystone | 21:24 |
dims_ | dhellmann: bknudson: ping | 21:24 |
bknudson | what's up? | 21:24 |
bknudson | if you do oslo.db[fixtures] it installs all oslo.db requirements and also fixtures extras | 21:24 |
*** shardy has quit IRC | 21:24 | |
dims_ | bknudson: so in nova, i want to use oslo.db[fixtures] instead of testresources in test-requirements. how do i do that | 21:25 |
dims_ | bknudson: i understand that part | 21:25 |
bknudson | test-requirements would have oslo.db[fixtures] | 21:25 |
dims_ | bknudson: currently oslo.db is in requirements.txt because we have imports for oslo_db package | 21:25 |
dims_ | bknudson: does not work :) | 21:25 |
bknudson | what happens? | 21:25 |
dims_ | bknudson: http://paste.openstack.org/show/476434/ | 21:26 |
dims_ | tried both tox.ini and in test-requirements | 21:26 |
dims_ | fail with the same problem | 21:26 |
dims_ | only thing will work is to add oslo.db[fixtures] in requirements.txt | 21:26 |
bknudson | looks like "Double requirement given: oslo.db[fixtures]" | 21:26 |
dims_ | but that's wrong since it adds test dependencies | 21:26 |
dims_ | y | 21:27 |
bknudson | the double requirement must be oslo.db | 21:27 |
bknudson | well, that's a great question... I don't know what the answer is. | 21:27 |
dims_ | so there does not seem to be a way to do it :( | 21:28 |
bknudson | maybe the double requirement checker need updating | 21:28 |
bknudson | what's putting out that error | 21:28 |
*** geoffarnold has quit IRC | 21:29 | |
dims_ | bknudson: pip | 21:29 |
dims_ | haha - https://github.com/pypa/pip/issues/2367 | 21:30 |
*** geoffarnold has joined #openstack-oslo | 21:30 | |
openstackgerrit | Eric Brown proposed openstack/oslo.config: Add PortOpt for integer with range 1 to 65535 https://review.openstack.org/231257 | 21:31 |
dims_ | bknudson: so relaxing that error would probably work for us then | 21:32 |
dims_ | mriedem: see url above ^^ | 21:32 |
*** achanda has quit IRC | 21:32 | |
bknudson | dims_: this seems a little different... since we don't want to ignore it all the time, just when extra is added | 21:32 |
dims_ | bknudson: right | 21:33 |
*** jecarey has quit IRC | 21:42 | |
*** vilobhmm111 has quit IRC | 21:46 | |
*** vilobhmm11 has joined #openstack-oslo | 21:47 | |
*** vilobhmm11 has quit IRC | 21:47 | |
*** vilobhmm11 has joined #openstack-oslo | 21:47 | |
*** dimtruck is now known as zz_dimtruck | 21:49 | |
*** geoffarnold has quit IRC | 21:50 | |
*** geoffarnold has joined #openstack-oslo | 21:51 | |
*** pballand has quit IRC | 21:59 | |
*** mriedem has quit IRC | 21:59 | |
*** geoffarnold has quit IRC | 22:01 | |
*** jecarey has joined #openstack-oslo | 22:05 | |
openstackgerrit | fumihiko kakuma proposed openstack/oslo.rootwrap: Python 3: encode or decode i/o data of Popen.communicate() https://review.openstack.org/230408 | 22:06 |
*** geoffarnold has joined #openstack-oslo | 22:06 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:08 | |
*** jecarey has quit IRC | 22:09 | |
*** achanda has joined #openstack-oslo | 22:12 | |
*** achanda has quit IRC | 22:13 | |
*** achanda has joined #openstack-oslo | 22:14 | |
*** jamielennox|away is now known as jamielennox | 22:16 | |
openstackgerrit | Stanislaw Pitucha proposed openstack/pycadf: Add authenticate and evaluate actions https://review.openstack.org/235601 | 22:27 |
*** pballand has joined #openstack-oslo | 22:32 | |
*** jecarey has joined #openstack-oslo | 22:40 | |
*** jecarey has quit IRC | 22:45 | |
*** exploreshaifali has quit IRC | 22:49 | |
openstackgerrit | tonytan4ever proposed openstack/taskflow: Register conductor information on jobboard https://review.openstack.org/234448 | 23:02 |
*** geoffarnold is now known as geoffarnoldX | 23:04 | |
*** david-lyle has quit IRC | 23:09 | |
*** pballand has quit IRC | 23:09 | |
*** david-lyle has joined #openstack-oslo | 23:09 | |
*** geoffarnoldX is now known as geoffarnold | 23:10 | |
*** geoffarnold has quit IRC | 23:11 | |
*** salv-orl_ has joined #openstack-oslo | 23:15 | |
*** salv-orlando has quit IRC | 23:18 | |
*** jecarey has joined #openstack-oslo | 23:21 | |
*** jecarey has quit IRC | 23:25 | |
*** nikhil_k has joined #openstack-oslo | 23:27 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Enable conversion of the tree nodes into a digraph https://review.openstack.org/209214 | 23:28 |
*** wendar_ has joined #openstack-oslo | 23:29 | |
*** pblaho_ has joined #openstack-oslo | 23:30 | |
*** mhorban1 has quit IRC | 23:30 | |
*** cburgess has quit IRC | 23:30 | |
*** wendar has quit IRC | 23:30 | |
*** pblaho has quit IRC | 23:30 | |
*** Kiall has quit IRC | 23:30 | |
*** achanda_ has joined #openstack-oslo | 23:30 | |
*** cburgess has joined #openstack-oslo | 23:30 | |
*** mhorban has joined #openstack-oslo | 23:31 | |
*** vipul has quit IRC | 23:31 | |
*** miguelgrinberg has quit IRC | 23:31 | |
*** Kiall has joined #openstack-oslo | 23:31 | |
*** miguelgrinberg has joined #openstack-oslo | 23:31 | |
*** vipul has joined #openstack-oslo | 23:31 | |
*** dansmith has quit IRC | 23:31 | |
*** dansmith has joined #openstack-oslo | 23:32 | |
*** achanda has quit IRC | 23:32 | |
*** nikhil has quit IRC | 23:32 | |
*** yamahata__ has joined #openstack-oslo | 23:33 | |
*** amaurymedeiros has quit IRC | 23:33 | |
*** hogepodge has quit IRC | 23:34 | |
*** binarin has quit IRC | 23:34 | |
*** bapalm has quit IRC | 23:34 | |
*** mtreinish has quit IRC | 23:34 | |
*** isq_ has quit IRC | 23:34 | |
*** amaurymedeiros has joined #openstack-oslo | 23:38 | |
*** amaurymedeiros has quit IRC | 23:39 | |
*** amaurymedeiros has joined #openstack-oslo | 23:39 | |
*** mtreinish has joined #openstack-oslo | 23:39 | |
*** bapalm has joined #openstack-oslo | 23:39 | |
*** alexpilotti has quit IRC | 23:54 | |
*** dstanek has quit IRC | 23:54 | |
*** BAKfr has quit IRC | 23:54 | |
*** amaurymedeiros has quit IRC | 23:54 | |
*** cburgess has quit IRC | 23:54 | |
*** pblaho_ has quit IRC | 23:54 | |
*** wendar_ has quit IRC | 23:54 | |
*** nikhil_k has quit IRC | 23:54 | |
*** vilobhmm11 has quit IRC | 23:54 | |
*** dims_ has quit IRC | 23:54 | |
*** DuncanT has quit IRC | 23:54 | |
*** pkholkin has quit IRC | 23:54 | |
*** rakhmerov has quit IRC | 23:54 | |
*** davidlenwell has quit IRC | 23:54 | |
*** jamielennox has quit IRC | 23:54 | |
*** waynr has quit IRC | 23:54 | |
*** xek has quit IRC | 23:54 | |
*** zigo has quit IRC | 23:54 | |
*** tonyb has quit IRC | 23:54 | |
*** krotscheck has quit IRC | 23:54 | |
*** gus has quit IRC | 23:54 | |
*** sigmavirus24_awa has quit IRC | 23:54 | |
*** jroll has quit IRC | 23:54 | |
*** timsim has quit IRC | 23:54 | |
*** hockeynut has quit IRC | 23:54 | |
*** otherwiseguy has quit IRC | 23:54 | |
*** pradk has quit IRC | 23:54 | |
*** droyal_ has quit IRC | 23:54 | |
*** fnordahl has quit IRC | 23:54 | |
*** jamespage has quit IRC | 23:54 | |
*** mhorban has quit IRC | 23:54 | |
*** freyes has quit IRC | 23:54 | |
*** rbradfor has quit IRC | 23:54 | |
*** elarson has quit IRC | 23:54 | |
*** sheeprine has quit IRC | 23:54 | |
*** shakamunyi has quit IRC | 23:54 | |
*** sirushti has quit IRC | 23:54 | |
*** ativelkov has quit IRC | 23:54 | |
*** Kiall has quit IRC | 23:54 | |
*** zqfan has quit IRC | 23:54 | |
*** zzzeek_ has quit IRC | 23:54 | |
*** therve has quit IRC | 23:54 | |
*** dguitarbite has quit IRC | 23:54 | |
*** rpodolyaka_afk has quit IRC | 23:54 | |
*** adam_g has quit IRC | 23:54 | |
*** kbyrne has quit IRC | 23:54 | |
*** Nakato has quit IRC | 23:54 | |
*** edleafe has quit IRC | 23:54 | |
*** toabctl has quit IRC | 23:54 | |
*** odyssey4me has quit IRC | 23:54 | |
*** ujjain has quit IRC | 23:54 | |
*** zz_dimtruck has quit IRC | 23:54 | |
*** bapalm has quit IRC | 23:54 | |
*** vipul has quit IRC | 23:54 | |
*** mmasaki has quit IRC | 23:54 | |
*** bknudson has quit IRC | 23:54 | |
*** jamespd has quit IRC | 23:55 | |
*** tristanC has quit IRC | 23:55 | |
*** jlvillal has quit IRC | 23:55 | |
*** david-lyle has quit IRC | 23:55 | |
*** lbragstad has quit IRC | 23:55 | |
*** ndipanov has quit IRC | 23:55 | |
*** sputnik13 has quit IRC | 23:55 | |
*** edmondsw has quit IRC | 23:55 | |
*** jaypipes has quit IRC | 23:55 | |
*** zhiyan has quit IRC | 23:55 | |
*** jraim has quit IRC | 23:55 | |
*** frickler has quit IRC | 23:55 | |
*** kashyap has quit IRC | 23:55 | |
*** morgan has quit IRC | 23:55 | |
*** achanda_ has quit IRC | 23:55 | |
*** alejandrito has quit IRC | 23:55 | |
*** openstackgerrit has quit IRC | 23:55 | |
*** gibi has quit IRC | 23:55 | |
*** haypo has quit IRC | 23:55 | |
*** kzaitsev_ws has quit IRC | 23:55 | |
*** tchaypo has quit IRC | 23:55 | |
*** jokke_ has quit IRC | 23:55 | |
*** ericksonsantos has quit IRC | 23:55 | |
*** dhellmann has quit IRC | 23:55 | |
*** Horgix has quit IRC | 23:55 | |
*** EmilienM has quit IRC | 23:55 | |
*** sabari has quit IRC | 23:55 | |
*** mtreinish has quit IRC | 23:55 | |
*** sreshetnyak has quit IRC | 23:55 | |
*** dulek has quit IRC | 23:55 | |
*** HenryG has quit IRC | 23:55 | |
*** d0ugal has quit IRC | 23:55 | |
*** ajo has quit IRC | 23:55 | |
*** pleia2 has quit IRC | 23:55 | |
*** _stowa has quit IRC | 23:55 | |
*** browne has quit IRC | 23:55 | |
*** peterstac has quit IRC | 23:55 | |
*** eezhova has quit IRC | 23:55 | |
*** SlickNik has quit IRC | 23:55 | |
*** mfedosin has quit IRC | 23:55 | |
*** jd__ has quit IRC | 23:55 | |
*** melwitt has quit IRC | 23:55 | |
*** miguelgrinberg has quit IRC | 23:55 | |
*** yamahata has quit IRC | 23:55 | |
*** Qwazerty has quit IRC | 23:55 | |
*** sileht has quit IRC | 23:55 | |
*** sontek has quit IRC | 23:55 | |
*** fungi has quit IRC | 23:55 | |
*** flaper87 has quit IRC | 23:55 | |
*** kragniz has quit IRC | 23:55 | |
*** bnemec has quit IRC | 23:55 | |
*** SergeyLukjanov has quit IRC | 23:55 | |
*** sc68cal has quit IRC | 23:55 | |
*** dstufft has quit IRC | 23:55 | |
*** dmitryme has quit IRC | 23:55 | |
*** dansmith has quit IRC | 23:55 | |
*** jasondotstar has quit IRC | 23:55 | |
*** pasquier-s has quit IRC | 23:55 | |
*** dukhlov has quit IRC | 23:55 | |
*** alaski_out has quit IRC | 23:55 | |
*** clayton has quit IRC | 23:55 | |
*** goodygum has quit IRC | 23:55 | |
*** yamahata__ has quit IRC | 23:55 | |
*** salv-orl_ has quit IRC | 23:55 | |
*** stevemar_ has quit IRC | 23:55 | |
*** amrith has quit IRC | 23:55 | |
*** mdbooth has quit IRC | 23:55 | |
*** harlowja has quit IRC | 23:55 | |
*** mordred has quit IRC | 23:55 | |
*** ttx has quit IRC | 23:55 | |
*** rpodolyaka has quit IRC | 23:55 | |
*** sdague has quit IRC | 23:55 | |
*** ekarlso has quit IRC | 23:55 | |
*** redrobot has quit IRC | 23:55 | |
*** kevinbenton has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!