| *** zhurong has quit IRC | 00:02 | |
| *** elynn_ has quit IRC | 00:33 | |
| *** yanyanhu has joined #senlin | 01:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/360160 | 01:11 |
|---|---|---|
| *** zhurong has joined #senlin | 01:17 | |
| yanyanhu | hi, Qiming, around? | 01:32 |
| Qiming | yes | 01:33 |
| yanyanhu | feilong just left some comments on this etherpad about message type receiver https://etherpad.openstack.org/p/senlin-message-type-receiver | 01:33 |
| Qiming | yes | 01:33 |
| yanyanhu | great new is zaqar supports posting notification with credential, e.g. token :) | 01:33 |
| yanyanhu | so we can leverage this ability to make the access to subscriber url much safer | 01:33 |
| yanyanhu | this will be different from webhook url access | 01:34 |
| Qiming | just a normal request | 01:34 |
| yanyanhu | yes, I think so | 01:34 |
| Qiming | as those that initiated by senlinclient? | 01:34 |
| Qiming | then we can just add an api | 01:34 |
| yanyanhu | no so sure about the detail, but will check the doc feilong pointed out | 01:35 |
| Qiming | okay | 01:35 |
| yanyanhu | Qiming, yes, it will be possible | 01:35 |
| Qiming | it is good if they have tokens in their request header | 01:35 |
| yanyanhu | yes | 01:35 |
| Qiming | just one thing I want to understand | 01:35 |
| Qiming | the targeted use case (for senlin) is as this | 01:35 |
| Qiming | a software (agent) running inside a VM sends a message to zaqar, and zaqar will reformat that request somehow, and then send a request to a "secret" API of senlin | 01:36 |
| Qiming | the goal is to eliminate the necessity of having the agent know the IP addresses of the controller nodes (where senlin-api/engine are running on) | 01:37 |
| Qiming | it would be great that they provide some help/doc/guidance on setting up such a scenario | 01:38 |
| yanyanhu | Qiming, I think they currently don't have a complete guide for building this kind of use case | 01:38 |
| Qiming | in other words, if the guest software (agent) has to know the IP address of the controller node, zaqar is still not that useful | 01:38 |
| yanyanhu | but I think this kind of use case can be supported | 01:39 |
| Qiming | we can have that software to initiate the "webhook" invocation directly | 01:39 |
| yanyanhu | not so sure, but hope zaqarclient can help to handle this | 01:39 |
| Qiming | if they don't have a guide on this, we should recommend them to write one | 01:39 |
| yanyanhu | since it doesn't make much sense that let enduser/software to access zaqar API directly | 01:39 |
| Qiming | yes, that is the core value of a user-facing messaing service | 01:40 |
| Qiming | just like Amazon SQS | 01:40 |
| yanyanhu | Qiming, or maybe we can help to write one based on our experience in future :) | 01:40 |
| yanyanhu | as part of autoscaling story | 01:40 |
| yanyanhu | oh, this is a little different | 01:40 |
| Qiming | but as the first step, we'll need to sync with zaqar team on this | 01:40 |
| yanyanhu | sure | 01:40 |
| Qiming | it should be them to author such an article | 01:41 |
| yanyanhu | will confirm the usage of zaqarclient | 01:41 |
| Qiming | they know what they are working on | 01:41 |
| yanyanhu | abosolutely | 01:41 |
| Qiming | maybe zaqarclient is not necessary as well | 01:41 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 01:41 |
| Qiming | there should be a possibility to create and post such a message in simple scripts | 01:42 |
| yanyanhu | Qiming, yes, I think there could be more than one feasible ways for enduer to post message | 01:42 |
| yanyanhu | yea | 01:42 |
| Qiming | or else, users will be forced to install zaqarclient | 01:42 |
| Qiming | absolutely | 01:42 |
| yanyanhu | yes. maybe something like what amazon do for waitcondition | 01:43 |
| yanyanhu | whose event can be triggered using curl simply | 01:43 |
| Qiming | yes, but amazon is requiring some cfntools installed as well | 01:43 |
| Qiming | when I have cycles, I'd like to start a project os-agent | 01:43 |
| yanyanhu | I see, will make some investigation here | 01:44 |
| Qiming | consolidating all requirements into the same agent software | 01:44 |
| yanyanhu | cool :) | 01:44 |
| Qiming | just like cloud-init | 01:44 |
| yanyanhu | agent inside VM is a REAL requirement for many scenarios | 01:44 |
| yanyanhu | although people usually don't like it that much :) | 01:44 |
| Qiming | but it will handle use cases like heat software-config, trove agent, ceilometer sample generation, zaqar message posting etc | 01:44 |
| yanyanhu | yes | 01:44 |
| yanyanhu | maybe this is a good topic for cross project coordination | 01:45 |
| Qiming | yes, it is a topic that has been brought up many times | 01:45 |
| Qiming | each project is reinventing the wheel ... | 01:45 |
| yanyanhu | yes... | 01:46 |
| Qiming | you can see this: http://git.openstack.org/cgit/openstack/trove-image-builder/ | 01:47 |
| Qiming | it is empty, but it really means that trove needs things injected to a image | 01:48 |
| Qiming | also sahara: http://git.openstack.org/cgit/openstack/sahara-image-elements/ | 01:49 |
| *** elynn has joined #senlin | 01:52 | |
| *** elynn has quit IRC | 01:56 | |
| *** elynn has joined #senlin | 01:56 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 02:08 |
| *** zhurong_ has joined #senlin | 02:09 | |
| *** zhurong has quit IRC | 02:11 | |
| openstackgerrit | Tony Breeds proposed openstack/senlin: [DNM] Test openstacksdk 0.9.4 https://review.openstack.org/360188 | 02:49 |
| *** yuanying has quit IRC | 02:49 | |
| *** yuanying has joined #senlin | 02:53 | |
| Qiming | xuhaiwei, online? | 02:56 |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/360160 | 03:05 |
| openstackgerrit | Merged openstack/python-senlinclient: Imported Translations from Zanata https://review.openstack.org/358169 | 03:05 |
| openstackgerrit | Merged openstack/python-senlinclient: Add profile validate operation to senlinclient https://review.openstack.org/358933 | 03:05 |
| openstackgerrit | Merged openstack/python-senlinclient: Add policy validate operation to senlinclient https://review.openstack.org/358936 | 03:06 |
| openstackgerrit | Merged openstack/python-senlinclient: py3:Rmove six.iteritems/iterkeys in python-senlinclient https://review.openstack.org/358926 | 03:06 |
| *** yuanying has quit IRC | 03:45 | |
| xuhaiwei | Qiming, yes | 03:48 |
| *** yuanying has joined #senlin | 03:48 | |
| *** elynn has quit IRC | 04:30 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 04:51 |
| *** elynn has joined #senlin | 05:03 | |
| *** elynn has quit IRC | 05:07 | |
| *** elynn has joined #senlin | 05:08 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: A spec for container cluster service https://review.openstack.org/281102 | 05:10 |
| Qiming | xuhaiwei, I'm working on patch 349906 | 05:19 |
| Qiming | moving logics down from engine/service to profile layer, will push a patch later today | 05:19 |
| Qiming | will let you know when it is submitted | 05:20 |
| xuhaiwei | ok, Qiming | 05:34 |
| Qiming | I'm removing 'host_cluster' property temporarily | 05:35 |
| Qiming | will add it later when we have a better idea about how to handle it | 05:35 |
| xuhaiwei | I have just updated the spec file, and added some rules on how to use host_node and host_cluster, I think we should make an agreement on it, Qiming | 05:39 |
| Qiming | sure | 05:40 |
| xuhaiwei | please review that patch, and write comments | 05:40 |
| Qiming | will do | 05:52 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 05:57 |
| Qiming | xuhaiwei, ^ is the new revision | 05:57 |
| Qiming | need to add test cases | 05:57 |
| Qiming | woops, finally -- [14:05] <openstackgerrit> Merged openstack/requirements: Bump openstacksdk version to 0.9.4 https://review.openstack.org/359486 | 06:07 |
| elynn | Great! | 06:08 |
| elynn | Now we can merge patches! | 06:08 |
| Qiming | hopefully, just kicked off a recheck, about the removal of mox/mox3 | 06:09 |
| xuhaiwei | thanks, Qiming | 06:18 |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/senlin: TrivialFix: Remove logging import usused https://review.openstack.org/360265 | 06:19 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Revise tools/setup-service to make it work with keystone v3 https://review.openstack.org/360271 | 06:31 |
| openstackgerrit | lvdongbing proposed openstack/senlin: Revise tools/setup-service to make it work with keystone v3 https://review.openstack.org/360271 | 06:39 |
| openstackgerrit | Merged openstack/senlin: Add zaqar v2 driver interfaces for claim resource https://review.openstack.org/355830 | 07:15 |
| openstackgerrit | Merged openstack/senlin: Minior revise pre/post_test_hook https://review.openstack.org/351549 | 07:15 |
| yanyanhu | hi, Qiming, just noticed those two integration test cases were run and passed correctly | 07:22 |
| yanyanhu | can start to work on more integration test cases | 07:22 |
| Qiming | great! | 07:24 |
| Qiming | are they experimental? | 07:25 |
| openstackgerrit | Merged openstack/senlin: py3:Remove six.iteritems/iterkeys https://review.openstack.org/357793 | 07:25 |
| yanyanhu | no, but they don't vote now | 07:25 |
| yanyanhu | they will be run, just don't vote | 07:25 |
| Qiming | em, that's fine | 07:25 |
| yanyanhu | yes, in current stage, maybe we don't make it vote always | 07:26 |
| yanyanhu | since the failure could be caused by other reasons rather than senlin's bug | 07:26 |
| Qiming | I was even considering skipping it most of the time | 07:26 |
| yanyanhu | anyway, it will be very helpful to verify whether senlin can work correctly with other services | 07:27 |
| Qiming | yes | 07:27 |
| yanyanhu | as a reference | 07:27 |
| yanyanhu | will try to add a new test case for lb policy | 07:27 |
| Qiming | okay, speaking of lb policy | 07:28 |
| Qiming | not sure if we can complete profile/policy versioning in time | 07:28 |
| Qiming | maybe we can treat the timeout option as a bug fix? | 07:29 |
| Qiming | and ... let it in? | 07:29 |
| yanyanhu | Qiming, yes, have the same concern... | 07:29 |
| yanyanhu | ok | 07:29 |
| openstackgerrit | Merged openstack/senlin: Remove usage of mox/mox3 https://review.openstack.org/358383 | 07:33 |
| openstackgerrit | Merged openstack/senlin: Get ready for os-api-ref sphinx theme change https://review.openstack.org/357931 | 07:33 |
| Qiming | note to all | 07:48 |
| Qiming | I'm gonna change the output format for network addresses for nova server profile | 07:49 |
| Qiming | the reason is that from client's perspective, we may need all those extra keys in addition to the 'addr' property | 07:49 |
| Qiming | this will impact nova server profile and lbaas driver | 07:49 |
| yanyanhu | Qiming, what impact there will be on nova server profile? you mean the get detail funtion? | 07:51 |
| yanyanhu | s/funtion/function | 07:51 |
| Qiming | yes | 07:51 |
| yanyanhu | I see. For lbaas driver, will you fix it as well at the same time? | 07:52 |
| yanyanhu | if not, I can provide some help here | 07:52 |
| openstackgerrit | Merged openstack/senlin: Reorganize receivers directory https://review.openstack.org/357585 | 07:52 |
| openstackgerrit | Merged openstack/senlin: Imported Translations from Zanata https://review.openstack.org/359014 | 07:52 |
| Qiming | yep | 07:52 |
| yanyanhu | ok, great | 07:52 |
| openstackgerrit | Merged openstack/senlin: Revise tools/setup-service to make it work with keystone v3 https://review.openstack.org/360271 | 07:52 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Faithful dump nova server addresses property https://review.openstack.org/360308 | 07:52 |
| Qiming | this one ^ | 07:53 |
| yanyanhu | ok, will check it | 07:53 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Minor tweak webhook receiver https://review.openstack.org/359636 | 08:04 |
| openstackgerrit | Merged openstack/senlin: Add new api for profile validation https://review.openstack.org/354549 | 08:17 |
| openstackgerrit | Merged openstack/senlin: Fix do_validate in heat profile https://review.openstack.org/355780 | 08:17 |
| openstackgerrit | Merged openstack/senlin: Updated from global requirements https://review.openstack.org/359529 | 08:41 |
| openstackgerrit | Merged openstack/senlin: Make lb timeout configurable https://review.openstack.org/346569 | 08:41 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: api-ref: OpenStack-Request-Id and location https://review.openstack.org/358986 | 08:46 |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** openstackgerrit has joined #senlin | 08:49 | |
| *** ChanServ sets mode: +v openstackgerrit | 08:49 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 09:07 |
| xuhaiwei | Qiming, made some changes to 349906 , please review | 09:08 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 09:09 |
| xuhaiwei | will add test cases | 09:09 |
| Qiming | okay, thanks | 09:11 |
| Qiming | just one comment posted | 09:15 |
| xuhaiwei | I thought about it a little, shall we let the exception raise up to upper layer? | 09:18 |
| xuhaiwei | because if we catch it, then raise what exception ? | 09:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/360373 | 09:31 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/senlin: Updated from global requirements https://review.openstack.org/360374 | 09:31 |
| *** elynn has quit IRC | 09:32 | |
| openstackgerrit | Merged openstack/senlin: Release note for Zaqar resource support https://review.openstack.org/359556 | 09:35 |
| openstackgerrit | Merged openstack/senlin: Add policy_validate function to engine https://review.openstack.org/358686 | 09:36 |
| openstackgerrit | Merged openstack/senlin: Add new api for policy validation https://review.openstack.org/358759 | 09:36 |
| openstackgerrit | Merged openstack/senlin: Some new typos need to be fixed https://review.openstack.org/358921 | 09:36 |
| *** zhurong_ has quit IRC | 10:01 | |
| *** yanyanhu has quit IRC | 10:02 | |
| *** elynn has joined #senlin | 11:07 | |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/360373 | 12:10 |
| openstackgerrit | Merged openstack/senlin: TrivialFix: Remove logging import usused https://review.openstack.org/360265 | 12:27 |
| *** zhurong has joined #senlin | 12:58 | |
| openstackgerrit | Merged openstack/senlin: Updated from global requirements https://review.openstack.org/360374 | 13:17 |
| *** openstackgerrit has quit IRC | 14:48 | |
| *** openstackgerrit has joined #senlin | 14:50 | |
| *** ChanServ sets mode: +v openstackgerrit | 14:50 | |
| *** Drago has joined #senlin | 15:03 | |
| *** Drago has quit IRC | 15:04 | |
| *** Drago has joined #senlin | 15:04 | |
| *** zhurong has quit IRC | 16:20 | |
| *** elynn has quit IRC | 16:51 | |
| *** Drago has quit IRC | 17:00 | |
| *** Drago has joined #senlin | 17:22 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting to implement some performance tuning adjustments, and should return to working order momentarily. | 19:47 | |
| *** elynn has joined #senlin | 23:35 | |
| *** Drago has quit IRC | 23:41 | |
| *** zhurong has joined #senlin | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!