Monday, 2014-04-07

« Sunday, 2014-04-06 Index Tuesday, 2014-04-08 »
*** yamahata has joined #openstack-oslo00:36
*** yamahata has quit IRC01:45
*** yamahata has joined #openstack-oslo03:30
*** zhiyan_ is now known as zhiyan03:50
*** zhiyan is now known as zhiyan_04:39
openstackgerritIan Wienand proposed a change to openstack/oslo-incubator: Don't call provision.py directly  https://review.openstack.org/8563405:11
openstackgerritIan Wienand proposed a change to openstack/oslo-incubator: Add OSLO_LOCK_PATH if not set  https://review.openstack.org/8563705:33
openstackgerritIan Wienand proposed a change to openstack/oslo-incubator: Add funcName to audit/info message output  https://review.openstack.org/8563805:45
*** ihrachyshka has joined #openstack-oslo06:10
*** ihrachyshka has quit IRC06:40
*** flaper87|afk is now known as flaper8707:04
*** ihrachyshka has joined #openstack-oslo07:23
*** I159 has joined #openstack-oslo07:35
*** dshulyak has joined #openstack-oslo08:16
*** rpodolyaka has quit IRC08:31
*** rpodolyaka1 has joined #openstack-oslo08:31
*** rpodolyaka1 has left #openstack-oslo08:46
*** rpodolyaka1 has joined #openstack-oslo08:46
*** arosen has quit IRC09:07
*** ihrachyshka has quit IRC09:15
*** stannie has joined #openstack-oslo09:32
*** ihrachyshka has joined #openstack-oslo09:48
*** killzone has quit IRC10:03
openstackgerritVictor Stinner proposed a change to openstack/oslo-incubator: Fix unit/rpc/test_common.py on Python 3  https://review.openstack.org/8053310:07
*** sreshetnyak has joined #openstack-oslo10:14
openstackgerritVictor Stinner proposed a change to openstack/oslo-incubator: Fix safe_encode(): return bytes on Python 3  https://review.openstack.org/8057110:17
*** sreshetnyak has quit IRC10:27
*** sreshetnyak has joined #openstack-oslo10:27
openstackgerritVictor Stinner proposed a change to openstack/oslo-incubator: Partial fix of test_strutils.py on Python 3  https://review.openstack.org/8057310:42
*** sreshetnyak has quit IRC11:50
*** sreshetnyak has joined #openstack-oslo11:52
*** lbragstad has quit IRC12:01
*** yamahata has quit IRC12:20
*** bogdando has quit IRC12:22
*** bogdando has joined #openstack-oslo12:22
*** jecarey has quit IRC12:29
*** lbragstad has joined #openstack-oslo12:34
*** sergmelikyan has joined #openstack-oslo12:36
sergmelikyandhellmann, I have a few questions about design of oslo.config.12:37
sergmelikyanWhy does it have global options state? (all options registered application wide) and does not follow, for example, Python config module design?12:38
*** ihrachyshka has quit IRC12:40
*** ihrachyshka has joined #openstack-oslo12:40
*** erecio has joined #openstack-oslo12:41
dhellmannsergmelikyan: lots of history there: https://wiki.openstack.org/wiki/Oslo#Why_does_oslo.config_have_a_CONF_object.3F_Global_object_SUCK.2112:55
sergmelikyandhellmann, thx!12:58
*** jecarey has joined #openstack-oslo13:05
openstackgerritVictor Stinner proposed a change to openstack/oslo.messaging: Add an optional timeout parameter to Listener.poll  https://review.openstack.org/7100313:08
*** rustlebee is now known as russellb13:08
*** mriedem has joined #openstack-oslo13:15
*** dims has joined #openstack-oslo13:22
mriedemhey oslophiles, question about this:13:35
mriedemhttps://github.com/openstack/oslo-incubator/blob/master/openstack/common/db/sqlalchemy/utils.py#L25213:35
mriedemturns out that doesn't work if yo'ure creating foreign keys because the metadata has to be shared between table objects for it to work13:35
mriedemi'm thinking about at least putting a NOTE in that code to warn against trying to use it for FK creation, any opinion on that? alternative is add kwarg for metadata and default to None13:36
openstackgerritVictor Stinner proposed a change to openstack/oslo-incubator: New module openstack.common.time_monotonic  https://review.openstack.org/8571713:44
*** yamahata has joined #openstack-oslo13:45
dhellmannmriedem: should that function take an optional metadata arg to handle the case you describe?13:46
mriedemdhellmann: that's what i was thinking....and was just writing a bug but then cancelled it13:46
openstackgerritgaryk proposed a change to openstack/oslo.vmware: Remove __del__ usage in oslo.vmware driver  https://review.openstack.org/8278113:46
mriedembecause if you provide metadata, the method is really only doing one thing for you, creating the table13:46
dhellmannmriedem: well, good point13:46
mriedemdhellmann: so the utility becomes less of a utility at that point13:46
mriedemdhellmann: at a minimum i'll push up a patch that adds a note to not use it's current form for FK creation13:47
mriedemso someone else doesn't lose a few days like me :)13:47
dhellmannmriedem: the note sounds good, maybe in the docstring so it ends up in the generated docs, too13:47
mriedemdhellmann: sure, will do13:47
dhellmannmriedem: I wonder how widely used that function is, if it's not useful for making tables with foreign keys. I guess it's still useful for examining the current table structure?13:48
mriedemdhellmann: yeah, you can still use it for other things like altering columns, creating/dropping unique constraints, etc13:48
dhellmannmriedem: makes sense13:49
dhellmanna nice ".. warning:: Do not use for creating fkeys because..." does seem in order13:49
mriedemdhellmann: yup, doing it now13:50
mriedemi'll open a bug to back it up with the stacktrace you get if you use it incorrectly for more details13:50
dhellmannmriedem: perfect, thanks13:53
openstackgerritMatt Riedemann proposed a change to openstack/oslo-incubator: Add a warning to not use get_table for working with ForeignKeys  https://review.openstack.org/8572214:18
openstackgerritVictor Stinner proposed a change to openstack/oslo-incubator: timeutils: add time_monotonic() function  https://review.openstack.org/8571714:21
openstackgerritVictor Stinner proposed a change to openstack/oslo-incubator: timeutils: add time_monotonic() function  https://review.openstack.org/8571714:49
*** yamahata has quit IRC15:04
*** yamahata has joined #openstack-oslo15:04
*** ihrachyshka has quit IRC15:24
openstackgerritDoug Hellmann proposed a change to openstack/cliff: Remove PrettyTable from documentation requirements  https://review.openstack.org/8451515:36
openstackgerritCyril Roelandt proposed a change to openstack/oslo.config: Fix test_version on Python 3.4  https://review.openstack.org/8560815:46
openstackgerritMatt Riedemann proposed a change to openstack/oslo-incubator: Add a warning to not use get_table for working with ForeignKeys  https://review.openstack.org/8572215:54
dhellmannwhat's the deal with run_tests.sh? is that still used (on purpose)? I see a few changes related to making it work, should we just remove it instead?16:01
openstackgerritRussell Bryant proposed a change to openstack/oslo.messaging: Update ensure() to catch all Exceptions.  https://review.openstack.org/8575016:01
*** I159 has quit IRC16:12
*** markmcclain has joined #openstack-oslo16:33
*** beekneemech is now known as bnemec16:34
bnemecdhellmann: It might be time to revisit removing run_tests.sh.  I know there have been some improvements to tox and friends to address the things people were still using run_tests for.16:35
bnemecAnd I think run_tests.sh is broken more often than not since hardly anyone uses it anymore. :-)16:35
*** harlowja has joined #openstack-oslo16:56
dhellmannbnemec: yeah, that's sort of what I thought16:58
dhellmannbnemec: we could also make it just run tox, I suppose16:58
*** markmcclain has quit IRC16:59
*** markmcclain has joined #openstack-oslo17:00
*** markmcclain has quit IRC17:00
openstackgerritJenkins proposed a change to openstack/cliff: Updated from global requirements  https://review.openstack.org/8312017:29
*** mriedem has quit IRC17:33
openstackgerritJenkins proposed a change to openstack/pycadf: Updated from global requirements  https://review.openstack.org/8576517:36
openstackgerritJenkins proposed a change to openstack/stevedore: Updated from global requirements  https://review.openstack.org/7559517:38
openstackgerritCyril Roelandt proposed a change to openstack/oslo.config: Fix test_version on Python 3.4  https://review.openstack.org/8560817:44
*** mriedem has joined #openstack-oslo17:44
*** arnaud has joined #openstack-oslo17:48
openstackgerritRussell Bryant proposed a change to openstack/oslo.messaging: Update ensure() to catch all Exceptions  https://review.openstack.org/8575017:53
russellbthat period in commit message check drives me crazy18:03
harlowjarussellb +1, i don't quite get it18:13
*** hartsocks has joined #openstack-oslo18:20
*** morganfainberg_Z is now known as morganfainberg18:39
*** markmcclain has joined #openstack-oslo18:39
openstackgerritRussell Bryant proposed a change to openstack/oslo.messaging: Update ensure() to catch all MessagingExceptions  https://review.openstack.org/8575018:47
openstackgerritRussell Bryant proposed a change to openstack/oslo.messaging: Update ensure() to catch MessagingError  https://review.openstack.org/8575018:47
*** markmcclain has quit IRC19:26
*** harlowja has quit IRC20:03
*** dims has quit IRC20:08
*** russellb_ has joined #openstack-oslo20:09
*** boris-42 has quit IRC20:10
*** russellb has quit IRC20:10
*** russellb_ is now known as russellb20:10
*** markmcclain has joined #openstack-oslo20:18
*** harlowja has joined #openstack-oslo20:19
*** boris-42 has joined #openstack-oslo20:28
*** markmcclain has quit IRC20:32
*** markmcclain has joined #openstack-oslo20:32
*** erecio has quit IRC20:49
*** arnaud has quit IRC20:59
openstackgerritJenkins proposed a change to openstack-dev/hacking: Updated from global requirements  https://review.openstack.org/8581621:06
*** GheRivero_ has joined #openstack-oslo21:10
openstackgerritJenkins proposed a change to openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/8583121:15
openstackgerritJenkins proposed a change to openstack/taskflow: Updated from global requirements  https://review.openstack.org/8584221:17
openstackgerritJenkins proposed a change to openstack/cliff: Updated from global requirements  https://review.openstack.org/8312021:19
*** harlowja is now known as harlowja_away21:31
*** harlowja_away is now known as harlowja21:35
*** GheRivero has quit IRC21:55
*** GheRivero_ is now known as GheRivero21:55
*** mriedem has quit IRC21:56
*** markmcclain has quit IRC22:02
*** mriedem has joined #openstack-oslo22:06
*** markmcclain has joined #openstack-oslo22:06
*** jecarey_ has joined #openstack-oslo22:19
*** jecarey has quit IRC22:19
*** lbragstad has quit IRC22:24
*** jecarey_ has quit IRC22:25
*** arnaud has joined #openstack-oslo22:34
*** zhiyan_ has quit IRC22:34
*** zhiyan_ has joined #openstack-oslo22:35
*** mriedem has quit IRC22:52
*** yamahata has quit IRC23:24
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Integrate urlparse for configuration augmentation  https://review.openstack.org/8549423:32
*** flaper87 is now known as flaper87|afk23:33
openstackgerritBen Nemec proposed a change to openstack/oslo-incubator: Add test case for config generator hostname check  https://review.openstack.org/8359423:33
*** markmcclain has quit IRC23:35
openstackgerritJoshua Harlow proposed a change to openstack/taskflow: Allow for only iterating over the most 'fresh' jobs  https://review.openstack.org/8552023:35
*** markmcclain has joined #openstack-oslo23:35
*** markmcclain has quit IRC23:52
« Sunday, 2014-04-06 Index Tuesday, 2014-04-08 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!