| *** yamahata has joined #openstack-oslo | 00:03 | |
| *** praneshp has joined #openstack-oslo | 00:11 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Make the runner a runtime provided property https://review.openstack.org/97656 | 00:14 |
|---|---|---|
| *** mriedem has joined #openstack-oslo | 00:19 | |
| *** arnaud has joined #openstack-oslo | 00:26 | |
| *** zhiyan_ is now known as zhiyan | 00:28 | |
| *** ajo|mac has quit IRC | 00:51 | |
| dhellmann | morganfainberg: probably _LE (error)? | 00:51 |
| morganfainberg | dhellmann, figured as much. but wanted to get an outside opinion | 00:52 |
| *** yamahata has quit IRC | 00:53 | |
| *** yamahata has joined #openstack-oslo | 00:53 | |
| dhellmann | morganfainberg: yeah, I think that's part of what I expected _LE would be used for | 01:00 |
| *** arnaud has quit IRC | 01:08 | |
| *** praneshp has quit IRC | 01:30 | |
| *** wingwj has joined #openstack-oslo | 01:31 | |
| *** bknudson has quit IRC | 01:37 | |
| *** mriedem has left #openstack-oslo | 02:27 | |
| *** mriedem has quit IRC | 02:27 | |
| *** harlowja is now known as harlowja_away | 02:42 | |
| *** dims has quit IRC | 02:45 | |
| *** wingwj has left #openstack-oslo | 03:58 | |
| *** prashanthr_ has joined #openstack-oslo | 03:58 | |
| *** amotoki has joined #openstack-oslo | 03:58 | |
| *** prashanthr_ has quit IRC | 04:05 | |
| *** prashanthr_ has joined #openstack-oslo | 04:09 | |
| *** zhiyan is now known as zhiyan_ | 04:15 | |
| *** zhiyan_ is now known as zhiyan | 04:17 | |
| *** amotoki has quit IRC | 04:39 | |
| *** prashanthr_ has quit IRC | 04:43 | |
| *** shakamunyi has joined #openstack-oslo | 05:04 | |
| *** praneshp has joined #openstack-oslo | 05:13 | |
| *** praneshp_ has joined #openstack-oslo | 05:27 | |
| *** praneshp has quit IRC | 05:29 | |
| *** praneshp_ is now known as praneshp | 05:29 | |
| *** prashanthr_ has joined #openstack-oslo | 05:30 | |
| *** prashanthr_1 has joined #openstack-oslo | 05:52 | |
| *** prashanthr_ has quit IRC | 05:54 | |
| *** prashanthr_1 has quit IRC | 05:57 | |
| *** stannie has joined #openstack-oslo | 06:12 | |
| *** prashanthr_ has joined #openstack-oslo | 06:13 | |
| *** mkoderer has joined #openstack-oslo | 06:20 | |
| *** stannie has quit IRC | 06:36 | |
| *** ihrachyshka has joined #openstack-oslo | 06:41 | |
| *** prashanthr_ has quit IRC | 06:45 | |
| *** ihrachyshka has quit IRC | 06:59 | |
| *** prashanthr_ has joined #openstack-oslo | 06:59 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Remove binary_decode from misc utils https://review.openstack.org/97696 | 07:05 |
| *** ihrachyshka has joined #openstack-oslo | 07:07 | |
| ihrachyshka | dhellmann: hey. can we discuss a backport candidate? | 07:11 |
| *** ihrachyshka has quit IRC | 07:14 | |
| openstackgerrit | James Polley proposed a change to openstack-dev/hacking: Clarify where the "following template" is https://review.openstack.org/97698 | 07:17 |
| *** prashanthr_ has quit IRC | 07:26 | |
| *** prashanthr_ has joined #openstack-oslo | 07:28 | |
| *** zhiyan is now known as zhiyan_ | 07:34 | |
| *** mrda is now known as mrda-away | 07:51 | |
| *** nacim has quit IRC | 07:58 | |
| *** nacim has joined #openstack-oslo | 07:58 | |
| *** praneshp has quit IRC | 08:06 | |
| *** nacim has quit IRC | 08:07 | |
| *** nacim has joined #openstack-oslo | 08:09 | |
| *** shakamunyi has quit IRC | 08:10 | |
| *** viktors has quit IRC | 08:14 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/oslo.db: replace string format arguments with function parameters https://review.openstack.org/94334 | 08:20 |
| *** prashanthr_1 has joined #openstack-oslo | 08:24 | |
| *** ihrachyshka has joined #openstack-oslo | 08:25 | |
| *** prashanthr_ has quit IRC | 08:26 | |
| *** markmc has joined #openstack-oslo | 08:30 | |
| *** flaper87|afk is now known as flaper87 | 08:35 | |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo.db: Remove common context from oslo.db https://review.openstack.org/93397 | 08:36 |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo.db: Remove common context usage from db model_query() https://review.openstack.org/93396 | 08:36 |
| *** markmcclain has joined #openstack-oslo | 08:40 | |
| *** mkoderer has quit IRC | 08:48 | |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** ajo has quit IRC | 08:48 | |
| *** ajo has joined #openstack-oslo | 08:48 | |
| *** mkoderer has joined #openstack-oslo | 08:48 | |
| *** openstackgerrit has joined #openstack-oslo | 08:50 | |
| *** ihrachyshka has quit IRC | 09:12 | |
| *** prashanthr_1 has quit IRC | 09:12 | |
| *** ihrachyshka has joined #openstack-oslo | 09:40 | |
| *** markmcclain has quit IRC | 09:44 | |
| *** prashanthr_ has joined #openstack-oslo | 09:45 | |
| *** markmcclain has joined #openstack-oslo | 09:45 | |
| *** prashanthr_ has quit IRC | 10:05 | |
| *** rdopiera has joined #openstack-oslo | 10:06 | |
| *** markmcclain has quit IRC | 10:11 | |
| *** shakamunyi has joined #openstack-oslo | 10:37 | |
| *** shakamunyi has quit IRC | 10:41 | |
| *** yamahata has quit IRC | 10:47 | |
| *** dims_ has joined #openstack-oslo | 10:50 | |
| ekarlso | is o.db rdy ? | 10:50 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/oslo-specs: Add service-status-interface spec https://review.openstack.org/97748 | 10:53 |
| *** nacim has quit IRC | 10:55 | |
| ajo | dhellmann, http://docs-draft.openstack.org/48/97748/1/check/gate-oslo-specs-docs/c4a464e/doc/build/html/specs/juno/service-status-interface.html | 10:57 |
| ajo | first version ready | 10:57 |
| openstackgerrit | Nikola Dipanov proposed a change to openstack/oslo.messaging: DO NOT MERGE: Fix can_send_version to check against the base client version https://review.openstack.org/97750 | 10:57 |
| *** amotoki has joined #openstack-oslo | 11:11 | |
| *** dims_ has quit IRC | 11:14 | |
| *** dims_ has joined #openstack-oslo | 11:20 | |
| *** flaper87 is now known as flaper87|afk | 11:25 | |
| *** nacim has joined #openstack-oslo | 11:34 | |
| *** shakamunyi has joined #openstack-oslo | 11:38 | |
| *** yamahata has joined #openstack-oslo | 11:41 | |
| *** shakamunyi has quit IRC | 11:43 | |
| *** viktors has joined #openstack-oslo | 11:49 | |
| *** yamahata has quit IRC | 11:50 | |
| *** yamahata has joined #openstack-oslo | 11:50 | |
| *** nacim has quit IRC | 11:54 | |
| *** dims_ has quit IRC | 12:29 | |
| *** dims_ has joined #openstack-oslo | 12:29 | |
| *** jecarey has quit IRC | 12:32 | |
| ajo | dhellmann, thanks for the review ;) addressing your comments | 12:36 |
| dhellmann | ajo: cool -- Just to be clear: I like this idea, I'm just not sure I get some of the details. | 12:37 |
| ajo | dhellmann, totally understood, it's my first "-spec" at openstack, the more complete, the clearer picture it will provide... | 12:38 |
| openstackgerrit | Gordon Sim proposed a change to openstack/oslo-specs: Added spec for proposed AMQP 1.0 driver https://review.openstack.org/96729 | 12:40 |
| dhellmann | ajo: right, we're all still figuring out the process for specs since they are new this cycle | 12:41 |
| ajo | dhellmann, it's actually a very good addition to the process :) | 12:41 |
| dhellmann | ajo: and of course, the details might have to change when you get into the implementation, but the more we can figure out up front the better off we'll be | 12:42 |
| dhellmann | ajo: I've definitely already found if valuable for my own planning | 12:42 |
| ajo | it always is... | 12:42 |
| dhellmann | I uncovered a couple of issues just by trying to figure out enough detail to explain how to graduate some of the libraries | 12:42 |
| ajo | dhellmann, I was missing it lately. Specially the sequence diagrams help to clean up the way classes talk to each other. I refined several local versions into this last one, which is much simpler/cleaner than initially, but sure, it can be improved yet :) | 12:43 |
| *** viktors has quit IRC | 12:44 | |
| ajo | dhellmann, yes, and the templates are good also, those make you think about performance, security, etc... API impact | 12:44 |
| ajo | in advance | 12:44 |
| *** viktors has joined #openstack-oslo | 12:53 | |
| *** jecarey has joined #openstack-oslo | 12:56 | |
| *** markmcclain has joined #openstack-oslo | 12:59 | |
| *** markmcclain1 has joined #openstack-oslo | 13:00 | |
| *** HenryG has quit IRC | 13:02 | |
| *** markmcclain has quit IRC | 13:03 | |
| *** mriedem has joined #openstack-oslo | 13:07 | |
| openstackgerrit | Ken Giusti proposed a change to openstack/oslo-incubator: Ensure routing key is specified in the address for a direct producer https://review.openstack.org/97780 | 13:09 |
| *** markmc has quit IRC | 13:09 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo-specs: add spec for graduate-oslo-log https://review.openstack.org/95929 | 13:10 |
| openstackgerrit | Ken Giusti proposed a change to openstack/oslo-incubator: Ensure routing key is specified in the address for a direct producer https://review.openstack.org/97783 | 13:16 |
| *** shakamunyi has joined #openstack-oslo | 13:16 | |
| *** bknudson has joined #openstack-oslo | 13:18 | |
| *** markmcclain1 has quit IRC | 13:31 | |
| ruhe | hi! are there any projects using https://github.com/openstack/oslo-incubator/blob/master/openstack/common/db/sqlalchemy/test_migrations.py ? it seems like every project has it's own copy of test_migrations. should i also go with a copy of test_migrations from another openstack project? | 13:51 |
| *** markmcclain has joined #openstack-oslo | 14:09 | |
| bnemec | ruhe: No, you should sync it from oslo-incubator: https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator | 14:16 |
| bnemec | Note that the db code is graduating to oslo.db, so if you don't need it immediately you might want to just wait until that's available. | 14:16 |
| *** rpodolyaka1 has joined #openstack-oslo | 14:16 | |
| ruhe | bnemec: ok. thank you. any ETA for oslo.db graduation and availability? | 14:17 |
| bnemec | ruhe: I think we're getting close. viktors and rpodolyaka are the ones driving that so they should know exactly what still needs to happen. | 14:18 |
| *** mrodden has quit IRC | 14:24 | |
| viktors | Hi bnemec, ruhe. Unfortunately, we still need to fix config usage, fix model_query() and add eventlt.tpool support. | 14:24 |
| viktors | Some of these patches are already approved, but they are not merged yet because of gate bugs :( | 14:24 |
| viktors | ruhe: as for test_migrations.py - feel free to reuse it, this code should be already synced to the most of OS projects | 14:26 |
| ruhe | viktors: willdo. thank you! | 14:27 |
| *** tkelsey has joined #openstack-oslo | 14:27 | |
| *** rdopiera has quit IRC | 14:28 | |
| tkelsey | hello openstack oslo people, I have a question about incubator, is this the best place to ask or is there an incubator specific channel? | 14:28 |
| viktors | bnemec: by the way, as for oslo.db - can you please re review patch https://review.openstack.org/#/c/93396/ (Remove common context usage from db model_query()), when you'll have a time? :) | 14:29 |
| viktors | tkelsey: yes, you can ask your question about incubator here | 14:30 |
| tkelsey | viktors: thanks, so i see changes like this one https://review.openstack.org/#/c/94881/6 that seem to copy portions of incubator in them, is this the correct way to use icubator stuff? im a little confused by it | 14:31 |
| tkelsey | perhaps there is an info page some place that answers this question, I read about projects graduating from incubation, but im not sure how it relates to stuff being used like this | 14:33 |
| viktors | tkelsey: you can read about syncing code from olso-incubator at https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator | 14:35 |
| viktors | In a few words - sync looks correct, but I prefer to have a first patch for sync and second patch for the changes in the project code. | 14:35 |
| bnemec | Yeah, that patch has a number of issues. :-) | 14:36 |
| tkelsey | viktors: I see, yes that would make more sense to me as well. I will read the page. Thanks for the info :-) | 14:36 |
| viktors | tkelsey: no problem :) | 14:36 |
| *** mrodden has joined #openstack-oslo | 14:37 | |
| bnemec | viktors: I will take another look. Just got out of my morning meeting. | 14:37 |
| viktors | bnemec: thanks. | 14:39 |
| * viktors don't like the big time difference | 14:39 | |
| *** hyakuhei has joined #openstack-oslo | 14:42 | |
| hyakuhei | While I was reviewing some of the Kite code I noticed a few oddities with the crypto/utils.py file, should I talk about it here, open an LP bug or maybe put something on the ML? | 14:44 |
| hyakuhei | Not a defect as such, just some things I wasn't sure about | 14:44 |
| dhellmann | hyakuhei: why don't you start here, and we'll see if it needs a ML discussion | 14:45 |
| hyakuhei | ok great, so looking at http://git.openstack.org/cgit/openstack/oslo-incubator/tree/openstack/common/crypto/utils.py there are a few things | 14:45 |
| *** markmcclain has quit IRC | 14:45 | |
| hyakuhei | First off all the comments refer to AES-128 but according to line 97 we are actually doing AES-256 everywhere | 14:46 |
| hyakuhei | I think ? | 14:46 |
| hyakuhei | The second thing I noticed was that the padding seemed a bit funky | 14:46 |
| hyakuhei | If my data is boundary-size I'll send a whole second block of padding - is that normal/desired behaviour? | 14:47 |
| dhellmann | hyakuhei: I wouldn't think so | 14:47 |
| hyakuhei | Comments in position (but the wrong CR) here : https://review.openstack.org/#/c/80902/4/kite/openstack/common/crypto/utils.py | 14:47 |
| hyakuhei | ^ That's where I commented on it before I realised it was oslo | 14:47 |
| bnemec | viktors: Okay, looks good now. I like the comment explaining the external bits (with the exception of the grammar nit ;-). | 14:47 |
| dhellmann | hyakuhei: the docstring thing can probably just be fixed with a patch to oslo-incubator but the other deserves a bug report | 14:49 |
| hyakuhei | Ok great, I'm happy to do both | 14:49 |
| dhellmann | hyakuhei: we'd be happy to have a patch for that, too, of course, but the bug will help track it :-) | 14:49 |
| hyakuhei | So it's ok for me to just patch little things without a bug? | 14:49 |
| dhellmann | hyakuhei: yeah, for something like that it's ok | 14:49 |
| hyakuhei | great, I'll do that now. Thanks dhellmann | 14:50 |
| dhellmann | hyakuhei: it's like to get a higher priority in reviews if you do file a bug, but it's not required | 14:50 |
| hyakuhei | Makes sense | 14:50 |
| * bnemec doesn't want to think what our bug list would look like if people opened one for every typo they fix | 14:50 | |
| viktors | bnemec: thanks for review. Can I ask you to be in touch a few minutes - let me fix you nit also. | 14:50 |
| bnemec | :-) | 14:50 |
| hyakuhei | I'd actually like to talk about getting the OSSG to do a code review of more of the OSLO base at some point soon | 14:50 |
| hyakuhei | bnemec: think of all those closed bugs! | 14:50 |
| bnemec | viktors: That would be great. | 14:50 |
| dhellmann | hyakuhei: that's a good idea, and it's planned for the libraries | 14:52 |
| bnemec | hyakuhei: As long as the launchpad integration with Gerrit works right. :-) | 14:52 |
| dhellmann | but it would be good to have someone look over the incubator, too | 14:52 |
| hyakuhei | Yeah, I agree | 14:52 |
| *** rmcall has joined #openstack-oslo | 14:53 | |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo.db: Remove common context from oslo.db https://review.openstack.org/93397 | 14:54 |
| openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo.db: Remove common context usage from db model_query() https://review.openstack.org/93396 | 14:54 |
| hyakuhei | Question: Do you actually want 256 bit encryption in here or are the comments right and the blocksize wrong? | 14:55 |
| viktors | bnemec: fixed | 14:55 |
| hyakuhei | (comments about algorithms and size are orthogonal for now | 14:56 |
| *** jungleboyj has joined #openstack-oslo | 15:01 | |
| *** jecarey has quit IRC | 15:02 | |
| *** jecarey has joined #openstack-oslo | 15:02 | |
| bnemec | viktors: +2, thanks! | 15:04 |
| viktors | bnemec: thank you! | 15:04 |
| *** mriedem has quit IRC | 15:04 | |
| *** mriedem has joined #openstack-oslo | 15:05 | |
| hyakuhei | So I'm slightly confused, when I pull down a fresh clone of oslo-incubator and do a commit without making any changes I'm seeing something waiting to be merged. | 15:07 |
| hyakuhei | something in common/log.py | 15:07 |
| hyakuhei | dhellmann: ^ ? | 15:12 |
| dhellmann | hyakuhei: can you pastebin the diff? | 15:13 |
| hyakuhei | Sure, though it doesn't show up in the diff | 15:14 |
| hyakuhei | Basically, I follow the gerrit workflow, pull down the clone, etc. | 15:14 |
| dhellmann | hyakuhei: so you see that file as having a pending modification, but there is no modification? | 15:14 |
| *** rmcall has quit IRC | 15:14 | |
| dhellmann | hyakuhei: which git repo are you cloning? | 15:14 |
| dhellmann | what url? | 15:14 |
| hyakuhei | git://git.openstack.org/openstack/oslo-incubator | 15:15 |
| dhellmann | ok, I thought maybe if you were using github it was out of sync | 15:15 |
| hyakuhei | Basically I pull down a fresh copy | 15:15 |
| hyakuhei | follow the guide, do the --ff-only etc | 15:15 |
| *** HenryG has joined #openstack-oslo | 15:15 | |
| hyakuhei | branch to a sensible branch | 15:15 |
| dhellmann | which guide are you following? | 15:15 |
| hyakuhei | GerritWorkflow | 15:15 |
| dhellmann | I don't ever use --ff-only, I don't know what that is | 15:15 |
| hyakuhei | ^ It's in the guide and it's what I've always done ;) | 15:16 |
| dhellmann | oh, hmm, I always just do "git pull" | 15:16 |
| hyakuhei | Anyway, lets assume that's not the broken bit for a second | 15:16 |
| dhellmann | sure | 15:16 |
| hyakuhei | So I branch as you should | 15:16 |
| dhellmann | ok | 15:16 |
| hyakuhei | then make my tiny change and do a commit --amend | 15:16 |
| hyakuhei | and there's some odd Merge stuff there | 15:16 |
| dhellmann | amend? | 15:17 |
| hyakuhei | heh, so I'm following https://wiki.openstack.org/wiki/GerritWorkflow | 15:17 |
| dhellmann | are you updating your own commit? | 15:17 |
| hyakuhei | No, I suppose I'm just used to running amend | 15:18 |
| dhellmann | ok, you only use --amend to change an existing commit before resubmitting it | 15:18 |
| dhellmann | so if you have a new change, you want to just commit it to make a new changeset | 15:19 |
| hyakuhei | Hmm. Magically it's not happening now, though I did get tkelsey to take a look and he replicated the oddness | 15:19 |
| hyakuhei | note oddness is a commit message about a change to some log.py file | 15:19 |
| hyakuhei | I'll nuke it and start from scratch | 15:19 |
| dhellmann | using --amend will do weird things to your tree, and then a git review (which rebases) may cause strangeness | 15:19 |
| hyakuhei | ok cool, rm -rf'd and re-working now | 15:19 |
| hyakuhei | thanks dhellmann | 15:19 |
| tkelsey | yeah i can confirm i did see it also | 15:20 |
| hyakuhei | well third time's a charm :) | 15:23 |
| openstackgerrit | Robert Clark proposed a change to openstack/oslo-incubator: Crypto utils file documentation referred to 128 bit crypt but appears to perform 256 bit changed documentation to match https://review.openstack.org/97822 | 15:23 |
| dhellmann | hyakuhei: :-) | 15:23 |
| bnemec | That GerritWorkflow page needs updating. It's been on my list for about six months now. :-/ | 15:24 |
| hyakuhei | So that oddness that I saw before, that's showing as the 'parent' of my change | 15:24 |
| dhellmann | some of the steps on the page seem to be related to the docs project specifically | 15:24 |
| dhellmann | hyakuhei: right, that was probably caused by the --amend | 15:24 |
| tkelsey | yeah i was also using --amend, its not needed for a fresh commit, i guess its tryting to ammend the last commit message, a merge doen by Jenkins | 15:24 |
| tkelsey | *done | 15:25 |
| dhellmann | right, --amend changes the most recent commit to include your new changes, which you only want to do if you're modifying a patch and resubmitting it | 15:25 |
| hyakuhei | Yeah that makes sense. Serves me right for expecting commands in the documentation to be in the order you would use them :P | 15:27 |
| *** viktors is now known as viktors|afk | 15:28 | |
| *** markmc has joined #openstack-oslo | 15:38 | |
| *** markmcclain has joined #openstack-oslo | 15:42 | |
| *** dims_ has quit IRC | 15:42 | |
| *** yamahata has quit IRC | 15:43 | |
| *** ihrachyshka_ has joined #openstack-oslo | 15:49 | |
| *** ihrachyshka has quit IRC | 15:52 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 15:53 |
| *** ihrachyshka_ has quit IRC | 15:55 | |
| *** markmc has quit IRC | 16:02 | |
| bnemec | dhellmann: Do we want to have a special policy on approving specs? I think in tripleo there was discussion that only the PTL would approve, and nova has their nova-drivers team that can approve. | 16:03 |
| bnemec | I'm thinking with the size of our team we probably want to stick with +2 +2/+A though. | 16:04 |
| *** markmcclain has quit IRC | 16:04 | |
| dhellmann | bnemec: maybe 3, for a spec? I don't think I need to be involved in all of them, though I'll obviously try | 16:32 |
| dhellmann | bnemec: this would be a good topic for the meeting this week, would you add it to the agenda (I have a meeting starting shortly) | 16:32 |
| bnemec | dhellmann: Yep, sounds good. | 16:33 |
| bnemec | At one point there was talk of requiring all cores to sign off on all specs in tripleo. I suspect that's not actually going to happen. ;-) | 16:33 |
| dhellmann | bnemec: the more I think about it, the more I like the idea of using more than 2 votes. For the governance repo no one has -2, and only ttx has +2. Maybe we could configure our specs repo like that. | 16:33 |
| dhellmann | bnemec: that might be excessive, but I would definitely like as much input as possible | 16:34 |
| dhellmann | s/that/all cores/ | 16:34 |
| bnemec | dhellmann: Yeah, and I think part of the motivation was to keep all the cores up on what's going on in the project too. | 16:34 |
| dhellmann | another good reason | 16:35 |
| bnemec | Oh look, I have a meeting too. :-) | 16:36 |
| *** dims_ has joined #openstack-oslo | 16:38 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/oslo-incubator: Fix Sphinx directive name https://review.openstack.org/97850 | 16:38 |
| *** dims_ has quit IRC | 16:43 | |
| *** jungleboyj has quit IRC | 16:47 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Handle . and - in translation domains https://review.openstack.org/92682 | 16:51 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Split up monolithic test file https://review.openstack.org/92681 | 16:51 |
| *** arnaud has joined #openstack-oslo | 16:58 | |
| openstackgerrit | Robert Clark proposed a change to openstack/oslo-incubator: Incorrect block-size in docstrings Crypto utils file documentation referred to 128 bit crypt but appears to perform 256 bit changed documentation to match https://review.openstack.org/97822 | 17:10 |
| *** shakamunyi has quit IRC | 17:13 | |
| *** dims_ has joined #openstack-oslo | 17:20 | |
| *** dims_ has quit IRC | 17:23 | |
| *** dims_ has joined #openstack-oslo | 17:23 | |
| *** praneshp has joined #openstack-oslo | 17:28 | |
| *** ildikov has joined #openstack-oslo | 17:34 | |
| openstackgerrit | Ildiko Vancsa proposed a change to openstack/oslo-incubator: Add source code generation to conf.py https://review.openstack.org/97862 | 17:38 |
| ildikov | dhellmann: hi. are you around? | 17:39 |
| *** harlowja_away is now known as harlowja | 17:39 | |
| dhellmann | ildikov: hi | 17:41 |
| ildikov | dhellmann: I just wanted ask if you thought something like this as docs job for oslo: https://review.openstack.org/97862 | 17:42 |
| dhellmann | ildikov: yes, we should do something like that and we should turn on the flag to prevent sphinx errors, too | 17:43 |
| dhellmann | we should probably move this auto-generation stuff into oslosphinx instead of coyping the function between conf.py files, but that can come later | 17:43 |
| ildikov | dhellmann: I saw a generate_autoindex in pbr too | 17:44 |
| dhellmann | ildikov: oh, interesting, maybe we can use that? | 17:44 |
| ildikov | but I'm not against to put it to oslosphinx later, if that seems to be a better option | 17:44 |
| dhellmann | oslosphinx makes more sense, but if it is already in pbr then that works, too | 17:45 |
| ildikov | it's here: http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/packaging.py#n658 | 17:45 |
| dhellmann | ah! | 17:45 |
| ildikov | not the same as the one we use, I did not have time to check what this is for especially | 17:45 |
| dhellmann | let's see if that produces something like what we want and use that instead of adding to conf.py | 17:45 |
| ildikov | I just saw it the last time, when we tried to figure out why the doc job does not fail in case of errors | 17:46 |
| dhellmann | oh, the difference there is it would only work if you run "python setup.py build_sphinx" where the change in conf.py works if you run sphinx-build directly | 17:46 |
| dhellmann | ildikov: pbr may need more documentation :-) | 17:47 |
| ildikov | but the previous one uses this only in this case: http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/packaging.py#n745 | 17:47 |
| ildikov | agreed ;) | 17:47 |
| dhellmann | http://docs.openstack.org/developer/pbr/ | 17:47 |
| dhellmann | that does mention the autodoc feature, but I don't see anything about turning it on | 17:47 |
| ildikov | me neither, I guess it's similar to the warnerrors, but I will try to check | 17:48 |
| dhellmann | ok, thanks | 17:48 |
| ildikov | if I can turn it on, than we can see, what should be done with it to use this version and get the same output as with the conf.py version | 17:49 |
| ildikov | dhellmann: cool, thanks, more work again :) | 17:50 |
| dhellmann | ildikov: I appreciate you taking all of this on. :-) | 17:50 |
| ildikov | dhellmann: the problem is that I like challenges and this doc clean up looks like one... :) | 17:52 |
| *** jungleboyj has joined #openstack-oslo | 17:52 | |
| ildikov | dhellmann: and also, if others do not have to do this again, then I think it worth the effort | 17:52 |
| ildikov | dhellmann: the other thing is that there were some not just sphinx formatting related errors in the sphinx build | 17:53 |
| dhellmann | ildikov: it is definitely helpful | 17:53 |
| dhellmann | oh, you found worse problems? | 17:53 |
| ildikov | dhellmann: at least in my env | 17:53 |
| *** tkelsey has quit IRC | 17:55 | |
| ildikov | dhellmann: I saw error messages like this: AttributeError: 'Module_six_moves_urllib_parse' object has no attribute 'SplitResult' | 17:55 |
| ildikov | dhellmann: but it was in my env | 17:55 |
| ildikov | dhellmann: I will check the log of the job on the gate | 17:56 |
| ildikov | dhellmann: I found some code related issues in Ceilometer with sphinx-build, so now at least I know that it is possible | 17:57 |
| dhellmann | ildikov: ok, I don't recognize that error message | 18:00 |
| ildikov | dhellmann: ok, cool, then the problem is in my env | 18:00 |
| dhellmann | well, it could be real and I just haven't seen it :-) | 18:00 |
| ildikov | dhellmann: human eyes have a selective way of working, so it's not impossible :) | 18:01 |
| ildikov | dhellmann: there was one error on the gate that I haven't seen locally, however it had to be there... | 18:02 |
| ildikov | dhellmann: I will let you know if I have any results or any more questions with pbr | 18:03 |
| ildikov | dhellmann: and I end this day now :) | 18:03 |
| dhellmann | ildikov: have a good evening :-) | 18:06 |
| ildikov | dhellmann: thanks :) | 18:07 |
| ildikov | dhellmann: have a nice day/afternoon | 18:08 |
| *** praneshp_ has joined #openstack-oslo | 18:08 | |
| *** praneshp has quit IRC | 18:09 | |
| *** praneshp_ is now known as praneshp | 18:09 | |
| *** arnaud has quit IRC | 18:51 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo-specs: add spec for remove-context-adapter https://review.openstack.org/95870 | 18:55 |
| dhellmann | dims: ping? | 18:57 |
| *** rpodolyaka1 has quit IRC | 19:13 | |
| *** devananda has joined #openstack-oslo | 19:15 | |
| *** praneshp has quit IRC | 19:20 | |
| *** rpodolyaka1 has joined #openstack-oslo | 19:29 | |
| rpodolyaka1 | dhellmann: hey! are you around? | 19:48 |
| *** HenryG has quit IRC | 19:51 | |
| openstackgerrit | Ken Giusti proposed a change to openstack/oslo.messaging: An initial implementation of a 1.0 based messaging driver https://review.openstack.org/75815 | 20:02 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add in default transcation isolation levels https://review.openstack.org/97902 | 20:03 |
| openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: Graduate oslo.serialization https://review.openstack.org/97315 | 20:04 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add in default transcation isolation levels https://review.openstack.org/97902 | 20:06 |
| *** rpodolyaka1 has quit IRC | 20:14 | |
| *** orzel has joined #openstack-oslo | 20:18 | |
| orzel | hello | 20:18 |
| orzel | I've been struggling with a problem for days. I've even re-installed the whole thing from scratch, only to get exactly the same error. I can create/delete volumes, but i can't attach them : http://pastebin.com/EV316LBS | 20:18 |
| orzel | the stacktrace is full of 'oslo', i'm not really sure about what this means | 20:19 |
| orzel | my openstack is icehouse, and oslo.messaging is 1.3.0 | 20:19 |
| *** ajo|mac has joined #openstack-oslo | 20:20 | |
| bnemec | orzel: That looks like a cinder problem (see lines 3 and 4 in your paste). I think oslo is just logging because it sees that an exception happened. | 20:22 |
| bnemec | dhellmann: I'm wondering if threadgroup and eventlet_backdoor were put into oslo.server because they use eventlet and we were trying to keep the contamination to a minimum. :-) | 20:25 |
| bnemec | They both look fairly generic otherwise. | 20:25 |
| orzel | bnemec: how can i make oslo less verbose (not diplayig "TRACE") ? | 20:30 |
| bnemec | orzel: I'm not positive, but you might be able to do it with default_log_levels in the service's conf file: https://github.com/openstack/oslo-incubator/blob/master/openstack/common/log.py#L166 | 20:36 |
| bnemec | I'm not sure off hand what level TRACE corresponds to though. | 20:37 |
| dhellmann | bnemec: you might be right about those modules | 20:42 |
| dhellmann | I might have thought the backdoor thing really only made sense in a server, too | 20:42 |
| dhellmann | rpodolyaka: here | 20:43 |
| bnemec | dhellmann: Yeah. It occurs to me that since we're cleaning greenthreads out of processutils, we probably don't want to introduce another module with an explicit dependency on them. | 20:43 |
| dhellmann | yeah | 20:43 |
| openstackgerrit | Ben Nemec proposed a change to openstack/oslo-specs: Graduate oslo.concurrency https://review.openstack.org/97296 | 20:44 |
| *** jecarey has quit IRC | 20:53 | |
| *** harlowja has quit IRC | 20:54 | |
| *** harlowja has joined #openstack-oslo | 20:54 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Check the lazy flag at runtime https://review.openstack.org/97931 | 20:59 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Update the public API of the library https://review.openstack.org/97932 | 20:59 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Improve initial documentation https://review.openstack.org/96961 | 21:17 |
| *** ajo|mac has quit IRC | 21:30 | |
| *** jungleboyj has left #openstack-oslo | 21:41 | |
| *** orzel has quit IRC | 21:44 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add in default transaction isolation levels https://review.openstack.org/97902 | 21:50 |
| *** mriedem has quit IRC | 21:54 | |
| *** markmcclain has joined #openstack-oslo | 21:58 | |
| *** mkoderer has quit IRC | 22:06 | |
| *** mkoderer has joined #openstack-oslo | 22:11 | |
| *** bknudson has quit IRC | 22:23 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Allow the mysql mode to be more than just TRADITIONAL https://review.openstack.org/97964 | 22:31 |
| *** arnaud has joined #openstack-oslo | 22:42 | |
| *** dims_ has quit IRC | 22:51 | |
| *** markmcclain has quit IRC | 23:08 | |
| *** dims_ has joined #openstack-oslo | 23:08 | |
| *** mrodden has quit IRC | 23:15 | |
| *** mrda-away is now known as mrda | 23:24 | |
| ekarlso | any new cut of of o.db coming soon ? | 23:31 |
| *** mrodden has joined #openstack-oslo | 23:36 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/hacking: Make import checks properly detect blank lines and comments https://review.openstack.org/97988 | 23:39 |
| *** praneshp has joined #openstack-oslo | 23:42 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!