Tuesday, 2015-03-31

« Monday, 2015-03-30 Index Wednesday, 2015-04-01 »
vkmcflwang, ^?00:00
*** achanda_ has joined #openstack-zaqar00:03
*** achanda has quit IRC00:07
*** achanda_ has quit IRC00:08
*** JAHoagie has quit IRC00:10
*** mpanetta has joined #openstack-zaqar00:12
*** mpanetta has quit IRC00:20
flwangvkmc: just back from lunch00:33
flwanglooking...00:33
vkmcflwang, thx00:36
*** mpanetta has joined #openstack-zaqar00:51
*** mpanetta has joined #openstack-zaqar00:52
*** amalagon has joined #openstack-zaqar01:04
*** mpanetta has quit IRC01:21
*** chandankumar has joined #openstack-zaqar01:49
*** chandankumar has quit IRC02:04
*** achanda has joined #openstack-zaqar02:34
flwangvkmc: ping02:38
*** achanda has quit IRC02:44
*** achanda has joined #openstack-zaqar02:45
vkmcflwang, pong02:59
*** amalagon has quit IRC03:00
flwangjust reviewed the code https://review.openstack.org/#/c/144803/14/zaqar/tests/unit/transport/websocket/v1_1/test_queue_lifecycle.py  would you mind explaining "I'm a bit constrained on the asserts I can do"?03:01
vkmcsure03:01
vkmcwell, for the queue tests it was ok03:02
vkmcI'm working on the messages tests03:02
vkmcand in some cases, I need to retrieve the response in order to make test cases more real03:02
vkmcgiven that we are using and event loop (server side)03:03
vkmcwe need to emulate it with side effects03:04
vkmcthat is... we create a mock for the onMessage function (triggered when the server receives a message)03:04
flwangyes, i saw that03:05
vkmcand then we retrieve the response with the validator function03:05
vkmcthing is... the response is not accessible from outside that function03:05
vkmcand I cannot think on an elegant solution for it03:05
flwangso you want a 'real' response instead of mocking it?03:08
vkmcI would need a way to get the response, as we are doing now, and be able to access it from other tests (not only inside validator())03:10
vkmchmm its hard to explain without the code03:18
*** mpanetta has joined #openstack-zaqar03:22
*** mpanetta has quit IRC03:27
*** amalagon has joined #openstack-zaqar03:31
*** jasondotstar has quit IRC04:12
*** achanda has quit IRC04:12
*** jasondotstar has joined #openstack-zaqar04:14
*** achanda has joined #openstack-zaqar04:19
*** mpanetta has joined #openstack-zaqar04:23
*** achanda has quit IRC04:27
*** mpanetta has quit IRC04:27
*** achanda has joined #openstack-zaqar04:30
*** achanda has quit IRC04:40
*** achanda has joined #openstack-zaqar04:40
*** chandankumar has joined #openstack-zaqar04:47
*** achanda has quit IRC04:51
*** JAHoagie has joined #openstack-zaqar05:21
*** achanda has joined #openstack-zaqar05:22
*** exploreshaifali has joined #openstack-zaqar05:40
*** reed has quit IRC05:47
*** exploreshaifali has quit IRC05:53
*** chandankumar has quit IRC06:06
*** achanda has quit IRC06:08
*** achanda has joined #openstack-zaqar06:08
*** chandankumar has joined #openstack-zaqar06:43
*** achanda has quit IRC07:05
*** mpanetta has joined #openstack-zaqar07:24
*** mpanetta has quit IRC07:29
*** JAHoagie has quit IRC07:33
*** amalagon has quit IRC08:41
*** amalagon has joined #openstack-zaqar08:42
*** amalagon has quit IRC08:46
-openstackstatus- NOTICE: CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment.08:55
*** ChanServ changes topic to "CI Check/Gate pipelines currently stuck due to a bad dependency creeping in the system. No need to recheck your patches at the moment."08:55
openstackgerritFlavio Percoco proposed openstack/zaqar: Remove QueueController from data to control plane  https://review.openstack.org/14434009:59
*** achanda has joined #openstack-zaqar10:06
*** achanda has quit IRC10:10
*** chandankumar has quit IRC10:50
*** chandankumar has joined #openstack-zaqar11:04
*** flaper87 has quit IRC11:10
*** vkmc has quit IRC11:10
*** flaper87 has joined #openstack-zaqar11:22
*** mpanetta has joined #openstack-zaqar11:26
*** flaper87 has quit IRC11:26
*** flaper87 has joined #openstack-zaqar11:26
*** mpanetta has quit IRC11:30
*** chandankumar has quit IRC11:38
*** amalagon has joined #openstack-zaqar11:41
*** ChanServ changes topic to "OpenStack Queuing and Notification Service || Smile :D || Meetings every Monday @ 21:00 UTC/15:00 UTC || Wiki: https://wiki.openstack.org/wiki/Zaqar || Paste: http://paste.openstack.org/ || Send messages and make some noise :D"11:51
-openstackstatus- NOTICE: Check/Gate unstuck, feel free to recheck your abusively-failed changes.11:51
*** vkmc has joined #openstack-zaqar11:51
vkmcmorning11:51
flaper87vkmc: morning girl12:26
flaper87I'm rechecking all the patches now that the gate is unstuck12:26
vkmcthx flaper8712:26
*** chandankumar has joined #openstack-zaqar12:27
*** mpanetta has joined #openstack-zaqar12:39
*** dynarro has joined #openstack-zaqar12:44
openstackgerritMerged openstack/zaqar: Implement redis driver for notifications  https://review.openstack.org/16489313:05
*** sriram has joined #openstack-zaqar13:05
flaper87w00000t13:05
openstackgerritMerged openstack/zaqar: Adds message processing to WebSockets driver  https://review.openstack.org/14480313:06
vkmc:D13:06
vkmcomg omg omg13:06
*** chandankumar has quit IRC13:23
*** JAHoagie has joined #openstack-zaqar13:27
openstackgerritFlavio Percoco proposed openstack/zaqar: Move devstack's Zaqar code into the codebase  https://review.openstack.org/16930513:30
openstackgerritFlavio Percoco proposed openstack/zaqar: Move devstack's Zaqar code into the codebase  https://review.openstack.org/16930513:33
flaper87vkmc: https://review.openstack.org/#/c/142432/13:36
openstackgerritFlavio Percoco proposed openstack/zaqar: Add websockets to our test suite  https://review.openstack.org/16931313:45
*** amitgandhinz has joined #openstack-zaqar13:46
flaper87vkmc: ^13:46
vkmcthx13:47
vkmcone nit13:47
vkmca temp file made it through https://review.openstack.org/#/c/169313/1/tests/unit/transport/websocket/test_v1_1.py~13:47
vkmcand let me see the devstack one13:48
flaper87fuck13:49
openstackgerritFlavio Percoco proposed openstack/zaqar: Add websockets to our test suite  https://review.openstack.org/16931313:50
flaper87vkmc: fixed13:50
vkmctx13:51
*** ametts has joined #openstack-zaqar13:59
*** cpallares has joined #openstack-zaqar14:00
*** JAHoagie has quit IRC14:13
*** kgriffs|afk is now known as kgriffs14:28
*** chandankumar has joined #openstack-zaqar14:50
flaper87vkmc: https://review.openstack.org/#/c/166145/14:55
flaper87vkmc: btw, https://review.openstack.org/#/c/144340/ <- all green!14:57
flaper87:)14:57
* vkmc nods14:57
vkmcw00000t14:57
flaper87flwang: ^^^^^^14:57
flaper87pls pls pls, review zaqar patches14:57
flaper87pls14:57
flaper87I know you're a busy man but PLEASE!14:57
vkmcflwang, ^^^^^^^^^14:57
vkmchahaha14:57
flaper87:P14:58
flaper87vkmc: team work!14:58
* flaper87 highfives vkmc14:58
* vkmc highfives back flaper87 14:58
*** JAHoagie has joined #openstack-zaqar15:06
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: API v1.1 Messages endpoints  https://review.openstack.org/16607615:13
*** reed has joined #openstack-zaqar15:13
vkmcflaper87, ^15:15
flaper87vkmc: already clicked !15:18
* flaper87 is one step ahead15:18
vkmcflaper87, ^15:18
* vkmc can be really annoying 15:18
flaper87vkmc: ^15:19
*** exploreshaifali has joined #openstack-zaqar15:25
*** chandankumar has quit IRC15:27
openstackgerritMerged openstack/zaqar: Updates the readme.rst file to add dependencies steps  https://review.openstack.org/16614515:31
*** amalagon has quit IRC15:52
*** chandankumar has joined #openstack-zaqar15:55
openstackgerritFlavio Percoco proposed openstack/zaqar: API v1.1 Messages endpoints  https://review.openstack.org/16607616:00
*** JAHoagie has quit IRC16:02
*** JAHoagie has joined #openstack-zaqar16:10
flaper87vkmc: https://review.openstack.org/#/c/142432/16:11
flaper87:P16:11
*** JAHoagie has quit IRC16:14
*** kgriffs is now known as kgriffs|afk16:21
*** JAHoagie has joined #openstack-zaqar16:42
*** chandankumar has quit IRC16:44
*** chandankumar has joined #openstack-zaqar16:48
*** dynarro has quit IRC16:57
*** amalagon has joined #openstack-zaqar17:03
*** JAHoagie has quit IRC17:04
*** chandankumar has quit IRC17:06
*** amalagon has quit IRC17:08
*** JAHoagie has joined #openstack-zaqar17:21
*** chandankumar has joined #openstack-zaqar17:22
*** achanda has joined #openstack-zaqar17:23
vkmcflaper87, I think I found a bug17:29
vkmcwhen queues are lazily created we don't perform any validation17:30
*** JAHoagie has quit IRC17:31
vkmchttps://github.com/openstack/zaqar/blob/master/zaqar/transport/wsgi/v2_0/messages.py#L176-L17717:31
flaper87vkmc: I think you're right17:32
flaper87:/17:32
flaper87We don't need to check if the queue exists but we do need to check the name17:32
flaper87As in, the lenght17:32
vkmcyeah17:32
vkmcand ifs ascii or not17:32
vkmcs/ifs/if it is17:33
flaper87right17:33
flaper87vkmc: good catch17:33
sriramI think I'm the one to blame here. I wrote that code :/ ( if I remember correctly)17:33
vkmc>:)17:33
vkmcno sriram... its kgriffs|afk fault17:33
flaper87sriram: hehe, it's always kgriffs|afk fault17:33
flaper87LOL17:33
vkmclol17:33
sriramheh17:33
flaper87sriram: we miss you here :(17:34
flaper87come back17:34
flaper87please17:34
vkmcyeah17:34
flaper87I can pay you with gummy bears17:34
* vkmc sad panda17:34
* flaper87 big sad, hungry panda17:34
sriramhaha, maybe I can pick up on some things to work on weekends :)17:34
sriramdo you still have low hanging fruits available? :P17:35
vkmcthere will be always a low hanging fruit in Zaqar17:38
*** mpanetta has quit IRC17:39
sriramcool, I'll go take a look, and pick something up to work on weekends.17:39
*** mpanetta has joined #openstack-zaqar17:39
vkmcyaay17:40
vkmcgo sriram :D17:40
sriram:)17:40
*** JAHoagie has joined #openstack-zaqar17:41
flaper87http://readwrite.com/2015/03/30/github-biggest-ddos-attack17:41
flaper87vkmc: flwang https://review.openstack.org/#/c/169305/17:44
flaper87vkmc: https://review.openstack.org/#/c/142432/17:44
* flaper87 keeps putting preasure on vkmc17:44
vkmcI want gummybears17:51
srirammight actually go out and buy some today :P17:52
vkmcflaper87, today is the last day for features17:52
vkmcsriram, you totally should17:52
mpanettasriram: Don't buy the sugar free ones!17:53
vkmcshh mpanetta17:53
vkmcthe sugar free ones are the most fun17:53
srirammpanetta: yeah, I heard they're no good. :P17:53
mpanettavkmc: haha, for a joke maybe :P17:53
mpanettasriram: something like that :P17:54
vkmcsriram, mpanetta, will you be attending the summit?17:54
mpanettaNot I17:54
sriramnope17:54
vkmcboo17:54
mpanettaamitgandhinz: is tho17:54
vkmcok, I won't bring gummybears then17:55
srirammalini might be as well ;)17:55
vkmc:D17:55
amitgandhinzyup, malini and I will be there17:59
vkmcyaay, hope we can finally meet!18:10
*** chandankumar has quit IRC18:31
openstackgerritMerged openstack/zaqar: Add REST API for notification/subscriptions  https://review.openstack.org/14243218:31
vkmcwut wut wut18:33
*** achanda has quit IRC18:49
*** kgriffs|afk is now known as kgriffs18:54
*** kgriffs is now known as kgriffs|afk18:54
*** cpallares has quit IRC18:55
*** ametts has quit IRC18:59
*** dynarro has joined #openstack-zaqar19:16
*** achanda has joined #openstack-zaqar19:27
*** reed has quit IRC19:36
*** reed has joined #openstack-zaqar19:36
flaper87vkmc: something urgent that needs to be taken care of ?19:54
flaper87flwang: around?19:54
vkmcflaper87, nothing I can recall of19:54
vkmcI'm fixing some stuff, I'll upload soon for review19:55
flaper87vkmc: coole, ping me19:55
flwangflaper87: i'm here19:59
flwangi'm reviewing your devstack patch19:59
flaper87flwang: hey :)19:59
flaper87flwang: thank you!19:59
flaper87so, most of your patches landed19:59
flaper87what's missing now?19:59
flwangand going to rebase the webhook patch19:59
flaper87how can I help?19:59
flaper87AWESOME!19:59
flwangif you can be around for 1 or 2 hours, i need your review :)20:00
flaper87I'll likely be20:00
flwangbut I assume it is not so urgent20:00
flwangflaper87: https://review.openstack.org/#/c/169305/2/devstack/settings line 320:00
flwangwhy "we have to add ec2-api to enabled services for screen_it to work"?20:01
flaper87LOL20:06
flaper87I copy/paste that20:06
flaper87it's jsut a comment, lemme fix it20:06
openstackgerritFlavio Percoco proposed openstack/zaqar: Move devstack's Zaqar code into the codebase  https://review.openstack.org/16930520:07
flaper87flwang: ^20:07
flwangflaper87: :)20:13
flwangflaper87: so do we need to keep the consistence with the plugin in devstack in the future?20:14
flaper87flwang: no, the code in devstack will go away20:16
flaper87we need to land this first20:16
flaper87then make the gate use the plugin20:16
flaper87and then remove the code in devstack20:16
flwangflaper87: ok, is it a new plan for all the services not the 'integrated' status?20:19
*** achanda has quit IRC20:20
*** dynarro has quit IRC20:29
vkmc:x20:48
vkmcsorry20:48
openstackgerritVictoria Martínez de la Cruz proposed openstack/zaqar: API v1.1 Messages endpoints  https://review.openstack.org/16607620:49
vkmcflaper87, flwang ^20:59
flwangvkmc: will do20:59
vkmcthx21:00
* flaper87 clicks21:00
vkmcgood morning :)21:00
flaper87flwang: it's more like most of the projects will follow that21:00
flwangflaper87: ok, cool21:03
*** sriram has quit IRC21:12
*** achanda has joined #openstack-zaqar21:14
*** mpanetta has quit IRC21:20
*** exploreshaifali has quit IRC21:25
*** amitgandhinz has quit IRC22:01
openstackgerritFei Long Wang proposed openstack/zaqar: Implement webhook notifier driver  https://review.openstack.org/15152922:07
*** flwang has quit IRC23:00
*** flwang has joined #openstack-zaqar23:13
*** davideagnello has quit IRC23:15
*** davideagnello has joined #openstack-zaqar23:24
*** achanda has quit IRC23:41
*** achanda has joined #openstack-zaqar23:55
« Monday, 2015-03-30 Index Wednesday, 2015-04-01 »

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