Tuesday, 2014-12-02

*** ajo has quit IRC00:02
*** russellb has joined #openstack-oslo00:04
*** dims has quit IRC00:06
*** takedakn has quit IRC00:06
*** takedakn has joined #openstack-oslo00:06
*** sigmavirus24 is now known as sigmavirus24_awa00:08
openstackgerritJoshua Harlow proposed openstack/taskflow: Rework the in-memory backend  https://review.openstack.org/12455200:13
openstackgerritJoshua Harlow proposed openstack/taskflow: Rework the in-memory backend  https://review.openstack.org/12455200:15
*** mtanino has quit IRC00:29
*** oomichi has joined #openstack-oslo00:31
*** raildo_ has joined #openstack-oslo00:33
*** raildo_ has quit IRC00:37
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821500:40
*** takedakn has quit IRC00:40
*** dims has joined #openstack-oslo00:48
*** raildo_ has joined #openstack-oslo00:48
*** dims_ has joined #openstack-oslo00:50
openstackgerritJoshua Harlow proposed openstack/oslo.concurrency: Add a reader/writer lock  https://review.openstack.org/13821700:50
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821500:50
openstackgerritJoshua Harlow proposed openstack/oslo.concurrency: Add a reader/writer lock  https://review.openstack.org/13821700:51
*** zzzeek has quit IRC00:52
*** dims has quit IRC00:54
openstackgerritJamie Lennox proposed openstack/oslo.utils: Import positional decorator used by keystoneclient  https://review.openstack.org/13821801:02
*** dims_ has quit IRC01:14
openstackgerritJoshua Harlow proposed openstack/oslo.concurrency: Add a reader/writer lock  https://review.openstack.org/13821701:37
*** arnaud_____ has quit IRC01:39
openstackgerritJoshua Harlow proposed openstack/oslo.concurrency: Add a reader/writer lock  https://review.openstack.org/13821701:42
openstackgerritJoshua Harlow proposed openstack/oslo.concurrency: Add a reader/writer lock  https://review.openstack.org/13821701:47
*** dims has joined #openstack-oslo01:50
openstackgerritJoshua Harlow proposed openstack/taskflow: Move the _pformat() method to be a classmethod  https://review.openstack.org/13651301:57
openstackgerritJamie Lennox proposed openstack/oslo.utils: Import positional decorator used by keystoneclient  https://review.openstack.org/13821801:57
*** sigmavirus24_awa is now known as sigmavirus2402:02
openstackgerritJoshua Harlow proposed openstack/taskflow: Cache immutable visible scopes in the runtime component  https://review.openstack.org/13724102:10
openstackgerritJoshua Harlow proposed openstack/taskflow: Add *basic* scope visibility constraints  https://review.openstack.org/13724502:10
openstackgerritJoshua Harlow proposed openstack/taskflow: Use a module level constant to provide the DEFAULT_LISTEN_FOR  https://review.openstack.org/13698302:15
openstackgerritMerged openstack/taskflow: Tweaks to setup.cfg  https://review.openstack.org/13655102:20
openstackgerritMerged openstack/taskflow: Include documentation of the utility modules  https://review.openstack.org/13772402:22
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow for the notifier to provide a 'details_filter'  https://review.openstack.org/13696902:23
openstackgerritJoshua Harlow proposed openstack/taskflow: Update listeners to ensure they correctly handle all atoms  https://review.openstack.org/13695802:23
openstackgerritMerged openstack/taskflow: Format failures via a static method  https://review.openstack.org/13100102:24
openstackgerritMerged openstack/taskflow: Add link to issue 17911  https://review.openstack.org/13786302:26
openstackgerritMerged openstack/taskflow: Avoid deepcopying exception values  https://review.openstack.org/13786102:29
openstackgerritMerged openstack/taskflow: Use a metaclass to dynamically add testcases to example runner  https://review.openstack.org/13548702:32
*** dims has quit IRC02:42
openstackgerritMerged openstack/taskflow: Move scheduler and completer classes to there own modules  https://review.openstack.org/13694402:51
*** dims has joined #openstack-oslo02:51
*** dims has quit IRC02:51
*** tsekiyam_ has joined #openstack-oslo02:56
*** tsekiyama has quit IRC02:59
*** tsekiyam_ has quit IRC03:00
*** sigmavirus24 is now known as sigmavirus24_awa03:02
*** arnaud_____ has joined #openstack-oslo03:10
*** arnaud_____ has quit IRC03:11
openstackgerritJoshua Harlow proposed openstack/taskflow: Split the scheduler into sub-schedulers  https://review.openstack.org/13823703:29
*** _amrith_ is now known as amrith03:31
*** raildo_ has quit IRC03:31
openstackgerritJoshua Harlow proposed openstack/taskflow: Update listeners to ensure they correctly handle all atoms  https://review.openstack.org/13695803:43
*** ishant has joined #openstack-oslo04:24
*** arnaud_____ has joined #openstack-oslo04:24
openstackgerritJoshua Harlow proposed openstack/taskflow: Update listeners to ensure they correctly handle all atoms  https://review.openstack.org/13695805:10
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Show what the threshold is being increased to  https://review.openstack.org/13825705:14
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a claims listener that connects job claims to engines  https://review.openstack.org/13692305:36
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a claims listener that connects job claims to engines  https://review.openstack.org/13692305:38
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a claims listener that connects job claims to engines  https://review.openstack.org/13692305:39
*** exploreshaifali has joined #openstack-oslo05:55
*** exploreshaifali has quit IRC06:02
*** vigneshvar has joined #openstack-oslo06:07
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Imported Translations from Transifex  https://review.openstack.org/13826606:10
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/13656606:13
*** exploreshaifali has joined #openstack-oslo06:24
*** harlowja_ is now known as harlowja_away06:45
*** exploreshaifali has quit IRC06:51
*** vigneshvar has quit IRC06:54
openstackgerritMerged openstack/oslo.messaging: Show what the threshold is being increased to  https://review.openstack.org/13825707:05
*** subscope has quit IRC07:12
*** arnaud_____ has quit IRC07:17
*** k4n0 has joined #openstack-oslo07:21
*** subscope has joined #openstack-oslo07:26
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Don't use oslo.cfg to set kombu in-memory driver  https://review.openstack.org/13804807:27
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: iterconsume must honor timeout  https://review.openstack.org/13820207:27
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: RabbitMQ heartbeat implementation  https://review.openstack.org/12633007:27
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Don't share connection pool between driver object  https://review.openstack.org/13802907:27
*** e0ne has joined #openstack-oslo07:35
*** arnaud_____ has joined #openstack-oslo07:43
*** e0ne has quit IRC07:47
*** e0ne has joined #openstack-oslo07:48
*** flaper87 has quit IRC08:20
*** flaper87 has joined #openstack-oslo08:20
*** stevemar has quit IRC08:23
openstackgerritMerged openstack/oslo.vmware: Switch to use requests/urllib3 and enable cacert validation  https://review.openstack.org/12195608:24
*** stannie has joined #openstack-oslo08:28
*** andreykurilin has joined #openstack-oslo08:45
*** noelbk has quit IRC08:51
*** pblaho_ has joined #openstack-oslo08:53
*** ajo has joined #openstack-oslo08:56
*** ajo has quit IRC09:08
*** ihrachyshka has joined #openstack-oslo09:14
*** i159 has joined #openstack-oslo09:16
openstackgerritMerged openstack/oslo.messaging: Don't share connection pool between driver object  https://review.openstack.org/13802909:31
*** andreykurilin has quit IRC09:32
*** viktors|afk has quit IRC09:34
*** i159 has quit IRC09:34
*** e0ne has quit IRC09:34
*** andreykurilin1 has quit IRC09:34
*** viktors has joined #openstack-oslo09:34
*** i159 has joined #openstack-oslo09:35
*** e0ne has joined #openstack-oslo09:35
*** e0ne has quit IRC09:37
*** viktors has quit IRC09:40
*** ozamiatin has quit IRC09:40
*** i159 has quit IRC09:40
*** dtantsur|afk is now known as dtantsur09:41
*** dmakogon_ has quit IRC09:45
*** arnaud_____ has quit IRC09:58
*** vigneshvar has joined #openstack-oslo09:59
*** denis_makogon has joined #openstack-oslo10:01
*** i159 has joined #openstack-oslo10:01
*** viktors has joined #openstack-oslo10:01
*** andreykurilin has joined #openstack-oslo10:01
*** e0ne has joined #openstack-oslo10:01
*** e0ne has quit IRC10:05
silehtflaper87, jd__ can you take a look to: https://review.openstack.org/#/c/138202/ et https://review.openstack.org/#/c/138048/ ?10:33
flaper87sileht: sure10:34
jd__yep10:34
silehtthanks a lot, I would like to land all timeout related fixes for the next release. (And perhaps the heartbeat patch)10:36
*** e0ne has joined #openstack-oslo10:37
openstackgerritMerged openstack/oslo.messaging: Have the timeout decrement inside the wait() method  https://review.openstack.org/13745611:10
*** alexpilotti_ has joined #openstack-oslo11:19
*** alexpilotti has quit IRC11:20
*** alexpilotti_ has quit IRC11:23
*** raildo_ has joined #openstack-oslo11:36
*** exploreshaifali has joined #openstack-oslo11:41
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: iterconsume must honor timeout  https://review.openstack.org/13820211:42
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: RabbitMQ heartbeat implementation  https://review.openstack.org/12633011:42
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: don't always start the heartbeart thread  https://review.openstack.org/13832511:42
*** raildo_ has quit IRC11:43
*** pblaho_ is now known as pblaho12:02
i159sileht: Hi! I need a description about your pool feature. I have a collision, that when I rebased on your merged patches,  FakeDriver.listen now require a pool argument. Can you help me, please?12:18
i159sileht: My question is — where do you initially get a pool name? Can't find it...12:20
silehti159, here:  https://github.com/openstack/oslo.messaging/blob/master/oslo/messaging/notify/listener.py#L10612:22
*** exploreshaifali has quit IRC12:27
*** k4n0 has quit IRC12:28
i159sileht:  from where it must be passed in the function?12:28
silehti159, https://github.com/openstack/oslo.messaging/blob/master/oslo/messaging/notify/dispatcher.py#L66 ?12:28
silehti159, and then the transport pass it to the driver12:29
i159sileht: so, NotificationDispatcher will be initialized by application which uses messaging lib with pool name. And we don't generate this name nowhere in the lib?12:36
silehti159, yes12:36
i159sileht: got it =) thanks you!12:37
silehti159, welcome12:39
*** jeckersb_gone is now known as jeckersb12:57
*** arnaud_____ has joined #openstack-oslo12:59
*** arnaud_____ has quit IRC13:03
openstackgerritMerged openstack/oslo-specs: Add adopt-oslo-versionedobjects spec  https://review.openstack.org/12753213:03
openstackgerritMerged openstack/oslo-specs: oslo.messaging: notification-listener-pools  https://review.openstack.org/13166613:03
openstackgerritMerged openstack/oslo-specs: log: Restore remove-context-adapter spec  https://review.openstack.org/12484713:03
*** kgiusti has joined #openstack-oslo13:12
openstackgerritDoug Hellmann proposed openstack/oslo-specs: rename spec to match bp  https://review.openstack.org/13835713:12
*** sigmavirus24_awa is now known as sigmavirus2413:13
*** ishant has quit IRC13:16
openstackgerritMerged openstack/oslo-specs: rename spec to match bp  https://review.openstack.org/13835713:24
*** exploreshaifali has joined #openstack-oslo13:31
*** dims has joined #openstack-oslo13:32
dhellmannjd__: are we waiting for infra to rename the tooz repository?13:32
*** palendae has quit IRC13:38
*** bknudson has joined #openstack-oslo13:40
*** bknudson1 has joined #openstack-oslo13:42
*** bknudson has quit IRC13:44
*** palendae has joined #openstack-oslo13:46
*** vigneshvar has quit IRC13:49
*** dims has quit IRC13:53
jd__dhellmann: yes13:53
*** dims has joined #openstack-oslo13:54
*** amrith is now known as _amrith_13:55
*** amotoki has quit IRC13:56
*** amotoki has joined #openstack-oslo13:57
*** gordc has joined #openstack-oslo14:00
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Don't use oslo.cfg to set kombu in-memory driver  https://review.openstack.org/13804814:09
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: iterconsume must honor timeout  https://review.openstack.org/13820214:09
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: RabbitMQ heartbeat implementation  https://review.openstack.org/12633014:09
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: don't always start the heartbeart thread  https://review.openstack.org/13832514:09
jd__dhellmann: clear to release oslo.middleware?14:11
dimsjd__: can you please bless https://review.openstack.org/#/c/135680/ - will help me get oslo.utils out14:17
jd__dims: commented14:19
dimsthanks!14:19
gordcjd__: no concerns from me.14:20
jd__gordc: I'm asking so you can do it :p14:20
gordci know you're waiting for mey opinion :)14:20
gordcjd__: smashing random keys on a keyboard makes a new release right? as long as i'm in oslo.middleware directory14:21
jd__gordc: in the end, yeah14:22
jd__might take time though14:22
gordcall i have is time.14:22
ttxdhellmann: I'm ready to release rootwrap... 1.4.0, right ?14:23
silehtIf someone can take a look to: https://review.openstack.org/#/c/138048/ and https://review.openstack.org/#/c/138202/14:23
dimssileht: done14:25
silehtthx14:26
*** stevemar has joined #openstack-oslo14:28
*** mriedem has joined #openstack-oslo14:29
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: Fixes debug message format  https://review.openstack.org/13837914:32
dimsdhellmann: going to push out oslo.utils (1.1.0), oslo.i18n(1.1.0) and oslo.vmware(0.8.0)14:33
*** _amrith_ is now known as amrith14:52
* ttx goes ahead14:56
*** yamahata has joined #openstack-oslo15:01
*** stevemar has quit IRC15:01
*** stevemar has joined #openstack-oslo15:02
*** zzzeek has joined #openstack-oslo15:02
ttxdhellmann: oslo.rootwrap 1.4.0 done15:06
openstackgerritDoug Hellmann proposed openstack/oslo-specs: Require spec filenames to match blueprint names  https://review.openstack.org/13839215:15
silehtjd__, dims can you take a look to https://review.openstack.org/#/c/138379/1 ?15:18
jd__sileht: don15:19
openstackgerritThomas Goirand proposed openstack-dev/pbr: Packagers vs package maintainers  https://review.openstack.org/13839415:19
silehtjd__, thx15:19
openstackgerritMerged openstack/oslo.messaging: Don't use oslo.cfg to set kombu in-memory driver  https://review.openstack.org/13804815:21
dhellmannjd__: release the hounds (or libraries, your choice)15:23
jd__gordc: ^ ready? :)15:23
gordcjd__: will do.15:25
gordc0.2.0?15:25
jd__gordc: I think so15:26
openstackgerritMerged openstack/oslo.messaging: Rabbit: iterconsume must honor timeout  https://review.openstack.org/13820215:27
openstackgerritDoug Hellmann proposed openstack/oslo-specs: Require spec filenames to match blueprint names  https://review.openstack.org/13839215:28
gordcjd__: err... 1.0.0? https://wiki.openstack.org/wiki/Oslo/VersioningPolicy15:28
gordcconfused myself.15:28
*** mtanino has joined #openstack-oslo15:31
openstackgerritMerged openstack/oslo-specs: Require spec filenames to match blueprint names  https://review.openstack.org/13839215:43
gordcdhellmann: oslo.middleware 1.0.0 cool? i take silence as confirmation :)15:43
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: RabbitMQ heartbeat implementation  https://review.openstack.org/12633015:52
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Rabbit: don't always start the heartbeart thread  https://review.openstack.org/13832515:52
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Add a info log when a reconnection occurs  https://review.openstack.org/13841115:52
*** zz_jgrimm is now known as jgrimm15:57
bnemecgordc: Was oslo.middleware graduated last cycle?15:58
*** alexpilotti has joined #openstack-oslo15:58
*** alexpilotti has quit IRC15:59
*** alexpilotti has joined #openstack-oslo15:59
gordcbnemec: around the feature-freeze time. i think we only started putting it into projects this cycle though.16:00
openstackgerritMerged openstack/oslo.messaging: Rabbit: Fixes debug message format  https://review.openstack.org/13837916:00
bnemecgordc: Hmm, I  think I would leave it <1.0 for this cycle then.  We want to have the freedom to make breaking changes during the first adoption cycle in case we find problems.16:02
bnemecAt the end of the cycle we can release 1.0.016:02
gordcbnemec: good thing i procrastinate.16:02
*** noelbk has joined #openstack-oslo16:02
bnemecgordc: :-)16:03
dimsdhellmann: wrapped up my 3 libs16:06
dhellmanndims: thanks16:09
dimsdhellmann: praying gate does not explode :)16:10
dhellmannbnemec, gordc : I like that logic.16:10
dhellmanndims: heh16:10
*** tsekiyama has joined #openstack-oslo16:12
openstackgerritVictor Sergeyev proposed openstack/oslo.db: Refactor database migration manager to use given engine  https://review.openstack.org/13488516:17
gordchas anyone seen this Unauthorized error when running release script? http://paste.openstack.org/show/143656/16:18
openstackgerritVictor Sergeyev proposed openstack/oslo.db: Refactor database migration manager to use given engine  https://review.openstack.org/13488516:18
viktorsgordc: I did. Are you have enough permissions on launchpad.16:19
viktors?16:19
openstackgerritOleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging functional tests proposal  https://review.openstack.org/13709416:20
gordcviktors: no clue. i added openstack-releasing with full access to my launchpad16:22
viktorsgordc: what library are you supposed to release?16:22
gordcviktors: oslo.middleware16:22
viktorsgordc: are you belong to Maintainer's group - https://launchpad.net/~oslo-drivers ?16:24
*** jecarey has joined #openstack-oslo16:25
gordcviktors: i thought i was... apparently just the gerrit groups16:26
gordcguess i can't do it. i'll pass it on16:26
viktorsgordc: I think, you should ask dhellmann to add required permissions to you16:28
bnemecdhellmann: oslo.concurrency 0.3.0 is in the wild.16:31
bnemecIt's possible it could break some people due to the lock-wrapper changes.16:31
bnemecBut if it does then they were already hosed, so we just need to help them fix the problem.16:31
*** ozamiatin has joined #openstack-oslo16:33
*** yamahata has quit IRC16:36
viktorsdhellmann: FYI - I really haven't had enough time today to test and release oslo.db - I will do it tomorrow16:40
openstackgerritDoug Hellmann proposed openstack/oslosphinx: Add links to the README  https://review.openstack.org/13843616:41
*** i159 has quit IRC16:45
openstackgerritDoug Hellmann proposed openstack/oslotest: Fix the URL for reporting bugs in the README  https://review.openstack.org/13843816:45
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Add more to the release notes  https://review.openstack.org/13843916:46
dhellmannviktors: ok, that's good. If you can't get to it tomorrow, you may want to wait until early next week to ensure you have time to address any issues.16:47
dhellmannbnemec, dims, jd__: I've tweaked the release note script some more in  https://review.openstack.org/13843916:48
jd__you're a tweaker16:48
* jd__ also checks http://www.urbandictionary.com/define.php?term=tweaker to be sure16:49
gordcjd__: nope... or maybe?..16:49
jd__42?16:49
gordcdhellmann: could i get added to appropriate launchpad group so i make oslo.middleware relesae?16:50
gordcor i could defer to jd__ :)16:51
dhellmanngordc: sure, let me see what we need to do16:51
dhellmanngordc: what's your launchpad id?16:52
gordcchungg... it should be that...16:52
gordci should've sync'd all my ids... chaos now.16:52
silehtdhellmann, I'm ready to release oslo.messaging 1.5.016:53
dhellmanngordc: ok, I added you to oslo-drivers to give you the privs you need for releases. Next we should check gerrit.16:53
dhellmannsileht: fire at will16:53
silehtdhellmann, I guess I will have the same issue as gordc16:53
dhellmannsileht: I added you, too16:53
gordcdhellmann: i should be ok for gerrit... or at least oslo-middleware-* groups16:53
dhellmanngordc: yep, I just verified that16:54
dhellmannsileht: you look like you're all set for releases in gerrit, too16:54
gordcdhellmann: cool cool. i'll give it a try again16:54
silehtdhellmann, thx, I start the process so16:54
* sileht launches: ./oslo_release.sh kilo 1.5.0 HEAD oslo.messaging16:56
* dhellmann crosses his fingers16:56
gordcgood on my end. will send email in a bit.16:57
*** vigneshvar has joined #openstack-oslo17:00
openstackgerritJulien Danjou proposed openstack/oslo.log: Replace RFCSysLogHandler by a syslog() based one  https://review.openstack.org/13684217:01
openstackgerritBen Nemec proposed openstack/oslo.concurrency: Fix bug link in README.rst  https://review.openstack.org/13844117:01
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Add more to the release notes  https://review.openstack.org/13843917:01
silehtdhellmann, looks like it has worked, I have just moved one BP manually: https://github.com/openstack/oslo.messaging/tree/1.5.017:03
dhellmannsileht: yes, launchpad looks ok. You need a new "next-kilo" milestone, but the 1.5.0 milestone looks ok17:04
dhellmannsileht: which bp did you move manually?17:05
silehtdhellmann, the listener-pools was tagged 1.5.0 but not target to kilo17:06
dhellmannsileht: ah, ok, the series target doesn't really matter17:06
dhellmannit's fine to set it, but it's not needed as long as the milestone is set properly17:06
dhellmannsileht: oh, I see an "in progress" bug in 1.5.0 - https://launchpad.net/oslo.messaging/kilo/1.5.017:07
dhellmannsileht: that should just move to next-kilo17:07
silehtdhellmann, ok I got it, I will move it17:08
dhellmannsileht: thanks17:08
gordcbnemec: how do you guys generate the stuff understand diffstat section?17:10
*** arnaud_____ has joined #openstack-oslo17:10
silehtdhellmann, done17:10
dhellmanngordc: look in oslo-incubator/tools for the release_notes.sh script17:10
gordcdhellmann: cool cool. will do17:11
*** viktors is now known as viktors|afk17:14
*** denis_makogon has quit IRC17:14
*** amotoki has quit IRC17:15
bnemecdhellmann: jd__ harlowja_away: Crap, I think the concurrency release is breaking something: https://jenkins07.openstack.org/job/gate-nova-python27/8299/console17:21
jd__bnemec: indeed17:21
*** dtantsur is now known as dtantsur|brb17:22
jd__I can't see how it breaks :/17:26
*** pblaho has quit IRC17:26
openstackgerritSean Dague proposed openstack/oslo.concurrency: test compatibility for old imports  https://review.openstack.org/13845117:29
dhellmannbnemec, jd__ : is something mocking the lock? with the namespace package change that might break17:31
jd__dhellmann: good lead, let me checj17:32
bnemecdhellmann: Ah, I bet it is.17:32
dhellmannjd__, bnemec : see sdague's patch ^^17:32
jd__nova/tests/unit/virt/libvirt/test_firewall.py:    @mock.patch.object(lockutils, "external_lock")17:32
jd__from a quick grep17:32
bnemecParticularly since that's a nova change and I think that's what they were doing.17:32
jd__so there are mock, maybe that's it indeed17:32
jd__        mock_lock.return_value = threading.Semaphore() hum17:34
jd__I don't think you can have a delay to a Semaphore ineed17:34
bnemecOh, phew.  At least not all of the red on the zuul status is from this.17:34
bnemecjd__: Yeah, that was an incorrect mock.  We should probably just convert it to use the new external lock fixture.17:35
* jd__ nods17:35
bnemecIt takes a really long time to install deps for Nova unit tests. :-/17:37
openstackgerritDoug Hellmann proposed openstack/oslo.concurrency: Remove noqa from test files  https://review.openstack.org/13845917:44
*** mriedem has quit IRC17:45
* dims is back 17:47
silehtdhellmann, the upload to pypi is done automagically ?17:48
dhellmannsileht: it should be17:48
silehtok I wait so17:48
*** e0ne has quit IRC17:52
*** boris-42 has joined #openstack-oslo17:52
*** fnordahl has joined #openstack-oslo17:54
*** exploreshaifali has quit IRC17:58
openstackgerritDoug Hellmann proposed openstack-dev/hacking: Allow import redirections  https://review.openstack.org/13846218:02
* bnemec is really glad he didn't try to release concurrency last night18:02
dhellmannbnemec: see #openstack-dev and https://review.openstack.org/13846218:07
bnemecdhellmann: Yep, looking18:07
dhellmannbnemec: when we merge that, I can backport it to the 0.9.x branch and make a new 0.9.x release18:08
openstackgerritDoug Hellmann proposed openstack-dev/hacking: Allow import redirections  https://review.openstack.org/13846218:10
*** mriedem has joined #openstack-oslo18:21
openstackgerritMerged openstack-dev/hacking: Allow import redirections  https://review.openstack.org/13846218:21
*** mriedem has quit IRC18:24
*** mriedem has joined #openstack-oslo18:25
*** harlowja_away is now known as harlowja_18:31
harlowja_bnemec u got it all under control?18:33
bnemecharlowja_: Yeah, I think we've got it.  There were actually a couple of separate problems, but fixes are in flight. :-)18:34
harlowja_kk18:34
*** dtantsur|brb is now known as dtantsur18:36
*** dtantsur is now known as dtantsur|afk18:54
*** e0ne has joined #openstack-oslo18:59
*** stevemar has quit IRC19:23
*** stevemar has joined #openstack-oslo19:24
*** e0ne has quit IRC19:25
*** e0ne has joined #openstack-oslo19:26
*** arnaud_____ has quit IRC19:29
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821519:33
*** rdobosz has quit IRC19:34
*** rdobosz has joined #openstack-oslo19:34
*** exploreshaifali has joined #openstack-oslo19:38
*** e0ne has quit IRC19:43
openstackgerritJoshua Harlow proposed openstack/taskflow: Properly handle and skip empty intermediary flows  https://review.openstack.org/13514619:46
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a testcase that can test + verify runtime task ordering  https://review.openstack.org/13547819:47
*** sigmavirus24 is now known as sigmavirus24_awa19:48
*** sigmavirus24_awa is now known as sigmavirus2419:50
*** ihrachyshka has quit IRC19:52
*** ihrachyshka has joined #openstack-oslo19:55
*** kgiusti1 has joined #openstack-oslo20:01
*** andreykurilin_ has joined #openstack-oslo20:02
*** kgiusti has quit IRC20:02
*** kgiusti1 has left #openstack-oslo20:02
*** e0ne has joined #openstack-oslo20:05
*** e0ne has quit IRC20:10
openstackgerritZhiQiang Fan proposed openstack/oslo.messaging: Keep __hash__ consistent with __eq__  https://review.openstack.org/13851720:11
*** exploreshaifali has quit IRC20:40
openstackgerritJoshua Harlow proposed openstack/taskflow: Raise value errors instead of asserts  https://review.openstack.org/13853320:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Raise value errors instead of asserts  https://review.openstack.org/13853320:47
*** jecarey has quit IRC20:54
*** ihrachyshka has quit IRC20:59
dimsdhellmann: so, when we get releases out, we can't update global-requirements now? - https://review.openstack.org/#/c/138407/21:09
*** stannie has quit IRC21:10
dhellmanndims: that's news to me. We'll have to figure out how to make that work.21:20
vishyharlowja_: bad news, the periodic tasks recover but the node fails to process concurrent messages properly21:22
vishyit seems to get stuck waiting on the socket and blocking other threads21:22
harlowja_:(21:22
vishythe whole timeout code is really poor so I’m hoping the new code from sileht fixes it21:23
harlowja_i think he just merged that21:23
harlowja_if u want to try21:23
harlowja_but agreed, the timeout code is hard to understand21:23
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821521:24
dimsour world is getting weirder by the minute :)21:28
*** e0ne has joined #openstack-oslo21:30
*** stevemar2 has joined #openstack-oslo21:30
*** stevemar has quit IRC21:31
*** openstackgerrit has quit IRC21:34
*** openstackgerrit has joined #openstack-oslo21:34
vishyharlowja_: he said he was going to propose something today to change the timeout for messages to be in messaging instead of getting passed down to kombu21:36
*** alexpilotti has quit IRC21:38
*** stevemar2 is now known as stevemar21:38
harlowja_hmmmm21:48
openstackgerritMerged openstack-dev/pbr: Packagers vs package maintainers  https://review.openstack.org/13839421:51
*** arnaud_____ has joined #openstack-oslo21:56
*** amrith is now known as _amrith_21:59
*** e0ne has quit IRC22:05
*** alexpilotti has joined #openstack-oslo22:08
*** arnaud_____ has quit IRC22:14
*** arnaud__ has joined #openstack-oslo22:15
silehtvishy, it's https://github.com/openstack/oslo.messaging/commit/2dd7de989f88c7c095d3c2ef1646d2dec87869a522:16
silehtvishy, the kombu timeout is always 1s now, like the kombu doc recommands, and the timeout can occurs when the reconnection fail, now22:18
silehtvishy, and the harlowja_ patch was merged too22:21
*** arnaud__ has quit IRC22:29
*** vigneshvar has quit IRC22:30
*** boris-42 has quit IRC22:37
vishysileht: ok its in22:43
vishywill test with the new code22:43
*** raildo_ has joined #openstack-oslo22:50
*** jgrimm is now known as zz_jgrimm22:55
*** harlowja_ is now known as harlowja_away23:07
*** alexpilotti has quit IRC23:09
*** zzzeek has quit IRC23:10
*** harlowja_away is now known as harlowja_23:12
*** alexpilotti has joined #openstack-oslo23:16
vishysileht: good news, it seems to recover properly with the new code23:19
harlowja_woot23:20
*** alexpilotti has quit IRC23:21
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a overly simplistic hello world example  https://review.openstack.org/13857223:23
*** gordc has quit IRC23:23
*** zzzeek has joined #openstack-oslo23:23
cburgessvishy: So sounds like the problem was the timeout logic?23:24
vishycburgess: yeah it appears so23:24
vishyunfortunately it doesn’t seem to be fixable in old releases23:24
vishycurious if anyone else is running into this23:24
cburgessvishy: Define old releases?23:24
vishyicehouse23:26
vishyspecifically23:26
vishyhere is how i’m testing:23:26
cburgessThis upsets me.23:26
vishysudo iptables -I OUTPUT -p tcp -m tcp -d $RABBIT/32 -j DROP; sudo iptables -I INPUT -p tcp -m tcp -s $RABBIT/32 -j DROP; sleep 130; sudo iptables -D OUTPUT -p tcp -m tcp -d $RABBIT/32 -j DROP; sudo iptables -D INPUT -p tcp -m tcp -s $RABBIT/32 -j DROP; tail -n100 -f /var/log/nova/nova-compute.log23:26
cburgessWhy is it not back portable?23:27
cburgessIs it just this change? https://review.openstack.org/#/c/138202/23:27
vishycburgess: well there are 4 or 5 related changes23:27
cburgessAhh...23:27
cburgessIs there a roll-up of them some place? Or can you send me the review/change ids?23:27
vishythe reconnect logic was moved into kombu23:27
vishytimeouts were changed23:27
vishycburgess: i think I’m just going to use a newer version of oslo.messaging23:28
cburgessMoving the reconnect logic is a win. I had solved a number of problems in that function by simply porting fixes to kombu to oslo.23:28
vishythe only annoying part of that is i have to pull in six, oslo.serialization, oslo.i18n, oslo.utils as well23:29
vishyso i think i’m going to pull in the juno versions of all of those23:29
cburgessUgh23:30
vishyand then build a 1.5 oslo messaging23:30
cburgessHow are you testing then? Or are you doing just that (running master of those)?23:31
vishycburgess: it would be awesome if you could repro those23:31
vishyso when i do the iptables thing above23:31
cburgessI ask because I will want to test CCNs to be sure they still work since thats one of the big things we added to oslo.messaging in icehouse.23:31
vishythere are two issues: 1) periodic tasks never start again23:31
vishymy first attempts at fixes solved that issue23:31
vishy2) messages to the node don’t get processed correctly, so doing a couple of launches fails23:32
vishyi’m also using only 2 greenthreads for messages so it is theoretically possible that 2) will not affect you if you use more23:32
cburgessHow do you set that?23:32
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821523:32
cburgessSo to repo, are you running an icehouse devstack then just checking out the master version of oslo.messaging and its deps?23:33
vishyno i copied all of the python files onto a running node23:33
vishy:)23:33
cburgessLOL ok.23:33
vishyit may be reproducible in devstack23:34
cburgessShould be.23:34
cburgessstack it as icehouse.23:34
cburgessThen change the branches, reinstall/dep solve by hand, restart.23:34
vishybut it is theoretically possible it only affects a certain kernel version or python version or eventlet version etc.23:34
vishyso once i got a repro on our running system i didn’t bother23:35
cburgessRight23:35
vishyyou will have to modify the iptables rule if you do it on devstack23:35
vishyto block by port instead of ip23:35
cburgessYeah23:35
cburgessI wonder if I can repro it on grizzly.23:36
vishyso we have not seen this issue before23:36
vishyso I suspect it doesn’t affect grizzly23:36
vishyi don’t have an old install to test against right now to confirm23:36
cburgessYou have a single rabbit node and you are just blocking access to it locally then restoring the access using iptables right/23:36
vishyright23:36
cburgessI can easily do the grizzly test.23:36
vishywe saw it initally when we had a network outage23:36
cburgessIts trivial.23:36
*** andreykurilin_ has quit IRC23:39
*** takedakn has joined #openstack-oslo23:40
openstackgerritMichael Bayer proposed openstack/oslo.db: Implement new oslo.db.sqlalchemy.enginefacade module  https://review.openstack.org/13821523:41
*** ftcjeff has joined #openstack-oslo23:43
*** takedakn has quit IRC23:56
*** takedakn has joined #openstack-oslo23:56

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