*** tongli has quit IRC | 00:20 | |
*** tongli has joined #openstack-oslo | 00:21 | |
*** dims has joined #openstack-oslo | 00:29 | |
*** zzzeek has quit IRC | 00:30 | |
*** dims has quit IRC | 00:34 | |
*** yamahata has joined #openstack-oslo | 00:57 | |
*** noelbk has quit IRC | 00:58 | |
*** zzzeek has joined #openstack-oslo | 00:59 | |
*** yamahata has quit IRC | 01:00 | |
*** yamahata has joined #openstack-oslo | 01:00 | |
*** gmurphy has quit IRC | 01:08 | |
*** gmurphy has joined #openstack-oslo | 01:08 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Allow tox tests with complex OS_TEST_DBAPI_CONNECTION URLs https://review.openstack.org/104428 | 01:12 |
---|---|---|
*** Alexei_987 has joined #openstack-oslo | 01:13 | |
*** zzzeek has quit IRC | 01:23 | |
*** dims has joined #openstack-oslo | 01:30 | |
*** dims has quit IRC | 01:34 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Make MySQL regexes generic across MySQL drivers https://review.openstack.org/106928 | 01:35 |
*** dims has joined #openstack-oslo | 01:56 | |
*** oomichi has joined #openstack-oslo | 02:32 | |
openstackgerrit | A change was merged to openstack-dev/hacking: Updated from global requirements https://review.openstack.org/108531 | 02:39 |
*** liusheng has quit IRC | 02:41 | |
*** amotoki has joined #openstack-oslo | 02:46 | |
amrith | gmurphy, many thanks. new bits on the way, just running tests now. | 02:49 |
openstackgerrit | amrith proposed a change to openstack/oslo-incubator: Mask any password in exception command string https://review.openstack.org/108215 | 02:58 |
*** tongli has quit IRC | 03:03 | |
*** mriedem has left #openstack-oslo | 03:17 | |
*** mriedem has quit IRC | 03:17 | |
*** dims has quit IRC | 03:25 | |
*** HenryG is now known as HenryG_afk | 03:43 | |
*** shakamunyi has joined #openstack-oslo | 04:05 | |
*** shakamunyi has quit IRC | 04:08 | |
*** YorikSar has joined #openstack-oslo | 04:13 | |
openstackgerrit | Angus Lees proposed a change to openstack/oslo.db: Specify raise_on_warnings=False for mysqlconnector https://review.openstack.org/107221 | 04:22 |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Revamp the symbol lookup mechanism https://review.openstack.org/108572 | 04:23 |
*** arnaud has quit IRC | 04:25 | |
*** arnaud__ has quit IRC | 04:25 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Revamp the symbol lookup mechanism https://review.openstack.org/108572 | 04:47 |
*** dims has joined #openstack-oslo | 04:54 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Revamp the symbol lookup mechanism https://review.openstack.org/108572 | 04:55 |
YorikSar | Hello. I've just founc out that oslo.local seems to be graduating in a separate library. Why isn't it graduating with oslo.concurrency? It would be more logical I think. | 04:56 |
*** dims has quit IRC | 04:59 | |
*** ildikov has joined #openstack-oslo | 05:21 | |
*** arnaud has joined #openstack-oslo | 05:34 | |
*** arnaud has quit IRC | 05:37 | |
*** arnaud has joined #openstack-oslo | 05:53 | |
*** k4n0 has joined #openstack-oslo | 05:59 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Revamp the symbol lookup mechanism https://review.openstack.org/108572 | 06:01 |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Revamp the symbol lookup mechanism https://review.openstack.org/108572 | 06:03 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Imported Translations from Transifex https://review.openstack.org/106938 | 06:03 |
openstackgerrit | A change was merged to openstack/oslo.utils: Setup for translation https://review.openstack.org/106339 | 06:27 |
*** mkoderer has joined #openstack-oslo | 06:30 | |
*** noelbk has joined #openstack-oslo | 06:32 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Revamp the symbol lookup mechanism https://review.openstack.org/108572 | 06:49 |
*** dims has joined #openstack-oslo | 06:57 | |
*** dims has quit IRC | 07:02 | |
*** harlowja is now known as harlowja_away | 07:04 | |
*** arnaud has quit IRC | 07:21 | |
*** k4n0 has quit IRC | 07:51 | |
*** i159 has joined #openstack-oslo | 07:54 | |
*** nacim has joined #openstack-oslo | 07:55 | |
*** dims has joined #openstack-oslo | 07:58 | |
*** dims has quit IRC | 08:03 | |
*** k4n0 has joined #openstack-oslo | 08:04 | |
*** ihrachyshka has joined #openstack-oslo | 08:22 | |
*** k4n0 has quit IRC | 08:32 | |
*** pblaho has joined #openstack-oslo | 08:43 | |
*** k4n0 has joined #openstack-oslo | 08:48 | |
*** dims has joined #openstack-oslo | 08:58 | |
*** dims has quit IRC | 09:04 | |
*** k4n0 has quit IRC | 09:19 | |
*** openstackgerrit has quit IRC | 09:31 | |
*** openstackgerrit has joined #openstack-oslo | 09:32 | |
openstackgerrit | Roman Vasilets proposed a change to openstack/oslo.db: Extract logging setup into a separate function https://review.openstack.org/106992 | 09:36 |
sileht | markmc, I'm trying to dig into the oslo.msg docs job issue, looks like sphinx 1.2b3 is installed, but the requirement says sphinx>=1.1.2,<1.2 , puting sphinx>=1.1.2,<1.1.999 should fix the issue | 09:37 |
*** k4n0 has joined #openstack-oslo | 09:37 | |
*** alexpilotti has joined #openstack-oslo | 09:38 | |
sileht | dhellmann, can you take a look to https://review.openstack.org/#/c/108624/ | 09:48 |
sileht | dhellmann, it's about the stable/icehouse docs job that fail on oslo.messaging | 09:48 |
*** yamahata has quit IRC | 09:51 | |
*** mrda is now known as mrda-away | 09:53 | |
*** arosen has quit IRC | 09:54 | |
*** hyakuhei has quit IRC | 09:54 | |
*** flaper87|afk has quit IRC | 09:54 | |
*** creiht has quit IRC | 09:54 | |
*** jogo has quit IRC | 09:54 | |
*** anteaya has quit IRC | 09:54 | |
*** yamahata__ has quit IRC | 09:54 | |
*** jogo has joined #openstack-oslo | 09:54 | |
*** creiht has joined #openstack-oslo | 09:54 | |
*** flaper87|afk has joined #openstack-oslo | 09:55 | |
*** hyakuhei has joined #openstack-oslo | 09:55 | |
*** yamahata__ has joined #openstack-oslo | 09:55 | |
*** flaper87|afk is now known as flaper87 | 09:55 | |
*** anteaya has joined #openstack-oslo | 09:55 | |
*** aaronorosen has joined #openstack-oslo | 09:56 | |
sileht | jd__, ^ | 09:56 |
*** oomichi has quit IRC | 09:59 | |
*** dims has joined #openstack-oslo | 09:59 | |
*** dims has quit IRC | 10:04 | |
*** pcm_ has joined #openstack-oslo | 10:07 | |
*** pcm_ has quit IRC | 10:08 | |
*** pcm_ has joined #openstack-oslo | 10:08 | |
*** viktors is now known as viktors|afk | 10:22 | |
*** cdent has joined #openstack-oslo | 10:23 | |
cdent | Is the problem with gate-oslo.messaging-python26 and argparse a known issue? | 10:25 |
cdent | ah, found it: https://bugs.launchpad.net/oslo/+bug/1346206 | 10:25 |
openstackgerrit | Pavel Kholkin proposed a change to openstack/oslo.db: Extension of DBDuplicateEntry exception https://review.openstack.org/108029 | 11:20 |
*** k4n0 has quit IRC | 11:28 | |
*** stannie1 has joined #openstack-oslo | 11:30 | |
*** dims has joined #openstack-oslo | 11:32 | |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Add a new greenio executor https://review.openstack.org/108652 | 11:34 |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Add a new greenio executor https://review.openstack.org/108652 | 11:37 |
*** haypo has joined #openstack-oslo | 11:47 | |
*** tongli has joined #openstack-oslo | 11:48 | |
jd__ | sileht: beuh is that a problem on all pip version? | 12:03 |
sileht | jd__, I guess | 12:14 |
jd__ | sileht: did you see the failure on py27 or only on py26 nodes? | 12:15 |
jd__ | because I learnt we have different pip versions on these nodes | 12:15 |
jd__ | anyway it's likely a good idea to have that :( | 12:15 |
sileht | jd__, my issue is with py27 | 12:15 |
sileht | jd__, I known we have the same issue with all oslo libs, this is why for alpha release we have put a additionnal .0 in the version before the a1, a2, .... | 12:16 |
jd__ | :-( | 12:17 |
sileht | jd__, I can't try it with py26, I haven't an old system near me | 12:19 |
jd__ | nevermind | 12:20 |
openstackgerrit | Christian Berendt proposed a change to openstack-dev/pbr: Bump hacking to 0.9.x series https://review.openstack.org/108668 | 12:23 |
*** stannie1 has quit IRC | 12:29 | |
*** stannie1 has joined #openstack-oslo | 12:32 | |
YorikSar | dims: Hi :) | 12:32 |
YorikSar | dims: When I wrote that email there was very silent here. | 12:33 |
*** dims has quit IRC | 12:33 | |
*** dims has joined #openstack-oslo | 12:33 | |
YorikSar | So.... What should I do to start graduating those libs? | 12:34 |
YorikSar | I see some detailed plan for oslo.serialization but no such plan for oslo.concurrency | 12:35 |
*** stannie1 has left #openstack-oslo | 12:36 | |
*** bnemec has joined #openstack-oslo | 12:56 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Implementation of Alembic as migration engine https://review.openstack.org/99965 | 12:59 |
*** bknudson has quit IRC | 13:10 | |
*** viktors has joined #openstack-oslo | 13:15 | |
*** HenryG_afk is now known as HenryG | 13:17 | |
*** mriedem has joined #openstack-oslo | 13:19 | |
*** jecarey has quit IRC | 13:28 | |
*** bknudson has joined #openstack-oslo | 13:29 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-dev/pbr: Allow using a .coveragerc file with --coverage https://review.openstack.org/108695 | 13:36 |
*** lbragstad has joined #openstack-oslo | 13:42 | |
*** lbragstad has quit IRC | 13:43 | |
*** lbragstad has joined #openstack-oslo | 13:43 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/108699 | 13:49 |
*** pblaho has quit IRC | 14:07 | |
*** jaypipes has joined #openstack-oslo | 14:20 | |
*** amotoki has quit IRC | 14:21 | |
*** pcm__ has joined #openstack-oslo | 14:37 | |
*** pcm__ has quit IRC | 14:37 | |
*** pcm__ has joined #openstack-oslo | 14:38 | |
*** pcm_ has quit IRC | 14:38 | |
*** ujjain has quit IRC | 14:44 | |
*** lbragstad has quit IRC | 14:46 | |
*** celttechie has joined #openstack-oslo | 14:49 | |
*** ujjain has joined #openstack-oslo | 14:49 | |
*** krotscheck has quit IRC | 14:50 | |
*** krotscheck has joined #openstack-oslo | 14:52 | |
*** lbragstad has joined #openstack-oslo | 14:56 | |
*** jecarey has joined #openstack-oslo | 14:59 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Utility function for robust schema drop https://review.openstack.org/106419 | 15:06 |
*** nacim has quit IRC | 15:09 | |
dhellmann | sileht: nice work! +2a | 15:12 |
*** yamahata has joined #openstack-oslo | 15:13 | |
openstackgerrit | A change was merged to openstack/oslo.version: Updated from global requirements https://review.openstack.org/108535 | 15:16 |
*** ihrachyshka has quit IRC | 15:21 | |
celttechie | hey all, I submitted a fix and am available if anyone has questions. https://review.openstack.org/#/c/108198/ | 15:22 |
*** cdent_ has joined #openstack-oslo | 15:23 | |
*** zzzeek has joined #openstack-oslo | 15:24 | |
*** cdent has quit IRC | 15:25 | |
*** cdent_ is now known as cdent | 15:25 | |
*** yamahata has quit IRC | 15:46 | |
*** i159 has quit IRC | 15:54 | |
*** arnaud has joined #openstack-oslo | 16:01 | |
*** arnaud has quit IRC | 16:03 | |
*** ildikov has quit IRC | 16:14 | |
celttechie | #openstack-meeting | 16:24 |
*** tsekiyama has joined #openstack-oslo | 16:26 | |
*** ihrachyshka has joined #openstack-oslo | 16:26 | |
*** ildikov has joined #openstack-oslo | 16:34 | |
*** lbragstad has quit IRC | 16:35 | |
*** markmcclain has joined #openstack-oslo | 16:41 | |
*** markmcclain1 has joined #openstack-oslo | 16:44 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/106591 | 16:44 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/105173 | 16:44 |
*** markmcclain has quit IRC | 16:46 | |
*** dims has quit IRC | 16:46 | |
*** lbragstad has joined #openstack-oslo | 16:51 | |
*** lbragstad has quit IRC | 16:52 | |
*** aaronorosen is now known as arosen | 16:55 | |
*** arosen has joined #openstack-oslo | 16:55 | |
*** arnaud has joined #openstack-oslo | 17:04 | |
*** lbragstad has joined #openstack-oslo | 17:05 | |
*** harlowja_away is now known as harlowja | 17:19 | |
*** ihrachyshka has quit IRC | 17:27 | |
*** ihrachyshka has joined #openstack-oslo | 17:30 | |
*** arnaud has quit IRC | 17:31 | |
*** arnaud has joined #openstack-oslo | 17:34 | |
*** arnaud has quit IRC | 17:41 | |
*** arnaud has joined #openstack-oslo | 17:41 | |
*** arnaud has quit IRC | 17:47 | |
*** arnaud has joined #openstack-oslo | 17:50 | |
*** arnaud has quit IRC | 17:53 | |
*** arnaud has joined #openstack-oslo | 17:54 | |
YorikSar | ttx: Around? | 17:54 |
YorikSar | ttx: I want to clarify about that late import in oslo.rootwrap. | 17:55 |
YorikSar | ttx: Is it necessary? Is fast failing a feature that should stay? | 17:55 |
*** arnaud has quit IRC | 17:59 | |
*** zzzeek has quit IRC | 18:00 | |
*** cdent has quit IRC | 18:02 | |
*** ihrachyshka has quit IRC | 18:06 | |
*** arnaud has joined #openstack-oslo | 18:17 | |
*** celttechie has quit IRC | 18:23 | |
*** lbragsta_ has joined #openstack-oslo | 18:27 | |
*** lbragstad has quit IRC | 18:30 | |
*** dims has joined #openstack-oslo | 18:30 | |
*** lbragsta_ has quit IRC | 18:31 | |
*** lbragstad has joined #openstack-oslo | 18:32 | |
*** lbragstad has quit IRC | 18:34 | |
*** lbragstad has joined #openstack-oslo | 18:34 | |
*** lbragstad has quit IRC | 18:39 | |
*** lbragstad has joined #openstack-oslo | 18:40 | |
*** arnaud has quit IRC | 18:44 | |
*** celttechie has joined #openstack-oslo | 18:49 | |
*** shakamunyi has joined #openstack-oslo | 18:56 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/106591 | 18:57 |
*** lbragstad has quit IRC | 19:03 | |
*** praneshp has quit IRC | 19:05 | |
*** lbragstad has joined #openstack-oslo | 19:08 | |
Alexei_987 | dhellmann: Hi thanks for reviewing my spec. I'll be glad to answer additional questions here if needed | 19:19 |
ttx | YorikSar: I have to remember why that was put in originally | 19:21 |
*** lbragstad has quit IRC | 19:22 | |
YorikSar | ttx: The only reason I can think of is to parse arguments and fail without importing the rest of the stdlib. | 19:22 |
YorikSar | ttx: I'm not sure that's still actual. | 19:23 |
ttx | YorikSar: Why are you declaring it global? | 19:23 |
ttx | Ah oh | 19:23 |
YorikSar | ttx: To not write another import in run_one_command. | 19:23 |
ttx | yeah, just figured hat the late import is local to main() | 19:24 |
*** lbragstad has joined #openstack-oslo | 19:24 | |
noelbk | Alexei_987: markmcclain1: I've got another fix for the rabbit reconnect problem: https://bugs.launchpad.net/oslo.messaging/+bug/1338732. I set passive=True in DirectPublisher so the publisher will fail and retry of the consumer hasn't reconnected yet | 19:24 |
noelbk | How does that sound to you? | 19:24 |
ttx | YorikSar: oh, I think I remember the reason now | 19:25 |
Alexei_987 | noelbk: Hi I will check now. btw I guess you want to ping markmc instead | 19:25 |
markmcclain1 | noelbk: I'm guessing you're looking for markmc who | 19:25 |
markmcclain1 | 's offline at the moment | 19:25 |
ttx | YorikSar: It's because we tweak the sys.path just above it | 19:25 |
YorikSar | ttx: Ah... | 19:25 |
ttx | YorikSar: but it's very likely to be useless those days | 19:26 |
dhellmann | Alexei_987: ok, it's better to have answers on the spec so all reviewers can see them | 19:26 |
jogo | so I have an oslo.messaging question | 19:26 |
jogo | dhellmann: ^ | 19:26 |
YorikSar | ttx: Can I please trash that pre-consolesctipts ahachronism? :) | 19:26 |
jogo | Bug 1311778 - Unit tests fail with MessagingTimeout errors | 19:26 |
ttx | YorikSar: It used to be a construct present everywhere so that you could run from branch | 19:26 |
dhellmann | hi, jogo | 19:26 |
jogo | dhellmann: can that be an oslo.messaging issue? | 19:26 |
ttx | YorikSar: yeah, I expect the whole thing can go away | 19:26 |
* dhellmann looks | 19:26 | |
jogo | example: http://logs.openstack.org/04/106704/2/gate/gate-nova-python26/7d61729/console.html#_2014-07-22_06_59_03_138 | 19:27 |
Alexei_987 | noelbk: so the fix is for the case when the reply is not yet sent? | 19:27 |
jogo | MessagingTimeout: No reply on topic scheduler | 19:27 |
YorikSar | ttx: Yes... Those times when someone had to add consolescripts and convince others to actually use them... | 19:27 |
ttx | YorikSar: maybe add an intermediate commit that gets rid of it | 19:27 |
Alexei_987 | noelbk: and it's different from "sent and lost" case? | 19:27 |
ttx | so if there is a regression somewhere we can trace it back there | 19:27 |
dhellmann | jogo: which messaging driver are the unit tests using? does it fail reliably? | 19:27 |
YorikSar | ttx: Ok, will do. | 19:27 |
jogo | fake | 19:27 |
jogo | http://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo/messaging/_drivers/impl_fake.py#n184 | 19:27 |
jogo | is where the exception comes from | 19:27 |
dhellmann | jogo: fake driver | 19:27 |
dhellmann | yeah, I'm flipping back and forth between irc and browser and finding answers as I ask :-) | 19:28 |
jogo | dhellmann: it doesn't fail reliably its a race condition | 19:28 |
jogo | dhellmann: the if queue empty, raise exception seems odd | 19:28 |
ttx | YorikSar: it's definitely not useful anymore, and keeping it in was always of a security question | 19:28 |
jogo | dhellmann: as if the queue is empty shouldn't we wait a bit | 19:28 |
ttx | since a lot of hte security model relies on sys.path | 19:29 |
Alexei_987 | noelbk: I guess we should start tracking failure reasons somehow. Cause it seems like 2 separate fixes for me in such case | 19:29 |
dhellmann | jogo: I think the assumption is if you're asking for data and the queue is empty that means the responder didn't give you any data | 19:29 |
ttx | YorikSar: thx! | 19:29 |
jogo | dhellmann: that isn't the best of assumptions | 19:29 |
jogo | IMHO | 19:29 |
noelbk | Alexei_987: my first fix was to always redeclare the reply queue in the publisher, but we decided it would be better to make the consumer responsible for declaring the reply queue and make the produce fail and retry when the consumer was still waiting for its reconnect delay | 19:29 |
YorikSar | ttx: oh... Oh... So we just pray that no service is run with vulnerable cwd? | 19:29 |
dhellmann | jogo: there's no message bus here, though, right? it's all in memory | 19:29 |
jogo | with eventlet and some of our bigger tests, we can be doing two things | 19:29 |
jogo | dhellmann: AFAIK correct | 19:29 |
Alexei_987 | noelbk: ah now I get it.. so it's still for the case when reply queue doesn't exist | 19:29 |
* jogo adds oslo.messaging to the bug for triage | 19:30 | |
ttx | YorikSar: no, we are pretty well covered thanks to sudo | 19:30 |
YorikSar | ttx: Ah, 2.x allows implicit relative imports anyway. | 19:30 |
ttx | which cleans the env | 19:30 |
ttx | back to system defined env | 19:30 |
ttx | which is assumed safe and only changeable by root | 19:30 |
YorikSar | ttx: But it doesnt chdir | 19:30 |
dhellmann | jogo: there's a timeout value, is the caller using it? | 19:30 |
noelbk | Alexei_987: yes, this just avoids a leak if the consumer never reconnects after a publisher posts a reply | 19:30 |
jogo | dhellmann: no idea | 19:30 |
jogo | dhellmann: I was hoping you would know this code better then me | 19:31 |
jogo | as this is the top gate bug | 19:31 |
jogo | which is sad | 19:31 |
ttx | YorikSar: oh right | 19:31 |
dhellmann | jogo: no, I'm not the messaging expert | 19:31 |
jogo | for nova | 19:31 |
YorikSar | ttx: So we can run rootwrap in a dir that has parent with oslo/__init__.py and... Run any code as root. | 19:31 |
jogo | dhellmann: any messaging experts up now? | 19:31 |
noelbk | Alexei_987: and prevents the publisher from delivering a reply to an exchange without a queue | 19:31 |
Alexei_987 | noelbk: In such case I think it's much better. You should post it on gerrit | 19:31 |
dhellmann | jogo: I think most of them are in europe (markmc, flaper87, sileht) | 19:31 |
noelbk | Alexei_987: will do, just wanted your blessing first :) | 19:31 |
dhellmann | jogo: some might be at oscon, though | 19:31 |
Alexei_987 | noelbk: I don't have much weight in oslo.messaging so my blessing won't help much | 19:32 |
ttx | YorikSar: not really | 19:32 |
ttx | YorikSar: hmm | 19:32 |
YorikSar | ttx: Ok, now I'm nervous... So on any machine that has it installed in sudoers anybody can run anything from root? | 19:32 |
Alexei_987 | noelbk: dhellmann and markmc should bless you instead :) | 19:32 |
ttx | YorikSar: htat's why you only allow to ru rootwarp in sudoers from a specific path | 19:32 |
YorikSar | ttx: Oh, wait. Not anybody. Just the user that is in sudoers. | 19:32 |
Alexei_987 | noelbk: however I'll be glad to review fix for that since we also suffering from this | 19:33 |
ttx | YorikSar: and that too | 19:33 |
ttx | So your sudoer should only allow /usr/bin/nova-roowrap, say | 19:33 |
ttx | IF you can install stuff under /usr/bin then yes you can escalate | 19:33 |
ttx | but then you're root already | 19:33 |
ttx | Anyway, it will be good to get rid of this | 19:34 |
YorikSar | ttx: Em... nova-rootwrap will import cmd and run main that will add cwd to sys.path and import wrapper from it. | 19:34 |
ttx | It adds os.path.abspath(execname)/../../oslo | 19:35 |
dhellmann | jogo: try adding a timeout in nova/conductor/rpcapi.py line 605 | 19:36 |
dhellmann | jogo: or find the default timeout and make it longer? | 19:36 |
ttx | and iiuc execname can only be the one that sudo authorizes | 19:36 |
YorikSar | ttx: Ok, I doudt someone has globally-writable /oslo dir :) | 19:36 |
*** praneshp has joined #openstack-oslo | 19:36 | |
YorikSar | ttx: Anyway, it should go. | 19:36 |
ttx | YorikSar: but yeah, getting rid of it will be awesome | 19:36 |
ttx | because it's a backdoor in the waiting | 19:37 |
jogo | not sure what the default timeout is | 19:37 |
dhellmann | jogo: sorry, that should have been nova/scheduler/rpcapi.py line 107 where the rpc call is actually made | 19:39 |
dhellmann | jogo: or even 105 where the call context is set up | 19:39 |
dhellmann | sheesh | 19:39 |
dhellmann | layers on layers here :-) | 19:40 |
*** viktors has quit IRC | 19:41 | |
jogo | yeah, | 19:41 |
jogo | way too many layers | 19:41 |
jogo | I want to set this once or twice | 19:41 |
jogo | not everywhere | 19:41 |
jogo | rpc_response_timeout=60 | 19:42 |
dhellmann | well, some calls take longer than others, right? | 19:42 |
jogo | but more then 60 seconds | 19:42 |
*** shakamunyi has quit IRC | 19:42 | |
*** Ish__ has joined #openstack-oslo | 19:42 | |
krotscheck | Is there any example code out there for oslo.messaging's notifier? | 19:43 |
dhellmann | krotscheck: a bit in the docs: http://docs.openstack.org/developer/oslo.messaging/notification_listener.html | 19:44 |
dhellmann | krotscheck: the ceilometer team uses is, irrc, so you might find more realistic code there | 19:44 |
krotscheck | dhellmann: Thanks, we'll check there. | 19:45 |
amrith | dhellmann, the intent in https://review.openstack.org/#/c/108216/ was to submit a change independent of https://review.openstack.org/#/c/108215/. I'm hoping both will go in together, would that address your concern? | 19:47 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Refactor common parts from cmd to wrapper https://review.openstack.org/107083 | 19:50 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add an option to run rootwrap as a daemon https://review.openstack.org/81798 | 19:50 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add basic benchmark https://review.openstack.org/107084 | 19:50 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Remove sys.path modification https://review.openstack.org/108821 | 19:50 |
YorikSar | ttx: ^ | 19:50 |
dhellmann | jogo: I'm reading this test code, and I wonder if something is just failing to mock out a method? | 19:52 |
dhellmann | jogo: and sometimes the fake driver 'works" and sometimes it doesn't, but we could just mock the method and not worry about it | 19:52 |
jogo | dhellmann: perhaps | 19:53 |
jogo | but not sure what method that would be | 19:54 |
dhellmann | no, me neither | 19:54 |
jogo | I am thinking the messaging stuff is a red herring | 19:56 |
jogo | will try to break this bug down into a few smaller ones | 19:56 |
jogo | we have things like InstanceInvalidState: Instance 460bdee5-d3bf-4127-a9e5-afdbb7b69bae in task_state powering-off. Cannot snapshot_volume_backed while the instance is in this state. | 19:56 |
jogo | so maybe one service stacktraces and never responds to the rpc or something | 19:57 |
*** zzzeek has joined #openstack-oslo | 19:59 | |
dhellmann | jogo: yeah, I noticed that happening too, so you might be right. I'm looking at how the tests set up the messaging stuff now but it's a bit hard to find :-/ | 20:05 |
*** lbragstad has quit IRC | 20:06 | |
*** lbragstad has joined #openstack-oslo | 20:08 | |
*** ihrachyshka has joined #openstack-oslo | 20:16 | |
*** ihrachyshka has quit IRC | 20:26 | |
*** ihrachyshka has joined #openstack-oslo | 20:26 | |
*** ihrachyshka has quit IRC | 20:26 | |
dims | dhellmann, bnemec - did Yuriy reach out on IRC? | 20:38 |
dhellmann | dims: if he did, I missed it | 20:39 |
dims | dhellmann, bnemec - since oslo.serialization repo is ready, i can take it through the rest of the process. ok? | 20:39 |
bnemec | dims: Not to me | 20:39 |
bnemec | dims: Yep | 20:39 |
dims | dhellmann, ok | 20:39 |
dims | bnemec, ack | 20:39 |
dhellmann | dims: how close is oslo.utils? | 20:39 |
* dims checking | 20:39 | |
dims | bnemec, could you please bless https://review.openstack.org/#/c/106778/ when you get a chance | 20:40 |
dims | dhellmann, that's the last one | 20:40 |
*** pcm_ has joined #openstack-oslo | 20:41 | |
dims | oops one more bnemec and dhellmann - https://review.openstack.org/#/c/108284/ | 20:41 |
*** pcm__ has quit IRC | 20:42 | |
*** pcm_ has quit IRC | 20:42 | |
*** pcm_ has joined #openstack-oslo | 20:43 | |
bnemec | dims: Okay, both done. | 20:49 |
dims | thanks! bnemec | 20:50 |
dhellmann | dims: so with those done, the API is tightened up enough for a release? | 20:51 |
dims | dhellmann, i believe so, importutils was the one we were worried about according to the spec and notes in etherpad | 20:51 |
dhellmann | dims: ok, cool | 20:52 |
*** Ish__ has quit IRC | 20:53 | |
*** lbragstad has quit IRC | 20:53 | |
openstackgerrit | A change was merged to openstack/pycadf: Remove docutils pin https://review.openstack.org/105626 | 20:54 |
dims | bnemec, dhellmann - either of you make it to beaverton? | 20:54 |
dhellmann | dims: I don't know what that is, so no? :-) | 20:54 |
openstackgerrit | A change was merged to openstack/pycadf: Bump hacking to version 0.9.2 https://review.openstack.org/107346 | 20:57 |
bnemec | dims: I'm in Raleigh this week. :-) | 20:57 |
dims | :) | 20:58 |
*** mrda-away is now known as mrda | 21:00 | |
*** Ish__ has joined #openstack-oslo | 21:04 | |
*** lbragstad has joined #openstack-oslo | 21:06 | |
*** jecarey has quit IRC | 21:07 | |
*** lbragstad has quit IRC | 21:08 | |
*** lbragstad has joined #openstack-oslo | 21:19 | |
*** jecarey has joined #openstack-oslo | 21:19 | |
dims | bnemec, did you try running oslo.serialization "tox -e pypy" | 21:26 |
bnemec | dims: I'm pretty sure I didn't. | 21:27 |
dims | ok, thanks py33 works, going to try pypy | 21:27 |
bnemec | Yeah, I'm pretty sure I did py27, py33, and pep8, but I don't know that I have pypy installed anywhere. :-) | 21:28 |
*** arnaud has joined #openstack-oslo | 21:30 | |
*** shakamunyi has joined #openstack-oslo | 21:32 | |
*** Ish__ has quit IRC | 21:37 | |
*** Ish__ has joined #openstack-oslo | 21:38 | |
dims | pypy: commands succeeded | 21:39 |
dims | congratulations :) | 21:39 |
dims | w00t | 21:39 |
bnemec | \o/ | 21:40 |
*** mriedem has left #openstack-oslo | 21:49 | |
*** mriedem has quit IRC | 21:49 | |
*** bnemec has quit IRC | 21:50 | |
*** shakamunyi has quit IRC | 21:56 | |
*** tongli has quit IRC | 21:59 | |
*** pcm_ has quit IRC | 22:10 | |
*** dims_ has joined #openstack-oslo | 22:23 | |
*** dims has quit IRC | 22:24 | |
*** celttechie has left #openstack-oslo | 22:56 | |
*** markmcclain1 has quit IRC | 22:56 | |
*** celttechie has joined #openstack-oslo | 22:56 | |
*** zzzeek has quit IRC | 22:57 | |
*** bknudson has quit IRC | 22:57 | |
*** zzzeek has joined #openstack-oslo | 23:09 | |
*** zzzeek has quit IRC | 23:14 | |
openstackgerrit | A change was merged to openstack/oslo.utils: export only try_import in __all__ https://review.openstack.org/106778 | 23:24 |
openstackgerrit | A change was merged to openstack/oslo.utils: Cleaning up index.rst file https://review.openstack.org/108284 | 23:24 |
*** lbragstad has quit IRC | 23:40 | |
openstackgerrit | amrith proposed a change to openstack/oslo-incubator: Mask any password in exception command string https://review.openstack.org/108215 | 23:42 |
*** tsekiyam_ has joined #openstack-oslo | 23:46 | |
*** tsekiyam_ has quit IRC | 23:47 | |
*** tsekiyam_ has joined #openstack-oslo | 23:49 | |
*** arnaud has quit IRC | 23:49 | |
*** tsekiyama has quit IRC | 23:49 | |
*** oomichi has joined #openstack-oslo | 23:50 | |
*** tsekiyama has joined #openstack-oslo | 23:54 | |
*** tsekiyam_ has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!