*** bauzas has quit IRC | 01:03 | |
*** bauzas has joined #openstack-climate | 01:12 | |
*** bauzas has quit IRC | 01:27 | |
*** chandan_kumar has joined #openstack-climate | 05:07 | |
*** saju_m has joined #openstack-climate | 06:50 | |
*** bauzas has joined #openstack-climate | 08:32 | |
openstackgerrit | A change was merged to stackforge/climate: Fix issue with devstack Climate config file https://review.openstack.org/76263 | 08:40 |
---|---|---|
*** SergeyLukjanov has quit IRC | 10:04 | |
*** SergeyLukjanov has joined #openstack-climate | 10:04 | |
openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Remove unecessary sys.path add from docs/config.py https://review.openstack.org/76478 | 10:11 |
openstackgerrit | Nikolay Starodubtsev proposed a change to stackforge/climate: Remove unecessary sys.path add from docs/config.py https://review.openstack.org/76478 | 10:53 |
openstackgerrit | Dina Belova proposed a change to stackforge/climate: Fix REST API docs https://review.openstack.org/76493 | 11:12 |
openstackgerrit | Dina Belova proposed a change to stackforge/climate: Fix REST API docs https://review.openstack.org/76493 | 11:56 |
*** casanch1 has joined #openstack-climate | 12:49 | |
bauzas | DinaBelova_: you there ? | 12:52 |
bauzas | I would like to discuss about API v2 and extra capabilities for Host | 12:53 |
openstackgerrit | Sylvain Bauza proposed a change to stackforge/climate: Add possibility to have separate config files for API and Manager https://review.openstack.org/76525 | 12:54 |
bauzas | DinaBelova_: as I discussed with Doug H., I have something to share with you | 12:54 |
DinaBelova_ | bauzas, ok, I'm here | 12:59 |
DinaBelova_ | I was looking into your conversation | 12:59 |
bauzas | DinaBelova_: I'll be quick | 12:59 |
DinaBelova_ | but it seems i have no last result | 13:00 |
bauzas | DinaBelova_: WSME imposes static API | 13:00 |
DinaBelova_ | yes, I got it | 13:00 |
bauzas | DinaBelova_: so, there are 2 possibilities for extra caps | 13:00 |
bauzas | DinaBelova_: 1/ we keep compatibility with client and V1, so we need to pass out the extra capas as keys within the body | 13:01 |
DinaBelova_ | 2/ - extras json? | 13:02 |
bauzas | DinaBelova_: on that case, I'll do some processing within the post() method by looking directly at the pecan.request.body and get the capas | 13:02 |
bauzas | DinaBelova_: pros: we keep compat, cons : we do some hack stuff within the controller | 13:02 |
DinaBelova_ | and the second variant? | 13:03 |
bauzas | 2/ I change the body specs by providing a fixed key (say it 'extra_caps') and any extra capas should be serialized to a JSON string as value for this key | 13:03 |
bauzas | pros: that allow to keep input validation 100% by WSME, cons: we change API specs | 13:04 |
bauzas | (and thus, need to patch the client) | 13:04 |
DinaBelova_ | ok | 13:04 |
bauzas | what is your opinion on it before doing the pathc ? | 13:04 |
DinaBelova_ | really it's difficult | 13:05 |
bauzas | well, both are easy to implement, that's just matter of deciding which is the best option :) | 13:05 |
DinaBelova_ | I love 2nd idea - but with having v2 client modules | 13:05 |
DinaBelova_ | so we'll have v1 and v2 | 13:05 |
DinaBelova_ | and that's normal v1!=v2 - as usual | 13:06 |
bauzas | indeed | 13:06 |
bauzas | but I'm a bit worried about incubation request | 13:06 |
bauzas | if we say such a change | 13:06 |
bauzas | oh, yet another unexpected meeting... | 13:06 |
DinaBelova_ | ok, I just only say | 13:07 |
bauzas | think about it, I'll focus first on patching | 13:07 |
bauzas | rebasing I mean | 13:07 |
DinaBelova_ | that if we'll keep v1 untouched | 13:07 |
DinaBelova_ | incubation request is ok | 13:07 |
bauzas | ok | 13:07 |
DinaBelova_ | that's why I think you may implement different v2 - but as v2, no influence on v1 | 13:07 |
*** cmart__ has joined #openstack-climate | 13:07 | |
DinaBelova_ | in this case it'll be ok | 13:07 |
bauzas | ok, taking option #2 then | 13:08 |
bauzas | I'll update how we show capas as well | 13:08 |
bauzas | thanks | 13:08 |
*** cmart__ has quit IRC | 13:10 | |
DinaBelova_ | ok, thank you :) | 13:10 |
*** cmart__ has joined #openstack-climate | 13:10 | |
cmart__ | hello | 13:10 |
DinaBelova_ | o/ | 13:10 |
*** bauzas has quit IRC | 13:12 | |
*** bauzas has joined #openstack-climate | 14:02 | |
*** cmart__ has quit IRC | 14:03 | |
openstackgerrit | Dina Belova proposed a change to stackforge/climate: Add development guides to docs https://review.openstack.org/76154 | 14:55 |
openstackgerrit | Dina Belova proposed a change to stackforge/climate: Start moving Climate docs from wiki to Sphinx docs https://review.openstack.org/76152 | 14:55 |
openstackgerrit | Dina Belova proposed a change to stackforge/climate: Add installation guides to docs https://review.openstack.org/76153 | 14:55 |
*** saju_m has quit IRC | 15:02 | |
*** bauzas has quit IRC | 15:11 | |
openstackgerrit | Cristian A Sanchez proposed a change to stackforge/climate: Sends notifications at lease events https://review.openstack.org/75189 | 15:32 |
*** chandan_kumar has quit IRC | 15:52 | |
*** chandan_kumar has joined #openstack-climate | 15:56 | |
*** bauzas has joined #openstack-climate | 16:28 | |
*** saju_m has joined #openstack-climate | 17:06 | |
*** casanch1_ has joined #openstack-climate | 17:12 | |
*** casanch1 has quit IRC | 17:12 | |
*** casanch1_ is now known as casanch1 | 17:13 | |
openstackgerrit | Pablo Fernando Cargnelutti proposed a change to stackforge/climate: Remove uuidutils imports in oslo modules https://review.openstack.org/76600 | 18:00 |
*** saju_m has quit IRC | 18:35 | |
*** chandan_kumar has quit IRC | 18:46 | |
openstackgerrit | Cristian A Sanchez proposed a change to stackforge/climate: Update openstack.common with latest oslo-incubator https://review.openstack.org/73363 | 18:50 |
*** casanch1_ has joined #openstack-climate | 19:42 | |
*** casanch1 has quit IRC | 19:47 | |
*** bauzas has quit IRC | 19:47 | |
*** casanch1_ is now known as casanch1 | 19:47 | |
*** casanch1_ has joined #openstack-climate | 20:12 | |
*** bauzas has joined #openstack-climate | 20:15 | |
*** casanch1 has quit IRC | 20:18 | |
*** casanch1_ is now known as casanch1 | 20:18 | |
openstackgerrit | Pablo Fernando Cargnelutti proposed a change to stackforge/climate: Remove uuidutils imports in oslo modules https://review.openstack.org/76600 | 20:20 |
*** casanch1 has quit IRC | 20:21 | |
*** casanch1 has joined #openstack-climate | 20:21 | |
*** casanch1 has quit IRC | 21:10 | |
*** jd__` has joined #openstack-climate | 21:17 | |
*** openstackgerrit has quit IRC | 21:20 | |
*** jd__ has quit IRC | 21:20 | |
*** jd__` is now known as jd__ | 21:20 | |
*** openstackgerrit has joined #openstack-climate | 21:21 | |
*** casanch1 has joined #openstack-climate | 23:26 | |
openstackgerrit | Jenkins proposed a change to stackforge/python-climateclient: Updated from global requirements https://review.openstack.org/76706 | 23:36 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!