2014-03-10T00:28:22 *** yamahata has quit IRC 2014-03-10T01:24:43 *** zhiyan_ is now known as zhiyan 2014-03-10T01:30:48 *** yamahata has joined #openstack-oslo 2014-03-10T01:36:21 *** gcb has joined #openstack-oslo 2014-03-10T01:45:13 *** xuhanp has joined #openstack-oslo 2014-03-10T02:01:18 *** mriedem has left #openstack-oslo 2014-03-10T02:09:29 *** yamahata has quit IRC 2014-03-10T03:44:19 *** yamahata has joined #openstack-oslo 2014-03-10T05:46:28 *** hdd_ has quit IRC 2014-03-10T06:08:32 *** regard has quit IRC 2014-03-10T08:14:04 *** flaper87|afk is now known as flaper87 2014-03-10T08:21:05 Ildiko Vancsa proposed a change to openstack/oslo-incubator: Add copyright text to openstack/common/__init__.py https://review.openstack.org/79273 2014-03-10T08:28:34 *** ihrachys|afk is now known as ihrachys 2014-03-10T10:18:59 *** xuhanp has quit IRC 2014-03-10T10:25:09 Nikola Dipanov proposed a change to openstack-dev/hacking: Make H202 check honor pep8 #noqa comment https://review.openstack.org/79287 2014-03-10T10:49:18 *** lbragstad has joined #openstack-oslo 2014-03-10T10:50:35 *** lbragstad__ has quit IRC 2014-03-10T11:03:36 Nikola Dipanov proposed a change to openstack-dev/hacking: Make H202 check honor pep8 #noqa comment https://review.openstack.org/79287 2014-03-10T11:17:08 Nikola Dipanov proposed a change to openstack-dev/hacking: Make H202 check honor pep8 #noqa comment https://review.openstack.org/79287 2014-03-10T11:51:03 *** lbragstad has quit IRC 2014-03-10T11:54:48 *** yamahata has quit IRC 2014-03-10T12:27:49 *** lbragstad has joined #openstack-oslo 2014-03-10T12:34:08 *** mriedem has joined #openstack-oslo 2014-03-10T12:40:03 Flavio Percoco proposed a change to openstack/oslo-incubator: User a more accurate max_delay for reconnects https://review.openstack.org/79304 2014-03-10T12:44:05 *** dims_ has quit IRC 2014-03-10T12:51:51 Cyril Roelandt proposed a change to openstack/oslosphinx: Add Python 3 trove classifiers https://review.openstack.org/79311 2014-03-10T12:56:27 *** dims_ has joined #openstack-oslo 2014-03-10T13:11:19 *** gcb has quit IRC 2014-03-10T13:26:23 *** rustlebee is now known as russellb 2014-03-10T13:44:08 *** yamahata has joined #openstack-oslo 2014-03-10T13:58:58 *** yamahata has quit IRC 2014-03-10T14:02:03 *** beekneemech is now known as bnemec 2014-03-10T14:04:05 *** yamahata has joined #openstack-oslo 2014-03-10T14:08:45 *** yamahata has quit IRC 2014-03-10T14:15:20 *** yamahata has joined #openstack-oslo 2014-03-10T14:20:09 *** yamahata has quit IRC 2014-03-10T14:24:35 Ildiko Vancsa proposed a change to openstack/oslo-incubator: Add copyright text to openstack/common/__init__.py https://review.openstack.org/79273 2014-03-10T14:26:19 *** yamahata has joined #openstack-oslo 2014-03-10T14:33:58 *** yamahata has quit IRC 2014-03-10T14:36:11 *** yamahata has joined #openstack-oslo 2014-03-10T14:41:11 *** yamahata__ has joined #openstack-oslo 2014-03-10T14:42:02 *** yamahata has quit IRC 2014-03-10T14:43:23 *** yamahata__ has quit IRC 2014-03-10T14:45:36 *** yamahata has joined #openstack-oslo 2014-03-10T14:48:18 *** yamahata has quit IRC 2014-03-10T14:50:55 *** yamahata has joined #openstack-oslo 2014-03-10T14:51:30 Jenkins proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/77644 2014-03-10T14:52:51 *** hdd_ has joined #openstack-oslo 2014-03-10T14:52:52 therese-mchale proposed a change to openstack/oslo-incubator: Retry running queries when not in a transaction https://review.openstack.org/76136 2014-03-10T14:52:55 Jenkins proposed a change to openstack/stevedore: Updated from global requirements https://review.openstack.org/75595 2014-03-10T15:07:08 *** corvus is now known as jeblair 2014-03-10T15:13:22 *** hdd_ has quit IRC 2014-03-10T15:45:01 *** hdd has joined #openstack-oslo 2014-03-10T15:46:32 anyone else having problems using run_tests.sh for oslo-incubator? 2014-03-10T15:46:44 I'm getting "ImportError: No module named openstack.common.db" 2014-03-10T15:48:35 hdd: I usually call tox directly to run the tests, have you tried that? 2014-03-10T15:54:08 dhellmann, I'll give it a whirl 2014-03-10T15:54:26 hdd, fwiw I did just try "run_tests.sh" and saw the same error 2014-03-10T15:54:42 I'm running tox now on master to see if it shows up there, too 2014-03-10T15:56:54 hdd: tests pass for me with "tox -e py27" so I don't know if the issue is with the virtualenv created by run_tests.sh or the different way it runs the tests 2014-03-10T15:58:51 yeah, tox works for me too 2014-03-10T15:58:53 thanks 2014-03-10T16:07:17 Zhi Yan Liu proposed a change to openstack/oslo-incubator: Making table utf-8 charset checking be optional for DB migration https://review.openstack.org/75356 2014-03-10T16:08:40 dims, did we ever announce that we were creating this channel on the mailing list? 2014-03-10T16:09:26 dhellmann, don't think so 2014-03-10T16:10:09 Zhi Yan Liu proposed a change to openstack/oslo-incubator: Making table utf-8 charset checking be optional for DB migration https://review.openstack.org/75356 2014-03-10T16:10:13 dims_: ok, I'll do that now 2014-03-10T16:13:30 *** yamahata has quit IRC 2014-03-10T16:14:02 Zhi Yan Liu proposed a change to openstack/oslo-incubator: Making table utf-8 charset checking be optional for DB migration https://review.openstack.org/75356 2014-03-10T16:20:02 hi folks, i'd like to raise a review request for https://review.openstack.org/75356 in this "preferred" channel ;) 2014-03-10T16:21:34 Joe Gordon proposed a change to openstack-dev/hacking: Revert "Make H202 check honor pep8 #noqa comment" https://review.openstack.org/79354 2014-03-10T16:25:16 *** manask has joined #openstack-oslo 2014-03-10T16:27:26 *** manask has left #openstack-oslo 2014-03-10T16:32:26 hi, zhiyan 2014-03-10T16:33:07 zhiyan: do you consider that change critical for the release candidate? 2014-03-10T16:33:55 dhellmann: i think it is a important "fix" for glance, at least 2014-03-10T16:34:27 zhiyan: ok, there wasn't a priority attached to the bug report yet, so I wasn't sure how big of an issue it was 2014-03-10T16:34:50 dhellmann: without it, deployer will failed to do db migration for glance 2014-03-10T16:35:01 zhiyan: that sounds pretty critical :-) 2014-03-10T16:35:26 zhiyan: how does the deployer set the option to not require utf8? or is that something glance would do? 2014-03-10T16:35:35 dhellmann: could you pls help set a bug priority for oslo one? 2014-03-10T16:35:49 dhellmann: 2 secs, i have one patch for it 2014-03-10T16:36:06 zhiyan: the change in glance is only marked medium 2014-03-10T16:36:18 dhellmann: https://review.openstack.org/#/c/75865/ 2014-03-10T16:36:44 dhellmann: yes, it now is. i can set it up if needed 2014-03-10T16:37:28 so every project that wants to use this flag would have to update the command they use to run the migrations to add the new switch? 2014-03-10T16:38:06 dhellmann: tbh i'm sure glance project only currently 2014-03-10T16:38:16 I wonder if this should have been opt-in. 2014-03-10T16:38:21 We hit it in heat in tripleo too. 2014-03-10T16:38:42 Our fix was to globally enable utf8 tables in our mysql config, but in retrospect that probably should have been a red flag. 2014-03-10T16:38:52 bnemec: yes, I'm starting to think so 2014-03-10T16:39:28 I wonder if we make any assumptions elsewhere that we are using utf8 in the database 2014-03-10T16:41:06 dhellmann: The original bug that prompted this was here: https://bugs.launchpad.net/heat/+bug/1261605 2014-03-10T16:42:06 Chris Buccella proposed a change to openstack/oslo-incubator: Don't store the request ID value in middleware https://review.openstack.org/79358 2014-03-10T16:43:49 bnemec: thanks 2014-03-10T16:44:46 bnemec: hi, iirc, there's a change which changes heat db create CLI script, to use utf8 as the db default charset, right? 2014-03-10T16:45:16 zhiyan: Yes, there was a separate bug raised and fixed about that: https://bugs.launchpad.net/heat/+bug/1276880 2014-03-10T16:45:17 dhellmann: btw, thanks for you review/pickup 2014-03-10T16:45:31 But it still doesn't help you if you have an existing install with incorrect encoding. 2014-03-10T16:46:07 Which is going to cause deployer headaches on H->I upgrades. 2014-03-10T16:46:13 bnemec: that original bug makes me think that if we're not consistent, we will continue to have failures with foreign keys between tables where the encodings don't match 2014-03-10T16:46:24 or charsets, I guess, which may not be the same thing as encoding 2014-03-10T16:46:47 Yeah, I don't know if they tested that. 2014-03-10T16:46:53 so I think we want all the tables to have the same charset, even if it isn't utf-8 2014-03-10T16:47:46 bnemec: indeed! that's what i want to say here, seems currently we can only give a flag to db_sync to make it be possible, i mean execute migration to auto "fix" utf8 issue (and foreign keys issue maybe) 2014-03-10T16:49:22 dhellmann: agree 2014-03-10T16:49:46 *** harlowja has joined #openstack-oslo 2014-03-10T16:50:38 zhiyan: so if we want the charsets to match, then just disabling the utf-8 check isn't going to do the right thing 2014-03-10T16:50:52 isn't there a migration to update the charsets of all tables? that's probably very expensive 2014-03-10T16:51:45 Here's now nova handled it: https://github.com/openstack/nova/blob/0a2ab98491cbf62e148e8289bc69c26357bd136c/nova/db/sqlalchemy/migrate_repo/versions/170_havana.py#L1395 2014-03-10T16:51:49 dhellmann: zhiyan: ^ 2014-03-10T16:52:54 bnemec: dhellmann: so i prepared this https://review.openstack.org/#/c/75898/5/glance/db/sqlalchemy/migrate_repo/versions/035_mysql_set_utf8_charset.py 2014-03-10T16:53:43 but, as we know, deployer need a chance to execute it. 2014-03-10T16:55:30 zhiyan: I think I need to see something explaining the bigger picture. What effect does it have on glance if the charset is no longer consistent across tables? How does a deployer fix up the charset and tables? That sort of thing. 2014-03-10T16:55:45 I think I understand some of the issues, but I'm not sure I understand how the proposed solution fixes them, if that makes sense. 2014-03-10T16:56:21 If you could add some details to the bug report answering some of those questions, that would help me with the review. 2014-03-10T16:57:19 dhellmann: ok, so pls raise you questions in the bug report. it will help us to track them. 2014-03-10T16:57:44 zhiyan: sure, I can do that 2014-03-10T16:59:45 dhellmann: thanks again. no rush, but tbh you know this issue might break glance now db_sync under particular deployment, so i think it is a worth fix for the coming release. 2014-03-10T17:07:06 *** yamahata__ has joined #openstack-oslo 2014-03-10T17:11:55 *** boris-42 has joined #openstack-oslo 2014-03-10T17:12:05 Nice 2014-03-10T17:12:11 Finially we have oslo chat! 2014-03-10T17:14:39 hi, boris-42 ! 2014-03-10T17:15:00 dhellmann hi! 2014-03-10T17:15:08 Ildiko Vancsa proposed a change to openstack/oslo-incubator: Add copyright text to openstack/common/__init__.py https://review.openstack.org/79273 2014-03-10T17:16:08 dhellmann I think this will be quite popular chat=) 2014-03-10T17:16:17 *** saikrishna_ has joined #openstack-oslo 2014-03-10T17:17:27 boris-42: :-) 2014-03-10T17:18:11 *** yamahata__ has quit IRC 2014-03-10T17:18:29 *** yamahata__ has joined #openstack-oslo 2014-03-10T17:18:33 *** yamahata__ has quit IRC 2014-03-10T17:18:55 *** yamahata has joined #openstack-oslo 2014-03-10T17:23:28 *** lbragstad has left #openstack-oslo 2014-03-10T17:23:38 *** dhellmann is now known as dhellmann_ 2014-03-10T17:24:39 *** lbragstad has joined #openstack-oslo 2014-03-10T17:57:41 *** saikrishna_ has quit IRC 2014-03-10T18:02:10 *** mriedem1 has joined #openstack-oslo 2014-03-10T18:04:04 *** mriedem has quit IRC 2014-03-10T18:08:14 *** zhiyan is now known as zhiyan_ 2014-03-10T18:13:59 dhellmann_ nice blog post btw, a+ 2014-03-10T18:14:00 :) 2014-03-10T18:14:37 for others, http://doughellmann.com/2014/03/04/the-intersection-of-the-openstack-and-python-communities.html 2014-03-10T18:14:50 “That is a lot of code. Why do we have so much code?”, lol 2014-03-10T18:15:41 *** SergeyLukjanov has joined #openstack-oslo 2014-03-10T18:19:43 *** zhiyan_ is now known as zhiyan 2014-03-10T18:23:10 harlowja: thanks :-) 2014-03-10T18:23:13 *** dhellmann_ is now known as dhellmann 2014-03-10T18:25:04 why do we have so much code, lol 2014-03-10T18:25:27 questions most people don't ask :-) 2014-03-10T18:28:18 :) 2014-03-10T18:31:05 Joshua Harlow proposed a change to openstack/taskflow: Lock test tweaks https://review.openstack.org/79401 2014-03-10T18:34:00 *** zhiyan is now known as zhiyan_ 2014-03-10T18:43:49 dims_: could you join #openstack-cinder quick? 2014-03-10T18:52:34 intersting, getting taskflow notifications in here also 2014-03-10T18:53:18 dims_: https://bugs.launchpad.net/cinder/+bug/1290503 2014-03-10T18:55:13 harlowja: all of the oslo repos should notify here now 2014-03-10T18:55:33 dhellmann k, good to know 2014-03-10T18:57:06 *** rpodolyaka has joined #openstack-oslo 2014-03-10T19:04:24 *** rpodolyaka has quit IRC 2014-03-10T19:06:37 *** rpodolyaka has joined #openstack-oslo 2014-03-10T19:10:57 *** mriedem has joined #openstack-oslo 2014-03-10T19:15:07 *** mriedem1 has quit IRC 2014-03-10T19:21:21 dhellmann: Do you want another look at https://review.openstack.org/#/c/78806/ or should I go ahead and approve? 2014-03-10T19:22:16 bnemec: lgtm, but you can approve 2014-03-10T19:22:54 dhellmann: Okay, cool. Just wanted to check because you had left a lot of comments on it. :-) 2014-03-10T19:23:02 bnemec: yep, thanks :-) 2014-03-10T19:34:43 mriedem, ack 2014-03-10T19:36:06 dims_: opened https://bugs.launchpad.net/cinder/+bug/1290503, was wondering if any of that made sense to you since you worked on that domain context stuff 2014-03-10T19:36:18 but now i'm wondering if it's related to bug 1288049 2014-03-10T19:41:12 *** rpodolyaka has quit IRC 2014-03-10T19:41:43 mriedem, see the differences in the log format string 2014-03-10T19:41:44 oslo-incubator/etc/openstack.conf.sample: #logging_context_format_string=%(asctime)s.%(msecs)03d %(process)d %(levelname)s %(name)s [%(request_id)s %(user_identity)s] %(instance)s%(message)s 2014-03-10T19:41:44 nova/etc/nova.conf.sample: #logging_context_format_string=%(asctime)s.%(msecs)03d %(process)d %(levelname)s %(name)s [%(request_id)s %(user)s %(tenant)s] %(instance)s%(message)s 2014-03-10T19:42:16 cinder may be using user 2014-03-10T19:44:02 mriedem, try switching https://github.com/openstack/cinder/blob/master/cinder/openstack/common/log.py#L144 to look like Nova's 2014-03-10T19:44:38 dims_: well nova doesn't use that stuff in it's request context i don't think, nova has it's own request context 2014-03-10T19:44:40 it doesn't use oslo's 2014-03-10T19:44:44 from what i remmeber 2014-03-10T19:52:33 Ben Nemec proposed a change to openstack/oslo-incubator: Be smarter about config generator default package name https://review.openstack.org/79429 2014-03-10T19:54:01 Hah! 20 pending reviews. Pretty sure that's a new record for me. :-) 2014-03-10T19:55:33 bnemec: ++ nice 2014-03-10T19:56:53 lbragstad: Clearly I need to get busy reviewing now. :-) 2014-03-10T19:57:04 :) 2014-03-10T19:57:08 mriedem, that was just the initial thought looking at the stack trace. i am heading out, should be back later today. will look deeper 2014-03-10T20:01:43 Heh, awesome. The utf8 thing came up on the list about an hour after we discussed it here. 2014-03-10T20:01:48 Guess we're not the only ones worried about it. 2014-03-10T20:29:18 gordon chung proposed a change to openstack/pycadf: add docstrings to functions https://review.openstack.org/79440 2014-03-10T20:41:37 *** rpodolyaka has joined #openstack-oslo 2014-03-10T20:46:28 *** rpodolyaka has quit IRC 2014-03-10T20:47:00 *** rpodolyaka has joined #openstack-oslo 2014-03-10T21:18:40 gordon chung proposed a change to openstack/pycadf: add docstrings to functions https://review.openstack.org/79440 2014-03-10T21:36:16 Jenkins proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/77644 2014-03-10T21:58:08 *** dims_ has quit IRC 2014-03-10T22:15:38 *** lbragstad has quit IRC 2014-03-10T22:15:58 *** dims_ has joined #openstack-oslo 2014-03-10T22:32:19 dhellmann off the top of your head do u know if there are bugs/blueprints tracking the oslo.messaging py3.3 adjustments? 2014-03-10T22:37:21 *** mriedem has left #openstack-oslo 2014-03-10T22:54:56 *** yamahata has quit IRC 2014-03-10T22:56:23 *** yamahata has joined #openstack-oslo 2014-03-10T23:04:07 *** rpodolyaka has quit IRC 2014-03-10T23:09:15 *** rpodolyaka has joined #openstack-oslo 2014-03-10T23:22:07 *** flaper87 is now known as flaper87|afk 2014-03-10T23:23:19 Joshua Harlow proposed a change to openstack/oslo.messaging: Start adding a futures executor based executor https://review.openstack.org/70914 2014-03-10T23:31:58 *** yamahata has quit IRC 2014-03-10T23:39:58 Monty Taylor proposed a change to openstack-dev/pbr: WIP: Support -S flag in scripts https://review.openstack.org/79492 2014-03-10T23:45:19 *** beav has quit IRC 2014-03-10T23:46:26 *** beav has joined #openstack-oslo 2014-03-10T23:50:19 *** beav has quit IRC 2014-03-10T23:58:27 *** bknudson has joined #openstack-oslo