Wednesday, 2015-04-01

« Tuesday, 2015-03-31 Index Thursday, 2015-04-02 »
*** JAHoagie has quit IRC00:08
*** flaper87 has quit IRC00:11
*** flaper87 has joined #openstack-zaqar00:15
openstackgerritFei Long Wang proposed openstack/zaqar: Implement webhook notifier driver  https://review.openstack.org/15152901:27
*** amalagon has joined #openstack-zaqar01:34
*** flwang1 has joined #openstack-zaqar02:29
flwang1vkmc: ping02:29
vkmcflwang1, pong02:53
*** chandankumar has joined #openstack-zaqar02:54
*** achanda has quit IRC02:59
flwang1vkmc: no worries, i have leave the comments on your patch03:03
flwang1will submit them soon03:03
vkmcflwang1, k, thanks :)03:09
vkmcloooool03:17
vkmcclipo is in gerrit03:17
*** chandankumar has quit IRC03:33
*** flwang1 has quit IRC04:01
*** achanda has joined #openstack-zaqar04:12
*** chandankumar has joined #openstack-zaqar04:50
*** reed has quit IRC05:29
*** chandankumar has quit IRC05:50
openstackgerritOpenStack Proposal Bot proposed openstack/zaqar: Imported Translations from Transifex  https://review.openstack.org/16962806:03
*** amalagon has quit IRC06:06
*** chandankumar has joined #openstack-zaqar06:06
*** Nutan has joined #openstack-zaqar06:17
*** chandankumar has joined #openstack-zaqar06:18
*** dynarro has joined #openstack-zaqar06:39
*** flaper87 has quit IRC06:45
*** flaper87 has joined #openstack-zaqar06:45
flaper87flwang: any vote on the devstack patch?06:51
flaper87flwang: btw, how do you test the notifications/webhook thing ?06:58
flaper87flwang: have you been using curl?06:58
flaper87or just unittests?06:58
openstackgerritMerged openstack/zaqar: Imported Translations from Transifex  https://review.openstack.org/16962807:16
openstackgerritMerged openstack/zaqar: Implement webhook notifier driver  https://review.openstack.org/15152907:19
flaper87flwang: ^07:23
*** achanda has quit IRC07:36
*** achanda has joined #openstack-zaqar07:42
*** achanda has quit IRC07:43
NutanWhat are common options in zaqar?08:29
flaper87Nutan: options that are used by different packages in zaqar08:30
Nutanflaper87: options are defined in bootstrap.py, right?08:32
flaper87Nutan: not all of them. For example, the mongodb options are defined in zaqar.storage.mongodb.options08:33
flaper87but the "global" ones yes, those are in bootstrap08:33
Nutanokay, I am actually wanted to know what these parameters mean, for example,  cfg.BoolOpt('daemon', default=False, help='Run Zaqar server in the background.'),  so first parameter is 'daemon', are these values pre-defined like 'debug' and so on?08:38
NutanWhere are they defined?08:38
flaper87Nutan: what do you mean with pre-defined ?08:39
flaper87We need `daemon` to fork zaqar properly when running as a daemon (or in devstack with SCREEN=False)08:39
NutanYes, just like deamon, 'debug' has been used and it has correponding help message08:40
*** dynarro has quit IRC08:40
NutanCan you tell me where is BoolOpt defined?08:41
*** dynarro has joined #openstack-zaqar08:41
flaper87Nutan: Ah, that's oslo_config08:41
flaper87it's a library08:41
flaper87The one we use to manage configuration and cli parameters08:42
openstackgerritFlavio Percoco proposed openstack/zaqar: Add websockets to our test suite  https://review.openstack.org/16931308:47
Nutanokay09:05
*** Nutan has quit IRC09:17
openstackgerritFlavio Percoco proposed openstack/zaqar: Add websockets to our test suite  https://review.openstack.org/16931309:35
openstackgerritFlavio Percoco proposed openstack/zaqar: Remove QueueController from data to control plane  https://review.openstack.org/14434010:01
*** dynarro has quit IRC10:21
*** achanda has joined #openstack-zaqar10:44
*** achanda has quit IRC10:48
*** amalagon has joined #openstack-zaqar11:32
vkmcmorning o/11:36
*** amalagon has quit IRC11:36
*** JAHoagie has joined #openstack-zaqar11:48
flaper87vkmc: yo yo!11:49
*** JAHoagie has quit IRC11:52
vkmcflaper87, watcha doin'?11:54
flaper87vkmc: OpenStack housekeeping :/11:55
flaper87We need to merge the remaining patches11:56
flaper87:D11:56
vkmcoh yes11:56
vkmccan I help with something?11:56
flaper87I'd like to set up an environment with latest Zaqar to test notifications11:56
* vkmc sends gummybears and coffee to flaper87 11:56
flaper87jasondotstar: there? You were working on the puppet manifests for Zaqar, right?11:59
flaper87vkmc: could you try to replicate this? http://logs.openstack.org/13/169313/4/check/gate-zaqar-python34/94359d4/console.html12:21
vkmcflaper87, sure12:22
flaper87vkmc: thank you :)12:23
vkmcno problema12:23
* flaper87 can't create a py34 env12:23
flaper87stupid python12:23
flaper87flwang: can we mark this blueprint as implemented? https://blueprints.launchpad.net/zaqar/+spec/notifications12:46
flaper87vkmc: what's left to do for websockets?12:47
vkmcflaper87, it depends12:47
vkmcI was planning to cover claims as well12:47
vkmcbut I'm not sure if we are ok with time12:47
vkmcI can have it done by this afternoon12:47
vkmcpools, flavors and notifications will be implemented in liberty12:47
flaper87I was thinking this morning that it might be a good idea to push this websockets feature to L12:48
flaper87it's in a good state but there are still missing pieces12:48
vkmcyes I agree12:48
flaper87that are important for it to be completely usable12:48
flaper87ok12:48
flaper87lets do that12:48
vkmcwe are not fully mapping all the Zaqar API12:48
vkmcthat is my only concern12:48
vkmcwe could have it in Kilo, but as experimental12:49
vkmcor tech preview12:49
vkmcwhat do you think?12:49
vkmcflaper87, ^12:50
flaper87vkmc: it will be shipped as-is for sure. One thing we should do, though, is to have a warning there saying it's experimental12:51
flaper87I can do that12:51
vkmcflaper87, that sounds grea12:51
vkmct12:51
*** chandankumar has quit IRC12:52
vkmcthe basic prod-cons scenario is being covered12:52
vkmcIMO that's good enough for a tech preview12:52
vkmcI'll try to work on the benchmarks soon, that will make it even better12:53
vkmcI have to adapt the client for that12:53
flaper87vkmc: sounds good, lemme know if I can help12:56
vkmcoki12:56
flaper87btw, pls, review the QueueController patch12:56
flaper87:)12:56
vkmcI'm on it12:56
flaper87awesome12:56
vkmcI know, thanks12:57
flaper87LOOOOOL12:59
*** dynarro has joined #openstack-zaqar13:02
*** sriram has joined #openstack-zaqar13:09
*** JAHoagie has joined #openstack-zaqar13:10
vkmcflaper87, I could reproduce the error in py3413:21
vkmcI gotta rush now, but I'll debug when I'm back13:21
flaper87vkmc: awesome, thanks!13:23
*** mpanetta has joined #openstack-zaqar13:26
-openstackstatus- NOTICE: gerrit has been restarted to restore event streaming. any change events missed by zuul (between 12:48 and 13:28 utc) will need to be rechecked or have new approval votes set13:29
*** sriram has quit IRC13:33
*** amitgandhinz has joined #openstack-zaqar13:34
*** sriram has joined #openstack-zaqar13:34
*** chandankumar has joined #openstack-zaqar13:53
*** ametts has joined #openstack-zaqar13:58
*** cpallares has joined #openstack-zaqar13:59
*** ametts has quit IRC14:27
*** exploreshaifali has joined #openstack-zaqar14:39
flaper87vkmc: back ?14:58
vkmcback14:58
flaper87vkmc: ok, lemme know if/when you have time to debug that py34 issue, it's annoying14:59
flaper87it has something to do with ddt's decorator15:00
flaper87but it seemed to work on other places15:00
flaper87so, no idea15:00
flaper87vkmc: btw, did you file a bug for that "queue validation" issue you found?15:02
vkmcnot yet15:02
vkmcI have it in my TODO: queue validation for lazy queue creation and another bug I don't remember now15:03
flaper87vkmc: ok, when you do, that sounds like a bug dynarro could work on, unless you fixed it alreay15:04
vkmcflaper87, no, I haven't fix it15:04
vkmcit has dynarro's name15:04
flaper87vkmc: WHAT? YOU HAVEN'T FIXED IT? You dissapoint me15:05
vkmcD:15:05
openstackgerritDoraly Navarro proposed openstack/zaqar: makes uri unique to avoid duplicated pools  https://review.openstack.org/16979815:12
*** dynarro has quit IRC15:21
*** JAHoagie has quit IRC15:23
vkmcoh ok ok15:31
*** exploreshaifali has quit IRC15:31
vkmcI think I get what this is about15:31
*** chandankumar has quit IRC15:35
*** reed has joined #openstack-zaqar15:35
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: Add websockets to our test suite  https://review.openstack.org/16931315:43
vkmcflaper87, ^15:43
*** JAHoagie has joined #openstack-zaqar16:17
*** exploreshaifali has joined #openstack-zaqar16:25
*** dynarro has joined #openstack-zaqar16:25
*** amitgandhinz has quit IRC16:26
*** amitgandhinz has joined #openstack-zaqar16:28
*** dynarro has quit IRC16:30
*** JAHoagie has quit IRC16:32
*** JAHoagie has joined #openstack-zaqar16:39
*** amitgandhinz has quit IRC16:48
*** achanda has joined #openstack-zaqar17:08
*** achanda has quit IRC17:10
*** achanda has joined #openstack-zaqar17:14
*** davideag_ has joined #openstack-zaqar17:30
*** davideagnello has quit IRC17:33
*** ametts has joined #openstack-zaqar17:43
*** exploreshaifali has quit IRC17:47
*** Akanksha08 has joined #openstack-zaqar17:58
Akanksha08Hi17:58
Akanksha08I am working on this issue https://bugs.launchpad.net/zaqar/+bug/142939317:59
openstackLaunchpad bug 1429393 in zaqar "Move debug option to common options" [Medium,In progress] - Assigned to Akanksha (akanksha-aha)17:59
Akanksha08did a grep and found this https://github.com/openstack/zaqar/blob/master/zaqar/transport/websocket/driver.py#L3618:00
Akanksha08so there is a debug option but that I could find that option for only this particular driver18:00
Akanksha08Is it expected to move only this option to the common options?18:02
*** dynarro has joined #openstack-zaqar18:05
vkmcAkanksha08, hi!18:17
vkmclet me check18:17
Akanksha08Hi18:18
Akanksha08okay18:18
openstackgerritDoraly Navarro proposed openstack/zaqar: makes uri unique to avoid duplicated pools  https://review.openstack.org/16979818:45
Akanksha08vkmc, did you check?18:47
*** achanda has quit IRC18:52
vkmcAkanksha08, I did18:56
vkmcas you say, the only appearance is in websocket18:56
vkmcso only fix it there18:56
Akanksha08okay18:56
vkmcprobably for future additions we are going to use the debug option from common18:56
*** dynarro has quit IRC18:59
Akanksha08okay18:59
Akanksha08By common do you mean the general options which are global19:01
Akanksha08https://github.com/openstack/zaqar/blob/ae7996869ff78f7377d9038ee09ef1deb2fbef6b/zaqar/bootstrap.py#L4719:01
vkmcAkanksha08, correct19:03
Akanksha08cool! Will do it then.19:04
Akanksha08Thanks a lot!19:04
vkmcnp, thanks to you19:04
Akanksha08:)19:07
flaper87vkmc: Thanks for fixing that issue with py34 <319:11
*** kgriffs|afk is now known as kgriffs19:11
flaper87now, review the freaking patch19:12
flaper87:P19:12
flaper87and the QueueController one19:12
flaper87brb, dinner19:12
*** kgriffs is now known as kgriffs|afk19:12
vkmcflaper87, np :)19:13
vkmcenjoy19:13
vkmcroger that19:13
vkmcI'll check the queuecontroller in a while, it looks good but I should re read it19:21
*** achanda has joined #openstack-zaqar19:23
*** mpanetta has quit IRC19:26
*** ametts has quit IRC19:27
openstackgerritAkanksha proposed openstack/zaqar: Moves debug option to common options  https://review.openstack.org/16989419:38
*** reed has quit IRC19:40
*** davideag_ has quit IRC19:57
*** Akanksha08 has quit IRC19:57
*** davideagnello has joined #openstack-zaqar19:58
*** davideagnello has quit IRC19:58
*** davideagnello has joined #openstack-zaqar19:58
flwangflaper87: done20:01
*** JAHoagie has quit IRC20:20
*** amitgandhinz has joined #openstack-zaqar20:25
*** JAHoagie has joined #openstack-zaqar20:30
flaper87flwang: can I get your review here? https://review.openstack.org/#/c/144340/20:41
flaper87We need to land that asap so pls, any feedback is important20:41
flaper87flwang: also, https://review.openstack.org/#/c/169313/20:42
*** mpanetta has joined #openstack-zaqar20:49
flwangflaper87: yep, sir20:51
flaper87flwang: PLEASE, PLEASE, PLEASE, PLEASE!20:52
flwangI need some time for https://review.openstack.org/#/c/144340/20:53
flwangit's a big change20:53
flaper87flwang: yes but please, I beg you to tackle it today if possible20:54
flwangflaper87: i will do that today :)20:56
flaper87flwang: awesome, thanks!20:57
flwanganytime, sir20:57
flwangbtw, I will be in vacation from tomorrow until next Thu20:57
*** achanda has quit IRC21:06
*** flwang1 has joined #openstack-zaqar21:10
flwang1flaper87: do we still need this https://review.openstack.org/#/c/146354/ ?21:11
*** sriram has quit IRC21:16
*** amitgandhinz has quit IRC21:17
*** achanda has joined #openstack-zaqar21:18
flaper87flwang1: yes, we need that!21:19
flaper87flwang1: thanks for the heads up21:19
flwang1flaper87: could you pls review it?21:19
flwang1i think we need to get it in to keep the design be consistent with the code21:19
*** boris-42 has quit IRC21:28
openstackgerritMerged openstack/zaqar: Add websockets to our test suite  https://review.openstack.org/16931321:32
*** boris-42 has joined #openstack-zaqar21:57
*** flwang1 has quit IRC22:03
*** cpallares has quit IRC22:14
*** mpanetta has quit IRC22:16
*** reed has joined #openstack-zaqar22:59
*** reed has left #openstack-zaqar22:59
*** JAHoagie has quit IRC23:11
« Tuesday, 2015-03-31 Index Thursday, 2015-04-02 »

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