Friday, 2015-09-04

« Thursday, 2015-09-03 Index Saturday, 2015-09-05 »
*** malini has quit IRC00:00
*** amitgandhinz has joined #openstack-zaqar00:32
*** amitgandhinz has quit IRC00:32
*** amitgandhinz has joined #openstack-zaqar00:33
*** Qiming_ has quit IRC02:17
*** amitgandhinz has quit IRC02:25
*** amitgandhinz has joined #openstack-zaqar02:25
*** amitgandhinz has quit IRC02:34
*** openstackgerrit_ has joined #openstack-zaqar03:11
*** flwang1 has quit IRC03:28
*** flwang1 has joined #openstack-zaqar03:34
*** openstackgerrit_ has quit IRC04:14
*** diga has joined #openstack-zaqar04:26
*** diga has quit IRC04:59
openstackgerritFlavio Percoco proposed openstack/zaqar: Policy support  https://review.openstack.org/20991007:38
*** boris-42 has quit IRC08:30
*** bradjones has joined #openstack-zaqar08:47
*** bradjones has joined #openstack-zaqar08:47
*** flwang has joined #openstack-zaqar09:25
flwangflaper87: ping09:25
flaper87flwang: pong09:26
flwangflaper87: can you remind me why the foreign key is linked to poolgroup instead of pools? see https://github.com/openstack/zaqar/blob/master/zaqar/storage/sqlalchemy/tables.py#L4909:26
flaper87flwang: because we associate a flavor to a group of pools that share the same capabilities09:27
flaper87flwang: for example, "Flavor durable" -> ["Mongodb-West", "MongoDB-APAC", "Swift-SouthAmerica"]09:29
flaper87flwang: when you create a queue with flavor durable, it may end-up in any of those pools09:29
flwangflaper87: so the pool in api is actually a pool group, instead of a pool, right?09:30
flaper87flwang: for flavors creation, yes09:30
flwang{   "pool": "fast_storage",   "capabilities": {       durable: true   } }09:30
flwangit's really confused i think09:30
flwangwhy don't we use pool_group instead of pool?09:31
flwangany specific consideration?09:31
flaper87flwang: wait, that's v1.109:31
flaper87flwang: remember that in v1.1 we had to specify capabilities manually09:31
flaper87in v2 it's a read-only field introspected from the pool group09:31
flaper87flwang: I'd really recommend to just use v209:32
flwangflaper87: so whats the post body for v2 to create a flavor?09:32
flaper87flwang: should be: {'name': ..., 'pool': ...}09:33
flaper87flwang: unless there's a bug09:34
flaper87:P09:34
flwangflaper87: and the pool in {'name': ..., 'pool': ...} is a pool or pool_group09:34
flaper87flwang: correct09:34
flaper87flwang: we could call it pool_group09:34
* flaper87 shrugs09:34
flaper87we still have time, I believe09:34
flwangflaper87: it would be nice for me :)09:35
flwangwe can do it in M09:35
flaper87flwang: mmh, if we do it, we should probably do it now.09:35
flwangflaper87: does that mean you don't mind me changing it in L? ;)09:35
flaper87flwang: right, because it's a breaking change09:36
flwangflaper87: ok, cool09:36
flaper87we actually released it in J09:36
flaper87but09:36
* flaper87 shrugs09:36
flwangok, next question, boss09:36
flwangIIUC, the flavor is only used by queue creation, right?09:37
flaper87yes09:37
* flwang is going to talk the topic flaper87 never want to talk09:38
flaper87LOOOOOOOOOOL09:38
* flaper87 prepares his shotgun09:38
flwangsoooooooo, another fancy feature depends on queue09:39
flwang:D09:39
flwangok, seriously, the question is we're marking the flavors endpoints as private09:39
flaper87yes09:40
flaper87I mean, it used to be09:40
flaper87BUT!09:40
flaper87now that policy is almost in09:40
flwangthat means end user can't know flavors by list, unless i missed something09:40
flaper87we should have it on always and have the post operations limited to users09:40
flaper87limited to admins09:40
flwangflaper87: you got my point09:40
* flaper87 reads minds09:40
flwangi think now the flavor is not really usable until we open the list09:41
flaper87flwang: which is why I think we can change `pool` to `pool_group`09:41
flaper87:D09:41
flaper87and noone will be hurt09:42
flwangflaper87: oh, man09:42
flwangyou're always correct09:42
flaper87lol, not really but I'll take that09:42
flaper87:D09:42
flwangflaper87: btw, so for creating queue with flavor, we just need to specify    flavor=flavor_name in queue's metadata, right?09:44
flaper87flwang: yes, that sounds correct09:45
flwangflaper87: cool09:45
flwangi just found some bugs as for flavor, patches are coming soon09:46
flaper87(brb)09:46
flaper87flwang: ah awesome09:46
flaper87flwang: thanks09:46
flaper87flwang: pls, file them on LP09:46
flaper87and feel free to ask for help09:46
flwangflaper87: sure, cheers man09:46
* flaper87 back09:59
openstackgerritFei Long Wang proposed openstack/zaqar: Fix typos in sqlachemy flavor  https://review.openstack.org/22044310:01
openstackgerritFei Long Wang proposed openstack/zaqar: Fix typos in sqlalchemy flavor.py  https://review.openstack.org/22044310:07
openstackgerritClaudiu Belu proposed openstack/zaqar: Adds oslo_cache dependency  https://review.openstack.org/21937410:13
*** Qiming_ has joined #openstack-zaqar10:34
openstackgerritMerged openstack/zaqar: Fix typos in sqlalchemy flavor.py  https://review.openstack.org/22044310:58
*** flwang has quit IRC11:08
*** flwang has joined #openstack-zaqar11:21
*** diga has joined #openstack-zaqar11:24
*** malini has joined #openstack-zaqar11:31
*** flwang has quit IRC11:39
openstackgerritFlavio Percoco proposed openstack/zaqar: Policy support  https://review.openstack.org/20991011:49
flaper87^ that's the one, I can feel it11:49
flaper87flwang1: https://review.openstack.org/#/c/218072/ Are you going to update that one?11:50
flaper87based on therve's comments11:50
therveflaper87, We talked about it and thought that we could try later on11:51
therveflwang1 thinks it may have a bigger impact that we don't know about11:51
flaper87therve: ah, ok! Thanks for the heads up11:51
openstackgerritFlavio Percoco proposed openstack/zaqar: Refactoring to make Websocket driver available in v2  https://review.openstack.org/21843011:51
flaper87vkmc: ^ I just rebased it11:51
flaper87therve: hate to ask you this but, could you rebase https://review.openstack.org/#/c/206080/ on top of https://review.openstack.org/#/c/206080/ ?11:56
flaper87feature wise, it doesn't make much sense to land that in v1_1 and then land the migration11:57
flaper87and I'd like to avoid causing more conflicts to vkmc's patch11:57
flaper87therve: I can do it if you want11:57
therveYep makes sense11:57
flaper87(or if you're busy)11:57
therveNo worries11:57
flaper87cool11:57
flaper87thanks11:57
therve(I guess you meant on top of https://review.openstack.org/218430)11:58
flaper87therve: lol, yeah. Failed to copy/paste11:58
therveI tested email notifications it seemed to work well btw11:59
flaper87therve: niiiice!12:00
openstackgerritMerged openstack/zaqar: Don't register the default pool  https://review.openstack.org/22017112:13
openstackgerritMerged openstack/zaqar: Fix duplicated notification  https://review.openstack.org/21807212:15
openstackgerritThomas Herve proposed openstack/zaqar: Notifications over websocket  https://review.openstack.org/20608012:17
vkmchey!12:18
flaper87vkmc: YO!12:20
vkmcflaper87, how are you my friend?12:21
flaper87vkmc: I'md doing great, you?12:21
* flaper87 is happy to see all the great progress in Zaqar12:21
*** malini has quit IRC12:21
vkmcflaper87, good good12:23
vkmc:D12:23
vkmcawesome! liberty has been really good for us12:23
*** diga has quit IRC12:27
flaper87it has, we've been silently making great progress12:27
flaper87I shall send an update to the mailing list and share the good news12:27
flaper87our gate logs are sooooo useless12:30
flaper87I'll work on that next week, I'm really tired of these logs not showing anything12:30
flaper87now that we have uwsgi in the gate, I'll get rid of that `fork` code we had added12:31
openstackgerritFlavio Percoco proposed openstack/zaqar: Policy support  https://review.openstack.org/20991012:32
*** Qiming_ has quit IRC12:44
flaper87therve: ryansb flwang1 it'd be awesome to get your reviews on this patch: https://review.openstack.org/#/c/218430/12:44
*** Qiming_ has joined #openstack-zaqar12:44
flaper87That'll help us close persistent-transports12:44
flaper87vkmc: should we mention the blueprint on the commit message?12:45
flaper87and DocImpact / APIImpact12:45
flaper87well, probably the flags are not needed12:45
flaper87since there's no docs about this yet12:46
vkmceerh, in which patch you mean?12:46
vkmcif the patch is part of the bp, yeah we should add that12:46
flaper87vkmc: moving websocket to v2.012:47
vkmcok yes12:47
flaper87it's kind of part of persistent-transports12:47
vkmcit is!12:47
ryansbflaper87: will do12:48
ryansband yes, please mention apiimpact12:48
flaper87vkmc: what did we say about the message_ids thing?12:49
flaper87I think we agreed on removing it, right?12:49
vkmcwe didn't chat about that again12:49
flaper87ah, good point12:49
flaper87lets not do it in this patch12:49
flaper87we'll discuss this again12:49
vkmcfor the Juno summit we said we should remove it12:50
flaper87right, I think we "just" forgot to12:50
vkmcbut I think we should discuss things again12:50
vkmcthings have changed a lot since then12:50
vkmcno, we were planning to take those off on v2.012:50
flaper87vkmc: which landed in Kilo :)12:52
vkmcyeah well12:52
vkmcI think we should talk more about s/queue/topic12:52
vkmcand if in that scenario makes sense to have get_by_id12:53
flaper87vkmc: even under a topic based messaging system, I kinda feel we shouldn't have get_by_id12:55
vkmcthen we shouldn't have delete_by_id either12:55
flaper87vkmc: we need it to remove mesages :)12:55
vkmcbut it doesn't make sense12:55
flaper87we can rename it to "ack" if you want12:55
flaper87it does, the difference is that you access messages in a certain order and then you delete them based on what you've processed12:56
vkmchmm12:56
vkmcbrb, meeting12:57
openstackgerritFlavio Percoco proposed openstack/zaqar: Refactoring to make Websocket driver available in v2  https://review.openstack.org/21843012:59
thervevkmc, flaper87: found one small issue13:02
flaper87therve: yes, let me update that since vkmc is in a meeting13:03
flaper87not sure if those reposnse schemas are really useful right now, they are used in functional tests (or supposed to be used)13:04
vkmcthey are not, we should remove those13:04
vkmcI was going to do that in a followup :)13:05
openstackgerritFlavio Percoco proposed openstack/zaqar: Refactoring to make Websocket driver available in v2  https://review.openstack.org/21843013:05
flaper87vkmc: cool13:05
flaper87ok, updated the patch13:05
vkmcthx13:05
flaper87vkmc: np13:05
openstackgerritFlavio Percoco proposed openstack/zaqar: Policy support  https://review.openstack.org/20991013:42
flaper87I'll get it right, I promisse13:42
* flaper87 too lazy to setup an env to test this locally w/ tempest13:42
flaper87if it fails again, I will13:42
flaper87promissed13:42
flaper87:d13:42
flaper87:D13:42
*** JAHoagie has joined #openstack-zaqar13:45
* flaper87 is excited about this policy feature13:46
*** ametts has joined #openstack-zaqar13:49
therveflaper87, Hum. It doesn't support websocket.13:57
*** dynarro has joined #openstack-zaqar13:57
flaper87therve: yeah, working on that :D13:58
therveAh cool :)13:58
flaper87therve: but likely to land in a follow-up patch13:58
therveThat's fine, was wondering it was on the map13:59
therveAh, I got that intermittent claim failure locally as well :/14:00
*** malini has joined #openstack-zaqar14:06
flaper87mmh, sounds like a race14:07
flaper87ah yeah, age is different14:08
openstackgerritThomas Herve proposed openstack/zaqar: Remove race condition in test_claim_lifecycle  https://review.openstack.org/22052114:08
therveflaper87, ^14:08
flaper87lets stop the time14:08
flaper87w000h00014:08
therveI actually opened a bug 2 months ago and forgot about it :/14:08
flaper87therve: it fell off my radar :/14:09
flaper87therve: +2, thank14:09
flaper87thanks*14:09
*** JAHoagie has quit IRC14:11
*** malini has quit IRC14:11
*** malini has joined #openstack-zaqar14:11
*** csoukup has joined #openstack-zaqar14:13
openstackgerritFlavio Percoco proposed openstack/zaqar: Refactoring to make Websocket driver available in v2  https://review.openstack.org/21843014:13
flaper87IT'S GREEEEEEEEN!14:16
dynarroflaper87, could you help me with this? I'm getting this error, but I don't understand the problem http://paste.openstack.org/show/445451/14:16
* flaper87 clicks14:17
flaper87dynarro: super(Client, self).__init__(self, url=url, version=version, conf=conf) <- you don't need the second `self`14:19
flaper87because `self` is already in the `super()` call14:19
dynarroooh! makes sense14:20
dynarrothanks!14:20
flaper87dynarro: anytime14:21
vkmcdone14:22
flaper87vkmc: ryansb https://review.openstack.org/#/c/209910/14:25
flaper87:D14:25
therveflaper87, Do we need the eventlet stuff?14:28
therveIt looks like it should be fairly easy to get the request object instead14:31
*** amitgandhinz has joined #openstack-zaqar14:34
*** dynarro has quit IRC14:48
*** malini has quit IRC14:54
*** kgriffs|afk is now known as kgriffs15:27
*** Qiming__ has joined #openstack-zaqar15:35
*** Qiming_ has quit IRC15:35
*** malini has joined #openstack-zaqar15:36
*** JAHoagie has joined #openstack-zaqar15:37
*** dynarro has joined #openstack-zaqar16:06
openstackgerritDoraly Navarro proposed openstack/python-zaqarclient: Added support to v2  https://review.openstack.org/20923816:08
*** Qiming__ has quit IRC16:13
*** lambdadoodle has joined #openstack-zaqar16:37
*** ametts has quit IRC16:37
*** davideagnello has quit IRC16:40
*** davideagnello has joined #openstack-zaqar16:41
lambdadoodlehey my tox instance used during genconfig isnt terminating ... any pointers ?16:48
*** malini has quit IRC16:48
*** amitgandhinz has quit IRC16:49
openstackgerritvenkatamahesh proposed openstack/zaqar: Fix accurate links in README.rst  https://review.openstack.org/22058816:52
flaper87lambdadoodle: what error is it throwing ?16:54
lambdadoodleno error .. just stuck at genconfig installdeps: -r/home/rohan/zaqar/requirements.txt, -r/home/rohan/zaqar/test-requirements.txt16:56
flaper87lambdadoodle: could it be that it's just taking longer to download dependencies?16:56
lambdadoodleflaper87: YES!16:57
lambdadoodleIt just terminated :)16:57
ryansbwell there you have it16:57
flaper87:)16:57
flaper87lambdadoodle: patience, young padawan. Patience.16:57
flaper87:D16:58
* flaper87 likes lambdadoodle's nick16:58
*** malini has joined #openstack-zaqar16:59
flaper87therve: I thought the threading local was something we had merged before!16:59
* flaper87 should learn to be patient17:00
flaper87therve: I'll take your code17:00
*** malini has quit IRC17:00
*** openstackgerrit_ has joined #openstack-zaqar17:02
openstackgerritFlavio Percoco proposed openstack/zaqar: Policy support  https://review.openstack.org/20991017:03
flaper87therve: ryansb vkmc ^17:03
*** exploreshaifali has joined #openstack-zaqar17:09
ryansbflaper87: when does z5 ship?17:10
*** amitgandhinz has joined #openstack-zaqar17:20
*** dynarro has quit IRC17:21
lambdadoodlepip install -e . returns an error ... No module named pbr_json17:23
flaper87lambdadoodle: pbr_json ?17:24
lambdadoodleImportError: No module named pbr_json17:24
flaper87o.017:24
lambdadoodleflaper87: thats the exact line17:24
* flaper87 tries17:25
flaper87mmh, it worked17:25
flaper87mmh17:25
flaper87lambdadoodle: is that master?17:25
lambdadoodleflaper87: Command "python setup.py egg_info" failed with error code 117:25
lambdadoodlewhat master?17:25
flaper87lambdadoodle: master branch17:25
lambdadoodleit is so i think17:26
lambdadoodledoes zaqar NEED openstack components?17:27
flaper87lambdadoodle: other than keystone for auth, it doesn't. And even keystone is optional17:28
flaper87vkmc: ryansb: https://review.openstack.org/#/c/218430/ https://review.openstack.org/#/c/209910/17:29
*** amitgandhinz has quit IRC17:30
lambdadoodleso how do i get around this pbr_json thing ?17:31
flaper87lambdadoodle: I honestly have no idea, it's the first time I see that. Try removing the local pbr copies if you have any17:33
* flaper87 has to go now17:33
*** lambdadoodle has quit IRC18:05
*** kgriffs is now known as kgriffs|afk18:19
*** amitgandhinz has joined #openstack-zaqar18:25
*** JAHoagie has quit IRC18:27
*** JAHoagie has joined #openstack-zaqar18:46
*** bradjones has quit IRC19:03
*** bradjones has joined #openstack-zaqar19:10
*** bradjones has joined #openstack-zaqar19:10
*** amitgandhinz has quit IRC19:11
*** kgriffs|afk is now known as kgriffs19:21
*** JAHoagie has quit IRC19:59
*** boris-42 has joined #openstack-zaqar20:03
*** openstackgerrit_ has quit IRC20:03
*** openstackgerrit_ has joined #openstack-zaqar20:04
*** malini has joined #openstack-zaqar20:05
*** EmilienM has quit IRC20:16
*** EmilienM has joined #openstack-zaqar20:17
*** malini has quit IRC20:26
*** exploreshaifali has quit IRC20:29
*** malini has joined #openstack-zaqar20:33
*** openstackgerrit_ has quit IRC20:47
*** openstackgerrit_ has joined #openstack-zaqar20:47
*** openstackgerrit_ has quit IRC20:48
*** openstackgerrit_ has joined #openstack-zaqar20:48
*** openstackgerrit_ has quit IRC20:53
*** malini has left #openstack-zaqar20:57
*** davideag_ has joined #openstack-zaqar21:06
*** ryansb has quit IRC21:07
*** davideagnello has quit IRC21:07
*** ryansb has joined #openstack-zaqar21:10
*** ryansb has quit IRC21:10
*** ryansb has joined #openstack-zaqar21:10
*** whydidyoustealmy has quit IRC21:11
*** whydidyoustealmy has joined #openstack-zaqar21:18
*** csoukup has quit IRC22:30
*** kgriffs is now known as kgriffs|afk23:34
« Thursday, 2015-09-03 Index Saturday, 2015-09-05 »

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