| *** zhurong has quit IRC | 00:00 | |
| Qiming | etherpad for ocata sessions: https://etherpad.openstack.org/p/ocata-senlin-sessions | 00:43 |
|---|---|---|
| *** zhurong has joined #senlin | 01:10 | |
| *** yanyanhu has joined #senlin | 01:18 | |
| openstackgerrit | Merged openstack/senlin: Add API tests for profile validation https://review.openstack.org/361739 | 01:22 |
| openstackgerrit | Merged openstack/senlin: Add negative API tests for profile validation https://review.openstack.org/361740 | 01:22 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Minor tweak webhook receiver https://review.openstack.org/359636 | 01:25 |
| openstackgerrit | Merged openstack/senlin: Update api-ref configuration https://review.openstack.org/361160 | 01:29 |
| openstackgerrit | Merged openstack/senlin: Remove 'description' from Action class https://review.openstack.org/359055 | 01:32 |
| *** elynn has joined #senlin | 01:36 | |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add API tests for profile validation https://review.openstack.org/361790 | 01:52 |
| elynn | Sorry, I forgot to include the real profile validation tests in last patch ^^ | 01:52 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: api-ref doc for policy validation https://review.openstack.org/361791 | 01:56 |
| openstackgerrit | Merged openstack/senlin: Fix reraise issue https://review.openstack.org/361048 | 01:58 |
| *** zhurong_ has joined #senlin | 02:02 | |
| *** xuhaiwei has joined #senlin | 02:05 | |
| *** zhurong has quit IRC | 02:06 | |
| openstackgerrit | Merged openstack/senlin: Add version support to schema and spec https://review.openstack.org/348709 | 02:12 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Minor tweak webhook receiver https://review.openstack.org/359636 | 02:19 |
| yanyanhu | hi, Qiming, around? | 02:25 |
| yanyanhu | I'm trying to add versioning support to lb policy since the schema version support has been merged | 02:25 |
| Qiming | ok | 02:26 |
| yanyanhu | but one question is should we allow user to define a lb policy spec with version=1.0 after we bump the lb policy version to 1.1? | 02:26 |
| Qiming | definitely | 02:26 |
| yanyanhu | I see | 02:26 |
| yanyanhu | so the lb version=1.1 implies the spec version <= 1.1 will be supported | 02:27 |
| openstackgerrit | Merged openstack/senlin: Add eval_status operation to cluster https://review.openstack.org/359177 | 02:27 |
| yanyanhu | if so, should we keep different version of lb policy plugin in senlin's repo? | 02:27 |
| Qiming | ideally, each profile/policy should have a dict property clarifying the mappings from versions to support status | 02:28 |
| yanyanhu | yes | 02:28 |
| Qiming | I'm not envisioning a drastical change to any policy | 02:28 |
| Qiming | if there have to be such changes, we can call them some different policy/profile type already | 02:29 |
| yanyanhu | yes. But there is still a problem that in case of properties are deprecated in new version of policy, if we don't keep the old version of policy plugin, some properties defined with old spec version actually doesn't work as expected | 02:30 |
| Qiming | they should fail validation | 02:32 |
| yanyanhu | yes | 02:32 |
| Qiming | if properties have been removed | 02:32 |
| yanyanhu | but that failure means old version spec is not supported yet | 02:32 |
| yanyanhu | that why I asked whether we should keep backward compatible since if so, we may need to keep old version plugin rather than bump the version directly | 02:33 |
| yanyanhu | s/that/that's | 02:33 |
| Qiming | yes, we are not supposed to carry on the support of all properties or spec versions for ever | 02:33 |
| yanyanhu | yes | 02:34 |
| Qiming | we will deprecate things | 02:34 |
| Qiming | keep them there for 2 cycles | 02:34 |
| Qiming | then remove them | 02:34 |
| Qiming | when things are removed, any spec that are still using those properties will become invalid | 02:34 |
| yanyanhu | Qiming, I see | 02:35 |
| Qiming | if you want to use the old specs, use the old version of the service | 02:35 |
| yanyanhu | so no strict backward compatible there | 02:35 |
| Qiming | that could be a rule | 02:35 |
| yanyanhu | Qiming, ok, makes sense to me | 02:35 |
| yanyanhu | since the policy framework could change as well over the time | 02:35 |
| Qiming | on the other hand, we should be very cautious when making such a decision | 02:35 |
| yanyanhu | in that case, we can't gaurantee the old version policy work correctly | 02:35 |
| yanyanhu | ok | 02:36 |
| Qiming | for example, we remove bdm v1 from nova server only when there is a clear indication from the community that BDM v1 is a history no one wants to support any more | 02:36 |
| Qiming | we actually cannot | 02:36 |
| Qiming | we cannot guarantee a 100% backward compatibility | 02:37 |
| yanyanhu | I see | 02:37 |
| yanyanhu | so using lb policy as an example, now the plugin name will be "senlin.policy.loadbalance-1.5". Although user is allowed to define a lb policy spec with version 1.0. But the spec validation could fail for property deprecation. | 02:38 |
| yanyanhu | but the plugin loading will succeed | 02:39 |
| yanyanhu | since it is still "lb policy" | 02:39 |
| yanyanhu | we just don't enforce user to define policy spec with latest version number | 02:40 |
| openstackgerrit | XueFeng Liu proposed openstack/python-senlinclient: Fix "openstack cluster policy binding list <cluster>" https://review.openstack.org/361800 | 02:40 |
| Qiming | that logic can be improved | 02:44 |
| Qiming | we can leave this to a summit discussion | 02:45 |
| *** yuanying has quit IRC | 02:51 | |
| *** yuanying has joined #senlin | 02:52 | |
| yanyanhu | Qiming, agee, this is an import topic need more discussion | 02:53 |
| yanyanhu | so I will stop making more changes here before we have a conclusion about it | 02:54 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add API tests for policy validation https://review.openstack.org/361808 | 03:02 |
| openstackgerrit | Ethan Lynn proposed openstack/senlin: Add negative API tests for policy validation https://review.openstack.org/361809 | 03:02 |
| *** yuanying has quit IRC | 03:44 | |
| openstackgerrit | Merged openstack/python-senlinclient: Fix cluster-policy-list https://review.openstack.org/361601 | 04:02 |
| openstackgerrit | Merged openstack/senlin: api-ref doc for policy validation https://review.openstack.org/361791 | 04:33 |
| *** yuanying has joined #senlin | 05:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/361874 | 06:17 |
| *** yuanying has quit IRC | 06:29 | |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/361874 | 06:47 |
| openstackgerrit | Merged openstack/senlin: Minor tweak webhook receiver https://review.openstack.org/359636 | 07:12 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: [WIP]Initial work for message type of receiver https://review.openstack.org/359042 | 07:16 |
| openstackgerrit | Merged openstack/senlin: Add API tests for policy validation https://review.openstack.org/361808 | 07:21 |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/senlin: Fix formatting strings when using multiple variables https://review.openstack.org/361907 | 07:24 |
| openstackgerrit | Merged openstack/senlin: Add negative API tests for policy validation https://review.openstack.org/361809 | 07:24 |
| openstackgerrit | Merged openstack/senlin: Add API tests for profile validation https://review.openstack.org/361790 | 07:25 |
| *** yuanying has joined #senlin | 07:51 | |
| *** elynn_ has joined #senlin | 08:27 | |
| *** elynn has quit IRC | 08:30 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 08:32 |
| openstackgerrit | Qiming Teng proposed openstack/python-senlinclient: OSC plugin command for cluster-run https://review.openstack.org/361934 | 08:34 |
| *** elynn__ has joined #senlin | 08:37 | |
| *** elynn_ has quit IRC | 08:40 | |
| openstackgerrit | Lu lei proposed openstack/senlin: Fix using filter() to meet python2,3 https://review.openstack.org/361939 | 08:42 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: [WIP]Initial work for message type of receiver https://review.openstack.org/359042 | 08:44 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to skip more actions https://review.openstack.org/361944 | 08:44 |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 08:45 |
| openstackgerrit | Merged openstack/senlin: Tweak cluster recover workflow https://review.openstack.org/361549 | 09:15 |
| openstackgerrit | Merged openstack/senlin: Always change desired_capacity for node_delete https://review.openstack.org/361598 | 09:15 |
| *** guoshan has joined #senlin | 09:30 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Enable health policy to skip more actions https://review.openstack.org/361944 | 09:36 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix exception handling in node_recover https://review.openstack.org/361967 | 09:36 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: [WIP]Initial work for message type of receiver https://review.openstack.org/359042 | 09:39 |
| *** yanyanhu has quit IRC | 09:52 | |
| *** zigo_ is now known as zigo | 09:56 | |
| *** zhurong_ has quit IRC | 10:04 | |
| *** guoshan has quit IRC | 10:26 | |
| *** guoshan has joined #senlin | 10:26 | |
| *** shu-mutou has quit IRC | 10:28 | |
| openstackgerrit | xu-haiwei proposed openstack/senlin: Fix some bugs for container creation https://review.openstack.org/349906 | 10:35 |
| *** guoshan has left #senlin | 10:43 | |
| openstackgerrit | Merged openstack/senlin: Enable health policy to skip more actions https://review.openstack.org/361944 | 10:46 |
| *** elynn__ has quit IRC | 11:02 | |
| openstackgerrit | Merged openstack/senlin: Fix exception handling in node_recover https://review.openstack.org/361967 | 11:25 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix exception handling in nova server recover https://review.openstack.org/362037 | 11:38 |
| *** elynn__ has joined #senlin | 12:14 | |
| *** zhurong has joined #senlin | 12:18 | |
| openstackgerrit | Lu lei proposed openstack/senlin: Fix oslo.i18n in senlin project https://review.openstack.org/362064 | 12:18 |
| openstackgerrit | Lu lei proposed openstack/python-senlinclient: Fix oslo.i18n problems in senlinclient Help msg in senlinclient project should support oslo.i18n. https://review.openstack.org/362072 | 12:28 |
| *** zzxwill has joined #senlin | 12:40 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix exception handling in server check https://review.openstack.org/362106 | 13:13 |
| *** elynn__ has quit IRC | 13:18 | |
| openstackgerrit | Merged openstack/senlin: Fix oslo.i18n in senlin project https://review.openstack.org/362064 | 13:51 |
| *** zzxwill has quit IRC | 14:19 | |
| *** zzxwill has joined #senlin | 14:20 | |
| *** Drago has joined #senlin | 14:31 | |
| *** Drago has quit IRC | 14:32 | |
| *** Drago has joined #senlin | 14:32 | |
| *** zzxwill has quit IRC | 14:38 | |
| *** zhurong has quit IRC | 15:55 | |
| *** Drago has quit IRC | 16:24 | |
| *** Drago has joined #senlin | 19:11 | |
| *** zzxwill has joined #senlin | 23:02 | |
| *** zzxwill has quit IRC | 23:03 | |
| *** zzxwill has joined #senlin | 23:10 | |
| *** zzxwill has quit IRC | 23:16 | |
| *** Drago has quit IRC | 23:30 | |
| *** zhurong has joined #senlin | 23:44 | |
| openstackgerrit | Merged openstack/senlin: Fix exception handling in nova server recover https://review.openstack.org/362037 | 23:45 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!