Thursday, 2015-01-08

*** takedakn has joined #openstack-oslo00:00
*** sigmavirus24 is now known as sigmavirus24_awa00:01
*** sigmavirus24_awa is now known as sigmavirus2400:01
*** takedakn has quit IRC00:03
*** jaosorior has quit IRC00:03
*** takedakn has joined #openstack-oslo00:03
*** harlowja_at_home has quit IRC00:05
*** salv-orlando has quit IRC00:06
*** salv-orlando has joined #openstack-oslo00:16
*** ajo has quit IRC00:17
*** jaypipes has quit IRC00:18
*** takedakn has quit IRC00:34
*** raildo has quit IRC00:35
*** dougwig is now known as dougwig_the_rude00:41
*** dougwig_the_rude is now known as dougwig00:42
*** alexpilotti has quit IRC01:08
*** tsekiyam_ has joined #openstack-oslo01:23
*** tsekiyama has quit IRC01:26
*** tsekiyam_ has quit IRC01:28
*** mtanino has quit IRC01:28
*** noelbk has quit IRC01:29
*** kgiusti has quit IRC01:31
*** sigmavirus24 is now known as sigmavirus24_awa01:34
*** dims__ has joined #openstack-oslo01:37
*** salv-orlando has quit IRC01:58
*** sabari is now known as zz_sabari02:07
*** zz_sabari is now known as sabari02:21
*** stevemar has quit IRC02:26
*** sabari is now known as zz_sabari02:34
*** gordc has quit IRC02:52
*** salv-orlando has joined #openstack-oslo03:00
*** salv-orlando has quit IRC03:07
*** gordc has joined #openstack-oslo03:09
*** yamahata has quit IRC03:31
openstackgerritMerged openstack/oslo.middleware: Fix bug tracker link in readme  https://review.openstack.org/14557803:49
*** salv-orlando has joined #openstack-oslo04:04
*** dims__ has quit IRC04:06
*** dims__ has joined #openstack-oslo04:08
*** salv-orlando has quit IRC04:08
*** tsufiev has quit IRC04:09
*** tsufiev has joined #openstack-oslo04:09
*** dims__ has quit IRC04:11
*** yamahata has joined #openstack-oslo04:24
*** david-lyle has quit IRC04:50
*** subscope has joined #openstack-oslo04:52
*** noelbk has joined #openstack-oslo05:10
*** gordc has quit IRC05:14
openstackgerritEli Qiao proposed openstack/oslo.db: Make sure sort_key_attr is QueryableAttribute when query  https://review.openstack.org/14363205:15
*** zz_sabari is now known as sabari05:19
*** arnaud__ has joined #openstack-oslo05:22
*** stevemar has joined #openstack-oslo05:22
*** salv-orlando has joined #openstack-oslo05:55
*** salv-orlando has quit IRC06:02
*** rpodolyaka2 has joined #openstack-oslo06:05
*** subscope has quit IRC06:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Imported Translations from Transifex  https://review.openstack.org/14325606:11
*** sabari is now known as zz_sabari06:19
*** zz_sabari is now known as sabari06:23
*** ajo has joined #openstack-oslo06:53
*** e0ne has joined #openstack-oslo07:11
*** rpodolyaka2 has quit IRC07:14
*** arnaud__ has quit IRC07:30
*** rpodolyaka2 has joined #openstack-oslo07:35
*** i159 has joined #openstack-oslo07:36
*** k4n0 has joined #openstack-oslo07:38
*** k4n0 has quit IRC07:42
ttxdhellmann, jogo: rootwrap config file is part of the code -- if users/distros want to customize they can do that in specific directories07:45
*** viktors|afk has quit IRC07:45
*** viktors has joined #openstack-oslo07:45
ttxdhellmann: In distros/FHS rootwrap config files get deployed in /usr/share and you can add your own stuff for example to /etc/rootwrap.d07:46
ttxdefault filters_path=/etc/oslo-rootwrap/filters.d,/usr/share/oslo-rootwrap07:46
ttxyou deploy the ones that come with code to /usr/share/oslo-rootwrap, you add your own to /etc/oslo-rootwrap/filters.d07:47
ttxhmm, I think I understood you wrong -- you're talking about the config file, not the filter files07:48
ttxthen the config file (generally /etc/*-rootwrap/rootwrap.conf) can be customized by usesr, it's a config file.07:49
ttxdistros generally update it to match where they put filters file in packages07:49
*** sabari is now known as zz_sabari07:50
*** zz_sabari is now known as sabari07:53
*** e0ne has quit IRC07:54
*** shardy_z is now known as shardy08:01
*** stevemar has quit IRC08:02
*** rpodolyaka2 has quit IRC08:05
*** alexpilotti has joined #openstack-oslo08:38
openstackgerritZhi Yan Liu proposed openstack/oslo-incubator: Change apiclient to use oslo_utils.uuidutils  https://review.openstack.org/14571708:42
openstackgerritLi Ma proposed openstack/oslo.messaging: Add matchmaker rings for OpenStack projects  https://review.openstack.org/14572208:48
*** salv-orlando has joined #openstack-oslo09:00
*** andreykurilin has joined #openstack-oslo09:01
*** dtantsur|afk is now known as dtantsur09:01
*** noelbk has quit IRC09:05
*** salv-orlando has quit IRC09:06
*** yamahata has quit IRC09:13
*** ihrachyshka has joined #openstack-oslo09:33
*** pblaho_ has joined #openstack-oslo09:40
*** andreykurilin has quit IRC09:45
*** dulek has joined #openstack-oslo09:55
openstackgerritRoman Podoliaka proposed openstack/oslo.db: Ensure mysql_sql_mode is set for MySQLOpportunisticTests  https://review.openstack.org/14562210:02
*** sabari is now known as zz_sabari10:11
*** e0ne has joined #openstack-oslo10:23
*** rushiagr_away is now known as rushiagr10:32
*** rushiagr is now known as rushiagr_away10:33
*** dims__ has joined #openstack-oslo10:34
*** salv-orlando has joined #openstack-oslo10:34
*** dims__ has quit IRC10:38
openstackgerritEli Qiao proposed openstack/oslo.db: Make sure sort_key_attr is QueryableAttribute when query  https://review.openstack.org/14363210:43
*** e0ne_ has joined #openstack-oslo10:48
*** e0ne has quit IRC10:48
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Declare DirectPublisher exchanges with passive=True  https://review.openstack.org/10937310:55
openstackgerritEli Qiao proposed openstack/oslo.db: Make sure sort_key_attr is QueryableAttribute when query  https://review.openstack.org/14363210:59
*** inc0 has joined #openstack-oslo11:03
*** miqui has joined #openstack-oslo11:10
*** denis_makogon has joined #openstack-oslo11:23
*** dims__ has joined #openstack-oslo12:05
*** miqui has quit IRC12:06
*** yassine has joined #openstack-oslo12:16
*** _amrith_ is now known as amrith12:21
*** dims__ has quit IRC12:25
*** dims__ has joined #openstack-oslo12:49
*** dulek has quit IRC12:50
*** inc0 has quit IRC12:52
*** dulek has joined #openstack-oslo12:53
*** alexpilotti has quit IRC13:26
openstackgerritMerged openstack/oslo.messaging: rabbit: fix timeout timer when duration is None  https://review.openstack.org/14552113:35
*** gordc has joined #openstack-oslo13:39
*** e0ne_ has quit IRC13:51
*** jeckersb_gone is now known as jeckersb13:56
*** jaypipes has joined #openstack-oslo13:58
*** e0ne has joined #openstack-oslo13:59
*** pblaho_ has quit IRC13:59
*** kgiusti has joined #openstack-oslo14:05
openstackgerritMerged openstack/oslo.db: Ensure mysql_sql_mode is set for MySQLOpportunisticTests  https://review.openstack.org/14562214:07
*** amrith is now known as _amrith_14:11
*** miqui has joined #openstack-oslo14:14
openstackgerritDoug Hellmann proposed openstack/oslo.utils: fix link to bug tracker in README  https://review.openstack.org/14578914:19
*** mriedem has joined #openstack-oslo14:21
dhellmannoslo.utils 1.2.0 released with the strutils.mask_password fix and ns package changes14:25
kragnizdhellmann: thanks!14:26
*** sigmavirus24_awa is now known as sigmavirus2414:27
sileht114:27
*** alexpilotti has joined #openstack-oslo14:30
dims__nice dhellmann!14:33
*** miqui_ has joined #openstack-oslo14:41
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: [TEST]  https://review.openstack.org/14579814:44
*** yamahata has joined #openstack-oslo14:57
*** _amrith_ is now known as amrith15:10
*** mtanino has joined #openstack-oslo15:11
*** subscope has joined #openstack-oslo15:17
openstackgerritIlya Pekelny proposed openstack/oslo.messaging: Required pyzmq for 0mq driver  https://review.openstack.org/12369115:18
openstackgerritIlya Pekelny proposed openstack/oslo.messaging: Topic object  https://review.openstack.org/14412015:18
openstackgerritIlya Pekelny proposed openstack/oslo.messaging: Reduce global CONF usage in zmq driver  https://review.openstack.org/12371115:18
openstackgerritIlya Pekelny proposed openstack/oslo.messaging: `publisher`function separated into ZmqProxy method  https://review.openstack.org/12409815:18
openstackgerritIlya Pekelny proposed openstack/oslo.messaging: Helper method to check and restrict socket name  https://review.openstack.org/12409915:18
*** takedakn has joined #openstack-oslo15:25
*** takedakn has quit IRC15:27
*** takedakn has joined #openstack-oslo15:27
*** takedakn has quit IRC15:28
*** takedakn has joined #openstack-oslo15:28
*** stevemar has joined #openstack-oslo15:29
*** takedakn has quit IRC15:35
*** jecarey has joined #openstack-oslo15:36
*** e0ne is now known as e0ne_15:47
*** zz_jgrimm is now known as jgrimm15:56
*** e0ne_ has quit IRC15:57
*** tsekiyama has joined #openstack-oslo15:58
*** raildo has joined #openstack-oslo16:00
*** dulek has quit IRC16:00
*** dulek has joined #openstack-oslo16:00
*** e0ne has joined #openstack-oslo16:09
ihrachyshkadhellmann, are you going to propose requirements bumps for new oslo libraries with code moved from namespaces?16:15
ihrachyshkaI only see middleware request16:15
ihrachyshkaah, config only16:16
ihrachyshka*too16:16
ihrachyshkaso i18n and utils are missing bumps16:16
*** dulek has quit IRC16:19
kragnizihrachyshka: isn't this the bump for utils? https://review.openstack.org/#/c/145790/16:22
ihrachyshkakragniz, ouch. I guess my gertty lags behind somehow... thanks16:23
kragnizihrachyshka: haha16:24
kragnizyou're welcome16:24
silehti159, I have found the issue about the heartbeat patch, I have put a comment in the review16:28
*** zzzeek has joined #openstack-oslo16:37
i159sileht: thank you! We even started to develop small testing env to reproduce it. I'll look at your comment tomorrow.16:38
sigmavirus24I wonder if anyone has opinions on https://bugs.launchpad.net/glance/+bug/110140416:40
*** dtantsur is now known as dtantsur|afk16:42
*** i159 has quit IRC16:42
silehti159, I reproduce the issue with a devstack+nova.fake_driver and the largeops tempest tests16:42
silehtarf16:42
*** raildo has quit IRC16:46
*** jecarey has quit IRC16:51
*** jecarey has joined #openstack-oslo17:00
ihrachyshkais cinder failure due to recent oslo.i18n release tracked by someone? http://logs.openstack.org/periodic-stableperiodic-cinder-python26-juno/dd57f60/testr_results.html.gz17:02
*** e0ne has quit IRC17:03
*** andymaier has joined #openstack-oslo17:05
ihrachyshkaah, found commit17:07
ihrachyshkadhellmann, https://review.openstack.org/#/c/145642/1/cinder/test.py May I ask you to check discussion there?17:07
*** gordc has quit IRC17:08
jecareyihrachyshka, I'm working with jungleboyj to look at the test cases and figure out what's going on and either, fix, remove or move them to oslo_i18n.17:09
*** yassine has quit IRC17:14
*** noelbk has joined #openstack-oslo17:15
*** yamahata has quit IRC17:23
*** andymaier has quit IRC17:24
*** zz_sabari is now known as sabari17:25
*** gordc has joined #openstack-oslo17:47
*** rushiagr_away is now known as rushiagr17:48
*** stevemar2 has joined #openstack-oslo17:50
*** stevemar has quit IRC17:50
*** stevemar2 has quit IRC17:56
*** stevemar2 has joined #openstack-oslo17:57
*** yamahata has joined #openstack-oslo17:57
* dhellmann is finally back online18:00
openstackgerritBen Nemec proposed openstack/oslo.utils: Return LOCALHOST if no default interface  https://review.openstack.org/14369118:03
openstackgerritBen Nemec proposed openstack/oslo.utils: Return LOCALHOST if no default interface  https://review.openstack.org/14369118:04
*** rushiagr is now known as rushiagr_away18:06
bnemecdhellmann: Having issues with the new oslo.utils release that I think is related to ^18:06
bnemecCan we get that merged and released today?18:06
bnemecdims__: ^18:07
dhellmannbnemec: should be able to18:07
gordcwe have one in ceilometer too but it may be our fault. specifically this: https://github.com/openstack/ceilometer/blob/master/ceilometer/tests/api/v2/test_event_scenarios.py#L14318:08
*** sabari is now known as zz_sabari18:08
dhellmanngordc: yeah, accessing private data is a no-no. do we need a new API or fixture for that case?18:10
gordcdhellmann: i'm trying to see if it was just laziness we reference the private attr or if we actually need it.18:11
gordcdhellmann: at quick glance it seems like the former18:11
dhellmanngordc: ok, cool18:12
dims__bnemec: looks like dhellmann +A'ed it18:13
*** e0ne has joined #openstack-oslo18:24
*** rpodolyaka2 has joined #openstack-oslo18:24
*** rpodolyaka2 has quit IRC18:31
*** e0ne has quit IRC18:34
*** e0ne has joined #openstack-oslo18:37
dhellmannsileht: is oslo.messaging in a state where we can stop approving changes until we get the ns package stuff done?18:37
*** zz_sabari is now known as sabari18:42
silehtdhellmann, yes18:43
dhellmannsileht: ok, I'm going to review some of the ones that have one +2 and block the rest of what's open18:44
*** stevemar2 has quit IRC18:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/14589118:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.concurrency: Updated from global requirements  https://review.openstack.org/14589218:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.db: Updated from global requirements  https://review.openstack.org/14589318:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.log: Updated from global requirements  https://review.openstack.org/14589418:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/14150918:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.middleware: Updated from global requirements  https://review.openstack.org/14589518:50
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821518:53
*** jecarey has quit IRC18:59
*** jecarey has joined #openstack-oslo19:05
*** stevemar has joined #openstack-oslo19:07
*** mriedem has quit IRC19:07
*** rpodolyaka2 has joined #openstack-oslo19:12
*** alexpilotti has quit IRC19:14
*** sabari is now known as zz_sabari19:18
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821519:21
openstackgerritMerged openstack/oslo.messaging: Add a info log when a reconnection occurs  https://review.openstack.org/13841119:22
*** mriedem has joined #openstack-oslo19:22
openstackgerritMerged openstack/oslo.messaging: Fix some comments in a backporting review session  https://review.openstack.org/14413219:23
*** jecarey has quit IRC19:33
*** jecarey has joined #openstack-oslo19:33
*** harlowja_at_home has joined #openstack-oslo19:36
*** stevemar has quit IRC19:37
*** stevemar has joined #openstack-oslo19:38
*** zz_sabari is now known as sabari19:38
*** e0ne is now known as e0ne_19:41
*** rpodolyaka2 has quit IRC19:43
openstackgerritMerged openstack/oslo.i18n: Add a fixture to let tests manage the lazy flag  https://review.openstack.org/14536419:44
*** e0ne_ is now known as e0ne19:48
*** jecarey has quit IRC19:59
*** jecarey has joined #openstack-oslo20:01
*** harlowja_at_home has quit IRC20:07
*** rpodolyaka2 has joined #openstack-oslo20:08
*** palendae has quit IRC20:10
*** palendae has joined #openstack-oslo20:10
*** raildo has joined #openstack-oslo20:12
*** sabari is now known as zz_sabari20:13
openstackgerritMichael Bayer proposed openstack/oslo.db: Ensure DBConnectionError is raised on failed revalidate  https://review.openstack.org/13973820:13
*** subscope_ has joined #openstack-oslo20:16
openstackgerritDoug Hellmann proposed openstack/oslo.vmware: Use ToggleLazy fixture from oslo.i18n  https://review.openstack.org/14592320:28
openstackgerritDoug Hellmann proposed openstack/oslo.vmware: Move files out of the namespace package  https://review.openstack.org/14592420:28
*** subscope_ has quit IRC20:28
mriedemzzzeek: what do you think about just approving this? https://review.openstack.org/#/c/142467/ not sure there are other sqla-migrate cores to wait around on20:36
*** harlowja_away is now known as harlowja20:37
*** jecarey has quit IRC20:40
*** jecarey has joined #openstack-oslo20:41
*** e0ne has quit IRC20:41
*** jecarey_ has joined #openstack-oslo20:42
*** zz_sabari is now known as sabari20:43
*** jecarey has quit IRC20:45
*** e0ne has joined #openstack-oslo20:46
zzzeekmriedem: well I’m a migrate core ….:)20:52
zzzeekoh +2 it.  dunno.  wahts acceptable20:52
mriedemzzzeek: i know, but i assume you're waiting for a 2nd +2, but those don't happen in that project20:52
mriedemzzzeek: depends on the project20:52
openstackgerritMerged openstack/oslo.utils: Return LOCALHOST if no default interface  https://review.openstack.org/14369120:52
zzzeekmriedem: OK who is going to come after me if i do so?   who actualyl releases this anyway?20:53
mriedemzzzeek: dripton used to do the releaes for a time20:53
mriedembut he's not involved anymore20:53
zzzeekmriedem: so, I can workflow this, but then, theres nobody to release it right :)20:53
mriedemzzzeek: so no one is going to come afer you20:53
* zzzeek has no idae how to release things from stackforge20:53
mriedemzzzeek: right, although there are some other changes that might be good to get in before a new release anyway20:53
mriedemlike ihar's20:54
*** david-lyle has joined #openstack-oslo20:54
zzzeekwell i clicked the button, im done20:54
zzzeekgood luck w/ the release :)20:54
mriedemfwiw, i'm the bug master in launchpad now i think20:55
*** ajo has quit IRC21:03
*** ajo has joined #openstack-oslo21:04
*** ajo has quit IRC21:05
*** e0ne has quit IRC21:09
*** e0ne has joined #openstack-oslo21:12
*** ihrachyshka has quit IRC21:13
*** mriedem has quit IRC21:13
*** rpodolyaka2 has quit IRC21:15
*** mriedem has joined #openstack-oslo21:15
*** ajo has joined #openstack-oslo21:15
openstackgerritDoug Hellmann proposed openstack/oslo.messaging: Move files out of the namespace package  https://review.openstack.org/14479421:15
*** ajo has quit IRC21:16
*** ajo has joined #openstack-oslo21:18
*** andreykurilin has joined #openstack-oslo21:23
*** ajo has quit IRC21:25
*** ajo has joined #openstack-oslo21:25
sdaguedhellmann: unrelated to other reviews, this should be a simple consistency fix - https://review.openstack.org/#/c/145497/21:26
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821521:27
dhellmannsdague: yep, +2a21:27
bnemecsdague: +2a21:27
dhellmannheh21:27
bnemecBah, ninja'd :-)21:27
dhellmannbnemec: oh, no, now it's going to merge twice!21:27
bnemec:-)21:28
bnemecdhellmann: While you're here, the oslo.utils fix merged.  I'm working on doing the release.21:28
openstackgerritDoug Hellmann proposed openstack/oslo.serialization: fix bug tracker link in README.rst  https://review.openstack.org/14593521:32
dhellmannoslo.serialization 1.2.0 released21:33
mriedemdhellmann: in case you haven't seen, bknudson figured out the deprecations warning logging issue21:34
mriedemdhellmann: turns out sqlalchemy-migrate is logging all deprecation warnings always https://github.com/stackforge/sqlalchemy-migrate/blob/master/migrate/changeset/__init__.py#L1321:34
dhellmannmriedem: oh, cool21:34
dhellmannah, right21:34
dhellmannthat bit us once before, but I couldn't remember which package was doing it21:35
mriedembknudson is pushing the fix21:35
dhellmannmriedem, bknudson : to remove that line completely, right?21:35
mriedemi'd assume so21:35
bknudsonthe plan is just don't call warnings.simplefilter.21:35
bknudsonhopefully the tests still work.21:35
bknudsonwithout it.21:35
dhellmannbknudson: if the tests depend on that, we can set up a temporary filter within the actual test code21:39
*** sabari is now known as zz_sabari21:39
*** zz_sabari is now known as sabari21:41
*** Mzoorikh has joined #openstack-oslo21:43
MzoorikhHi.21:43
MzoorikhI have a simple question.21:43
MzoorikhWhere is the rabbitmq.config in devstack installation. In manual installation the config file is under /etc/rabbitmq directory. But in devstack installation this folder is empty. Where is the config file in the case of devstack installation?21:43
openstackgerritBen Nemec proposed openstack/oslo.utils: Fix bug link in README.rst  https://review.openstack.org/14593921:45
bknudsonthere is a test that fails... I'll try the temp filter21:46
dhellmannbnemec: https://review.openstack.org/14578921:46
MzoorikhI can not find rabbitmq.config file in my devstack installation. Where is the it? Thanks in advance.21:47
openstackgerritDoug Hellmann proposed openstack-dev/oslo-cookiecutter: Fix link to bug tracker in README.rst template  https://review.openstack.org/14594021:47
bnemecdhellmann: Heh, oops21:48
dhellmannbnemec: :-)21:48
bnemecAlso, oslo.utils 1.2.1 is released.21:48
Mzoorikh...21:48
dhellmannbnemec: thanks!21:49
dhellmannI'm going offline to head over to the meetup site.21:49
bnemecdhellmann: np, enlightened self interest is a strong motivator. :-)21:49
*** e0ne has quit IRC21:54
Mzoorikh Where is the rabbitmq.config in devstack installation. In manual installation the config file is under /etc/rabbitmq directory. But in devstack installation this folder is empty. Where is the config file in the case of devstack installation?21:54
bknudsondhellmann: actually the test is kind of tricky so can't just override it in the test... http://git.openstack.org/cgit/stackforge/sqlalchemy-migrate/tree/migrate/tests/versioning/test_shell.py#n45521:55
*** raildo has quit IRC21:55
openstackgerritTerry Wilson proposed openstack/oslo.concurrency: Add fail_loglevel to processutils.execute  https://review.openstack.org/14594221:56
*** e0ne has joined #openstack-oslo21:58
*** e0ne has quit IRC22:05
*** Mzoorikh has quit IRC22:09
openstackgerritEli Qiao proposed openstack/oslo.db: Make sure sort_key_attr is QueryableAttribute when query  https://review.openstack.org/14363222:12
*** david-lyle has quit IRC22:28
*** e0ne has joined #openstack-oslo22:34
*** gordc has quit IRC22:36
*** dims__ has quit IRC22:36
*** dims__ has joined #openstack-oslo22:37
*** amrith is now known as _amrith_22:37
*** jecarey_ has quit IRC22:38
*** dims__ has quit IRC22:41
*** meghal has joined #openstack-oslo22:43
*** flaper87 is now known as TheGuy22:45
*** e0ne has quit IRC22:47
meghalzzzeek: had a qq about sqlalchemy "for update" lock if you have a minute22:47
zzzeekmeghal: sure22:47
meghalzzzeek:  thanks. I read on this page - http://dev.mysql.com/doc/refman/5.6/en/innodb-locking-reads.html22:48
meghalzzzeek:  that for update will not work if autocommit is set to true22:48
*** mriedem has quit IRC22:49
meghalzzzeek:  and I believe openstack autocommit set to true in icehouse - https://github.com/openstack/nova/blob/stable/icehouse/nova/openstack/common/db/sqlalchemy/session.py#L80522:49
meghalzzzeek:  so I was wondering if there could be any race conditions in this query here - https://github.com/openstack/nova/blob/stable/icehouse/nova/db/sqlalchemy/api.py#L77122:49
zzzeekmeghal: that autocommit has nothing to do with mysql’s native autocommit mode22:50
meghalzzzeek:  ahh did not know that22:50
zzzeekmeghal: Python DBAPI never sets autocommit natively unless it is specifcialyl selected, and I dont think SQLAclhemy even exposes this in the case of MySQLdb or other mysql drivers at the moment22:51
zzzeekmeghal: jay pipes just wrote this: http://www.joinfu.com/2015/01/understanding-reservations-concurrency-locking-in-nova/22:52
zzzeekmeghal: way more detail than I have in my head22:53
meghalzzzeek:  got it…I introduced some wait in the session where we select ip from the pool and was observing some race condition…so was trying to look for a cause and read about autocommit22:54
meghalzzzeek:  thanks…will go through the link22:54
zzzeekmeghal: short asnwer is we need to do away with SELECT FOR UPDATE in any case as it isn’t compatible with galera22:54
*** stevemar has quit IRC22:55
meghalzzzeek:  interesting22:56
harlowjadamn, jaypipes all about the diagrams and stuff22:56
harlowjasuch professional blogging, lol22:57
harlowjajaypipes how long did u spend on those pictures ;)22:57
*** sigmavirus24 is now known as sigmavirus24_awa23:01
bknudsonI think the new oslo.utils has broken keystonemiddleware tests... I'm guessing it's the timeutils.utcnow mock.23:02
bknudsonprobably need to change it to oslo_utils.23:02
morganfainbergbknudson, only keystoneclient or both keystoneclient and keystonemiddleware?23:02
morganfainbergbknudson, keystoneclient.middleware that is23:03
bknudsonmorganfainberg: I believe the tests are the same in both.23:03
morganfainbergbknudson, good to know23:03
bknudsonand they're both broken.23:03
morganfainbergbknudson, thanks.23:03
bknudsonI'll try it out and make sure.23:04
*** TheGuy is now known as flaper8723:06
bknudsonwhen I switch to oslo_utils for the mock.patch etc, the tests pass.23:07
jaypipesharlowja: a few minutes :)23:33
harlowjadamn, u master23:33
sdaguezzzeek: so I think it was decided that you should do some core group mods on sqlalchemy-migrate core, as the existing core group is basically defunct23:42
zzzeekwhat exactly does “core group mods” mean in this case23:43
sdaguezzzeek: https://review.openstack.org/#/admin/groups/186,members23:44
sdagueI believe any existing core member can add / remove to that23:44
sdagueI would at minimum add mriedem23:44
zzzeeksdague: ahha23:45
zzzeek“Add” is greyed out for me23:45
sdaguehmmm23:45
sdaguelet me go ask in infra23:45
sdaguezzzeek: ok, your permissions sure be updated. Reload the page and see if it works now23:49
zzzeekyup23:49
zzzeeksdague: mriedem is added23:50
*** meghal has quit IRC23:55
sdaguegreat23:56

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