Eva-i | flwang1: I like this https://review.openstack.org/#/c/284555/ spec now. Though you have proposed to include also "region" inside confirmation message from Zaqar, but wanghao didn't included it. Is it okay for you? | 00:00 |
---|---|---|
Eva-i | *include | 00:00 |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Add Zaqar installation guide https://review.openstack.org/268851 | 00:14 |
*** shu-mutou has joined #openstack-zaqar | 00:15 | |
openstackgerrit | Eva Balycheva proposed openstack/zaqar: Add Zaqar installation guide https://review.openstack.org/268851 | 00:16 |
*** rcernin has joined #openstack-zaqar | 00:17 | |
wanghao | Eva-i, flwang1: hi, guys, about the 'region', since we can't get it from token, unless we get it from keystone. | 00:17 |
wanghao | Eva-i, flwang1: So I'm thinking do we need it? | 00:18 |
wanghao | Eva-i, flwang1: 'region' in keystone is related to endpoint, I didn't find the relationship between region and project. | 00:20 |
wanghao | Eva-i: About the modifying devstack plugion, did you mean to modify Zaqar's plugion to let devstack can host the web page, didn't it? | 00:21 |
*** rcernin has quit IRC | 00:22 | |
wanghao | Eva-i: So what I mean in my comment is I think we can let users to host the page by themselves. | 00:24 |
*** tqtran has quit IRC | 00:33 | |
*** rcernin has joined #openstack-zaqar | 00:48 | |
*** rcernin has quit IRC | 00:53 | |
*** rcernin has joined #openstack-zaqar | 01:05 | |
openstackgerrit | Merged openstack/zaqar-ui: Imported Translations from Zanata https://review.openstack.org/328886 | 01:09 |
*** rcernin has quit IRC | 01:10 | |
openstackgerrit | wangxiyuan proposed openstack/zaqar: [WIP]Subscription Confirmation Support for MongoDB https://review.openstack.org/316601 | 01:28 |
*** tqtran has joined #openstack-zaqar | 01:30 | |
*** tqtran has quit IRC | 01:34 | |
Eva-i | wanghao: hello | 01:53 |
flwang1 | Eva-i: ping | 01:54 |
Eva-i | wanghao: I'm not sure about region, that's why I decided to ask flwang | 01:54 |
Eva-i | flwang1: pong | 01:54 |
Eva-i | wanghao: "did you mean to modify Zaqar's plugion to let devstack can host the web page, didn't it" yes, I mean this. But devstack is not for users, it's for us, developers, to test things. It just will be convenient to have devstack setup hosting confirmation web page for us. | 01:56 |
flwang1 | Eva-i: pls revisit https://review.openstack.org/#/c/323059/ | 01:56 |
flwang1 | as for the 'region' | 01:56 |
Eva-i | flwang1: oki | 01:56 |
flwang1 | it's not a must have | 01:57 |
flwang1 | generally, when deploy a service, operator will give it a unique domain name | 01:57 |
flwang1 | for different region | 01:57 |
flwang1 | so that said | 01:58 |
flwang1 | user should be able to figure out the region they're using | 01:58 |
flwang1 | however, if the page is hosted along with horizon, it will be tricky | 01:59 |
flwang1 | since the url to doashboard(horizon) is only one | 01:59 |
wanghao | Eva-i: hi, yes, you're right, my point is let developers/users whoever want to use this feature in devstack to host this page. It will be convenient to have devstack setup hosting this page. It cloud be done later I think. | 02:09 |
wanghao | flwang1: yes, so I perfer to show project only. | 02:10 |
Eva-i | wanghao: okay =/ | 02:11 |
flwang1 | wanghao: i'm fine with only project now and we can figure out if we can/should add the region in code review | 02:13 |
flwang1 | it's not a blocker for me | 02:13 |
wanghao | flwang1: sure | 02:13 |
Eva-i | flwang1: so for now we should not include devstack plugin modify task into this spec? If so, I can merge it. | 02:14 |
Eva-i | flwang1: check out https://review.openstack.org/#/c/268851/ | 02:16 |
flwang1 | Eva-i: yep, it's not a blocker for me | 02:18 |
flwang1 | Eva-i: +1 if you're happy with its current shape :) | 02:23 |
Eva-i | flwang1: hm, why I can't mark it as +2? I have no such option for some reason now. | 02:28 |
flwang1 | Eva-i: spec core team is a separated team :) | 02:30 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: [WIP]Subscription Confirmation Support for MongoDB https://review.openstack.org/316601 | 02:31 |
Eva-i | flwang1: aha, I see | 02:31 |
wanghao | Eva-i: yes, I also only can +1. | 02:31 |
Eva-i | flwang1: so we should ask some other developer to give it second +2? | 02:32 |
flwang1 | Eva-i: not really, i can approve it :) | 02:32 |
Eva-i | If I'm not mistaken, flaper87 was going to review it | 02:33 |
flwang1 | Eva-i: yep, i have asked him give it a final review | 02:33 |
wxy | Hello, everyone. the subscription confirm patch can be reviewed now. It only missed releasenote and new test code. | 02:34 |
Eva-i | wanghao: your spec now looks super nice =) | 02:34 |
wanghao | Eva-i: that can't be donw without your help. :) | 02:35 |
wanghao | done | 02:35 |
wanghao | Eva-i, flwang1, wxy, flaper87: I want to thanks guys' help for this spec, very much great reviews in this work. | 02:36 |
flwang1 | wanghao: thanks for the patience ;) | 02:38 |
Eva-i | wanghao: yes, thank you too =) | 02:42 |
wanghao | flwang1, Eva-i: you're welcome. :) | 02:42 |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Add Queues for api ref https://review.openstack.org/323059 | 02:50 |
*** tqtran has joined #openstack-zaqar | 03:31 | |
*** tqtran has quit IRC | 03:36 | |
Eva-i | flwang1: Hi. I want to remind you that we should rename the official name of Zaqar from "Message" to "Messaging". | 03:39 |
flwang1 | Eva-i: hmm... background? | 03:39 |
Eva-i | flwang1: we discussed a long time ago that in our docs and programs zaqar has different names "Messaging", "Message", "Queues", "Messaging and Notification". Our current official name is "Message": https://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml#n4495. You have decided that we should rename it to "Messaging" and start using "Messaging" in our content. | 03:46 |
flwang1 | Eva-i: oh, yes | 03:47 |
flwang1 | i will follow up that can figure out where we need to change | 03:48 |
flwang1 | thanks for the reminder | 03:48 |
Eva-i | flwang1: okay, so it's good that I have reminded you, you started to forget | 03:48 |
openstackgerrit | Merged openstack/zaqar: Add Queues for api ref https://review.openstack.org/323059 | 03:57 |
*** njohnston|afk is now known as njohnston | 03:59 | |
Eva-i | flwang1: BTW personally I'm okay with "Message". | 04:06 |
Eva-i | flwang1: I think I like it even more. | 04:08 |
Eva-i | flwang: also, before we make publish job for jenkins, we need to merge the initial commit of install guide. See https://review.openstack.org/#/c/327470/1/doc/contributor-guide/source/project-install-guide.rst@28. Or should I just try to use "Depends-On" tag before making changes to "project-config" repository? | 04:11 |
*** GB21 has joined #openstack-zaqar | 04:13 | |
*** njohnston is now known as njohnston|sleep | 04:59 | |
*** GB21 has quit IRC | 05:05 | |
*** GB21 has joined #openstack-zaqar | 05:37 | |
*** flwang has quit IRC | 06:05 | |
*** openstackgerrit has quit IRC | 06:18 | |
*** openstackgerrit has joined #openstack-zaqar | 06:18 | |
*** rcernin has joined #openstack-zaqar | 06:29 | |
*** flwang has joined #openstack-zaqar | 06:30 | |
*** GB21 has quit IRC | 06:53 | |
*** tesseract has joined #openstack-zaqar | 07:03 | |
*** rcernin has quit IRC | 07:04 | |
*** rcernin has joined #openstack-zaqar | 07:04 | |
*** GB21 has joined #openstack-zaqar | 07:05 | |
*** GB21 has quit IRC | 07:30 | |
openstackgerrit | wanghao proposed openstack/zaqar: [WIP]Subscription Confirmation Support for MongoDB https://review.openstack.org/316601 | 07:32 |
*** openstackgerrit has quit IRC | 07:33 | |
*** openstackgerrit has joined #openstack-zaqar | 07:33 | |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Subscription Confirmation Support for MongoDB https://review.openstack.org/316601 | 08:02 |
*** shu-mutou is now known as shu-mutou-AFK | 08:15 | |
*** GB21 has joined #openstack-zaqar | 08:24 | |
*** tqtran has joined #openstack-zaqar | 08:45 | |
*** tqtran has quit IRC | 08:50 | |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Subscription Confirmation Support for MongoDB https://review.openstack.org/316601 | 08:53 |
openstackgerrit | wangxiyuan proposed openstack/zaqar: Subscription Confirmation Support-1 https://review.openstack.org/316601 | 08:57 |
*** GB21 has quit IRC | 09:33 | |
*** GB21 has joined #openstack-zaqar | 10:01 | |
*** GB21 has quit IRC | 10:23 | |
*** GB21 has joined #openstack-zaqar | 10:48 | |
*** GB21 has quit IRC | 12:24 | |
*** tqtran has joined #openstack-zaqar | 12:47 | |
*** tqtran has quit IRC | 12:51 | |
*** rcernin has quit IRC | 13:14 | |
*** rcernin has joined #openstack-zaqar | 13:28 | |
*** rcernin has quit IRC | 13:54 | |
*** njohnston|sleep is now known as njohnston | 13:55 | |
*** mpanetta has joined #openstack-zaqar | 14:03 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/zaqar: Updated from global requirements https://review.openstack.org/322589 | 14:05 |
*** rcernin has joined #openstack-zaqar | 14:09 | |
*** mpanetta has quit IRC | 14:10 | |
*** rcernin has quit IRC | 14:18 | |
*** pt_15 has joined #openstack-zaqar | 14:28 | |
*** rcernin has joined #openstack-zaqar | 14:31 | |
*** Kevin_Zheng has quit IRC | 15:41 | |
*** pcaruana has quit IRC | 15:50 | |
*** rcernin has quit IRC | 15:53 | |
*** tesseract has quit IRC | 15:55 | |
*** srwilkers has joined #openstack-zaqar | 16:08 | |
openstackgerrit | Steve Linabery proposed openstack/zaqar: Relocates cli options registration from bootstrap to server https://review.openstack.org/327268 | 16:20 |
eggmaster | (just a rebase) | 16:21 |
eggmaster | (eggmaster=steve linabery) | 16:21 |
*** srwilkers has quit IRC | 16:45 | |
*** srwilkers has joined #openstack-zaqar | 16:46 | |
*** pcaruana has joined #openstack-zaqar | 17:21 | |
*** tqtran has joined #openstack-zaqar | 17:31 | |
*** tqtran has quit IRC | 17:35 | |
*** mpanetta has joined #openstack-zaqar | 18:07 | |
*** mpanetta has quit IRC | 18:11 | |
*** tqtran has joined #openstack-zaqar | 18:13 | |
*** srwilkers has left #openstack-zaqar | 18:25 | |
*** pt_15 has quit IRC | 18:36 | |
*** aimeeu has quit IRC | 18:39 | |
*** aimeeu has joined #openstack-zaqar | 18:41 | |
flwang1 | therve: ping | 19:20 |
flwang1 | therve: i'm wondering if this patch https://review.openstack.org/#/c/329520/1 is targeting the mistral notification | 19:20 |
therve | flwang1, yup | 19:21 |
flwang1 | therve: awesome | 19:21 |
therve | flwang1, Except without mistral, but yeah :) | 19:21 |
flwang1 | if so, i think i can work on the mistral part | 19:21 |
therve | Well that's the thing there is no mistral part | 19:22 |
flwang1 | therve: i see. but your work could be leveraged by the mistral spec, no? | 19:22 |
therve | But there is one potential controversial change I think, interested what you think about it | 19:22 |
therve | I mean there is everything that you need in that patch, in theory | 19:22 |
therve | It ought to be generic enough to work for mistral | 19:23 |
flwang1 | therve: yep, that's what i'm thinking. with your work, we should be able easily support any openstack services | 19:23 |
flwang1 | i was thinking the task name should be 'os_service_task' :D | 19:24 |
therve | task name? Instead of trust you mean? | 19:25 |
flwang1 | therve: no, i mean your work is a great middle layer | 19:26 |
therve | flwang1, But that's what I mean earlier, I'm not sure what you need to put on top of it | 19:27 |
flwang1 | therve: sorry for the confusion | 19:27 |
flwang1 | therve: my initial thought is not good, so now the idea is more clear base don your current work | 19:27 |
flwang1 | so now if we want to support mistral, i think we can easily make the mistral_task as a sub class of the trust_task | 19:28 |
flwang1 | and the only thing we need to do is define the post body, IIUC | 19:28 |
therve | That's what I'm saying: you can already talk to mistral using the trust task :) | 19:28 |
therve | My patch adds a subscription option to tweak the post body | 19:28 |
flwang1 | therve: yep, i know | 19:29 |
flwang1 | therve: i really love this patch, it's a great work | 19:29 |
therve | flwang1, Somewhat related, is there a work in progress to move documentation away from XML? | 19:32 |
flwang1 | therve: are you talking about this https://review.openstack.org/323059 ? | 19:32 |
flwang1 | therve: and this https://review.openstack.org/268851 | 19:33 |
*** openstackgerrit has quit IRC | 19:33 | |
therve | flwang1, No | 19:33 |
*** openstackgerrit has joined #openstack-zaqar | 19:33 | |
therve | I'm talking about the user guide | 19:33 |
flwang1 | user guide? oh, yes | 19:34 |
flwang1 | it's on my list, but we don't have limited resource you know | 19:34 |
flwang1 | we have done the config ref last cycle | 19:34 |
therve | Yeah I was hoping there was a script to run somewhere | 19:34 |
flwang1 | and this cycle we're focusing on api ref and install guide | 19:35 |
flwang1 | i can give it a higher priority of the user guide | 19:35 |
flwang1 | given my api ref patch has been merged | 19:36 |
therve | https://wiki.openstack.org/wiki/Documentation/Migrate#Migration_Using_a_Text_Editor looks like the procedure | 19:40 |
flwang1 | therve: awesome, i will take a look | 19:49 |
flwang1 | but why you think it's related? ;) | 19:49 |
flwang1 | just wondering | 19:49 |
therve | Because I won't write any documentation in XML format | 19:50 |
therve | But I might in RST :) | 19:50 |
flwang1 | therve: wow, that's a good reason | 19:52 |
flwang1 | for that case, i'm happy to get it done so that i can get your docs about this feature :D | 19:52 |
flwang1 | therve: btw, where is heat's user guide? in tree or in the docs.openstack.org's repo? | 19:58 |
therve | flwang1, In tree | 20:01 |
*** flwang1 has quit IRC | 20:04 | |
*** flwang1 has joined #openstack-zaqar | 21:36 | |
*** openstackgerrit has quit IRC | 22:02 | |
*** openstackgerrit has joined #openstack-zaqar | 22:05 | |
openstackgerrit | Fei Long Wang proposed openstack/zaqar: Integrate OSprofiler with Zaqar https://review.openstack.org/141356 | 22:06 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!