| *** yanyanhu has joined #senlin | 00:42 | |
| *** ruijie_ has joined #senlin | 01:15 | |
| *** hanwei has joined #senlin | 01:26 | |
| openstackgerrit | miaohb proposed openstack/senlin: Return support status in profile type show https://review.openstack.org/418678 | 01:51 |
|---|---|---|
| *** elynn has joined #senlin | 01:58 | |
| elynn | Morning | 01:59 |
| ruijie_ | Morning | 02:12 |
| yanyanhu | morning :) | 02:32 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for events https://review.openstack.org/423892 | 03:18 |
| elynn | Hi Qiming | 03:35 |
| Qiming | yes? | 03:35 |
| elynn | Is there a way to expose node_ips in heat cluster resource? | 03:35 |
| elynn | I can not find a easy to support cluster-collect in heat. | 03:36 |
| Qiming | you will need to do a cluster-collect api call | 03:36 |
| elynn | easy way | 03:36 |
| Qiming | you can expose an attribute | 03:36 |
| elynn | Since the in heat, _resolve_attribute only accept a name | 03:37 |
| Qiming | and when implement that attribute, do a cluster-collect call | 03:37 |
| Qiming | in this case, you are exposing an attribute named 'node_ips' for example | 03:37 |
| elynn | I can add a node_ips attribute, but that seems too specific. | 03:37 |
| Qiming | right, that is where I dislike heat | 03:38 |
| elynn | okay, maybe that is the only way I can do. | 03:39 |
| elynn | What about the format of node_ips | 03:39 |
| elynn | Since one node maybe have more than one IP | 03:39 |
| elynn | like this ? [[ip1, ip2, floating_ip], [ip1, ip2]] | 03:40 |
| Qiming | oh no | 03:40 |
| Qiming | you don't have such a use case do you? | 03:40 |
| elynn | lixinhui are drafting the proposal, and in her solution, they use heat to create senlin vdu cluster | 03:41 |
| Qiming | I'm looking at software deployment group resource | 03:41 |
| elynn | But some cluster required other cluster's IP | 03:41 |
| elynn | For her case, one cluster have only one node for now. | 03:41 |
| Qiming | get attribute can have a path parameter | 03:42 |
| elynn | yes, so the template would looks like get_attr: [cluster1, node_ips, 0, 0] to get a port ip | 03:43 |
| Qiming | the second argument can be more general | 03:44 |
| elynn | I'm afraid not.. | 03:44 |
| Qiming | then the third one can be a json path | 03:44 |
| elynn | I go through the codes this morning, and the third parameter couldn't pass to res._resolve_attribute | 03:45 |
| Qiming | get_attr: [cluster1, collect, addresses.private[0].addr] | 03:46 |
| elynn | What heat do is res._resolve_attribute(node_ips) and then get item 0, then get item 0 | 03:47 |
| Qiming | if we have such a general solution, why cannot we override both get_attribute and _resolve_attribute? | 03:47 |
| elynn | Qiming, that's the ideal way. | 03:47 |
| elynn | hmm... override get_attribute | 03:47 |
| Qiming | when override get_attribute, we decide how to deal with 'collect', and 'addresses.private[0].addr | 03:47 |
| elynn | Maybe that can do. | 03:47 |
| elynn | Never thought of that! | 03:49 |
| elynn | Thanks | 03:49 |
| Qiming | okay, then try if it works ... | 03:49 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for node 1 https://review.openstack.org/423913 | 06:03 |
| yanyanhu | hi, Qiming, around? | 06:43 |
| yanyanhu | will sdk has 0.9.13 release by the end of this week? | 06:44 |
| yanyanhu | since we need to release senlinclient before this friday | 06:44 |
| yanyanhu | if it doesn't, I guess we may need to revert some changes in senlinclient | 06:44 |
| Qiming | not sure | 06:48 |
| Qiming | need to check with briancurtain | 06:48 |
| Qiming | and also a lot of other patches have to be reverted I think | 06:48 |
| Qiming | because the only version that known working for all is 0.9.10 | 06:49 |
| yanyanhu | yes, a lot of them | 06:49 |
| yanyanhu | so hope there will be 0.9.13 sdk release | 06:49 |
| Qiming | look here: https://review.openstack.org/#/c/418663/ | 06:50 |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/423688 | 06:51 |
| openstackgerrit | miaohb proposed openstack/senlin: Return support status in policy type show https://review.openstack.org/418694 | 06:52 |
| yanyanhu | Qiming, sigh, this is really a headache. | 06:53 |
| yanyanhu | I'm afraid they can't finish that patch before this weekend | 06:54 |
| yanyanhu | will sdk 0.9.13 fix that network issue? | 06:54 |
| yanyanhu | if not, I think we should consider to start reverting those changes in senlinclient | 06:54 |
| yanyanhu | or at least provide an old commit id for release | 06:55 |
| yanyanhu | last release(1.1.0) of python-senlinclient was done based on this commit | 06:57 |
| yanyanhu | https://review.openstack.org/#/c/405158/ | 06:57 |
| yanyanhu | at least these three patches need to be reverted | 07:02 |
| yanyanhu | http://paste.openstack.org/show/595997/ | 07:02 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise versioned objects' exception message https://review.openstack.org/423953 | 07:08 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for node 1 https://review.openstack.org/423913 | 07:12 |
| *** hanwei has quit IRC | 07:13 | |
| *** hanwei has joined #senlin | 07:14 | |
| *** ruijie_ is now known as ruijie | 07:15 | |
| openstackgerrit | miaohb proposed openstack/senlin: Fix typo in policy-type-list related code https://review.openstack.org/423968 | 07:46 |
| elynn | Hi Qiming yanyanhu | 07:56 |
| elynn | Dose openstacksdk support cluster_collect? | 07:56 |
| yanyanhu | hi, elynn, I'm not sure, Qiming, do you have any idea? | 07:57 |
| elynn | Saw it. | 08:01 |
| yanyanhu | nice | 08:01 |
| elynn | But found a but in senlinclient | 08:01 |
| elynn | bug | 08:01 |
| yanyanhu | collect_cluster_attrs | 08:01 |
| yanyanhu | this is supported | 08:01 |
| yanyanhu | elynn, you mean? | 08:02 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise versioned objects' exception message https://review.openstack.org/423953 | 08:05 |
| openstackgerrit | Ethan Lynn proposed openstack/python-senlinclient: Fix cluster_collect interface https://review.openstack.org/423977 | 08:05 |
| elynn | yanyanhu, ^^ | 08:05 |
| yanyanhu | thanks :) | 08:06 |
| yanyanhu | hi, elynn, you may need to update the unit test as well | 08:09 |
| yanyanhu | oh | 08:10 |
| yanyanhu | my fault, the unit test is only for osc client | 08:10 |
| openstackgerrit | Merged openstack/senlin: revise tempest api test for cluster 5 https://review.openstack.org/423613 | 08:10 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise versioned objects' exception message https://review.openstack.org/423953 | 08:11 |
| elynn | yanyanhu, Is this possible to release senlinclient with this patch this release | 08:11 |
| yanyanhu | hi, elynn, that depends on which version of sdk release includes cluster collect support | 08:12 |
| yanyanhu | let me check | 08:12 |
| yanyanhu | elynn, no problem I think | 08:13 |
| elynn | 0.9.10 is fine. | 08:13 |
| yanyanhu | this is a feature added Aug last year | 08:13 |
| yanyanhu | yep | 08:13 |
| elynn | But since heat actually calls senlinclient instead of openstacksdk | 08:13 |
| elynn | So I need this fixed in senlinclient | 08:14 |
| yanyanhu | elynn, yes, we can include in into senlinclient's ocata final release | 08:14 |
| yanyanhu | include it | 08:14 |
| elynn | Haven't got time to directly call openstacksdk in heat yet.. | 08:14 |
| yanyanhu | elynn, you can start that work in next cycle :) | 08:15 |
| elynn | I prepare a patch actually, but don't have time to do a fully tests. | 08:16 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for node 2 https://review.openstack.org/423980 | 08:21 |
| openstackgerrit | Merged openstack/senlin: Return support status in profile type show https://review.openstack.org/418678 | 08:46 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for profile/policy types https://review.openstack.org/423991 | 08:48 |
| openstackgerrit | miaohb proposed openstack/senlin: Return support status in policy type show https://review.openstack.org/418694 | 08:52 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise versioned objects' exception message https://review.openstack.org/423953 | 08:56 |
| openstackgerrit | Merged openstack/python-senlinclient: Fix cluster_collect interface https://review.openstack.org/423977 | 09:08 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise versioned objects' exception message https://review.openstack.org/423953 | 09:15 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for profile/policy types https://review.openstack.org/423991 | 09:17 |
| openstackgerrit | RUIJIE YUAN proposed openstack/senlin: revise tempest api test for node 2 https://review.openstack.org/423980 | 09:17 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add create and update actions for policy https://review.openstack.org/421119 | 09:31 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add delete action for cluster https://review.openstack.org/421667 | 09:32 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add update action for cluster https://review.openstack.org/421711 | 09:34 |
| *** yanyanhu has quit IRC | 09:37 | |
| *** ruijie has quit IRC | 09:41 | |
| *** hanwei has quit IRC | 10:21 | |
| openstackgerrit | Kenji Ishii proposed openstack/senlin-dashboard: Implement action updating cluster policies https://review.openstack.org/423209 | 10:21 |
| openstackgerrit | Merged openstack/senlin-dashboard: Add angular policies table https://review.openstack.org/389076 | 10:30 |
| openstackgerrit | Merged openstack/senlin-dashboard: Add details view into Policies panel https://review.openstack.org/421072 | 10:31 |
| openstackgerrit | Merged openstack/senlin-dashboard: Add delete action for policies panel https://review.openstack.org/421078 | 10:31 |
| openstackgerrit | Kenji Ishii proposed openstack/senlin-dashboard: Implement action updating cluster policies https://review.openstack.org/423209 | 10:38 |
| *** hanwei has joined #senlin | 10:52 | |
| *** x00350071_ has joined #senlin | 11:10 | |
| *** edisonxiang has quit IRC | 11:10 | |
| openstackgerrit | Merged openstack/senlin-dashboard: Implement action updating cluster policies https://review.openstack.org/423209 | 11:17 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add create and update actions for policy https://review.openstack.org/421119 | 11:38 |
| *** x00350071_ is now known as edisonxiang | 11:39 | |
| openstackgerrit | XueFeng Liu proposed openstack/senlin: Fix health registry claim bug https://review.openstack.org/424073 | 11:49 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add delete action for cluster https://review.openstack.org/421667 | 12:11 |
| openstackgerrit | Shu Muto proposed openstack/senlin-dashboard: Add update action for cluster https://review.openstack.org/421711 | 12:13 |
| *** shu-mutou is now known as shu-mutou-AWAY | 12:17 | |
| *** catintheroof has joined #senlin | 12:45 | |
| *** catintheroof has quit IRC | 12:50 | |
| *** catintheroof has joined #senlin | 12:58 | |
| *** catinthe_ has joined #senlin | 14:29 | |
| *** catintheroof has quit IRC | 14:32 | |
| *** hanwei has quit IRC | 14:59 | |
| *** Drago1 has joined #senlin | 16:00 | |
| *** Jeffrey4l_ has quit IRC | 16:06 | |
| *** Jeffrey4l_ has joined #senlin | 16:06 | |
| *** Jeffrey4l__ has joined #senlin | 17:57 | |
| *** Jeffrey4l_ has quit IRC | 17:59 | |
| *** catintheroof has joined #senlin | 18:45 | |
| *** catinthe_ has quit IRC | 18:47 | |
| *** elynn has quit IRC | 19:01 | |
| *** elynn has joined #senlin | 19:02 | |
| *** catinthe_ has joined #senlin | 19:03 | |
| *** catintheroof has quit IRC | 19:06 | |
| *** Jeffrey4l_ has joined #senlin | 21:34 | |
| *** Jeffrey4l__ has quit IRC | 21:35 | |
| *** catinthe_ has quit IRC | 23:23 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!