*** achanda has quit IRC | 00:00 | |
*** achanda_ has quit IRC | 00:01 | |
*** Qiming has joined #openstack-zaqar | 01:03 | |
*** mpanetta has joined #openstack-zaqar | 02:03 | |
*** mpanetta has quit IRC | 02:04 | |
*** mpanetta has joined #openstack-zaqar | 02:04 | |
*** boris-42 has quit IRC | 02:10 | |
*** csoukup has joined #openstack-zaqar | 02:21 | |
*** achanda has joined #openstack-zaqar | 02:28 | |
flwang | vkmc: any lucky you around? | 02:33 |
---|---|---|
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 02:38 |
vkmc | flwang, :) course | 02:40 |
vkmc | flwang, morning | 02:40 |
flwang | vkmc: what's your current time? | 03:03 |
flwang | i think i need your help | 03:03 |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix duplicated notification https://review.openstack.org/218072 | 03:20 |
*** mpanetta_ has joined #openstack-zaqar | 03:26 | |
*** kgriffs|afk has quit IRC | 03:28 | |
*** VeggieMeat_ has joined #openstack-zaqar | 03:28 | |
*** mpanetta has quit IRC | 03:28 | |
*** flwang has quit IRC | 03:28 | |
*** VeggieMeat has quit IRC | 03:28 | |
*** flwang has joined #openstack-zaqar | 03:29 | |
*** kgriffs|afk has joined #openstack-zaqar | 03:33 | |
*** kgriffs|afk is now known as kgriffs | 03:34 | |
*** achanda has quit IRC | 03:41 | |
*** VeggieMeat_ is now known as VeggieMeat | 03:57 | |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Add email driver for notification https://review.openstack.org/202374 | 03:58 |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix GET queue metadata for sqlalchemy https://review.openstack.org/218083 | 04:15 |
*** mpanetta_ has quit IRC | 04:49 | |
*** flwang has quit IRC | 05:15 | |
*** csoukup has quit IRC | 05:52 | |
*** mpanetta has joined #openstack-zaqar | 06:13 | |
openstackgerrit | Yang Shengming proposed openstack/zaqar: Update 'doc/source/development-environment.rst' https://review.openstack.org/217035 | 06:13 |
*** mpanetta has quit IRC | 06:14 | |
*** mpanetta has joined #openstack-zaqar | 06:14 | |
*** akanksha has joined #openstack-zaqar | 06:46 | |
*** flwang has joined #openstack-zaqar | 06:51 | |
*** achanda has joined #openstack-zaqar | 07:05 | |
*** flwang has quit IRC | 07:11 | |
flaper87 | o/ | 07:31 |
*** Ram_ has joined #openstack-zaqar | 07:40 | |
*** Ram_ has quit IRC | 07:42 | |
*** dynarro has joined #openstack-zaqar | 07:49 | |
*** achanda has quit IRC | 07:56 | |
*** mpanetta has quit IRC | 08:07 | |
*** mpanetta has joined #openstack-zaqar | 08:38 | |
*** mpanetta has quit IRC | 08:42 | |
*** exploreshaifali has joined #openstack-zaqar | 09:04 | |
*** mpanetta has joined #openstack-zaqar | 09:08 | |
exploreshaifali | morning everyone! | 09:08 |
*** dynarro has quit IRC | 09:10 | |
*** mpanetta has quit IRC | 09:16 | |
flaper87 | exploreshaifali: MORNING! | 09:32 |
openstackgerrit | Flavio Percoco proposed openstack/python-zaqarclient: Make v1.1 the default CLI version https://review.openstack.org/218169 | 09:32 |
exploreshaifali | \o | 09:33 |
flaper87 | exploreshaifali: ready to do some tests ? | 09:35 |
flaper87 | exploreshaifali: I'm still finishing up the TestDay guid while I'm testing things | 09:36 |
flaper87 | there are many CLI ops missing | 09:36 |
flaper87 | :( | 09:36 |
exploreshaifali | yes I know | 09:36 |
exploreshaifali | its my fault | 09:36 |
exploreshaifali | for flavor at lest | 09:36 |
flaper87 | exploreshaifali: there are more things missing. For instance, you can't post messages from the CLI | 09:38 |
*** akanksha has quit IRC | 09:38 | |
flaper87 | :/ | 09:38 |
flaper87 | not your fault, really | 09:38 |
flaper87 | it's just more work to do | 09:38 |
flaper87 | anyway | 09:38 |
flaper87 | lets file all these | 09:38 |
exploreshaifali | ok! | 09:39 |
exploreshaifali | my zaqar cli still not working | 09:41 |
* exploreshaifali starting everything from scratch! | 09:41 | |
flaper87 | exploreshaifali: awesome, let me know how that goes | 09:42 |
exploreshaifali | sure, Thanks :) | 09:43 |
openstackgerrit | Flavio Percoco proposed openstack/python-zaqarclient: Make sure the API version is passed down https://review.openstack.org/218173 | 09:44 |
exploreshaifali | so flaper87 I am sure again plugin will not work, so I will install devstack without zaqar and then will install zaqar and its client separately | 09:45 |
*** flwang has joined #openstack-zaqar | 09:45 | |
exploreshaifali | will you able to help me for getting endpoints of zaqar to keystone? | 09:46 |
flwang | therve: ping | 09:46 |
exploreshaifali | flwang, \o | 09:46 |
flwang | exploreshaifali: good evening | 09:46 |
exploreshaifali | good evening! | 09:46 |
flwang | therve: ping | 09:49 |
flaper87 | exploreshaifali: yup, are you using devstack ? | 09:52 |
exploreshaifali | flaper87, yes | 09:52 |
flaper87 | exploreshaifali: http://paste.openstack.org/show/430730/ | 09:52 |
flaper87 | that's what your local.conf should look like | 09:52 |
exploreshaifali | flaper87, ooooooooook! | 09:53 |
* exploreshaifali still afraid | 09:53 | |
exploreshaifali | :P | 09:53 |
exploreshaifali | let see | 09:53 |
exploreshaifali | :D | 09:53 |
flwang | flaper87: what's the plan of today? | 09:57 |
flaper87 | flwang: install Zaqar and test all the endpoints | 09:58 |
flaper87 | Ideally using zaqarclient but that's going to be hard | 09:58 |
flaper87 | Unless you write a script for that | 09:59 |
flaper87 | because there's no CLI support | 09:59 |
flaper87 | actually, I should probably write a script that calls all the APIs | 09:59 |
flwang | flaper87: should we allocate the endpoints so that each of us can focus on a range of endpoints | 10:01 |
flwang | which would be more efficient | 10:01 |
flwang | meanwhile, make sure at least all the endpoints are covered by 2+ people | 10:02 |
flaper87 | I think we should all test all the endpoints based on different scenarios: https://wiki.openstack.org/wiki/Zaqar/TestingDay/Liberty | 10:02 |
flaper87 | I'm trying to finish that guide | 10:03 |
flaper87 | :/ | 10:03 |
*** angvp has joined #openstack-zaqar | 10:03 | |
flwang | I can cover the last | 10:04 |
flwang | control : sqlalchemy data: redis | 10:04 |
flwang | that's what i'm using | 10:04 |
flaper87 | flwang: that'd be awesome | 10:04 |
flaper87 | thanks | 10:04 |
flaper87 | flwang: if you encounter issues just file bugs, don't fix them right away | 10:05 |
flaper87 | we'll collect them and fix them later | 10:05 |
flaper87 | tag the bugs as zaqar-liberty-test-day | 10:05 |
flwang | flaper87: ok, i have found some today :) | 10:05 |
flaper87 | actually, just liberty-test-day | 10:05 |
flwang | https://bugs.launchpad.net/zaqar/+bug/1489701 | 10:05 |
openstack | Launchpad bug 1489701 in zaqar "Got 503 when get queue with sqlalchemy" [Low,In progress] - Assigned to Fei Long Wang (flwang) | 10:05 |
flaper87 | sweeet | 10:06 |
flaper87 | brb, lunch | 10:07 |
flwang | btw, IIRC, we still support update queue's metadata, right? | 10:08 |
*** Qiming has quit IRC | 10:25 | |
*** kgriffs has quit IRC | 10:27 | |
*** VeggieMeat has quit IRC | 10:27 | |
*** tmu_ has quit IRC | 10:27 | |
*** flaper87 has quit IRC | 10:27 | |
*** dguerri` has quit IRC | 10:27 | |
*** davideagnello has quit IRC | 10:27 | |
*** davideagnello has joined #openstack-zaqar | 10:29 | |
*** kgriffs has joined #openstack-zaqar | 10:30 | |
*** VeggieMeat has joined #openstack-zaqar | 10:30 | |
*** tmu_ has joined #openstack-zaqar | 10:30 | |
*** flaper87 has joined #openstack-zaqar | 10:30 | |
*** dguerri` has joined #openstack-zaqar | 10:30 | |
*** flaper87 has quit IRC | 10:30 | |
*** flaper87 has joined #openstack-zaqar | 10:30 | |
*** kgriffs has quit IRC | 10:30 | |
*** VeggieMeat has quit IRC | 10:30 | |
*** tmu_ has quit IRC | 10:30 | |
*** flaper87 has quit IRC | 10:30 | |
*** dguerri` has quit IRC | 10:30 | |
*** kgriffs has joined #openstack-zaqar | 10:33 | |
*** VeggieMeat has joined #openstack-zaqar | 10:33 | |
*** tmu_ has joined #openstack-zaqar | 10:33 | |
*** flaper87 has joined #openstack-zaqar | 10:33 | |
*** dguerri` has joined #openstack-zaqar | 10:33 | |
*** malini has joined #openstack-zaqar | 10:35 | |
*** malini has quit IRC | 10:35 | |
*** malini has joined #openstack-zaqar | 10:35 | |
* flaper87 back | 10:50 | |
flaper87 | flwang: yes, that's in v1.1 | 10:50 |
flaper87 | it's set_metadata get_metadata | 10:50 |
flwang | flaper87: it's in v2 as well | 10:51 |
flwang | but I think it doesn't work | 10:51 |
flaper87 | flwang: yes, that's how we are supposed to set flavors and stuff | 10:51 |
flaper87 | oh, it doesn't work ? | 10:51 |
flaper87 | :( | 10:51 |
flwang | the metadata can be saved when creating queue | 10:51 |
flwang | but after that, the metadata can't be updated but still return 204 | 10:52 |
flwang | i'm not sure if it's a regression bug after splitting the queue controller from data plane to control plane | 10:53 |
flwang | flaper87: btw, this conf file is not correct http://paste.openstack.org/show/429438/ | 10:55 |
flwang | the management_store and message_store should be switched | 10:56 |
flaper87 | flwang: ah, feel free to update it | 10:59 |
flaper87 | thanks for spotting that | 10:59 |
flaper87 | actually, I'll do it | 11:00 |
flaper87 | flwang: http://paste.openstack.org/show/430790/ | 11:01 |
flaper87 | fixed | 11:01 |
flwang | cool | 11:02 |
flaper87 | thanks for all the help | 11:02 |
* flaper87 is working on the testing script | 11:03 | |
flaper87 | this script is becoming just another functional test | 11:08 |
flaper87 | not good, not good | 11:09 |
flaper87 | flwang: could you help writing this: https://wiki.openstack.org/wiki/Zaqar/TestingDay/Liberty#Redis | 11:09 |
flwang | flaper87: sure, no problem | 11:11 |
*** malini has quit IRC | 11:13 | |
exploreshaifali | flaper87, still it is giving same error http://paste.openstack.org/show/430805/ | 11:19 |
exploreshaifali | I am sure credentials are correct, this is openrc file http://paste.openstack.org/show/430810/ | 11:21 |
flwang | exploreshaifali: are you using keystone auth? | 11:28 |
exploreshaifali | flwang, yes | 11:28 |
flwang | exploreshaifali: does it work if just using rest api instead of client? | 11:28 |
exploreshaifali | flwang, with curl way, I am able to create queue | 11:31 |
exploreshaifali | but listing queue is not working | 11:31 |
flaper87 | that's weird | 11:32 |
flwang | can you show me your conf | 11:32 |
* flaper87 will let flwang handle this weirdness | 11:32 | |
* flaper87 just found another bug | 11:32 | |
flaper87 | ... in the client library | 11:32 |
exploreshaifali | flaper87, do you want to see zaqar.conf? | 11:32 |
flaper87 | flwang: ^ | 11:32 |
flwang | exploreshaifali: yes | 11:33 |
exploreshaifali | oh sorry | 11:33 |
flaper87 | exploreshaifali: I believe he'll need zaqar.conf and your keystonerc file | 11:33 |
flwang | it would be nice | 11:33 |
flaper87 | erm, openrc | 11:33 |
flaper87 | :D | 11:33 |
exploreshaifali | yea | 11:33 |
exploreshaifali | flwang, zaqar.conf http://paste.openstack.org/show/430814/ | 11:35 |
exploreshaifali | openrc http://paste.openstack.org/show/430810/ | 11:35 |
*** Qiming has joined #openstack-zaqar | 11:37 | |
flaper87 | https://bugs.launchpad.net/bugs/+bugs?field.tag=zaqar-liberty-test-day <- so far | 11:37 |
flwang | exploreshaifali: so you're using websocket | 11:38 |
flwang | does it work with wsgi? | 11:38 |
exploreshaifali | flwang, I didn't configured zaqar.conf file at all, it all came wtih devstack plugin | 11:39 |
exploreshaifali | that mean, haven't tested with wisgi | 11:39 |
flwang | oh? we're setting websocket as default transport protocol? | 11:40 |
flwang | flaper87: ^ | 11:40 |
flaper87 | flwang: not as the default transport, AFAIK | 11:40 |
flaper87 | it's just configured | 11:40 |
flaper87 | but not the default | 11:41 |
flaper87 | therve: ^ | 11:41 |
flaper87 | you changed that lately | 11:41 |
flaper87 | :D | 11:41 |
exploreshaifali | yes its therve's fault | 11:41 |
exploreshaifali | :P | 11:41 |
exploreshaifali | haha | 11:41 |
flaper87 | LOL | 11:42 |
flaper87 | brb | 11:42 |
flwang | exploreshaifali: then can you try wsgi, it would be more stable | 11:42 |
exploreshaifali | ok so now if I will modify conf file to not use websocket, I need to replace wisgi instead of websocket? | 11:42 |
exploreshaifali | flwang, ^ | 11:42 |
exploreshaifali | of just remove websocket | 11:42 |
flwang | exploreshaifali: just use this line transport = wsgi | 11:43 |
flwang | to replace transport = websocket | 11:43 |
exploreshaifali | ok ok | 11:43 |
exploreshaifali | got it! | 11:43 |
exploreshaifali | flwang, Thanks :) | 11:43 |
exploreshaifali | flwang, after updating do I need to run stack.sh again? to let environment read the changes? | 11:47 |
flwang | exploreshaifali: you just need to restart the zaqar service | 11:48 |
exploreshaifali | or there is some other way ? | 11:48 |
exploreshaifali | ok | 11:48 |
flwang | not necessary restack | 11:48 |
flaper87 | flwang: vkmc ryansb we're currently sending back the `links` information on every request regardless there are items being returned or not in the listing operation | 11:54 |
flaper87 | does that sound reasonable to you? | 11:54 |
ryansb | can you paste the body? | 11:55 |
flaper87 | it doesn't sit very well with me because there's not reall a "next" page | 11:55 |
flaper87 | ryansb: sure | 11:55 |
flwang | flaper87: if there is no item, we can/should just return [] or {} | 11:55 |
flaper87 | ryansb: http://paste.openstack.org/show/430842/ | 11:55 |
flaper87 | flwang: my feeling exactly | 11:56 |
flaper87 | ryansb: you're in API-WG, shed some light | 11:56 |
flaper87 | :D | 11:56 |
flwang | and the marker=230 looks weird | 11:56 |
flaper87 | flwang: ignore it, it's just from my current env | 11:56 |
flaper87 | :D | 11:56 |
flwang | flaper87: hah | 11:56 |
ryansb | yeah, so the general case is that you shouldn't elide top level keys | 11:57 |
ryansb | for some transient cases (e.g. empty queues) | 11:57 |
flaper87 | ryansb: does 'links': [] sound reasonable ? | 11:57 |
ryansb | yes, that makes sense | 11:57 |
flwang | flaper87: ryansb: btw, can you guys revisit the email patch https://review.openstack.org/#/c/202374/11 ? | 11:57 |
flaper87 | or is that what you meant with not eliding it ? | 11:57 |
flaper87 | flwang: will do | 11:57 |
ryansb | so {messages: [], links: []} | 11:57 |
ryansb | I meant "don't just do {messages: []}H | 11:58 |
flaper87 | ok, cool | 11:58 |
flaper87 | then we know were to fix this | 11:58 |
ryansb | so having the list be empty gets a +1 from me | 11:58 |
ryansb | flaper87: sure thing | 11:58 |
ryansb | err, I meant flwang | 12:00 |
ryansb | premature tab completion | 12:00 |
flwang | :-D | 12:00 |
flaper87 | info here: https://bugs.launchpad.net/zaqar/+bug/1489849 | 12:00 |
openstack | Launchpad bug 1489849 in zaqar "Stream iterator goes to an infinity loop" [High,New] - Assigned to Flavio Percoco (flaper87) | 12:00 |
flaper87 | remember, the tag is zaqar-liberty-test-day | 12:01 |
flaper87 | mmh, I guess we should consider that a breaking change | 12:03 |
flaper87 | actually, not | 12:03 |
flaper87 | nvm | 12:04 |
openstackgerrit | Flavio Percoco proposed openstack/zaqar: WIP: Return empty `links` on empty collections https://review.openstack.org/218235 | 12:12 |
*** dynarro has joined #openstack-zaqar | 12:32 | |
*** exploreshaifali has quit IRC | 12:32 | |
vkmc | hey | 12:35 |
*** exploreshaifali has joined #openstack-zaqar | 12:37 | |
flaper87 | vkmc: MORNING! | 12:38 |
*** exploreshaifali has quit IRC | 12:40 | |
*** exploreshaifali has joined #openstack-zaqar | 12:41 | |
*** exploreshaifali has quit IRC | 12:41 | |
openstackgerrit | Flavio Percoco proposed openstack/python-zaqarclient: WIP: v1.1 and v2 claims return document not list https://review.openstack.org/218248 | 12:43 |
openstackgerrit | Flavio Percoco proposed openstack/python-zaqarclient: WIP: Send claims `limit` as a query param https://review.openstack.org/218249 | 12:43 |
flaper87 | vkmc: FYI https://bugs.launchpad.net/bugs/+bugs?field.tag=zaqar-liberty-test-day | 12:43 |
flaper87 | vkmc: are you joining the test sessions? | 12:45 |
flaper87 | :D | 12:45 |
vkmc | flaper87, sure! | 12:46 |
flaper87 | vkmc: liar | 12:46 |
* flaper87 hides | 12:46 | |
vkmc | is there some guidelines or something? | 12:46 |
vkmc | I was reading the backlog | 12:46 |
vkmc | s/is/are | 12:46 |
flaper87 | vkmc: https://wiki.openstack.org/wiki/Zaqar/TestingDay/Liberty I started that yesterday, I'm completing it as we speak | 12:46 |
flaper87 | and I'm working on a zaqarclient script that should hit most of the APIs | 12:47 |
flaper87 | vkmc: if you want, you can take mongodb+mongodb | 12:47 |
flaper87 | I've been testing mongodb+sqlalchemy | 12:47 |
vkmc | oki | 12:47 |
vkmc | :) | 12:47 |
flaper87 | vkmc: you'll need all these patches to test things properly: https://bugs.launchpad.net/bugs/+bugs?field.tag=zaqar-liberty-test-day :( | 12:48 |
flaper87 | or at least be aware of those bugs | 12:48 |
flaper87 | vkmc: if you find bugs, tag them as zaqar-liberty-test-day | 12:48 |
vkmc | btw, about the links thing... unnecessary payload, I'd say remove it | 12:48 |
vkmc | oki | 12:48 |
vkmc | flaper87, dunno if dynarro told you, but the functional tests for python-zaqarclient are failing in master | 12:48 |
vkmc | which lead me to the question: when was the last time we run those and worked? | 12:49 |
*** Caleb-- has quit IRC | 12:49 | |
flaper87 | vkmc: TBH, not long ago. I had fixed those tests when Liberty started | 12:49 |
flaper87 | no idea why they are broken, honestly | 12:50 |
vkmc | flaper87, I found two things that maybe I can check | 12:50 |
flaper87 | vkmc: that'd be a good testday task to take *hint* | 12:50 |
vkmc | yes | 12:50 |
vkmc | :D | 12:50 |
dynarro | flaper87, it's your fault | 12:50 |
flaper87 | vkmc: if you can get those tests to pass or at least identify the issues, that'd be awesome | 12:50 |
vkmc | k | 12:50 |
flaper87 | dynarro: it's my fault it had worked, it's your fault it's broken | 12:50 |
flaper87 | :D | 12:51 |
dynarro | flaper87, whatever happens it's always your fault | 12:51 |
dynarro | just accept it | 12:51 |
dynarro | =D | 12:51 |
* flaper87 redirects dynarro to vkmc | 12:52 | |
* flaper87 sets up ipfaultstable | 12:52 | |
vkmc | flaper87, you heard her, its your fault | 12:52 |
* flaper87 walks away heads down | 12:52 | |
* dynarro gives a high five to vkmc | 12:53 | |
* vkmc highfives back dynarro | 12:53 | |
* vkmc gives a gummybear to flaper87 | 12:54 | |
vkmc | we like you anyway | 12:54 |
* dynarro gives a cookie to flaper87 | 12:54 | |
* flaper87 highfives a million angels and ignores dynarro and vkmc | 12:55 | |
dynarro | hahaha | 12:55 |
vkmc | lol | 12:55 |
openstackgerrit | Doraly Navarro proposed openstack/python-zaqarclient: Added v1 folder https://review.openstack.org/216159 | 12:57 |
flwang | flaper87: vkmc: ryansb: does this make sense for you https://bugs.launchpad.net/zaqar/+bug/1489879 ? | 13:05 |
openstack | Launchpad bug 1489879 in zaqar "Got 500 error when addding new mongodb pool" [Undecided,New] - Assigned to Fei Long Wang (flwang) | 13:05 |
*** Caleb-- has joined #openstack-zaqar | 13:06 | |
flaper87 | flwang: mmh, wait, are you adding it with a different pool name ? | 13:06 |
flwang | yes | 13:06 |
flwang | i'm going to have two pools, one for redis, one for mongodb, can it be supported? | 13:07 |
flwang | at least, i didn't see any doc said we can't support that | 13:07 |
flaper87 | flwang: yes but it's important to set a different group name for each | 13:08 |
flaper87 | mongodb and redis don't share the same capabilities | 13:08 |
flwang | flaper87: ok, i will try give them different group name | 13:08 |
*** Qiming has quit IRC | 13:08 | |
flwang | flaper87: works now | 13:09 |
*** sriram has joined #openstack-zaqar | 13:09 | |
flwang | flaper87: do we have a corner to document this? | 13:10 |
ryansb | better would be to make that error message more helpful | 13:12 |
flaper87 | ryansb: +1 to make the error more helpful | 13:13 |
flaper87 | and flwang +1 for documenting this | 13:13 |
flaper87 | we currently don't have docs for pools | 13:13 |
flaper87 | :( | 13:13 |
flaper87 | in fact, this just happened: https://twitter.com/sigmavirus24/status/637247547984056320 | 13:14 |
flaper87 | :/ | 13:14 |
flwang | flaper87: ok, i will restore the bug to make the error message more clear | 13:14 |
flaper87 | flwang: awesome | 13:15 |
flaper87 | We'll do a docs sprint before the end of the cycle | 13:15 |
flwang | flaper87: that would be great | 13:16 |
flwang | flaper87: btw, as for the policy feature, the code is ready and work, now i'm just struggling with the unit test | 13:16 |
flaper87 | flwang: awesome, I was looking at it the other day | 13:17 |
flaper87 | this script is helping me find so many bugs | 13:18 |
flaper87 | I'll upload it in a bit | 13:18 |
flwang | flaper87: i need to have a sleep | 13:21 |
flwang | maybe back after several hours | 13:22 |
flaper87 | flwang: sure thing. Thanks a lot flwang | 13:22 |
flaper87 | sleep well and thanks for all the great help | 13:22 |
flwang | flaper87: that's what i should do :) | 13:23 |
flwang | ttyl | 13:23 |
flwang | vkmc: ryansb: have fun, let's rock on | 13:23 |
flaper87 | nnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnice, sqlalchemy doesn't have support for flavors | 13:25 |
flaper87 | this is a big hole | 13:25 |
ryansb | sounds like someone found a bug | 13:26 |
flaper87 | "# NOTE(flaper87): Needed to avoid `abc` errors" | 13:27 |
flaper87 | my comment in that method is even worse | 13:27 |
flaper87 | :P | 13:27 |
flaper87 | oh boy | 13:27 |
flaper87 | I guess I'll have to fix it because I wrote that comment | 13:28 |
flaper87 | :( | 13:28 |
flaper87 | should be simple enough | 13:28 |
flaper87 | I guess | 13:28 |
ryansb | lol | 13:28 |
vkmc | haha | 13:33 |
flaper87 | no idea why we have a `project` field in flavors, TBH | 13:37 |
ryansb | ¯\_ツ_/¯ | 13:52 |
flaper87 | ryansb: do you happen to know what error sqlalchemy raises when a required ForeignKey is not provided ? | 13:55 |
flaper87 | sorry, when the ForeignKey provided doesn't exist | 13:55 |
flaper87 | is it NoReferenceError ? | 13:55 |
ryansb | not offhand, no | 13:55 |
* flaper87 tries that out | 13:55 | |
*** amitgandhinz has joined #openstack-zaqar | 14:06 | |
*** exploreshaifali has joined #openstack-zaqar | 14:08 | |
*** malini has joined #openstack-zaqar | 14:09 | |
*** Qiming has joined #openstack-zaqar | 14:15 | |
ryansb | hey, can someone else +2A this patch https://review.openstack.org/#/c/135991/ ? | 14:20 |
ryansb | I want to get proposal bot off our backs :) | 14:20 |
flaper87 | and I'm happy to say that flavors for sqla have been implemented | 14:21 |
flaper87 | w/o tests, OBVIOUSLY | 14:21 |
vkmc | ryansb, done | 14:22 |
ryansb | cool, thanks | 14:22 |
ryansb | flaper87: w/o tests :( | 14:22 |
flaper87 | ryansb: ungrateful | 14:22 |
flaper87 | >.> | 14:23 |
flaper87 | :P | 14:23 |
flaper87 | I'm marking all my patches as WIP | 14:23 |
ryansb | yeah, saw that | 14:23 |
flaper87 | because they are lacking tests | 14:23 |
openstackgerrit | Flavio Percoco proposed openstack/zaqar: WIP: Add support for flavors in sqlalchemy https://review.openstack.org/218291 | 14:26 |
openstackgerrit | Flavio Percoco proposed openstack/zaqar: WIP: s/update/_update/ sqlalchemy's pool controller https://review.openstack.org/218292 | 14:26 |
flaper87 | w0000h000 | 14:26 |
flaper87 | ah and expect pep8 to complain | 14:26 |
flaper87 | :D | 14:26 |
flaper87 | just trying to make sure everyone can get those patches and move on with the testing session | 14:27 |
flaper87 | is anyone testing zaqar? | 14:27 |
flaper87 | or is it just me? | 14:27 |
ryansb | I'm working on my swift patch - keystone is being weird | 14:28 |
ryansb | so I'm testing my unreleased changes to zaqar | 14:28 |
flaper87 | ryansb: sounds good! w000h00000 | 14:28 |
ryansb | so it's sort of like being in a test day | 14:29 |
openstackgerrit | chandan kumar proposed openstack/zaqar: Removing unused dependency: discover https://review.openstack.org/218297 | 14:30 |
openstackgerrit | Merged openstack/python-zaqarclient: Updated from global requirements https://review.openstack.org/135991 | 14:31 |
* flaper87 takes a break | 14:33 | |
*** exploreshaifali has quit IRC | 14:39 | |
*** boris-42 has joined #openstack-zaqar | 15:13 | |
*** csoukup has joined #openstack-zaqar | 15:23 | |
*** Qiming has quit IRC | 15:25 | |
openstackgerrit | Merged openstack/python-zaqarclient: Implement CLI support for flavor https://review.openstack.org/212467 | 15:32 |
* ryansb goes on review spree | 15:37 | |
*** malini has quit IRC | 15:38 | |
* flaper87 will go through all the patches and add tests to them | 15:40 | |
flaper87 | vkmc: https://review.openstack.org/#/c/218173/ | 15:40 |
flaper87 | vkmc: https://review.openstack.org/#/c/218169/1 | 15:41 |
*** malini has joined #openstack-zaqar | 15:45 | |
*** malini has quit IRC | 15:50 | |
*** malini has joined #openstack-zaqar | 15:51 | |
openstackgerrit | Merged openstack/python-zaqarclient: Make v1.1 the default CLI version https://review.openstack.org/218169 | 15:51 |
openstackgerrit | Merged openstack/python-zaqarclient: Make sure the API version is passed down https://review.openstack.org/218173 | 15:56 |
*** malini has quit IRC | 16:13 | |
*** mpanetta has joined #openstack-zaqar | 16:16 | |
*** kgriffs is now known as kgriffs|afk | 16:23 | |
*** exploreshaifali has joined #openstack-zaqar | 16:34 | |
*** amitgandhinz has quit IRC | 16:46 | |
exploreshaifali | flaper87, around? | 17:00 |
*** dynarro has quit IRC | 17:04 | |
*** dynarro has joined #openstack-zaqar | 17:04 | |
*** malini has joined #openstack-zaqar | 17:06 | |
*** amitgandhinz has joined #openstack-zaqar | 17:07 | |
*** dynarro has quit IRC | 17:09 | |
*** ametts has joined #openstack-zaqar | 17:09 | |
*** kgriffs|afk is now known as kgriffs | 17:18 | |
*** achanda has joined #openstack-zaqar | 17:56 | |
*** exploreshaifali has quit IRC | 18:21 | |
*** stanchan has joined #openstack-zaqar | 18:22 | |
*** achanda has quit IRC | 18:44 | |
*** exploreshaifali has joined #openstack-zaqar | 18:47 | |
*** achanda has joined #openstack-zaqar | 19:11 | |
*** ametts has quit IRC | 19:19 | |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/zaqar: Refactoring to make Websocket driver available in v2 https://review.openstack.org/218430 | 19:25 |
flaper87 | vkmc: ^ <3 | 19:26 |
flaper87 | exploreshaifali: now I am | 19:26 |
flaper87 | sorta | 19:26 |
vkmc | kinda | 19:26 |
vkmc | beer o'clock! | 19:26 |
flaper87 | maybe | 19:26 |
flaper87 | wine:30 | 19:27 |
*** malini has quit IRC | 19:28 | |
exploreshaifali | flaper87, do you remember what you wanted to suggest here https://review.openstack.org/#/c/202248/2/zaqarclient/queues/v1/cli.py | 19:28 |
exploreshaifali | I can remove if check | 19:28 |
exploreshaifali | that okay | 19:28 |
exploreshaifali | but than you said "create the flavor instance again to get the latest data from the server and use that." | 19:28 |
exploreshaifali | does that mean "client.flavor(args, auto_create=False)" is not creating flavor instance? | 19:29 |
exploreshaifali | flaper87, ^ | 19:29 |
flaper87 | exploreshaifali: that was a random. just remove those 2 lines and I believe you need auto_create=True | 19:30 |
exploreshaifali | yes | 19:30 |
exploreshaifali | I agree with this | 19:30 |
exploreshaifali | Thanks flaper87 :) | 19:30 |
flaper87 | exploreshaifali: thank you :) | 19:31 |
exploreshaifali | ;) | 19:31 |
*** malini has joined #openstack-zaqar | 19:32 | |
*** ametts has joined #openstack-zaqar | 19:32 | |
openstackgerrit | Shaifali Agrawal proposed openstack/python-zaqarclient: Implement CLI support for flavor https://review.openstack.org/202248 | 19:33 |
*** malini has quit IRC | 19:37 | |
exploreshaifali | btw, vkmc flaper87 my GSoC result is out! I am PASSED! | 19:38 |
flaper87 | exploreshaifali: AWESOME! | 19:38 |
vkmc | wohoooooooooooooo | 19:38 |
exploreshaifali | flaper87, wait I am doing making it true | 19:38 |
flaper87 | congratulations, exploreshaifali! | 19:38 |
flaper87 | that's so exciting | 19:38 |
vkmc | congrats Shaifali! | 19:38 |
exploreshaifali | Thanks!!! | 19:38 |
exploreshaifali | :D | 19:39 |
exploreshaifali | also flaper87 you don't know I will be joining you people in coming openstack summit | 19:39 |
exploreshaifali | :) | 19:39 |
flaper87 | wuuuuuuuuuuuuuuuuuuuuuut ? | 19:40 |
flaper87 | that's even better, we'll get to celebrate there :D | 19:40 |
sriram | congrats exploreshaifali! | 19:40 |
exploreshaifali | Thanks sriram :) | 19:40 |
exploreshaifali | flaper87, yes yes we will! | 19:40 |
*** achanda has quit IRC | 19:43 | |
*** kgriffs is now known as kgriffs|afk | 19:45 | |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/zaqar: Refactoring to make Websocket driver available in v2 https://review.openstack.org/218430 | 19:47 |
*** achanda has joined #openstack-zaqar | 19:47 | |
openstackgerrit | Shaifali Agrawal proposed openstack/python-zaqarclient: Implement CLI support for flavor https://review.openstack.org/202248 | 19:48 |
exploreshaifali | flaper87, ^ go ahead with this! | 19:48 |
* flaper87 clicks | 19:49 | |
flaper87 | vkmc: ^ | 19:49 |
exploreshaifali | flaper87, does that mean in v2 users will not able to set capabilities of flavor? | 19:54 |
flaper87 | exploreshaifali: no because those are introspected from the pool | 19:58 |
flaper87 | which is why we made it a read-only field | 19:58 |
exploreshaifali | ok! | 19:58 |
exploreshaifali | Does a pool can have only one queue and a queue can be registered to only one pool? | 19:58 |
exploreshaifali | flaper87, ^ | 19:58 |
openstackgerrit | Shaifali Agrawal proposed openstack/python-zaqarclient: Implement CLI support for flavor https://review.openstack.org/202248 | 20:00 |
flaper87 | exploreshaifali: a queue can be registered to only one pool but a pool can have many queues | 20:01 |
exploreshaifali | ok so still queues are there, its just that they are not necessary to post a message | 20:03 |
exploreshaifali | right? | 20:03 |
exploreshaifali | one can post a message without queue with having a topic inside metadata of message | 20:03 |
flaper87 | exploreshaifali: right, although the topic is not part of the metadata but the url | 20:04 |
*** kgriffs|afk is now known as kgriffs | 20:06 | |
exploreshaifali | so now if I am posting a message with queue, url is somehting like - "http://178.62.30.120:8888/v1/queues/<queue name>/messages" | 20:07 |
exploreshaifali | but without queue, it will be "http://178.62.30.120:8888/v1/<topic>/messages" | 20:08 |
exploreshaifali | flaper87, ^ | 20:08 |
flaper87 | exploreshaifali: no, the url doesn't change. It'll still be http://178.62.30.120:8888/v1/queues/<queue name>/messages | 20:12 |
flaper87 | the queue will still be created under-the-hood | 20:12 |
flaper87 | the thing is that it's not behaving as a queue | 20:12 |
exploreshaifali | flaper87, so since queue is not ceated, what i will put in place of <queue name>? | 20:18 |
exploreshaifali | topic? | 20:18 |
exploreshaifali | yes of course | 20:18 |
flaper87 | :) | 20:22 |
exploreshaifali | flaper87, so here a topic will belong to a pool? | 20:24 |
exploreshaifali | sorry if I am irritating you today | 20:24 |
exploreshaifali | :P | 20:24 |
exploreshaifali | s/sorry/slap me | 20:25 |
exploreshaifali | :D | 20:25 |
flaper87 | exploreshaifali: no irritating at all | 20:26 |
flaper87 | exploreshaifali: yes, a topic can live just in 1 pool | 20:26 |
exploreshaifali | ok ok :) | 20:26 |
exploreshaifali | Thanks! | 20:27 |
*** exploreshaifali has quit IRC | 20:28 | |
*** achanda has quit IRC | 20:28 | |
openstackgerrit | Merged openstack/zaqar: Fix typos in test_suite.rst and test_utils.py https://review.openstack.org/216930 | 20:31 |
openstackgerrit | Merged openstack/zaqar: The link for plugin samples is updated https://review.openstack.org/216617 | 20:32 |
openstackgerrit | Merged openstack/zaqar: Removing unused dependency: discover https://review.openstack.org/218297 | 20:32 |
*** sriram has quit IRC | 21:11 | |
*** ametts has quit IRC | 21:23 | |
*** amitgandhinz has quit IRC | 21:40 | |
*** amitgandhinz has joined #openstack-zaqar | 21:41 | |
*** amitgandhinz has quit IRC | 21:46 | |
*** csoukup has quit IRC | 21:48 | |
*** achanda has joined #openstack-zaqar | 21:48 | |
*** mpanetta has quit IRC | 21:55 | |
*** flwang2 has joined #openstack-zaqar | 22:10 | |
*** flwang has quit IRC | 22:13 | |
*** flwang2 has quit IRC | 22:45 | |
*** david-lyle has quit IRC | 23:24 | |
*** kgriffs is now known as kgriffs|afk | 23:25 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!