| openstackgerrit | Merged openstack/senlin master: Updated from global requirements https://review.openstack.org/466956 | 00:57 |
|---|---|---|
| *** xinhuili has joined #senlin | 01:17 | |
| openstackgerrit | Hongbin Li proposed openstack/python-senlinclient master: remove redundant value when cluster-show https://review.openstack.org/468592 | 01:35 |
| *** XueFeng has joined #senlin | 01:36 | |
| openstackgerrit | lu.li proposed openstack/senlin master: update test.attr() to decorators.attr() https://review.openstack.org/468596 | 01:46 |
| *** XueFeng has quit IRC | 01:51 | |
| *** XueFeng has joined #senlin | 01:52 | |
| XueFeng | QiMing, I want to integrate senlin to openstack K version productio environment. which version should I use? | 01:55 |
| XueFeng | production | 01:55 |
| Qiming_ | ... latest version I think | 01:58 |
| Qiming_ | K is too old I'm afraid | 01:58 |
| Qiming_ | but anyway | 01:58 |
| Qiming_ | openstacksdk doesn't have a strict version checking or a flexible version negotiation capability in place | 01:59 |
| Qiming_ | so .. when talking to an old version of nova or heat, there might be some subtle bugs | 01:59 |
| XueFeng | OK, I will try latest senlin version in one environment first | 02:37 |
| chenyb4 | Hi, I have a question, when extend receiver_update api , allways appear "<ValidationError: "Additional properties are not allowed (u'receiver' was unexpected)">＂ | 02:54 |
| chenyb4 | PATCH body params "{ | 02:55 |
| chenyb4 | "receiver": { | 02:55 |
| chenyb4 | "name": "cluster_inflate", | 02:55 |
| chenyb4 | "params": { | 02:55 |
| chenyb4 | "count": "1" | 02:55 |
| chenyb4 | } | 02:55 |
| chenyb4 | } | 02:55 |
| chenyb4 | }" | 02:55 |
| openstackgerrit | lu.li proposed openstack/senlin master: update test.attr() to decorators.attr() https://review.openstack.org/468596 | 03:00 |
| Qiming_ | where is your ReceiverUpdateRequest? | 03:03 |
| chenyb4 | Qiming_, senlin/objects/requests/receivers.py class ReceiverUpdateRequest(base.SenlinObject): | 03:05 |
| chenyb4 | fields = { | 03:06 |
| chenyb4 | 'identity': fields.StringField(), | 03:06 |
| chenyb4 | 'name': fields.NameField(nullable=True), | 03:06 |
| chenyb4 | 'params': fields.JsonField(nullable=True, default={}) | 03:06 |
| chenyb4 | } | 03:06 |
| Qiming_ | obviously, your request object is not accepting a 'receiver' property | 03:07 |
| chenyb4 | Qiming_, What should I do ? | 03:09 |
| chenyb4 | Qiming_, I has add update to policy.json | 03:10 |
| Qiming_ | check how cluster update api is supported | 03:10 |
| Qiming_ | this has nothing to do with policy | 03:10 |
| *** xinhuili has quit IRC | 04:53 | |
| *** XueFeng has quit IRC | 05:25 | |
| *** XueFeng has joined #senlin | 05:25 | |
| openstackgerrit | chenyb4 proposed openstack/senlin master: Versioned object for receiver update request https://review.openstack.org/468612 | 05:27 |
| openstackgerrit | lu.li proposed openstack/senlin master: update test.attr() to decorators.attr() https://review.openstack.org/468596 | 05:52 |
| openstackgerrit | Merged openstack/senlin master: delete profile base invalid variable https://review.openstack.org/467895 | 06:10 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr Function 'tempest.test.attr()' has moved to 'tempest.lib.decorators .attr()' in Pike and will be removed in a future version[1]. This patch replaces the 'tempest.test.attr()' with the 'tempest.lib .decorators.attr(). https://review.openstack.org/468596 | 06:25 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:30 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:38 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:44 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:46 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:50 |
| chenyb4 | Hi, Qiming_ I don't find about "Additional properties are not allowed", I only find test about "Additional properties are not allowed" message. | 06:54 |
| chenyb4 | I find I find "API layer support to node adoption" https://review.openstack.org/#/c/462354/ but don't find about it message | 06:54 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:55 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 06:58 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 07:01 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 07:09 |
| Qiming_ | chenyb4, cannot understand what you are saying | 07:31 |
| chenyb4 | Qiming_, I add new receiver update api interface ,configure policy.json, add router map, but i patch params after appear "<ValidationError: "Additional properties are not allowed (u'receiver' was unexpected)">＂ | 07:33 |
| Qiming_ | what kind of a JSON are you passing to the API? | 07:34 |
| Qiming_ | how are you modeling the ReceiverUpdateRequest object? | 07:34 |
| chenyb4 | {"receiver":{"name": "test"}} | 07:34 |
| Qiming_ | is your ReceiverUpdateRequest prepared to receive something like {'receiver': {...}} ? | 07:35 |
| Qiming_ | then the object must contain a 'receiver' field at the top level | 07:35 |
| chenyb4 | https://review.openstack.org/#/c/468612/ this is request about receiver | 07:35 |
| chenyb4 | Qiming_, http://paste.ubuntu.com/24676140/ api code | 07:38 |
| Qiming_ | so ... you still don't understand what I am saying | 07:39 |
| Qiming_ | and what I was suggesting | 07:39 |
| Qiming_ | In your ReceiverUpdateRequest class, there is no such a field named 'receiver', right? | 07:39 |
| Qiming_ | how could API layer accept such a JSON, {'receiver': {...}} ? | 07:40 |
| Qiming_ | previously I told you to learn about this from ClusterUpdateRequest, and you didn't do that either | 07:40 |
| Qiming_ | okay, my fault | 07:42 |
| Qiming_ | you have been faithfully following the cluster update example | 07:42 |
| Qiming_ | let me check where the problem is | 07:42 |
| Qiming_ | how are you invoking the API now? | 07:43 |
| Qiming_ | via SDK? | 07:43 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 07:45 |
| chenyb4 | Qiming_, use Postman tool PATCH body | 07:46 |
| Qiming_ | what is the body look like? | 08:00 |
| chenyb4 | Hi, Qiming_ I see the data error. params = body i must change params = receiver_data. I'm sorry | 08:01 |
| chenyb4 | So, appear jsonschema format error | 08:02 |
| chenyb4 | receiver_data = body.get('receiver', None) | 08:02 |
| chenyb4 | params = body | 08:02 |
| chenyb4 | change params = receiver_data right | 08:03 |
| Qiming_ | ... | 08:12 |
| openstackgerrit | chenyb4 proposed openstack/senlin master: Add receiver new api interface https://review.openstack.org/468627 | 08:29 |
| openstackgerrit | lu.li proposed openstack/senlin master: Replace test.attr with decorators.attr https://review.openstack.org/468596 | 08:31 |
| openstackgerrit | chenyb4 proposed openstack/senlin master: Add receiver new api interface https://review.openstack.org/468627 | 08:37 |
| *** XueFeng has quit IRC | 09:56 | |
| openstackgerrit | chenyb4 proposed openstack/senlin master: Add receiver new api interface https://review.openstack.org/468627 | 10:12 |
| *** openstackgerrit has quit IRC | 12:02 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting to clear some hung API connections and should return to service momentarily. | 12:05 | |
| *** Rita has joined #senlin | 12:23 | |
| *** Rita is now known as chenyb4_ | 12:24 | |
| *** openstackgerrit has joined #senlin | 14:57 | |
| *** ChanServ sets mode: +v openstackgerrit | 14:57 | |
| openstackgerrit | chenyb4 proposed openstack/senlin master: Add receiver update interface engine service support https://review.openstack.org/468612 | 14:57 |
| *** chenyb4_ has quit IRC | 14:59 | |
| *** chenyb4_ has joined #senlin | 15:07 | |
| *** chenyb4_ has quit IRC | 15:07 | |
| *** chenyb4_ has joined #senlin | 15:07 | |
| *** chenyb4_ has quit IRC | 16:02 | |
| *** chenyb4_ has joined #senlin | 16:03 | |
| *** chenyb4_ has quit IRC | 16:07 | |
| *** openstackgerrit has quit IRC | 16:47 | |
| *** chenyb4_ has joined #senlin | 23:43 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!