*** xuhaiwei has quit IRC | 00:03 | |
openstackgerrit | Merged stackforge/senlin: Updated from global requirements https://review.openstack.org/204431 | 00:14 |
---|---|---|
*** jruano has quit IRC | 00:36 | |
*** lixinhui_ has joined #senlin | 00:37 | |
*** xuhaiwei has joined #senlin | 00:42 | |
*** hightall has joined #senlin | 00:48 | |
*** Qiming_ has joined #senlin | 01:11 | |
*** hightall has quit IRC | 01:31 | |
*** jruano has joined #senlin | 01:32 | |
*** mathspanda has joined #senlin | 01:36 | |
xuhaiwei | Qiming_, in this patch https://review.openstack.org/#/c/204538/, gate is always NOT_REGISTERED, do you know the reason | 01:40 |
Qiming_ | gerrit was down last night | 01:40 |
Qiming_ | it is now okay | 01:40 |
xuhaiwei | I rechecked it, but nothing changed | 01:40 |
Qiming_ | -openstackstatus/#openstack-dev- NOTICE: CI is slowly recovering, please be patient while the backlog is worked through. | 01:41 |
xuhaiwei | ok | 01:41 |
Qiming_ | zuul has experienced some problems afict | 01:41 |
*** hightall has joined #senlin | 01:44 | |
*** Yanyanhu has joined #senlin | 01:49 | |
Qiming_ | lixinhui_, there? | 01:53 |
lixinhui_ | yes | 01:54 |
Qiming_ | https://review.openstack.org/#/c/204637/ | 01:54 |
lixinhui_ | okay, thanks | 01:55 |
lixinhui_ | I will try it right now | 01:55 |
Qiming_ | ok | 01:55 |
Yanyanhu | hi, Qiming_ this patch https://review.openstack.org/204441. When I tried to check pep8 error log, I got an error 'File Not Found' | 02:01 |
xuhaiwei | I think this is also the gate's problem | 02:02 |
Qiming_ | just do a recheck? | 02:02 |
Yanyanhu | ok, let me have a try | 02:02 |
Yanyanhu | xuhaiwei, me too | 02:02 |
Yanyanhu | by the way, will start work on test case for lb_policy, please help to review :) | 02:03 |
Yanyanhu | thanks | 02:03 |
Qiming_ | sure | 02:04 |
*** Qiming_ is now known as Qiming | 02:16 | |
lixinhui_ | Qiming_, the patch works | 02:22 |
Qiming | \o/ | 02:24 |
lixinhui_ | _get_create_body this method can be called when server new? | 02:34 |
Qiming | it will be called automatically | 02:37 |
lixinhui_ | okay, thanks, Qiming | 02:38 |
Qiming | the weird thing is: scheduler_hints is only meant to be an out-of-band body entry for server creation | 02:39 |
Qiming | you won't be able to get it back when retrieving server details | 02:40 |
Qiming | so ... I didn't add that into the Server class definition as a property | 02:40 |
lixinhui_ | en, I saw your revision to resource body when create. maybe because hints is designed for use of nova scheduler, but server-group, as a result of allocation/scheduling, will be part of server deails | 02:44 |
lixinhui_ | by the way, when openstacksdk to finish their release, the extension to server-group is also very useful to commit in | 02:45 |
Qiming | yes, but we have to respect their decision to do a code freeze before release | 02:46 |
xuhaiwei | gate seems to be recovered | 02:47 |
*** Zhenqi has joined #senlin | 02:48 | |
Qiming | it was recovering late last night BJ time | 02:48 |
*** hightall has quit IRC | 03:19 | |
Yanyanhu | hi, Qiming, the cred attr in Credential DB model is a list of trusts? | 03:22 |
Yanyanhu | or a single trust | 03:22 |
Qiming | a single | 03:23 |
* Qiming is heading for lunch | 03:26 | |
Yanyanhu | ok | 03:26 |
*** Qiming is now known as Qiming_afk | 03:26 | |
*** mathspanda has quit IRC | 03:38 | |
openstackgerrit | Qiming Teng proposed stackforge/senlin: Remove some unused dependencies https://review.openstack.org/204441 | 04:07 |
*** Zhenqi has quit IRC | 04:09 | |
*** Zhenqi has joined #senlin | 04:10 | |
*** mathspanda has joined #senlin | 04:43 | |
openstackgerrit | OpenStack Proposal Bot proposed stackforge/senlin: Updated from global requirements https://review.openstack.org/204853 | 04:43 |
*** Qiming_afk is now known as Qiming | 05:09 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add test case for LoadBalancer driver(1) https://review.openstack.org/203913 | 05:22 |
openstackgerrit | Merged stackforge/senlin: Updated from global requirements https://review.openstack.org/204853 | 05:28 |
*** Tennyson has joined #senlin | 05:30 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add test case for LoadBalancer driver(1) https://review.openstack.org/203913 | 05:40 |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Add test case for LoadBalancer driver(2) https://review.openstack.org/203956 | 05:49 |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Move webhook test case to tests/engine directory https://review.openstack.org/204890 | 05:53 |
*** jruano has quit IRC | 06:06 | |
*** Tennyson has quit IRC | 06:11 | |
*** Tennyson has joined #senlin | 06:11 | |
*** Zhenqi has quit IRC | 06:31 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Complete test case of policy base module https://review.openstack.org/204933 | 07:22 |
Qiming | successfully get myself trapped, sigh | 07:51 |
Yanyanhu | ... | 07:51 |
Yanyanhu | what happened | 07:51 |
Qiming | a policy has 'signa' and 'handlers' section | 07:52 |
Qiming | s/signa/signal | 07:52 |
Qiming | the handler could be a webhook generated out of a CLUSTER_RESIZE action, and the signal can be a ceilometer alarm | 07:53 |
Qiming | so far so good | 07:53 |
Qiming | then this policy cannot be used | 07:54 |
Yanyanhu | why | 07:55 |
Qiming | don't now how to specify its TARGET property | 07:55 |
Yanyanhu | oh | 07:56 |
Qiming | it doesn't need one, because the triggering signal comes from external systems | 07:56 |
Qiming | so don't need a ('BEFORE', consts.CLUSTE_RESIZE) for example | 07:56 |
Yanyanhu | yes | 07:56 |
Qiming | it won't be checked before or after any action execution | 07:57 |
Qiming | this is fine too | 07:57 |
Yanyanhu | em | 07:57 |
Qiming | although we previously defined policy to be something more restrictive, we can revise that definition | 07:57 |
Qiming | the real problem is that the handlers may trigger other policies | 07:58 |
Qiming | it could be a ScalingInPolicy for example | 07:58 |
Yanyanhu | so the ComplicatedScalingPolicy is not just a 'policy' we imagined before | 07:59 |
Qiming | no, it is not | 07:59 |
Yanyanhu | hmm | 07:59 |
Qiming | it is gonna break our existing definition | 07:59 |
Yanyanhu | it's a workflow spec | 08:00 |
Qiming | It doesn't qualify to be a workflow by itself | 08:01 |
Qiming | it is just a hook, hooking external signals to senlin actions, via the webhook interface, in a secured way | 08:01 |
Yanyanhu | yes | 08:01 |
Qiming | then what are we gonna do with the other policies | 08:02 |
Yanyanhu | but really not a policy which used to regulated the behavior of cluster action | 08:02 |
Yanyanhu | Qiming, not sure whether we should put them together | 08:04 |
Yanyanhu | treat them as the same thing | 08:05 |
Qiming | I'm still trying to shepherd all things into the same pen | 08:07 |
Yanyanhu | yes, that's our goal I think, just feel may be there should be some detail classification inside the big pen | 08:09 |
*** lixinhui_ has quit IRC | 08:42 | |
*** mathspanda has quit IRC | 09:24 | |
*** Tennyson has quit IRC | 09:32 | |
*** Yanyanhu has quit IRC | 10:08 | |
*** Qiming has quit IRC | 10:55 | |
*** Qiming has joined #senlin | 11:44 | |
*** Qiming has quit IRC | 12:16 | |
*** Qiming has joined #senlin | 12:20 | |
*** lkarm has joined #senlin | 13:09 | |
*** jruano has joined #senlin | 13:11 | |
openstackgerrit | Qiming Teng proposed stackforge/senlin: Fix environment test case error https://review.openstack.org/205051 | 13:12 |
jruano | qiming, i spent some time yesterday tracing through the magnum code for bay create | 13:14 |
Qiming | hi | 13:14 |
Qiming | any findings? | 13:15 |
jruano | they use heat to orchestrate their container deployment, and put together a pretty complex template | 13:15 |
jruano | i need a little more time today to dig into this a little more. will let you know what i find out | 13:15 |
Qiming | great, thanks | 13:16 |
jruano | there is a lot in that template, so im trying to distill some necessary elements to satisfy a poc | 13:16 |
Qiming | okay, that sounds practical | 13:17 |
jruano | the other thing is that there is a process to get kube on coreos | 13:17 |
jruano | docker comes built in | 13:18 |
jruano | will you require kube for now, or will docker suffice | 13:18 |
jruano | ? | 13:18 |
Qiming | maybe just docker | 13:18 |
jruano | cool, for now that is the fastest way to a supported runtime | 13:19 |
Qiming | it seems magnum is using a lot of softwareconfig to deploy kube | 13:20 |
jruano | yes | 13:23 |
jruano | so a lot of elements in the template to coordinate this | 13:24 |
jruano | im hoping the docker template is a little simpler | 13:24 |
Qiming | right | 13:24 |
jruano | i can send you a very raw dict output of the template if you want. maybe i can try to pretty it up a little | 13:24 |
Qiming | that sounds good, don't waste time pretty it up though, :) | 13:25 |
*** jdandrea has joined #senlin | 13:34 | |
openstackgerrit | Merged stackforge/senlin: Fix environment test case error https://review.openstack.org/205051 | 13:41 |
*** lixinhui_ has joined #senlin | 15:39 | |
*** Qiming has quit IRC | 15:42 | |
*** lixinhui_ has quit IRC | 15:59 | |
*** lkarm has left #senlin | 19:35 | |
*** Qiming has joined #senlin | 23:32 | |
*** jruano has quit IRC | 23:34 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!