*** tej has joined #openstack-zaqar | 00:00 | |
*** tej has quit IRC | 00:00 | |
*** tej has joined #openstack-zaqar | 00:03 | |
*** shakamunyi has joined #openstack-zaqar | 00:09 | |
*** mpanetta has quit IRC | 00:20 | |
flwang | flaper87: ryansb: vkmc: can you hep review this one https://review.openstack.org/#/c/250125/ ? | 00:30 |
---|---|---|
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix queue create failure when metadata is {} https://review.openstack.org/247867 | 00:35 |
*** tej has quit IRC | 00:37 | |
*** rcernin has quit IRC | 00:37 | |
*** kui has joined #openstack-zaqar | 00:45 | |
*** malini has joined #openstack-zaqar | 00:48 | |
*** kgriffs is now known as kgriffs|afk | 00:51 | |
*** kui has quit IRC | 00:54 | |
*** shakamunyi has quit IRC | 01:02 | |
*** shakamunyi has joined #openstack-zaqar | 01:03 | |
*** csoukup has joined #openstack-zaqar | 01:18 | |
*** eliqiao has joined #openstack-zaqar | 01:22 | |
*** malini has quit IRC | 01:25 | |
*** eliqiao has quit IRC | 01:51 | |
*** eliqiao has joined #openstack-zaqar | 01:51 | |
*** lei-zh1 has joined #openstack-zaqar | 01:59 | |
*** dmowrer has joined #openstack-zaqar | 02:00 | |
*** Qiming has joined #openstack-zaqar | 02:03 | |
*** dmowrer has quit IRC | 02:04 | |
*** kui has joined #openstack-zaqar | 02:35 | |
*** malini has joined #openstack-zaqar | 02:50 | |
*** malini has quit IRC | 03:05 | |
mdnadeem | Eva-i: ping | 03:20 |
flwang | mdnadeem: anything i can help? | 03:24 |
Eva-i | mdnadeem: hello | 03:25 |
Eva-i | flwang: congratulations with fixing functional test! =) | 03:25 |
flwang | Eva-i: YEAH! | 03:25 |
flwang | i can't make it without your help | 03:26 |
flwang | Eva-i: thank you for your greeeat catch ;) | 03:26 |
mdnadeem | Eva-i: While reviewing your patch https://review.openstack.org/#/c/251023/, It fail because of Authorization Failed: Cannot authenticate without an auth_url | 03:26 |
Eva-i | flwang: no, it's not great, but thanks | 03:26 |
flwang | Eva-i: btw, are you working on the user guide spec? | 03:26 |
flwang | Eva-i: our config ref spec has been merged YEAH! | 03:27 |
Eva-i | flwang: I wanted to discuss it with you when you will be less busy | 03:27 |
flwang | Eva-i: maybe tomorrow :) | 03:27 |
Eva-i | mdnadeem: ok, let me think | 03:27 |
Eva-i | flwang: alright | 03:27 |
flwang | Eva-i: or, just talk now, i have 30 mins before i leave, but i may reply slow, sorry | 03:28 |
Eva-i | flwang: alright, I'll try to make it quick | 03:29 |
mdnadeem | Eva-i: Now, zaqar client uses bydefault keystone, so only passing project name in config is not enough, so you need to _setup_auth_params, flwang patch may be help you out https://review.openstack.org/#/c/250125 | 03:29 |
Eva-i | flwang: when I asked where should zaqar user guide be hosted, you said in "http://docs.openstack.org/" | 03:30 |
Eva-i | flwang: but there are many types of user guides for different categories of people | 03:30 |
flwang | Eva-i: yes | 03:30 |
Eva-i | flwang: for example, there is " http://docs.openstack.org/admin-guide-cloud/ " | 03:31 |
flwang | Eva-i: since i'm the right guy to answer that questions :) | 03:31 |
Eva-i | flwang: and this: http://docs.openstack.org/user-guide/ | 03:31 |
Eva-i | flwang: so I shouldn't ask you and should ask guys in doc channel? | 03:32 |
mdnadeem | flwang, Eva-i, vkmc, ryansb : I am flying to my home town tomorrow , So i will not be available 1-2 day, See you guys on monday :) | 03:35 |
Eva-i | mdnadeem: so | 03:35 |
flwang | Eva-i: i would say both | 03:35 |
Eva-i | mdnadeem: I need to rebase my patch? | 03:35 |
flwang | Eva-i: generally, my idea comes from the manila project | 03:35 |
flwang | it's not in core and it's not a 'tc-approved' project | 03:35 |
flwang | so where you can see 'shared file systems' related doc, we would like see zaqar is around :) | 03:36 |
Eva-i | flwang: manila project doesn't have any docs in docs.openstack.org? | 03:36 |
flwang | http://docs.openstack.org/admin-guide-cloud/shared_file_systems.html | 03:36 |
flwang | for back to your above question, both user-guide and admin-guide-cloud are the goal we would like to work for | 03:37 |
flwang | but | 03:37 |
flwang | we don't have much bandwidth | 03:37 |
flwang | and personally, i think user-guide is a little bit easier | 03:38 |
flwang | but admin-guide-cloud is more important | 03:38 |
flwang | we have to balance :) | 03:38 |
Eva-i | flwang: oh, I see. So one type of doc manilla decided to host in docs.openstack.org | 03:39 |
Eva-i | flwang: the sources are here: https://github.com/openstack/openstack-manuals/tree/master/doc/admin-guide-cloud/source | 03:39 |
flwang | Eva-i: yes | 03:39 |
mdnadeem | Eva-i: you need to update your patch to set _setup_auth_param, using flwang patch as i mentioned above :) | 03:40 |
Eva-i | so the spec I need to implement for https://blueprints.launchpad.net/openstack-manuals/+spec/zaqar-user-guide is one that will allow Zaqar user guide to be hosted in http://docs.openstack.org/user-guide/ ? | 03:41 |
Eva-i | mdnadeem: okay, I'll do it, thanks =) | 03:41 |
mdnadeem | Eva-i: np :) | 03:41 |
flwang | Eva-i: yep, it should be easier and a good size for Mitaka | 03:42 |
flwang | Eva-i: i don't want to hold you 3 months only for docs, you can do more :) | 03:43 |
Eva-i | flwang: and docbook user guide will be converted to rst to be hosted in http://docs.openstack.org/user-guide/ ? | 03:43 |
flwang | i think so | 03:43 |
Eva-i | flwang: okay, thanks | 03:43 |
flwang | you may need to confirm with #openstack-doc | 03:44 |
flwang | Eva-i: thank you! | 03:44 |
Eva-i | flwang: also our team have different vision of what should Zaqar docs be in the near future | 03:44 |
*** lei-zh1 has quit IRC | 03:45 | |
flwang | Eva-i: what's that mean 'different visions'? | 03:45 |
flwang | we're a small team, we should focus | 03:46 |
Eva-i | flwang: one person want some docs to be hosted in one place, another person wants them to be hosted in another | 03:46 |
Eva-i | flwang: for example ^ | 03:47 |
Eva-i | flwang: I created "Doc places and statuses" section in https://etherpad.openstack.org/p/zaqar-mitaka-docs so we can discuss where they should be. | 03:47 |
Eva-i | flwang: okay, I have no questions now | 03:47 |
Eva-i | flwang: and yes, I'll visit guys in #openstack-doc | 03:48 |
Eva-i | flwang: thanks for today | 03:48 |
*** AAzza has quit IRC | 03:48 | |
*** AAzza has joined #openstack-zaqar | 03:49 | |
flwang | Eva-i: different opinions is not a problem, we can workout an agreement to avoid wasting your effort | 03:51 |
Eva-i | flwang: "i don't want to hold you 3 months only for docs, you can do more :)" I agree, docs are a bit boring thing =) | 03:51 |
flwang | haha | 03:51 |
flwang | i have to run | 03:52 |
Eva-i | bye | 03:52 |
flwang | Eva-i: thank you, ttyl | 03:52 |
*** flwang1 has quit IRC | 03:56 | |
*** lei-zh1 has joined #openstack-zaqar | 04:33 | |
*** kui has quit IRC | 04:36 | |
*** sonuk has joined #openstack-zaqar | 04:36 | |
*** mdnadeem has quit IRC | 05:10 | |
*** diga has joined #openstack-zaqar | 05:19 | |
*** kui has joined #openstack-zaqar | 05:23 | |
*** kui has quit IRC | 05:24 | |
*** csoukup has quit IRC | 05:46 | |
*** dmowrer has joined #openstack-zaqar | 06:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/zaqar: Imported Translations from Zanata https://review.openstack.org/250621 | 06:26 |
*** dmowrer has quit IRC | 06:27 | |
*** rcernin has joined #openstack-zaqar | 07:19 | |
*** rcernin_ has joined #openstack-zaqar | 07:24 | |
*** vkmc has quit IRC | 07:43 | |
*** vkmc has joined #openstack-zaqar | 07:44 | |
*** flaper87 has quit IRC | 07:44 | |
*** flaper87 has joined #openstack-zaqar | 07:49 | |
*** diga has quit IRC | 08:12 | |
*** dynarro has joined #openstack-zaqar | 08:33 | |
*** rcernin has quit IRC | 08:50 | |
*** dmowrer has joined #openstack-zaqar | 08:50 | |
*** rcernin_ has quit IRC | 08:51 | |
*** dmowrer has quit IRC | 08:54 | |
*** malini has joined #openstack-zaqar | 09:39 | |
*** malini has left #openstack-zaqar | 09:50 | |
*** rcernin has joined #openstack-zaqar | 09:59 | |
*** lei-zh1 has quit IRC | 10:33 | |
*** lei-zh1 has joined #openstack-zaqar | 10:34 | |
*** lei-zh1 has quit IRC | 10:37 | |
*** dynarro has quit IRC | 10:45 | |
*** Qiming has quit IRC | 10:58 | |
*** dmowrer has joined #openstack-zaqar | 11:33 | |
*** dmowrer has quit IRC | 11:37 | |
*** Qiming has joined #openstack-zaqar | 11:43 | |
*** dmowrer has joined #openstack-zaqar | 12:09 | |
*** sonuk has quit IRC | 12:16 | |
*** flaper87 has quit IRC | 12:17 | |
*** flaper87 has joined #openstack-zaqar | 12:17 | |
*** malini1 has joined #openstack-zaqar | 12:24 | |
*** openstackgerrit has quit IRC | 12:32 | |
*** openstackgerrit has joined #openstack-zaqar | 12:33 | |
*** vkmc has quit IRC | 12:50 | |
*** vkmc has joined #openstack-zaqar | 12:50 | |
*** malini1 has quit IRC | 13:03 | |
*** njohnston has joined #openstack-zaqar | 13:07 | |
*** njohnston has quit IRC | 13:11 | |
*** njohnston has joined #openstack-zaqar | 13:11 | |
*** njohnston has quit IRC | 13:12 | |
*** openstackgerrit has quit IRC | 13:17 | |
*** njohnston has joined #openstack-zaqar | 13:17 | |
*** openstackgerrit has joined #openstack-zaqar | 13:17 | |
*** bradjones has quit IRC | 13:27 | |
*** bradjones has joined #openstack-zaqar | 13:31 | |
*** bradjones has quit IRC | 13:31 | |
*** bradjones has joined #openstack-zaqar | 13:31 | |
*** tej has joined #openstack-zaqar | 13:40 | |
*** dmowrer_ has joined #openstack-zaqar | 13:42 | |
*** dmowrer has quit IRC | 13:42 | |
*** dmowrer_ has quit IRC | 13:47 | |
*** dmowrer has joined #openstack-zaqar | 13:49 | |
*** dmowrer has quit IRC | 13:54 | |
*** dmowrer has joined #openstack-zaqar | 13:57 | |
*** tej has quit IRC | 13:59 | |
*** dmowrer has quit IRC | 14:01 | |
*** csoukup has joined #openstack-zaqar | 14:09 | |
*** dmowrer has joined #openstack-zaqar | 14:14 | |
*** sriram has joined #openstack-zaqar | 14:15 | |
*** njohnston has quit IRC | 14:38 | |
*** njohnston has joined #openstack-zaqar | 14:38 | |
*** ametts has joined #openstack-zaqar | 14:39 | |
*** tej has joined #openstack-zaqar | 14:41 | |
*** malini has joined #openstack-zaqar | 14:44 | |
openstackgerrit | Ryan Brown proposed openstack/python-zaqarclient: Add cli support for flavor create v1 https://review.openstack.org/248508 | 14:52 |
*** Qiming has quit IRC | 15:02 | |
openstackgerrit | Merged openstack/python-zaqarclient: Using clouds.yaml to get the credentials https://review.openstack.org/250125 | 15:03 |
openstackgerrit | Ryan Brown proposed openstack/python-zaqarclient: Add cli support for flavor create v1 https://review.openstack.org/248508 | 15:05 |
openstackgerrit | Ryan Brown proposed openstack/python-zaqarclient: Flavor create support for v2 https://review.openstack.org/250248 | 15:08 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-zaqarclient: Updated from global requirements https://review.openstack.org/252999 | 15:08 |
*** mpanetta has joined #openstack-zaqar | 15:12 | |
*** dynarro has joined #openstack-zaqar | 15:19 | |
*** kgriffs|afk is now known as kgriffs | 15:30 | |
*** amitgandhinz has joined #openstack-zaqar | 15:37 | |
openstackgerrit | Merged openstack/python-zaqarclient: Updated from global requirements https://review.openstack.org/252999 | 16:09 |
*** rcernin has quit IRC | 16:12 | |
jasondotstar | flwang, flaper87: ping | 16:13 |
jasondotstar | did you guys see the email from Balaji ? | 16:13 |
ryansb | jasondotstar: was it on the ml/what subject? | 16:15 |
jasondotstar | well flwang was following up on an action item to provide me with a sample zaqar conf | 16:27 |
jasondotstar | ryansb: ^ | 16:27 |
jasondotstar | so we had a separate email thread that we sent to Balaji re: the zaqar config a rackspace | 16:30 |
jasondotstar | the reply was that they were still running marconi code :-/ | 16:34 |
jasondotstar | ryansb: ^^ | 16:34 |
jasondotstar | that's the gist essentially... | 16:34 |
*** tej has quit IRC | 16:51 | |
*** dmowrer has quit IRC | 16:51 | |
*** rcernin has joined #openstack-zaqar | 16:53 | |
*** kgriffs is now known as kgriffs|afk | 17:00 | |
*** dmowrer has joined #openstack-zaqar | 17:08 | |
*** tej has joined #openstack-zaqar | 17:08 | |
*** dmowrer has quit IRC | 17:09 | |
*** dmowrer has joined #openstack-zaqar | 17:09 | |
*** njohnsto_ has joined #openstack-zaqar | 17:23 | |
*** njohnston has quit IRC | 17:27 | |
*** flwang1 has joined #openstack-zaqar | 17:35 | |
*** dynarro has quit IRC | 17:38 | |
flwang1 | vkmc: ryansb: flaper87: anybody working on the dup auth section issue? | 17:40 |
ryansb | flwang1: I am, yeah | 17:40 |
flwang1 | i'm working on that, but i haven't work out a good solution | 17:40 |
ryansb | my first fix was, as you noted, out of line with global reqs | 17:40 |
ryansb | yeah, same. | 17:40 |
flwang1 | ryansb: yep | 17:40 |
flwang1 | we have to find another way :( | 17:41 |
flwang1 | let's work together | 17:41 |
flwang1 | guys, now it's our first priority | 17:42 |
ryansb | yeah, so I found that deleting line 62 of zaqar/transport/auth.py "fixes" it | 17:49 |
flwang1 | let me see | 17:49 |
ryansb | but I'm not sure how to validate that everything is still configured right | 17:49 |
ryansb | because it does pass the tests, and printing the `conf` seems to have the right bits | 17:49 |
*** amitgandhinz has quit IRC | 17:49 | |
flwang1 | ryansb: run zaqar and run functional test? | 17:49 |
ryansb | I haven't run the functionals against that yet, no | 17:49 |
flwang1 | seems it works | 17:51 |
flwang1 | i'm running functional test after removing the line | 17:51 |
ryansb | k, cool | 17:52 |
flwang1 | hmm...wait | 17:55 |
flwang1 | i added a breakpoint at line 62, but it didn't stopped :( | 17:55 |
flwang1 | i need to double confirm | 17:55 |
ryansb | ? that's odd | 17:55 |
ryansb | are you running with keystone? | 17:56 |
ryansb | or standalone | 17:56 |
flwang1 | yep, i'm running with keystone | 17:57 |
ryansb | hm | 17:58 |
ryansb | can you paste your config? | 17:58 |
openstackgerrit | Ryan Brown proposed openstack/python-zaqarclient: Add cli support for flavor show v1 https://review.openstack.org/248509 | 17:59 |
flwang1 | ryansb: sure, wait a se | 18:03 |
* ryansb rebasing intensifies | 18:03 | |
flwang1 | ryansb: did u get any comments from keystone team about the usage of keystoneclient.auth? | 18:03 |
ryansb | no | 18:03 |
flwang1 | paste is very slow | 18:04 |
flwang1 | ryansb: ok, i will talk with stevemar | 18:04 |
* ryansb hops into #openstack-keystone as well | 18:04 | |
*** malini has quit IRC | 18:07 | |
*** davideagnello has quit IRC | 18:26 | |
*** davideagnello has joined #openstack-zaqar | 18:26 | |
flwang1 | nobody care you and me, hah | 18:27 |
flwang1 | ryansb: i have to run to office, ttyl | 18:27 |
ryansb | k | 18:27 |
flwang1 | thanks | 18:27 |
*** flwang1 has quit IRC | 18:28 | |
*** malini has joined #openstack-zaqar | 18:38 | |
*** wanghao_ has joined #openstack-zaqar | 18:43 | |
*** wanghao has quit IRC | 18:44 | |
*** wanghao_ is now known as wanghao | 18:44 | |
*** malini has quit IRC | 18:49 | |
jasondotstar | hey flwang | 18:58 |
jasondotstar | did you see my msg earlier? | 18:58 |
*** tej has quit IRC | 19:00 | |
*** dmowrer has quit IRC | 19:00 | |
*** tej has joined #openstack-zaqar | 19:04 | |
*** dmowrer_ has joined #openstack-zaqar | 19:04 | |
*** kgriffs|afk is now known as kgriffs | 19:09 | |
*** tej_ has joined #openstack-zaqar | 19:26 | |
*** dmowrer_ has quit IRC | 19:31 | |
flwang | jasondotstar: no, would you mind repeating ? | 19:41 |
ryansb | flwang: "11:13:57 jasondotstar | did you guys see the email from Balaji ?" | 19:42 |
jasondotstar | there it is :) | 19:42 |
flwang | jasondotstar: ah, i saw it, but, i would still want to see their config and adjust it based on the new changes | 19:43 |
jasondotstar | ack. | 19:43 |
flwang | jasondotstar: i can reply that | 19:43 |
jasondotstar | +1 | 19:43 |
jasondotstar | get him to send it along anyway | 19:43 |
flwang | jasondotstar: yes | 19:43 |
*** davideagnello has quit IRC | 19:50 | |
flwang | jasondotstar: thanks for the heads up, i will reply the mail soon | 19:50 |
*** dmowrer has joined #openstack-zaqar | 19:50 | |
*** amitgandhinz has joined #openstack-zaqar | 19:50 | |
*** davideagnello has joined #openstack-zaqar | 19:50 | |
*** flwang1 has joined #openstack-zaqar | 19:53 | |
jasondotstar | cool np. | 19:55 |
flwang | ryansb: seems we have to upgrade our code | 19:57 |
ryansb | :( oh well | 19:58 |
* ryansb will start on that | 19:58 | |
ryansb | I see the session docs there | 19:58 |
flwang | ryansb: personally, i don't really want to upgrade to sessions atm | 20:00 |
flwang | if we have a choice | 20:00 |
ryansb | yeah, but I'm not sure we have one | 20:00 |
ryansb | I'd actually prefer to break globalreqs for the milestone | 20:01 |
ryansb | which is evil, but it seems *less* evil | 20:01 |
flwang | ryansb: we can't merge it, IIRC | 20:03 |
ryansb | oh. Well :( | 20:03 |
*** dmowrer has quit IRC | 20:03 | |
flwang | i mean jenkins won't allow us to do that | 20:03 |
*** dmowrer has joined #openstack-zaqar | 20:13 | |
*** dmowrer_ has joined #openstack-zaqar | 20:21 | |
*** dmowrer has quit IRC | 20:24 | |
*** dmowrer_ has quit IRC | 20:27 | |
*** dmowrer has joined #openstack-zaqar | 20:29 | |
*** dmowrer_ has joined #openstack-zaqar | 20:30 | |
*** dmowrer has quit IRC | 20:33 | |
flwang1 | ryansb: you said we're using a different type for auth_section option, right? | 20:34 |
*** amitgandhinz has quit IRC | 20:34 | |
ryansb | no, it's the different keystone libs (keystoneclient for us, keystoneauth1 for the middleware) | 20:34 |
flwang1 | ryansb: and we are using oslo_config's String, seems there is not way to change it, is it? | 20:34 |
flwang1 | ryansb: so the best way to fix it is use keystoneauth1 ? | 20:35 |
ryansb | it's ksclient that's using the options, we're just using keystoneclient which is doing the registering | 20:35 |
ryansb | That may work, I don't know what we depend on in ksclient | 20:35 |
openstackgerrit | Merged openstack/python-zaqarclient: Add cli support for flavor show v1 https://review.openstack.org/248509 | 20:36 |
*** dmowrer_ has quit IRC | 20:44 | |
flwang1 | ryansb: i tried to just comment the line 62 out and it works by manual test | 20:44 |
*** dmowrer has joined #openstack-zaqar | 20:44 | |
ryansb | oh? | 20:44 |
flwang | ryansb: let me throw a patch and see what's the response from gate | 20:45 |
ryansb | k | 20:45 |
flwang | we have functional test gate | 20:45 |
*** dmowrer_ has joined #openstack-zaqar | 20:45 | |
*** dmowrer has quit IRC | 20:47 | |
*** openstackgerrit has quit IRC | 20:47 | |
*** openstackgerrit has joined #openstack-zaqar | 20:47 | |
*** dmowrer has joined #openstack-zaqar | 20:49 | |
*** dmowrer has quit IRC | 20:50 | |
*** dmowrer_ has quit IRC | 20:50 | |
*** dmowrer has joined #openstack-zaqar | 20:50 | |
*** tej_ has quit IRC | 20:57 | |
*** tej has quit IRC | 20:57 | |
flwang | flaper87: http://logs.openstack.org/38/252638/1/check/gate-zaqar-python27/d862004/ | 20:59 |
flwang | it would be nice if you can take a look, since most of the code is written by you :) | 20:59 |
*** tej_ has joined #openstack-zaqar | 21:02 | |
*** tej has joined #openstack-zaqar | 21:02 | |
*** rcernin has quit IRC | 21:05 | |
*** ryansb has quit IRC | 21:09 | |
*** ryansb has joined #openstack-zaqar | 21:12 | |
*** ryansb has quit IRC | 21:12 | |
*** ryansb has joined #openstack-zaqar | 21:12 | |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Fix duplicate auth_section issue https://review.openstack.org/253220 | 21:19 |
flwang | Eva-i: around? | 21:20 |
flwang | vkmc: ? | 21:20 |
vkmc | flwang, hey! | 21:24 |
*** njohnsto_ has quit IRC | 21:25 | |
*** njohnston has joined #openstack-zaqar | 21:26 | |
flwang1 | vkmc: can you give this patch a try ? https://review.openstack.org/#/c/253220/ | 21:27 |
flwang1 | it works for me and pass the unit test | 21:27 |
flwang1 | but i would like there is a double confirm | 21:27 |
flwang1 | it's breaking our gate | 21:27 |
*** sonuk has joined #openstack-zaqar | 21:30 | |
ryansb | flwang1: I'm bringing up a fresh devstack to test it as well | 21:32 |
flwang | ryansb: awesome, many thanks | 21:33 |
flwang | ryansb: i'm testing it as well | 21:33 |
ryansb | cool deal | 21:33 |
*** malini has joined #openstack-zaqar | 21:40 | |
*** malini1 has joined #openstack-zaqar | 21:42 | |
*** malini has quit IRC | 21:44 | |
*** sriram has quit IRC | 21:48 | |
vkmc | sure, I'm on it | 21:52 |
*** tej has quit IRC | 21:55 | |
*** tej_ has quit IRC | 21:55 | |
*** dmowrer has quit IRC | 21:55 | |
*** malini1 has quit IRC | 21:57 | |
*** amitgandhinz has joined #openstack-zaqar | 22:09 | |
*** malini has joined #openstack-zaqar | 22:16 | |
*** njohnsto_ has joined #openstack-zaqar | 22:18 | |
*** ametts has quit IRC | 22:19 | |
*** amitgandhinz has quit IRC | 22:19 | |
*** njohnston has quit IRC | 22:21 | |
flwang | vkmc: ryansb: any feedback? at least it works for me | 22:23 |
ryansb | it's working for me too | 22:23 |
ryansb | I'm thinking it's good - I've tried from CLI, integration tests, and unittests | 22:23 |
flwang | ryansb: awesome | 22:24 |
ryansb | (unittests aren't too likely to actually fail though) | 22:24 |
ryansb | has the gate spoken? | 22:24 |
flwang | ryansb: not yet, it's slow | 22:25 |
flwang | as normal :D | 22:25 |
vkmc | working for me too | 22:26 |
vkmc | :) | 22:26 |
flwang | vkmc: coooooooooool | 22:26 |
ryansb | gimme a few more minutes, I'm going to try some curl tests | 22:26 |
flwang | could you pls vote the patch? if there is any other issue i need to fix, i may give it a nijia approve in case your guys left | 22:26 |
flwang | since today is the deadline of m-1 | 22:27 |
vkmc | I'll stay here | 22:27 |
ryansb | I'll be around | 22:27 |
ryansb | but yeah, right up on the deadline | 22:27 |
flwang | awesome, appreciate it | 22:28 |
flwang | after this one merged, we need to get https://review.openstack.org/252154 and https://review.openstack.org/252638 in | 22:28 |
*** dmowrer has joined #openstack-zaqar | 22:35 | |
ryansb | ok, +2'd | 22:39 |
*** dmowrer has quit IRC | 22:39 | |
ryansb | gate looks happy with it (redis still isn't done though) | 22:43 |
flwang | ryansb: can you paste me the zuul link? | 22:45 |
ryansb | http://zuul.openstack.org/ | 22:45 |
ryansb | then put "openstack/zaqar,openstack/python-zaqarclient" (without the quotes) in the filter box | 22:45 |
flwang | got it | 22:46 |
flwang | i have seen it | 22:46 |
flwang | very good | 22:46 |
flwang | vkmc: ^ | 22:46 |
ryansb | flwang: or vkmc if you +A https://review.openstack.org/#/c/253220/ now, it'll merge | 22:50 |
ryansb | then once it's in we'll just need to recheck https://review.openstack.org/#/c/252154/ | 22:50 |
ryansb | and https://review.openstack.org/#/c/252638/ | 22:50 |
flwang | ryansb: i'm going to approve it | 22:54 |
ryansb | cool | 22:54 |
Eva-i | hello | 22:54 |
flwang | Eva-i: hi | 22:56 |
Eva-i | I have some concerns about how this patch was implemented https://review.openstack.org/#/c/245526/ | 22:56 |
flwang | Eva-i: listening... | 22:57 |
flwang | vkmc: we did it at the same time :D | 22:58 |
flwang | vkmc: can you review https://review.openstack.org/#/c/252154/ ? | 22:58 |
Eva-i | maybe it would be better if this patch was changing default authentification backend only for CLI and not for both CLI and python API | 22:58 |
flwang | so that we just need recheck after 253220 merged | 22:59 |
flwang | Eva-i: why? | 22:59 |
Eva-i | flwang: there are two reasons | 23:11 |
Eva-i | 1. Zaqar server by default has noauth authentication and the client now has keystone by default. I think it's a bit strange. | 23:11 |
Eva-i | 2. It breaks compatibility with simple programs that were using no authentication, for example with zaqar-bench or code examples in the internet. Also 3 of 4 examples in "examples" directory in python-zaqarclient repo will need more refactoring (more fat "conf" passed as parameter for client.Client() instantiation) to make them work. They were not working even before this patch, but this patch adds problems. | 23:11 |
Eva-i | I'm sorry I haven't reviewed this patch before it was merged. | 23:11 |
flwang | personally, i don't think noauth is the direction of Zaqar | 23:12 |
flwang | Zaqar is designed as a messaging service targeting for large(huge) scale, not for small/simple user case | 23:13 |
flwang | besides, just like we said in the project description, it's a multi-tenancy messaging service, so i can't imagine how noauth can meet the multi-tenancy | 23:14 |
flwang | but i agree, we need to make the server and client be compatible | 23:14 |
flwang | that's my 2 cents | 23:14 |
flwang | brb, lunch | 23:15 |
*** sonuk has quit IRC | 23:15 | |
*** mpanetta has quit IRC | 23:16 | |
*** sonuk has joined #openstack-zaqar | 23:16 | |
*** Qiming has joined #openstack-zaqar | 23:21 | |
Eva-i | flwang: maybe Zaqar was designed like that to make new users get involved easily: from small code examples without complicated parameters, without thinking about complicated things. But of course maybe Zaqar just evolves. http://paste.openstack.org/show/b7Ynxf5pSHtlRFrgFLyW/ | 23:23 |
*** csoukup has quit IRC | 23:25 | |
ryansb | uh what | 23:29 |
ryansb | test fail with stack overflow? https://jenkins06.openstack.org/job/gate-zaqar-python34/189/console | 23:29 |
Eva-i | flwang: I still don't know how Zaqar handles compatibility. I don't know for example should be stable Kilo release be compatible with stable Liberty release. Or maybe I should look at Zaqar major version numbers, but I don't know where these numbers can be found. If I had a products programmed that rely on Zaqar I would be very cautious before updating Zaqar. | 23:33 |
Eva-i | *had products | 23:33 |
vkmc | Eva-i, the client instantiation has always been like what you call "now" in that paste | 23:38 |
vkmc | Eva-i, compatibility is handled considering versioning, not OpenStack releases | 23:39 |
vkmc | those numbers are the versions for the API | 23:39 |
Eva-i | vkmc: do you mean Zaqar server's API? | 23:40 |
vkmc | yeah | 23:41 |
Eva-i | vkmc: I think python-zaqarclient has API too | 23:42 |
Eva-i | vkmc: not web API, but python code API | 23:43 |
Eva-i | vkmc: does the client's API it has versioning too? | 23:43 |
vkmc | Eva-i, we are using the same versioning than the client | 23:44 |
vkmc | sorry, than the server | 23:44 |
*** tej_ has joined #openstack-zaqar | 23:49 | |
*** tej has joined #openstack-zaqar | 23:51 | |
*** Qiming has quit IRC | 23:52 | |
Eva-i | vkmc: I'm not telling it was not possible to instantiate client.Client() like "now", I'm telling it's not possible to instantiate client like "before". | 23:53 |
Eva-i | vkmc: so it can work | 23:53 |
vkmc | Eva-i, if the client was being instantiated like that in the bench, then the bench was wrong | 23:54 |
*** malini has quit IRC | 23:55 | |
Eva-i | vkmc: and "examples" directory in python-zaqarclient repo were wrong too? | 23:57 |
Eva-i | vkmc: I believe they were working nicely | 23:57 |
vkmc | Eva-i, I'd say so, I remember commenting that some time ago | 23:57 |
vkmc | whatever the case, we should move on and focus in those aspects the next time | 23:58 |
vkmc | make sure everything keeps compatibility | 23:58 |
vkmc | and make sure we are clear wrt versioning | 23:58 |
ryansb | hey folks, it's EOD here, but I'll check back in in ~1hr to see where the gate is at on https://review.openstack.org/#/c/253220/ | 23:59 |
* ryansb heads to dinner | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!