Monday, 2017-07-31

*** wanghao has joined #openstack-mogan00:20
zhenguomorning mogan!00:40
*** openstackgerrit has joined #openstack-mogan00:55
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893100:55
*** liusheng has joined #openstack-mogan01:07
*** litao__ has joined #openstack-mogan01:11
wanghaomorning!01:18
wanghaoliusheng: ping01:18
liushengwanghao: pong01:18
wanghaoliusheng: do you test guru in uwsgi. it do not work?01:19
liushengzhenguo: yes, I don't know why :(01:19
wanghaoliusheng: it just work under 2en way01:19
liushengwanghao: yes, I just add the config option to mogan.conf, and then restart api server, but I cannot find any log01:20
wanghaoliusheng: did you touch the file?01:20
liushengwanghao: yes01:21
wanghaoliusheng: mogan-api run with 2>/var/log/mogan/mogan-api-err.log?01:22
liushengwanghao: yes,01:23
wanghaook I will check it01:24
liushengwanghao: let me try again01:24
wanghaoliusheng: or you can withouth the 2>01:24
wanghaoliusheng: and you will see the guru report in  uwsgi console output01:24
liushengwanghao: just restart mogan-api ?01:25
wanghaoyes01:25
wanghaoand then touch the file01:25
wanghaojust uwsgi --ini /etc/mogan/mogan-uwsgi.ini01:26
liushengwanghao: no, I use systemctl to restart api server01:27
wanghaoliusheng: so you also can't see the uwsgi console output01:28
liushengwanghao: but currently, the common way  to restart services in devstack is using systemctl01:29
liushengwanghao: it cannot support this way ?01:29
wanghaoliusheng: what is your mogan-api process looking like.  ps -ef | grep mogan-api01:34
liushengwanghao: /usr/local/bin/uwsgi --ini /etc/mogan/mogan-uwsgi.ini01:35
zhenguoliusheng: do you know which config file to use with uwsgi01:36
zhenguoliusheng: like /etc/mogan/mogan.cnof01:37
wanghaoliusheng: /usr/local/bin/uwsgi --ini /etc/mogan/mogan-uwsgi.ini 2>/var/log/mogan/mogan-api-err.log01:37
wanghaoI use this,  and it works in my env01:37
wanghaodid you have path /var/log/mogan?01:37
liushengwanghao: you mean add "2>/var/log/mogan/mogan-api-err.log" after the command ?01:37
liushengwanghao: yes01:38
wanghaoyes01:38
wanghaoand then touch the file later01:38
liushengwanghao: no I use "service devstack@mogan-api restart" to restart my api server01:38
liushengwanghao: I touched the file01:38
wanghaoso you need to add 2>/var/log/mogan/mogan-api-err.log01:38
wanghaoand then see the report in this log file01:39
liushengwanghao: so I cannot add that, unless I modify the systemd script :(01:39
liushengwanghao: if so, the gmr cannot directly work with systemd ?01:39
wanghaoliusheng: I think it can,  with the systemd,  where is your log?01:41
liushengwanghao: with "journalctl", e.g. "journalctl -u devstack@mogan-api"01:42
wanghaoliusheng: could you see other uwsgi log in journalctl?01:46
liushengwanghao: yes01:46
liushengwanghao: just specify the systemd startup script name after -u parameter01:47
wanghaoliusheng: I find the devstack@mogan-api.service is not uwsgi way01:50
wanghaoliusheng: it just wsgi: /usr/bin/python /usr/local/bin/mogan-api --config-file /etc/mogan/mogan.conf01:51
liushengwanghao: why ??01:51
liushengwanghao: no, maybe you didn't re-stack your env recently, hah01:51
wanghaoliusheng: what is your script in devstack@mogan-api.service01:52
wanghaoliusheng: never mind,  I will update it.01:52
liushengwanghao: after the https://review.openstack.org/#/c/482057/ patch merged, if you install your devstack env, the api server will run under uwsgi01:53
wanghaoI see01:53
wanghaoliusheng: it work fine with systemctl01:58
wanghaoliusheng: Jul 31 09:58:18 openstack uwsgi[17255]: ====                         Green Threads                          ====01:58
wanghaoI can find the log in my env01:59
wanghaoI also use the systemctl way01:59
liushengwanghao: really, let me try01:59
wanghaoif you didn't find the log,  you can wait for a seconds,  and then touch again01:59
wanghaoand you will search the key word guru in your whole log02:00
liushengwanghao: no I didn't fond that, I am installing a new env, will test in another devstack env02:03
wanghaookay02:08
wanghaohttps://docs.openstack.org/zaqar/latest/admin/gmr.html02:13
wanghaohere is doc for uwsgi mode02:13
wanghaoA GMR can be generated by “touch”ing the file which was specified in file_event_handler. The GMR will then output to standard error for that particular process.02:13
wanghaoSo technically, you can check it in standard error.02:13
openstackgerritwanghao proposed openstack/mogan master: Support Guru in uwsgi way  https://review.openstack.org/48773802:16
openstackgerritwanghao proposed openstack/mogan master: Support quota for keypairs in Mogan(part-three)  https://review.openstack.org/48546102:37
*** deep-book-gk has joined #openstack-mogan02:46
*** deep-book-gk has left #openstack-mogan02:48
openstackgerritZhenguo Niu proposed openstack/mogan master: Add aggregates tests  https://review.openstack.org/48829603:54
openstackgerritZhenguo Niu proposed openstack/mogan master: Add aggregate nodes API  https://review.openstack.org/48728403:54
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893104:48
openstackgerritZhenguo Niu proposed openstack/mogan master: Add aggregates tests  https://review.openstack.org/48829604:54
openstackgerritZhenguo Niu proposed openstack/mogan-specs master: Node aggregates  https://review.openstack.org/47092704:56
openstackgerritZhenguo Niu proposed openstack/mogan master: Trivial-Fix: Correct test case name in tempest  https://review.openstack.org/48908905:00
wanghaozhenguo: add some comments on node aggregates spec.06:07
zhenguowanghao: thanks06:07
wanghaozhenguo: : )06:10
zhenguowanghao: please find my replies06:17
wanghaozhenguo: sure06:17
* zhenguo brb06:26
litao__zhenguo: I found the latest codes has added some flavor columns. How can i apply it to my devstack rapidly06:35
litao__zhenguo: not restack my devstack.06:36
zhenguolitao__: seems you need to recreat your db06:36
litao__zhenguo: How?06:37
zhenguolitao__: maybe just drop mogan database, and use dbsync  but which will lose all data06:37
litao__zhenguo: OK,  i can restore my db data06:38
zhenguolitao__: ok06:38
zhenguoliusheng: does placement only need a config which has database and keystone config options?06:39
wanghaozhenguo: I see your reply,  so I think maybe we need to make the alternaives more clear, since we implement the aggregates on mogan side basically.06:40
zhenguowanghao: ok, the alternatives should be something like, we don't use placement any more, hah06:41
liushengzhenguo: I think so, at least the username of placement need to be confiured in [keystone_authtoken] in nova.conf06:41
wanghaozhenguo: or just None, that's better I think06:42
zhenguoliusheng: ok06:42
zhenguowanghao: hah, ok06:42
wanghaozhenguo: I check the placement API,  since nova has host aggregates, so it put the aggregates id to resource provider06:43
wanghaozhenguo: so for Mogan, it's node aggregates like Nova did.06:43
zhenguowanghao: yes, and we also need to put our node aggregate id there.06:43
zhenguowanghao: so, with placement nova can't handle bare metal aggregates as well :D06:44
openstackgerritZhenguo Niu proposed openstack/mogan-specs master: Node aggregates  https://review.openstack.org/47092706:45
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893107:00
zhenguoliusheng: I would like to add an aggregates attribute to flavor, wdyt?07:19
liushengzhenguo: how to use that ?07:20
zhenguoliusheng: with key/value pairs07:20
zhenguoliusheng: as now we use query_filters of rp list instead of filters node key/values like before07:21
zhenguoliusheng: we can't use an extra specs to include everything07:21
zhenguoliusheng: we need to know which flavor attribute matches which filter07:21
liushengzhenguo: we don't have extra now07:22
zhenguoliusheng: rp list only support resources, traits, aggregates query filter07:22
zhenguoliusheng: and now our flavor have resource and resource traits07:22
liushengzhenguo:yes07:22
zhenguoliusheng: so we can't put aggregates info to those field right?07:23
zhenguoliusheng: before, I think we can put aggregate info to traits, but it's hard to use07:23
zhenguoliusheng: and I don't want to add extra specs back07:23
liushengzhenguo: so we don't need schedule_hints ?07:23
zhenguoliusheng: not related to scheduler_hints I think :D07:24
zhenguoliusheng: only for aggregates07:24
zhenguoliusheng: I want to construct the list rp query filters07:24
liushengzhenguo: sounds ok, but not sure if we need to add simlar things to flavor in future07:25
openstackgerritTao Li proposed openstack/mogan master: Manage existing BMs: Part-2  https://review.openstack.org/48154407:25
zhenguoliusheng: when there's a new query filter, we may add a new filed in flavor07:25
zhenguoliusheng: but seems that's all, quanlitive, quantitive, and group07:26
zhenguo*qualitative07:27
liushengzhenguo: for now, it looks like ok :)07:27
zhenguoliusheng: then every specified aggregate key/value will cost a rp list API call07:27
zhenguoliusheng: then will set set1 | set 2| setn to get the final rps07:28
zhenguoliusheng: mogan aggregate metadta key/value may result in a list of aggregates07:29
liushengzhenguo: that I think is because the placement not matrue yet07:29
zhenguoliusheng: seems not07:29
zhenguoliusheng: one mogan metadata key, may have a list of aggregate07:29
zhenguoliusheng: but not sure whether rp will support all that conditions in future07:30
zhenguoliusheng: for now, we can just do like that07:30
liushengzhenguo: ok, we can improve if placment provide more functionalities07:31
zhenguoliusheng: ok07:31
liushengzhenguo: so we won't have something like metadata of flavor ?07:32
liushengzhenguo: seems resources_traits isn't07:32
zhenguoliusheng: yes07:32
zhenguoliusheng: as we don't know how to handle the metadata07:32
zhenguoliusheng: we only extract the 3 query opetions from flavor07:32
zhenguoliusheng: *options07:32
liushengzhenguo: seems yes07:33
zhenguoliusheng: it's not like the old way of filtering07:33
liushengzhenguo: sure, we won't have filter like old ones07:34
zhenguoliusheng: ok, will draft a patch to see07:34
liushengzhenguo: ok07:34
* zhenguo brb07:38
openstackgerritTao Li proposed openstack/mogan master: Manage existing BMs: Part-2  https://review.openstack.org/48154407:43
zhenguoliusheng: when will we clean up /etc/systemd/system/devstack@placement-api.service?08:07
liushengzhenguo: seems unstack.sh08:07
zhenguoliusheng: on my test it's not08:07
liushengzhenguo: oh, maybe in stack.sh, not sure08:07
zhenguoliusheng: and if you changed some parameters, it's not override08:08
liushengzhenguo: didn't try that08:08
openstackgerritMerged openstack/mogan master: Remove legacy auth loading  https://review.openstack.org/48893708:17
openstackgerritMerged openstack/mogan master: Trivial-Fix: Correct KeypairExists exception name  https://review.openstack.org/48893408:17
openstackgerritMerged openstack/mogan master: Trivial-Fix: Correct test case name in tempest  https://review.openstack.org/48908908:17
zhenguohttp://insight.qingcloud.com/#insight-live , QingCloud new bare metal service08:22
zhenguoliusheng, shaohe_feng, wanghao, litao__ ^^08:22
liushengzhenguo: thanks for the sharing :)08:24
zhenguoliusheng: hah08:24
zhenguoliusheng: they use the same API :(08:27
* litao__ pls review: https://review.openstack.org/#/c/481544/08:28
liushengzhenguo: what ?08:28
zhenguoliusheng: same API for vm and bm08:28
liushengzhenguo: oh, like Nova does :(08:29
zhenguoliusheng: seems not08:29
zhenguoliusheng: they just follow the same API pattern08:29
liushengzhenguo: oh, don't fimilar with08:31
zhenguoliusheng: hah08:32
openstackgerritZhenguo Niu proposed openstack/mogan master: Add aggregates tests  https://review.openstack.org/48829608:37
zhenguoliusheng: when to create this  /etc/systemd/system/devstack@placement-api.service ?08:38
zhenguoliusheng: after manually clean up it, it's not created with stack.sh08:38
zhenguoliusheng: now I can succeed to start placement-api with uwsgi08:39
zhenguoliusheng: but can't under systemd08:39
liushengzhenguo: really, it should be created by stack.sh08:39
liushengzhenguo: let me change08:39
liushengzhenguo: check08:39
liushengzhenguo: did you try: systemctl daemon-reload08:45
zhenguoliusheng: I'm restacking now08:45
zhenguoliusheng: still, devstack@placement-api isn't created09:18
zhenguoliusheng: /etc/systemd/system/devstack@placement-api.service09:18
liushengzhenguo: I guess may devstack will handle by "enabled services"09:21
liushengzhenguo: you just start placement in mogan devstack plugin09:21
zhenguoliusheng: how does plugin work09:21
liushengzhenguo: devstack may will create the syetemd script by the name of project in enabled services09:22
liushengzhenguo: let me check09:23
zhenguoliusheng: seems yes, all mogan services there09:23
liushengzhenguo: may you can call write_user_unit_file method in our plugin09:26
*** wanghao has quit IRC09:26
liushengzhenguo: it is defined in devstack/common-functions09:27
zhenguoliusheng: let me check09:27
liushengzhenguo: devstack/functions-common09:27
openstackgerritZhenguo Niu proposed openstack/mogan master: Add aggregates filed to flavor  https://review.openstack.org/48915109:29
* zhenguo brb09:31
zhenguoliusheng: yes, it will check whether service enabled in run_process09:45
zhenguo:q09:46
zhenguoliusheng: stop_process will also check enabled services09:52
liushengzhenguo: but i didn09:53
liushengzhenguo: didn't found any place to generate the file in devstack09:53
liushengzhenguo: besides write_user_unit_file method function ,but the function only be called once for etcd309:54
zhenguoliusheng: no, it will called under run_process09:54
liushengzhenguo: oh09:54
zhenguoliusheng: I will add a own run_proces sand stop_process method for placement09:56
liushengzhenguo: may you can leverage the write_user_unit_file method :)09:57
zhenguoliusheng: not only that,09:57
zhenguoliusheng: run process will do other things09:57
zhenguoliusheng: you can check under the is_service_enabled09:57
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893109:58
openstackgerritZhenguo Niu proposed openstack/mogan-specs master: Update new-flavor specs  https://review.openstack.org/48916710:23
openstackgerritZhenguo Niu proposed openstack/mogan master: Add resource aggregates filed to flavor  https://review.openstack.org/48915110:28
openstackgerritZhenguo Niu proposed openstack/mogan master: Add aggregates tests  https://review.openstack.org/48829610:38
zhenguoliusheng: hah, it finally passed the gate https://review.openstack.org/#/c/488931/10:41
liushengzhenguo: awesome :D10:42
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893110:43
zhenguoliusheng: will remove the depends-on to test if it's work when user enable placement-api service with nova10:43
liushengzhenguo: ok, let's see it10:44
zhenguoliusheng: ok10:44
zhenguoliusheng: please help to test this https://review.openstack.org/#/c/484690/ when you got time10:45
liushengzhenguo: ok10:47
openstackgerritZhenguo Niu proposed openstack/mogan-specs master: Node aggregates  https://review.openstack.org/47092710:48
* zhenguo brb11:03
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893111:19
*** litao__ has quit IRC11:50
openstackgerritZhenguo Niu proposed openstack/mogan master: Add placement with mogan devstack plugin  https://review.openstack.org/48893112:08
* zhenguo away12:09
*** openstackgerrit has quit IRC14:33
*** openstackgerrit has joined #openstack-mogan16:20
openstackgerritSpencer Yu proposed openstack/python-moganclient master: Switch from oslosphinx to openstackdocstheme  https://review.openstack.org/48926916:20
*** liusheng has quit IRC18:09
*** liusheng has joined #openstack-mogan18:09
*** harlowja has joined #openstack-mogan18:11

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